opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: Move context from tenant to project_id https://review.opendev.org/c/openstack/designate/+/739394 | 05:00 |
---|---|---|
opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: [WIP] Remove unused central code https://review.opendev.org/c/openstack/designate/+/824503 | 05:40 |
opendevreview | Erik Olof Gunnar Andersson proposed openstack/designate master: [WIP] Remove unused central code https://review.opendev.org/c/openstack/designate/+/824503 | 05:42 |
opendevreview | Merged openstack/designate-tempest-plugin master: Adds an API test for list transfer requests sudo https://review.opendev.org/c/openstack/designate-tempest-plugin/+/805320 | 06:29 |
opendevreview | Merged openstack/designate-tempest-plugin master: Update the constraints url https://review.opendev.org/c/openstack/designate-tempest-plugin/+/684539 | 06:47 |
opendevreview | Arkady Shtempler proposed openstack/designate-tempest-plugin master: New test cases for tsigkey tests suite https://review.opendev.org/c/openstack/designate-tempest-plugin/+/798277 | 10:32 |
eandersson | frickler I wonder why grenade failed on my WIP patch above. Is maybe the designate-api not restarted properly since we moved to a wsgi endpoint? | 21:17 |
eandersson | https://zuul.opendev.org/t/openstack/build/4cac468f320e410692b683c8866fb1f2/log/controller/logs/grenade.sh_log.txt#18259 | 21:17 |
johnsom | zone status unknown... hmm | 21:19 |
eandersson | https://zuul.opendev.org/t/openstack/build/4cac468f320e410692b683c8866fb1f2/log/controller/logs/screen-designate-central.txt?severity=4#2697 | 21:20 |
eandersson | Pretty sure it is this | 21:20 |
johnsom | eandersson https://review.opendev.org/c/openstack/designate/+/823624/11/designate/central/rpcapi.py#67 | 21:21 |
eandersson | Yea - I see in the logs that central is restarted, but not api | 21:21 |
eandersson | oh.. | 21:22 |
eandersson | maybe because previous was apache and current is uwsgi | 21:22 |
eandersson | mod_wsgi vs. uwsgi | 21:22 |
eandersson | https://zuul.opendev.org/t/openstack/build/4cac468f320e410692b683c8866fb1f2/log/controller/logs/screen-designate-api.txt | 21:22 |
eandersson | no requests at all on that endpoint | 21:23 |
eandersson | https://zuul.opendev.org/t/openstack/build/4cac468f320e410692b683c8866fb1f2/log/controller/logs/apache/designate-api_log.txt probably going here instead | 21:23 |
johnsom | Yeah, the "start" in the api log matches the "upgrading designate" timestamp from grenade log | 21:24 |
johnsom | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_4ca/824503/2/check/designate-grenade-pdns4/4cac468/controller/logs/grenade.sh.log_summary.txt | 21:24 |
eandersson | Yep - but apache config still pointing at old endpoint, not new https://zuul.opendev.org/t/openstack/build/4cac468f320e410692b683c8866fb1f2/log/controller/logs/apache_config/designate-api_conf.txt | 21:25 |
johnsom | Yeah, I was just going to look for that | 21:26 |
eandersson | It's weird thou because mod_wsgi should still work fine | 21:26 |
johnsom | Yeah, it should have restarted apache and still picked up the new code | 21:27 |
eandersson | I def see apache being restarted before the upgrade and not after | 21:29 |
eandersson | 2022-01-13 06:22:24.295 | + functions-common:restart_service:2299 : sudo /bin/systemctl restart apache2 | 21:30 |
eandersson | 2022-01-13 06:22:43.311 | + /opt/stack/new/designate/devstack/upgrade/upgrade.sh:main:71 : upgrade_project designate /opt/stack/new/designate/devstack/upgrade stable/xena master | 21:30 |
eandersson | That is the last apache restart I could find | 21:30 |
eandersson | this is def where uwsgi is much better | 21:31 |
eandersson | Actually totally makes sense | 21:32 |
eandersson | Since we migrated to uwsgi, only uwsgi is restarted | 21:32 |
johnsom | Looking in the grenade scripts, I don't see it bouncing apache | 21:33 |
johnsom | I guess under mod_wsgi no one bumped the rpc version... | 21:34 |
johnsom | So... We might need a devstack/upgrade/from-xena/upgrade-designate script that calls the devstack code to write out the apache config and restart apache | 21:41 |
eandersson | Yea - that makes sense. I wonder why the previous patch didn't break. | 23:58 |
eandersson | 6.2 (current) -> 6.3 -> 7.0 | 23:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!