| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-07-08 | |||
| 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 | |
| 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. | |