| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-25 | |||
| 02:12:06 | gmann | bauzas: did you get chance to get placement project created? You need to ask one of the member of "OpenStack Administrators" in LP to create the placement and assign it to "nova-driver" group to further maintain that. project https://launchpad.net/~openstack-admins/+members#active | |
| 02:12:28 | gmann | bauzas: may be fungi can help on this ^^ | |
| 02:13:16 | gmann | please let me know once done, I will create the BP there for my placement spec | |
| 03:29:35 | opendevreview | Ghanshyam proposed openstack/placement master: Modify the placement API policies defaults and scope_type https://review.opendev.org/c/openstack/placement/+/865618 | |
| 03:30:44 | opendevreview | Nobuhiro MIKI proposed openstack/nova-specs master: Add maxphysaddr support for Libvirt https://review.opendev.org/c/openstack/nova-specs/+/861033 | |
| 03:51:33 | opendevreview | Ghanshyam proposed openstack/placement master: Modify the placement API policies defaults and scope_type https://review.opendev.org/c/openstack/placement/+/865618 | |
| 04:12:29 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/864055 | |
| 07:38:15 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: add regression test case for bug 1978983 https://review.opendev.org/c/openstack/nova/+/864168 | |
| 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 | |