14:00:36 <whoami-rajat> #startmeeting cinder
14:00:36 <opendevmeet> Meeting started Wed Mar  6 14:00:36 2024 UTC and is due to finish in 60 minutes.  The chair is whoami-rajat. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:36 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:36 <opendevmeet> The meeting name has been set to 'cinder'
14:00:40 <whoami-rajat> #topic roll call
14:00:56 <Luzi> o/
14:01:17 <jbernard> o/
14:01:17 <tosky> o/
14:01:24 <rosmaita> o/
14:01:26 <simondodsley> o/
14:01:41 <happystacker> o/
14:01:43 <jungleboyj> o/
14:02:10 <whoami-rajat> #link https://etherpad.opendev.org/p/cinder-caracal-meetings
14:02:13 <akawai> o/
14:03:37 <whoami-rajat> hello everyone
14:03:40 <whoami-rajat> let's get started
14:03:42 <whoami-rajat> #topic announcements
14:03:47 <whoami-rajat> Results of TC and PTL elections
14:03:51 <whoami-rajat> #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/OLLRWKP2FSSG5YK5F6M4TOEIHXSLCJ2X/
14:03:55 <whoami-rajat> #link https://governance.openstack.org/election/#2024.2-ptl-candidates
14:04:01 <whoami-rajat> #link https://governance.openstack.org/election/#2024.2-tc-candidates
14:04:05 <whoami-rajat> the TC and PTL results are out
14:04:19 <whoami-rajat> we already know who the next cinder PTL is so no surprises there
14:04:27 <whoami-rajat> but if you would like to check who is leading other projects
14:04:34 <whoami-rajat> and our newly added TC members
14:04:42 <whoami-rajat> please check the links above
14:05:10 <rosmaita> i think there's going to be a TC election, so that one's not finalized yet
14:05:31 <jayaanand> hi
14:05:59 <whoami-rajat> oh
14:06:04 <whoami-rajat> i didn't understand this statement in the email
14:06:05 <whoami-rajat> Now begins the campaigning period where candidates and electorate
14:06:06 <whoami-rajat> may debate their statements.
14:06:16 <rosmaita> yeah, that's for the TC
14:06:53 <whoami-rajat> so we have 7 people on the TC list but TC only has 6 members right?
14:07:19 <rosmaita> 9 members total, not sure if we replace 4 or 5 this cycle
14:07:59 <rosmaita> you serve for 1 year on the tc, but some people are elected in fall, some in spring
14:08:21 <rosmaita> the idea is that way, there's some continuity
14:09:27 <whoami-rajat> okay, i should gather more info on how the TC works before commenting, but glad we have rosmaita to let us know
14:09:38 <rosmaita> :D
14:09:59 <whoami-rajat> so that was about the elections
14:10:04 <whoami-rajat> next, Transition victoria -> xena to unmaintained
14:10:14 <whoami-rajat> #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/XHN5T4HQHUQYCRVUUCBPMEIMCDOMTXQC/
14:10:21 <whoami-rajat> #link https://review.opendev.org/c/openstack/releases/+/910409
14:10:34 <whoami-rajat> initially we moved yoga from stable to EOM to test out the process
14:10:45 <whoami-rajat> and now we are moving victoria to xena in EOM
14:10:51 <whoami-rajat> the idea is to remove the stable branches
14:10:54 <whoami-rajat> and create new unmaintained branches
14:11:09 <whoami-rajat> The release team has a patch for each project
14:11:40 <whoami-rajat> for cinder i verified hashes for all projects from victoria to xena but wanted to let cinder team take a look as well
14:11:43 <whoami-rajat> since it's a big change
14:11:53 <whoami-rajat> but as i can see they have merged the patch with my approval ...
14:12:25 <simondodsley> makes you feel good as the outgoing PTL, eh...
14:12:59 <whoami-rajat> :D
14:13:55 <whoami-rajat> anyway, so now we will have unmaintained branches from victoria -> yoga and all patches needs to be proposed there
14:15:41 <whoami-rajat> there will be a difference in case of SLURP releases but that won't be until next to next release so we can worry about it later
14:15:50 <rosmaita> i noticed that the auto-generated patches to master for the releasenotes have been posted ... they pull from unmaintained/vwx ... i think we want to use the -eom tags instead?
14:16:03 <whoami-rajat> rosmaita, yes
14:16:09 <whoami-rajat> at least thats what we decided for yoga
14:16:16 <rosmaita> i can modify those
14:16:16 <whoami-rajat> was there a discussion about it in TC?
14:16:32 <rosmaita> no, i didn't bring it up, it's up to the projects to decide
14:16:37 <whoami-rajat> that cinder will be taking a different route than other projects on this?
14:16:42 <whoami-rajat> okay, if that's the case
14:16:44 <whoami-rajat> we are good
14:16:51 <rosmaita> well, glance is taking the same route :)
14:17:02 <whoami-rajat> I was worried if something changes and cinder is the only one broken
14:17:03 <rosmaita> anyway, don'
14:17:19 <rosmaita> t  merge them, i'll get the revisions up later today
14:17:36 <whoami-rajat> sure, thanks!
14:18:01 <whoami-rajat> ok, next, Upcoming Deadlines & Dates
14:18:07 <whoami-rajat> #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/5LVI4E425B3XCXD227NCZOFDWV7BSJLK/
14:18:14 <whoami-rajat> Cycle Highlights Due: March 7th, 2024 (R-4 week)
14:18:15 <whoami-rajat> RC1 deadline: week of March 11, 2024 (R-3 week)
14:18:15 <whoami-rajat> Final RC deadline: week of  March 25, 2024 (R-1 week)
14:18:15 <whoami-rajat> Final 2024.1/Caracal release: April 3, 2024
14:18:15 <whoami-rajat> Dalmatian PTG: April 8-12, 2024
14:18:21 <whoami-rajat> we have cycle highlights deadline this week
14:18:31 <whoami-rajat> but i was planning to get the FFE features in
14:18:38 <whoami-rajat> before proposing that patch
14:18:45 <whoami-rajat> I've added a topic for it
14:19:16 <whoami-rajat> anyone would like to announce anything else?
14:20:49 <whoami-rajat> okay, let's move to topics then
14:20:52 <whoami-rajat> #topic FFE features
14:21:00 <whoami-rajat> 1. [NetApp] LUN space-allocation support for iSCSI
14:21:10 <whoami-rajat> #link https://review.opendev.org/c/openstack/cinder/+/893106
14:21:23 <whoami-rajat> i went through it and it looks good
14:21:26 <whoami-rajat> netapp ci is also passing
14:21:38 <whoami-rajat> rosmaita, had some comments on it
14:22:00 <whoami-rajat> if those are addressed, i think we are good on this
14:22:14 <rosmaita> i'll look right after the meeting
14:22:29 <jayaanand> rosmaita comments are addressed
14:22:29 <whoami-rajat> great, thanks
14:22:52 <jayaanand> thank you
14:23:00 <whoami-rajat> thanks jayaanand
14:23:07 <whoami-rajat> moving on
14:23:10 <whoami-rajat> 2. Dell EMC: PowerMax - Configurable SRDF snapshots
14:23:17 <whoami-rajat> #link https://review.opendev.org/c/openstack/cinder/+/910320
14:23:33 <whoami-rajat> anyone from dell team here?
14:23:47 <happystacker> yes
14:23:53 <whoami-rajat> so this is waiting on the powermax CI
14:24:01 <whoami-rajat> is there an ETA on when it will be able to report?
14:24:13 <happystacker> need to ping the lab team again
14:24:32 <happystacker> there was an array upgrade in progress
14:24:45 <whoami-rajat> okay, please let them know we are in the feature freeze exception and this needs to merge today or tomorrow else it will be moved to next cycle
14:24:58 <happystacker> understood
14:25:09 <whoami-rajat> also if dell team can confirm that this feature works in their lab, i think that is also a valid reason to merge it
14:25:25 <whoami-rajat> at least that's what we've done in the past, rosmaita can confirm ^
14:25:58 <happystacker> I'll let them know
14:26:00 <rosmaita> yes, i think that's acceptable as long as no one objects
14:26:13 <rosmaita> so if anyone has objections, now is the time to state them!
14:27:09 <whoami-rajat> looks like there are no objections
14:27:15 <whoami-rajat> so happystacker please try getting at least one of the two done and we can go ahead and merge the patch
14:27:18 <jungleboyj> In the past for these types of situations if they can add output from a successful run in their lab we would temporarily accept that.
14:27:26 <happystacker> that would be great! thanks team
14:27:47 <happystacker> ok thks whoami-rajat
14:28:01 <whoami-rajat> as jungleboyj said, running tempest locally and adding output as a comment on the patch is acceptable
14:28:10 <jungleboyj> :-)
14:28:32 <bryanneumann> Thank you whoami-rajat & rosmaita for all of the reviews.
14:29:10 <whoami-rajat> np
14:29:17 <whoami-rajat> 3. [HPE XP] Support HA
14:29:25 <whoami-rajat> #link https://review.opendev.org/c/openstack/cinder/+/892608
14:29:41 <whoami-rajat> i was reviewing this and i mostly agree with rosmaita's comments
14:29:50 <whoami-rajat> the safe get part makes sense
14:30:07 <whoami-rajat> and also fixing the issues with child drivers could be a separate patch if there are too many of similar issues
14:30:12 <whoami-rajat> and it doesn't affect the current feature
14:31:34 <whoami-rajat> the hitachi team mentioned that they are working on addressing the comments
14:31:43 <whoami-rajat> so probably we can expect an update today or tomorrow?
14:32:26 <akawai> whoami-rajat: I see. we'll try.
14:33:14 <whoami-rajat> okay, just to mention again, we are in feature freeze exception and all patches should be merged by tomorrow since we need to mention them in cycle highlights and cut branch for 2024.1
14:33:28 <whoami-rajat> after which all patches will go into 2024.2
14:33:34 <akawai> I see.
14:34:48 <whoami-rajat> so i feel we are good on this
14:35:16 <whoami-rajat> moving on
14:35:17 <rosmaita> akawai: please ping me in #openstack-cinder when you post a revision
14:36:39 <akawai> rosmaia: I see.
14:37:26 <whoami-rajat> 4. Dell PowerFlex driver: Add Cinder active-active support
14:37:32 <whoami-rajat> #link https://review.opendev.org/c/openstack/cinder/+/899795
14:37:53 <whoami-rajat> so not an A/A expert but compared the code to pure driver (that has A/A) supported and it looked good
14:37:57 <whoami-rajat> the changes are very minimal
14:38:01 <whoami-rajat> the CI is passing
14:38:07 <whoami-rajat> so i didn't find any issues
14:38:13 <whoami-rajat> requires a second reviewer on it though
14:38:58 <yian> thank you whoami-rajat
14:39:04 <whoami-rajat> if geguileo has some time, maybe he can verify the A/A part, but anyone is welcome to review
14:39:14 <happystacker> thank you
14:40:44 <whoami-rajat> since we have less time, we still need a second reviewer though ...
14:41:57 <rosmaita> i will review
14:42:01 <jungleboyj> whoami-rajat: I can look now.  I am not an expert but have looked at some of the other patches.
14:42:16 <jungleboyj> Between rosmaita  and I it should be good.
14:42:31 <rosmaita> sounds good
14:42:39 <whoami-rajat> oh great, thanks for volunteering!
14:42:49 <yian> thanks!
14:42:54 <happystacker> excellent, thank you
14:43:00 <rosmaita> i just want to grab another A/A patch and make sure the correct functions are implemented & tested
14:43:14 <rosmaita> happystacker: i assume you looked at an already merged dell patch?
14:43:23 <happystacker> yes
14:43:26 <whoami-rajat> Pure is a good example, that's what i looked at
14:43:37 <happystacker> that's what we looked at as well
14:43:44 <simondodsley> :D
14:43:45 <whoami-rajat> :D
14:43:46 <rosmaita> simondodsley: ^^ hooray!
14:43:59 <simondodsley> I'm just looking at the patch as well
14:44:04 <whoami-rajat> Pure is setting an example for A/A implementation
14:44:09 <rosmaita> this patch is getting a lot of looks!
14:44:35 <happystacker> ;-)
14:45:51 <whoami-rajat> okay, this looks sorted out
14:46:05 <whoami-rajat> last one
14:46:07 <whoami-rajat> 5. Ceph: Add option to keep only last n snapshots per backup
14:46:15 <whoami-rajat> #link https://review.opendev.org/c/openstack/cinder/+/810457
14:46:26 <whoami-rajat> zaitcev, had a comment to which Chrsitian has replied
14:47:07 <whoami-rajat> i haven't taken a look and honestly the ceph backup driver scares me
14:47:35 <whoami-rajat> so i can't guarantee if we will be able to get it in before validating with some ceph experts
14:47:44 <zaitcev> I'll take another look.
14:48:13 <zaitcev> But I think core members should take a look without regard to my question anyway.
14:48:33 <zaitcev> Eric and Gorka at the minimum.
14:50:15 <jungleboyj> ++ Yeah, for Ceph backup we really need one of those two.
14:50:52 <whoami-rajat> I would be happy to delegate the review responsibility, but let's see if we are able to get it in
14:52:19 <whoami-rajat> there was also a request from Fujitsu but i feel we already have a lot to be reviewed so unfortunately we will target it in next cycle
14:52:37 <whoami-rajat> other than that, I think we are covered on features
14:53:12 <rosmaita> maybe we can prioritize fujitsu for immediately after RC-time
14:53:32 <rosmaita> get it in really early into Dalmatian
14:53:36 <whoami-rajat> +1
14:53:47 <whoami-rajat> we have Fujitsu and Hitachi that have a long list of patches
14:53:50 <whoami-rajat> #link https://etherpad.opendev.org/p/cinder-2024.1-caracal-features
14:53:57 <whoami-rajat> so better to target them early next cycle
14:54:37 <rosmaita> well, if we are short on ptg topics, maybe we can have a fujitsu/hitachi review festival during ptg week
14:55:21 <whoami-rajat> maybe jbernard would like to be reminded about it :D
14:55:45 <jbernard> whoami-rajat: thanks :)
14:55:53 <whoami-rajat> I have no issues with the idea
14:57:37 <whoami-rajat> okay quickly i will go through the next topic
14:57:40 <whoami-rajat> #topic Deprecate python-cinderclient-brick-ext
14:57:44 <whoami-rajat> I was going through the history of changes in this project and found out the last functional change was 3 years ago
14:57:48 <whoami-rajat> #link https://github.com/openstack/python-brick-cinderclient-ext/commit/26495b75bb2ace278214d5961eefbd21014811b9
14:57:51 <whoami-rajat> Recently all patches proposed to this project are maintainance related and I was wondering if we want to maintain this project or just deprecate it and recommend using older versions?
14:57:55 <whoami-rajat> Do we have contributors that actively want to work on it or keep releases ongoing for other reasons?
14:58:09 <whoami-rajat> (good thing i wrote all this before)
14:58:19 <rosmaita> the only person i know who really uses it is hemna
14:59:25 <whoami-rajat> okay, but there is no active development on it for a long time so there is maintenance overhead
14:59:42 <whoami-rajat> if hemna- is good with older versions
15:00:07 <whoami-rajat> then we can probably deprecate it
15:01:33 <whoami-rajat> we are out of time, we can get back to it next week or during PTG
15:01:36 <whoami-rajat> thanks everyone for attending
15:01:39 <whoami-rajat> #endmeeting