| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-08-17 | |||
| 16:29:35 | sean-k-mooney | tkajinam: did the mock path fix merge yet | |
| 16:29:52 | tkajinam | sean-k-mooney, not yet | |
| 16:29:57 | sean-k-mooney | ack | |
| 16:30:12 | gmaan | gibi: sean-k-mooney on threading jobs series, I was in between of this one on friday but will check today, rest all were good https://review.opendev.org/c/openstack/nova/+/999121/3 | |
| 16:30:55 | tkajinam | there is one small simplification change related to SNP, waiting for 2nd +2 https://review.opendev.org/c/openstack/nova/+/998285 | |
| 16:31:19 | tkajinam | so it would be nice if we can merge that too | |
| 16:31:31 | tkajinam | https://review.opendev.org/c/openstack/nova/+/999780 is the mock path fix which sean-k-mooney mentioned, just for record | |
| 16:31:44 | sean-k-mooney | tkajinam: ill review that shortly but ya only checkign x86 makes sense | |
| 16:32:00 | tkajinam | yup | |
| 16:32:05 | tkajinam | a similar change is proposed for tdx as well (found in the series) | |
| 16:32:07 | sean-k-mooney | tkajinam: im ok with merging that as is and defering the rest till later | |
| 16:32:24 | sean-k-mooney | but just want to make sure gibi and dansmith are ok with that before +2ing | |
| 16:32:35 | sean-k-mooney | that for https://review.opendev.org/c/openstack/nova/+/999780 | |
| 16:32:39 | tkajinam | sean-k-mooney, you are talking about 999780, right ? | |
| 16:32:41 | tkajinam | ok | |
| 16:33:26 | sean-k-mooney | we coudl do more but i would prefer to fix the test and followup rather then continue to refine that change but i dont really mind in this case as long as we fix it before the release | |
| 16:33:35 | sean-k-mooney | * brefore rc1 | |
| 16:34:00 | sean-k-mooney | i dont wnat those test to start randomly failing when distos stat doign buils for the RC | |
| 16:34:34 | opendevreview | Takashi Kajinami proposed openstack/nova master: DNM: Test unit tests on kvm https://review.opendev.org/c/openstack/nova/+/999729 | |
| 16:34:56 | tkajinam | agree | |
| 16:35:27 | Uggla | I guess we are done with the topic ? | |
| 16:35:33 | sean-k-mooney | +1 | |
| 16:35:42 | tkajinam | I think that's all from my end. We can continue discussion about 999780 in reviews (and that can be mereged after FF because that is basically test only change) | |
| 16:35:43 | tkajinam | yup | |
| 16:36:05 | Uggla | thx antia, tkajinam, gibi. | |
| 16:36:11 | Uggla | #topic Bug scrubbing | |
| 16:36:17 | Uggla | #info up to 73 (-9) | |
| 16:36:22 | Uggla | #link https://etherpad.opendev.org/p/nova-bug-triage-roster | |
| 16:36:28 | Uggla | #link https://truc.uggla.fr/ to follow the trend. | |
| 16:36:35 | Uggla | Next meeting (this week): [public] Upstream bug triage. Wednesday, August 19th · 15:30 – 16:00 UTC. Video call link: meet.google.com/zjr-rxus-hzj | |
| 16:36:50 | Uggla | #topic Open discussion | |
| 16:37:09 | Uggla | Reminder: PTL Candidacy: I wanted to do another round for the 'I' cycle, but my manager suggested that it would be better to have someone else step in. I agree that, if possible, rotating the role is good for the project. | |
| 16:37:33 | Uggla | So if someone interested to run as candidate please let me know. | |
| 16:37:41 | Uggla | (Note: The deadline is August 19th.) | |
| 16:38:02 | Uggla | all on my side, something you'd like to discuss. | |
| 16:38:37 | lajoskatona | I have another 2 topics for migration mostly if you have some time for review that would be nice: | |
| 16:38:57 | lajoskatona | https://review.opendev.org/c/openstack/nova/+/982085 (Suppress error in cleanup of pre live migration ) | |
| 16:39:31 | lajoskatona | and the other one is for handling queue_size/mtu related values: https://review.opendev.org/c/openstack/nova/+/995841/3 | |
| 16:39:40 | lajoskatona | thanks in advance | |
| 16:39:41 | Uggla | lajoskatona 👍, I will add to the list if not already done. | |
| 16:39:55 | gmaan | one thing on gate status, i observed nova-grenade-multinode job failing for test_shelve_unshelve_server test more frequently last week | |
| 16:40:02 | gmaan | i reported bug for that | |
| 16:40:04 | gmaan | #link https://bugs.launchpad.net/nova/+bug/2163552 | |
| 16:40:15 | lajoskatona | Uggla: thanks, please let me know if I need for some extra meeting or similar | |
| 16:40:26 | gmaan | i did not debug it yet but in case anyone see similar failure, this is known one | |
| 16:40:54 | Uggla | gmann, I have seen it. I mean the bug. | |
| 16:41:26 | gmaan | k, I will try to debug it after FF and happen more frequently | |
| 16:41:52 | Uggla | gmaan any idea of the frequency ? | |
| 16:41:53 | gmaan | at least 4-5 times I saw this in different changes | |
| 16:42:01 | gmaan | #link https://zuul.opendev.org/t/openstack/builds?job_name=nova-grenade-multinode&skip=0 | |
| 16:42:09 | gmaan | this ^^ is state currenyty whihc is not bad | |
| 16:42:30 | sean-k-mooney | lajoskatona: updatign the mtu durring live migratrion is not a thing we supprot | |
| 16:42:54 | sean-k-mooney | lajoskatona: and there is a good reason fro that which si we do not allow the mtu to be chagne on a port that is atched to a vm | |
| 16:43:03 | sean-k-mooney | i kno nuetron can allow that but nova has never supproted that | |
| 16:43:18 | lajoskatona | sean-k-mooney: it 's not updating the value, but not changing if on the dst there is a different thing in the cfg, and breaking things in the VM | |
| 16:43:20 | sean-k-mooney | also adding or removign the mtu elemetn changes the virtio options | |
| 16:43:26 | sean-k-mooney | which make the vm un live mgiratable | |
| 16:43:35 | sean-k-mooney | so updating it woudl only work if the element already exitsed | |
| 16:44:18 | sean-k-mooney | lajoskatona: so the value of the mtu in the xml shoudl not change if the dest port bining has a diffent value | |
| 16:44:41 | sean-k-mooney | i.e. if you enable jumb frames on a network | |
| 16:44:51 | sean-k-mooney | that shoudl only reuslt in teh xml value updatign after a hard reboot | |
| 16:44:55 | sean-k-mooney | not on a live migration | |
| 16:46:09 | lajoskatona | the patch handles only the case when the nova-comp.cfg is different, nothing from Neutron side today (life is difficult even without that) and the same can happen with min/max_queue_size as the bug you opened (https://bugs.launchpad.net/nova/+bug/1854844) | |
| 16:46:40 | sean-k-mooney | we only use the mtu values form teh nova config as a fallback | |
| 16:46:47 | sean-k-mooney | and that is really only there for nova-networks | |
| 16:46:58 | sean-k-mooney | so the real fix is to stop usign those values entirly | |
| 16:47:09 | sean-k-mooney | or even remove them at this point | |
| 16:47:24 | lajoskatona | ok, that can be also a good point to avoid confusion with config | |
| 16:47:53 | sean-k-mooney | for the quese leght and count that is difent | |
| 16:47:58 | sean-k-mooney | as that is entrily config driven | |
| 16:48:01 | lajoskatona | so than the MTU part can be forgotten now (and let's check if the cfg can be deprecated and removed) | |
| 16:48:11 | sean-k-mooney | but for mtu you shoudl not config that in nova if your using neutron | |
| 16:48:33 | sean-k-mooney | ya for the queue config those cant change | |
| 16:48:40 | sean-k-mooney | and we need to use the souce value on live migriton | |
| 16:48:55 | sean-k-mooney | as that changes the way the nic is virutalsied to the guest | |
| 16:49:10 | sean-k-mooney | i think qemu will not alow the queue count to chagne whiel the guest is running | |
| 16:49:40 | lajoskatona | actually I got it from field where they upgraded from old nova/libvirt and during migrating around the VMs the MTU value mismatch had broken their apps, but if that is not used any more let's remove it | |
| 16:50:13 | lajoskatona | but I don't want to keep the meeting open, we can discuss this topic offline, or in gerrit | |
| 16:50:36 | Uggla | lajoskatona 👍 | |
| 16:50:42 | Uggla | anything else ? If not I think we can close and keep this meeting. | |
| 16:51:33 | sean-k-mooney | lajoskatona: it was common when changing form ml2/ovs to ml2/ovn | |
| 16:51:33 | bryanfraschetti | Hi folks, I'm hoping to get some reviews on this series: https://review.opendev.org/q/topic:%22improve_attachment_resiliance_against_timeout%22 | |
| 16:51:40 | sean-k-mooney | but the fix was to cold migrate | |
| 16:51:50 | sean-k-mooney | not to hack live meigtion to chnge the mtu | |
| 16:52:05 | sean-k-mooney | lajoskatona: we can chat about this more later | |
| 16:52:30 | lajoskatona | sean-k-mooney: ack, thanks | |
| 16:52:56 | Uggla | bryanfraschetti, at least I will put your series in the list. | |
| 16:53:28 | bryanfraschetti | Thank you! | |
| 16:54:10 | Uggla | anything else ? | |
| 16:55:01 | Uggla | No so Closing, thanks for joining this meeting. Have a nice day/evening. | |
| 16:55:11 | Uggla | Thanks for joining this meeting. Have a nice day/evening. | |
| 16:55:17 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-17-16.00.log.html | |
| 16:55:17 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-17-16.00.txt | |
| 16:55:17 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-17-16.00.html | |
| 16:55:17 | opendevmeet | Meeting ended Mon Aug 17 16:55:17 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:55:17 | Uggla | #endmeeting | |
| 16:55:23 | lajoskatona | o/ | |
| 17:30:59 | dansmith | gmaan: I'm not sure I understand this: https://github.com/openstack/nova/blob/master/nova/compute/rpcapi.py#L981 | |
| 17:31:56 | dansmith | this lets us start new live migrations after shutdown is already going.. I think your comment is about conductor having already checked with both source and dest, but is there any reason we need to actually proceed during shutdown there? | |
| 17:32:01 | dansmith | like some state conductor has altered? | |