| Posted | Nick | Remark | |
|---|---|---|---|
| #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 | |
| 09:05:09 | antia | removing the need for -cpu host was marked as future work, but not sure where it is today | |
| 09:05:09 | gibi | you mean attestation? | |
| 09:05:10 | antia | removing the need for -cpu host was marked as future work | |
| 09:05:13 | antia | ops | |
| 09:05:17 | antia | https://lists.nongnu.org/archive/html/qemu-devel/2025-04/msg00097.html | |
| 09:05:23 | sean-k-mooney | yes enabel attestation in a portable way is a seprate feature in my view | |
| 09:05:35 | gibi | sean-k-mooney: sure nova does nothing really with attestation | |
| 09:05:37 | sean-k-mooney | so i woudl have wanted that to be its own spec adn have its own extra spec | |
| 09:05:49 | sean-k-mooney | gibi: but we used too | |
| 09:06:03 | sean-k-mooney | we had supprot for opens attstation and trusted boot before | |
| 09:06:04 | gibi | the sev one works out of the box via the cpu itself. the tdx works via a local socket added automatically by libvirt to the guest | |
| 09:06:06 | sean-k-mooney | and we riped it out | |
| 09:06:24 | gibi | so we don't carry any code | |
| 09:06:26 | sean-k-mooney | and we have been asked looks at supprot keylime in the future | |
| 09:06:28 | gibi | and documented nothing | |
| 09:06:42 | gibi | regarding attestation | |
| 09:06:59 | sean-k-mooney | gibi: rgith btu we dotn supprot atested boot today | |
| 09:07:04 | sean-k-mooney | meaning you cant file a bug agasint nova if it does not work | |
| 09:07:13 | antia | this is not attested boot though | |
| 09:07:14 | gibi | sure, that is fine by me :) | |
| 09:07:44 | sean-k-mooney | as long as we dont have to fix bugs related to attestation for tdx im ok with it | |
| 09:08:16 | antia | yeah that is more or less why it was included in the documentation to remove any expectation of Nova doing attestations | |
| 09:08:21 | sean-k-mooney | antia: if the tdx attestation feature is not related to messered secure boot what is it | |
| 09:08:54 | sean-k-mooney | antia: well the docs dont say that its unsuprpoted right. can we just not allcoate teh socket? | |
| 09:09:24 | sean-k-mooney | so that thing we are not testing or cliaming to supprot is not enabled | |
| 09:10:42 | gibi | sean-k-mooney: I would let libvirt do the default that allows anybody who want to try a chance to make it work | |
| 09:11:11 | sean-k-mooney | so i dont htink these type of side channle shoud lbe aviabel to folks to hook up like that | |
| 09:11:28 | sean-k-mooney | like if live migration for exmaple get implemtne only when the socket is not used | |
| 09:11:48 | sean-k-mooney | we woudl not want ot have to chosoe betwen possiebl breakign folks that set it up and making live migration work | |