| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-09-29 | |||
| 14:41:02 | sean-k-mooney | unless thats also in the journal dump? | |
| 14:41:56 | gibi | we have journal logs from both runs, and I do an extra service restart now at the end to get the resource usage after the second run as well. Then I pull the data out from the journal into eg https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_eb1/openstack/eb1c16166a9e42b5be0987f5d8ca0d01/results/resource-usage-devstack%40n-sch.service.txt | |
| 14:43:23 | sean-k-mooney | ah cool | |
| 14:46:00 | jssfr | dansmith (and maybe others following https://review.opendev.org/c/openstack/nova/+/955657 ): I'm going to be AFK starting october 1st until november 25th. Is there anything else I need to do before that to get it over the line? | |
| 14:47:31 | dansmith | jssfr: sorry, just been really busy with other stuff.. I think it mostly looks good to me now but I was just going to do some tests locally that I haven't had a chance to do yet | |
| 14:47:45 | dansmith | but others may feel more comfortable just looking at it | |
| 14:48:25 | jssfr | related question: is it possible to take over another person's gerrit patch? | |
| 14:48:34 | jssfr | I might find a substitute who can continue to look into it while I'm away. | |
| 14:48:56 | dansmith | yep, easy and common | |
| 14:50:05 | jssfr | is there anything that needs to be taken into account when doing that? or can another gerrit user just take my patch and run `git review`? | |
| 14:50:36 | opendevreview | Merged openstack/nova stable/2025.1: Add admin context when filling metadata on ImagePropsWeigher https://review.opendev.org/c/openstack/nova/+/961806 | |
| 14:50:40 | dansmith | no, anyone can git review it right now | |
| 14:50:59 | jssfr | thanks | |
| 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 | with nova_context.target_cell(ctxt, cell_mapping): | |
| 16:49:03 | bauzas | insts.fill_metadata() | |
| 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 | bauzas | thanks | |
| 16:53:00 | dansmith | cool, yeah then just need to target the one you give to the InstanceList | |
| 17:00:41 | bauzas | with nova_context.target_cell(ctxt, cell_mapping) as cell_ctxt: | |
| 17:00:41 | bauzas | insts = objects.InstanceList(cell_ctxt, | |
| 17:00:41 | bauzas | objects=host_state.instances.values()) | |
| 17:00:41 | bauzas | insts.fill_metadata() | |
| 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: Opt-in to new TPM secret security via resize https://review.opendev.org/c/openstack/nova/+/962052 | |
| 19:47:28 | opendevreview | melanie witt proposed openstack/nova master: DNM vtpm tempest https://review.opendev.org/c/openstack/nova/+/957477 | |
| #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 | bauzas | tic toc | |
| 16:01:57 | Uggla | #startmeeting nova | |
| 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 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:57 | opendevmeet | The meeting name has been set to 'nova' | |
| 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 | Uggla | 👍 | |
| 16:07:07 | gibi_ | I monitor the nova-tox-py312-threading job but I might miss things | |
| 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/ | |