| johnsom | New water pump ordered. Let me know if there is something I should do on that tomorrow. | 00:29 |
|---|---|---|
| eandersson | Pretty sure I am on to something here. The tests are passing, but could be pure chance. | 00:46 |
| eandersson | Gonna switch to.. uwsgi.. because... no designate-api logs with apache atm | 00:47 |
| johnsom | Sigh, I still need to get back to the uwsgi issue. | 00:47 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: [DNM] Testing https://review.opendev.org/c/openstack/designate/+/796167 | 01:11 |
| eandersson | I was totally wrong btw johnsom | 01:46 |
| eandersson | .get is magic and does not care about case | 01:46 |
| eandersson | having that said the code seems to work fine. I see the headers actually setting all_tenants | 01:50 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Updated middleware https://review.opendev.org/c/openstack/designate/+/796171 | 02:44 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Updated middleware https://review.opendev.org/c/openstack/designate/+/796171 | 02:48 |
| opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Updated middleware https://review.opendev.org/c/openstack/designate/+/796171 | 02:55 |
| eandersson | Not really able to confirm any of these big reports tbh | 03:57 |
| eandersson | Things go wrong for sure, but not sure if I can spot what is mentioned in the bug reports. | 03:58 |
| eandersson | In one of the tests failing I see "status": "PENDING" for the zone it is missing which makes snese. | 04:00 |
| eandersson | It happens. Sometimes pdns and designate takes time to sync. | 04:01 |
| eandersson | I'll probably look at this again tomorrow, but haven't been able to find any obvious re d flags here. | 04:08 |
| eandersson | It just looks to me like we are looking for zones stuck in pending | 04:09 |
| eandersson | Maybe this week I'll set up a local env to catch them locally instead of using the CI | 04:09 |
| eandersson | (btw could totally be wrong here) | 04:34 |
| eandersson | btw if we cant fix uwsgi we should at least fix logging for apache. That did work... at some point :D but it still wasn't as good as with uwsgi enabled | 04:42 |
| *** kinpaa12389 <kinpaa12389!~kinpaa123@59.94.50.75> has joined #openstack-dns | 12:24 | |
| kinpaa12389 | @johnsom https://review.opendev.org/c/openstack/designate-specs/+/794705 is updated. please check | 12:24 |
| *** gregraka <gregraka!~gregraka@99.29.115.253> has joined #openstack-dns | 13:01 | |
| opendevreview | Michael Johnson proposed openstack/designate master: Fix migration for sqlalchemy 1.4 https://review.opendev.org/c/openstack/designate/+/795906 | 21:45 |
| opendevreview | Michael Johnson proposed openstack/designate master: Re-enable the tempest scenario jobs https://review.opendev.org/c/openstack/designate/+/796321 | 21:46 |
| johnsom | eandersson nicolasbock If you agree with my approach on those two patches to get the SQLA fix in to unstick the basic gates, please review those two patches. Thanks | 21:47 |
| opendevreview | Michael Johnson proposed openstack/designate-tempest-plugin master: Update the constraints url https://review.opendev.org/c/openstack/designate-tempest-plugin/+/684539 | 21:51 |
| opendevreview | Michael Johnson proposed openstack/designate-tempest-plugin master: Use TOX_CONSTRAINTS_FILE https://review.opendev.org/c/openstack/designate-tempest-plugin/+/792319 | 21:55 |
| eandersson | LGTM johnsom | 22:08 |
| johnsom | Thanks | 22:08 |
| nicolasbock | Aren't 792319 and 684539 the same johnsom ? | 23:10 |
| nicolasbock | Your changes look good to me johnsom | 23:11 |
| johnsom | Thanks | 23:11 |
| johnsom | One patch was part of the URL, the other was for the variable name | 23:11 |
| nicolasbock | Oh, I missed that | 23:15 |
| nicolasbock | I reviewed them both now | 23:16 |
| johnsom | +1 Old was pretty old, so.... | 23:18 |
| johnsom | Old-One. Sigh, typing while in a meeting... | 23:49 |
| opendevreview | Merged openstack/designate master: Fix migration for sqlalchemy 1.4 https://review.opendev.org/c/openstack/designate/+/795906 | 23:57 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!