| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-09-18 | |||
| 12:17:34 | sean-k-mooney | for cold migration we do not need an api change | |
| 12:17:38 | sean-k-mooney | that only for rebuild | |
| 12:17:40 | nicolairuckel | ah | |
| 12:17:44 | sean-k-mooney | i misspoke before | |
| 12:18:02 | sean-k-mooney | for cold migrate we whosul always preseve the nvram | |
| 12:18:14 | nicolairuckel | The other reason was complexity. My patch turned out to be quite simple since we were able to use libvirt. | |
| 12:18:36 | nicolairuckel | Is this the problem for rebuild? https://bugs.launchpad.net/nova/+bug/2139077 | |
| 12:18:59 | sean-k-mooney | no for rebuild we have 2 usecases | |
| 12:19:21 | sean-k-mooney | some people use rebuild for backup and restore or to upgrade the software in teh vm | |
| 12:19:36 | sean-k-mooney | so for that cohort we want to preserve the nvram | |
| 12:19:53 | sean-k-mooney | but you can also use rebuild to change the image entrily and even disable secure boot | |
| 12:20:08 | sean-k-mooney | so for that second group clearing the nvram is more desireable | |
| 12:20:27 | sean-k-mooney | my prefernce woudl be to preserve by default and have an new api option to ask for it to be cleared | |
| 12:20:46 | nicolairuckel | So I guess we should split that up even further: one patch for the cold migration and one for rebuild? | |
| 12:21:04 | sean-k-mooney | https://bugs.launchpad.net/nova/+bug/2139077 is a third edgecase but its a less common one | |
| 12:21:21 | sean-k-mooney | yes i would keep it split | |
| 12:21:29 | sean-k-mooney | im not sure what the curret rebuild behivor is | |
| 12:22:04 | sean-k-mooney | shelve is also the final edgbecase we did not adress | |
| 12:22:17 | sean-k-mooney | which si it would be nice to save the nvram somewhere on shelve | |
| 12:22:29 | sean-k-mooney | we just didnt agree where | |
| 12:22:53 | sean-k-mooney | all of the edgecase for nvram also applies ot the tpm data too | |
| 12:23:27 | sean-k-mooney | for windows in partical preserving the nvram and loosing the tpm data on cold migrate will require bitlocker recovery | |
| 12:24:05 | sean-k-mooney | so if i was to work on this i would do cold-migrate/resize next for both nvram and tpm | |
| 13:13:47 | nicolairuckel | In one patch or separate patches? | |
| 13:13:56 | bauzas | gibi: hope your laptop is back, can I start to look at your patches ? | |
| 13:14:34 | nicolairuckel | I'm not sure if we need TPM | |
| 13:15:57 | gibi | bauzas: give me 5 to push what I have (some missing tests) | |
| 13:16:40 | bauzas | ack, I'll stop in ~1.5hour | |
| 13:21:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix PCI allocation leak on failed cold migration https://review.opendev.org/c/openstack/nova/+/1006003 | |
| 13:21:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Clean PCI claim of failed resize in periodic task https://review.opendev.org/c/openstack/nova/+/1006209 | |
| 13:22:09 | gibi | bauzas: so ^^ I have 3 patches | |
| 13:22:35 | gibi | 1. reproducer https://review.opendev.org/c/openstack/nova/+/1004658 | |
| 13:22:52 | gibi | 2. clean old leaks from periodics https://review.opendev.org/c/openstack/nova/+/1006209 | |
| 13:23:03 | gibi | 3. prevent new leaks https://review.opendev.org/c/openstack/nova/+/1006003 | |
| 13:23:06 | bauzas | ok | |
| 13:23:25 | gibi | the 3rd is RPC sensitve :) | |
| 13:23:46 | gibi | and I have to add unit test to the 2nd patch | |
| 13:36:17 | dansmith | gibi: I found a deadlock in my own stuff that was definitely related (but haven't solved yet) so that may have been my only issue, but other test workers are also hanging, so not sure | |
| 13:37:29 | dansmith | I guess it feels like maybe we should not wait there but fail if any tasks are left in some way instead of wait | |
| 13:40:20 | gibi | yeah that is a good idea to implement | |
| 15:55:38 | gmaan | yeah failing is also good, better than hanging. | |
| 16:00:12 | gmaan | gibi: RE:resize_instance: on source it will not hang the task as task on source resize_instance is tracked via RPC common wrapper and it will always end the task as soon RPC requsted is ended either by success or fail | |
| 16:00:28 | gmaan | gibi: but yes, event cleanup will be left in failure case | |
| 16:01:33 | gmaan | gibi: dansmith but on dest side we have case of task leakage same as live migration so what you think of it and we can backport it to stable/2026.2 https://review.opendev.org/c/openstack/nova/+/1006095 | |
| 20:09:45 | gmaan | dansmith: I think I am seeing it in my patch (https://review.opendev.org/c/openstack/nova/+/1004160) also where test are hanging due to wait on executors https://zuul.opendev.org/t/openstack/build/9263ab2ac8c244f3a81f94efd50ad1c7/log/job-output.txt | |
| 20:10:20 | gmaan | let me convert the wait to failure and see | |
| 21:22:35 | dansmith | gmaan: I need to go EOD soon but are you saying we can't make the functional test fail instead of wait because of the task leakage on live migration? | |
| 22:53:55 | gmaan | dansmith: we can, I am working on something to propose soon | |
| 22:54:37 | gmaan | dansmith: live migration task leakage is when we will bump the manager_shutdown_timeout which is 0 currently | |
| #openstack-nova - 2026-09-19 | |||
| 04:06:46 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Fail the test if threadpool executor hang during cleanup https://review.opendev.org/c/openstack/nova/+/1006351 | |
| #openstack-nova - 2026-09-21 | |||
| 09:19:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix PCI allocation leak on failed cold migration https://review.opendev.org/c/openstack/nova/+/1006003 | |
| 09:19:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Clean PCI claim of failed resize in periodic task https://review.opendev.org/c/openstack/nova/+/1006209 | |
| 09:19:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reproduce bug 2166786 https://review.opendev.org/c/openstack/nova/+/1004658 | |
| 09:19:50 | gibi | bauzas: fixed your comment. I would appreciate some oppinion about the 3rd patch touching RPC if that is considered an OK move from you prespective | |
| 09:20:03 | bauzas | gibi: on my radar shortly | |
| 09:20:11 | gibi | thanks | |
| 09:41:35 | opendevreview | ribaudr proposed openstack/nova master: Add regression test for bug #2120927 https://review.opendev.org/c/openstack/nova/+/991294 | |
| 09:41:36 | opendevreview | ribaudr proposed openstack/nova master: Fix shelve-offload/unshelve race wiping instance host https://review.opendev.org/c/openstack/nova/+/991295 | |
| 11:47:13 | gibi | bauzas: left comment on the BDM race https://review.opendev.org/c/openstack/nova/+/1005727/2#message-8b58cc364909d72a12803ec86a2e92a3d9841b71 | |
| 11:47:38 | gibi | I tink we can make it more safe and still backward compatible | |
| 11:50:50 | bauzas | gibi: ack will look when I have time :( | |
| 12:04:06 | gibi | bauzas: OK cool | |
| 15:26:33 | Uggla | gmaan, can you have a look at https://bugs.launchpad.net/nova/+bug/2164931, I have the feeling it can be closed or at least set to something else than 'new'. | |
| 15:33:17 | Uggla | Reminder upstream meeting in ~30mn | |
| 16:00:36 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:36 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:36 | opendevmeet | Meeting started Mon Sep 21 16:00:36 2026 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:36 | Uggla | #startmeeting nova | |
| 16:00:48 | Uggla | Hello everyone | |
| 16:00:50 | tkajinam | o/ | |
| 16:00:55 | lajoskatona | o/ | |
| 16:00:56 | elodilles | o/ | |
| 16:00:57 | sambork | o/ | |
| 16:00:59 | gibi | o/ | |
| 16:01:07 | dansmith | o/ | |
| 16:01:53 | Uggla | Let's go | |
| 16:01:56 | Uggla | #topic Bugs (stuck/critical) | |
| 16:02:00 | sean-k-mooney | o/ | |
| 16:02:04 | Uggla | #info No Critical bug | |
| 16:02:16 | Uggla | #topic Gate status | |
| 16:02:23 | Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:02:30 | Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | |
| 16:02:36 | Uggla | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&branch=stable%2F*&branch=master&pipeline=periodic-weekly&skip=0 Nova&Placement periodic jobs status | |
| 16:02:41 | Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | |
| 16:02:49 | Uggla | #info Please try to provide a meaningful comment when you recheck | |
| 16:02:54 | Uggla | #info gibi, gmann, I have refreshed 991295: Fix shelve-offload/unshelve race wiping instance host | https://review.opendev.org/c/openstack/nova/+/991295. (fix for some job instabilities) | |
| 16:02:55 | Uggla | #topic Release Planning | |
| 16:03:37 | gibi | Uggla: ack | |
| 16:03:47 | Uggla | note: tox-cover job still disabled. | |
| 16:04:00 | dansmith | not disabled, just not voting right/ | |
| 16:04:25 | Uggla | dansmith yes you are right | |
| 16:04:50 | Uggla | something to add about gate status? | |
| 16:05:43 | Uggla | seems not so moving on | |
| 16:05:45 | gibi | gmaan started looking into the hangs | |
| 16:05:58 | gibi | trying to translate the hanging results to clear failures | |
| 16:06:17 | gibi | https://review.opendev.org/c/openstack/nova/+/1006351 | |
| 16:06:25 | gibi | this is a good move | |
| 16:06:36 | Uggla | thx gibi | |
| 16:06:44 | Uggla | #topic Release Planning | |
| 16:06:51 | Uggla | #link https://releases.openstack.org/hibiscus/schedule.html | |