| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-07-08 | |||
| 17:05:06 | sean-k-mooney | i.e. if this si between netapp san and ceph ectra | |
| 17:05:44 | sean-k-mooney | this is realted to how we store the nvram | |
| 17:06:10 | sean-k-mooney | but lets loop back to this when we have had tim to look ath this properly | |
| 17:06:28 | sean-k-mooney | there is potically some race condtions happenign here too so i think we need to look at this in detail | |
| 17:07:43 | Uggla | sean-k-mooney, no problem. I agree the bug is not accurate enough. But the description of the patch looks clearer. | |
| 17:08:40 | Uggla | I think I'll ask for a reproducer. Because that will clarify the need. | |
| 17:10:57 | sean-k-mooney | i think i know why this si happeing and i think its a side aeffect to a previous incorrect bugfix | |
| 17:11:58 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/922354 | |
| 17:12:45 | sean-k-mooney | that reverts https://review.opendev.org/c/openstack/nova/+/906380 which was written for https://launchpad.net/bugs/1997352 but i dont think that patchs was correct | |
| 17:13:32 | sean-k-mooney | there fix is properly makign the nvram deletion contionall https://review.opendev.org/c/openstack/nova/+/954210/2/nova/virt/libvirt/guest.py | |
| 17:13:51 | sean-k-mooney | and making sure we dont delete it as part of swap volume | |
| 17:14:10 | sean-k-mooney | but we also shoudl not be undefining it in most cases | |
| 17:14:43 | sean-k-mooney | the only time its valid to delete it is when deleting the vm or when rebuilding form uefi ot non uefi | |
| 23:25:00 | gmaan | sean-k-mooney: stephenfin: manager role code is ready, all tempest test are also green https://review.opendev.org/q/topic:%22bp/policy-manager-role-default%22+status:open | |
| 23:26:49 | sean-k-mooney | awsome ill try and do a pass tomorow | |
| #openstack-nova - 2025-07-09 | |||
| 00:38:21 | gmaan | thanks | |
| 07:20:17 | opendevreview | benlei proposed openstack/nova master: FIX: instance live migration failed after swap volume https://review.opendev.org/c/openstack/nova/+/954210 | |
| 09:52:52 | opendevreview | Kamil Sambor proposed openstack/nova master: Replace eventlet.event.Event with threading.Event https://review.opendev.org/c/openstack/nova/+/949754 | |
| 12:05:33 | opendevreview | Rajesh Tailor proposed openstack/nova master: Add support for using cell-name in cell_v2 commands https://review.opendev.org/c/openstack/nova/+/954460 | |
| 13:02:42 | gibi | folks I don't have news / agenda for the eventlet meeting staring in 90 minutes. If you have feel free to add it. But if not then I will expect a short meeting today. | |
| 13:17:53 | sambork | gibi I added my update but it shouldn't take too much except maybe discussion/question about timeout replacement | |
| 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 | |