| gmann | ade_lee: done, I am ok with those but let's add the job enable the fips and setup then we can go with the config option or so. I have commented in gerrit too. | 00:16 |
|---|---|---|
| opendevreview | Merged openstack/devstack master: Add additional project personas for secure RBAC https://review.opendev.org/c/openstack/devstack/+/780033 | 00:22 |
| *** pojadhav|afk is now known as pojadhav | 04:01 | |
| *** brinzhang0 is now known as brinzhang | 06:10 | |
| *** soniya29|ruck is now known as soniya29|ruck|lunch | 07:13 | |
| *** pojadhav- is now known as pojadhav | 08:08 | |
| *** soniya29|ruck|lunch is now known as soniya29|ruck | 08:11 | |
| *** jpena|off is now known as jpena | 08:36 | |
| opendevreview | Slawek Kaplonski proposed openstack/devstack master: Add missing configure_qos_ml2 function to devstack https://review.opendev.org/c/openstack/devstack/+/818355 | 09:38 |
| slaweq | simondodsley: clarkb I think I missed rehome of one function earlier, here is fix https://review.opendev.org/c/openstack/devstack/+/818355 | 09:40 |
| slaweq | sorry for that | 09:40 |
| slaweq | simondodsley: please let me know if that will work for You fine or not :) | 09:40 |
| *** pojadhav is now known as pojadhav|brb | 11:35 | |
| *** jpena is now known as jpena|lunch | 12:23 | |
| *** pojadhav|brb is now known as pojadhav | 12:31 | |
| opendevreview | Merged openstack/stackviz master: Updated Node.js Installation Command https://review.opendev.org/c/openstack/stackviz/+/817015 | 13:01 |
| frickler | anyone interested in centos might want to look into these appstream repo issues, tripleo also noticed this earlier in #opendev https://zuul.opendev.org/t/openstack/build/2402c562b08f445b8aa8460fb628321f | 13:03 |
| *** jpena|lunch is now known as jpena | 13:16 | |
| *** pojadhav is now known as pojadhav|afk | 15:14 | |
| *** akahat|rover is now known as akahat|dinner | 15:19 | |
| opendevreview | Slawek Kaplonski proposed openstack/devstack master: Add missing ml2, L2 and L3 agent functions to devstack https://review.opendev.org/c/openstack/devstack/+/818355 | 15:43 |
| opendevreview | mitya-eremeev-2 proposed openstack/tempest master: Take domain_id from config in some tests. https://review.opendev.org/c/openstack/tempest/+/818433 | 16:30 |
| ade_lee | gmann, responded to your comments in https://review.opendev.org/c/openstack/tempest/+/807465 | 16:33 |
| opendevreview | mitya-eremeev-2 proposed openstack/tempest master: Take domain_id from config in some tests. https://review.opendev.org/c/openstack/tempest/+/818433 | 16:36 |
| *** akahat|dinner is now known as akahat|rover | 16:58 | |
| gthiemonge | Hi Folks, this change in devstack https://review.opendev.org/c/openstack/devstack/+/780417/9/stack.sh is breaking the Octavia gates, our devstack plugin relies on OS_AUTH_URL to configure the octavia.conf file | 17:23 |
| gthiemonge | after this patch, OS_AUTH_URL is not set, and our configuration file no longer works | 17:23 |
| gthiemonge | so the question is: can we use OS_AUTH_URL in the plugins? or should we change for KEYSTONE_SERVICE_URI? | 17:25 |
| *** jpena is now known as jpena|off | 17:39 | |
| *** tkajinam is now known as Guest6220 | 18:30 | |
| TheJulia | https://review.opendev.org/c/openstack/devstack/+/780417 <-- shot ironic's gate in the foot by forcing an explicit OS_CLOUD setting override | 18:32 |
| TheJulia | gthiemonge: I suspect KEYSTONE_SERVICE_URI. It is what we use in ironic. | 18:34 |
| clarkb | TheJulia: would export OS_CLOUD=${OS_CLOUD:devstack-admin} work for your use case? It isn't clear to me if you are already setting that value or not. | 18:35 |
| TheJulia | clarkb: likely would fix us, tbh | 18:35 |
| TheJulia | ours gets set super early on in the run, where as this is coming in much later it seems | 18:36 |
| gthiemonge | TheJulia: thanks, I proposed a change in the octavia plugin to switch to KEYSTONE_SERVICE_URI | 18:36 |
| clarkb | TheJulia: do you want to write that change or should I? I can +2 on that repo and that seems within scope of something I'd be safe +2'ing if you want to push it | 18:37 |
| TheJulia | clarkb: yeah, give me a minute, just double checking one thing | 18:37 |
| clarkb | er I needed OS_CLOUD=${OS_CLOUD:-devstack-admin} | 18:37 |
| TheJulia | heh | 18:37 |
| TheJulia | yeah | 18:37 |
| opendevreview | Julia Kreger proposed openstack/devstack master: Make OS_CLOUD be able to be configured https://review.opendev.org/c/openstack/devstack/+/818449 | 18:42 |
| clarkb | gmann: ^ fyi I +2'd it and marked it as a gate blocker since it is affecting ironic's testing | 18:43 |
| TheJulia | test change uploaded to verify this fixes Ironic's CI | 18:43 |
| gmann | clarkb: checking | 18:43 |
| gmann | lgtm, waiting until ironic testing patch | 18:49 |
| gmann | gthiemonge: yeah, tacker did the same when they were broken. | 19:08 |
| frickler | TheJulia: which of the ironic jobs are failing currently? if that patch is causing issues, I wonder how it was working before | 19:09 |
| TheJulia | frickler: https://review.opendev.org/c/openstack/ironic/+/818299 is where I spotted it, specifically in ironic-tempest-functional-rbac-scope-enforced where we launch the job in a scope enforcing mode and it sets OS_CLOUD as part of the job setup in order to configure/interact with services | 19:11 |
| TheJulia | frickler: https://zuul.opendev.org/t/openstack/build/a59ac5f44a25423cae70da28e01cee2d is where you can see the errors being reported, since we're using OS_CLOUD in the job | 19:11 |
| frickler | TheJulia: ah, so you are setting OS_CLOUD very early, but unset the other OS_* stuff much later. if you'd do both at the same time, there wouldn't be an issue I think | 19:14 |
| TheJulia | https://zuul.opendev.org/t/openstack/build/b58f9464114d4e368028622405a532f4 passes now with that being pulled in | 19:14 |
| frickler | but anyway, your patch is fine IMO | 19:14 |
| TheJulia | the unsets are from Wallaby development | 19:14 |
| TheJulia | when we were not going to wait on devstack to have perfect support | 19:15 |
| TheJulia | I should be able to rip the unsets out now | 19:15 |
| frickler | gmann: could you drop your w-1 now? ^^ | 19:18 |
| gmann | frickler: TheJulia done | 19:27 |
| TheJulia | gmann: frickler: clarkb: much appreciated! | 19:27 |
| TheJulia | https://zuul.opendev.org/t/openstack/build/6db102bfcb714ebdab8e81ca27ee2ed5 <-- I wonder if the cinder library is broken | 19:47 |
| clarkb | programming error choosing an endpoint | 19:49 |
| clarkb | neat | 19:49 |
| frickler | TheJulia: clarkb: that's on keystone https://opendev.org/openstack/keystoneauth/src/branch/master/keystoneauth1/access/service_catalog.py#L281-L283 | 19:51 |
| frickler | but maybe the trigger is that cinder isn't ready for system scope yet | 19:56 |
| TheJulia | possibly, I don't think we force that in that job, so we'll see | 19:58 |
| TheJulia | some time() | 19:58 |
| frickler | TheJulia: well you are forcing it now if you set OS_CLOUD=devstack-admin-system early like we saw above. that setting had no effect on devstack until after you did your unset OS_* before 780417. after that, if forces system scope for all of devstack | 20:00 |
| frickler | s/after that/after 818449/ | 20:01 |
| TheJulia | hmm, yeah | 20:25 |
| TheJulia | I guess some of the commands were happy before hand, now not so much | 20:26 |
| opendevreview | Merged openstack/devstack master: Make OS_CLOUD be able to be configured https://review.opendev.org/c/openstack/devstack/+/818449 | 22:17 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!