ralonsoh | hello folks, can you review https://review.opendev.org/c/openstack/devstack/+/887790? | 08:30 |
---|---|---|
ralonsoh | That patch unblocks the stable/wallaby CI of Neutron | 08:31 |
ralonsoh | frickler, dansmith ^^ if you don't mind | 08:32 |
lucasagomes | hi frickler, we need https://review.opendev.org/c/openstack/devstack/+/887790 because a job in neutron stable/wallaby is currently broken without it. I left a comment there | 08:47 |
frickler | lucasagomes: so is this because you are deploying a different ovn version now? maybe neutron can also simply skip failing jobs in EM releases? | 09:07 |
lucasagomes | frickler, it started because the OVN_BRANCH devstack config was pointing to a commit hash but that was failing because wallaby lacks this: https://review.opendev.org/c/openstack/devstack/+/887184 | 09:08 |
lucasagomes | turns out this patch introduces a regression as you know | 09:09 |
lucasagomes | so we updated the OVN_BRANCH config in the wallaby job to not use a hash | 09:09 |
lucasagomes | and now it's failing due to this other permission problem | 09:09 |
lucasagomes | frickler, I guess, if none of these patches get in we will need to disable it yeah | 09:09 |
frickler | yes. but the job was working some time ago, wasn't it? so neutron must have changed something, is that still a newer OVN version that you were using originally? | 09:10 |
lucasagomes | frickler, yaeh I need to investigate but deffo something changed for sure | 09:10 |
lucasagomes | I think the OVN_BRANCH was updated in the base config | 09:11 |
lucasagomes | and that's shared for all branches *I think* | 09:11 |
lucasagomes | I will track it down | 09:11 |
opendevreview | Merged openstack/tempest master: Revert "Mark "test_live_migration_with_trunk" as unstable" https://review.opendev.org/c/openstack/tempest/+/887682 | 16:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!