| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-25 | |||
| 09:42:30 | auniyal | please review these - https://review.opendev.org/c/openstack/nova/+/864169 | |
| 09:50:20 | bauzas | sean-k-mooney: so, should we be non-voting nova-next ? | |
| 09:50:48 | sean-k-mooney[m] | just add the trunk test to the exclude regex | |
| 09:51:09 | sean-k-mooney[m] | for nova next and the nova ovs job | |
| 09:51:11 | bauzas | lemme fire a change | |
| 09:51:16 | bauzas | I'm done with rechecking | |
| 09:52:06 | bauzas | looks like we also have another problem https://zuul.opendev.org/t/openstack/build/6b766705d27d4308999ddd36d763e325 | |
| 09:55:22 | sean-k-mooney[m] | thats just volume detach | |
| 09:55:44 | sean-k-mooney[m] | qemu broke that 2 years ago and never fixed it | |
| 09:56:34 | sean-k-mooney[m] | for some reason the lvm job is showing it more often then everything else | |
| 09:56:37 | gibi | we did fixed detach in tempest so those failures are for something new | |
| 09:57:02 | sean-k-mooney[m] | fix is a stong word but yes we worked around the bootup issue in most cases | |
| 09:57:10 | sean-k-mooney[m] | i still think we were missing one patch | |
| 09:57:13 | gibi | yes but that work around means we should not see this failure | |
| 09:58:39 | bauzas | sean-k-mooney: do we have a launchpad bug tracking the trunk problem ? | |
| 09:58:56 | sean-k-mooney[m] | neutron do its not a nova issue | |
| 09:59:47 | bauzas | ralonsoh: do you have a launchpad bug about the live-migration bug with trunk ports ? | |
| 10:00:02 | bauzas | ralonsoh: context https://zuul.opendev.org/t/openstack/build/c6dee3859e354e6d8ed69c7ac7826761 | |
| 10:04:15 | gibi | bauzas: https://bugs.launchpad.net/nova/+bug/1940425 | |
| 10:04:37 | sean-k-mooney[m] | https://review.opendev.org/c/openstack/tempest/+/842240 is open for waiting before volume attach | |
| 10:05:27 | bauzas | gibi: excellent thanks | |
| 10:06:12 | gibi | sean-k-mooney[m]: good find | |
| 10:06:36 | ralonsoh | bauzas, https://bugs.launchpad.net/neutron/+bug/1997025 | |
| 10:06:48 | sean-k-mooney[m] | i just looked at https://review.opendev.org/q/topic:wait_until_sshable_pingable | |
| 10:08:03 | gibi | maybe gmann has context for that https://review.opendev.org/c/openstack/tempest/+/842240 . I already lost the context | |
| 10:08:12 | gibi | the linked other tempest patch was merged | |
| 10:08:22 | gibi | so maybe we just need to merge https://review.opendev.org/c/openstack/tempest/+/842240 | |
| 10:09:10 | 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 | |
| 10:09:28 | bauzas | gibi: sean-k-mooney: zuul needs your help ^ | |
| 10:09:35 | gibi | I've rebased https://review.opendev.org/c/openstack/tempest/+/842240 to get it out from W -1 | |
| 10:09:41 | gibi | bauzas: on it | |
| 10:09:54 | bauzas | I'm trying to figure out the occurrences | |
| 10:10:18 | bauzas | but after 3 days trying to merge my patches, I dare people to tell me the problem is only intermittent :) | |
| 10:10:37 | gibi | probably the frequency changed with the jammy switch | |
| 10:10:42 | gibi | it was intermittent in the past | |
| 10:11:05 | bauzas | oh, surely | |
| 10:11:07 | bauzas | the bug is old | |
| 10:11:17 | bauzas | the frequency changed tho | |
| 10:11:31 | sean-k-mooney[m] | its not something we can fix in nova | |
| 10:11:32 | bauzas | and that's why I'd like to eventually find the number | |
| 10:11:40 | bauzas | sean-k-mooney: don't disagree | |
| 10:11:51 | bauzas | the bug report says it's invalid for nhova | |
| 10:11:56 | ralonsoh | bauzas, gibi since we merged os-vif 3.1.0, the problem is not always reproducible but happens 90% of the executions | |
| 10:12:15 | ralonsoh | I'm finishing https://review.opendev.org/c/openstack/neutron/+/837780/ that shoudl avoid this issue in ML2/OVS | |
| 10:12:30 | ralonsoh | testing patch: https://review.opendev.org/c/openstack/neutron/+/865039/ | |
| 10:12:34 | gibi | ralonsoh: cool, thanks for working on it | |
| 10:14:15 | bauzas | ralonsoh: great, thanks for working on it | |
| 10:15:17 | bauzas | https://zuul.opendev.org/t/openstack/builds?job_name=nova-ovs-hybrid-plug&skip=0 | |
| 10:15:25 | bauzas | that's... interesting | |
| 10:16:19 | sean-k-mooney[m] | we have actully merged things on master in the last few days | |
| 10:16:24 | sean-k-mooney[m] | just not your patch | |
| 10:16:42 | sean-k-mooney[m] | but the failure rate is certenly high enough to be a problem | |
| 10:17:11 | bauzas | yeah, but despite finding occurences of the exact issue, I just see that the number of builds from this job becomes flakey | |
| 10:17:20 | sean-k-mooney[m] | bauzas: +1 on the zuul patch can you make the changes in line | |
| 10:17:38 | bauzas | sean-k-mooney: I wasn't pointing to my own patch and I understand we can continue to merge | |
| 10:17:52 | bauzas | sean-k-mooney: my only concern was the failure rate which is unknown | |
| 10:18:09 | bauzas | and I suspect it to be higher than previously | |
| 10:18:26 | sean-k-mooney[m] | oh i know i have noticed it while reviewing code over the last week or so | |
| 10:18:30 | bauzas | (just to clarify my words above) | |
| 10:18:48 | bauzas | sean-k-mooney: ack for the zuul +1, looking now | |
| 10:19:24 | bauzas | ralonsoh: good spot then, I'll change the regex | |
| 10:27:16 | 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 | |
| 10:27:25 | bauzas | gibi: sean-k-mooney: ralonsoh: updated ^ | |
| 10:28:50 | bauzas | please don't argue on the regex :) | |
| 10:29:27 | bauzas | and I preferred to have the full test name | |
| 10:29:52 | bauzas | (in case, a good regex trainer is https://regex101.com/= | |
| 10:36:37 | sean-k-mooney | sure that works i just prefer keping it short but correct | |
| 10:36:54 | sean-k-mooney | its on its way | |
| 10:50:12 | gibi | stephenfin: if you have time the pci in placement series next 7 patches are ready to land and was reviewed by sean-k-mooney[m] already. https://review.opendev.org/c/openstack/nova/+/852771/13 | |
| 11:10:00 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 11:17:21 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 11:22:31 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 11:26:50 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 11:41:29 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 12:18:25 | opendevreview | John Garbutt proposed openstack/nova-specs master: Ironic shard_key to replace peer_list https://review.opendev.org/c/openstack/nova-specs/+/862833 | |
| 12:26:20 | fungi | gmann: bauzas: i don't think openstack-admins needs to create projects on lp. anyone can do it | |
| 12:31:39 | fungi | at least i don't think i've ever created lp projects or groups for individual teams in the past, and i assume they always handled that themselves | |
| 13:17:54 | opendevreview | Fabian Wiesel proposed openstack/nova master: Add more password generation options https://review.opendev.org/c/openstack/nova/+/865669 | |
| 13:26:31 | Uggla | gibi, sorry I have just seen your msg :( | |
| 13:27:28 | Uggla | gibi, if you have time, sure you can continue with the virtiofs serie review. | |
| 13:53:10 | stephenfin | gibi: sorry, was looking at k8s CAPO issues. Will take a look, sure | |
| 14:18:56 | bauzas | f*** the zuul patch got hit by the nova-lvm job failure with volume detach | |
| 14:19:05 | bauzas | f*cking Friday | |
| 14:19:25 | bauzas | Black Friday is well suited name | |
| 14:22:11 | sahid | o/ bauzas, dansmith how do you feel about https://review.opendev.org/c/openstack/nova-specs/+/857838 ? the code is ready I hope without too many issues :-) | |
| 14:35:12 | bauzas | sahid: today's holiday for US folks | |
| 14:35:41 | bauzas | sahid: but I can take a look if I have time | |
| 14:36:03 | sahid | bauzas: oh.. did not noticed that | |
| 14:36:05 | bauzas | gibi: https://zuul.opendev.org/t/openstack/builds?job_name=nova-lvm&skip=0 makes me feel nova-lvm is broken on master since a couple of days | |
| 14:36:24 | sahid | sure please, but only if it's to put a +2 ;) | |
| 15:13:07 | opendevreview | Sylvain Bauza proposed openstack/nova-specs master: Proposes cpu power managment in libvirt https://review.opendev.org/c/openstack/nova-specs/+/861591 | |
| 15:13:24 | bauzas | gibi: sean-k-mooney: updated my cpu power mgmt spec | |
| 15:45:51 | gibi | bauzas: ack on both | |
| 15:46:05 | gibi | stephenfin: thanks! | |
| 15:47:50 | gibi | Uggla: ack, I will try to get to it | |
| 15:50:08 | bauzas | gibi: yet again, nova-lvm failed | |
| 15:50:13 | bauzas | https://review.opendev.org/c/openstack/nova/+/865658 | |
| 15:50:21 | gibi | yeah it seems pretty brokenish | |
| 15:50:54 | bauzas | I'm gonna amend the patch above to skip the other test in nova-lvm | |
| 15:52:07 | gibi | yeah we need to. but we will forget about them unfortunately | |