| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-09-28 | |||
| 15:29:44 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 16:13:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 17:06:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 18:38:30 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 19:28:35 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| #openstack-nova - 2025-09-29 | |||
| 08:20:20 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 08:46:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 09:43:31 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 10:00:07 | opendevreview | Jonas Schäfer proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | |
| 10:11:49 | opendevreview | Kamil Sambor proposed openstack/nova master: Switch nova-conductor to use global executor https://review.opendev.org/c/openstack/nova/+/962478 | |
| 11:12:10 | opendevreview | Jonas Schäfer proposed openstack/nova master: Preserve vTPM state between power off and power on https://review.opendev.org/c/openstack/nova/+/955657 | |
| 11:28:28 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rally job for eventlet-removal https://review.opendev.org/c/openstack/nova/+/960130 | |
| 12:14:16 | gibi | dansmith: fyi we can catch threading slowdowns now with rally https://review.opendev.org/c/openstack/nova/+/960130/30#message-1f12ccedc335aa0e59d469a478012a41d76fd67c | |
| 12:14:33 | gibi | (this was a simulated slowdown by injecting sleep to nova) | |
| 12:40:42 | opendevreview | Kamil Sambor proposed openstack/nova master: Switch nova-conductor to use global executor https://review.opendev.org/c/openstack/nova/+/962478 | |
| 13:09:06 | opendevreview | Merged openstack/nova stable/2025.1: Regression test for ImagePropsWeigher due to missing context https://review.opendev.org/c/openstack/nova/+/961805 | |
| 13:58:58 | dansmith | gibi: nice.. hopefully that will be stable enough to be useful.. even within a single worker performance has varied a lot in the past (due to noisy neighbors) but even if we get a smoke signal that will be good | |
| 14:04:47 | gibi | if the performance of the worker fluctuates in a ~10 mintues window then yeah we will get false postive signals | |
| 14:05:20 | gibi | the window size depends one length of our rally scenarios | |
| 14:17:39 | sean-k-mooney-pto | while we could have noise neighber effects i think its a lease worth trying this for a while and see if it becomes a problem | |
| 14:18:15 | sean-k-mooney-pto | gibi: presumable the rally job wont be voting at least not in the short term | |
| 14:18:32 | gibi | won't be voting for a while | |
| 14:18:35 | sean-k-mooney-pto | so if it a perodic or its non voting | |
| 14:18:43 | sean-k-mooney-pto | i think this ok for now | |
| 14:19:19 | sean-k-mooney-pto | gibi: do you think it need to run in check longterm or would you jsut rung this in weekly perodic? | |
| 14:23:29 | gibi | right now it is a cheap job so I would run it non-voting. But if we add more scenarios (I planning to ) then it might be expensive enough to run it only periodically | |
| 14:23:55 | gibi | or if we move it to a label with many subnodes | |
| 14:24:11 | sean-k-mooney-pto | ack | |
| 14:24:47 | sean-k-mooney-pto | this is the kind fo job where it would be nice if there as a persitent mysql db where we coudl log some data across runs | |
| 14:25:12 | sean-k-mooney-pto | i.e. to track over all trends across time | |
| 14:26:24 | sean-k-mooney-pto | but ya i was thining the same start we non voting and hten move to perodic + experimental if its becomes expensive | |
| 14:27:12 | sean-k-mooney-pto | havign it in experminetal so you can spot check specific changes is ueful if we go the perodic route | |
| 14:29:45 | sean-k-mooney | gibi: the last day before i went on pto i spend a little time tryign to see if we could get any better infro form atop or the logs on overall load. i made some progress but the binary format and way that tool works is a littel painful | |
| 14:40:14 | gibi | sean-k-mooney: I actually noticed that when a service is stopped systemd already nicely logs the memory swap high watermark and the cpu time of the service | |
| 14:40:20 | gibi | so I went with that | |
| 14:40:32 | sean-k-mooney | yes i saw that also | |
| 14:40:52 | sean-k-mooney | however we only have the logs form the second run correct | |
| 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 | |