| opendevreview | Anthony Galica proposed openstack/cinder master: [Hitachi] Bug fix: Introduce Host Group and target/WWN caching and batching to address severe performance issues (especially when using custom Host Groups, or creating several VMs at once). https://review.opendev.org/c/openstack/cinder/+/977285 | 01:02 |
|---|---|---|
| opendevreview | mengxiangzhi proposed openstack/cinder master: Support zte vstorage driver https://review.opendev.org/c/openstack/cinder/+/969207 | 02:14 |
| *** erlon2 is now known as erlon | 03:28 | |
| opendevreview | Merged openstack/devstack-plugin-ceph master: Add nova ceph jobs for more coverage https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/977154 | 04:14 |
| *** deepa__ is now known as deepa_ | 05:31 | |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Doc: Advertise clone across pool as driver feature https://review.opendev.org/c/openstack/cinder/+/977924 | 06:38 |
| opendevreview | mengxiangzhi proposed openstack/cinder master: Support zte vstorage driver https://review.opendev.org/c/openstack/cinder/+/969207 | 07:07 |
| hong-p[m] | hi jbernard: please review for a small but critical bug fix https://review.opendev.org/c/openstack/cinder/+/961906 | 10:48 |
| opendevreview | Merged openstack/cinder master: Pure Storage - bump version numbers for 2026.1 (Gazpacho) https://review.opendev.org/c/openstack/cinder/+/962866 | 11:04 |
| opendevreview | Abhishek Gupta proposed openstack/cinder master: Dell PowerFlex : Fix multiple broken functions https://review.opendev.org/c/openstack/cinder/+/950546 | 12:05 |
| opendevreview | Eric Harney proposed openstack/cinder master: HPE 3par: Handling session leak and multithreading issues https://review.opendev.org/c/openstack/cinder/+/976596 | 12:37 |
| opendevreview | Brian Rosmaita proposed openstack/cinder stable/2025.1: [stable-only] pep8, unit tests, doc fixes https://review.opendev.org/c/openstack/cinder/+/978791 | 13:28 |
| jbernard | #startmeeting cinder | 14:02 |
| opendevmeet | Meeting started Wed Mar 4 14:02:38 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 | 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 | 14:02 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-gazpacho-meetings | 14:02 |
| jbernard | #help | 14:03 |
| agalica_ | O/ | 14:03 |
| kaisers | o/ | 14:03 |
| rosmaita | o/ | 14:03 |
| harsh | o/ | 14:03 |
| jungleboyj | o/ lurking | 14:03 |
| Anoop_Shukla | O/ | 14:03 |
| jbernard | jungleboyj: good to see you | 14:04 |
| raghavendrat | hi | 14:04 |
| jungleboyj | Thanks. Sorry I have been so absent. | 14:04 |
| jbernard | no worries | 14:04 |
| Jyotsna | hi | 14:06 |
| erlon | o/ | 14:06 |
| jbernard | hello everyone | 14:07 |
| jbernard | #topics annoucements | 14:08 |
| jbernard | we're doing a final review gathering on friday | 14:08 |
| jbernard | we've been reviewing as much as possible, but there are still a lot of patches that need attention | 14:08 |
| jbernard | rosmaita: stable updates? | 14:08 |
| rosmaita | yeah | 14:09 |
| rosmaita | we have new stable/2026.1 branches for os-brick, cinderclient, and the brick-cinderclient-extension | 14:09 |
| rosmaita | cinder stable gates are borked | 14:09 |
| rosmaita | but, i think we have a solution | 14:10 |
| rosmaita | https://review.opendev.org/c/openstack/cinder/+/978238 | 14:10 |
| rosmaita | that's for flamingo, will need to be backported to epoxy, and i think dalmatian (need to check) | 14:10 |
| rosmaita | i guess that's it | 14:11 |
| erlon | btw, thanks jbernard and rosmaita for the help with the os-brick patch. Really appreciated your effort to get the iSCSI patch in | 14:11 |
| rosmaita | np, sure hope it doesn't break anything :P | 14:12 |
| erlon | :O, fingers crossed | 14:12 |
| opendevreview | Nimesh Desai proposed openstack/cinder master: [IBM SVf Driver] Define meaningful names for error constants https://review.opendev.org/c/openstack/cinder/+/959683 | 14:12 |
| agalica_ | Yeah, thank you guys for your help with our patches as well. | 14:14 |
| jbernard | since we're here, quick update on upcoming PTG | 14:15 |
| jbernard | it's late April, 20 - 24 | 14:15 |
| nimeshdesai | opendevreview: That patch got merge conflicts due to merge of some other patch. I'll be resolving it and uploading again. Will require +2s. | 14:15 |
| jbernard | the release name is 'Hibiscus' | 14:15 |
| jbernard | and our working etherpad is https://etherpad.opendev.org/p/cinder-hibiscus-ptg | 14:16 |
| jbernard | feel free to add topics and details as you wish, ill construct an agenda from those notes and fill gaps with review time | 14:17 |
| agalica_ | Guessing those dates are just copy/paste for now | 14:17 |
| jayaanand | i need core attention for this patch https://review.opendev.org/c/openstack/cinder/+/976335 | 14:17 |
| jbernard | agalica_: yes | 14:17 |
| jbernard | it's that week, but not necessarily all of those days, we typically skip monday, mondays suck | 14:18 |
| agalica_ | Mondays do suck | 14:19 |
| jbernard | rosmaita: i haven't looked yet, how is the stable gate? did you squash the problematic commits? | 14:19 |
| jungleboyj | Lol. Indeed. | 14:19 |
| rosmaita | jbernard: yes, that patch is in the gate now | 14:19 |
| jbernard | awesome | 14:19 |
| rosmaita | approved about 40 minutes ago, so we're probably looking at 2.5 more hours | 14:20 |
| jbernard | kk | 14:20 |
| Anoop_Shukla | We have multiple patches from NetApp pending for first reviews. Our concern is a lot of these features are being awaited by customers on our side. Wanted to understand how to address these reviews. | 14:20 |
| rosmaita | what's your CI situation? | 14:20 |
| jbernard | re netapp patches, what is the CI status? ive looked a few of them and i see it's not passing | 14:20 |
| Anoop_Shukla | NetApp CI has some issues currently (which seems temporary) and we are trying to address it ASAP. | 14:22 |
| jbernard | k, let us know when that's resolved | 14:22 |
| harsh | Hi.. two of our patches have multiple +2s. Waiting for the workflow +1 so that they can be merged: https://review.opendev.org/c/openstack/cinder/+/916722/91 and https://review.opendev.org/c/openstack/cinder/+/929906/62 | 14:24 |
| sanketh_shetty | Requesting one more +2 for - https://review.opendev.org/c/openstack/cinder/+/973908. Very small change. | 14:24 |
| jbernard | harsh: i looked at at least one of those earlier, you need to stop making changes once it's been approved, a new change removes the +W | 14:25 |
| jbernard | harsh: https://review.opendev.org/c/openstack/cinder/+/929906 for example | 14:25 |
| jbernard | harsh: it's been approved 5 times | 14:25 |
| jbernard | but a rebase clobbers the +W on each attempt | 14:26 |
| harsh | ouh. yea there were some rebasing going on on some related patches. sorry about that. We have asked the team to stop the rebasing for a while. | 14:26 |
| jbernard | which costs time and could be frustrating | 14:26 |
| harsh | Could you please approve them once again .. sorry for the multiple rebasing.. | 14:26 |
| jbernard | does "for a while" mean that the patches are not truely ready for review? | 14:26 |
| jbernard | the continous updates could be interpreted that way, potentially | 14:27 |
| harsh | these ones are ready. there are some newer patches that we submitted and are still in review. | 14:27 |
| hong-p[m] | jbernard: hi I requested some bug fix could you review this? https://review.opendev.org/c/openstack/cinder/+/961906 | 14:28 |
| jbernard | ok, i will try to circle back today, raise them in friday's meeting if not | 14:28 |
| harsh | but since all these features are dependent on each other and if anyone from the team rebases one of the new patch, the whole chain looses the +W | 14:28 |
| jbernard | yeah, i understand the dilema, ideally things should settle at some point | 14:29 |
| harsh | i totally agree. and trust me we are working towards it. These older patches have been ready for a while now. but the newer ones are getting customer reviews as well. Hence the multiple rebasing. Team is building some process to drastically reduce the rebasing which will save your and our time as well. | 14:31 |
| jbernard | hong-p[m]: ack, noted for our friday meeting (https://etherpad.opendev.org/p/cinder-festival-of-reviews#L23) unless we get to it sooner | 14:31 |
| hong-p[m] | thank you! | 14:31 |
| jbernard | harsh: ok, will take a look at the ones you've mentioned, remind us friday if no movement | 14:32 |
| jbernard | noonedeadpunk: your patch is on the list too | 14:32 |
| Anoop_Shukla | @jbernard: while we are trying to fix our CI - can we not get exceptions for the patches where NetApp CI has failed? At least can we have the review process start on these patches becuase for us to address the comment will anyway be a cyclic review process | 14:32 |
| opendevreview | Merged openstack/cinder stable/2025.2: [stable-only] pep8, unit tests, doc fixes https://review.opendev.org/c/openstack/cinder/+/978238 | 14:32 |
| harsh | thankyou ! appreciate your patience :) | 14:32 |
| rosmaita | Anoop_Shukla: the problem is that as you can see, everyone needs reviews, so the priority is to review the driver patches with green third party CI, because those can merge | 14:34 |
| rosmaita | so even though theoretically, your patches could be reviewed | 14:34 |
| rosmaita | as a practical matter, they get back-burnered until the CI shows that they are working | 14:35 |
| noonedeadpunk | jbernard: isn't we're past feature freeze? | 14:35 |
| rosmaita | or at least not causing regressions | 14:35 |
| jbernard | noonedeadpunk: technically, but we will evaluate a ff exception | 14:35 |
| noonedeadpunk | ++ gotcha, would be much appreciated | 14:36 |
| kaisers | rosmaita: regarding gate checks, do we need to rebase changes on stable/2025.2 in order to profit from the change that just merged (see above, pep8...) or can we simply recheck? | 14:36 |
| jbernard | noonedeadpunk: if you're available for friday's review meeting, i added an agenda item (https://etherpad.opendev.org/p/cinder-festival-of-reviews#L25) | 14:37 |
| jbernard | noonedeadpunk: if questions come up, having you there synchronously will be most efficient | 14:37 |
| rosmaita | kaisers: not entirely sure, i think for the gate you may be ok, but for the initial check you may need a rebase | 14:37 |
| kaisers | rosmaita: k, thnx | 14:37 |
| hong-p[m] | And I also requested a patch for python-openstackclient about cinder. Is this the right place to discuss here too? https://review.opendev.org/c/openstack/python-openstackclient/+/973349 | 14:38 |
| rosmaita | hong-p[m]: sure | 14:38 |
| noonedeadpunk | yes, I will be around, but but potentially not following all discussions. | 14:38 |
| noonedeadpunk | but a ping would work for sure to bring my attention :) | 14:39 |
| nimeshdesai | Plugin Register has also received +2 and =1 for workflow. it would be good if this gets another +2 and gets merged - [IBM SVf driver] Adding support for CINDER plugin registration | https://review.opendev.org/c/openstack/cinder/+/951829 | 14:39 |
| jbernard | hong-p[m], rosmaita: clients have already released, or are at least frozen for .1, i believe | 14:39 |
| nimeshdesai | in fact, Plugin Register has 2 +2s. | 14:39 |
| jbernard | nimeshdesai: and +W, im not immediately sure why that didn't go in, maybe need to nudge it, will look today | 14:41 |
| jbernard | noonedeadpunk: sounds good | 14:42 |
| rosmaita | jbernard: maybe just +2A it again, i'm surprised the workflow didn't clear when the patch got stopped on feb 19 | 14:42 |
| hong-p[m] | Got it, thanks for letting me know. I’d appreciate your feedback whenever possible. | 14:43 |
| jbernard | nimeshdesai, rosmaita: i re-voted, im hoping that will get things moving | 14:43 |
| rosmaita | looks like it got picked up: https://zuul.openstack.org/status?change=951829 | 14:44 |
| harsh | awesome ! :) | 14:46 |
| agalica_ | Now to cross your fingers and pray for a successful Zuul lottery, haha | 14:46 |
| harsh | :D | 14:46 |
| jbernard | ok, that's all from me | 14:46 |
| jbernard | #topic open discussion | 14:47 |
| Jyotsna | we have hot customer fix in review https://review.opendev.org/c/openstack/cinder/+/976596 from HPE requires immediate attention. Both CI and zuul passed till 24th Feb. | 14:47 |
| Jyotsna | We are looking for +2 to get it merged, as we are approaching the deadline for bugs, request reviewers to look into this patch. | 14:48 |
| Jyotsna | eharney and jbernard can you please help us with reviews | 14:50 |
| eharney | yes this one is on my list to get back to | 14:51 |
| jbernard | i think eharney has that on his list | 14:51 |
| Jyotsna | we also had new driver for review https://review.opendev.org/c/openstack/cinder/+/969373 which was in review, at last moment due to our backend library was not open source it was not approved | 14:52 |
| Jyotsna | now that we have the library open source and internally approved by HPE can we have a chance of exception for new driver merge ? | 14:53 |
| jbernard | that patch is not ready | 14:55 |
| jbernard | at least, it references a package in driver-requirements but it doesn't exist currently https://pypi.org/project/hpe-storage-flowkit/ | 14:55 |
| Jyotsna | due to the opensource comment we had moved to DNM state, but in case we have chance we can open for review | 14:56 |
| jbernard | it's very unlikely at this stage, there remains far more to do than we have bandwidth for | 14:56 |
| Jyotsna | we just got approval for hpe-storage-flowkit repo to be opensource and yet to create pypi | 14:56 |
| Jyotsna | ok thanks for all reviewers patience | 14:57 |
| Jyotsna | https://review.opendev.org/c/openstack/cinder/+/976596 we will await for any further feedback or +2 | 14:58 |
| jbernard | sounds good, that one should receive some feedback soon | 14:58 |
| Jyotsna | sure thanks | 14:59 |
| jbernard | ok, last call | 15:00 |
| raghavendrat | thanks | 15:00 |
| jbernard | thanks everyone | 15:01 |
| jbernard | #endmeeting | 15:01 |
| opendevmeet | Meeting ended Wed Mar 4 15:01:23 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-03-04-14.02.html | 15:01 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-03-04-14.02.txt | 15:01 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-03-04-14.02.log.html | 15:01 |
| opendevreview | Nimesh Desai proposed openstack/cinder master: [IBM SVf Driver] Define meaningful names for error constants https://review.opendev.org/c/openstack/cinder/+/959683 | 15:05 |
| opendevreview | Brian Rosmaita proposed openstack/cinder stable/2025.1: [stable-only] pep8, unit tests, doc fixes, more https://review.opendev.org/c/openstack/cinder/+/978791 | 17:46 |
| opendevreview | Manideep proposed openstack/cinder stable/2025.2: [WIP] NetApp cinder logs https://review.opendev.org/c/openstack/cinder/+/978919 | 17:56 |
| opendevreview | Brian Rosmaita proposed openstack/os-brick stable/2025.1: [stable-only] constrain setuptools https://review.opendev.org/c/openstack/os-brick/+/978786 | 18:09 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org will be offline momentarily for a software upgrade | 18:11 | |
| opendevreview | Sean McGinnis proposed openstack/cinder master: lvm: Correctly error when no VGs present https://review.opendev.org/c/openstack/cinder/+/978965 | 21:36 |
| *** erlon2 is now known as erlon | 21:53 | |
| opendevreview | Sean McGinnis proposed openstack/cinder master: Fix manage_existing silently ignoring missing LV https://review.opendev.org/c/openstack/cinder/+/978968 | 22:10 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!