| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-09-29 | |||
| 14:51:09 | clarkb | `git review -d changenumber` make edits then `git review` to push back | |
| 16:48:48 | bauzas | dansmith: given we have yet another issue with the ImageMetaWeigher https://bugs.launchpad.net/nova/+bug/2125935 I wonder why I can't just target a cell context for fill_metadata this way : | |
| 16:49:03 | bauzas | insts.fill_metadata() | |
| 16:49:03 | bauzas | with nova_context.target_cell(ctxt, cell_mapping): | |
| 16:49:24 | bauzas | I guess I need to change the object context itself, right? | |
| 16:49:25 | dansmith | well, that's targeting a context but you're not using it | |
| 16:49:57 | dansmith | all the instances are from the same cell right? If so, you need to target the context, then pass that to InstanceList.__init__() and then fill metadata | |
| 16:50:19 | dansmith | the usage is "with target_cell(...) as child_ctx: " | |
| 16:52:00 | bauzas | yeah, given we look at all the instances from the same host, they are on the same context | |
| 16:52:07 | bauzas | s/context/cell | |
| 16:52:54 | bauzas | gotcha, I got that | |
| 16:53:00 | dansmith | cool, yeah then just need to target the one you give to the InstanceList | |
| 16:53:00 | bauzas | thanks | |
| 17:00:41 | bauzas | insts.fill_metadata() | |
| 17:00:41 | bauzas | objects=host_state.instances.values()) | |
| 17:00:41 | bauzas | insts = objects.InstanceList(cell_ctxt, | |
| 17:00:41 | bauzas | with nova_context.target_cell(ctxt, cell_mapping) as cell_ctxt: | |
| 17:00:51 | bauzas | that thing doesn't give me the metadata yet | |
| 17:14:26 | dansmith | do the fake instances have sysmeta set on the object to an empty dict? | |
| 17:21:16 | bauzas | yup | |
| 17:27:19 | bauzas | I'm lost, I need to get some fresh air, ending my long day | |
| 17:35:27 | dansmith | if they do then the fill will just skip them all, so need to drop those from the object to make it work | |
| 19:47:27 | opendevreview | melanie witt proposed openstack/nova master: FUP for vTPM live migration https://review.opendev.org/c/openstack/nova/+/962309 | |
| 19:47:28 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| 19:47:28 | opendevreview | melanie witt proposed openstack/nova master: Opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| #openstack-nova - 2025-09-30 | |||
| 05:34:08 | jssfr | dansmith, thanks for the review! | |
| 06:02:00 | opendevreview | Rajesh Tailor proposed openstack/nova master: Fixing the 500 HTTP code in the metadata service for incorrect queries https://review.opendev.org/c/openstack/nova/+/914249 | |
| 09:20:17 | opendevreview | Taketani Ryo proposed openstack/nova-specs master: Add a spec for 2026.1 for libvirt launching Arm CCA instances https://review.opendev.org/c/openstack/nova-specs/+/960777 | |
| 10:52:45 | opendevreview | Taketani Ryo proposed openstack/nova-specs master: Adjust the SEV/SEV-ES code for multiple cc support https://review.opendev.org/c/openstack/nova-specs/+/962578 | |
| 10:58:06 | aravindh | hello folks, I see some specs for detaching boot cinder volumes from a VM, but most of them are abandoned. What is the offical stance on this feature? We had some usecase where we had to restore boot disks from snapshots. I want to understand what the current consenses about this feature in the community. | |
| 14:22:57 | bauzas | dansmith: back to our discussion yesterday, I found the bug (this was in fill_metadata). I'll provide a functional test for testing the weigher in the fix change | |
| 14:23:18 | bauzas | hopefully tomorrow given all the today meetings | |
| 14:23:32 | dansmith | bauzas: you said that the fake instances had blank sysmeta in them, which will (intentionally) be skipped by fill_metadata | |
| 14:23:41 | dansmith | that's not a bug, if that's what you're describing | |
| 14:25:15 | bauzas | dansmith: https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L1608 there we need to pass the dict value | |
| 14:25:37 | bauzas | ie.inst.system_metadata = utils.instance_sys_meta(updated[inst.uuid) | |
| 14:25:55 | bauzas | anyway, working now on a functional test | |
| 14:29:58 | dansmith | bauzas: um | |
| 14:30:42 | dansmith | if that's wrong, seems like the image props weigher can't be working? | |
| 14:31:13 | bauzas | yeah, that's why I'm creating a functional test | |
| 14:31:30 | bauzas | we were only having tests that were checking that the weigher was running | |
| 14:31:38 | bauzas | and we were having unittests for checking the results | |
| 14:32:20 | dansmith | well, I was definitely testing that locally, but perhaps a refactor before merge broke it.. it was QAd downstream though, I'm pretty sure | |
| 14:33:56 | bauzas | honestly, that's my fault here | |
| 14:34:00 | bauzas | the whole one | |
| 14:34:51 | bauzas | I was thinking this was simple to create a weigher but the fact is that this specific weigher needs to get some values not from the host state, and we didn't really tested it, just myself by testing once | |
| 14:36:24 | dansmith | the unit test for fill_metadata uses the db and seems solid to me, but I'll wait for your patch | |
| 15:06:29 | opendevreview | Balazs Gibizer proposed openstack/placement master: Reproduce GET a_c slowness https://review.opendev.org/c/openstack/placement/+/962596 | |
| 15:31:06 | Uggla | Nova upstream meeting in ~30mn | |
| 16:01:57 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:01:57 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:57 | opendevmeet | Meeting started Tue Sep 30 16:01:57 2025 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:01:57 | Uggla | #startmeeting nova | |
| 16:01:57 | bauzas | tic toc | |
| 16:02:17 | Uggla | Hello everyone | |
| 16:02:20 | bauzas | \o | |
| 16:02:28 | r-taketn | Hello | |
| 16:02:43 | elodilles | o/ | |
| 16:03:01 | tkajinam | o/ | |
| 16:03:07 | Uggla | awaiting a couple of minutes, so people can join | |
| 16:03:28 | Uggla | I know some of them are already in another meeting | |
| 16:03:49 | gibi_ | o/ | |
| 16:04:07 | Uggla | Let's start | |
| 16:04:16 | Uggla | #topic Bugs (stuck/critical) | |
| 16:04:22 | Uggla | #info No Critical bug | |
| 16:04:36 | Uggla | #topic Gate status | |
| 16:04:44 | Uggla | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:04:48 | fwiesel | o/ | |
| 16:04:56 | Uggla | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | |
| 16:05:02 | 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:05:09 | Uggla | #info Please look at the gate failures and file a bug report with the gate-failure tag. | |
| 16:05:17 | Uggla | #info Please try to provide a meaningful comment when you recheck | |
| 16:05:42 | Uggla | #info I have seen (last week) "nova-tox-py312-threading fails intermittently" https://bugs.launchpad.net/nova/+bug/2125185 from gibi | |
| 16:06:04 | Uggla | gibi is it something you want to discuss here or later in the eventlet part ? | |
| 16:06:11 | gibi_ | I will recheck the fix there as it failed on the gate | |
| 16:06:15 | gibi_ | 961781 | |
| 16:06:19 | gibi_ | https://review.opendev.org/c/openstack/nova/+/961781 | |
| 16:06:37 | gibi_ | my point here is just if you see nova-tox-py312-threading failing on your patch | |
| 16:06:43 | gibi_ | but the other py312 job does not | |
| 16:06:47 | gibi_ | then please let me know | |
| 16:07:07 | gibi_ | I monitor the nova-tox-py312-threading job but I might miss things | |
| 16:07:07 | Uggla | 👍 | |
| 16:07:13 | gibi_ | that is it here | |
| 16:07:51 | Uggla | Anything else to discuss about the gate ? I have not seen anything special but... ? | |
| 16:08:20 | gibi | maybe one | |
| 16:08:57 | gibi | if you see strange failures in nova-next regarding scheduling then let me no too :) nova-next uses threading mode for n-sch n-api n-metadata | |
| 16:09:11 | gibi | /let me no/ let me know/ | |
| 16:09:38 | gibi | I spotted some strange failurs there and even filed a bug | |
| 16:09:49 | gibi | but I lost the link :) | |
| 16:10:18 | gibi | https://bugs.launchpad.net/nova/+bug/2125598 | |
| 16:10:20 | gibi | here it is | |
| 16:10:28 | gibi | I need to get back to this and look deeper | |
| 16:10:47 | gibi | that is it really for me now :) | |
| 16:11:05 | Uggla | ok* | |
| 16:11:13 | Uggla | Moving on to next topic skipping gmann's one due to PTO. | |
| 16:11:23 | Uggla | #topic Release Planning | |
| 16:11:31 | Uggla | #link https://releases.openstack.org/flamingo/schedule.html | |
| 16:11:38 | Uggla | #info Nova deadlines are set in the above schedule | |
| 16:11:49 | Uggla | #info release in progress. | |
| 16:11:57 | Uggla | #info PTG etherpad for 2026.1 is available: https://etherpad.opendev.org/p/nova-2026.1-ptg | |