| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-05-28 | |||
| 15:33:22 | sean-k-mooney | and this was ment ot prevent start an agent when you have instnace that need vtpm supprot but you have it turn off | |
| 15:34:00 | sean-k-mooney | JayF: ill try and write this up as a bug noting the ironic impact | |
| 15:34:14 | sean-k-mooney | and see if i can hack somethign togheter for this. | |
| 15:34:38 | sean-k-mooney | JayF: but also feel free to push any patches ye have if ye get time | |
| 15:42:06 | opendevreview | sean mooney proposed openstack/nova master: Fix unified limits to include all resource types https://review.opendev.org/c/openstack/nova/+/975872 | |
| 17:38:33 | sean-k-mooney | JayF: https://bugs.launchpad.net/nova/+bug/2154495 is the bug for that by the way | |
| 17:39:06 | sean-k-mooney | adding the new capablity trait is one of the ways to fix that but i am goign to see if i can first create a repoducer and then we can see what the best way forward is | |
| 17:39:40 | sean-k-mooney | im condiering if i just want to wrap the body of the function in a check for the verit driver that is in use or if this check shoudl move into the driver instead | |
| 17:40:09 | sean-k-mooney | all 3 approchs woudl work its just a question fo what is the better long term approch | |
| 17:41:40 | sean-k-mooney | im really not sure why this is not part of self.driver.init_host() | |
| 17:41:50 | sean-k-mooney | given its a driver sepcific check in the first palce | |
| 18:16:30 | isaacvicente[m] | there's a patch for this but seems to be abandoned, what you guys think? | |
| 18:16:30 | isaacvicente[m] | hi o/ is this bug still being tracked? https://bugs.launchpad.net/nova/+bug/2065599 | |
| 18:30:18 | sean-k-mooney | isaacvicente[m]: having both imageRef": "f2285517-a996-40d3-b331-c8214ec66b77", and destion_type: volume is not invlid nessisarly | |
| 18:30:37 | sean-k-mooney | ppofied the bdm and the imageref are the same | |
| 18:31:51 | sean-k-mooney | the metadata in the domain xml is a debug interface so wether tie image is incldue is mostly cosmetic | |
| 18:32:41 | sean-k-mooney | the real pain poitn woudl be if you were rlying on the presence or absance of the image refe to determin if its a BFV guest | |
| 18:32:57 | sean-k-mooney | that is curerntly really a hack | |
| 18:33:32 | sean-k-mooney | i have argued in the past that the image refence shoudl be aviabel for instnace that are booted form volume | |
| 18:34:04 | sean-k-mooney | if they are created like this or is storead on the volume as volume/image metadata | |
| 18:34:47 | sean-k-mooney | isaacvicente[m]: what is the issue you are actuly having and tryign to adress here? | |
| 18:34:58 | sean-k-mooney | this is not somethign that is beign activly worked on | |
| 18:35:25 | sean-k-mooney | but its also not a priorty to fix as its mostly cosmetic and its unclear that this request is invilad | |
| 18:36:11 | sean-k-mooney | fixing it woudl likely need a new microver version unless we decied to make this a 400 but that would break a set of exiting user where funcitally the call works and had no real negitive side effect | |
| 18:36:39 | isaacvicente[m] | I was searching for bugs to work on and found this, so I wish to know if the issue reported is relevant. If so, I would continue to work on the patch that already exists | |
| 18:36:58 | sean-k-mooney | isaacvicente[m]: oh ok | |
| 18:37:10 | sean-k-mooney | it a inconsitency in the api | |
| 18:37:17 | sean-k-mooney | but likely not one that is worth yoru time | |
| 18:37:35 | isaacvicente[m] | So, as fair as I understand this is not a bug, right? And could break some users' workflow | |
| 18:37:51 | sean-k-mooney | its boarderline | |
| 18:38:03 | sean-k-mooney | fixing it woudl break client that pass both | |
| 18:38:13 | sean-k-mooney | unless we do it via a new microverion | |
| 18:38:22 | sean-k-mooney | which would need a spec and woudl not be backportable | |
| 18:38:46 | isaacvicente[m] | Hmm I get it | |
| 18:38:49 | sean-k-mooney | and if we were to add a new microversion i woudl arrgue it woudl be better to add a boot_form_volume boolean to server show instead | |
| 18:39:19 | sean-k-mooney | and always show the galnce image if aviabel for the root disk regardless of if its boot form voluem or not | |
| 18:46:26 | sean-k-mooney | isaacvicente[m]: https://bugs.launchpad.net/nova/+bug/2108980 and https://review.opendev.org/c/openstack/nova/+/954460 | |
| 18:46:33 | sean-k-mooney | woudl be nicer to finish | |
| 18:49:31 | isaacvicente[m] | Thanks sean-k-mooney! I will check it out | |
| 18:50:26 | sean-k-mooney | if im being enfitly hone i dont think the lowhangin frut list for nova is really curated to low haning fruit | |
| 18:54:49 | isaacvicente[m] | yeah... some of them are definitely not for newcomers haha | |
| 18:55:06 | sean-k-mooney | i woudl almost say sorting bug by age and fining one that is triage but not in progess is a bettwer way https://bugs.launchpad.net/nova/+bug/2154428 for exmaple could be a nice one if fixe as sugested | |
| 18:58:48 | isaacvicente[m] | thats interesting, this will help me finding some bugs to work on | |
| 19:05:58 | isaacvicente[m] | that nova nova-scheduler aggregate bug seems a nice one, thanks again sean! | |
| 19:19:12 | opendevreview | Ghanshyam Maan proposed openstack/nova-specs master: Spec for the graceful shutdown part2: Task tracking https://review.opendev.org/c/openstack/nova-specs/+/986447 | |
| 19:20:27 | gmaan | dansmith: ^^ added the generic way of marking untracked tasks | |
| 19:29:28 | opendevreview | sean mooney proposed openstack/nova master: Add vTPM startup validation reproducer https://review.opendev.org/c/openstack/nova/+/990551 | |
| 19:29:29 | opendevreview | sean mooney proposed openstack/nova master: Limit vTPM startup check to libvirt https://review.opendev.org/c/openstack/nova/+/990552 | |
| 19:39:42 | sean-k-mooney | JayF: ^ i think makes sense but as part of that i notice that nova (at least for the libvir driver) is doing 2 seprate calls to objects.InstanceList.get_by_host during startup | |
| 19:40:38 | sean-k-mooney | so we can likely impove that again | |
| 21:18:12 | opendevreview | Isaac Silva proposed openstack/nova master: Disable interactive prompt on LVM image creation https://review.opendev.org/c/openstack/nova/+/990576 | |
| 21:21:31 | isaacvicente[m] | I found this easy one sean-k-mooney, could you review please? | |
| 22:03:42 | opendevreview | Isaac Silva proposed openstack/nova master: Disable interactive prompt on LVM image creation https://review.opendev.org/c/openstack/nova/+/990576 | |
| 22:04:08 | sean-k-mooney | i say that in the list too but the lvm driver is a little more neice | |
| 22:04:19 | sean-k-mooney | it is howver potentially a quick fix | |
| 22:05:42 | sean-k-mooney | so adding -y is an option the reason this happens is becasue lvm does not delete the disk content by defualt | |
| 22:05:48 | sean-k-mooney | when yo udelete a previous voluem | |
| 22:06:51 | sean-k-mooney | so if you set https://docs.openstack.org/nova/latest/configuration/config.html#libvirt.volume_clear to none | |
| 22:07:03 | sean-k-mooney | then when you recret a voluem you get that prompt | |
| #openstack-nova - 2026-05-29 | |||
| 02:25:40 | opendevreview | huanhongda proposed openstack/nova master: Fix scheduler aggregate cache race on concurrent add_host https://review.opendev.org/c/openstack/nova/+/990437 | |
| 09:29:55 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron extensions https://review.opendev.org/c/openstack/nova/+/962270 | |
| 09:31:19 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron floating IPs https://review.opendev.org/c/openstack/nova/+/962604 | |
| 09:35:25 | opendevreview | Balazs Gibizer proposed openstack/nova master: Poison eventlet import in native threading mode https://review.opendev.org/c/openstack/nova/+/986282 | |
| 11:02:46 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron Ports https://review.opendev.org/c/openstack/nova/+/969298 | |
| 11:03:00 | opendevreview | Lajos Katona proposed openstack/nova master: Use SDK for Neutron security-groups https://review.opendev.org/c/openstack/nova/+/981141 | |
| 12:02:35 | sean-k-mooney | gibi: thanks for reviewign the repoducer, will you have time to review https://review.opendev.org/c/openstack/nova/+/975872 today or next week im hoping between you an melwitt i can close that out soon | |
| 13:34:46 | opendevreview | Masanori Ueno proposed openstack/nova master: NUMA live-migration: ensure allocation_ratio is respected https://review.opendev.org/c/openstack/nova/+/990648 | |
| 13:42:21 | songwenping | sean-k-mooney: hi, is NVIDIA A100 MIG devices can passthrough to KVM VM? | |
| 13:54:44 | sean-k-mooney | songwenping: yes and no :) | |
| 13:55:11 | sean-k-mooney | songwenping: it depenson who waht you eman by passthoughah and your driver version/kernel | |
| 13:55:26 | sean-k-mooney | songwenping: yes if you are creating an mdev on top of the mig vf | |
| 13:56:08 | sean-k-mooney | i dont know if a100 suport the newer way fo using a nvidaia vfio-pci variatnt driver | |
| 13:56:12 | sean-k-mooney | to allow direct assignment | |
| 13:56:16 | sean-k-mooney | of the vf | |
| 13:56:27 | sean-k-mooney | that is supproted on blackwell and hopper i belive | |
| 14:06:03 | songwenping | yes we use vf+mdev | |
| 14:07:25 | songwenping | we did it at 2022,but now it is not worked. | |
| 14:08:23 | songwenping | our qemu-kvm version is 4.2.0, the version of 6.2.0 is not support, i wonder if the qemu is changed. | |
| 14:46:44 | sean-k-mooney | i dont belive this is a qemu chage nessiarly but it might eb a direr one | |
| 14:47:37 | sean-k-mooney | bauzas: prably know more but i think some verion fo the nvida driver (grid 20?) may have added the vfrio variant driver code path for a100 | |
| 14:47:45 | sean-k-mooney | i dont have acess to a a100 to test | |
| 14:48:47 | sean-k-mooney | im not aware of anythin in nova/libvirt/qemu that woudl have broken this however so i stongly suspect if it has broken in your env its related to a change made by nvidia | |
| 14:49:46 | sean-k-mooney | if it is because they remvoed mdev suprpot form the driver then you need this featrue https://specs.openstack.org/openstack/nova-specs/specs/2025.1/implemented/enable-vfio-devices-with-kernel-variant-drivers.html instaead | |
| 14:53:17 | opendevreview | sean mooney proposed openstack/nova master: Add agentic coding guidance and docs https://review.opendev.org/c/openstack/nova/+/984540 | |
| 14:57:06 | opendevreview | Joan Gilabert proposed openstack/nova-specs master: Repropose and update cyborg vGPU (mdev) support https://review.opendev.org/c/openstack/nova-specs/+/967515 | |
| 15:59:23 | masahito | hi nova team, please review the small lock API bug fix patch. https://review.opendev.org/c/openstack/nova/+/946223/5 thanks you | |
| 16:01:35 | opendevreview | Dan Smith proposed openstack/nova master: Add unpin AZ functionality https://review.opendev.org/c/openstack/nova/+/990682 | |
| 16:17:23 | sean-k-mooney | masahito: just reading the commit message that seams reasonable to me | |
| 16:18:02 | sean-k-mooney | i woudl have to check exactly what we docuemetn the preconditon fo the lock api as bu tif we have not finished spawning a vm for the first time i agree we shoudl nto allow you to lock it | |
| 16:18:29 | sean-k-mooney | we need the inaly isntce create to either finush successfuly or at least compel in the error state | |
| 16:18:41 | sean-k-mooney | before locking the isnstnace is really valid | |
| 16:20:05 | sean-k-mooney | we dont expclity docuemn a precondition but a pre condition for any server actrion is impleicly we have complete the inical server creation | |
| 16:21:04 | sean-k-mooney | if that is still inpugres we shoudl not allow you do do any server action api on teh instnace until the create is complete | |
| 16:52:51 | opendevreview | sean mooney proposed openstack/nova master: Use CirrOS test image in Nova CI jobs https://review.opendev.org/c/openstack/nova/+/990728 | |
| 16:52:52 | opendevreview | sean mooney proposed openstack/nova master: Revert "Use CirrOS test image in Nova CI jobs" https://review.opendev.org/c/openstack/nova/+/990729 | |
| 20:59:28 | opendevreview | Adam McArthur proposed openstack/placement master: Placement spec for bulk read API of resource providers https://review.opendev.org/c/openstack/placement/+/990761 | |
| 21:10:30 | melwitt | priteau: hey for the test_resize_reschedule_uses_host_lists_3_fails fail you see on stable/2026.1 you will want https://review.opendev.org/c/openstack/nova/+/989619 | |
| 21:10:52 | melwitt | cc elodilles ^ | |
| 21:20:35 | opendevreview | melanie witt proposed openstack/nova master: Fix swap disk creation skipped on NFS during cold migration https://review.opendev.org/c/openstack/nova/+/988547 | |
| 21:25:35 | melwitt | sean-k-mooney: I applied your suggested edit on the NFS file check thing ^ sorry I didn't see it earlier :( | |