| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-10-14 | |||
| 16:42:06 | gibi | then first couple of patches are reviewable on https://review.opendev.org/c/openstack/nova/+/956089/ | |
| 16:42:16 | gibi | and the ptg etherpad as the longer term plan | |
| 16:42:53 | gibi | Also please report any strange test failures from nova-next, since last week we run conduct there in threading mode as well | |
| 16:43:08 | gibi | (top of scheduler api, and metadata) | |
| 16:43:13 | gibi | that is ti | |
| 16:43:15 | gibi | it | |
| 16:43:18 | sean-k-mooney | hum on the rbd thread pool i guess everually we will wnat to use the asinc api instead | |
| 16:43:30 | sean-k-mooney | so we can actully pass timeouts ectra | |
| 16:44:17 | sean-k-mooney | but i guess the intial plan is to just not swapn it on a tread pool at all which i guess is ok | |
| 16:44:19 | gibi | sean-k-mooney: this will not prevent doing that later when we actaully go and translate the nova-compute | |
| 16:44:44 | sean-k-mooney | ack ya just chekcign that still the medium term plan | |
| 16:44:46 | gibi | the patches on that series are for unblocking unit testing | |
| 16:44:47 | opendevreview | Merged openstack/nova master: Move cleanup of vTPM secret from driver to compute https://review.opendev.org/c/openstack/nova/+/962007 | |
| 16:45:17 | sean-k-mooney | ah ok | |
| 16:47:03 | gibi | I think we can move on | |
| 16:47:11 | Uggla | thx gibi | |
| 16:47:20 | Uggla | #topic Open discussion | |
| 16:47:45 | Uggla | Just a reminder for the poll: Pool for the new schedule of upstream meeting: https://framadate.org/ao12zK6wR3K3nfbF | |
| 16:48:10 | Uggla | Anything you want to discuss ? | |
| 16:48:25 | nicolairuckel | I worked on a patch to preserve the NVRAM after reboots that got mentioned a few weeks ago. | |
| 16:48:39 | nicolairuckel | I'm not sure if that's the correct place to talk about that. | |
| 16:48:47 | Uggla | yes | |
| 16:48:54 | nicolairuckel | https://review.opendev.org/c/openstack/nova/+/959682 | |
| 16:49:29 | sean-k-mooney | looks like it needs a rebase | |
| 16:49:59 | nicolairuckel | The only caveat is that at the moment you have to set the permissions for the NVRAM file manually. I wasn't able to figure out if there's a way to change the permissions through libvirt. | |
| 16:50:08 | nicolairuckel | sean-k-mooney, I'll do that! | |
| 16:50:35 | sean-k-mooney | why do you need to do that? | |
| 16:51:27 | nicolairuckel | If you don't add read permissions, there will be a permissions error on a reboot and it won't be preserved. | |
| 16:52:04 | sean-k-mooney | wait why are you copying the file to the instnce dir an back | |
| 16:52:04 | nicolairuckel | The libvirt people told me that the permissions are like than intentionally. It's just a bit weird to me that the disk file has read permissions. | |
| 16:52:23 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/959682/3/nova/virt/libvirt/driver.py#10919 | |
| 16:52:32 | nicolairuckel | The file contains the UEFI variables that I want to preserve. | |
| 16:52:42 | sean-k-mooney | i tought we were goign to preseve this by passign the libvirt flag to the doamin cleanup to preserve the file | |
| 16:53:36 | nicolairuckel | You mean I could skip copying the file? | |
| 16:53:42 | sean-k-mooney | yes | |
| 16:53:58 | nicolairuckel | I see. I'm going to try that. | |
| 16:53:59 | sean-k-mooney | so for the vtpm data we can pass a flag to tell libvirt not to delete it when the domein is undefiend | |
| 16:54:05 | sean-k-mooney | i belive we can do the same for the nvram | |
| 16:54:19 | sean-k-mooney | i could be mistaken but i belive there was a flag for the nvram as well | |
| 16:54:24 | nicolairuckel | That would be a better solution for sure. | |
| 16:54:26 | sean-k-mooney | so we would not need ot copy the file | |
| 16:54:37 | nicolairuckel | I'm going to look into that, thank you. | |
| 16:55:12 | nicolairuckel | How can I find a reviewer when that is done? Should I just bring it up again next week? | |
| 16:56:47 | sean-k-mooney | you can also ping on irc | |
| 16:56:52 | nicolairuckel | okay | |
| 16:57:01 | sean-k-mooney | we will proably see it updated but you can reach out if we dont | |
| 16:57:03 | Uggla | nicolairuckel, ping on the channel | |
| 16:57:16 | nicolairuckel | thanks | |
| 16:57:23 | nicolairuckel | that would be all from my side then | |
| 16:57:56 | gibi | nicolairuckel: thanks for working on it | |
| 16:58:24 | sean-k-mooney | VIR_DOMAIN_UNDEFINE_KEEP_NVRAM | |
| 16:58:39 | sean-k-mooney | https://github.com/libvirt/libvirt/blob/b42a12174c787b99cd6fcb29b44e4b13bd64ee58/include/libvirt/libvirt-domain.h#L2373 | |
| 16:58:52 | sean-k-mooney | that the flag we need to pass to not delete it when the domain is undeifed | |
| 16:58:57 | Uggla | nicolairuckel, yes thanks for working on it | |
| 17:00:14 | Uggla | time to close | |
| 17:00:32 | Uggla | Thank you for participating in this meeting. | |
| 17:00:39 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-10-14-16.01.log.html | |
| 17:00:39 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-10-14-16.01.txt | |
| 17:00:39 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-10-14-16.01.html | |
| 17:00:39 | opendevmeet | Meeting ended Tue Oct 14 17:00:39 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 17:00:39 | Uggla | #endmeeting | |
| 17:01:29 | elodilles | thanks o/ | |
| 17:03:27 | gibi | hanks | |
| 17:03:29 | gibi | thanks | |
| 17:03:30 | gibi | :) | |
| 17:06:58 | elodilles | tom? o:) | |
| 17:10:33 | gibi | nemtom (sorry this only funny in hungarian) | |
| 17:18:07 | sean-k-mooney | by the way im on pto retruning on tusday | |
| 17:18:28 | sean-k-mooney | just an fyi that i wont be here for the rest of the week | |
| 17:19:38 | gibi | sean-k-mooney: have a nice one | |
| 18:45:51 | opendevreview | Merged openstack/nova master: Switch nova-conductor to use global executor https://review.opendev.org/c/openstack/nova/+/962478 | |
| 20:09:24 | opendevreview | Henry Richter proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | |
| 22:01:54 | opendevreview | Fernando Ferraz proposed openstack/nova master: [DNM] Get processutils logs from unprivileged_convert_image https://review.opendev.org/c/openstack/nova/+/964095 | |
| 23:22:09 | opendevreview | melanie witt proposed openstack/nova master: Add handling for vTPM secret permission error https://review.opendev.org/c/openstack/nova/+/963648 | |
| #openstack-nova - 2025-10-15 | |||
| 02:06:31 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `user` secret security https://review.opendev.org/c/openstack/nova/+/942502 | |
| 02:06:32 | opendevreview | melanie witt proposed openstack/nova master: Add vtpm_secret_(uuid|value) to LibvirtLiveMigrateData https://review.opendev.org/c/openstack/nova/+/952628 | |
| 02:06:32 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `host` secret security https://review.opendev.org/c/openstack/nova/+/941795 | |
| 02:06:33 | opendevreview | melanie witt proposed openstack/nova master: TPM: bump service version and require it for live migration https://review.opendev.org/c/openstack/nova/+/962051 | |
| 02:06:33 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `host` secret security https://review.opendev.org/c/openstack/nova/+/941483 | |
| 02:06:34 | opendevreview | melanie witt proposed openstack/nova master: TPM: support live migration of `deployment` secret security https://review.opendev.org/c/openstack/nova/+/925771 | |
| 02:06:34 | opendevreview | melanie witt proposed openstack/nova master: TPM: support instances with `deployment` secret security https://review.opendev.org/c/openstack/nova/+/942021 | |
| 02:06:35 | opendevreview | melanie witt proposed openstack/nova master: TPM: test live migration between hosts with different security https://review.opendev.org/c/openstack/nova/+/952629 | |
| 02:06:36 | opendevreview | melanie witt proposed openstack/nova master: TPM: opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 02:06:36 | opendevreview | melanie witt proposed openstack/nova master: TPM: add late check for supported TPM secret security https://review.opendev.org/c/openstack/nova/+/956975 | |
| 02:06:38 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 02:06:38 | opendevreview | melanie witt proposed openstack/nova master: TPM: add documentation and reno for live migration https://review.opendev.org/c/openstack/nova/+/962889 | |
| 07:57:21 | opendevreview | Balazs Gibizer proposed openstack/placement master: Prune a_c search space by invalid prefixes https://review.opendev.org/c/openstack/placement/+/962776 | |
| 08:48:45 | opendevreview | Kamil Sambor proposed openstack/nova master: [hacking] Improve N373 to catch also other primitives https://review.opendev.org/c/openstack/nova/+/958408 | |
| 13:20:19 | dansmith | gibi: I think I can proxy sean's +2 on that since you just changed some english and added an assert to the test.. agree? | |
| 13:23:45 | gibi | dansmith: I agree. If you are confortable with the patch then I think Sean will not disagree approving it based on Sean's past review rounds | |
| 13:28:36 | gibi | dansmith: thank you for your continued effort on that patch I appreciate it. And sorry again that I forced you to re-review it. | |
| 14:33:33 | dansmith | gibi: eventlet sync? | |
| 14:36:41 | opendevreview | Merged openstack/placement master: Prune a_c search space by invalid prefixes https://review.opendev.org/c/openstack/placement/+/962776 | |
| 17:59:24 | gmaan | melwitt: +2 on the config option logging fix. I checked in logs (added links in gerrit comment) and it is working fine now https://review.opendev.org/c/openstack/nova/+/963619 | |
| 17:59:49 | gmaan | I added the log flow in comment so that it will be quick for 2nd reviewer to check the result | |
| 17:59:58 | melwitt | gmaan: hey thanks :D /me was not expecting reviews so quick | |
| 19:55:01 | Zhan[m] | `_rollback_live_migration` function cannot properly rollback the changes to port bindings. Does it make sense for me to write some code in the rollback function to basically recreate and reactivate the port bindings on the source side if needed, so that we can have the previous change back (start port binding earlier)? thanks! | |
| 19:55:01 | Zhan[m] | hey team, just wanna ask for feedbacks on some ideas on live migrations (using libvirt and QEMU): I saw in this [commit](https://review.opendev.org/c/openstack/nova/+/434870) that we'll be activating port bindings when the VM is paused on the source side, but this change was reverted in this [commit](https://github.com/openstack/nova/commit/aa87b9c288d316b85079e681e0df24354ec1912c) because if the migration fails, the | |
| 21:43:17 | gmaan | bauzas: gibi: I added some comment in ImagePropertiesWeigher fix, nothing blocker so feel free to +w if those does not need updates or reply and I will +w tomorrow https://review.opendev.org/c/openstack/nova/+/963358 | |
| #openstack-nova - 2025-10-16 | |||
| 07:52:00 | jlejeune | hello bauzas, sean-k-mooney[m], gibi and others :) I would like to backport my changes related to these topics: https://review.opendev.org/q/topic:%22bug-2044235%22 and https://review.opendev.org/q/topic:%22bug/2085135%22, what do you think about them ? thanks | |