Wednesday, 2024-03-06

« Monday, 2024-03-04 Index Thursday, 2024-03-07 »
*** geguileo is now known as Guest200410:19
*** Guest2004 is now known as geguileo10:19
*** tosky_ is now known as tosky13:57
whoami-rajat#startmeeting cinder14:00
opendevmeetMeeting 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
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
opendevmeetThe meeting name has been set to 'cinder'14:00
whoami-rajat#topic roll call14:00
Luzio/14:00
jbernardo/14:01
toskyo/14:01
rosmaitao/14:01
simondodsleyo/14:01
happystackero/14:01
jungleboyjo/14:01
whoami-rajat#link https://etherpad.opendev.org/p/cinder-caracal-meetings14:02
akawaio/14:02
whoami-rajathello everyone14:03
whoami-rajatlet's get started14:03
whoami-rajat#topic announcements14:03
whoami-rajatResults of TC and PTL elections14:03
whoami-rajat#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/OLLRWKP2FSSG5YK5F6M4TOEIHXSLCJ2X/14:03
whoami-rajat#link https://governance.openstack.org/election/#2024.2-ptl-candidates14:03
whoami-rajat#link https://governance.openstack.org/election/#2024.2-tc-candidates14:04
whoami-rajatthe TC and PTL results are out14:04
whoami-rajatwe already know who the next cinder PTL is so no surprises there14:04
whoami-rajatbut if you would like to check who is leading other projects14:04
whoami-rajatand our newly added TC members14:04
whoami-rajatplease check the links above14:04
rosmaitai think there's going to be a TC election, so that one's not finalized yet14:05
jayaanandhi14:05
whoami-rajatoh14:05
whoami-rajati didn't understand this statement in the email14:06
whoami-rajatNow begins the campaigning period where candidates and electorate14:06
whoami-rajatmay debate their statements.14:06
rosmaitayeah, that's for the TC14:06
whoami-rajatso we have 7 people on the TC list but TC only has 6 members right?14:06
rosmaita9 members total, not sure if we replace 4 or 5 this cycle14:07
rosmaitayou serve for 1 year on the tc, but some people are elected in fall, some in spring14:07
rosmaitathe idea is that way, there's some continuity14:08
whoami-rajatokay, i should gather more info on how the TC works before commenting, but glad we have rosmaita to let us know14:09
rosmaita:D14:09
whoami-rajatso that was about the elections14:09
whoami-rajatnext, Transition victoria -> xena to unmaintained14:10
whoami-rajat#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/XHN5T4HQHUQYCRVUUCBPMEIMCDOMTXQC/14:10
whoami-rajat#link https://review.opendev.org/c/openstack/releases/+/91040914:10
whoami-rajatinitially we moved yoga from stable to EOM to test out the process14:10
whoami-rajatand now we are moving victoria to xena in EOM14:10
whoami-rajatthe idea is to remove the stable branches14:10
whoami-rajatand create new unmaintained branches14:10
whoami-rajatThe release team has a patch for each project14:11
whoami-rajatfor cinder i verified hashes for all projects from victoria to xena but wanted to let cinder team take a look as well14:11
whoami-rajatsince it's a big change14:11
whoami-rajatbut as i can see they have merged the patch with my approval ...14:11
simondodsleymakes you feel good as the outgoing PTL, eh...14:12
whoami-rajat:D14:12
whoami-rajatanyway, so now we will have unmaintained branches from victoria -> yoga and all patches needs to be proposed there14:13
whoami-rajatthere 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 later14:15
rosmaitai 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:15
whoami-rajatrosmaita, yes14:16
whoami-rajatat least thats what we decided for yoga14:16
rosmaitai can modify those14:16
whoami-rajatwas there a discussion about it in TC?14:16
rosmaitano, i didn't bring it up, it's up to the projects to decide14:16
whoami-rajatthat cinder will be taking a different route than other projects on this?14:16
whoami-rajatokay, if that's the case14:16
whoami-rajatwe are good14:16
rosmaitawell, glance is taking the same route :)14:16
whoami-rajatI was worried if something changes and cinder is the only one broken14:17
rosmaitaanyway, don'14:17
rosmaitat  merge them, i'll get the revisions up later today14:17
whoami-rajatsure, thanks!14:17
whoami-rajatok, next, Upcoming Deadlines & Dates14:18
whoami-rajat#link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/5LVI4E425B3XCXD227NCZOFDWV7BSJLK/14:18
whoami-rajatCycle Highlights Due: March 7th, 2024 (R-4 week)14:18
whoami-rajatRC1 deadline: week of March 11, 2024 (R-3 week)14:18
whoami-rajatFinal RC deadline: week of  March 25, 2024 (R-1 week)14:18
whoami-rajatFinal 2024.1/Caracal release: April 3, 202414:18
whoami-rajatDalmatian PTG: April 8-12, 202414:18
whoami-rajatwe have cycle highlights deadline this week14:18
whoami-rajatbut i was planning to get the FFE features in14:18
whoami-rajatbefore proposing that patch14:18
whoami-rajatI've added a topic for it14:18
whoami-rajatanyone would like to announce anything else?14:19
whoami-rajatokay, let's move to topics then14:20
whoami-rajat#topic FFE features14:20
whoami-rajat1. [NetApp] LUN space-allocation support for iSCSI14:21
whoami-rajat#link https://review.opendev.org/c/openstack/cinder/+/89310614:21
whoami-rajati went through it and it looks good14:21
whoami-rajatnetapp ci is also passing14:21
whoami-rajatrosmaita, had some comments on it14:21
whoami-rajatif those are addressed, i think we are good on this14:22
rosmaitai'll look right after the meeting14:22
jayaanandrosmaita comments are addressed14:22
whoami-rajatgreat, thanks14:22
jayaanandthank you14:22
whoami-rajatthanks jayaanand14:23
whoami-rajatmoving on14:23
whoami-rajat2. Dell EMC: PowerMax - Configurable SRDF snapshots14:23
whoami-rajat#link https://review.opendev.org/c/openstack/cinder/+/91032014:23
whoami-rajatanyone from dell team here?14:23
happystackeryes14:23
whoami-rajatso this is waiting on the powermax CI 14:23
whoami-rajatis there an ETA on when it will be able to report?14:24
happystackerneed to ping the lab team again14:24
happystackerthere was an array upgrade in progress14:24
whoami-rajatokay, 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 cycle14:24
happystackerunderstood14:24
whoami-rajatalso if dell team can confirm that this feature works in their lab, i think that is also a valid reason to merge it14:25
whoami-rajatat least that's what we've done in the past, rosmaita can confirm ^14:25
happystackerI'll let them know14:25
rosmaitayes, i think that's acceptable as long as no one objects14:26
rosmaitaso if anyone has objections, now is the time to state them!14:26
whoami-rajatlooks like there are no objections14:27
whoami-rajatso happystacker please try getting at least one of the two done and we can go ahead and merge the patch14:27
jungleboyjIn 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
happystackerthat would be great! thanks team14:27
happystackerok thks whoami-rajat14:27
whoami-rajatas jungleboyj said, running tempest locally and adding output as a comment on the patch is acceptable14:28
jungleboyj:-)  14:28
bryanneumannThank you whoami-rajat & rosmaita for all of the reviews.14:28
whoami-rajatnp14:29
whoami-rajat3. [HPE XP] Support HA14:29
whoami-rajat#link https://review.opendev.org/c/openstack/cinder/+/89260814:29
whoami-rajati was reviewing this and i mostly agree with rosmaita's comments14:29
whoami-rajatthe safe get part makes sense14:29
whoami-rajatand also fixing the issues with child drivers could be a separate patch if there are too many of similar issues14:30
whoami-rajatand it doesn't affect the current feature14:30
whoami-rajatthe hitachi team mentioned that they are working on addressing the comments14:31
whoami-rajatso probably we can expect an update today or tomorrow?14:31
akawaiwhoami-rajat: I see. we'll try.14:32
whoami-rajatokay, 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.114:33
whoami-rajatafter which all patches will go into 2024.214:33
akawaiI see.14:33
whoami-rajatso i feel we are good on this14:34
whoami-rajatmoving on14:35
rosmaitaakawai: please ping me in #openstack-cinder when you post a revision14:35
akawairosmaia: I see.14:36
whoami-rajat4. Dell PowerFlex driver: Add Cinder active-active support14:37
whoami-rajat#link https://review.opendev.org/c/openstack/cinder/+/89979514:37
whoami-rajatso not an A/A expert but compared the code to pure driver (that has A/A) supported and it looked good14:37
whoami-rajatthe changes are very minimal14:37
whoami-rajatthe CI is passing14:38
whoami-rajatso i didn't find any issues14:38
whoami-rajatrequires a second reviewer on it though14:38
yianthank you whoami-rajat14:38
whoami-rajatif geguileo has some time, maybe he can verify the A/A part, but anyone is welcome to review14:39
happystackerthank you 14:39
whoami-rajatsince we have less time, we still need a second reviewer though ...14:40
rosmaitai will review14:41
jungleboyjwhoami-rajat: I can look now.  I am not an expert but have looked at some of the other patches.14:42
jungleboyjBetween rosmaita  and I it should be good.14:42
rosmaitasounds good14:42
whoami-rajatoh great, thanks for volunteering!14:42
yianthanks!14:42
happystackerexcellent, thank you14:42
rosmaitai just want to grab another A/A patch and make sure the correct functions are implemented & tested14:43
rosmaitahappystacker: i assume you looked at an already merged dell patch?14:43
happystackeryes14:43
whoami-rajatPure is a good example, that's what i looked at14:43
happystackerthat's what we looked at as well14:43
simondodsley:D14:43
whoami-rajat:D 14:43
rosmaitasimondodsley: ^^ hooray!14:43
simondodsleyI'm just looking at the patch as well14:43
whoami-rajatPure is setting an example for A/A implementation14:44
rosmaitathis patch is getting a lot of looks!14:44
happystacker;-)14:44
whoami-rajatokay, this looks sorted out14:45
whoami-rajatlast one14:46
whoami-rajat5. Ceph: Add option to keep only last n snapshots per backup14:46
whoami-rajat#link https://review.opendev.org/c/openstack/cinder/+/81045714:46
whoami-rajatzaitcev, had a comment to which Chrsitian has replied14:46
whoami-rajati haven't taken a look and honestly the ceph backup driver scares me14:47
whoami-rajatso i can't guarantee if we will be able to get it in before validating with some ceph experts14:47
zaitcevI'll take another look.14:47
zaitcevBut I think core members should take a look without regard to my question anyway.14:48
zaitcevEric and Gorka at the minimum.14:48
jungleboyj++ Yeah, for Ceph backup we really need one of those two.14:50
whoami-rajatI would be happy to delegate the review responsibility, but let's see if we are able to get it in14:50
whoami-rajatthere 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 cycle14:52
whoami-rajatother than that, I think we are covered on features14:52
rosmaitamaybe we can prioritize fujitsu for immediately after RC-time14:53
rosmaitaget it in really early into Dalmatian14:53
whoami-rajat+114:53
whoami-rajatwe have Fujitsu and Hitachi that have a long list of patches14:53
whoami-rajat#link https://etherpad.opendev.org/p/cinder-2024.1-caracal-features14:53
whoami-rajatso better to target them early next cycle14:53
rosmaitawell, if we are short on ptg topics, maybe we can have a fujitsu/hitachi review festival during ptg week14:54
whoami-rajatmaybe jbernard would like to be reminded about it :D14:55
jbernardwhoami-rajat: thanks :)14:55
whoami-rajatI have no issues with the idea14:55
whoami-rajatokay quickly i will go through the next topic14:57
whoami-rajat#topic Deprecate python-cinderclient-brick-ext14:57
whoami-rajatI was going through the history of changes in this project and found out the last functional change was 3 years ago14:57
whoami-rajat#link https://github.com/openstack/python-brick-cinderclient-ext/commit/26495b75bb2ace278214d5961eefbd21014811b914:57
whoami-rajatRecently 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
whoami-rajatDo we have contributors that actively want to work on it or keep releases ongoing for other reasons?14:57
whoami-rajat(good thing i wrote all this before)14:58
rosmaitathe only person i know who really uses it is hemna14:58
whoami-rajatokay, but there is no active development on it for a long time so there is maintenance overhead14:59
whoami-rajatif hemna- is good with older versions14:59
whoami-rajatthen we can probably deprecate it15:00
whoami-rajatwe are out of time, we can get back to it next week or during PTG15:01
whoami-rajatthanks everyone for attending15:01
whoami-rajat#endmeeting15:01
opendevmeetMeeting ended Wed Mar  6 15:01:39 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:01
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-03-06-14.00.html15:01
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-03-06-14.00.txt15:01
opendevmeetLog:            https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-03-06-14.00.log.html15:01
jungleboyjThank you!15:03
happystacker:-)15:04
bryanneumannThanks for the suggestion jungleboyj about attaching Tempest results.15:06
*** geguileo is now known as Guest205221:13
*** kopecmartin_ is now known as kopecmartin22:50
« Monday, 2024-03-04 Index Thursday, 2024-03-07 »

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