| opendevreview | Tobias Urdin proposed openstack/puppet-neutron master: allow to use access_log_env_var for apache wsgi https://review.opendev.org/c/openstack/puppet-neutron/+/939590 | 08:21 |
|---|---|---|
| opendevreview | Tobias Urdin proposed openstack/puppet-keystone master: Fix broken OIDC auth due to OIDCRedirectURI change https://review.opendev.org/c/openstack/puppet-keystone/+/939777 | 10:11 |
| tobias-urdin | tkajinam: ^ if you could check above for regression that needs backporting | 10:11 |
| tkajinam | tobias-urdin, I guess we have to backport https://review.opendev.org/c/openstack/puppet-keystone/+/910637 instead ? | 10:49 |
| tkajinam | tobias-urdin, I've left a few comments. This made me aware that I should have added a release note for https://review.opendev.org/c/openstack/puppet-keystone/+/928755 which was released in 2024.2 ... | 10:52 |
| opendevreview | Tobias Urdin proposed openstack/puppet-keystone stable/2023.1: Fix wrong OIDCRedirectURI https://review.opendev.org/c/openstack/puppet-keystone/+/939781 | 10:59 |
| tobias-urdin | tkajinam: ah thx! created backport of that fix for 2023.1 instead | 10:59 |
| tobias-urdin | yeah release notes on both these changes could have saved some headache | 10:59 |
| tobias-urdin | i even forgot it myself and i approved it :D | 10:59 |
| tkajinam | ugh I made 2023.2 backport but did not create 2023.1 backport... | 11:16 |
| tkajinam | tobias-urdin, 2023.1 is on its way to EOL. I can discuss it with the release team if you want but it might be too later to merge that backport... | 11:17 |
| tkajinam | https://review.opendev.org/c/openstack/releases/+/934488 | 11:17 |
| tkajinam | if we merge it then we may need a new release | 11:17 |
| tobias-urdin | tkajinam: that's unfortunate, that it didn't make it into the final release, would you be ok checking with the release team? | 12:19 |
| tobias-urdin | would really help skip having to fork or pull all code for the template internally for fixing 4 bytes in a string | 12:20 |
| tobias-urdin | but if it's too late, then i'll have to fix it somehow myself | 12:21 |
| tkajinam | tobias-urdin, I've asked for opinions in #openstack-release so let's wait for it | 12:36 |
| tkajinam | if having it in an upstream branch helps then we can attempt to merge it after the branch is transitioned to unmaintained/2023.1 but adopting CI for unmaintained migration is messy, you know | 12:37 |
| tobias-urdin | tkajinam: thanks, that would work fine as wel | 12:38 |
| tobias-urdin | well* | 12:38 |
| tobias-urdin | i'd be fine with setting integration jobs as nonvoting and just merge the fix directly into unmaintained/2023.1 branch tbh, if you're ok with it | 12:38 |
| tkajinam | ok. I'll keep that in my mind. | 13:01 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-keystone stable/2023.1: Prepare final stable/2023.1 release again https://review.opendev.org/c/openstack/puppet-keystone/+/939804 | 14:22 |
| opendevreview | Merged openstack/puppet-keystone stable/2023.1: Fix wrong OIDCRedirectURI https://review.opendev.org/c/openstack/puppet-keystone/+/939781 | 15:30 |
| opendevreview | Merged openstack/puppet-keystone stable/2023.1: Prepare final stable/2023.1 release again https://review.opendev.org/c/openstack/puppet-keystone/+/939804 | 15:30 |
| tobias-urdin | tkajinam: thanks! | 16:04 |
| opendevreview | Merged openstack/puppet-neutron master: allow to use access_log_env_var for apache wsgi https://review.opendev.org/c/openstack/puppet-neutron/+/939590 | 16:18 |
| opendevreview | Merged openstack/puppet-openstack-integration master: nova: Enable management of ceph client https://review.opendev.org/c/openstack/puppet-openstack-integration/+/939675 | 16:40 |
| opendevreview | Tobias Urdin proposed openstack/puppet-openstack-integration master: dnm: test using openvox https://review.opendev.org/c/openstack/puppet-openstack-integration/+/939903 | 18:37 |
| opendevreview | Tobias Urdin proposed openstack/puppet-openstack-integration master: dnm: test using openvox https://review.opendev.org/c/openstack/puppet-openstack-integration/+/939903 | 18:53 |
| tobias-urdin | tkajinam: LarsErikP ^fun test to just straight up replace puppet | 19:51 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily for a restart to put some database compaction config changes into effect, and will return within a few minutes | 22:55 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!