bauzas | good morning Nova | 07:22 |
---|---|---|
opendevreview | Konrad Gube proposed openstack/nova-specs master: Re-propose using extend volume completion action for 2024.1 https://review.opendev.org/c/openstack/nova-specs/+/895648 | 10:59 |
sean-k-mooney | fyi https://github.com/mtreinish/stestr/issues/350 is going to make the gate explode | 11:36 |
opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Do not store InstanceInvalidState as instance_fault https://review.opendev.org/c/openstack/nova/+/860702 | 12:02 |
bauzas | sean-k-mooney: at least if the gate was actually working... | 12:09 |
sean-k-mooney | im pushing a uc fix for this now | 12:10 |
bauzas | sean-k-mooney: you know that we fail hard on https://launchpad.net/bugs/1940425 right? | 12:15 |
sean-k-mooney | ish | 12:16 |
sean-k-mooney | they reverted a particl fix on the neutron side for ovn | 12:16 |
sean-k-mooney | and have another fix | 12:16 |
sean-k-mooney | btu i dont think that is merged | 12:16 |
sean-k-mooney | it passes with ml2/ovs | 12:16 |
sean-k-mooney | so this is not a nova issue | 12:16 |
sean-k-mooney | its a neutorn one its also not an os-vif issues as far as i am aware | 12:17 |
sean-k-mooney | i have not looked that this in a while however | 12:17 |
sean-k-mooney | so i still stand by the invaldifor nova and incomplte/invalid for os-vif tatus | 12:17 |
sean-k-mooney | this is a neutron bug only when using ml2/ovn | 12:18 |
sean-k-mooney | we can skip that test if we want by default and leave it on in the hybrid plug job which uses ml2/ovs | 12:18 |
sean-k-mooney | until they fix it | 12:19 |
sean-k-mooney | bauzas: fyi i explained why this is a neutron bug in https://bugs.launchpad.net/neutron/+bug/1940425/comments/23 | 12:20 |
bauzas | yup, we know this is a neutron test and ralonsoh works on it AFAIK | 12:21 |
bauzas | context : https://review.opendev.org/c/openstack/nova/+/894945 is getting trampled a lot due to this | 12:21 |
sean-k-mooney | ya so in https://review.opendev.org/c/openstack/nova/+/893540 | 12:22 |
sean-k-mooney | i mentioned to dan that we could skip it in the other jobs where its failing and nova-ovs-hybrid-plug | 12:23 |
sean-k-mooney | would still give us coverage | 12:23 |
sean-k-mooney | want me to do that now? | 12:23 |
sean-k-mooney | we should unskip it in nova when they fix the bug but there is no reason to keep testing somethign that is broken in the interim | 12:23 |
bauzas | let's wait for dan to be here, but meh | 12:33 |
bauzas | both the changes I want to merge are in the gate by nox | 12:34 |
sean-k-mooney | ill put up a patch and ye can review | 12:39 |
bauzas | ++ | 12:46 |
opendevreview | sean mooney proposed openstack/nova master: disable ovn based testing of test_live_migration_with_trunk https://review.opendev.org/c/openstack/nova/+/895655 | 12:50 |
sean-k-mooney | i think ^ is correct but lets see what the ci says | 12:50 |
bauzas | ack | 12:53 |
opendevreview | sean mooney proposed openstack/nova-specs master: nova-specs shoudl use upper-constraints https://review.opendev.org/c/openstack/nova-specs/+/895683 | 13:41 |
opendevreview | Merged openstack/nova master: Revert "Add nova-manage ironic-compute-node-move" https://review.opendev.org/c/openstack/nova/+/894945 | 13:41 |
sean-k-mooney | bauzas: ok so nova-specs was not using UC so it got broken by the new release. since then there has also been a new release of stestr that corects the probelm | 13:46 |
sean-k-mooney | so i abandoned the uc clamp patch and made nova-spec actully use upper-constraits | 13:46 |
sean-k-mooney | it will also resolve itself once stestr 4.1.0 makes it way into the pip caches of the relvent ci providers so we should be unblokced there | 13:47 |
bauzas | huzzah, one last merge todo :) | 13:47 |
dansmith | bauzas: yeah we need to get that last one landed | 13:51 |
bauzas | yup | 13:56 |
bauzas | I'll also click on the nova-specs | 13:56 |
sahid | o/ | 14:37 |
sahid | qucik question regarding event vif-plug | 14:38 |
sahid | how nova neutron are communicating? | 14:38 |
sahid | we have lot of timout vif plug creation and are not able to find why | 14:38 |
dansmith | neutron calls to nova's api to post an event | 14:38 |
dansmith | it's been that way since icehouse, so nothing new | 14:39 |
sahid | ah yes perfect | 14:42 |
*** ykarel is now known as ykarel|away | 15:04 | |
opendevreview | Sylvain Bauza proposed openstack/nova master: Add a Bobcat prelude section https://review.opendev.org/c/openstack/nova/+/894940 | 15:41 |
bauzas | dansmith: sean-k-mooney: gmann: we need some approval of the prelude above before RC1 ^ | 15:42 |
opendevreview | Merged openstack/nova master: Revert "Limit nodes by ironic shard key" https://review.opendev.org/c/openstack/nova/+/894946 | 19:07 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!