| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-01-24 | |||
| 06:48:28 | masanori | Hello. I have created this bug fix: https://review.opendev.org/c/openstack/nova/+/936910 Could you please review it when you have time. > gibi, stephenfin or anyone :) This fixes the process to get the image's metadata from instance.image_meta instead of request_spec.image during resize. | |
| 10:11:09 | stephenfin | sean-k-mooney: Would you be able to take a look at https://review.opendev.org/c/openstack/nova/+/937377/ today? | |
| 10:18:15 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Allow min/max_version arguments to expected_errors https://review.opendev.org/c/openstack/nova/+/936363 | |
| 10:18:15 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Allow min/max_version arguments to response https://review.opendev.org/c/openstack/nova/+/936364 | |
| 10:18:16 | opendevreview | Stephen Finucane proposed openstack/nova master: trivial: Remove legacy API artifact https://review.opendev.org/c/openstack/nova/+/937377 | |
| 10:18:16 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Adjust validation helpers for a single-method future https://review.opendev.org/c/openstack/nova/+/936365 | |
| 10:18:17 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Stop using wsgi.Controller.api_version to switch between API versions https://review.opendev.org/c/openstack/nova/+/936366 | |
| 10:18:17 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add new, simpler api_version decorator https://review.opendev.org/c/openstack/nova/+/936367 | |
| 10:18:18 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only run format checks on strings https://review.opendev.org/c/openstack/nova/+/936368 | |
| 10:18:19 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Simplify parameter types https://review.opendev.org/c/openstack/nova/+/936369 | |
| 10:18:19 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Ensure all APIs have a response body schema https://review.opendev.org/c/openstack/nova/+/924598 | |
| 10:18:21 | opendevreview | Stephen Finucane proposed openstack/nova master: doc: Add missing API samples https://review.opendev.org/c/openstack/nova/+/937012 | |
| 10:18:21 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use UUIDs in TestOpenStackClient https://review.opendev.org/c/openstack/nova/+/937013 | |
| 10:18:23 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hosts APIs https://review.opendev.org/c/openstack/nova/+/937047 | |
| 10:18:23 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for instance actions https://review.opendev.org/c/openstack/nova/+/937048 | |
| 10:18:25 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (1/3) https://review.opendev.org/c/openstack/nova/+/937245 | |
| 10:18:25 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (2/3) https://review.opendev.org/c/openstack/nova/+/937246 | |
| 10:18:27 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for hypervisors APIs (3/3) https://review.opendev.org/c/openstack/nova/+/937247 | |
| 10:18:27 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server IPs APIs https://review.opendev.org/c/openstack/nova/+/937608 | |
| 10:18:29 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for remote consoles https://review.opendev.org/c/openstack/nova/+/940114 | |
| 10:18:29 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for keypairs APIs https://review.opendev.org/c/openstack/nova/+/940115 | |
| 14:24:42 | bauzas | sean-k-mooney: reading your whiteboard comment on https://blueprints.launchpad.net/nova/+spec/image-metadata-props-weigher | |
| 14:24:55 | bauzas | sean-k-mooney: are we clear that I *need* to create those sets , | |
| 14:24:57 | bauzas | ? | |
| 14:29:15 | sean-k-mooney | can you repahse that | |
| 14:29:31 | sean-k-mooney | they dont already exist but they are trivial to construct from the data you have | |
| 14:29:55 | sean-k-mooney | i was trying to descibe the algorithum i would use and its the one i would have documented in a spec if we were to have had a spec | |
| 14:31:42 | sean-k-mooney | are you askign are free to come up with an entirly diffent design? from my perspecitve i was agreeing to treat this treat it as a specless blueprint on the understadign that you would implemnt the design in the whiteboard but if you ahve a diffent approch i can review it | |
| 14:33:14 | sean-k-mooney | bauzas: if ^ is not what you were asking can you restate the question | |
| 14:33:19 | bauzas | nevermind, your answers are fine | |
| 14:33:40 | bauzas | sean-k-mooney: your whiteboard description looked confusing to me | |
| 14:33:53 | bauzas | but OK, fair enough, your response helped me | |
| 14:34:17 | sean-k-mooney | well the clip notes version is score = common_props / total_props | |
| 14:34:55 | sean-k-mooney | the rest was just defining how to count the common and total sets | |
| 14:34:59 | bauzas | so, yeah, I'll lookup the instances flavor field for generating the sets and as as we discussed https://meetings.opendev.org/meetings/nova/2025/nova.2025-01-07-16.01.log.html#l-86 we avoid a DB lookup call | |
| 14:35:41 | bauzas | okay, I see, you provided some high level algorithm for calculating the weigh by intersecting sets | |
| 14:35:41 | sean-k-mooney | ya so if you not adding db calls, but decisde to use a dict or caluate the value with omething other then a set that is ok | |
| 14:36:15 | bauzas | sets are interesting because they come with batteries for calculating the intersects or the joined elemenbts | |
| 14:36:27 | sean-k-mooney | the set approch should be fast yes and that^ | |
| 14:36:28 | bauzas | so your idea sounds good to me | |
| 14:36:44 | bauzas | ok, now I better understand, thanks | |
| 14:36:54 | bauzas | and thanks for the idea | |
| 14:37:38 | sean-k-mooney | no worries. i likely wont get to it today but if you have a poc that you want me to look at ping me when its pushed | |
| 14:38:06 | opendevreview | Masahito Muroi proposed openstack/nova master: Add managed='no' flag to a guest xml definition https://review.opendev.org/c/openstack/nova/+/940135 | |
| 14:38:29 | sean-k-mooney | am ... | |
| 14:39:56 | sean-k-mooney | oh that is seprate form the pci thing | |
| 14:50:13 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for keypairs APIs https://review.opendev.org/c/openstack/nova/+/940115 | |
| #openstack-nova - 2025-01-25 | |||
| 02:11:06 | opendevreview | melanie witt proposed openstack/nova master: pwr mgmt: power down free PCPUS when updating compute node https://review.opendev.org/c/openstack/nova/+/932926 | |
| 05:48:52 | opendevreview | melanie witt proposed openstack/nova master: pwr mgmt: power down free PCPUS when updating compute node https://review.opendev.org/c/openstack/nova/+/932926 | |
| 12:41:06 | opendevreview | Artom Lifshitz proposed openstack/os-traits master: Add vTPM secret policy traits https://review.opendev.org/c/openstack/os-traits/+/940191 | |
| 17:32:05 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Allow vTPM live migrations for admins https://review.opendev.org/c/openstack/nova/+/925771 | |
| 17:32:05 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add [libvirt]default_tpm_secret_security https://review.opendev.org/c/openstack/nova/+/940194 | |
| 17:32:06 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add [libvirt]supported_tpm_secret_security https://review.opendev.org/c/openstack/nova/+/940195 | |
| 17:32:06 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add hw_tpm_secret_security image property https://review.opendev.org/c/openstack/nova/+/940196 | |
| 17:32:07 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197 | |
| #openstack-nova - 2025-01-26 | |||
| 01:47:52 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add [libvirt]default_tpm_secret_security https://review.opendev.org/c/openstack/nova/+/940194 | |
| 01:47:52 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add [libvirt]supported_tpm_secret_security https://review.opendev.org/c/openstack/nova/+/940195 | |
| 01:47:53 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add hw_tpm_secret_security image property https://review.opendev.org/c/openstack/nova/+/940196 | |
| 01:47:53 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197 | |
| 01:47:54 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Allow vTPM live migrations for admins https://review.opendev.org/c/openstack/nova/+/925771 | |
| 15:24:12 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add [libvirt]supported_tpm_secret_security https://review.opendev.org/c/openstack/nova/+/940195 | |
| 15:24:13 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add hw_tpm_secret_security image property https://review.opendev.org/c/openstack/nova/+/940196 | |
| 15:24:13 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Add hw:tpm_secret_security extra spec validation https://review.opendev.org/c/openstack/nova/+/940197 | |
| 15:24:14 | opendevreview | Artom Lifshitz proposed openstack/nova master: WIP: Allow vTPM live migrations for admins https://review.opendev.org/c/openstack/nova/+/925771 | |
| #openstack-nova - 2025-01-27 | |||
| 03:08:43 | opendevreview | Masanori Kuroha proposed openstack/nova-specs master: Allow only one custom trait with the same prefix in provider.yaml https://review.opendev.org/c/openstack/nova-specs/+/940225 | |
| 03:10:46 | opendevreview | Masanori Kuroha proposed openstack/nova-specs master: Allow only one custom trait with the same prefix in provider.yaml https://review.opendev.org/c/openstack/nova-specs/+/937587 | |
| 03:26:06 | opendevreview | Masanori Kuroha proposed openstack/nova-specs master: Allow only one custom trait with the same prefix in provider.yaml https://review.opendev.org/c/openstack/nova-specs/+/937587 | |
| 10:57:41 | priteau | Hello Nova team. Could another core approve this backport? https://review.opendev.org/c/openstack/nova/+/939665 | |
| 11:01:26 | gibi | priteau: done, thanks for backporting | |
| 12:20:45 | opendevreview | Biser Milanov proposed openstack/nova master: StorPool: Pass the instance UUID and device_name to os-brick https://review.opendev.org/c/openstack/nova/+/930297 | |
| 13:14:58 | opendevreview | Merged openstack/nova stable/2024.2: libvirt: Wrap un-proxied listDevices() and listAllDevices() https://review.opendev.org/c/openstack/nova/+/939665 | |
| 13:19:08 | opendevreview | Pierre Riteau proposed openstack/nova stable/2024.1: libvirt: Wrap un-proxied listDevices() and listAllDevices() https://review.opendev.org/c/openstack/nova/+/939695 | |
| 13:21:21 | priteau | Thank you gibi. The Caracal backport is also ready to review ^ | |
| 13:29:34 | opendevreview | Biser Milanov proposed openstack/nova master: StorPool: Pass the instance UUID and device_name to os-brick https://review.opendev.org/c/openstack/nova/+/930297 | |
| 14:44:29 | bauzas | sean-k-mooney: hrmmm, looks like system_metadata needs to be lazy-loaded https://paste.opendev.org/show/boOPqY0RzgGM3SYO8k5M/ | |
| 14:45:16 | bauzas | sean-k-mooney: context being https://meetings.opendev.org/meetings/nova/2025/nova.2025-01-07-16.01.log.html#l-86 | |
| 14:45:18 | sean-k-mooney | bauzas: it depend where. its in the default filed so it should be loaded by default unless you change that | |
| 14:45:44 | bauzas | sean-k-mooney: I tested on the HostState.instances value | |
| 14:46:31 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L69-L71 | |
| 14:47:18 | sean-k-mooney | maybe we are not using that for what i tought we were | |
| 14:50:07 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L875-L896 | |
| 14:50:52 | sean-k-mooney | so it looks like we are building fake instance object wiht only the uuid set | |
| 14:52:28 | sean-k-mooney | bauzas: do you have track_instance_changes enabled | |
| 14:53:23 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for showing image properties in server show response https://review.opendev.org/c/openstack/nova/+/939649 | |
| 14:54:05 | bauzas | sean-k-mooney: yes indeed, but I'll doublecheck | |
| 14:54:42 | sean-k-mooney | bauzas: what we likely need to do is repalce the call to just get the uuid and expiclty get the uuid and system metadata as teh only requested fieds | |
| 14:54:57 | sean-k-mooney | by calling https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L1441-L1446 | |
| 14:57:05 | sean-k-mooney | although i migh tbe looking at the wrong place | |
| 14:57:20 | sean-k-mooney | i was looking at teh host manager. its fine for it to have just the id | |
| 14:57:42 | sean-k-mooney | its https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L144 that we really care about | |
| 14:58:44 | sean-k-mooney | i suspect its also jsut a dict with the minimal object | |
| 15:00:47 | sean-k-mooney | they are consutcted here https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L790-L827 | |
| 15:01:30 | sean-k-mooney | so the instnace info is populated by https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L880 | |
| 15:02:15 | sean-k-mooney | so unfortunetly it will have a minor db impact to lookup taht data instead of just the uuid | |
| 15:04:35 | artom | Uggla, since you mentioned doing the os-traits release, the TPM traits patch is pending: https://review.opendev.org/c/openstack/os-traits/+/940191 | |
| 15:08:01 | bauzas | sean-k-mooney: you know what ? we could just say that it could have some db performance if we use the weigher, but fwiw, my change for moment doesn't default to use this weigher | |
| 15:08:12 | bauzas | so let's discuss this on the review then | |
| 15:12:54 | sean-k-mooney | sure, we have 2 options as far as i see, 1 lazy load if you enable the weigher, 2 perload the filed either based on the weigher multipler or a new config | |
| 15:13:09 | sean-k-mooney | *feild | |
| 15:13:36 | sean-k-mooney | in either case you are opting in to the schdulelr memory/db overhead | |