| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-08-08 | |||
| 10:03:23 | jkulik | hi folks, one of my colleagues found a bug in the code ^ do you also need a bug-report in launchpad? imho this is a quite obvious one :) | |
| 10:39:10 | frickler | jkulik: since the bug appears to be 4 years old, I think a bug report would be better to allow easier backporting to stable branches, even though the fix looks obvious | |
| 10:39:22 | jkulik | ok | |
| 10:43:33 | opendevreview | Johannes Kulik proposed openstack/nova master: db: Fix api_db_opts reading main_db_opts https://review.opendev.org/c/openstack/nova/+/956869 | |
| 10:48:16 | tkajinam | Can https://review.opendev.org/c/openstack/placement/+/953367 get some attention. it's just one of the hosekeeping updates. | |
| 14:12:29 | opendevreview | Stefan Hoffmann proposed openstack/nova master: POC: setup tap interface by nova https://review.opendev.org/c/openstack/nova/+/956907 | |
| 14:14:53 | opendevreview | Stefan Hoffmann proposed openstack/nova master: POC: setup tap interface by nova https://review.opendev.org/c/openstack/nova/+/956907 | |
| 14:40:15 | sthoffmann[m] | Like discussed at the last PTG, I provided a POC for letting neutron wait till ovn-controller on destination host has the change and let therefore nova create the tap interface.... (full message at <https://matrix.org/oftc/media/v1/media/download/AcK7tVlei75Wjt7vLary51xZAItVJ4c5v-eSjBPuJhQN-w4LXaTO8jzvgi_S84-s7GhXW08xw4TvM1xnZCT5SnxCeY0jkb2AAG1hdHJpeC5vcmcvb0NZaVRvTWhJTWpCWnpEaGtnRG1ITWt0>) | |
| 15:21:38 | sean-k-mooney | sthoffmann[m]: if we are goign to create the tap atead of tiem weo would do that in os-vif | |
| 15:22:22 | sean-k-mooney | we would jsut have nova pass a flag to tell os-vif to do it | |
| 16:51:38 | opendevreview | Merged openstack/nova master: Fix image ID in libvirt metadata when unshelving https://review.opendev.org/c/openstack/nova/+/942973 | |
| 17:13:57 | opendevreview | Pavlo Shchelokovskyy proposed openstack/nova master: Remove max config limit for libvirt.num_pcie_ports https://review.opendev.org/c/openstack/nova/+/956942 | |
| 19:20:04 | opendevreview | Ghanshyam proposed openstack/nova master: Disable server reset state API by default https://review.opendev.org/c/openstack/nova/+/956959 | |
| 20:35:24 | opendevreview | Ghanshyam proposed openstack/nova master: Disable server reset state API by default https://review.opendev.org/c/openstack/nova/+/956959 | |
| #openstack-nova - 2025-08-09 | |||
| 01:15:35 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502 | |
| 01:15:35 | opendevreview | melanie witt proposed openstack/nova master: TPM: migrate legacy instances to new security policy https://review.opendev.org/c/openstack/nova/+/942501 | |
| 01:15:36 | opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | |
| 01:15:36 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795 | |
| 01:15:37 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 01:15:37 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 01:15:38 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 01:15:38 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 01:15:39 | opendevreview | melanie witt proposed openstack/nova master: TPM: update instance request_spec with secret security https://review.opendev.org/c/openstack/nova/+/952630 | |
| 01:15:40 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 01:15:40 | opendevreview | melanie witt proposed openstack/nova master: TPM: confirm secret security via hard reboot https://review.opendev.org/c/openstack/nova/+/955847 | |
| 01:21:38 | opendevreview | melanie witt proposed openstack/nova master: Add hw_tpm_secret_security image property https://review.opendev.org/c/openstack/nova/+/940196 | |
| 01:21:39 | opendevreview | melanie witt proposed openstack/nova master: TPM: migrate legacy instances to new security policy https://review.opendev.org/c/openstack/nova/+/942501 | |
| 01:21:39 | opendevreview | melanie witt proposed openstack/nova master: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197 | |
| 01:21:40 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795 | |
| 01:21:40 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502 | |
| 01:21:41 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 01:21:41 | opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | |
| 01:21:42 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 01:21:42 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 01:21:44 | opendevreview | melanie witt proposed openstack/nova master: TPM: update instance request_spec with secret security https://review.opendev.org/c/openstack/nova/+/952630 | |
| 01:21:44 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 01:21:46 | opendevreview | melanie witt proposed openstack/nova master: TPM: confirm secret security via hard reboot https://review.opendev.org/c/openstack/nova/+/955847 | |
| 01:21:46 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 08:15:38 | opendevreview | Takashi Kajinami proposed openstack/nova master: Drop unused method https://review.opendev.org/c/openstack/nova/+/927680 | |
| 08:16:12 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Remove tpm supoort detection for libvirt < 8.0.0 https://review.opendev.org/c/openstack/nova/+/952308 | |
| 08:17:20 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove logic for unsupported old libvirt/qemu https://review.opendev.org/c/openstack/nova/+/952318 | |
| 08:31:52 | opendevreview | Merged openstack/nova master: Add more flavor metadata to libvirt guest XML https://review.opendev.org/c/openstack/nova/+/942974 | |
| 08:47:45 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 08:47:45 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate MEM_ENCRYPTION_CONTEXT from root provider https://review.opendev.org/c/openstack/nova/+/921814 | |
| 08:47:46 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 08:47:46 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | |
| 12:46:17 | opendevreview | Merged openstack/nova master: Add image meta to libvirt XML metadata https://review.opendev.org/c/openstack/nova/+/942766 | |
| 12:57:32 | opendevreview | Takashi Kajinami proposed openstack/nova master: Detect AMD SEV-ES support https://review.opendev.org/c/openstack/nova/+/925685 | |
| 12:57:50 | opendevreview | Takashi Kajinami proposed openstack/nova master: Add hw_mem_encryption_model image property https://review.opendev.org/c/openstack/nova/+/927706 | |
| 12:58:00 | opendevreview | Takashi Kajinami proposed openstack/nova master: libvirt: Launch instances with SEV-ES memory encryption https://review.opendev.org/c/openstack/nova/+/926106 | |
| 21:11:01 | opendevreview | Callum Dickinson proposed openstack/nova master: Fix libvirt metadata upgrade release note https://review.opendev.org/c/openstack/nova/+/956988 | |
| 21:49:52 | opendevreview | Callum Dickinson proposed openstack/nova master: Fix libvirt metadata upgrade release note https://review.opendev.org/c/openstack/nova/+/956988 | |
| #openstack-nova - 2025-08-10 | |||
| 16:32:06 | opendevreview | Masanori Kuroha proposed openstack/nova master: Copy applied provider config https://review.opendev.org/c/openstack/nova/+/948304 | |
| #openstack-nova - 2025-08-11 | |||
| 04:47:59 | jayaanand | Hi All, good morning. i am planning to co-author and own patch https://review.opendev.org/c/openstack/nova/+/680648 as part of our NFS extend feature. I will address any pending review comments. please complete this review. we need this change for This relase | |
| 05:26:17 | jssfr | jayaanand, what do you mean by "NFS extend feature"? | |
| 06:44:26 | jayaanand | i am trying to fix BUG https://bugs.launchpad.net/cinder/+bug/1878913 from ONTAP | |
| 06:45:08 | jayaanand | to improve ERROR reporting. we need fix https://review.opendev.org/c/openstack/nova/+/680648 from NOVA side | |
| 07:03:21 | jssfr | uhuh | |
| 07:59:41 | sthoffmann[m] | Thanks for the feedback. Should I add this to the POC or will we include it to the spec for later implementation? | |
| 08:36:59 | jssfr | jayaanand, we had submitted specs for that feature: https://review.opendev.org/c/openstack/nova-specs/+/949504 and https://review.opendev.org/c/openstack/cinder-specs/+/866718 but we didn't get around to re-submit them for Flamingo. | |
| 08:37:06 | jssfr | we plan on resuming that work in the G cycle | |
| 08:40:59 | jssfr | *get it merged for Flamingo | |
| 09:44:49 | sean-k-mooney | Uggla: https://blueprints.launchpad.net/nova/+spec/xml-image-meta is now merged if you want to mark it as implemnted | |
| 09:53:52 | jayaanand | can we consider https://bugs.launchpad.net/cinder/+bug/1878913 and https://review.opendev.org/c/openstack/nova/+/680648 as stop gap arrangement. right now, customer is not able extend NFS volumes attached to Nova VM | |
| 09:59:38 | sean-k-mooney | jayaanand: if i recall correctly the spec for that was nto accpated this cycle but ill check | |
| 10:00:16 | sean-k-mooney | https://review.opendev.org/c/openstack/nova-specs/+/949504 | |
| 10:00:24 | sean-k-mooney | jayaanand: ^ | |
| 10:01:14 | sean-k-mooney | jayaanand: extended for nfs i think is techinally not supproted today | |
| 10:02:28 | sean-k-mooney | """ | |
| 10:02:30 | sean-k-mooney | Many remotefs-based volume drivers in Cinder use the ``qemu-img resize`` | |
| 10:02:32 | sean-k-mooney | command to extend volume files. | |
| 10:02:34 | sean-k-mooney | However, when the volume is attached to a guest, QEMU will lock the file and | |
| 10:02:36 | sean-k-mooney | ``qemu-img`` will be unable to resize it. | |
| 10:02:38 | sean-k-mooney | In this case, only the QEMU process holding the lock can resize the volume, | |
| 10:02:40 | sean-k-mooney | which can be triggered through the QEMU monitor command ``block-resize``. | |
| 10:02:42 | sean-k-mooney | There is currently no adequate way for Cinder to use this feature, so the NFS, | |
| 10:02:44 | sean-k-mooney | NetApp NFS, Powerstore NFS, and Quobyte volume drivers all disable extending | |
| 10:02:46 | sean-k-mooney | attached volumes.""" | |
| 10:02:48 | sean-k-mooney | from teh spec | |
| 10:03:38 | sean-k-mooney | jayaanand: we dont consdier that to be bug becasue it has never work and there is a proposal to add it as a feature the problem is that the folks workign on it have done so very intermitently | |
| 10:04:44 | sean-k-mooney | jayaanand: https://review.opendev.org/c/openstack/nova/+/873560 that is the poc | |
| 10:05:15 | sean-k-mooney | jayaanand: part of the problem is the cider side is not done yet https://review.opendev.org/q/topic:%22bp/extend-volume-completion-action%22 | |
| 10:06:13 | sean-k-mooney | although part of it merged in epocy i belive so we may be at the poitn where the nova side can progress but that will be 2026.1 at this rate | |
| 10:08:56 | jayaanand | we tested https://review.opendev.org/c/openstack/nova/+/680648 with customer NFS volumes attached to Nova VM. except encrypted volumes other scenarios NFS volumes expanded. This is ok with most of the customers. Without this changes. user need expand volume in NetApp filer and reboot VM to reflect size change. | |
| 10:09:14 | kgube | sean-k-mooney, the cinder dependency is actually merged | |
| 10:09:36 | jayaanand | this changes https://review.opendev.org/c/openstack/nova/+/680648 has extra error scenarios no logic changers will help customer to get information about errors. | |
| 10:09:59 | kgube | sean-k-mooney, at least the first part, that the nova changes depend on | |
| 10:10:30 | sean-k-mooney | kgube: the new api exits but is not hooked up right | |
| 10:10:51 | sean-k-mooney | i.e. cinder is not actully waiting for the completion callback correct? | |
| 10:11:27 | kgube | well, it has to be enabled in the drivers, but that depends on nova support again | |
| 10:12:28 | sean-k-mooney | jayaanand: we can consdier improving the error handeling but from a nova perspective extending an nfs voluem wont be supproted until kgube's feateure is complete | |
| 10:12:54 | kgube | sean-k-mooney, it's a cross-dependency, so we have to do a sort of a ping-pong of merges | |
| 10:13:20 | kgube | currently the ball is on the nova side | |
| 10:13:20 | sean-k-mooney | kgube: ack, i assume your going to reporpose the nova sepc for 2026.1 | |
| 10:13:30 | kgube | yes | |
| 10:14:05 | jayaanand | thank you! sean-k-mooney for considering error handling. is it possible to merge https://review.opendev.org/c/openstack/nova/+/680648 for this release. | |
| 10:14:17 | sean-k-mooney | kgube: have the cidner folks reviewd the cinder parts? if we are tat the critical poing we proably need to corrdiate the revew on both side to get it finally landed | |
| 10:16:42 | sean-k-mooney | jayaanand: i have not looked at it in detail but maybe. it likely will need a release note and moer testing. ideally a functional test to repoduce the incorrect event state in a preceedign patch then this patch woudl update the functional test to show that the instance action event was properly set to error | |
| 10:17:15 | sean-k-mooney | the curret unit test does not actully show it fixes the reporte bug in reporting | |
| 10:23:13 | kgube | sean-k-mooney, cinder folks have merged the initial patch: https://review.opendev.org/c/openstack/cinder-specs/+/866718, the next cinder patch is a minor change that will depend on a new nova microversion to enable the new behavior: https://review.opendev.org/c/openstack/cinder/+/891602 | |