| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-07-08 | |||
| 16:57:56 | gibi | so I suggest tabling this | |
| 16:58:09 | sean-k-mooney | by the way {address="0000", managed=no} is technially valid today | |
| 16:58:14 | gibi | maybe a PTG discussion is in order with preparation listing all the cases that accepted today | |
| 16:58:27 | sean-k-mooney | but that only because we supprot partlial adresses | |
| 16:58:48 | Uggla | Can we move to the next one. As I'm unsure we manage to get out with something here. | |
| 16:58:53 | gibi | so address="0" is also a partial address as well as "" | |
| 16:58:59 | gibi | where is the line? | |
| 16:59:18 | sean-k-mooney | we draw it based on what we agreed in the specs orginally | |
| 16:59:47 | sean-k-mooney | and if thre was a bug in the spec we fix it with a new one or a bug that we dicusss in detail | |
| 17:00:05 | gibi | I agree that we need to discuss in detail | |
| 17:00:26 | Uggla | yep please can we stop on that one. | |
| 17:00:48 | gibi | we need a detailed plan what to rip off we cannot rip off address="" support if we keep address="0" I tink | |
| 17:00:50 | bauzas | I need to leave now | |
| 17:00:53 | bauzas | o/ | |
| 17:01:21 | Uggla | Can we have a look at the latest one: https://bugs.launchpad.net/nova/+bug/2115870 I guess it is valid looking aat the patch attached. | |
| 17:02:10 | gibi | I have no context on volume retype | |
| 17:02:33 | sean-k-mooney | lets leave that for now and come back next week | |
| 17:03:08 | Uggla | ok end for today. | |
| 17:03:29 | Uggla | Thanks all | |
| 17:03:35 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-07-08-16.00.log.html | |
| 17:03:35 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2025/nova.2025-07-08-16.00.txt | |
| 17:03:35 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-07-08-16.00.html | |
| 17:03:35 | opendevmeet | Meeting ended Tue Jul 8 17:03:35 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 17:03:35 | Uggla | #endmeeting | |
| 17:04:12 | sean-k-mooney | Uggla: for what its worth im not sure we have enough info in the bug to triage that | |
| 17:04:42 | sean-k-mooney | we dont know the souce/dest volume types, we dont knwo if it just change the qos ro actull moved data | |
| 17:05:00 | elodilles | thanks o/ | |
| 17:05:03 | Uggla | sean-k-mooney, I guess the intend is clearer in the patch. | |
| 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: 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 | |
| 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 '[api] auth_strategy', NoAuthMiddlware https://review.opendev.org/c/openstack/nova/+/952272 | |
| 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:05 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: api: Remove version controller split https://review.opendev.org/c/openstack/nova/+/954103 | |
| 10:24:05 | opendevreview | Stephen Finucane proposed openstack/nova master: api: Deprecate v2 API https://review.opendev.org/c/openstack/nova/+/954102 | |
| 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 group APIs https://review.opendev.org/c/openstack/nova/+/952281 | |
| 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: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: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: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: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: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: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: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: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: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: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: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: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: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 | |
| 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 | |
| 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 | |