Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2025-07-08
16:49:14 gibi device_spec = { "vendor_id":"8086", "product_id":"10c9", "address": "0000:0c:"}
16:49:15 sean-k-mooney i would consieder supprot {} as a wild card to be a feature nto a bug
16:49:18 gibi this works today
16:49:18 Uggla ok so, can we consider it triaged or valid ?
16:49:43 Uggla As I understand something is required here.
16:49:53 sean-k-mooney Uggla: we both agree its valid but not what the actull issue is and the actual fix
16:49:55 bauzas let's just document this
16:50:04 sean-k-mooney i consider it a input validation bug
16:50:10 bauzas at least for now
16:50:22 sean-k-mooney im not okk wiht documenting it as supproted
16:50:37 sean-k-mooney if we want to say that it work but is not supproted andy you cant rely on it then sure
16:50:55 sean-k-mooney but this was never inteded to work so im not oke with commiting to supprot it going forward
16:50:57 bauzas sean-k-mooney: I'm not saying we should document it as "it's supported"
16:51:18 bauzas I just want to document something like "note: if you use {}, then blah"
16:51:31 bauzas or "bug note" if you prefer
16:51:40 bauzas or known issue
16:51:50 sean-k-mooney i can by into this as a known issue
16:52:05 gibi My original plan was to describe the two types of dev_spec fields. The one that we use to filter the devices and the one that we use to enhance the devices. Then in the filtering fields I would go an detail how partial match works. But based on Sean's view I would go -1 or -2 so I'm not going to do it
16:52:30 sean-k-mooney gibi: those are not seperate things
16:52:44 sean-k-mooney you are to enrich devices sepereatly form filtering
16:53:01 sean-k-mooney there are ways to do that today beucase of bug
16:53:19 sean-k-mooney but that was not how this was intened to work form my perspective
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 :)

Earlier   Later