| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-08-18 | |||
| 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 | |
| 08:55:59 | gibi | assume | |
| 08:56:08 | sean-k-mooney | and it normlaly works on amd | |
| 08:56:16 | sean-k-mooney | but i have seen it not work in some cases | |
| 08:56:36 | sean-k-mooney | have you tested host-model | |
| 08:56:39 | antia | amd has a cpu model with AMD SEV-SNP support (EPYC-v4), but there is none with Intel TDX support | |
| 08:56:45 | gibi | in this case it triggers the feature precondition check without proper image meta passed causing the issue in that bug | |
| 08:58:34 | antia | TDX requires qemu to have -cpu host which is where the requirement for host-passthrough comes from | |
| 08:58:49 | sean-k-mooney | antia: but is that being worked on to fix in qemu | |
| 08:59:10 | sean-k-mooney | antia: its totally fine if that is a limaition in the current qemu | |
| 08:59:28 | sean-k-mooney | but long term that is not somethign we shoudl continue to require | |
| 08:59:58 | sean-k-mooney | we generaly try not ot add feature that do not supprot live migration | |
| 09:00:08 | sean-k-mooney | the excption to that is if its a hardare limiation | |
| 09:00:17 | sean-k-mooney | which this mostly is today | |
| 09:00:26 | sean-k-mooney | (mix of hardare and qemu limiation) | |
| 09:01:36 | sean-k-mooney | as long as there is a path to eventuly supproting live migration with TDX or SEV instance that fine but i woudl hate to force operator to choose between secure vms and 5 9s of uptime for workloads | |
| 09:01:40 | gibi | yeah regardless of the model support live migration is not supported for SEV or TDX | |
| 09:03:20 | sean-k-mooney | gibi: by the way, our current cirros image dont actully supprot uefi boot | |
| 09:03:31 | sean-k-mooney | you can enabel it but the guest wont actully boot | |
| 09:04:11 | sean-k-mooney | qemu/libirt will boot the vm but i think the image just dont have the efi partion requried and the relevent grub config | |
| 09:04:15 | gibi | I use an ubuntu image locally to be able to try attestation as that anyhow needs an extra kernel module in each case | |
| 09:04:27 | sean-k-mooney | ya so on that | |
| 09:04:40 | sean-k-mooney | im kind wish we didnt enable that as part of this feature | |
| 09:04:52 | sean-k-mooney | that really fells like it shoudl have been out of scope | |