| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-07-09 | |||
| 13:17:56 | gibi | OK I see topics fro sambork | |
| 13:18:04 | gibi | :) | |
| 14:28:37 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Use updated_at to purge shadow_instance_extra https://review.opendev.org/c/openstack/nova/+/953984 | |
| 14:28:38 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Add a functional test to reproduce bug #2111129 https://review.opendev.org/c/openstack/nova/+/954504 | |
| 14:31:04 | gibi | eventlet sync https://meet.google.com/bcy-uqoz-hje | |
| 14:32:55 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Use updated_at to purge shadow_instance_extra https://review.opendev.org/c/openstack/nova/+/953984 | |
| 14:38:16 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Use created_at to purge shadow_instance_extra https://review.opendev.org/c/openstack/nova/+/953984 | |
| 15:03:23 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Add a functional test to reproduce bug #2111129 https://review.opendev.org/c/openstack/nova/+/954504 | |
| 15:03:23 | opendevreview | Alexey Stupnikov proposed openstack/nova master: Use created_at to purge shadow_instance_extra https://review.opendev.org/c/openstack/nova/+/953984 | |
| 23:31:26 | melwitt | Uggla: just wanted to let you know about: #opendev peeps ran into this bug recently https://bugs.launchpad.net/nova/+bug/1856329 while troubleshooting some zuul provider errors. there is an old patch proposed here for it that could use some review https://review.opendev.org/c/openstack/nova/+/699176 | |
| 23:34:05 | melwitt | related discussion: https://meetings.opendev.org/irclogs/%23opendev/%23opendev.2025-07-09.log.html#opendev.2025-07-09.log.html#t2025-07-09T20:35:52 | |
| #openstack-nova - 2025-07-10 | |||
| 07:50:53 | Uggla | Thank you melwitt | |
| 09:24:00 | opendevreview | ribaudr proposed openstack/nova stable/2025.1: FUP Update pci-passthrough and virtual-gpu documentation https://review.opendev.org/c/openstack/nova/+/948254 | |
| 10:24:03 | opendevreview | Stephen Finucane proposed openstack/nova master: conf: Rename '[api] neutron_default_tenant_id' https://review.opendev.org/c/openstack/nova/+/952270 | |
| 10:24:04 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove deprecated pipeline_factory https://review.opendev.org/c/openstack/nova/+/952271 | |
| 10:24:04 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Remove '[api] auth_strategy', NoAuthMiddlware https://review.opendev.org/c/openstack/nova/+/952272 | |
| 10:24:05 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Deprecate v2 API https://review.opendev.org/c/openstack/nova/+/954102 | |
| 10:24:05 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: api: Remove version controller split https://review.opendev.org/c/openstack/nova/+/954103 | |
| 10:40:43 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server external events API https://review.opendev.org/c/openstack/nova/+/940508 | |
| 10:40:44 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server password APIs https://review.opendev.org/c/openstack/nova/+/945736 | |
| 10:40:44 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server group APIs https://review.opendev.org/c/openstack/nova/+/952281 | |
| 10:40:45 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Address issues with server group APIs https://review.opendev.org/c/openstack/nova/+/953281 | |
| 10:40:45 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for removed APIs https://review.opendev.org/c/openstack/nova/+/952282 | |
| 10:40:46 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for networks API https://review.opendev.org/c/openstack/nova/+/952283 | |
| 10:40:46 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for images APIs https://review.opendev.org/c/openstack/nova/+/952284 | |
| 10:40:47 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Separate volume, snapshot and volume attachments https://review.opendev.org/c/openstack/nova/+/952347 | |
| 10:40:47 | opendevreview | Stephen Finucane proposed openstack/nova master: tests: Use valid UUIDs for cinder resources https://review.opendev.org/c/openstack/nova/+/952935 | |
| 10:40:49 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Only apply "soft" additionalProperties validation to requests https://review.opendev.org/c/openstack/nova/+/952936 | |
| 10:40:49 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volumes APIs https://review.opendev.org/c/openstack/nova/+/952348 | |
| 10:40:51 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for snapshots APIs https://review.opendev.org/c/openstack/nova/+/952349 | |
| 10:40:51 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for volume attachments APIs https://review.opendev.org/c/openstack/nova/+/952350 | |
| 10:40:53 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for floating IP APIs https://review.opendev.org/c/openstack/nova/+/952972 | |
| 10:40:53 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for security group APIs https://review.opendev.org/c/openstack/nova/+/952973 | |
| 10:40:55 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for services APIs https://review.opendev.org/c/openstack/nova/+/953196 | |
| 10:40:55 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Add response body schemas for server usage audit log APIs https://review.opendev.org/c/openstack/nova/+/953209 | |
| 13:15:09 | opendevreview | Merged openstack/nova master: api: Address issues with keypairs APIs https://review.opendev.org/c/openstack/nova/+/952214 | |
| 13:15:21 | opendevreview | Merged openstack/nova stable/2025.1: FUP Update pci-passthrough and virtual-gpu documentation https://review.opendev.org/c/openstack/nova/+/948254 | |
| 13:24:37 | opendevreview | Markus Hentsch proposed openstack/nova master: Adapt to Cinder and Glance changes for image encryption https://review.opendev.org/c/openstack/nova/+/926326 | |
| 13:37:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: [pci]Keep used dev in Placement regardless of dev_spec https://review.opendev.org/c/openstack/nova/+/954149 | |
| 13:37:57 | opendevreview | Balazs Gibizer proposed openstack/nova master: [PCI tracker]Remove non configured devs when freed https://review.opendev.org/c/openstack/nova/+/954613 | |
| 13:57:42 | gibi | ^^ this was a tough one | |
| 14:18:54 | sean-k-mooney | ah i see the edge case this adresed | |
| 14:35:01 | gibi | sambork: FYI https://review.opendev.org/c/openstack/nova/+/954324/comment/41b996aa_19440791/ | |
| 14:44:46 | sambork | gibi, thanks I will take a look | |
| 14:50:41 | sambork | "It generally looks like the timeout work needs to be done in the same patch to ensure that we raise the error at the same level and update _InstanceEvent accordingly | |
| 14:53:55 | gibi | OK I can live with that :) | |
| 14:57:26 | opendevreview | Takashi Kajinami proposed openstack/nova master: sqlalchemy: Use built-in declarative https://review.opendev.org/c/openstack/nova/+/953751 | |
| 15:36:30 | sean-k-mooney | sambork: kamil is this your upstream nic? | |
| 17:22:39 | gmaan | sean-k-mooney[m]: thanks for review on manager role. I agree to merge both change (policy granularity and default change to manager). I replied to this comment, if that make sense - https://review.opendev.org/c/openstack/nova/+/953063/7/nova/policies/migrate_server.py#101 | |
| 17:23:34 | gmaan | oslo.policy implementation does not cover the policy granularity which is something we can implement in future but I used the deprecated_rule way so that we can handle the operator use case | |
| 17:25:39 | gmaan | I am mentioning it in deprecation_reason that policy is not deprecated and used to handle the custom policy case | |
| 17:32:19 | gmaan | sean-k-mooney[m]: I notice that we still have time for oslo lib freeze for this cycle, so let me try implementing the granular policy case in oslo.policy and then our implementation will looks good. | |
| 17:32:29 | gmaan | and that will handle the future case of policy granularity also | |
| 18:00:41 | opendevreview | Merged openstack/nova master: Add spawn_on https://review.opendev.org/c/openstack/nova/+/948079 | |
| #openstack-nova - 2025-07-11 | |||
| 04:17:22 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/954688 | |
| 06:54:18 | sambork | sean-k-mooney[m], yes thats my nick | |
| 08:59:43 | opendevreview | benlei proposed openstack/nova master: FIX: instance live migration failed after swap volume https://review.opendev.org/c/openstack/nova/+/954210 | |
| 09:02:58 | opendevreview | benlei proposed openstack/nova master: FIX: instance live migration failed after swap volume https://review.opendev.org/c/openstack/nova/+/954210 | |
| 09:07:31 | opendevreview | benlei proposed openstack/nova master: FIX: instance live migration failed after swap volume https://review.opendev.org/c/openstack/nova/+/954210 | |
| 09:15:04 | jssfr | Fun question. I'm looking at a Nova 2023.1 installation with vTPM enabled. As far as I can tell, the vTPM state gets completely deleted on VM power on, e.g. after poweroff from within the VM. The code path leading up to this is apparently power_on -> driver._hard_reboot -> driver.destroy -> driver.cleanup -> driver._cleanup -> libvirt undefine, and libvirt drops the swtpm directory on undefine unless | |
| 09:15:04 | jssfr | `VIR_DOMAIN_UNDEFINE_KEEP_TPM` is passed, which isn't passed anywhere in nova. | |
| 09:15:12 | jssfr | am I holding this wrong? | |
| 09:17:11 | jssfr | meanwhile I'm going to try to reproduce this on 2024.1, but given that I traced back the codepath on 2024.1, I'm not hopeful... | |
| 11:55:14 | jssfr | I reproduced the same issue in 2024.1 :( | |
| 12:55:17 | opendevreview | Balazs Gibizer proposed openstack/nova master: FUP: Translate scatter-gather to futurist https://review.opendev.org/c/openstack/nova/+/953338 | |
| 13:48:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move ConductorManager to use spawn_on https://review.opendev.org/c/openstack/nova/+/948187 | |
| 13:48:58 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make nova.utils.pass_context private https://review.opendev.org/c/openstack/nova/+/948188 | |
| 13:48:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Rename DEFAULT_GREEN_POOL to DEFAULT_EXECUTOR https://review.opendev.org/c/openstack/nova/+/948086 | |
| 13:48:59 | opendevreview | Balazs Gibizer proposed openstack/nova master: Make the default executor configurable https://review.opendev.org/c/openstack/nova/+/948087 | |
| 13:49:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Print ThreadPool statistics https://review.opendev.org/c/openstack/nova/+/948340 | |
| 13:49:00 | opendevreview | Balazs Gibizer proposed openstack/nova master: Document native threading mode and tuneables https://review.opendev.org/c/openstack/nova/+/949364 | |
| 13:49:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow services to start with threading https://review.opendev.org/c/openstack/nova/+/948311 | |
| 13:49:01 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-next with n-sch in threading mode https://review.opendev.org/c/openstack/nova/+/948450 | |
| 13:49:02 | opendevreview | Balazs Gibizer proposed openstack/nova master: Do not yield in threading mode https://review.opendev.org/c/openstack/nova/+/950994 | |
| 13:49:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run nova-api and -metadata in threaded mode https://review.opendev.org/c/openstack/nova/+/951957 | |
| 13:49:03 | opendevreview | Balazs Gibizer proposed openstack/nova master: Warn on long task wait time for executor https://review.opendev.org/c/openstack/nova/+/952666 | |
| 13:49:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Allow to start unit test without eventlet https://review.opendev.org/c/openstack/nova/+/953436 | |
| 13:49:05 | opendevreview | Balazs Gibizer proposed openstack/nova master: Run unit test with threading mode https://review.opendev.org/c/openstack/nova/+/953475 | |
| 13:49:07 | opendevreview | Balazs Gibizer proposed openstack/nova master: [test]RPC using threading or eventlet selectively https://review.opendev.org/c/openstack/nova/+/953815 | |
| #openstack-nova - 2025-07-12 | |||
| 00:39:48 | melwitt | jssfr: it looks like that flag may be relatively new i.e. newer than when vTPM was added to nova https://patchew.org/Libvirt/20220822120554.3529999-1-stefanb@linux.ibm.com/20220822120554.3529999-7-stefanb@linux.ibm.com/ so it seems like it's something we need to add | |
| 06:48:12 | jssfr | melwitt, thanks for the feedback. so this fell on our feet because we have users with Windows 11 who enabled bitlocker and ... well, they're a bit sad now after they powered their VMs off ;). | |
| 06:48:25 | jssfr | we can't be the first with this type of usage, can we? | |
| #openstack-nova - 2025-07-14 | |||
| 07:04:31 | opendevreview | Takashi Kajinami proposed openstack/nova master: Remove unused utilities https://review.opendev.org/c/openstack/nova/+/954174 | |
| 07:04:32 | opendevreview | Takashi Kajinami proposed openstack/nova master: Migrate ExceptionHelper to test utilities https://review.opendev.org/c/openstack/nova/+/954176 | |
| 08:01:18 | gibi | The nova-ceph-multistore job is broken and blocking the gate since 2025-07-11 08:33:54 I reported the bug ttps://bugs.launchpad.net/nova/+bug/2116852 | |
| 08:43:38 | mikal | Uggla: what is your preferred way for me to solicit a second set of +2's on https://review.opendev.org/c/openstack/nova/+/926126 onwards? sean-k-mooney[m] has already +2'ed all four patches. | |
| 11:49:03 | opendevreview | Merged openstack/placement master: Replace deprecated assertItemsEqual https://review.opendev.org/c/openstack/placement/+/953582 | |
| 14:54:46 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: Force librbd1 19.2.0 to fix ceph job https://review.opendev.org/c/openstack/nova/+/954956 | |
| 15:23:25 | fungi | looks like the recent breaking change in cloud-init has reawakened interest in https://launchpad.net/nova/+bug/1663304 | |
| 15:38:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: DNM: Force librbd1 19.2.0 to fix ceph job https://review.opendev.org/c/openstack/nova/+/954956 | |
| 16:15:59 | dansmith | gibi: I think maybe adding a global monotonic deadline to the RecordSortContext that we can check and use to limit how long we run would probably be all we need there | |
| 16:16:29 | dansmith | the timer is mostly just to avoid the threads querying another batch if the whole process is taking too long | |
| 16:16:39 | dansmith | I can try to propose something if you want | |
| 16:17:41 | gibi | feel free to propose, I'm mostly done for the day anyhow. And I will start with running some local trials with this batching and shorting scatter gather locally hopefully tomorrow to understand it more | |
| 16:19:38 | gibi | I agree if we can check a global deadline before yielding the next item and exit instead that would workh | |
| 16:20:32 | dansmith | cool | |
| 17:16:01 | sean-k-mooney | fungi: ya.. the problem with that is they interface they are suign is kind of a private implation detail | |
| 17:16:15 | sean-k-mooney | fungi: so it snot clear that we shoudl do that | |
| 18:09:21 | opendevreview | Dan Smith proposed openstack/nova master: Remove eventlet timer from multi_cell_list https://review.opendev.org/c/openstack/nova/+/954990 | |
| 18:16:09 | opendevreview | Dan Smith proposed openstack/nova master: Remove eventlet timer from multi_cell_list https://review.opendev.org/c/openstack/nova/+/954990 | |