| opendevreview | Sam Clippinger proposed openstack/cinder master: Add additional rbd options to improve download speed. For some rudimentary benchmarks, see: https://bugs.launchpad.net/cinder/+bug/2120509 https://review.opendev.org/c/openstack/cinder/+/957190 | 01:29 |
|---|---|---|
| opendevreview | wanghongtao proposed openstack/cinder master: fix: validate extend size using admin metadata https://review.opendev.org/c/openstack/cinder/+/993707 | 02:02 |
| opendevreview | wanghongtao proposed openstack/cinder master: Fix volume extend size validation logic https://review.opendev.org/c/openstack/cinder/+/993707 | 02:03 |
| opendevreview | wanghongtao proposed openstack/cinder master: Fix retype failure between DM and Huawei storage https://review.opendev.org/c/openstack/cinder/+/992851 | 03:05 |
| opendevreview | Liron Kuchlani proposed openstack/cinder-tempest-plugin master: Add support for volume replication https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/952882 | 07:00 |
| opendevreview | Amit Uniyal proposed openstack/cinder master: Add CI job to run cinder-tempest-plugin tests with native threading (no-eventlet) https://review.opendev.org/c/openstack/cinder/+/991065 | 07:05 |
| opendevreview | Amit Uniyal proposed openstack/cinder master: Add CI jobs to run tests with native threading (no-eventlet) https://review.opendev.org/c/openstack/cinder/+/988747 | 07:05 |
| opendevreview | Amit Uniyal proposed openstack/cinder master: Adds threading mode tests for semaphore funcs https://review.opendev.org/c/openstack/cinder/+/990681 | 07:05 |
| opendevreview | Amithabh proposed openstack/cinder master: Shield backend driver exceptions from API response https://review.opendev.org/c/openstack/cinder/+/993715 | 07:19 |
| *** elodilles_pto is now known as elodilles | 07:50 | |
| opendevreview | Lucas Lafontaine proposed openstack/cinder master: Fujitsu Driver: Improve the SSH connection method https://review.opendev.org/c/openstack/cinder/+/907127 | 07:53 |
| opendevreview | Liron Kuchlani proposed openstack/cinder-tempest-plugin master: Add failover/failback API tests for volume replication https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/993659 | 07:54 |
| opendevreview | Saikumar Pulluri proposed openstack/cinder master: NetApp ONTAP: begin closing ZAPI-to-REST gaps https://review.opendev.org/c/openstack/cinder/+/993363 | 10:09 |
| opendevreview | Nimesh Desai proposed openstack/cinder master: [IBM SVf Driver] Added support of clone operation for volumegroup https://review.opendev.org/c/openstack/cinder/+/982617 | 10:18 |
| opendevreview | Nimesh Desai proposed openstack/cinder master: [IBM SVf driver] Adding support for individual volume snapshots https://review.opendev.org/c/openstack/cinder/+/949006 | 10:18 |
| opendevreview | Amithabh proposed openstack/cinder master: Shield backend driver exceptions from API response https://review.opendev.org/c/openstack/cinder/+/993715 | 11:19 |
| jbernard | #startmeeting cinder | 14:00 |
| opendevmeet | Meeting started Wed Jun 17 14:00:50 2026 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 | #link https://etherpad.opendev.org/p/cinder-hibiscus-meetings | 14:00 |
| 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:01 |
| jbernard | #topic roll call | 14:01 |
| jbernard | o/ | 14:01 |
| auniyal | o/ | 14:01 |
| rosmaita | o/ | 14:01 |
| whoami-rajat | hi | 14:01 |
| * fungi is around for his topic too | 14:02 | |
| agalica | o/ | 14:02 |
| agalica | oh snap, it's Rajat | 14:02 |
| Jyotsna | o/ | 14:03 |
| whoami-rajat | lol, didn't realize i was famous :D | 14:03 |
| agalica | :D | 14:03 |
| fungi | infamous | 14:03 |
| jbernard | welcome everyone | 14:07 |
| jbernard | #topic annoucements | 14:07 |
| jbernard | periodic weekly jobs... | 14:08 |
| jbernard | I think we're in good shape | 14:08 |
| jbernard | yes | 14:08 |
| rosmaita | \o/ | 14:09 |
| fungi | congrats! that's no small feat | 14:09 |
| rosmaita | i anticipate something breaking after M-2, but i am a pessimist | 14:09 |
| jbernard | if it's going to break, it'll break late ;) | 14:10 |
| rosmaita | at the most inconvenient time | 14:10 |
| jbernard | but that time is not now, so yay! | 14:11 |
| rosmaita | :D | 14:11 |
| jbernard | that's all for annoucements, fungi you're up | 14:11 |
| fungi | i'll try to make this quick | 14:11 |
| fungi | #topic Bridging the Gap Gazpacho Cycle Survey and Metrics Analysis | 14:12 |
| jbernard | #topic survey and metrics analysis (fungi) | 14:12 |
| jbernard | #topic Bridging the Gap Gazpacho Cycle Survey and Metrics Analysis | 14:12 |
| jbernard | sorry | 14:12 |
| fungi | no problem, i wasn't sure if you were waiting for me to #topic (not sure if i can as not a chair anyway) | 14:12 |
| fungi | ildiko posted openstack-wide 2026.1 (gazpacho) cycle retrospective contributor/maintainer survey results and metrics to openstack-discuss recently: | 14:13 |
| fungi | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/3POML6PYPDWQNYM5IFB54O4ASXEPK4Z5/ Bridging the Gap Gazpacho Cycle Retrospective Survey Results | 14:13 |
| fungi | #link https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/CVLQIFEVNNJJ7BJSFZJ4RNZZA7ELQKDS/ Bridging the Gap Gazpacho Cycle Retrospective Metrics Analysis | 14:13 |
| fungi | i and the other community managers on the openinfra foundation staff have also been digging into team-specific details and i'm doing a round of outreach similar to past cycles, to go over how things may have changed | 14:13 |
| fungi | cinder had more survey responses than most teams in openstack (4 contributors and 2 maintainers), so this is great though additional participants for the next round would, as always, help us draw even more accurate conclusions | 14:13 |
| fungi | most contributors indicated spending 2 or more years in the project, half the respondents had experience in at least 1 other open source project while the other half only participate in this one | 14:13 |
| fungi | average ratings on the contributor survey questions were encouragingly positive overall, with actionable reviewer feedback averaging highest at 4.5 out of 5 while timeliness of review feedback was scored lowest at 2.75 average out of 5 | 14:13 |
| fungi | the most common contribution challenge selected was getting reviewer attention, followed by struggles with unstable testing, conflicting feedback from different maintainers, and nit-picky reviews at times while too shallow at other times | 14:14 |
| fungi | established contributors remarked that the review experience in cinder had improved compared to prior cycles, noting the team is small and overloaded by the volume, but indicated disappointment in some features not landing for gazpacho | 14:14 |
| fungi | the new survey questions about ai tools indicate only a quarter of respondents used any ai assistance (copilot for writing some unit tests), while the majority used no ai tools when contributing | 14:14 |
| fungi | the rankings on the maintainer survey responses were slightly lower, ranging from 4 (contributor documentation being comprehensive and up to date) down to 2 (receiving actionable and timely feedback from other reviewers) | 14:14 |
| fungi | contributing challenges among maintainers were very similar to those noted by contributors, most commonly trouble getting review attention from other maintainers but also unclear priorities, scope creep from reviewers, and unstable testing | 14:15 |
| fungi | when reviewing, maintainers saw many that needed further discussion but change owners were unavailable, incomplete changes (missing tests and docs), as well as priority mismatches | 14:15 |
| fungi | one maintainer indicated using copilot to help review changes (to better understand the changes and their risks), while the other did not rely on any ai assistance in their code review workflow | 14:15 |
| fungi | both maintainers used some ai assistance in preparing their own changes however, to help write unit tests and cleaner code while improving their development speed and efficiency; both used copilot, but one of them additionally used cursor | 14:15 |
| fungi | maintainers also left similar comments as contributors with regard to review efficiency challenges, the overloaded maintainer team's inability to tackle larger reviews, and priority features slipping to later cycles | 14:15 |
| fungi | on the recurring priority mismatch, contributors indicated looking mainly at collaborative docs like an etherpad and irc/mailing lists but maintainers said they track them with gerrit change topics/priority votes and specs/blueprints | 14:16 |
| erlon | \o | 14:16 |
| Anoop_Shukla | O/ | 14:16 |
| fungi | as for metrics, active maintainer count held steady at the same level as flamingo (still down significantly from epoxy) while reviewer count rose by a modest 1% (now only 5% away from the level it was at in bobcat) | 14:16 |
| fungi | interestingly, even though change volume increased 17% the team still managed to basically keep up by closing 91% as many as were opened, and a whopping 53% more than were closed in flamingo! | 14:16 |
| jayaanand | O/ | 14:16 |
| rosmaita | \o/ | 14:16 |
| fungi | an important note about metrics this time: since flamingo we've switched from bitergia to lfx insights as our data source, and while the numbers between them are similar there are some slight differences in how activities are counted | 14:16 |
| fungi | because of this, we re-ran metrics analyses for previous cycles against the new system in order to make sure we weren't trending numbers from different backends and drawing misleading conclusions due to changing measurement methods | 14:16 |
| fungi | anyway, that was a quick dump, i know it's a lot to take in but i didn't want to eat up too much of your meeting, so i'll put this on the agenda again for next week to give everyone time to digest and come up with questions or ideas | 14:17 |
| fungi | one thing we're interested in finding out is whether the team implemented any new contributor or reviewer practices over the course of the last cycle, for example techniques that were discussed in the previous round of analyses | 14:17 |
| fungi | though i'm happy to answer any immediate feedback now too if there's time | 14:17 |
| rosmaita | "whether the team implemented any new contributor or reviewer practices over the course of the last cycle" -- i don't think so? | 14:19 |
| jbernard | rosmaita: for G? | 14:19 |
| rosmaita | yeah, i don't think we did anything different from F | 14:19 |
| fungi | yeah, for example we did see a lot more changes closed last cycle compared to flamingo, but that could just be attributed to the number of changes that were opened also increasing | 14:20 |
| jbernard | not officially, no. i think any changes were individually motivated | 14:20 |
| rosmaita | yeah, i wonder whether the current contributors just got used to how we work | 14:20 |
| fungi | not mentioned above, it looked like the time to response on new changes improved dramatically | 14:20 |
| rosmaita | well, that would be pretty great! | 14:21 |
| fungi | the mean dropped from 22 days to 5, and the average from 69 days to 30 | 14:21 |
| jbernard | that's unexpected, and encouraging | 14:21 |
| rosmaita | wow, that is impressive | 14:21 |
| fungi | so someone seems to be reviewing new changes a lot more often/quickly | 14:21 |
| fungi | note that could be basically any comment or vote, even if it's just to flag review priority +/-1 or something | 14:22 |
| rosmaita | still, any action on a new patch is good action | 14:22 |
| jbernard | i agree | 14:22 |
| fungi | yes, it helps especially casual contributors know their change has been at least noticed/seen | 14:23 |
| fungi | and in gazpacho that seemed to happen within a week most of the time, but still within a month on average | 14:23 |
| fungi | (very long-unaddressed changes skew the averages significantly, so the mean is a better representation of day-to-day activity) | 14:24 |
| fungi | anyway, i didn't have anything else specific for this week, but i'll pop this back onto the agenda for next week and give people some time to ponder | 14:25 |
| jbernard | thanks fungi, as always, much appreciated | 14:26 |
| rosmaita | thanks! it is nice to have some good news | 14:26 |
| fungi | my pleasure! | 14:26 |
| agalica | always the most useful mushroom in the chat | 14:26 |
| rosmaita | fungi: are you a project-config core? | 14:26 |
| fungi | yes. ping me outside the meeting if you need me to look at something there | 14:26 |
| jbernard | rosmaita: i see what you did there ;) | 14:26 |
| rosmaita | fungi: will do | 14:27 |
| jbernard | as a side annoucement, rosmaita has a patch to make the review priority persist across rechecks, updates, rebases, etc | 14:28 |
| jbernard | https://review.opendev.org/c/openstack/project-config/+/993177 | 14:28 |
| erlon | Not sure how much of that counted for that measure but I do have a filter to triage the last XM/S changes. Those are usualy minor work quick gains I think it worth spending time: | 14:28 |
| erlon | https://review.opendev.org/q/project:openstack/cinder+status:open+is:mergeable+-is:wip+label:Code-Review%3E%3D0+-label:Code-Review%3C%3D-1+label:Verified%3D%252B1+-age:30d++-label:Code-Review%3D1%2Cuser%3Dself+-label:Code-Review%3D-1%2Cuser%3Dself+-path:%22%5Ecinder%5C/volume%5C/drivers.*%24%22 | 14:28 |
| fungi | oh yep, i saw rosmaita's change, will take a look shortly | 14:29 |
| jbernard | this should help with our prioritization piece in fungi's earlier messages | 14:29 |
| fungi | erlon: yeah if you were actively triaging changes like that, it could easily make the observed impact, great call-out! | 14:29 |
| agalica | jay has been reviewing like a madman this cycle too | 14:30 |
| agalica | jayaanand | 14:30 |
| jbernard | yep, we have serveral folks stepping up recently, many thanks for everyone contributing | 14:30 |
| jbernard | #topic feedback on contributer doc changes (rosmaita) | 14:31 |
| jbernard | #link https://review.opendev.org/c/openstack/cinder/+/993467 | 14:32 |
| rosmaita | this is a follow-up from the ptg | 14:32 |
| rosmaita | agalica "volunteered" to review the curent doc | 14:32 |
| rosmaita | https://etherpad.opendev.org/p/contributor-review-agalica | 14:32 |
| jbernard | voluteered, voluntoold, all same :) | 14:33 |
| rosmaita | and erlon give me feedback on the first draft | 14:33 |
| agalica | nah, I actually voluteered | 14:33 |
| agalica | haha | 14:33 |
| rosmaita | so would be helpful for some new-ish contributors to take a look | 14:33 |
| rosmaita | let us know if there's anything missing that would be helpful | 14:33 |
| rosmaita | but thanks to agalica all the info on the doc now is accurate | 14:34 |
| Anoop_Shukla | Can we add the logging guidelines to this? | 14:34 |
| agalica | happy to help | 14:34 |
| fungi | as a follow-on to the last topic, making sure the contributor doc helps clarify where and how change review prioritization is tracked by the maintainers may solve some of the impedance mismatch we saw in the survey responses | 14:34 |
| jayaanand | anjalica, jbernard: thanks, appreciate it! trying to keep reviews moving this cycle without bothering everyone too much :) | 14:35 |
| rosmaita | Anoop_Shukla: maybe at the bottom, in the "special info for driver maintainers" section? | 14:35 |
| Anoop_Shukla | yep.. | 14:35 |
| rosmaita | good idea, i can add that | 14:35 |
| Anoop_Shukla | https://etherpad.opendev.org/p/OpenStack_Cinder_Logging_Guidelines | 14:36 |
| rosmaita | thanks! | 14:36 |
| rosmaita | though we should probably turn that etherpad into a "real" document | 14:36 |
| Anoop_Shukla | I can convert it to a doc and float a PR | 14:37 |
| jbernard | that would be great | 14:38 |
| rosmaita | i agree, in the meantime we can use the etherpad link and then update once the other one lands | 14:38 |
| jbernard | rosmaita: you also have a patch that needs attention (https://review.opendev.org/c/openstack/cinder/+/978080)? | 14:38 |
| rosmaita | yeah, Rajat and Eric noticed a possible race condition in PS 4 on that review | 14:39 |
| rosmaita | it has been addressed in PS 5 | 14:39 |
| rosmaita | but race conditions are tricky (at least I think so) | 14:40 |
| rosmaita | so it would be good if a few people could take a look to verify that it looks ok | 14:40 |
| rosmaita | that's all | 14:41 |
| jbernard | thanks | 14:43 |
| jbernard | #topic open discussion | 14:43 |
| Jyotsna | New HPE Alletra MP driver patch : https://review.opendev.org/c/openstack/cinder/+/969373 https://review.opendev.org/c/openstack/requirements/+/974322 | 14:43 |
| Jyotsna | We truly appreciate the reviewer's valuable feedback. We have addressed all comments to improve the patch. As this is critical for us to provide timelines to customers, we would be grateful for help in moving it towards merge when possible. | 14:43 |
| agalica | Festival of Reviews friday since no one's mentioned it (that I saw) | 14:45 |
| rosmaita | arrgh ... it's on a holiday | 14:47 |
| agalica | oh, right | 14:47 |
| rosmaita | this seems to happen a lot | 14:47 |
| agalica | I don't even have work that day, I forgot | 14:47 |
| agalica | push it to next week? | 14:47 |
| rosmaita | for non-USA people, it's the "Juneteenth" holiday | 14:47 |
| fungi | in other news, rosmaita's sticky review-priority vote change should be taking effect here in the next few minutes (the deploy job to update the project acl in production is running now) | 14:49 |
| rosmaita | \o/ | 14:49 |
| rosmaita | thanks fungi | 14:49 |
| Jyotsna | I also want to bring one point that we have been waiting for the new driver merge almost for a year so request reviewers to provide their valuable feedback consolidated so we can address them at stretch. | 14:49 |
| Jyotsna | As an improvement we have transitioned our CI to Jenkins for new driver to make the CI runs more smooth and easy to debug in case of failures. | 14:50 |
| rosmaita | Jyotsna: yes, you and your team have been patient and have responded to reviews requesting big changes | 14:50 |
| Jyotsna | Thanks | 14:50 |
| jbernard | rosmaita: should we move the review festival do you think? or skip | 14:51 |
| Anoop_Shukla | We have some netapp patches we wanted to bring up in review festival. | 14:52 |
| rosmaita | maybe have it, but keep in mind that the turnout may be low | 14:52 |
| Anoop_Shukla | But if its going to be moved we can add it in etherpad for offline look | 14:52 |
| agalica | I won't be there friday, but i'm going to set aside some time next week to do some reviewing regardless of whether or not we do the fest this week or next | 14:53 |
| erlon | I'll be around both Fridays, so for me either of them are good | 14:54 |
| rosmaita | maybe what we do is keep the current schedule, move the meeting to meetpad, and i will mention in the email that people who want to participate async can look at the etherpad | 14:54 |
| rosmaita | to get some idea of patches to concentrate on | 14:55 |
| jbernard | that sounds good | 14:55 |
| rosmaita | ok | 14:55 |
| agalica | oh, since we're still here: | 14:55 |
| agalica | can we encourage people to start using the review request etherpad again? | 14:56 |
| agalica | https://etherpad.opendev.org/p/cinder-hibiscus-reviews | 14:56 |
| agalica | I know there's far more patches than that that people want reviewed | 14:56 |
| jbernard | yes, that's a good call | 14:57 |
| jbernard | im hoping we can combine that with brian's persistent priorty change | 14:58 |
| agalica | yeah, good idea | 14:58 |
| jbernard | thanks for mentioning that | 14:58 |
| agalica | np, always at service my leige | 14:58 |
| agalica | or is it liege | 14:59 |
| agalica | i before e except after c and the billion exceptions | 14:59 |
| jbernard | can't forget those :) | 14:59 |
| jbernard | ok, last call for things and/or stuff | 15:00 |
| jbernard | thanks everyone | 15:00 |
| jbernard | #endmeeting | 15:01 |
| opendevmeet | Meeting ended Wed Jun 17 15:01:00 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:01 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-06-17-14.00.html | 15:01 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-06-17-14.00.txt | 15:01 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-06-17-14.00.log.html | 15:01 |
| erlon | \o | 15:01 |
| agalica | somewhere there's a very slow typist crying to themselves about not getting in their question in time | 15:01 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Remove F824 skip https://review.opendev.org/c/openstack/cinder/+/993772 | 15:08 |
| opendevreview | Eric Harney proposed openstack/cinder master: Fix F824 hacking errors in rpc.py https://review.opendev.org/c/openstack/cinder/+/993773 | 15:08 |
| opendevreview | Merged openstack/cinder master: typing: "Import" objects https://review.opendev.org/c/openstack/cinder/+/985090 | 15:33 |
| opendevreview | Merged openstack/cinder stable/2025.1: Fix missing encryption params in Quobyte driver https://review.opendev.org/c/openstack/cinder/+/981129 | 16:41 |
| opendevreview | Merged openstack/cinder master: Remove driver-requirements.txt https://review.opendev.org/c/openstack/cinder/+/991215 | 18:28 |
| opendevreview | Merged openstack/cinder master: Remove TODO re: Semaphore for eventlet-removal https://review.opendev.org/c/openstack/cinder/+/992947 | 18:29 |
| opendevreview | Ghanshyam Maan proposed openstack/cinder-tempest-plugin master: Use the new variable to enable the RBAC new defaults https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/993837 | 20:24 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!