jbernard | #startmeeting cinder | 14:00 |
---|---|---|
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
opendevmeet | The meeting name has been set to 'cinder' | 14:00 |
jbernard | #topic roll call | 14:00 |
jbernard | o/ | 14:00 |
Luzi | o/ | 14:00 |
ccokeke[m] | o/ | 14:00 |
jbernard | good morning/afternoon/evening everyone! | 14:00 |
whoami-rajat | hi | 14:00 |
akawai | o/ | 14:00 |
jungleboyj | o/ | 14:00 |
rosmaita | o/ | 14:02 |
jbernard | #link https://etherpad.opendev.org/p/cinder-dalmatian-meetings | 14:02 |
simondodsley | o/ | 14:02 |
tosky | o/ | 14:03 |
jbernard | ok, lets get started | 14:05 |
jbernard | #topic annoucements | 14:06 |
jbernard | our first midcycle is next wednesday (june 12) | 14:06 |
jbernard | ive created an etherpad to collect topics | 14:06 |
jbernard | #link https://etherpad.opendev.org/p/cinder-dalmatian-midcycles | 14:07 |
jbernard | feel free to add any topics or talking points there and we can organize them into a schedule for the session | 14:07 |
jbernard | I think the final cinder-specific schedule will merge shortly | 14:08 |
jbernard | #link https://review.opendev.org/c/openstack/releases/+/919908 | 14:08 |
jbernard | a few minor tweaks (thanks rajat) | 14:08 |
zaitcev | Wow, running this show is no sinecure. | 14:09 |
jbernard | that's all i have for annoucements, things have been pretty steady in the last weeks | 14:10 |
jbernard | #topic docstrings! | 14:11 |
zaitcev | Oh, snap. I've only finished half of mine. | 14:11 |
jbernard | if 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 us | 14:11 |
jbernard | for context, in the PTG we agreed to consolidate our driver documentation into the cinder interface definition and references those from the driver declarations | 14:12 |
jbernard | #link https://etherpad.opendev.org/p/document-driver-interface | 14:13 |
rosmaita | probably the best thing would be to use a common topic for the reviews | 14:13 |
jbernard | im just going to spin through the assignments really quickly | 14:13 |
jbernard | rosmaita: +1 | 14:13 |
jbernard | rosmaita: maybe just 'docstrings' would work | 14:14 |
rosmaita | sure | 14:14 |
jbernard | rosmaita: can i apply such a topic to the collection, or must that be done by each respective author? | 14:14 |
rosmaita | i think any core can set a topic on a review (though i'm not sure about that) | 14:15 |
jbernard | ok, 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 |
jbernard | the 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-rajat | cinder-driver-docstrings | 14:16 |
zaitcev | I used "improve-driver-docstring" topic. | 14:16 |
jbernard | 'cinder-docstrings-dalmatian' | 14:16 |
jbernard | ok, ill pick something unique, no worries | 14:17 |
zaitcev | Too late, mine is merged. I'll do that next. | 14:17 |
jbernard | lol | 14:17 |
whoami-rajat | I've my patches under "document-driver-interface-image" https://review.opendev.org/q/topic:%22document-driver-interface-image%22 | 14:17 |
whoami-rajat | but I'm fine with changing it | 14:17 |
jbernard | manage/unmanage still needs a volunteer | 14:17 |
jbernard | any takers on taht one? | 14:17 |
jbernard | #link https://etherpad.opendev.org/p/document-driver-interface#L32 | 14:18 |
jbernard | the list contains several duplicates, probably not as bad as it initially looks | 14:18 |
jbernard | ok, ill nominate myself then | 14:19 |
jbernard | next is replication, still needs a volunteer | 14:19 |
jbernard | added to the list ;) | 14:20 |
jbernard | basic create/delete | 14:20 |
jbernard | whoami-rajat: you've posted a patch for those? | 14:20 |
whoami-rajat | jbernard, I've a patch up for create volume https://review.opendev.org/c/openstack/cinder/+/918336 | 14:21 |
jbernard | whoami-rajat: does that include the snapshot and delete routines? | 14:22 |
whoami-rajat | jbernard, nope, i will propose separate patches for those, only done create_volume for now | 14:22 |
jbernard | ok | 14:23 |
jbernard | transer | 14:23 |
jbernard | Desire, are you here? | 14:23 |
jbernard | i don't see any posted patches from a quick scan, can revisit soon | 14:24 |
jbernard | attach/detach | 14:25 |
jbernard | this is gorka | 14:25 |
jbernard | he's not here today, do we know the status of those? | 14:25 |
jbernard | scheduler | 14:26 |
jbernard | rosmaita: ^ | 14:27 |
jbernard | how are those going? | 14:27 |
rosmaita | well, i have some of the functions doc'd locally, need to finish | 14:27 |
jbernard | me too | 14:27 |
jbernard | you're also on the 'assorted' list as well | 14:28 |
jbernard | image | 14:28 |
rosmaita | that one i have not started | 14:28 |
jbernard | whoami-rajat: you've already posted patches in the etherpad for those | 14:29 |
whoami-rajat | yes, all patches are up for image | 14:29 |
jbernard | and that just leaves init and backup from pete | 14:29 |
jbernard | which are in progress | 14:29 |
jbernard | i'll ask gorka offline, im sure it wont take him long to get those posted | 14:30 |
jbernard | ok, that's all for docstring scraping | 14:30 |
jbernard | we can do review requests shortly, ill open for discussion now if anyone has concerns or general questions | 14:32 |
jbernard | #topic open discussion | 14:32 |
zaitcev | Looks like nobody has anything. | 14:35 |
jbernard | #topic review requests | 14:35 |
jbernard | antelope backport | 14:35 |
jbernard | #link https://review.opendev.org/c/openstack/os-brick/+/920232 | 14:35 |
jbernard | ^ that's an easy one | 14:35 |
jbernard | akawai needs a review for Hitachi tests | 14:36 |
jbernard | #link https://review.opendev.org/c/openstack/cinder/+/916724 | 14:36 |
jbernard | simondodsley needs a review for Pure API upgrade | 14:37 |
jbernard | #link https://review.opendev.org/c/openstack/cinder/+/903095 | 14:37 |
simondodsley | Please | 14:37 |
jbernard | ^ this is blocking other patches, so needs to be reviewed | 14:37 |
jbernard | simondodsley: am i correct that your local CI is passing on that patch? | 14:38 |
simondodsley | Yes. I can provide tempest results if needed | 14:39 |
jbernard | ok, that's helpful to know | 14:39 |
jbernard | and Luzi needs reviews for image encryption spec and user visible info in volume types | 14:39 |
jbernard | #link https://review.opendev.org/c/openstack/cinder-specs/+/919499 | 14:39 |
jbernard | and | 14:39 |
jbernard | #link https://review.opendev.org/c/openstack/cinder-specs/+/909195 | 14:39 |
Luzi | yes please | 14:40 |
jbernard | will the review-priority flag cause patches like this to become more visible in dashboard listings? | 14:41 |
jbernard | whoami-rajat, rosmaita: ^ | 14:41 |
rosmaita | well, it's supposed to | 14:41 |
rosmaita | but i don't think it's worked well in practice | 14:42 |
whoami-rajat | i think we can filter out patches which has RP set but we haven't gone through that list in a while | 14:43 |
jbernard | ok, lets try for an initial pass and any outstanding concerns can be worked through in the midcycle next week | 14:43 |
jbernard | that's all i have for today | 14:44 |
jbernard | anything else? | 14:44 |
jbernard | ok, thanks everyone! | 14:45 |
jbernard | #endmeeting | 14:46 |
opendevmeet | Meeting ended Wed Jun 5 14:46:00 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:46 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-06-05-14.00.html | 14:46 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-06-05-14.00.txt | 14:46 |
opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2024/cinder.2024-06-05-14.00.log.html | 14:46 |
jungleboyj | Thank you! | 14:46 |
whoami-rajat | thanks! | 14:46 |
*** chuanm1 is now known as chuanm | 18:32 | |
*** dasm is now known as Guest8716 | 20:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!