Wednesday, 2024-06-05

jbernard#startmeeting cinder14:00
opendevmeetMeeting started Wed Jun  5 14:00:17 2024 UTC and is due to finish in 60 minutes.  The chair is jbernard. 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
jbernard#topic roll call14:00
jbernardo/14:00
Luzio/14:00
ccokeke[m]o/14:00
jbernardgood morning/afternoon/evening everyone!14:00
whoami-rajathi14:00
akawaio/14:00
jungleboyjo/14:00
rosmaitao/14:02
jbernard#link https://etherpad.opendev.org/p/cinder-dalmatian-meetings14:02
simondodsleyo/14:02
toskyo/14:03
jbernardok, lets get started14:05
jbernard#topic annoucements14:06
jbernardour first midcycle is next wednesday (june 12)14:06
jbernardive created an etherpad to collect topics14:06
jbernard#link https://etherpad.opendev.org/p/cinder-dalmatian-midcycles14:07
jbernardfeel free to add any topics or talking points there and we can organize them into a schedule for the session14:07
jbernardI think the final cinder-specific schedule will merge shortly14:08
jbernard#link https://review.opendev.org/c/openstack/releases/+/91990814:08
jbernarda few minor tweaks (thanks rajat)14:08
zaitcevWow, running this show is no sinecure.14:09
jbernardthat's all i have for annoucements, things have been pretty steady in the last weeks14:10
jbernard#topic docstrings!14:11
zaitcevOh, snap. I've only finished half of mine.14:11
jbernardif we can, I'd like to get a collection of the docstring patches either merged or in progress in hopes of creating a reveiw dashboard for us14:11
jbernardfor context, in the PTG we agreed to consolidate our driver documentation into the cinder interface definition and references those from the driver declarations14:12
jbernard#link https://etherpad.opendev.org/p/document-driver-interface14:13
rosmaitaprobably the best thing would be to use a common topic for the reviews14:13
jbernardim just going to spin through the assignments really quickly14:13
jbernardrosmaita: +114:13
jbernardrosmaita: maybe just 'docstrings' would work14:14
rosmaitasure14:14
jbernardrosmaita: can i apply such a topic to the collection, or must that be done by each respective author?14:14
rosmaitai think any core can set a topic on a review (though i'm not sure about that)14:15
jbernardok, ill try to to field that one and let everyone keep working :)14:15
jbernard#action apply common topic to outstanding docstring patches for dashboard (jbernard)14:16
jbernardthe list as it is:14:16
whoami-rajat'docstrings' might be too generic and could be used by other projects (or probably already used?), maybe something more cinder specific?14:16
whoami-rajatcinder-driver-docstrings14:16
zaitcevI used "improve-driver-docstring" topic.14:16
jbernard'cinder-docstrings-dalmatian'14:16
jbernardok, ill pick something unique, no worries14:17
zaitcevToo late, mine is merged. I'll do that next.14:17
jbernardlol14:17
whoami-rajatI've my patches under "document-driver-interface-image" https://review.opendev.org/q/topic:%22document-driver-interface-image%2214:17
whoami-rajatbut I'm fine with changing it14:17
jbernardmanage/unmanage still needs a volunteer14:17
jbernardany takers on taht one?14:17
jbernard#link https://etherpad.opendev.org/p/document-driver-interface#L3214:18
jbernardthe list contains several duplicates, probably not as bad as it initially looks14:18
jbernardok, ill nominate myself then14:19
jbernardnext is replication, still needs a volunteer14:19
jbernardadded to the list ;)14:20
jbernardbasic create/delete14:20
jbernardwhoami-rajat: you've posted a patch for those?14:20
whoami-rajatjbernard, I've a patch up for create volume https://review.opendev.org/c/openstack/cinder/+/91833614:21
jbernardwhoami-rajat: does that include the snapshot and delete routines?14:22
whoami-rajatjbernard, nope, i will propose separate patches for those, only done create_volume for now14:22
jbernardok14:23
jbernardtranser14:23
jbernardDesire, are you here?14:23
jbernardi don't see any posted patches from a quick scan, can revisit soon14:24
jbernardattach/detach14:25
jbernardthis is gorka14:25
jbernardhe's not here today, do we know the status of those?14:25
jbernardscheduler14:26
jbernardrosmaita: ^14:27
jbernardhow are those going?14:27
rosmaitawell, i have some of the functions doc'd locally, need to finish14:27
jbernardme too14:27
jbernardyou're also on the 'assorted' list as well14:28
jbernardimage14:28
rosmaitathat one i have not started14:28
jbernardwhoami-rajat: you've already posted patches in the etherpad for those14:29
whoami-rajatyes, all patches are up for image14:29
jbernardand that just leaves init and backup from pete 14:29
jbernardwhich are in progress14:29
jbernardi'll ask gorka offline, im sure it wont take him long to get those posted14:30
jbernardok, that's all for docstring scraping14:30
jbernardwe can do review requests shortly, ill open for discussion now if anyone has concerns or general questions14:32
jbernard#topic open discussion14:32
zaitcevLooks like nobody has anything.14:35
jbernard#topic review requests14:35
jbernardantelope backport14:35
jbernard#link https://review.opendev.org/c/openstack/os-brick/+/92023214:35
jbernard^ that's an easy one14:35
jbernardakawai needs a review for Hitachi tests14:36
jbernard#link https://review.opendev.org/c/openstack/cinder/+/91672414:36
jbernardsimondodsley needs a review for Pure API upgrade14:37
jbernard#link https://review.opendev.org/c/openstack/cinder/+/90309514:37
simondodsleyPlease 14:37
jbernard^ this is blocking other patches, so needs to be reviewed14:37
jbernardsimondodsley: am i correct that your local CI is passing on that patch?14:38
simondodsleyYes. I can provide tempest results if needed14:39
jbernardok, that's helpful to know14:39
jbernardand Luzi needs reviews for image encryption spec and user visible info in volume types14:39
jbernard#link https://review.opendev.org/c/openstack/cinder-specs/+/91949914:39
jbernardand14:39
jbernard#link https://review.opendev.org/c/openstack/cinder-specs/+/90919514:39
Luziyes please14:40
jbernardwill the review-priority flag cause patches like this to become more visible in dashboard listings?14:41
jbernardwhoami-rajat, rosmaita: ^14:41
rosmaitawell, it's supposed to14:41
rosmaitabut i don't think it's worked well in practice14:42
whoami-rajati think we can filter out patches which has RP set but we haven't gone through that list in a while14:43
jbernardok, lets try for an initial pass and any outstanding concerns can be worked through in the midcycle next week14:43
jbernardthat's all i have for today14:44
jbernardanything else?14:44
jbernardok, thanks everyone!14:45
jbernard#endmeeting14:46
opendevmeetMeeting ended Wed Jun  5 14:46:00 2024 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:46
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-06-05-14.00.html14:46
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-06-05-14.00.txt14:46
opendevmeetLog:            https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-06-05-14.00.log.html14:46
jungleboyjThank you!14:46
whoami-rajatthanks!14:46
*** chuanm1 is now known as chuanm18:32
*** dasm is now known as Guest871620:46

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