| opendevreview | Merged openstack/cinder master: Replace eventlet sleep calls in volume drivers https://review.opendev.org/c/openstack/cinder/+/950695 | 00:43 |
|---|---|---|
| opendevreview | Amit Uniyal proposed openstack/cinder master: Adds urllib3 lib in requirements.txt https://review.opendev.org/c/openstack/cinder/+/993615 | 07:12 |
| opendevreview | Maximilian Brandt proposed openstack/cinder master: RBD: server-side copy when uploading a volume to an image https://review.opendev.org/c/openstack/cinder/+/994467 | 07:49 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: NetApp: Optimize ensure_export to skip LUN table refresh https://review.opendev.org/c/openstack/cinder/+/994600 | 07:51 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: NetApp: Optimize ensure_export to skip LUN table refresh https://review.opendev.org/c/openstack/cinder/+/994600 | 08:21 |
| opendevreview | Franciszek Przewoźny proposed openstack/cinder master: NetApp: Fix in-use boot and attach extension https://review.opendev.org/c/openstack/cinder/+/952900 | 10:15 |
| opendevreview | Fuli Qi proposed openstack/cinder-specs master: Add spec for retype volume including snapshots https://review.opendev.org/c/openstack/cinder-specs/+/989322 | 10:20 |
| opendevreview | Claudiu Belu proposed openstack/cinder master: tests: Enforce autospec=True on mock patches https://review.opendev.org/c/openstack/cinder/+/994152 | 11:22 |
| opendevreview | Claudiu Belu proposed openstack/cinder master: tests: Enforce autospec=True on mock patches https://review.opendev.org/c/openstack/cinder/+/994152 | 11:43 |
| opendevreview | Claudiu Belu proposed openstack/cinder master: tests: Enforce autospec on mock objects and patches https://review.opendev.org/c/openstack/cinder/+/994686 | 11:43 |
| hong-p[m] | jbernard: please review this spec, Add spec for incremental backup with CBT optimization | https://review.opendev.org/c/openstack/cinder-specs/+/985089 already got +2, +1 | 12:37 |
| eharney | i'll make another review pass on that one soon | 12:40 |
| opendevreview | Elod Illes proposed openstack/os-brick master: DNM: gate health test https://review.opendev.org/c/openstack/os-brick/+/994720 | 13:19 |
| opendevreview | Elod Illes proposed openstack/python-brick-cinderclient-ext master: DNM: gate health test https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/994723 | 13:20 |
| opendevreview | Elod Illes proposed openstack/python-cinderclient master: DNM: gate health test https://review.opendev.org/c/openstack/python-cinderclient/+/994724 | 13:20 |
| jbernard | #startmeeting cinder | 14:02 |
| opendevmeet | Meeting started Wed Jun 24 14:02:56 2026 UTC and is due to finish in 60 minutes. The chair is jbernard. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:02 |
| opendevmeet | The meeting name has been set to 'cinder' | 14:02 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-hibiscus-meetings | 14:03 |
| jbernard | hmm, the courtesy ping nicks have disappeared | 14:03 |
| rosmaita | probably just as well, we can ask people to re-populate | 14:04 |
| rosmaita | otherwise it's difficult to tell who doesn't care anymore | 14:04 |
| * fungi is around for his topic again | 14:04 | |
| jbernard | fixed | 14:04 |
| jbernard | oh, well, we could do that too | 14:04 |
| jbernard | feel free to unfix ;) | 14:05 |
| rosmaita | fixed is good | 14:05 |
| jbernard | courtesy ping: jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney jbernard hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy hillpd msaravan sp-bmilanov Luzi sfernand simondodsley zaubea nileshthathagar flelain wizardbit agalica lutimura kaisers aloke_dev Anoop_Shukla erlon | 14:05 |
| Anoop_Shukla | O/ | 14:06 |
| VolodymyrBoiko[m] | Hi | 14:06 |
| harsh | o/ | 14:06 |
| jbernard | o/ | 14:06 |
| fungi | protip: i just add "#startmeeting whatever" to the channel highlights in my irc client, never miss a meeting i care about | 14:07 |
| agalica_ | o/ | 14:07 |
| erlon | \o | 14:07 |
| fqi | o/ | 14:07 |
| rosmaita | o/ | 14:07 |
| Jyotsna | o/ | 14:07 |
| auniyal | o/ | 14:08 |
| rosmaita | fungi: that is a good tip! | 14:08 |
| fungi | mostly helpful when you have a dozen or more different weekly irc meetings to keep track of all over the place | 14:09 |
| jbernard | ok, let's get started | 14:09 |
| jbernard | #topic annoucements | 14:09 |
| jbernard | periodic jobs are still looking good | 14:10 |
| rosmaita | \o/ | 14:10 |
| jbernard | or, they're all false negatives and everything is broken | 14:10 |
| jbernard | either way, green! | 14:10 |
| jbernard | thanks to whoever shortened that link | 14:11 |
| jayaanand | O/ | 14:11 |
| jbernard | last minute addition,it looks like we have a 3rd party CI that has become an issue | 14:11 |
| jbernard | the "Ranjith" CI | 14:12 |
| jbernard | does anyone know who's running this? | 14:12 |
| agalica_ | Why, Ranjith is! | 14:12 |
| jbernard | lol, but of course! | 14:12 |
| erlon | Is it floading in the patches? | 14:13 |
| erlon | ow I see | 14:14 |
| rosmaita | https://review.opendev.org/q/cc:goldranjith@gmail.com | 14:15 |
| jbernard | ^ anyone know who might be running that account? | 14:16 |
| rosmaita | i think there is only one patch where that CI actually did not fail: https://review.opendev.org/q/reviewer:goldranjith@gmail.com | 14:17 |
| erlon | One of the posts have a link named vmstore...something | 14:17 |
| erlon | so, they must be trying to run their ci | 14:17 |
| rosmaita | not very successfully | 14:18 |
| erlon | And I think the only handle we have on him would be his email | 14:18 |
| fungi | note that the opendev sysadmins can disable a gerrit account if it's being a nuisance and the operator for it isn't responsive | 14:19 |
| fungi | but that's typically a last resort | 14:19 |
| rosmaita | i wonder whether this is the driver (it's not in-tree): https://github.com/Tintri/vmstore-cinder-driver | 14:19 |
| jbernard | hmm, i will send a query to that email and see if we get a response | 14:20 |
| erlon | there is a "cinder/volume/drivers/vmstore/nfs.py' driver | 14:21 |
| jbernard | they're welcome to run ci on proposed patches, but if the driver is out-of-tree it shouldn't be reporting on them | 14:21 |
| erlon | But other than the uncommon (personal) name, I don't see that much difference from other CIs that fail most of the time | 14:21 |
| rosmaita | erlon is correct, i was looking in an old branch | 14:22 |
| agalica_ | so the CI is just failing and that's it, or is it spamming? | 14:23 |
| rosmaita | two issues: it is failing like 100%, and (2) shouldn't be using a user account | 14:23 |
| jbernard | and 3, testing an out-of-tree driver | 14:24 |
| agalica_ | gotcha- yeah, the user account thing is what I complained about a few weeks ago. Most everyone has fixed that now | 14:24 |
| rosmaita | jbernard: it is actually in-tree, erlon found it | 14:24 |
| erlon | hmm, got it, I remember theres a need to request a different account for cis | 14:24 |
| jbernard | ahh, ok i was looking at the wrong thing, sorry | 14:24 |
| *** agalica_ is now known as agalica | 14:25 | |
| jbernard | do we have a human vmstore contact? | 14:25 |
| agalica | ah, no wonder I didn't see the courtesy ping | 14:25 |
| jbernard | oh yeah, courtesy ping: agalica :) | 14:26 |
| agalica | :D | 14:26 |
| rosmaita | looks like the lead developer is akhodos@ddn.com | 14:26 |
| jbernard | i pulled from a previous revision, i think i went too far back and missed some recent ones | 14:26 |
| jbernard | ok | 14:26 |
| jbernard | #action jbernard to email vmstore (akhodos and goldranjith) about their CI, report in next meeting if not before | 14:27 |
| auniyal | jbernard on quring ranjit, may be on openstack-discuss mailing list instead of directly, so if Ranjit is not available to reply others from there team/org who knows the person might reply, may be in other projects too | 14:27 |
| erlon | this was the submitter: eivanov@ddn.com | 14:28 |
| jbernard | auniyal: good call | 14:28 |
| auniyal | or *@ddn | 14:28 |
| erlon | Eugene Ivanov | 14:28 |
| jbernard | erlon: ill add that to the cc, thanks | 14:28 |
| jbernard | #topic gap bridging discussion (fungi) | 14:30 |
| jbernard | fungi: take it away | 14:30 |
| fungi | briefly revisiting this topic from last week now that folks have hopefully had time to digest it better, i've got a few more points that came out of a closer look into the data but also wanted to see if anyone had additional thoughts | 14:30 |
| fungi | one thing i missed calling out, which is exciting, is that the typical time to response on reviews improved drastically in gazpacho compared to flamingo; twice as fast on average (mean) but the median was actually five times faster! | 14:30 |
| fungi | i don't know if the team's reviewing habits changed in gazpacho, but that would be something to continue if so as it seems to have made a big impact, both the mean and median were significantly better than in any of the prior 5 cycles | 14:30 |
| fungi | granted, flamingo was anomalously bad in that regard, but gazpacho numbers were still easily twice as fast (on both median and mean) as any of the prior 5 cycles too, so something definitely improved dramatically | 14:30 |
| fungi | i guess this could also be attributed to the small/medium patch triage dashboard that was mentioned in last week's discussion? conversely, multiple survey respondents did talk about larger patches not getting attention or being ignored | 14:30 |
| fungi | we had takeaway suggestions, some similar to the previous round... contributors still aren't clear on where maintainers are tracking review priorities, so continuing to work on addressing that communication gap may help | 14:30 |
| fungi | even maintainers didn't necessarily agree on where and how review priorities were being tracked, which could be contributing to the confusion there | 14:31 |
| fungi | also the change and reviewer numbers are steadily growing while the active maintainer count is not, so if possible try to find some time to evaluate new maintainer candidates from the pool of established reviewers and code contributors | 14:31 |
| fungi | and on the topic of larger changes not getting as much review attention, maybe this is where the broader community can help by coming up with a set of recommendations to contributors for how to split up and make patches easier to review | 14:31 |
| fungi | if anybody has follow-up thoughts or questions on any of the above, as well as for the survey and metrics analysis from last week now that you've had time to mull it over, i'm happy to answer questions and entertain feedback here | 14:31 |
| fungi | (or in the #openstack-cinder irc channel after the meeting, or on the openstack-discuss mailing list, or even directly/in private for that matter) | 14:31 |
| fungi | i wonder if i just broke everyone's irc clients, or merely left you all speechless | 14:33 |
| jbernard | some of both, i suspect | 14:34 |
| jbernard | :) | 14:34 |
| jbernard | processing… | 14:34 |
| harsh | still her o/ :D | 14:34 |
| harsh | here* | 14:34 |
| fungi | ;) | 14:34 |
| jbernard | big patches remain a challenge, there is room for improvement there certainly | 14:35 |
| erlon | I think that the delays are particularly bigger for driver maintainers, in new drivers submitters. For new drivers is really hard to split the code | 14:35 |
| jbernard | we did just recently add 2 new cores, and have more coming up to speed | 14:35 |
| fungi | yes, i am getting the feeling that, as a community, maybe we should start seeing "proposing large patches" as the actual problem, rather than "failing to get around to reviewing large patches" | 14:35 |
| agalica | I'm going to be doing reviews on friday, hopefully that helps | 14:36 |
| agalica | :D | 14:36 |
| fungi | excellent news on the core team additions, congrats to you all! | 14:36 |
| erlon | But I think that, if we collect the list of things that we usually require, and flag on new changes, we can delegate part of that (locally) to LLMs to assist to point those | 14:36 |
| jbernard | yeah, there is some /size/ where reviews take much more time and begin to collide with other commitments, it can be hard to balance/juggle everything | 14:36 |
| fungi | it seems like llms are making this problem worse too, based on what i'm hearing from various maintainers | 14:37 |
| agalica | i think that's a possibility too, but it might resolve down to people leaning on vibes rather than actually making sure the code it spits out is sane | 14:37 |
| fungi | and that people who insist on using an llm to create patches should be giving it explicit instructions to break the work up into logical reviewable chunks while avoiding making unrelated changes in the files they touch | 14:38 |
| erlon | it makes worse because people can generate much more code now :) | 14:38 |
| auniyal | somewhat offtopic, I would like suggest to create a groups of interested people for particular topic , eventlet-removal, CBT if required separate syncup too - this will help fast feedback | 14:38 |
| erlon | so, if the reviewers don't get to new tools, we will be drawned | 14:38 |
| auniyal | *separate from upstream meet I meant | 14:38 |
| fungi | some project teams already do hold subteam meetings, yes | 14:39 |
| fungi | you can see them if you skim the list at meetings.opendev.org | 14:39 |
| fungi | it's certainly a pattern that could be replicated | 14:39 |
| jbernard | fungi: for review response time improvements, do we know if it's across all reviewers, or mainly driven by one or two? | 14:41 |
| erlon | fungi: is there any discussions regarding to adding AGENTS.md, CLAUDE.md files into projects? That could steer the models into generating better and more concise code adhering to the projects guidelines | 14:41 |
| fungi | i'm a little hesitant at the suggestions i keep seeing in various places to throw new-found ai tools at reviewing tasks, so far people who have done that have told me that while it helped them do more thorough reviews and maybe catch more bugs, it didn't improve their review bandwidth at all | 14:41 |
| fungi | erlon: yes, get involved in the agentic workflows effort under the tact sig, they have a #openstack-agentic-workflows irc channel for those topics too | 14:41 |
| fungi | but they're collaborating on coming up with standard agents files and skills and whatnot | 14:42 |
| jbernard | erlon: https://review.opendev.org/c/openstack/cinder/+/994007 | 14:42 |
| jbernard | fungi: for key takeaways for improvements, | 14:43 |
| jbernard | * better transparency on review priorities | 14:43 |
| fungi | the other concern i have with suggestions to just lean harder on ai is that so far all of the root problems we've found stemmed from insufficient communication between humans, so removing humans even farther is likely to make things worse rather than better | 14:43 |
| jbernard | * faster response times for larger patches | 14:43 |
| fungi | or maybe having fewer large patches (which probably means openstack doing a better job of educating contributors) | 14:45 |
| fungi | and yeah, the third takeaway was growing the core team, which it sounds like is in progress for hibiscus already | 14:46 |
| jbernard | in progress | 14:46 |
| erlon | jbernard: I woke up with this in mind some time ago, may be a start: https://etherpad.opendev.org/p/cinder-priority-guidelines | 14:46 |
| jbernard | it's defintely a start, ordering them and being able to identify the different categories quickly would help immensely | 14:48 |
| agalica | I usually dream about awesome things like the apocalypse. It's depressing to me to know that erlon dreams of openstack | 14:49 |
| fungi | i dream of the openstackapocalypse | 14:49 |
| erlon | haha lol, I have a morning notepad, for those kind of goodies. they always come unexpectedly | 14:50 |
| agalica | I usually get mine in the shower where there isn't a place to write it down, haha | 14:50 |
| fungi | you need a greaseboard in the shower where you can take notes | 14:51 |
| Anoop_Shukla | A quick suggestion - Why don’t we have openstackathons? | 14:51 |
| fungi | or one of those pads divers use | 14:51 |
| erlon | It would be nice if gerrit had tags | 14:51 |
| fungi | Anoop_Shukla: like hackathons? we probably do and have different terms for them | 14:51 |
| jbernard | fungi: from your understanding, if you could me one, or two, recommendations to this group, do you think we've missed anything? priorities and reponse time; anything else? | 14:51 |
| Anoop_Shukla | Can be a good way across openstack projects where contributers come together and solve problems.. | 14:51 |
| fungi | erlon: gerrit has hashtags, is that what you're looking for? | 14:51 |
| jbernard | fungi: s/me/make | 14:51 |
| Anoop_Shukla | Cool..would be good to know when they happen and participate | 14:52 |
| fungi | jbernard: keep growing the core team if you can, because the incoming volume seems to be on a steady rise | 14:52 |
| jbernard | fungi: got it | 14:52 |
| erlon | does it, I mean, you tag something like, 'low-prio', 'ci-blocker', then you can filter all of those in a list | 14:52 |
| agalica | yes | 14:53 |
| jbernard | erlon: i think that's how it works | 14:53 |
| agalica | look at the "tags" section | 14:53 |
| fungi | Anoop_Shukla: some happen in the scope of local meetups for user groups, some happen at per-team midcycle events, some happen in ptg sessions (this is a lot of what the ptg was originally meant for) | 14:53 |
| jbernard | we do that with eventlet-removal at lease | 14:53 |
| agalica | on the left hand side around the checklist | 14:53 |
| Anoop_Shukla | Okay good to know..thanks for the info @fungi | 14:54 |
| agalica | hmm, I'm not seeing them now, weird. maybe it only shows up if there's actual tags | 14:54 |
| erlon | ah, yes, topic | 14:54 |
| fungi | Anoop_Shukla: you could also consider cinder's semi-frequent "festival of reviews" a sort of recurring hackathon but for reviewing instead of coding | 14:54 |
| eharney | topic is a different thing | 14:54 |
| erlon | But that is hard to use for that kind of organization, since gerrit auto-create then, and I think you can only add one topic, and anyone can add | 14:55 |
| eharney | the hashtags field is under "Show All" | 14:55 |
| fungi | right, we've been discouraging continued use of teh change "topic" field in gerrit for a while, because upstream gerrit has switched from treating it as informal metadata to having some configurable behaviors tied to same-topic changes | 14:55 |
| agalica | ahh, there it is, thanks eharney | 14:55 |
| fungi | what people previously relied on topics for is mostly better handled with gerrit hashtags now | 14:55 |
| jayaanand | I updated NetApp pending patches in festival-of-reviews etherpad. Need cors attention for these patches | 14:56 |
| jbernard | 4 minutes remaining, | 14:56 |
| jbernard | #topic open discussion | 14:56 |
| fungi | i didn't mean to consume your entire meeting, sorry! | 14:56 |
| fqi | Could you please tell me the deadline of Cinder Spec for Hibiscus? | 14:57 |
| agalica | consumption with mushrooms usually goes the other way around | 14:57 |
| Jyotsna | New HPE Alletra MP driver patch : https://review.opendev.org/c/openstack/cinder/+/969373 Thanks and appreciate consolidated feedback from <erlon>. We are in process of addressing the feedback. Soon will update the patch. We have 2 queries on patch feedback, whenever possible <erlon> can help answering them. We have made an improvement in CI process, transitioned from zuul to Jenkins which made the CI runs smooth and easy to debug. | 14:57 |
| jbernard | fungi: no worries, thanks so much for the data collection and summaries, much much appreciated | 14:57 |
| harsh | I have added a couple of small bug fix patches for IBM driver. They have multiple +1s and reviews so would appreciate your attention in closing those small patches quickly. | 14:58 |
| fungi | if cinder has a formal spec freeze you could consider including it in the release schedule like nova and manila do: https://releases.openstack.org/hibiscus/schedule.html | 14:59 |
| erlon | Jyotsna: Ill address then later today | 14:59 |
| Jyotsna | Thanks <erlon> | 15:00 |
| erlon | harsh: link? | 15:00 |
| jbernard | fqi: not formally defined, but soon, do you have something that needs attention? | 15:00 |
| harsh | erlon : https://etherpad.opendev.org/p/cinder-hibiscus-reviews | 15:00 |
| harsh | Last patches in the bugs section | 15:01 |
| erlon | thanks | 15:03 |
| jbernard | #endmeeting | 15:04 |
| opendevmeet | Meeting ended Wed Jun 24 15:04:30 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:04 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-06-24-14.02.html | 15:04 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-06-24-14.02.txt | 15:04 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-06-24-14.02.log.html | 15:04 |
| jbernard | thanks everyone | 15:04 |
| amoralej | Hi, I'm working in some watcher integration with cinder. I've found that while the volume_type in Volume cinder API is the volume_type name, in the notifications payload for volumes, volume_type is ID and name is not reflected. Is that expected and documented somewhere? | 15:04 |
| fqi | jbernard: thank you. I would like to merge the spec retype volume including snapshots into Hibiscus. | 15:05 |
| hemna | hey sorry I missed the meeting. busy work day. I have a few patches I'd like to request for reviews. | 15:09 |
| hemna | https://review.opendev.org/c/openstack/cinder/+/992667 | 15:09 |
| hemna | https://review.opendev.org/c/openstack/cinder/+/991876 | 15:09 |
| opendevreview | Merged openstack/cinder master: Fungible: use volume_utils to upload volume as image https://review.opendev.org/c/openstack/cinder/+/983237 | 15:24 |
| opendevreview | Merged openstack/cinder master: Add correct mock for rados.Error https://review.opendev.org/c/openstack/cinder/+/991867 | 15:24 |
| opendevreview | Saikumar Pulluri proposed openstack/cinder master: NetApp ONTAP: begin closing ZAPI-to-REST gaps https://review.opendev.org/c/openstack/cinder/+/993363 | 17:50 |
| opendevreview | Vinícius Fernando Rocha Lobo proposed openstack/cinder master: Fix service_uuid not set on cinder manage volumes https://review.opendev.org/c/openstack/cinder/+/994819 | 20:14 |
| opendevreview | Vinícius Fernando Rocha Lobo proposed openstack/cinder master: Fix service_uuid not set on cinder manage volumes https://review.opendev.org/c/openstack/cinder/+/994819 | 21:11 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!