Wednesday, 2022-01-19

*** hemna6 is now known as hemna07:37
*** dviroel|out is now known as dviroel10:58
*** dasm|off is now known as dasm13:25
*** fiooodooor is now known as mionsz13:54
rosmaita#startmeeting cinder14:01
opendevmeetMeeting started Wed Jan 19 14:01:10 2022 UTC and is due to finish in 60 minutes.  The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot.14:01
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:01
opendevmeetThe meeting name has been set to 'cinder'14:01
rosmaita#topic roll call14:01
whoami-rajatHi14:01
rosmaitalooks like i am at least in the correct channel this week14:01
fabiooliveirahi14:01
whoami-rajat:D14:01
walshh_Hi14:01
jungleboyjo/14:01
eharneyhi14:02
rosmaita#link https://etherpad.openstack.org/p/cinder-xena-meetings14:02
sfernandhi14:02
enriquetasohi14:02
eharneyhttps://etherpad.opendev.org/p/cinder-yoga-meetings14:02
jungleboyj:-)14:03
simondodsleyhi14:03
raghavendrat_hi14:03
caiquemello[m]hi14:03
rosmaitahello everyone, looks like a good turnout14:03
rosmaitalet's get started14:04
rosmaita#topic announcements14:04
rosmaitaFirst, a reminder about the Festival of XS reviews on Friday 1400-1600 UTC14:04
rosmaitathe video link is https://meetpad.opendev.org/cinder-festival-of-reviews14:04
rosmaitarumor is that some people have been having meetpad problems, so here is the etherpad link:14:05
rosmaitahttps://etherpad.opendev.org/p/cinder-festival-of-reviews14:05
rosmaitaif you have meetpad problems, put a note on the etherpad on Friday, and we can communicate with you in IRC14:05
rosmaitaNext, "Z" release name nominations open until 24 January14:05
rosmaita#link http://lists.openstack.org/pipermail/openstack-discuss/2022-January/026620.html14:06
rosmaitathis is your chance at immortality14:06
rosmaitasimilarly, the release schedule for Z has been proposed:14:06
rosmaita#link https://review.opendev.org/c/openstack/releases/+/82448914:06
rosmaitaif any of the milestones etc smack up against local holidays etc, it would be worth pointing out14:08
rosmaitanot that they will change, but it might be good for the release team to indicate that on the actual schedule so that people know when there will be review slowdowns14:08
rosmaitanext topic: Berlin Summit Call For Presentations14:09
rosmaitathe CFP closes on 9 February14:09
rosmaitahere's where you go to make a proposal:14:09
rosmaita#link http://cfp.openinfra.dev/app/berlin-202214:09
rosmaitanext item is on everyone's minds, i imagine14:09
rosmaitaNew driver merge deadline is Friday (this week)14:10
rosmaitaanother event sneaking up on us fast is14:10
rosmaitaThe R-9 midcycle 14:10
rosmaitascheduled for next Wednesday (26 Jan)14:10
rosmaitathere is still room for topics14:11
rosmaita#link https://etherpad.opendev.org/p/cinder-yoga-midcycles14:11
rosmaitawe will vote on the time in a few minutes14:11
whoami-rajatThere's a public holiday in India on 26th :/14:11
rosmaitaarrrrrgh!14:11
whoami-rajatbut i can try to attend14:11
whoami-rajatdon't know how i missed this before, anyway my bad 14:12
rosmaitaok, let's discuss in a minute14:12
whoami-rajatok14:12
rosmaitafinal announcement is that vkmc posted a proposal for a pop-up team for using cephadm in devstack14:12
rosmaita#link http://lists.openstack.org/pipermail/openstack-discuss/2022-January/026778.html14:12
rosmaitayou should read the email, but the tl;dr is that discussion will happen in #openstack-cephadm in OFTC14:13
rosmaitaincluding selecting when the popup team will meet14:13
rosmaitathat's all from me, anyone else have an announcement of general interest?14:14
rosmaitaok, guess not14:15
sfernandis this week the deadline for os-brick as well?14:15
rosmaita#topic R-9 midcycle time14:16
rosmaitaok, i guess the first question is ... do we want to postpone it for 1 week and hold it on 2 February?14:17
rosmaita(please check your calendars!)14:18
jungleboyjThat is then part of the Spring Festival in China.  Does that impact anyone that was planning to attend?14:18
simondodsleyHow do you know that??14:18
jungleboyjMost of my team is in China.  :-)  They are gone for two weeks.14:18
sfernandhe works for a chinese company 14:18
* enriquetaso is on holiday on 2 Feb 14:18
jungleboyjSo, it is a good time for me.14:18
jungleboyj:-)14:18
jungleboyjsfernand: Bingo!14:18
simondodsleyIt's also Groundhog Day14:19
rosmaitathis gets better and better14:19
jungleboyjWoo hoo!14:19
enriquetaso(but it's okay if most of people wants to postpone) 14:19
rosmaitaok, i think we should keep it as is14:19
jungleboyjsimondodsley:  Now you are just googling.14:19
simondodsleyIt's on my calender14:19
whoami-rajatI will try to attend and if not will watch the recording later14:20
rosmaitawhoami-rajat: you can always watch the video14:20
whoami-rajatyep14:20
rosmaitayeah, if you don't have a topic you want discussed, you should take your holiday14:20
rosmaitaok, so let's figure out the time14:20
rosmaitafirst, to be completely clear, we are talking about Wednesday 26 January14:21
rosmaitaI'd like to do our usual thing where we use 1 hour from the weekly meeting time14:21
rosmaita(since everyone has that blocked off already)14:21
rosmaitaand add an hour either before or after the meeting14:21
simondodsleyAfter is better for me14:21
rosmaitawe can vote on this right now14:21
rosmaitaoptions are:14:21
rosmaitaA 1300-1500 UTC14:21
rosmaitaB 1400-1600 UTC14:21
rosmaitaC either one of the above14:21
rosmaitaD need more options14:21
rosmaita#startvote time for the R-9 (26 January) midcycle? A, B, C, D14:22
opendevmeetBegin voting on: time for the R-9 (26 January) midcycle? Valid vote options are A, B, C, D.14:22
opendevmeetVote using '#vote OPTION'. Only your last vote counts.14:22
enriquetaso#vote C14:22
simondodsley#vote B14:22
rosmaita#vote B14:22
jungleboyj#vote B14:22
SachinMore#vote C14:22
sfernand#vote C14:22
fabiooliveira#vote C14:23
felipe_rodrigues#vote C14:23
caiquemello[m]#vote C14:23
TusharTgite#vote C14:24
rosmaitavoting will close in 1 minute14:24
mionsz#vote B14:25
eharney#vote C14:25
rosmaita#showvote14:25
opendevmeetC (8): sfernand, fabiooliveira, enriquetaso, caiquemello[m], TusharTgite, eharney, felipe_rodrigues, SachinMore14:25
opendevmeetB (4): simondodsley, jungleboyj, rosmaita, mionsz14:25
rosmaitalast call ...14:26
rosmaita#endvote14:26
opendevmeetVoted on "time for the R-9 (26 January) midcycle?" Results are14:26
opendevmeetC (8): sfernand, fabiooliveira, enriquetaso, caiquemello[m], TusharTgite, eharney, felipe_rodrigues, SachinMore14:26
opendevmeetB (4): simondodsley, jungleboyj, rosmaita, mionsz14:26
rosmaitaOK, we'll meet from 1400-1600 UTC14:26
rosmaitajust as a reminder ... if you have a topic and only have 1400-1500 open, leave a note on the etherpad and we'll try to get to you during that time14:27
rosmaitathanks everyone14:27
rosmaitaso to summarize:14:27
rosmaitathe R-9 Midcycle is on 26 January 2022 1400=1600 UTC14:28
rosmaita#topic core reviewer situation14:28
rosmaitaok, it will come as a surprise to no one that we are having review bandwidth problems14:29
rosmaitai just want to mention that if anyone has interest in becoming a cinder core, please contact me or one of the other cores to discuss what it entails14:29
rosmaitawe may also be able to give you helpful arguments to use with your manager about why it is a good use of your time14:29
rosmaitadon't worry about your review count at the moment, we can discuss expectations and how to meet them14:30
rosmaitathat's all I have to say about that right now14:30
rosmaitaexcept for a related topic14:30
rosmaitai am being PM'd on IRC, which is OK in general14:31
rosmaitabut i am being pinged about doing reviews ... which is sort of neutral14:31
rosmaitayou should definitely ping me if i left a comment on your patch and you have revised14:31
rosmaitabecause i don't always get back to those quickly14:32
rosmaitabut if you do a cold IRC ping requesting a review, i would like you to also include links to 2 reviews you have done on other people's patches14:32
rosmaitaand these should be reviews that have a helpful comment or 2, not simply a +114:32
rosmaitathat's my personal policy, just thought i should say it out loud14:33
rosmaita#topic block-box situation14:33
rosmaitafirst thing is that last week i learned that this thing was "block" box, not black box14:33
rosmaitabut i digress14:33
rosmaitai sent an email to the ML14:34
rosmaita#link http://lists.openstack.org/pipermail/openstack-discuss/2022-January/026639.html14:34
rosmaitathe silence has been deafening14:34
rosmaitaso i think we can go ahead and remove it14:34
rosmaita#link https://review.opendev.org/c/openstack/cinder/+/82447214:34
eharneyremoving it sounds good14:34
rosmaitapatch already has a +2!14:34
jungleboyjAgreed.14:34
sfernand+1 remove14:35
rosmaitai think we are all in agreement there, so review at your leisure14:35
rosmaita#topic allow driver feature backport?14:36
raghavendrat_hi14:36
rosmaita#link https://review.opendev.org/c/openstack/cinder/+/82062814:36
jungleboyjOh boy.14:36
rosmaitai'll give everyone time to look it over14:37
simondodsleyI see 2 issues here. One being Ussuri is EM and the othe ris IMHO I think this sia feature backport, but given it was approved as a backport for Victoria...14:37
rosmaitawell, we can revert it from Victoria14:37
rosmaitaso i don't see that as an issue14:37
whoami-rajatI might be missing something here but we've allowed feature backports that are driver specific before right?14:37
rosmaitamy concern just has to do with setting expectations about when this will be allowed14:37
simondodsleyThey also have a patch open for this in Train14:37
eharneywell, the code itself isn't much of a feature, it's just removing a restriction to not run against certain array versions, i think?14:38
rosmaitawhoami-rajat: i am not sure14:38
raghavendrat_right eharney14:38
jungleboyjHmm, I reviewed and hadn't interpreted as a feature.  14:38
simondodsleysimple code change I agree, but this is effectively adding a new driver by adding a new backplane14:38
raghavendrat_Initially Primera hardware had only FC support... Later iSCSI support was added14:38
jungleboyjI see it is kind of a feature though.14:39
whoami-rajatrosmaita, i think we've had tons of backports that are driver specific since the changes are isolated to the driver itself and doesn't affect cinder as a whole14:39
rosmaitaeharney: yeah, but my concern is that there may be differences between master and older versions that make the change unproblematic in master but maybe not in older versions14:39
sfernandI believe at least start talking about this possibility is important because some clouds just moved to Train due to OSP 16... We should have OSP 17 comming in a near future but adoption may be quite slow14:39
sfernandand it is really a pain to deliver improvements to customers..14:39
simondodsleyFor OSP16 RH will do downstream backports if you raise a BZ14:39
raghavendrat_we got customer ask for Ussuri14:40
simondodsleyThe Ussuri ask is for Canonical I think14:40
simondodsleyThe Train is for RH - 2 customers14:40
raghavendrat_in link: https://ubuntu.com/about/release-cycle ... pls search for "ussuri"14:40
raghavendrat_it has extended support till 203014:41
simondodsleyUbunut's support is not the same as OpenStack support cycles14:41
sfernandsimondodsley: all bzs I've opneded was required to be merged in official repos first =x14:41
simondodsleyussuroi is EM14:41
sfernandeven old backports such as queens...14:41
sfernandfor osp 1314:41
raghavendrat_whereas V, W and X have support of 2 years. 14:41
simondodsleyi have been told that downstream ports are possible - I'm doing one now for Manila14:41
rosmaitayes, but upstream backports spread the goodness to everyone14:42
rosmaitabut also the risk14:42
rosmaitawe can talk about this next week at the midcycle14:43
sfernandyep that is a nice approach in teory but impossible to mantain a CI running14:43
sfernandfor too many releases in the bast..14:43
sfernandpast*14:43
sfernandthat would be nice!14:43
rosmaitaour position could be that it's possible on a case-by-case basis14:43
raghavendrat_ok. will wait for midcycle14:44
simondodsleythat could lead to many discussions whereas a blanket - no feature backports would be better14:44
rosmaitabut i think we want to require at least an assurance from the vendor that there have been clean CI runs for the appropriate branches14:44
simondodsleyeven if this is backported to Ussuri, who is cutting a new release?14:44
raghavendrat_we will definitely run CI 14:45
raghavendrat_and ensure that there is no regression14:45
rosmaitaraghavendrat_: yes, i am not saying that you wouldn't ... just that we should have some formal documentation of that14:45
raghavendrat_ok14:46
rosmaitaok, everyone please think about this, and we will discuss next week in video ... should be easier to have the discussion14:46
raghavendrat_thanks everyone. Appreciate your feedback.14:46
rosmaita#topic new driver questions/comments/etc14:47
rosmaita#link https://etherpad.opendev.org/p/cinder-yoga-new-drivers14:47
rosmaitaok, let's look at the etherpad14:47
rosmaitaso far we've had TOYOU merge14:48
rosmaitai still haven't heard from YADRO14:48
rosmaitathe HPE X_ an dNEC Storage V require some patches to merge first14:48
rosmaitalines 36-37 in the etherpad14:49
rosmaitaafter that, the drivers themselves should be easy to review, assuming that the third party CI is running14:49
whoami-rajatI see 3 drivers following the format of just inheriting all useful methods from existing drivers and not implementing anything14:49
simondodsleyjust to let everyone know the Pure NVMe driver will miss the deadline as our Software Factory CI build has hit issues and we have no one with good SF knowledge to help out. We are looking for a short-term contractor with SF knowledge to help get this over the line if anyone knows of someone14:49
rosmaitasorry to hear about missing the deadline14:49
rosmaitabut thanks for letting us know so we can focus review effort elsewhere14:50
yuvalHey simon, we took the jenkins-gerrit-trigger path14:50
rosmaitathe Dell NFS Driver looks not ready for Yoga14:50
yuvalif you would like to setup this env, we can help14:50
simondodsleywe will refactor the driver to multiple files and add NVMe-FC and sync rep support for Z14:50
rosmaitasimondodsley: that sounds like a good plan14:50
rosmaitaok, the big one then, is Lightbits14:51
rosmaitawe have the driver, an os-brick connector, and there's a nova driver too14:51
simondodsleyyuval: thanks for the suggestion. It might be something we can investigate, but we did want to do SF as that is the preferred way 14:51
yuvalgot it14:51
rosmaitayeah, we were hoping that Software Factory would be a useful thing for all our third-party CIs14:52
yuvalas far as we know, the cinder driver is ready14:53
yuvalwe upload yesterday updated code14:53
rosmaitagreat, i will look at the most recent version today14:53
yuvalthanks14:53
rosmaitathanks to Gorka and Simon for thorough reviews on the Lightbits driver14:54
yuvalright, that was very helpful!14:54
rosmaita#topic open discussion14:55
rosmaitareview priorities: new drivers, and os-brick ... there are some important NVMe connector patches 14:55
rosmaitadon't forget the Festival of XS reviews on Friday14:55
rosmaitaremember, if you want reviews it helps to do reviews14:56
rosmaitai have a suspicion that i am forgetting something14:56
rosmaita#action rosmaita reminder to the ML about R-9 midcycle next week14:57
*** dviroel is now known as dviroel|lunch14:57
bingyanh_hi i have a question :which version will the master branch be released to? Y?14:59
rosmaitayes, right now master branch is also known as the yoga development branch14:59
rosmaitaat release time, we will branch master to stable/yoga14:59
rosmaitaand then master will become the Z development branch14:59
bingyanh_thanks15:00
rosmaitaok, we are out of time15:00
rosmaitathanks, everyone, and see you next week at the midcycle!15:00
bingyanh_so how to merge the code of the master branch to other versions?15:00
bingyanh_ Cherry Pick?15:00
bingyanh_when a driver have bug15:00
whoami-rajatthanks!15:00
rosmaita#endmeeting15:00
opendevmeetMeeting ended Wed Jan 19 15:00:56 2022 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:00
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-01-19-14.01.html15:00
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-01-19-14.01.txt15:00
opendevmeetLog:            https://meetings.opendev.org/meetings/cinder/2022/cinder.2022-01-19-14.01.log.html15:00
yuvalGuys thanks for all the help with the driver15:00
bingyanh_thanks15:01
rosmaitabingyanh_: yes, i think the process may be explained in our contributor docs somewhere15:01
rosmaitaor ask in #openstack-cinder if you have questions15:01
bingyanh_thank you ~15:01
*** dviroel|lunch is now known as dviroel16:15
*** dviroel is now known as dviroel|out21:38
*** dasm is now known as dasm|off23:33

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