14:03:08 <jbernard> #startmeeting cinder
14:03:08 <opendevmeet> Meeting 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:08 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:03:08 <opendevmeet> The meeting name has been set to 'cinder'
14:03:10 <jbernard> #topic roll call
14:03:12 <jbernard> o/
14:03:14 <jbernard> #link https://etherpad.opendev.org/p/cinder-flamingo-meetings
14:03:16 <whoami-rajat> hey
14:03:22 <Sai> o/
14:03:41 <Ashish_> Hi
14:03:45 <sfernand> hi
14:03:54 <simondodsley> o/
14:06:01 <gireesh> o/
14:06:56 <jbernard> welcome everyone
14:06:58 <rosmaita> o/
14:07:03 <vdhakad> Hi
14:07:14 <keerthivasansuresh> o/
14:07:15 <hvlcchao1> hi
14:07:27 <jbernard> our agenda is spase today, so the say the least ;)
14:07:34 <jbernard> s/spase/sparse
14:08:30 <jbernard> we are in R-23 right now
14:08:39 <jbernard> this means 23 weeks until flamingo's release
14:08:42 <jbernard> #link https://releases.openstack.org/flamingo/schedule.html
14:09:24 <jbernard> which means there is plenty of time to get patches and reviews done! :)
14:09:42 <simondodsley> you hope :)
14:09:58 <jbernard> other than that, it is very quiet with the exception of the mechanical switches in my keyboard
14:10:23 <jbernard> so we can have open discussion now if it's needed
14:10:32 <jbernard> i dont' have anything pressing to communicate
14:10:37 <jbernard> #topic open discussion
14:10:43 <vdhakad> Is this a good time to get this patch merged, now that we have F branch: https://review.opendev.org/c/openstack/cinder/+/925450
14:10:48 <simondodsley> there are still outstanding patches that need to be done for caracal so they can backport to bobcat
14:10:54 <rosmaita> don't forget about bobcat
14:11:02 <jbernard> ahh, great
14:12:05 <jbernard> vdhakad: genearlly yes, it looks like you have some minor feedback
14:12:15 <rosmaita> #link https://etherpad.opendev.org/p/cinder-stable-bobcat-eol
14:12:32 <jbernard> simondodsley: what are the outstanding ones?
14:12:47 <simondodsley> listed at the top of https://etherpad.opendev.org/p/cinder-stable-bobcat-eol
14:12:59 <jbernard> ok, i see
14:13:26 <vdhakad> jbernard yeah, Simon left a comment regarding documentation. I’ll address it as part of a subsequent specifically patch for documentation
14:14:45 <whoami-rajat> don't we require a deprecation period for driver opts? i see 'storwize_volume_group' directly removed
14:14:57 <rosmaita> i was saying "April 30" like it's a long way away, but it's basically next week
14:15:04 <jbernard> rosmaita: ill work on this today
14:15:44 <rosmaita> ok
14:16:19 <jbernard> whoami-rajat: im not aware of it applying to opts, but i could very well be missing it
14:16:38 <jbernard> whoami-rajat: we have a period for removal (https://docs.openstack.org/cinder/latest/drivers-all-about.html)
14:18:46 <whoami-rajat> IMO 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 change
14:19:09 <whoami-rajat> what do you think jbernard rosmaita ? ^
14:19:50 <rosmaita> i agree with whoami-rajat
14:20:38 <jbernard> it makes sense, don't we have that written somewhere?
14:20:47 <vdhakad> This 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:21:02 <rosmaita> i think it's just the normal deprecation process for openstack
14:21:33 <rosmaita> vdhakad: so what you are saying is, currently the option exists, but it's a no-op?
14:22:03 <whoami-rajat> hmm, that's interesting
14:23:27 <vdhakad> We 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:30 <whoami-rajat> i do see a lot of code using that option but maybe that code path is never triggered
14:23:51 <simondodsley> we 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 ok
14:23:53 <vdhakad> whoami-rajat: yup
14:24:37 <whoami-rajat> yeah, in this specific case, it looks reasonable
14:25:38 <rosmaita> though i do agree with Simon's comment that this should be mentioned in the release note
14:26:28 <whoami-rajat> +1
14:26:31 <rosmaita> i know it will show up in the docs eventually, but i think it's better to have it in the same patch
14:27:19 <vdhakad> Cool, I’ll update the release notes
14:27:31 <rosmaita> great
14:28:47 <rosmaita> i 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:29:48 <vdhakad> rosmaita: oh! okay. Thanks for the help!
14:30:26 <rosmaita> np, take a look through the existing release notes for an example
14:31:24 <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:32 <jbernard> i have added it to the review reqeust list, just so i dont loose track
14:34:00 <rosmaita> Ashish_: 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 do
14:34:44 <rosmaita> we have ssh_utls but i think that's just so that the drivers who need it can reuse the same code
14:35:20 <rosmaita> (though, i am happy to be corrected)
14:35:40 <rosmaita> (well, not "happy" ... "willing", maybe)
14:36:36 <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:37:47 <Ashish_> But I think overriding the ssh_utils to customise it for our driver might make more sense
14:38:19 <rosmaita> Ashish_: i was just going to suggest that you do your work in ssh_utils!
14:38:55 <rosmaita> i don't think that we are necessarily tied to paramiko if there is a better library for this
14:41:25 <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:42:09 <rosmaita> sounds good
14:44:22 <jbernard> ok, i think we might be at the end, anything else?
14:46:04 <jbernard> thank you everyone
14:46:08 <jbernard> #endmeeting