| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-11-24 | |||
| 16:32:36 | Uggla | That's all for me today. | |
| 16:32:56 | Uggla | Anything else someone wants to add ? | |
| 16:33:47 | Uggla | Seems not, so time to close. | |
| 16:33:51 | Uggla | Thanks for joining this meeting. Have a nice day/evening. | |
| 16:33:58 | Uggla | #endmeeting | |
| 16:33:58 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-11-24-16.00.html | |
| 16:33:58 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-11-24-16.00.txt | |
| 16:33:58 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-11-24-16.00.log.html | |
| 16:33:58 | opendevmeet | Meeting ended Mon Nov 24 16:33:58 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:34:14 | tkajinam | thx | |
| 16:34:14 | elodilles | thanks o/ | |
| 16:35:48 | kgube | o/ | |
| 17:17:17 | opendevreview | Phillip Toohill proposed openstack/nova-specs master: WIP: Improve image caching behavior https://review.opendev.org/c/openstack/nova-specs/+/968065 | |
| 18:22:31 | nicolairuckel | ||
| #openstack-nova - 2025-11-25 | |||
| 00:47:25 | opendevreview | Seyeong Kim proposed openstack/nova master: libvirt: Support boot_index for multiple block devices https://review.opendev.org/c/openstack/nova/+/963665 | |
| 01:34:18 | opendevreview | Seyeong Kim proposed openstack/nova master: libvirt: Support boot_index for multiple block devices https://review.opendev.org/c/openstack/nova/+/963665 | |
| 09:17:33 | nicolairuckel | For my patch (https://review.opendev.org/c/openstack/nova/+/959682), I tried to make cold migration work. Sean suggested using `libvirt_utils.copy_image` to copy the NVRAM file. Unfortunately, it still fails because of missing read permissions. When I set those manually the file will be present after the migration with the same permissions but my test UEFI variable is gone. Does anyone have an idea what I | |
| 09:17:33 | nicolairuckel | could be missing here? This is what I tried: https://paste.opendev.org/show/bWX0QudxanRHdz8wpZs9/ | |
| 11:31:32 | stephenfin | gibi: sean-k-mooney: Not urgent, but this series does the same pyproject.toml migration and wsgi_script removal for placement that we recently did for nova, if you want to take a look https://review.opendev.org/c/openstack/placement/+/919582 I think ye reviewed the nova ones | |
| 11:51:08 | sean-k-mooney | stephenfin: those all look good to me | |
| 11:55:13 | stephenfin | thanks | |
| 12:14:32 | gibi | stephenfin: I have request in https://review.opendev.org/c/openstack/placement/+/967602 I'm +2+A on the rest | |
| 12:14:53 | nicolairuckel | sean-k-mooney: Would you have time to look at my issue with copy_image? | |
| 12:32:38 | sean-k-mooney | nicolairuckel: so to workaroudn the perrmeion issue we will need to invoke the copy with privsep (basically sudo cp vs cp) | |
| 13:11:22 | opendevreview | Stephen Finucane proposed openstack/placement master: pre-commit: Bump versions https://review.opendev.org/c/openstack/placement/+/967602 | |
| 13:12:19 | stephenfin | gibi: Good spot on https://review.opendev.org/c/openstack/placement/+/967602. Resolve now with comment left explaining where I was coming from (I did actually think about it at the time and still got it wrong 😅) | |
| 13:12:34 | stephenfin | *Resolved | |
| 13:36:47 | nicolairuckel | I'll look into that, thanks | |
| 14:02:09 | opendevreview | Merged openstack/placement master: setup: Remove pbr's wsgi_scripts https://review.opendev.org/c/openstack/placement/+/919582 | |
| 14:02:11 | opendevreview | Merged openstack/placement master: Correct default oslo.conf file https://review.opendev.org/c/openstack/placement/+/967600 | |
| 14:02:13 | opendevreview | Merged openstack/placement master: Migrate setup configuration to pyproject.toml https://review.opendev.org/c/openstack/placement/+/967601 | |
| 15:02:35 | Uggla | dansmith, yesterday I promised Luzy to ping you about the review for the image encryption bp & glance spec. | |
| 15:08:30 | gibi | stephenfin: thanks +Ad it | |
| 15:10:23 | dansmith | Uggla: thanks, but I get an email every time it's updated, it was just revised again yesterday and it's in my list to re-review.. but I have lots of things in the queue | |
| 15:13:05 | Uggla | dansmith, it is what I explained yesterday, that you were busy with escalations etc... I don't know the timelines for Glance, maybe there is still time before spec freeze too. | |
| 15:20:09 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: Use built-in firmware auto-selection for UEFI firmware https://review.opendev.org/c/openstack/nova-specs/+/966584 | |
| 16:11:22 | opendevreview | Merged openstack/nova stable/2024.2: Add managed='no' flag to libvirt XML definition for VIF type TAP https://review.opendev.org/c/openstack/nova/+/967569 | |
| 16:14:45 | opendevreview | Merged openstack/placement master: pre-commit: Bump versions https://review.opendev.org/c/openstack/placement/+/967602 | |
| 16:18:33 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: Use built-in firmware auto-selection for UEFI firmware https://review.opendev.org/c/openstack/nova-specs/+/966584 | |
| 16:21:32 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: Use built-in firmware auto-selection for UEFI firmware https://review.opendev.org/c/openstack/nova-specs/+/966584 | |
| 16:32:59 | opendevreview | Takashi Kajinami proposed openstack/nova-specs master: libvirt: Use built-in firmware auto-selection for UEFI firmware https://review.opendev.org/c/openstack/nova-specs/+/966584 | |
| 17:22:05 | opendevreview | Merged openstack/nova stable/2024.2: nova-conductor puts instance in error state https://review.opendev.org/c/openstack/nova/+/961775 | |
| 18:42:15 | nelljerram | FYI all backports for the "libvirt-tap-managed-no" topic have been merged now. Thanks so much to elodilles for nudging these along. masahito you may like to abandon https://review.opendev.org/c/openstack/nova/+/940135 now. If there are any related loose ends that I should work on, please do let me know. (I'm already thinking about how to plumb in a third-party Calico CI...) | |
| 20:43:48 | nicolairuckel | Does migrate_disk_and_power_off not get called for cold migration? I thought that would be a good place to copy the NVRAM. | |
| 21:28:09 | opendevreview | Ghanshyam proposed openstack/nova master: [Trivial]: Remove unused RBAC default alias https://review.opendev.org/c/openstack/nova/+/968410 | |
| 21:29:52 | gmaan | Uggla: ^^ adding some notes and removing the common alias which should not be used. | |
| 22:48:34 | opendevreview | Sean Mooney proposed openstack/nova-specs master: add spec for resouce notifications https://review.opendev.org/c/openstack/nova-specs/+/967712 | |
| #openstack-nova - 2025-11-26 | |||
| 09:39:55 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.2: Update start_service() function in test https://review.opendev.org/c/openstack/nova/+/963848 | |
| 09:39:56 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.2: Adds regression test for bug LP#2085135 https://review.opendev.org/c/openstack/nova/+/963849 | |
| 09:39:57 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.2: Reset the mapped field of nodes at service deletion https://review.opendev.org/c/openstack/nova/+/963850 | |
| 09:58:53 | Uggla | gmaan, lgtm. +1 on https://review.opendev.org/c/openstack/nova/+/968410, I have added bauzas and gibi because I think we can go ahead. | |
| 11:27:03 | opendevreview | Dominik proposed openstack/nova master: Remove Placement allocations in the broken build cleanup https://review.opendev.org/c/openstack/nova/+/968446 | |
| 13:55:12 | gibi | dansmith: bauzas: Uggla: Kamil: I need to skip the eventlet sync today. Sorry. Feel free to have it without me. | |
| 13:55:20 | gibi | sean-k-mooney: ^^ | |
| 13:59:33 | sean-k-mooney | ack, i have a meeting before it which might over run so i may or may not be there either | |
| 14:04:08 | sean-k-mooney | gibi: heh fun, you know the way we say we dont reallly supprot runing addtional libvirt vms on the same host even though its techinally possible | |
| 14:04:11 | sean-k-mooney | https://paste.opendev.org/show/beuKA1lA9Pt5xDTYYwJR/ | |
| 14:04:24 | sean-k-mooney | this is one of hte way tha tcan break things | |
| 14:05:23 | sean-k-mooney | basicaly if nova cant access the disk images then it would break update aviable resouces | |
| 14:41:02 | noonedeadpunk | sean-k-mooney: hey! have a discussion point around one of the comment on https://review.opendev.org/c/openstack/nova-specs/+/900296/7/specs/2026.1/approved/cross-az-instance-scheduling.rst if you have a minute | |
| 14:41:18 | noonedeadpunk | I've addressed all points (did not push them yet), except `cross_az_attach` | |
| 14:41:50 | noonedeadpunk | As my way of thinking was that I don't need care about that in this spec, as it will be handled here anyway: https://opendev.org/openstack/nova/src/commit/23b462d77df1a1d09c43d0918bca853ef3af1e3f/nova/compute/api.py#L1429-L1430 | |
| 14:42:22 | noonedeadpunk | so if it won't be matching, `MismatchVolumeAZException` will be raised | |
| 14:49:02 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.2: Update start_service() function in test https://review.opendev.org/c/openstack/nova/+/963848 | |
| 14:49:03 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.2: Adds regression test for bug LP#2085135 https://review.opendev.org/c/openstack/nova/+/963849 | |
| 14:49:04 | opendevreview | Julien LE JEUNE proposed openstack/nova stable/2024.2: Reset the mapped field of nodes at service deletion https://review.opendev.org/c/openstack/nova/+/963850 | |
| 14:49:56 | sean-k-mooney | noonedeadpunk: so cross_az attach is a per compute node opiton | |
| 14:50:27 | noonedeadpunk | it seems it's currently verified on api side already? | |
| 14:50:47 | noonedeadpunk | as iirc nova.compute.api is api-side? | |
| 14:51:09 | sean-k-mooney | my point is if you dont configure in the api and only configure it the comptue nodes which is allowed then it will fail late | |
| 14:51:19 | noonedeadpunk | oh, yes | |
| 14:51:49 | noonedeadpunk | but I think it's already the case today? I am just thinking if this is smth which should be addressed in spec at all | |
| 14:52:16 | sean-k-mooney | we just need to document that in the spec | |
| 14:52:48 | sean-k-mooney | in generally i woudl prefer to remove that config option entirly and either replace it with nothing or a flavor extra spec | |
| 14:52:57 | sean-k-mooney | this shoudl never have been setable on a per compute basis | |
| 14:53:11 | sean-k-mooney | but that proably not somethign we can do at this point | |
| 14:53:36 | noonedeadpunk | the reason why I'm raising this, is that we are actually having cross_az_attach=False here.... | |
| 14:53:46 | noonedeadpunk | And made it working *somehow* | |
| 14:53:52 | sean-k-mooney | it built on the false premiss that nova AZ names and AZ names in any other service shoudl align | |
| 14:53:52 | noonedeadpunk | In *some* cases... | |
| 14:54:48 | sean-k-mooney | so if you set cross_az_attach=False in the api then we will end up pinnign the vm to the az of the voluem if and only if you pass in a volume | |
| 14:54:53 | noonedeadpunk | So I'd really love to leave "status quo" for this specific spec wrt cross_az_attach | |
| 14:55:14 | sean-k-mooney | if on the other hand nova creates the voluem i dont think we pin it because we create teh volume after schdulign i think | |
| 14:55:32 | noonedeadpunk | Yes, I think it's true actually | |
| 14:55:33 | sean-k-mooney | noonedeadpunk: yes im not asking you to change the behvior fomr today | |
| 14:55:37 | sean-k-mooney | just document what it iss | |
| 14:55:40 | noonedeadpunk | ++ | |
| 14:55:53 | noonedeadpunk | ok, shoot... | |
| 14:56:15 | noonedeadpunk | so basically, it would make things worse, or indeed I'd need to address that in implementation.... | |
| 14:56:24 | noonedeadpunk | or well... | |
| 14:56:42 | sean-k-mooney | you can litrally punt an just say "this will not change the behvior of cross_az_attach" and if its disabel then it will fail if there is a confilct | |
| 14:57:22 | noonedeadpunk | ++ | |
| 14:57:59 | noonedeadpunk | I am literally failing now to set in my head cross_az_attach behavior in all scenarios.... | |
| 14:58:13 | noonedeadpunk | ok, thanks, will do that then | |
| 14:59:26 | sean-k-mooney | yep because its stupily complicated. the default is true which mean sthe api does nto condier the voluem az by default | |
| 14:59:40 | sean-k-mooney | if you only set it to false on the compute we only check it there | |
| 15:00:05 | sean-k-mooney | if you onely set it on the api we will validate teh az request in the build request vs the volume | |
| 15:00:22 | sean-k-mooney | and we will use the volume's az if there isnt a conflict | |
| 15:00:40 | sean-k-mooney | there is some other behvior thace chages as well | |
| 15:00:56 | noonedeadpunk | unless you create an instance with `--boot-fropm-volume` | |