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