| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: [DNM] Testing https://review.opendev.org/c/openstack/designate/+/817501 | 01:10 |
|---|---|---|
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix dns.query.tcp/udp not always handling ipv6 properly https://review.opendev.org/c/openstack/designate/+/813722 | 01:38 |
| eandersson | https://zuul.opendev.org/t/openstack/build/3198313c3ae049a0a7e8955e7d6c5ecc/log/controller/logs/screen-designate-mdns.txt?severity=4 | 01:42 |
| eandersson | This is an interesting failure | 01:43 |
| eandersson | I wonder if I caused it, but basically xfr cannot handle a hostname | 01:43 |
| eandersson | > 'host': 'ns1.devstack.org.' | 01:43 |
| eandersson | Not sure how this has ever worked, or if I changed something to cause it to use the hostname instead of an ip... but I cannot see that | 01:44 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: [DNM] Testing https://review.opendev.org/c/openstack/designate/+/817501 | 02:06 |
| johnsom | I think that was a dnspython change, but I thought we fixed those tests | 04:01 |
| eandersson | With the above code change it fails gracefully | 05:05 |
| eandersson | > socket.gaierror: [Errno -2] Name or service not known | 05:05 |
| eandersson | btw that call is coming from a test using a SECONARY | 05:05 |
| johnsom | Oh, maybe I got this confused with another issue. We knew secondary zones were broken. There is a launchpad for it. | 05:23 |
| eandersson | btw is this the bad call? 00:05:53.599681 | 05:45 |
| eandersson | https://zuul.opendev.org/t/openstack/build/bf74d54bf03e47a693fe6231345e64d7/log/controller/logs/screen-designate-api.txt | 05:45 |
| eandersson | johnsom was able to reproduce the 5XX locally! | 06:35 |
| eandersson | Lets see if I can get it a second time lol | 06:35 |
| eandersson | nvm it was a 503, not a 502 :'( | 06:52 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: [DNM] Testing https://review.opendev.org/c/openstack/designate/+/817501 | 07:19 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix dns.query.tcp/udp not always handling ipv6 properly https://review.opendev.org/c/openstack/designate/+/813722 | 07:20 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Fix dns.query.tcp/udp not always handling ipv6 properly https://review.opendev.org/c/openstack/designate/+/813722 | 07:37 |
| frickler | all the negative API tests make it essentially impossibly to locate real issues in testing logs. makes me wonder whether we would only want to run them in a dedicated job | 09:10 |
| frickler | eandersson: the failure with ns1.devstack.org used to be a socket error, that changed with dnspython2, 813722 would be the patch that should fix it | 09:18 |
| frickler | actually I wonder whether the negative tests might actually be be trigger for our issues. seems uwsgi always closes the socket on such errors. if another request is happening at the same time, it might get the 502 | 09:24 |
| frickler | https://paste.opendev.org/show/810925/ would look to be like a 30ms window which could trigger it. happening around 400 times during our tempest run | 09:24 |
| kevko | Hi \o/ | 13:28 |
| kevko | please, should be this backpored to previous stable branches ? https://review.opendev.org/c/openstack/designate/+/814290 ? Thank you | 13:29 |
| kevko | * shouldn't be | 13:29 |
| *** gmann_ is now known as gmann | 13:45 | |
| *** johnsom_ is now known as johnsom | 13:46 | |
| frickler | kevko: certainly for xena, not sure for older branches. you can propose the backport yourself if you want (cherry-pick in the UI, that's assuming it doesn't need editing) | 13:55 |
| opendevreview | Michal Arbet proposed openstack/designate stable/victoria: Fix race condition in the sink when deleting records https://review.opendev.org/c/openstack/designate/+/817540 | 14:26 |
| opendevreview | Michal Arbet proposed openstack/designate stable/wallaby: Fix race condition in the sink when deleting records https://review.opendev.org/c/openstack/designate/+/817541 | 14:26 |
| opendevreview | Michal Arbet proposed openstack/designate stable/xena: Fix race condition in the sink when deleting records https://review.opendev.org/c/openstack/designate/+/817542 | 14:27 |
| eandersson | frickler figured out the issue | 17:47 |
| eandersson | My code was broken :p | 17:48 |
| johnsom | opps | 17:48 |
| opendevreview | Michael Johnson proposed openstack/designate stable/wallaby: Fix race condition in the sink when deleting records https://review.opendev.org/c/openstack/designate/+/817541 | 18:39 |
| opendevreview | Michael Johnson proposed openstack/designate stable/victoria: Support filtering on zone import/export list https://review.opendev.org/c/openstack/designate/+/817545 | 18:41 |
| opendevreview | Michael Johnson proposed openstack/designate stable/victoria: Fix race condition in the sink when deleting records https://review.opendev.org/c/openstack/designate/+/817540 | 18:42 |
| eandersson | johnsom do we wanna back port a release note for the sink fix too? | 20:08 |
| eandersson | It's the only reason I hadn't back ported the patches yet | 20:08 |
| johnsom | I am a fan of release notes. Grin | 20:15 |
| opendevreview | Merged openstack/designate stable/xena: Fix race condition in the sink when deleting records https://review.opendev.org/c/openstack/designate/+/817542 | 21:11 |
| opendevreview | Michael Johnson proposed openstack/designate stable/ussuri: Support filtering on zone import/export list https://review.opendev.org/c/openstack/designate/+/817547 | 22:34 |
| opendevreview | Michael Johnson proposed openstack/designate stable/ussuri: Fix pool update with duplicate masters https://review.opendev.org/c/openstack/designate/+/805143 | 22:34 |
| opendevreview | Michael Johnson proposed openstack/designate master: DNM: Debug uWSGI https://review.opendev.org/c/openstack/designate/+/786506 | 23:07 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!