Thursday, 2024-10-10

sean-k-mooney[m]testing via thunderbird00:53
sean-k-mooneyseams to work00:53
opendevreviewAmit Uniyal proposed openstack/nova master: VMware: updates resource provider trait list  https://review.opendev.org/c/openstack/nova/+/92652202:42
opendevreviewAmit Uniyal proposed openstack/nova master: zvm: updates resource provider trait list  https://review.opendev.org/c/openstack/nova/+/92652302:42
opendevreviewAmit Uniyal proposed openstack/nova master: Ironic: updates resource provider trait list  https://review.opendev.org/c/openstack/nova/+/92652402:42
semanticHello! Could someone please look into https://review.opendev.org/c/openstack/nova/+/931737 ?08:34
opendevreviewribaudr proposed openstack/nova master: Amend ShareMappingStatus due to asynchronous call  https://review.opendev.org/c/openstack/nova/+/90886408:51
opendevreviewribaudr proposed openstack/nova master: Amend DB model add a unique constraint.  https://review.opendev.org/c/openstack/nova/+/91251808:51
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction)  https://review.opendev.org/c/openstack/nova/+/83119408:51
opendevreviewribaudr proposed openstack/nova master: Use client token when talking to manila  https://review.opendev.org/c/openstack/nova/+/92527708:51
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part)  https://review.opendev.org/c/openstack/nova/+/83309008:51
opendevreviewribaudr proposed openstack/nova master: Mounting the shares as part of the initialization process  https://review.opendev.org/c/openstack/nova/+/88007508:51
opendevreviewribaudr proposed openstack/nova master: Deletion of associated share mappings on instance deletion  https://review.opendev.org/c/openstack/nova/+/88147208:51
opendevreviewribaudr proposed openstack/nova master: Add metadata for shares  https://review.opendev.org/c/openstack/nova/+/85050008:51
opendevreviewribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part)  https://review.opendev.org/c/openstack/nova/+/85482308:51
opendevreviewribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute manager part)  https://review.opendev.org/c/openstack/nova/+/85482408:51
opendevreviewribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part)  https://review.opendev.org/c/openstack/nova/+/86028408:51
opendevreviewribaudr proposed openstack/nova master: Support resuming an instance with shares (compute manager part)  https://review.opendev.org/c/openstack/nova/+/86028508:51
opendevreviewribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares  https://review.opendev.org/c/openstack/nova/+/86028608:51
opendevreviewribaudr proposed openstack/nova master: Support rescuing an instance with shares  https://review.opendev.org/c/openstack/nova/+/86028708:51
opendevreviewribaudr proposed openstack/nova master: Allow to mount manila share using Cephfs protocol  https://review.opendev.org/c/openstack/nova/+/88386208:51
opendevreviewribaudr proposed openstack/nova master: Check shares support (compute manager)  https://review.opendev.org/c/openstack/nova/+/88575108:51
opendevreviewribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (API)  https://review.opendev.org/c/openstack/nova/+/83683008:51
opendevreviewribaudr proposed openstack/nova master: Add helper methods to attach/detach shares  https://review.opendev.org/c/openstack/nova/+/88575308:52
opendevreviewribaudr proposed openstack/nova master: Add instance.share_attach notification  https://review.opendev.org/c/openstack/nova/+/85050108:52
opendevreviewribaudr proposed openstack/nova master: Add instance.share_detach notification  https://review.opendev.org/c/openstack/nova/+/85102808:52
opendevreviewribaudr proposed openstack/nova master: Add shares to InstancePayload  https://review.opendev.org/c/openstack/nova/+/85102908:52
opendevreviewribaudr proposed openstack/nova master: Add instance.share_attach_error notification  https://review.opendev.org/c/openstack/nova/+/86028208:52
opendevreviewribaudr proposed openstack/nova master: Add instance.share_detach_error notification  https://review.opendev.org/c/openstack/nova/+/86028308:52
opendevreviewribaudr proposed openstack/nova master: Reports instance events to the DB regarding attaching and detaching a share  https://review.opendev.org/c/openstack/nova/+/92708808:52
opendevreviewribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working.  https://review.opendev.org/c/openstack/nova/+/85208608:52
opendevreviewribaudr proposed openstack/nova master: Add virt/libvirt error test cases  https://review.opendev.org/c/openstack/nova/+/85208708:52
opendevreviewribaudr proposed openstack/nova master: Manila shares admin guide documentation  https://review.opendev.org/c/openstack/nova/+/87164208:52
opendevreviewTakashi Natsume proposed openstack/python-novaclient master: Replace neutronclient with openstacksdk  https://review.opendev.org/c/openstack/python-novaclient/+/93205711:41
opendevreviewTakashi Natsume proposed openstack/python-novaclient master: Replace neutronclient with openstacksdk  https://review.opendev.org/c/openstack/python-novaclient/+/93205711:46
opendevreviewsean mooney proposed openstack/nova stable/2024.1: repoduce post liberty pre vicoria instance numa db issue  https://review.opendev.org/c/openstack/nova/+/93206412:38
opendevreviewsean mooney proposed openstack/nova stable/2024.1: allow upgrade of pre-victoria InstanceNUMACells  https://review.opendev.org/c/openstack/nova/+/93206512:38
sean-k-mooney1elodilles: ^ you have some downstream victoria clouds right. that would technially be useful for them but im hoping you dont have downstream pre victoria clouds that have nto upgraded to at least victoira so it might now be relevent anymore12:41
*** sean-k-mooney1 is now known as sean-k-mooney12:42
sean-k-mooneygiven how old that bug is and that it does not afffect any isntance after victoria im not very motivated to backport this upstream but just wanted to highlight it to you in case it was useful to you12:43
sean-k-mooneytkajinam: i think you are the 4th person to start replacing neutron clinet with the sdk if you have that working your amazing if you dont i guess that makes you human but still pretty awsome12:44
sean-k-mooneyoh your patches are for nova-client not nova 12:45
sean-k-mooneystill nice to see although im hoping we can remove nova-client in the next 1-2 years entirly12:45
tkajinamI'm not really touching "kill clients" so far but I know it may give me a lot of thing to do in heat :-P12:46
sean-k-mooneyit would be nice to start the converstaion "can we formally deprecate the python binding in 2025.1, for removal in 2026.1"12:46
sean-k-mooneytkajinam: i know neutorn want to kill there client in the next 1-2 slurps and i think we should try to do the same12:46
sean-k-mooneybut ya it wont happen for a while12:46
tkajinamI understand that point but honestly I'm not too sure about its feasibility12:47
tkajinamfor the past 1 year I'm almost the only person taking care of all heat repos.12:47
sean-k-mooneywell the only reaosn we didnt freeze the python bidnign before last cycle was osc12:47
sean-k-mooneythat is no longer a usecase as osc is now using the sdk12:48
tkajinamok12:48
sean-k-mooneyso i dont realy expect use to continue to extend the python binding in nova-clinet anymore for new api. just the sdk12:48
tkajinamI'm ok to freeze the client code now. I may need some more time before it is actually retired.12:48
tkajinamfollowing what was done for neutronclient in the past12:49
elodillessean-k-mooney: ACK, thanks, i'll review and see if i can backport :) maybe it can help others :]12:52
tkajinamas we expect some big changes needed for eventlet removal and py313 support I'm hoping that we don't add yet another big topic for a few cycles12:54
tkajinamthat's my own hope but people may want more short timeline for different things.12:54
sean-k-mooneywell the move to sdk started before the eventlet topic12:56
sean-k-mooneybut ya we will see how things pan out.12:57
sean-k-mooneyi know we are all pretty streched so we dont want to break things just for the sake of it12:57
sean-k-mooneywhat i would hope however is that heat for example could elelct to only use the sdk for new apis to not accure new tech debt12:57
sean-k-mooneyeven if it does not start the removal of the old client code in the near term12:58
sean-k-mooneyelodilles: alot of our downstream customer are startign to jump form train to wallaby so that bug is now finally manafesting. out side of enterpise distrobutions i would not expect many clouds to really be running pre victoira code today13:02
elodilleswell, we always see really old versions when community asks around about used versions in deployments, so who knows13:03
sean-k-mooneybecause of hardware age reasons many of our custoemr using wallaby before did fresh installs on new hardware instead of upgrading so this only came up now after 4 years.13:03
sean-k-mooneyya i mean it happens, compaines deploy a cloud, it works well enough for there needed and dont upgrade it because of small teams and then they realsie they are massivly out of date with no clear way to catch up13:04
elodillesthat's true, though, probably new install is easier for many operators13:05
opendevreviewmelanie witt proposed openstack/os-traits master: DNM trait  https://review.opendev.org/c/openstack/os-traits/+/93215222:55
opendevreviewmelanie witt proposed openstack/nova master: api: Add microversion 2.97 to add image_type to block_device_mapping_v2  https://review.opendev.org/c/openstack/nova/+/93096823:02
opendevreviewmelanie witt proposed openstack/nova master: WIP trait, filter, request spec  https://review.opendev.org/c/openstack/nova/+/93215323:02

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!