| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-25 | |||
| 07:38:16 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/864169 | |
| 07:38:17 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Block unsupported instance ops with accelerators. https://review.opendev.org/c/openstack/nova/+/865624 | |
| 07:54:10 | Uggla | good morning Nova ! | |
| 07:57:07 | gibi | o/ | |
| 07:57:32 | gibi | Uggla: do you have anything I should review for you? | |
| 08:01:56 | opendevreview | Jorge San Emeterio proposed openstack/nova-specs master: Review usage of oslo-privsep library on Nova https://review.opendev.org/c/openstack/nova-specs/+/865432 | |
| 08:04:14 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 08:09:19 | opendevreview | Jorge San Emeterio proposed openstack/nova-specs master: Review usage of oslo-privsep library on Nova https://review.opendev.org/c/openstack/nova-specs/+/865432 | |
| 08:13:56 | bauzas | hello Nova | |
| 08:14:14 | bauzas | gmann: ack, I didn't had time yet, but I can ask indeed | |
| 08:14:32 | bauzas | I was thinking we were having the placement LP | |
| 08:26:33 | gibi | sean-k-mooney: finally the stable/train version of https://review.opendev.org/c/openstack/nova/+/841444 is green, it took a while to land the fix that switched back the jobs to focal | |
| 08:28:23 | sean-k-mooney[m] | ah that was till in flight i saw that it was proposed but had assumed that the job change had landed already | |
| 08:29:30 | gibi | me too but | |
| 08:29:46 | sean-k-mooney[m] | gibi: can you repond to artoms comment on the previous patch https://review.opendev.org/c/openstack/nova/+/841288/3 | |
| 08:30:41 | sean-k-mooney[m] | im technically off today but im not going car shopping for 3 hours so i can take a look at a few things in the meantime | |
| 08:32:13 | gibi | done | |
| 08:32:27 | gibi | I'm not sure I see the issue artom saw | |
| 08:49:09 | sean-k-mooney[m] | i just added some more comments | |
| 08:49:37 | sean-k-mooney[m] | there is one nit but im ok with the contnet im going to quickly reivew the actual fix | |
| 08:55:37 | sean-k-mooney[m] | gibi: ok im +2 on both of those | |
| 08:55:42 | gibi | thanks | |
| 09:42:20 | auniyal | Hi | |
| 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 | |