| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-08-18 | |||
| 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 | |
| 05:18:19 | opendevreview | Bhavana proposed openstack/nova-specs master: Add spec for MIG-aware PCI Host Weigher https://review.opendev.org/c/openstack/nova-specs/+/998753 | |
| 05:54:31 | opendevreview | chandan kumar proposed openstack/nova master: Add libvirt driver MDEV ARQ support for Cyborg https://review.opendev.org/c/openstack/nova/+/995840 | |
| 06:06:21 | opendevreview | Goutham Pacha Ravi proposed openstack/nova master: Support cold migration with virtiofs shares https://review.opendev.org/c/openstack/nova/+/989633 | |
| 08:07:19 | opendevreview | Lajos Katona proposed openstack/nova master: libvirt: Add test for MTU overwrite during live migration https://review.opendev.org/c/openstack/nova/+/995803 | |
| 08:36:24 | opendevreview | Lajos Katona proposed openstack/nova master: libvirt: Preserve source MTU during live migration https://review.opendev.org/c/openstack/nova/+/995809 | |
| 08:36:57 | opendevreview | Lajos Katona proposed openstack/nova master: Add regression test to repoduce bug 1854844 https://review.opendev.org/c/openstack/nova/+/991772 | |
| 08:44:38 | gibi | sean-k-mooney: thanks for the TDX review, and good catch on the cgroups problem. I agree with having a config option until libvirts add capacity reporting | |
| 08:44:57 | gibi | I will test out scsi support on my machine | |
| 08:45:03 | sean-k-mooney | ack | |
| 08:45:08 | gibi | and I confirmed that serial access works | |
| 08:45:15 | sean-k-mooney | that good | |
| 08:45:22 | sean-k-mooney | at least one of the concoles will | |
| 08:45:28 | gibi | yeah :) | |
| 08:45:36 | gibi | super secure VM, you cannot even access it :) | |
| 08:45:43 | sean-k-mooney | :) | |
| 08:45:44 | gibi | nobody can. done. | |
| 08:45:48 | sean-k-mooney | most secure | |
| 08:46:07 | sean-k-mooney | so i was debating soemting as well | |
| 08:46:16 | sean-k-mooney | if you have enabel tdx | |
| 08:46:36 | sean-k-mooney | shoudl we automaticlly select hw_video_model="none" for you | |
| 08:46:44 | sean-k-mooney | if its not set in the image | |
| 08:47:01 | sean-k-mooney | technially when its not set in the image the libvirt driver choose for you | |
| 08:47:15 | sean-k-mooney | it used to chosoe cirros and we updated it to virtio-gpu | |
| 08:47:32 | sean-k-mooney | but it could take into account tdx | |
| 08:48:08 | gibi | hm that is not a bad idea. I would say we can add that as a follow up | |
| 08:48:17 | sean-k-mooney | yep can be a followup | |
| 08:48:32 | gibi | currently we document that you need to set none explicitly, and we can relax that later | |
| 08:48:45 | gibi | that is a nice UX improvement | |
| 08:48:58 | sean-k-mooney | the real reason i said this feature feel like it shoudl be experimatal is the amount of specifcal casing you need to do to make it work at all combiend with the fact it only works with host-passthough | |
| 08:49:12 | sean-k-mooney | that last part makes me quite uncofrrotable | |
| 08:49:25 | sean-k-mooney | i like host-passthough for performace | |
| 08:49:39 | sean-k-mooney | but we have never had a feature that only works when that is used before | |
| 08:50:05 | sean-k-mooney | so i think if you ahve not selected host-passtouhg we shoudl likely avoid reporting tdx supprot as a trait | |
| 08:50:11 | sean-k-mooney | even when the host supprot it | |
| 08:50:35 | sean-k-mooney | with that sait | |
| 08:50:37 | sean-k-mooney | *said | |
| 08:50:48 | sean-k-mooney | im hoping the host-passthough requirement is tempory | |
| 08:51:01 | sean-k-mooney | so if future qemu cpu models supprot tdx | |
| 08:51:15 | sean-k-mooney | then we dont want the extra code in nova to filter the triats | |
| 08:51:42 | opendevreview | Lajos Katona proposed openstack/nova master: libvirt: Preserve queue_size during live migration https://review.opendev.org/c/openstack/nova/+/995841 | |
| 08:51:49 | sean-k-mooney | taht why i didnt suggest that in the review but ya it makes me a bit uneasy | |
| 08:53:51 | gibi | I'm not sure that host-passthrough requirement is valid on cpu level or just a consequence of this bug https://bugs.launchpad.net/nova/+bug/2158927/comments/1 | |
| 08:54:37 | gibi | antia: ^^ could you confirm which is the case? | |
| 08:55:21 | gibi | sean-k-mooney: antia: fyi I'm on PTO Thru/Fri this week but I'm back on Monday | |
| 08:55:33 | sean-k-mooney | gibi: ack | |
| 08:55:57 | gibi | we have FF next Thursdays I assuem | |
| 08:55:59 | sean-k-mooney | so cpu_models = Nehalem is an intel model | |