| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-08-17 | |||
| 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 | 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:33 | sean-k-mooney | lajoskatona: it was common when changing form ml2/ovs to ml2/ovn | |
| 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 | Uggla | #endmeeting | |
| 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 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-17-16.00.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 | Log: https://meetings.opendev.org/meetings/nova/2026/nova.2026-08-17-16.00.log.html | |
| 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? | |
| 17:32:25 | dansmith | point being, you can continue issuing new live migration requests during a shutdown which will eventually run out the timeout clock and cause a failure | |
| 17:33:42 | gmaan | dansmith: no, new live migratin will be rejected by check_can_live_migrate when conductor check it on source and dest. check_can_live_migrate_ is listneing on rpcserver which is stopped durign shutdown | |
| 17:34:17 | dansmith | ah okay, cool | |
| 17:34:32 | gmaan | that is why check_can_live_migrate make sure both source and dest are not shutting down so proceed rest of the live migration on rpc server alt in cae shutdown happen in betwen | |
| 17:34:42 | dansmith | gotcha | |
| 17:56:51 | gouthamr | dansmith: hey there, ack on the virtiofs cold-migration notes from the nova meeting, and ty for explaining stuff on the patch. i think i agree with your hardening suggestion more than any pointed fix i can make to the recovery paths.. i'm testing an update | |
| 17:58:26 | gouthamr | dansmith: ack also on the PTO :) i'm out on and for a couple days after feature freeze day too (27th Aug) .. but i'll react to any review feedback as soon as i can | |
| 17:59:01 | gouthamr | and i'll bother melwitt, Uggla and anyone else that was looking at this :) ty! | |
| 18:26:06 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Use service shutdown related config options in nova-graceful-shutdown https://review.opendev.org/c/openstack/nova/+/1001017 | |
| 18:38:32 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Improve the graceful shutdown job ansible tasks name https://review.opendev.org/c/openstack/nova/+/1001212 | |
| 19:04:14 | opendevreview | Alexandre Engloner proposed openstack/nova master: Typo fix in List Hypervisors details Policy path. https://review.opendev.org/c/openstack/nova/+/1001216 | |
| 20:01:43 | opendevreview | Clif Houck proposed openstack/nova master: perf(ironic): eliminate O(N²) ProviderTree deepcopy at startup https://review.opendev.org/c/openstack/nova/+/980676 | |
| 20:01:43 | opendevreview | Clif Houck proposed openstack/nova master: Parallelize per-node resource updates https://review.opendev.org/c/openstack/nova/+/980679 | |
| 20:35:14 | opendevreview | Merged openstack/nova master: Stabilize func test waiting for VM ERROR state https://review.opendev.org/c/openstack/nova/+/998822 | |
| 22:45:11 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | |
| #openstack-nova - 2026-08-18 | |||
| 01:02:21 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | |
| 01:59:12 | opendevreview | Merged openstack/nova master: Stabilize functional test for slow instance.save https://review.opendev.org/c/openstack/nova/+/999121 | |
| 06:15:53 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | |
| 09:09:50 | gibi | gmaan: I did some digging into the logs of https://bugs.launchpad.net/nova/+bug/2163552 it is strange as everything seems to correct, except that nova-api does not forwards the vif-plugged event to the compute even though the compute already waiting for it. | |
| 09:14:03 | gibi | gmaan: funnily that sounds very similar to the downstream bug report you triaged recently https://redhat.atlassian.net/browse/OSPRH-31354 | |
| 09:14:46 | gibi | and that is reported to Antelope | |
| 09:14:52 | gibi | and we see it on master | |
| 09:14:55 | gibi | crazy | |
| 09:44:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: [DNM]: Troubleshoot race condition in unshelve vif plug https://review.opendev.org/c/openstack/nova/+/1001294 | |
| 09:44:19 | gibi | gmaan: ^^ I added extra logs | |
| 11:33:08 | opendevreview | Balazs Gibizer proposed openstack/nova master: [DNM]: Troubleshoot race condition in unshelve vif plug https://review.opendev.org/c/openstack/nova/+/1001294 | |
| 12:45:28 | opendevreview | Merged openstack/nova master: Make functional-py313-threading voting https://review.opendev.org/c/openstack/nova/+/997216 | |
| 13:35:17 | dansmith | gibi: what is the purpose of the StaticallyDelayingCancellableTaskExecutorWrapper ? | |
| 13:36:34 | dansmith | oh for filtering out stops when a start is right on the heels | |
| 13:47:48 | opendevreview | Merged openstack/nova master: libvirt: Check only x86_64 arch for SEV capability detection https://review.opendev.org/c/openstack/nova/+/998285 | |
| 13:51:27 | gibi | dansmith: yeah that is an old behavior of the libvirt virt driver | |
| 13:51:40 | gibi | just eventlet had a native submit_after facility to handle that | |
| 13:51:42 | dansmith | I remember discussing it now | |
| 13:51:55 | dansmith | it's a very (impressively) long class name | |
| 13:52:22 | gibi | I'm open to suggestions :) (we can still change futurist before the deadline);5;13~ | |
| 13:53:37 | dansmith | are you building that into futurist? | |
| 13:58:15 | gibi | yes | |
| 13:58:18 | gibi | it is in now | |
| 13:58:29 | gibi | waiting for a lib release to use it from there | |
| 13:58:53 | gibi | https://github.com/openstack/futurist/commit/350e39d788a04fdbd38529564a0f44f3e395f80f | |
| 13:59:27 | gibi | https://review.opendev.org/c/openstack/nova/+/997410 this is the patch that will usit if from futurist | |
| 13:59:34 | dansmith | okay those seem like more reasonable names already :) | |
| 13:59:42 | gibi | */use it from/ | |
| 13:59:52 | gibi | thanks to Kamil :) | |
| 14:00:08 | gibi | and thanks to the fact that the delay is not static now | |
| 14:00:21 | gibi | we basically implemented submit_after for native thread pools | |
| 14:07:54 | dansmith | cool | |
| 14:58:49 | opendevreview | Ashish Gupta proposed openstack/nova master: Fix init_host crash migration test under native threading https://review.opendev.org/c/openstack/nova/+/1001369 | |
| 15:27:05 | gmaan | gibi: ok, i see, it seems similar to that downstream bug. I have some idea to add retry in server external event when it drop the network-vif-plugged but not sure if that will solve the actual root cause or hide. because form downstream bug, it was ~23 sec when resource claim happened (which set the host on instance) and nova-api still did not see the host set and dropped the event | |
| 15:28:26 | gmaan | thanks for the adding more logging, will do multiple recheck if we can hit it | |
| 15:28:32 | gibi | 23 seconds very long | |
| 15:28:55 | gmaan | yeah | |
| 15:29:18 | gibi | the current run of the DNM patch hit the issue but I haven't had the time to check the logs | |
| 15:51:18 | gmaan | nice, will check in noon | |
| 16:17:21 | opendevreview | Daniel Marques Caires proposed openstack/nova master: Add functional regression test for bug #2156927 https://review.opendev.org/c/openstack/nova/+/1000548 | |
| 16:17:22 | opendevreview | Daniel Marques Caires proposed openstack/nova master: nova: cast set_host_enabled on disable https://review.opendev.org/c/openstack/nova/+/998464 | |
| 16:20:14 | opendevreview | Daniel Marques Caires proposed openstack/nova master: Add functional regression test for bug #2156927 https://review.opendev.org/c/openstack/nova/+/1000548 | |
| 16:47:26 | opendevreview | Daniel Marques Caires proposed openstack/nova master: nova: cast set_host_enabled on disable https://review.opendev.org/c/openstack/nova/+/998464 | |
| 17:21:37 | opendevreview | Clif Houck proposed openstack/nova master: perf(ironic): eliminate O(N²) ProviderTree deepcopy at startup https://review.opendev.org/c/openstack/nova/+/980676 | |
| 17:21:38 | opendevreview | Clif Houck proposed openstack/nova master: Parallelize per-node resource updates https://review.opendev.org/c/openstack/nova/+/980679 | |
| 17:43:04 | opendevreview | Merged openstack/nova stable/2026.1: Restore 'fork' start method for daemon mode on Python 3.14 https://review.opendev.org/c/openstack/nova/+/999667 | |
| 19:14:55 | gmaan | dansmith: I replied to the unify thread pool comments, I see the value of initializing executors during service start so I am ok to change it in this effort only instead of doing it again. we still have 2 weeks for FF, i hope that is enough time to review. but let me know if you are busy for other things to review before FF then I will add the missing test coverage and refactor the initialization part later. | |
| 19:23:58 | gmaan | melwitt: this might fix the failure you saw for graceful shutdown job fail to stop service https://review.opendev.org/c/openstack/nova/+/1001017 | |
| 19:24:41 | melwitt | ah cool, will look | |
| 19:24:50 | gmaan | melwitt: and on top of it I improved that job tasks name so that it is easy to know which operation tests it is running https://review.opendev.org/c/openstack/nova/+/1001212 | |
| 19:25:34 | gmaan | currently ansible top level block name are not appended in console logs so adding operation name in front of those tasks | |
| 19:25:41 | gmaan | thanks | |
| 19:45:00 | dansmith | gmaan: don't change it now. | |
| 19:48:43 | gmaan | dansmith: ok | |
| 19:49:23 | dansmith | I mean, you can change it now, but don't change it in this commit.. let's not jeopardize landing the functionality :) | |
| 19:50:29 | gmaan | dansmith: got it, let me add the missing test coverage in this change and propose new one independent of graceful shutdown series for refactoring | |
| 19:51:05 | dansmith | yup thanks | |
| 21:34:51 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Unify the thread pool executors into a central factory https://review.opendev.org/c/openstack/nova/+/998571 | |
| 22:38:59 | gmaan | dansmith: ^^ a few jobs are still running in CI but it is ready | |
| #openstack-nova - 2026-08-19 | |||
| 02:50:22 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Shutdown thread pool executors during graceful shutdown https://review.opendev.org/c/openstack/nova/+/1000211 | |