14:00:59 <rosmaita> #startmeeting cinder
14:00:59 <opendevmeet> Meeting started Wed Mar 12 14:00:59 2025 UTC and is due to finish in 60 minutes.  The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:59 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:59 <opendevmeet> The meeting name has been set to 'cinder'
14:01:07 <rosmaita> #topic roll call
14:01:08 <simondodsley> o/
14:01:24 <akawai> o/
14:02:46 <Sai> o/
14:03:00 <whoami-rajat> hey
14:03:39 <rosmaita> looks like a light turnout ... i'll wait a few minutes, we went to daylight saving time in the USA this week, so people may be a bit confused
14:04:02 <rosmaita> Jon has a conflict that is keeping him offline, so I'm covering for him today
14:04:27 <simondodsley> Good old DST...
14:04:40 <msaravan> Hi
14:04:45 <rosmaita> good in some ways, bad in some ways
14:04:46 <simondodsley> shall we keep it or not? that is the question...
14:05:11 <tosky> o/
14:05:25 <rosmaita> simondodsley: are you pro- or anti- DST?
14:05:33 <simondodsley> TBH I don't care
14:06:14 <simondodsley> i live in FL so any change doesn't reallt affect me
14:06:19 <jungleboyj> I would be happy keeping the time where it is now.  Makes it better for me working with China.
14:06:49 <rosmaita> i think we should have micro-time-zones
14:07:13 <simondodsley> lol - can I have my house always be at 5pm
14:07:28 <rosmaita> :D
14:07:36 <rosmaita> ok, i guess we should get started
14:07:42 <jungleboyj> LOL.
14:07:51 <jungleboyj> Well, it is always 5 O'Clock somewhere.
14:07:56 <rosmaita> #topic announcements
14:08:08 <rosmaita> this is RC-1 target week
14:08:23 <rosmaita> what that means is that the first Release Candidate will be released and a stable/2025.1 branch will be cut
14:08:38 <rosmaita> the deadline for RC-1 is friday
14:09:14 <rosmaita> after friday, bugs will have to be fixed in master and then backported to stable/2025.1
14:09:28 <rosmaita> until the coordinated OpenStack release on Wednesday April 2, only release-critical bugs will be allowed to be backported
14:09:49 <simondodsley> that is concerning to me as I have a number of feature patches and bug fixes that have been waiting for some time
14:11:03 <rosmaita> yeah, and i'm not sure what to say about that
14:11:42 <simondodsley> i just need some core reviews - mainly for https://review.opendev.org/c/openstack/cinder/+/933675 as that is a big one we have been promising to customers. Needs one more core
14:11:45 <rosmaita> in the meantime, if you come across an issue that you think is release-critical, you can put it on this etherpad
14:12:01 <rosmaita> #link https://etherpad.opendev.org/p/cinder-epoxy-release-critical-bugs
14:12:26 <whoami-rajat> I remember reviewing some of them last week, do we have them listed on basis of priority?
14:12:33 <rosmaita> yikes, -56 +785
14:12:54 <rosmaita> whoami-rajat: not yet, that i am aware of
14:13:11 <simondodsley> romaita: exactly - it's been there a long time
14:13:33 <yuval> I have a very small patch only doc's need another core review: https://review.opendev.org/c/openstack/cinder/+/942672
14:13:34 <rosmaita> simondodsley: i will commit to looking that over today
14:13:44 <simondodsley> thank you
14:14:15 <rosmaita> yuval: done
14:14:22 <yuval> Thansk
14:14:37 <jungleboyj> simondodsley:  Me as well.  Just opened a tab.
14:14:44 <rosmaita> just a reminder to cores, the cinder project policy on doc-only changes is that they only need one +2
14:14:53 <jungleboyj> ++
14:15:49 <rosmaita> jungleboyj: do you have time for https://review.opendev.org/c/openstack/cinder/+/933675 ?
14:16:09 <rosmaita> (or were you talking about something else?)
14:16:21 <jungleboyj> rosmaita:  Yes, that one.
14:16:40 <rosmaita> ok, if you can take that, i'll grab something else
14:17:21 <jungleboyj> rosmaita: Sounds good.
14:17:26 <rosmaita> actually, i will grab https://review.opendev.org/c/openstack/cinder/+/916408 since it fixes a feature
14:18:04 <simondodsley> my other outstanding patches are small
14:18:28 <whoami-rajat> this one is not release critical but it's a small one correcting DEBUG logging in RBD QoS code, the commit message mentions the before/after https://review.opendev.org/c/openstack/cinder/+/943666
14:18:58 <whoami-rajat> it turns out we were logging the module instead of the actual QoS values being applied
14:19:23 <rosmaita> that would be misleading
14:20:26 <whoami-rajat> yeah i was puzzled to not see the QoS properties in logs and later i realized what was happening ...
14:20:48 <rosmaita> whoami-rajat: i will review that one if you hit this: https://review.opendev.org/c/openstack/cinder/+/915908
14:21:40 <whoami-rajat> we now have NVMe over RDMA so who needs iSER :D
14:21:48 <whoami-rajat> sure, will take a look
14:22:44 <rosmaita> thanks!
14:22:56 <rosmaita> it has a child patch that would be an easy review, too
14:23:13 <rosmaita> ok, but back to the main point, i.e., what's the top priority
14:23:37 <rosmaita> there was a encryption standardization patch that is important
14:25:11 <rosmaita> looks like the glance part is on hold for Flamingo, though
14:25:59 <rosmaita> this is the cinder patch: https://review.opendev.org/c/openstack/cinder/+/926298
14:26:12 <rosmaita> this is the glance one: https://review.opendev.org/c/openstack/glance/+/926295
14:27:12 <whoami-rajat> yeah, it would be good to get Glance team opinion on it, I would rather have all the code in same cycle to avoid backward compat issues
14:27:29 <rosmaita> ok, makes sense ... we'll hold off on that one
14:27:48 <rosmaita> the other patch i was thinking of was abishop's display migration progress
14:28:04 <rosmaita> i believe the nova side of that will not be in epoxy
14:30:07 <whoami-rajat> :/
14:32:18 <rosmaita> this is the cinder patch: https://review.opendev.org/c/openstack/cinder/+/942716
14:33:09 <rosmaita> it has a depends-on against the nova patch, so guess it will have to wait for Flamingo
14:35:25 <rosmaita> guess that's all for topics
14:35:33 <rosmaita> anyone have any concerns they want to raise?
14:36:16 <whoami-rajat> I've two patches added to the critical etherpad, the RBD one is pretty critical to me, but otherwise we will need to add them to KNOWN ISSUES in reno
14:36:33 <whoami-rajat> s/critical/release critical
14:37:17 <rosmaita> those both look worthy to me
14:38:26 <rosmaita> vdhakad: are you or someone from ibm here?
14:40:14 <rosmaita> ok, i left a note about that one on the agenda etherpad
14:40:47 <rosmaita> looks like the pure patches mostly have one +2 already, so those shouldn't be too bad
14:41:03 <rosmaita> and actually, jungleboyj already committed to the one on line 88
14:41:31 <rosmaita> the hitachi & oem patch ... already had 2 +2s, so I just +w'd it
14:42:03 <akawai> rosmaita: thx
14:42:38 <rosmaita> ok, if there is nothing else to discuss, we have 18 minutes of free reviewing time
14:43:35 <whoami-rajat> just to confirm, we are not pursuing the two features i.e. standardize encryption and report migration status?
14:44:33 <rosmaita> migration status : not pursuing for sure, since the nova patch is a dependency, and it won't be in nova epoxy
14:45:27 <rosmaita> standardize encryption: hate to leave it sitting, but it would be good to understand the glance concerns and whether it makes sense to merge the cinder patch without the glance one
14:45:45 <whoami-rajat> ack, thanks
14:46:12 <rosmaita> so i will ping Jon ... he can contact the core team if he thinks it should make it into Flamingo
14:46:22 <rosmaita> will have to merge tomorrow, though
14:46:32 <rosmaita> anything else?
14:47:29 <rosmaita> go forth and review!!!
14:47:29 <rosmaita> see you next week
14:47:29 <rosmaita> #endmeeting