| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2026-07-13 | |||
| 16:53:55 | gibi | I think it might qualify for specless in Indri but we have to wait | |
| 16:54:18 | gibi | anything else before we close the meeting? | |
| 16:54:29 | tkajinam | nothing else from me ! | |
| 16:55:01 | gibi | #endmeeting | |
| 16:55:01 | opendevmeet | Meeting ended Mon Jul 13 16:55:01 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:55:01 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2026/nova.2026-07-13-16.00.html | |
| 16:55:01 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2026/nova.2026-07-13-16.00.txt | |
| 16:55:01 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2026/nova.2026-07-13-16.00.log.html | |
| 16:55:05 | gibi | thanks folks | |
| 16:55:11 | gibi | antia: we can continue | |
| 16:55:19 | elodilles | thanks too o/ | |
| 16:55:21 | tkajinam | thanks ! | |
| 16:59:10 | antia | gibi: here is gitlog and gerrit warning https://paste.openstack.org/show/b3hGRYq7qtxpuY2DRUpk/ | |
| 16:59:27 | sean-k-mooney | gibi: tkajinam so speaking of conflict im inclinet too say since i need to rebase my change anyway lets just land https://review.opendev.org/c/openstack/nova/+/994342 | |
| 16:59:42 | sean-k-mooney | so im going to go review that properly now with that in mind | |
| 16:59:58 | sean-k-mooney | well | |
| 17:00:26 | sean-k-mooney | im still not sure about the new is_supported_mem_encryption_model fucntion and if its ok to remvoe that again | |
| 17:01:23 | tkajinam | checking | |
| 17:02:25 | sean-k-mooney | antia: so if you did "git review -R" that disables rebasing in git review | |
| 17:02:35 | gibi | eb455009ac this is an old version of tkajinam's series | |
| 17:02:43 | sean-k-mooney | antia: so provided you didnt already rebase the inital commits in the series they wont be updated | |
| 17:03:03 | antia | I need to update this based on the changes | |
| 17:03:51 | gibi | (in pure git without git review, this means pulling the ps 13 from https://review.opendev.org/c/openstack/nova/+/994764 and rebasing your commit on top) | |
| 17:04:12 | tkajinam | sean-k-mooney, hmm a problem is that memory encryption support is not in driver capabilities yet. we could add it but that'd be removed later once start up check is delegated to drivers. | |
| 17:05:43 | antia | yes I rebased in pure git before so it is just about rebasing with the new ps | |
| 17:05:50 | tkajinam | sean-k-mooney, are you ok if I propose a follow-up to add that capability ? though I'd abandon that once your validation delegation is merged. | |
| 17:06:28 | sean-k-mooney | adding and removing a cpablity is cheaper then a dirver function | |
| 17:06:51 | sean-k-mooney | but once we add and remove either i think and dont do a release in between we are sort of ok | |
| 17:07:19 | sean-k-mooney | there wont have been a release for any one to depend on it | |
| 17:07:30 | sean-k-mooney | so no need for deprecation of your new methond | |
| 17:07:52 | sean-k-mooney | i just want to make sure other agree that i can jsut delete is as part fo the refactor | |
| 17:15:19 | gibi | antia: yepp one more rebase :) | |
| 17:16:16 | antia | I think it is clear then :) | |
| 17:16:44 | antia | I'll ask again if I run into something else | |
| 17:18:08 | gibi | antia: sure | |
| 17:36:40 | tkajinam | https://zuul.opendev.org/t/openstack/builds?job_name=nova-alt-configurations&project=openstack/nova | |
| 17:37:09 | tkajinam | it seems nova-alt-configurations POST_FAILURE is now broken now due to missing cinder service | |
| 17:42:47 | tkajinam | https://bugs.launchpad.net/python-openstackclient/+bug/2160553 | |
| 17:45:06 | melwitt | OSC server create requires cinder? that seems ... wrong | |
| 17:47:59 | tkajinam | yeah | |
| 17:48:37 | tkajinam | I roughly suspect it is an regression caused by https://review.opendev.org/c/openstack/python-openstackclient/+/994996 | |
| 17:54:42 | opendevreview | Takashi Kajinami proposed openstack/nova master: DNM: Testing https://review.opendev.org/c/openstack/nova/+/997104 | |
| 17:55:24 | tkajinam | https://review.opendev.org/c/openstack/python-openstackclient/+/996626 | |
| 17:55:30 | tkajinam | oh slawek already proposed a fix | |
| 17:56:33 | melwitt | ahh nice | |
| 18:03:03 | opendevreview | sean mooney proposed openstack/nova stable/2026.1: Make QEMU_IMG_LIMITS process limits configurable https://review.opendev.org/c/openstack/nova/+/997027 | |
| 18:03:19 | sean-k-mooney | melwitt: elodilles if that work for you ill update the rest | |
| 18:03:46 | sean-k-mooney | although ill probly to the rest tomorrow | |
| 18:03:53 | sean-k-mooney | so no rush | |
| 18:05:12 | melwitt | cool thanks sean-k-mooney, will review | |
| 18:31:24 | opendevreview | Merged openstack/nova master: Speed up functional test in threading mode https://review.opendev.org/c/openstack/nova/+/996500 | |
| 20:05:10 | opendevreview | sean mooney proposed openstack/nova master: Delegate startup instance processing to drivers https://review.opendev.org/c/openstack/nova/+/990552 | |
| 20:07:43 | opendevreview | sean mooney proposed openstack/nova master: Add vTPM startup validation reproducer https://review.opendev.org/c/openstack/nova/+/990551 | |
| 20:07:44 | opendevreview | sean mooney proposed openstack/nova master: Delegate startup instance processing to drivers https://review.opendev.org/c/openstack/nova/+/990552 | |
| 20:07:44 | opendevreview | sean mooney proposed openstack/nova master: Reduce ProviderTree copy cost for Ironic https://review.opendev.org/c/openstack/nova/+/980676 | |
| 20:08:29 | opendevreview | sean mooney proposed openstack/nova master: Parallelize per-node resource updates https://review.opendev.org/c/openstack/nova/+/980679 | |
| 20:08:59 | opendevreview | sean mooney proposed openstack/nova master: perf: add ironic startup performance simulator https://review.opendev.org/c/openstack/nova/+/980641 | |
| 20:09:00 | opendevreview | sean mooney proposed openstack/nova master: docs: add Ironic startup performance measurements https://review.opendev.org/c/openstack/nova/+/980680 | |
| 20:25:59 | JayF | dansmith: if you get an extra moment; https://review.opendev.org/c/openstack/ironic/+/997105 is the corollary fix in Ironic for OSSN-0101 if you wanna take a look. Just wanted to let you know since you authored the Nova one. | |
| 22:51:15 | gmaan | gibi: sean-k-mooney: thanks for the more details in bug and I am good on the attachment_id 404 case and it does not need microverison. The only use case I was thinking was if user monitoring it to know if the attachment is ready but that is not valid or guranteed case. So returning attachment_id=None is not actually a success case even return code is 200. -1 on that change only for the test cases | |
| 22:51:15 | gmaan | https://review.opendev.org/c/openstack/nova/+/996757 | |
| 23:01:53 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Add doc and release notes for the graceful shutdown task tracking https://review.opendev.org/c/openstack/nova/+/997141 | |
| 23:34:47 | opendevreview | KWAK JOONGKEE proposed openstack/nova master: policy: initialize enforcer before publishing it https://review.opendev.org/c/openstack/nova/+/997142 | |
| #openstack-nova - 2026-07-14 | |||
| 00:42:51 | opendevreview | Ghanshyam Maan proposed openstack/nova master: Add doc and release notes for the graceful shutdown task tracking https://review.opendev.org/c/openstack/nova/+/997141 | |
| 00:42:56 | gmaan | dansmith: melwitt: graceful shutdown changes are ready for review https://review.opendev.org/q/topic:%22bp/nova-services-graceful-shutdown-part2-task-tracking%22 | |
| 02:50:03 | opendevreview | yaguang proposed openstack/nova master: libvirt: enable Hyper-V evmcs on Intel VMX hosts https://review.opendev.org/c/openstack/nova/+/995620 | |
| 04:20:18 | opendevreview | Masahito Muroi proposed openstack/nova master: Add bug 2106085 reproducer test case https://review.opendev.org/c/openstack/nova/+/971153 | |
| 04:20:18 | opendevreview | Masahito Muroi proposed openstack/nova master: Add check_instance_state validation to the four action API https://review.opendev.org/c/openstack/nova/+/946223 | |
| 04:37:36 | opendevreview | Bhavana proposed openstack/nova master: Add custom GPU PCI weigher for scheduler optimization Add Minimal Displacement Solver for MIG GPU Repartitioning https://review.opendev.org/c/openstack/nova/+/996058 | |
| 07:26:59 | ralonsoh | hi folks, if you have a few minutes, please check https://review.opendev.org/c/openstack/os-vif/+/995933 | |
| 07:27:14 | ralonsoh | dansmith, ^ thanks in advance | |
| 07:58:36 | gibi | gmaan: thanks I will check your feedback | |
| 09:03:19 | gibi | sean-k-mooney: elodilles: the oslo.concurrency 7.6.0 cross test failure turns out to be a real nova bug we now see but was hidden before https://bugs.launchpad.net/nova/+bug/2160596 Two VMs on the host using the same multi attach volume being deleted in parallel can result in danging volume attachments | |
| 09:03:34 | gibi | as we are not serializing the deletion by volume id just by VM id | |
| 09:04:32 | gibi | as the attachment delete (in the manager) is far from the volume disconnect check (in the driver) it is not easy to just add a new lock. | |
| 09:17:23 | gibi | not dangling volume attachements, but dangling volume host mounts | |
| 09:36:48 | elodilles | gibi: ah, ACK, thanks for the information | |
| 09:46:41 | sean-k-mooney | gibi: hum i really tought i had properly fixed that | |
| 09:46:50 | sean-k-mooney | gibi: did we regress it | |
| 09:47:36 | sean-k-mooney | gibi: that what i was fixing in https://bugs.launchpad.net/nova/+bug/2048837 | |
| 09:48:26 | sean-k-mooney | gibi: if this only happens with the new oslo.concurrency then its a but in oslo | |
| 09:49:38 | sean-k-mooney | gibi: i added per volume locking in https://opendev.org/openstack/nova/commit/72132f89ee4ca7367a6a5639f763f9b20d2c441a | |
| 09:50:19 | sean-k-mooney | sorry in https://opendev.org/openstack/nova/commit/22012360c40045ac1bf6dd0dc95aea3e15fed0cd | |
| 09:52:18 | gibi | I have to look deeper as my simple reproduction does not work without the oslo change so something important changed in oslo | |
| 10:20:20 | gibi | the only real change is oslo.concurrency is eventlet related | |
| 10:23:46 | gibi | reverting https://review.opendev.org/c/openstack/oslo.concurrency/+/988652 stops my reproduction to recreate the issue | |
| 10:24:53 | sean-k-mooney | hum i looked at that and discounted it | |
| 10:25:13 | sean-k-mooney | i kwno that eventlet has import order dependent behvior | |
| 10:25:59 | sean-k-mooney | i dont know why that would break locking | |
| 10:26:11 | gibi | yeah me neither but I will dig further | |
| 10:26:14 | sean-k-mooney | espically when we are not chanign the patching | |
| 10:28:01 | sean-k-mooney | gibi: does the cross job run wiht our without monkey patching? | |
| 10:28:20 | sean-k-mooney | i assume with eventlet for now | |
| 10:28:30 | sean-k-mooney | since that stil technially our default | |
| 10:30:04 | sean-k-mooney | i think this this early out https://review.opendev.org/c/openstack/oslo.concurrency/+/988652/1/oslo_concurrency/lockutils.py#111 | |
| 10:30:25 | sean-k-mooney | that is shared across all locks | |
| 10:30:35 | sean-k-mooney | but https://review.opendev.org/c/openstack/oslo.concurrency/+/988652/1/oslo_concurrency/lockutils.py#131 | |
| 10:30:54 | sean-k-mooney | need to run for all rwlocks when we are patched | |
| 10:31:07 | sean-k-mooney | withihs chagne it will hapen for the first lock but not the rest | |
| 10:31:26 | sean-k-mooney | so _EVENTLET_CHECKED should just be removed | |