| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-28 | |||
| 13:16:55 | gibi | but gmann hopefully knows more about it | |
| 13:17:15 | bauzas | I'm just wrapping up all the LP bug reports | |
| 13:17:15 | gibi | so I think we need to disable those volume tests that are failing currently, not all volume tests | |
| 13:17:18 | sean-k-mooney | https://review.opendev.org/c/openstack/tempest/+/840410 | |
| 13:17:26 | sean-k-mooney | this one might also be interesting ^ | |
| 13:17:42 | gibi | yes, we need that too to disable trunk | |
| 13:17:49 | gibi | there we know ralonsoh working on a fix | |
| 13:18:11 | sean-k-mooney | https://review.opendev.org/c/openstack/tempest/+/840410 fixes a issue witht he cleanup function ordering | |
| 13:18:17 | sean-k-mooney | related to volumes | |
| 13:18:57 | gibi | sorry i mixed up things | |
| 13:19:03 | gibi | https://review.opendev.org/c/openstack/tempest/+/840410 is new to me | |
| 13:19:06 | sean-k-mooney | bauzas: could you perhaps also add a depend on for that | |
| 13:19:46 | gibi | probably we need a single nova patch that disables trunk (as in https://review.opendev.org/c/openstack/nova/+/865658) and disable the failing volume tests | |
| 13:20:15 | sean-k-mooney | ya that would be simpler to merge | |
| 13:20:24 | bauzas | sean-k-mooney: sec, writing an email to inform the world we're busted | |
| 13:28:37 | bauzas | I'm here | |
| 13:28:40 | bauzas | so | |
| 13:29:19 | sean-k-mooney | joing an internal call shortly but https://review.opendev.org/c/openstack/tempest/+/840410 may help with the volume issues | |
| 13:29:29 | sean-k-mooney | if you can add that as a second depends on it might help | |
| 13:29:34 | bauzas | sean-k-mooney: I can try | |
| 13:29:48 | bauzas | that's gonna be fun | |
| 13:29:54 | sean-k-mooney | alternitivly we can tempoerly disable the volume tests in the same patch that is skiping the tunk tests | |
| 13:30:08 | sean-k-mooney | and then try that in the followup that turns them back on | |
| 13:30:10 | bauzas | https://lists.openstack.org/pipermail/openstack-discuss/2022-November/031335.html | |
| 13:30:41 | bauzas | ralonsoh: you also have a different tracking bug number for the wrong port state | |
| 13:30:59 | bauzas | ralonsoh: maybe we should merge those 2 bug reports we have | |
| 13:31:21 | bauzas | sean-k-mooney: editing inline in gerrit, so we'll see | |
| 13:32:55 | opendevreview | Sylvain Bauza proposed openstack/nova master: Skip live_migration_with_trunk tempest test on ovs-hybrid and nova-next https://review.opendev.org/c/openstack/nova/+/865658 | |
| 13:39:54 | ralonsoh | bauzas, yes, one sec | |
| 13:40:43 | ralonsoh | bauzas, https://bugs.launchpad.net/neutron/+bug/1997025 | |
| 13:40:51 | ralonsoh | patch: https://review.opendev.org/c/openstack/neutron/+/837780 | |
| 13:41:48 | ralonsoh | I've increased the priority of th ereview | |
| 13:41:50 | bauzas | ralonsoh: would you disagree if I'm closing https://bugs.launchpad.net/neutron/+bug/1997025 as duplicate of https://bugs.launchpad.net/neutron/+bug/1940425 which was already triaged for Neutron ? | |
| 13:42:19 | ralonsoh | bauzas, that issue was something different | |
| 13:42:41 | ralonsoh | in https://bugs.launchpad.net/neutron/+bug/1997025, the port is not activated | |
| 13:42:49 | ralonsoh | and this is because of the os-vif release | |
| 13:43:02 | bauzas | Hmmm, OK | |
| 13:43:12 | ralonsoh | --> https://bugs.launchpad.net/neutron/+bug/1997025/comments/1 | |
| 14:46:55 | opendevreview | Sylvain Bauza proposed openstack/nova master: Skip live_migration_with_trunk tempest test on ovs-hybrid and nova-next https://review.opendev.org/c/openstack/nova/+/865658 | |
| 14:47:37 | bauzas | yet another try ^ | |
| 14:47:58 | bauzas | the previous run was failing | |
| 17:13:09 | bauzas | sean-k-mooney: gibi: so, I'm about to skip the nova-lvm tests that fail | |
| 17:13:53 | bauzas | fwiw, nova-grenade-multinode also hits the port not active isue https://zuul.opendev.org/t/openstack/build/16cc990d758348999cc5c1b2b30aa9a2 | |
| 17:15:11 | gibi | bauzas: ack | |
| 17:18:55 | ralonsoh | bauzas, https://review.opendev.org/c/openstack/neutron/+/837780 | |
| 17:19:02 | ralonsoh | we have merged the patch in Neutron | |
| 17:19:08 | ralonsoh | I think you should be OK now | |
| 17:19:12 | ralonsoh | (I hope so) | |
| 17:19:35 | bauzas | ah, nice to hear | |
| 17:19:45 | bauzas | I can try to recheck my original patch | |
| 17:19:56 | bauzas | ralonsoh: kudos to the neutron team | |
| 17:20:00 | ralonsoh | yw | |
| 17:25:01 | sean-k-mooney | ralonsoh ah an ye removed the config option great | |
| 17:25:06 | sean-k-mooney | then it should just work | |
| 17:25:15 | sean-k-mooney | without us needing to modify jobs | |
| 17:25:20 | ralonsoh | yes referring to the Nova patch | |
| 17:26:08 | bauzas | ralonsoh: sean-k-mooney: I just rechecked https://review.opendev.org/c/openstack/nova/+/838976/8 | |
| 17:26:12 | bauzas | we will see | |
| 17:26:24 | ralonsoh | bauzas, I'll keep an eye on it | |
| 17:26:39 | sean-k-mooney | bauzas: we still need to skip the lvm tests | |
| 17:26:41 | sean-k-mooney | right | |
| 17:26:43 | bauzas | either way, it will fail due to the nova-lvm issue but we'll see whether ovn and nova-next jobs work | |
| 17:26:50 | bauzas | sean-k-mooney: yup | |
| 17:27:04 | bauzas | this is just for checking against something which is not my skiptest patch | |
| 17:27:17 | sean-k-mooney | ya ok well it might pass put its unlikely to do it twice to get passed both gate and check | |
| 17:27:28 | bauzas | yup | |
| 17:27:39 | bauzas | nova-lvm seems to fail 100% of times | |
| 17:29:19 | sean-k-mooney | is nova-lvm using pc or q35 by the way i assume pc | |
| 17:30:33 | sean-k-mooney | the ohter thing we could try is putting it back to focal temporilly | |
| 17:30:53 | sean-k-mooney | thats more invasive however then skiping the flaky tests | |
| 17:31:42 | sean-k-mooney | not set in the nova.conf so it should be pc | |
| 17:31:49 | sean-k-mooney | https://zuul.opendev.org/t/openstack/build/f076244738904c949e63b21571a5fc9a/log/controller/logs/etc/nova/nova-cpu_conf.txt#118 | |
| 17:46:11 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/863806 | |
| 17:46:12 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864055 | |
| 17:57:40 | gmann | gibi: bauzas sean-k-mooney: hi, for detach issue in nova-lvm job. tempest patch need more time it seems. I was making server sshable by default which need more work in other tests. | |
| 17:58:55 | gmann | gibi: sean-k-mooney bauzas: I am moving my house this week so might need some more time on that. For now, I think we can go with skipping the failing tests temporarily in nova-lvm | |
| 17:59:06 | bauzas | ack | |
| 17:59:09 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864007 | |
| 17:59:15 | gibi | gmann: thanks for looking into it | |
| 18:00:54 | sean-k-mooney | so i think we shoudl be ok to skip them for a few weeks to unblock things | |
| 18:01:16 | gibi | yepp I agree | |
| 18:01:25 | sean-k-mooney | the cidner code paths really shoudl not depend on the images_backend that is in use at least not lvm | |
| 18:01:26 | gibi | sady we have no better option now | |
| 18:01:29 | gmann | but thing is those failing rescue test, all use ssh-able server. there is something else might be causing the failure | |
| 18:01:45 | sean-k-mooney | gmann: yep there likely is | |
| 18:01:55 | gmann | something to do on Jammy as lvm on focal working fine | |
| 18:02:11 | sean-k-mooney | possibly | |
| 18:02:28 | sean-k-mooney | it defiently happens more on jammy im not sure it would not happen on focal | |
| 18:02:43 | sean-k-mooney | it might be related to a default config delta between the two or the qemu/libvirt verions | |
| 18:02:59 | gmann | may be | |
| 18:02:59 | sean-k-mooney | but we dont currently have a lot of capsity to investigate | |
| 18:03:40 | gmann | yeah, do we have patch up to skip the tests? | |
| 18:04:05 | sean-k-mooney | i think bauzas has one but im not sure they updated for the volume tests | |
| 18:04:19 | gmann | k | |
| 18:04:51 | gmann | this one? https://review.opendev.org/c/openstack/nova/+/865658 | |
| 18:05:13 | sean-k-mooney | that was to solve the other issue so that is not needed | |
| 18:05:14 | gmann | this does not include nova-lvm failing tests | |
| 18:05:21 | gmann | k | |
| 18:05:23 | sean-k-mooney | we shoudl pull that form the gate | |
| 18:06:22 | sean-k-mooney | gmann: i do not see one form sylvain | |