| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-03-13 | |||
| 16:04:50 | dansmith | does not have to be server id | |
| 16:05:21 | dansmith | I dunno why but that just worked.. I spent a stupid amount of time trying to get that to work (granted on a libvirt non-nova VM, maybe there was some config difference) | |
| 16:05:30 | dansmith | so let me see if that sticks after the reboot | |
| 16:07:16 | dansmith | sho'nuf.. works as expected | |
| 16:07:44 | dansmith | I dunno, I got myself very turned around with efivar before and thought I'd just manipulate the boot config since that was less obscure anyway | |
| 16:07:58 | dansmith | there must be something going on that overwrites that boot config on startup or something, I dunno why | |
| 16:08:13 | dansmith | but the efivar method to verify is what I originally wanted, so thanks for that :) | |
| 16:08:19 | nicolairuckel | You're welcome | |
| 16:08:36 | nicolairuckel | I just used efivar because my colleague showed me the commands I needed. | |
| 16:08:52 | nicolairuckel | That's why I never tried efibootmgr. | |
| 16:10:33 | dansmith | yeah, well, efivar is very undocumented and obscure so I had trouble getting ti to behave *shrug* | |
| 16:11:12 | nicolairuckel | I think that's why I never bothered to figure out why at some point the UUID had to be the server ID. | |
| 16:11:35 | nicolairuckel | But I added that to my notes that this isn't actually the case. | |
| 16:11:37 | dansmith | but to be clear, it doesn't.. I just used uuigen to get one | |
| 16:11:40 | dansmith | yeah, cool | |
| 16:12:00 | nicolairuckel | I wonder if that was a problem in my first implementation where I copied the variables manually instead of using libvirt. | |
| 16:12:19 | nicolairuckel | And since using the server ID still works, I never realized that it's not necessary anymore. | |
| 16:15:41 | nicolairuckel | thanks for testing it again | |
| 16:16:22 | dansmith | yup, I was doing something similar for vTPM to prove it works | |
| 16:16:28 | dansmith | again requiring obscure and temperamental tooling | |
| 16:16:38 | dansmith | s/works/persists | |
| 16:16:40 | nicolairuckel | Even though you scared me a bit. For a few minutes I thought I'd have to go back to work on that after telling everyone it was done and merged. :D | |
| 16:16:45 | dansmith | heh | |
| 18:21:59 | gmaan | does anyone knows why we are keeping nova/keymgr/conf_key_mgr.py ? https://github.com/openstack/nova/blob/master/nova/keymgr/conf_key_mgr.py | |
| 18:22:33 | gmaan | I think it is only for testing and because barbican ins not installed by default in devstack, devstack also use this as key_manager backendhttps://github.com/openstack/devstack/blob/master/lib/nova#L460C23-L460C34 | |
| 18:23:47 | gmaan | melwitt: ^^ maybe you know, as enabling the barbican in vTPM jobs modify the key_manager backend to barbican which actually test the secrets store/get etc https://github.com/openstack/barbican/blob/master/devstack/lib/barbican#L94 | |
| 18:24:48 | gmaan | any other place we use this fixed_key backend (nova/keymgr/conf_key_mgr.py) then testing in unit/functional or devstack testing? | |
| 18:25:29 | gmaan | or stephenfin ^^ as you modified it last (though 3 years ago :)) | |
| 18:42:31 | stephenfin | gmaan: yeah, it's purely for testing. I think I had plans to move it to castellan (e.g. castellan.test) but there were a few references in other projects, as you've seen | |
| 18:44:04 | gmaan | stephenfin: ack, thanks. that is enough information for now. but ++ on moving | |
| 20:00:29 | melwitt | Uggla, gmaan: I noticed a pending spec amendment patch and thought it should merge first before the "spec move" patch. it will cause a merge conflict in the "spec move" patch, sorry about that https://review.opendev.org/c/openstack/nova-specs/+/978144 | |
| 20:01:51 | gmaan | melwitt: oh, there is another one. I thought vtpm amendment was the only one left | |
| 20:02:31 | melwitt | yeah, I noticed it :/ | |
| 20:02:50 | melwitt | alternatively we could move the specs and then make the amendment have to update | |
| 20:03:44 | gmaan | as amenedment one is in gate, maybe let's merge that and Uggla can rebase the spec move one | |
| 20:05:55 | melwitt | I thought it should go first and then second guessed it. | |
| 20:06:22 | gmaan | yeah | |
| 20:07:31 | melwitt | I did not expect it to merge conflict bc the spec move is just renaming files. but I guess the file content at the time matters too | |
| 20:11:03 | gmaan | ++ | |
| 20:13:26 | opendevreview | Merged openstack/nova-specs master: Update remove-os-volumes_boot-api to reflect changes https://review.opendev.org/c/openstack/nova-specs/+/978144 | |
| 22:05:19 | opendevreview | Ashish Gupta proposed openstack/nova master: Add native threading support to test case ID propagation fixtures https://review.opendev.org/c/openstack/nova/+/980179 | |
| 22:32:15 | opendevreview | Stephen Finucane proposed openstack/nova stable/2025.2: api: Handle empty imageRef alongside null for local BDM check https://review.opendev.org/c/openstack/nova/+/980540 | |
| 22:32:40 | opendevreview | Stephen Finucane proposed openstack/nova stable/2025.1: api: Handle empty imageRef alongside null for local BDM check https://review.opendev.org/c/openstack/nova/+/980541 | |
| 22:32:59 | opendevreview | Stephen Finucane proposed openstack/nova stable/2025.1: api: Handle empty imageRef alongside null for local BDM check https://review.opendev.org/c/openstack/nova/+/980541 | |
| #openstack-nova - 2026-03-14 | |||
| 02:31:03 | opendevreview | melanie witt proposed openstack/nova master: Add nova-vtpm job https://review.opendev.org/c/openstack/nova/+/978841 | |
| 04:36:22 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/980544 | |
| #openstack-nova - 2026-03-15 | |||
| 23:14:36 | opendevreview | sean mooney proposed openstack/nova master: [WIP] perf: add ironic startup perf simulator and analysis https://review.opendev.org/c/openstack/nova/+/980641 | |
| 23:30:34 | opendevreview | sean mooney proposed openstack/nova master: [WIP] perf: add ironic startup perf simulator and analysis https://review.opendev.org/c/openstack/nova/+/980641 | |
| #openstack-nova - 2026-03-16 | |||
| 07:57:26 | opendevreview | Thibaut Démaret proposed openstack/nova master: libvirt: add disk rotation_rate support for local disks https://review.opendev.org/c/openstack/nova/+/979693 | |
| 09:03:05 | rmnijgh | Hi! It came to my attention that nova live migrate also calls the periodic task update_available_resource() in the post_live_migration... | |
| 09:03:44 | rmnijgh | However, this tasks can take a few minutes. And in those minutes the migration task is still seen as running, even though the migration is already finished. | |
| 09:04:21 | rmnijgh | Is this necessary to run at post_live_migrate? Since this task already runs periodically (I believe every 2 minutes by default) | |
| 09:05:16 | rmnijgh | Not having this function at post_live_migrate completes migration tasks significantly faster | |
| 09:49:06 | rmnijgh | I've just found a bug report with the exact problem I am experiecing #2085709 :D | |
| 10:10:31 | opendevreview | ribaudr proposed openstack/nova-specs master: Move Gazpacho implemented specs https://review.opendev.org/c/openstack/nova-specs/+/979283 | |
| 10:37:10 | opendevreview | sean mooney proposed openstack/nova master: [WIP] perf(ironic): eliminate O(N²) ProviderTree deepcopy at startup (B6) https://review.opendev.org/c/openstack/nova/+/980676 | |
| 10:37:10 | opendevreview | sean mooney proposed openstack/nova master: [WIP] perf: add ironic startup perf simulator and analysis https://review.opendev.org/c/openstack/nova/+/980641 | |
| 10:37:11 | opendevreview | sean mooney proposed openstack/nova master: [WIP] perf(ironic): background _error_out_interrupted during startup (B4) https://review.opendev.org/c/openstack/nova/+/980678 | |
| 10:37:11 | opendevreview | sean mooney proposed openstack/nova master: [WIP] perf(ironic): suppress redundant _refresh_cache calls during startup (B1) https://review.opendev.org/c/openstack/nova/+/980677 | |
| 10:37:12 | opendevreview | sean mooney proposed openstack/nova master: [WIP] docs: update startup.md with post-optimisation measurements https://review.opendev.org/c/openstack/nova/+/980680 | |
| 10:37:12 | opendevreview | sean mooney proposed openstack/nova master: [WIP] perf(compute): parallelise per-node update_available_resource loop (B2) https://review.opendev.org/c/openstack/nova/+/980679 | |
| 10:50:15 | sean-k-mooney | rmnijgh: it calls it because it was oen of the ways we heal the resouces asyncornsly, it runs every 5 mintues by defualt. it used to be used to move pci device form claimed to allocated. although on newer release i belive we do that seperatly | |
| 10:51:51 | sean-k-mooney | i dont think that call shoudl be needed in antelop+ possibly a little eailer but | |
| 10:52:04 | sean-k-mooney | we have not exaimed if it can be remvoed again | |
| 11:05:53 | rmnijgh | I can confirm this bug report #2085709, we have a large kolla-ansible openstack cluster. And nodes with 50+ instances take ages to migrate (or even timeout), but also the regular periodic task update_available_resource takes ages. Can the reviews in this report perhaps get review priority? I think I'm able to make changes in these two commits when neccesary. | |
| 12:33:29 | opendevreview | Lajos Katona proposed openstack/nova master: docs: Update libvirt version support matrix for 26.1 Gazpacho https://review.opendev.org/c/openstack/nova/+/980703 | |
| 12:51:15 | opendevreview | Merged openstack/nova stable/2026.1: Update .gitreview for stable/2026.1 https://review.opendev.org/c/openstack/nova/+/980404 | |
| 13:02:06 | opendevreview | Merged openstack/nova stable/2026.1: Update TOX_CONSTRAINTS_FILE for stable/2026.1 https://review.opendev.org/c/openstack/nova/+/980405 | |
| 13:59:27 | zigo | sean-k-mooney: Does some of them ring a bell? | |
| 13:59:27 | zigo | https://paste.opendev.org/show/bgO4WRXAvotrU1s66cx8/ | |
| 13:59:27 | zigo | Building Nova backport to Trixie leads me to these unit test failures: | |
| 14:04:51 | zigo | As much as I can see, I have 5 types of failures. | |
| 14:07:41 | nicolairuckel | elodilles: Thanks for restarting the pipeline. The same test failed a few times already though so I'm wondering if that's a bigger issue with the CI. And when it doesn't fail, another (unrelated) test fails. | |
| 14:14:21 | elodilles | nicolairuckel: i was having the same feeling - that something introduced a bug in tempest (?) - but then i saw a successful job run on stable/2025.2 branch, so it's either got fixed, or it's disappeared somehow o:) so i gave it another try now :) | |
| 14:19:19 | zigo | The aquire_lock thingy looks like probably comming from the fasterners version, I'm fixing that. | |
| 14:26:00 | nicolairuckel | Fingers are crossed that it works now. :D | |
| 14:27:00 | elodilles | nicolairuckel: grenade-skip-level passed already ( https://zuul.opendev.org/t/openstack/build/337549cbec7f480a8f36bba5cd23f208 ) now only the others should do the same. fingers crossed, indeed ;) | |
| 14:34:45 | opendevreview | Elod Illes proposed openstack/placement stable/2025.1: [CI][stable-only] Fix docs job https://review.opendev.org/c/openstack/placement/+/980761 | |
| 14:52:18 | opendevreview | Elod Illes proposed openstack/osc-placement stable/2024.2: [CI][stable-only] Workaround for missing pkg_resources https://review.opendev.org/c/openstack/osc-placement/+/979691 | |
| 14:56:44 | zigo | Ok, that fixed 10 unit tests, now only got the oslo.middleware api-paste.ini loading thingy to fix ... (plus one unit test that looks like non-deterministic). | |
| 15:05:28 | zigo | The healthcheck thing is probably due to my own patch. | |
| 15:11:48 | zigo | Yeah, all is passing now. :P | |
| 15:13:11 | zigo | Any reason why oslo.policy was taken away from nova.conf namespace list when generating it? | |
| 15:13:22 | zigo | Is this handled in nova.conf namespace? | |
| 15:19:29 | Uggla | Nova upstream meeting in ~40mn | |
| 15:32:52 | worteks-thidem | Hey, I try to propose a PR for rotatation rate in Libvirt, I know it is a old subject. But I have few applications use this informations to make a choice on specific scheduler. My PR is here : https://review.opendev.org/c/openstack/nova/+/979693. If somebody can review my code and explain me if I forget something. | |
| 15:37:19 | opendevreview | Takashi Kajinami proposed openstack/placement master: Use consistent program name for wsgi scripts and entry points https://review.opendev.org/c/openstack/placement/+/980775 | |
| 15:38:52 | opendevreview | Takashi Kajinami proposed openstack/placement master: Use consistent program name for wsgi scripts and entry points https://review.opendev.org/c/openstack/placement/+/980775 | |
| 15:41:51 | opendevreview | Takashi Kajinami proposed openstack/placement master: Use consistent program name for wsgi scripts and entry points https://review.opendev.org/c/openstack/placement/+/980775 | |
| 15:56:06 | s3rj1k | Hi, I would like to ask for review of https://review.opendev.org/c/openstack/nova/+/973924, this is mainly needed for Metal3 support and complementary PR is already proposed to Cloud-Init, I am hoping that Nova can consider merging this so cloud-init feature gets unblocked for merge | |
| 16:01:33 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:01:33 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:01:33 | opendevmeet | Meeting started Mon Mar 16 16:01:33 2026 UTC and is due to finish in 60 minutes. The chair is Uggla. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:01:33 | Uggla | #startmeeting nova | |
| 16:01:41 | Uggla | Hello everyone | |
| 16:02:01 | dansmith | o/ | |
| 16:02:04 | r-taketn | hello | |
| 16:02:16 | gmaan | o/ | |
| 16:02:21 | lajoskatona | o/ | |
| 16:02:50 | elodilles | o/ | |