Wednesday, 2025-04-23

*** mhen_ is now known as mhen01:19
jbernard#startmeeting cinder14:03
opendevmeetMeeting started Wed Apr 23 14:03:08 2025 UTC and is due to finish in 60 minutes.  The chair is jbernard. Information about MeetBot at http://wiki.debian.org/MeetBot.14:03
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:03
opendevmeetThe meeting name has been set to 'cinder'14:03
jbernard#topic roll call14:03
jbernardo/14:03
jbernard#link https://etherpad.opendev.org/p/cinder-flamingo-meetings14:03
whoami-rajathey14:03
Saio/14:03
Ashish_Hi14:03
sfernandhi14:03
simondodsleyo/14:03
gireesho/14:06
jbernardwelcome everyone14:06
rosmaitao/14:06
vdhakadHi14:07
keerthivasansuresho/14:07
hvlcchao1hi14:07
jbernardour agenda is spase today, so the say the least ;)14:07
jbernards/spase/sparse14:07
jbernardwe are in R-23 right now14:08
jbernardthis means 23 weeks until flamingo's release14:08
jbernard#link https://releases.openstack.org/flamingo/schedule.html14:08
jbernardwhich means there is plenty of time to get patches and reviews done! :)14:09
simondodsleyyou hope :)14:09
jbernardother than that, it is very quiet with the exception of the mechanical switches in my keyboard14:09
jbernardso we can have open discussion now if it's needed14:10
jbernardi dont' have anything pressing to communicate14:10
jbernard#topic open discussion14:10
vdhakadIs this a good time to get this patch merged, now that we have F branch: https://review.opendev.org/c/openstack/cinder/+/92545014:10
simondodsleythere are still outstanding patches that need to be done for caracal so they can backport to bobcat14:10
rosmaitadon't forget about bobcat14:10
jbernardahh, great14:11
jbernardvdhakad: genearlly yes, it looks like you have some minor feedback14:12
rosmaita#link https://etherpad.opendev.org/p/cinder-stable-bobcat-eol14:12
jbernardsimondodsley: what are the outstanding ones?14:12
simondodsleylisted at the top of https://etherpad.opendev.org/p/cinder-stable-bobcat-eol14:12
jbernardok, i see14:12
vdhakad jbernard yeah, Simon left a comment regarding documentation. I’ll address it as part of a subsequent specifically patch for documentation14:13
whoami-rajatdon't we require a deprecation period for driver opts? i see 'storwize_volume_group' directly removed14:14
rosmaitai was saying "April 30" like it's a long way away, but it's basically next week14:14
jbernardrosmaita: ill work on this today14:15
rosmaitaok14:15
jbernardwhoami-rajat: im not aware of it applying to opts, but i could very well be missing it14:16
jbernardwhoami-rajat: we have a period for removal (https://docs.openstack.org/cinder/latest/drivers-all-about.html)14:16
whoami-rajatIMO the deprecation period should be valid for all config opts, if a deployment is using any config option and when they upgrade, the option doesn't work without any prior deprecation warning, it doesn't give them any time to react to the change14:18
whoami-rajatwhat do you think jbernard rosmaita ? ^14:19
rosmaitai agree with whoami-rajat14:19
jbernardit makes sense, don't we have that written somewhere?14:20
vdhakadThis conf option was introduced in Antelope release, but all the patches of the functionality weren’t merged due to disruption in team. Specifically the patch that updates documentation for the feature also wasn’t merged. We don’t believe any user is able to use the functionality.14:20
rosmaitai think it's just the normal deprecation process for openstack14:21
rosmaitavdhakad: so what you are saying is, currently the option exists, but it's a no-op?14:21
whoami-rajathmm, that's interesting14:22
vdhakadWe do not believe any of our users is using the ‘storwize-volume-group’ config along with the volume group support, because the functionality never merged fully. So upgrade shouldn’t be a problem.14:23
whoami-rajati do see a lot of code using that option but maybe that code path is never triggered14:23
simondodsleywe should definatley document the deprecation process for opts, but in this case, if the opt has not been previously documented and doesn't work anyway, then I think skipping the deprecation period in this case might be ok14:23
vdhakadwhoami-rajat: yup14:23
whoami-rajatyeah, in this specific case, it looks reasonable14:24
rosmaitathough i do agree with Simon's comment that this should be mentioned in the release note14:25
whoami-rajat+114:26
rosmaitai know it will show up in the docs eventually, but i think it's better to have it in the same patch14:26
vdhakadCool, I’ll update the release notes14:27
rosmaitagreat14:27
rosmaitai think we put stuff like that in the "upgrade" section ... "IBM svf: the '<name>' option, which was never fully operational, has been removed in this release"14:28
vdhakadrosmaita: oh! okay. Thanks for the help!14:29
rosmaitanp, take a look through the existing release notes for an example14:30
Ashish_I was wondering if we currently support SSH host certificates for authentication. If not, is there any plan to add support for them in the near future? From what I can tell, it seems that the version of Paramiko we’re using may not support this feature, but I might be mistaken. Just wanted to double-check in case I’m overlooking something.14:31
jbernardi have added it to the review reqeust list, just so i dont loose track14:31
rosmaitaAshish_: i don't think we do ... i don't think cinder itself uses ssh at all, it's something that some of the drivers might do14:34
rosmaitawe have ssh_utls but i think that's just so that the drivers who need it can reuse the same code14:34
rosmaita(though, i am happy to be corrected)14:35
rosmaita(well, not "happy" ... "willing", maybe)14:35
Ashish_rosmaita: I’m working on the IBM SVC driver. We’re at a point where when a partition IP floats from a site to another, the cinder backend loses connectivity due to a mismatch with the host keys. We’re looking at host certificates as an alternative.14:36
Ashish_But I think overriding the ssh_utils to customise it for our driver might make more sense14:37
rosmaitaAshish_: i was just going to suggest that you do your work in ssh_utils!14:38
rosmaitai don't think that we are necessarily tied to paramiko if there is a better library for this14:38
Ashish_ rosmaita : Ah! Makes sense. So appending the host certificates support to ssh_utils would make our job easier, I’ll take a look into this. Thank you!14:41
rosmaitasounds good14:42
jbernardok, i think we might be at the end, anything else?14:44
jbernardthank you everyone14:46
jbernard#endmeeting14:46
opendevmeetMeeting ended Wed Apr 23 14:46:08 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:46
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-04-23-14.03.html14:46
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-04-23-14.03.txt14:46
opendevmeetLog:            https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-04-23-14.03.log.html14:46
SaiHad something to announce !! Sorry jon was typing. 14:46
jbernardSai: sorry!14:46
jbernardSai: what's up?14:46
SaiWe have new team member from NetApp14:47
SaiPlease join me in welcoming our newest team member pragya from NetApp. 14:47
SaiShe brings a wealth of expertise in Python and is ready to contribute to the openStack-cinder community. Welcome and all the very best Pragya !! 14:47
Sai> Sai: sorry!14:47
SaiNo worries, was just this! Thanks.14:47
pragyaThanks Sai! Hello everyone.14:47
jbernardpragya: welcome!14:48
pragyaThanks Jon!14:49
SaiThank you Jon!! That's all. Good day everyone! 14:49

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