| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2025-07-08 | |||
| 16:53:27 | gibi | traits, resource_class, otu are clearly not filtering anything | |
| 16:53:45 | gibi | they are just enhancing the already filtered data | |
| 16:53:46 | sean-k-mooney | correct but they shoudl only be applied to the same devspec that they are defiend on | |
| 16:54:04 | gibi | and they do | |
| 16:54:24 | sean-k-mooney | ok so your not advocating for somethign like {managed=no} | |
| 16:54:29 | sean-k-mooney | to set that on all devices | |
| 16:54:53 | sean-k-mooney | like that is an invalid devspec form my perspetive | |
| 16:55:21 | gibi | if you say {managed=no} then your dev_spec is matching to all devices today and enhancing them with the managed flag | |
| 16:55:35 | sean-k-mooney | ya so that is not valid in my opion | |
| 16:55:49 | sean-k-mooney | that is a very severy bug that should prevent startup | |
| 16:55:49 | gibi | same as {address="", managed=no} | |
| 16:55:55 | sean-k-mooney | which is also not valid | |
| 16:56:18 | gibi | does {address="0000", managed=no} valid? | |
| 16:56:29 | gibi | it is accepted today | |
| 16:56:34 | gibi | and matching most of the devices :) | |
| 16:56:40 | sean-k-mooney | if you read our docs then no | |
| 16:56:49 | sean-k-mooney | and i don tcondier what works to defien the contact | |
| 16:56:52 | gibi | so the docs is incomplete | |
| 16:56:55 | sean-k-mooney | wha twe wrote in the specs does | |
| 16:57:06 | gibi | then we would never fix a doc bug :) | |
| 16:57:07 | bauzas | I agree with gibi here | |
| 16:57:14 | bauzas | we need a doc, please | |
| 16:57:16 | sean-k-mooney | the docs are correct | |
| 16:57:20 | sean-k-mooney | the code is wrong | |
| 16:57:35 | gibi | anyhow. We need a list of what is supported and what is not supported and we need to agree on it | |
| 16:57:42 | gibi | then we can fix whatever need fixing | |
| 16:57:52 | gibi | we currently cannot agree on what is supported and what is not | |
| 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 | Uggla | #endmeeting | |
| 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 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2025/nova.2025-07-08-16.00.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 | Log: https://meetings.opendev.org/meetings/nova/2025/nova.2025-07-08-16.00.log.html | |
| 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: 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 | |