| opendevreview | Anoop Kumar Shukla proposed openstack/cinder master: Added additional capabilities to ASAr2 driver https://review.opendev.org/c/openstack/cinder/+/977911 | 06:33 |
|---|---|---|
| zigo | Just got the CI saying: | 08:08 |
| zigo | cinder/volume/drivers/vmstore/api.py:122:44: E0601: Using variable 'response' before assignment (used-before-assignment) | 08:08 |
| zigo | which I haven't change. Is anyone working on that one? | 08:08 |
| zigo | BTW, it looks like a false positive. It's in a try: / except:, and probably, the linter fails to see that the variable in the except was set in the try block. | 08:09 |
| opendevreview | Thomas Goirand proposed openstack/cinder master: Fix pylint failure https://review.opendev.org/c/openstack/cinder/+/979960 | 08:13 |
| zigo | This patch should fix it. :P | 08:13 |
| hong-p[m] | hi team, this patch has one +1, look forward for +2 and +w https://review.opendev.org/c/openstack/cinder/+/961906 | 10:34 |
| zigo | hong-p[m]: Wouldn't you write *NOT* rather than ~ ? | 11:06 |
| zigo | I know it's the same at the end, but just, not very readable ... | 11:06 |
| *** tobias-u1 is now known as tobias-urdin | 11:33 | |
| opendevreview | Eric Harney proposed openstack/cinder master: Remove eventlet.sleep calls from scheduler https://review.opendev.org/c/openstack/cinder/+/959388 | 12:26 |
| eduardomorais | Hi all, I have a Cinder issue in my OpenStack cloud. | 12:42 |
| eduardomorais | I added a new controller and removed an old one, but the removed controller was running cinder-volume. I didn’t migrate the volumes before removing it, so now those volumes are still associated with that old cinder-volume service. | 12:42 |
| eduardomorais | Because of this, I can’t migrate or manage those volumes. If I bring the old controller back with the same cinder-volume service, will I be able to migrate the volumes to another backend? | 12:42 |
| hong-p[m] | zigo: thanks for the feedback! I agree that NOT can be more readable. However, I used ~ to stay consistent with the existing patterns in Cinder (e.g., in cinder/db/sqlalchemy/api.py). https://opendev.org/openstack/cinder/src/commit/d5f967dd35aea2ee7040b1ce2666a9c9374028b1/cinder/db/sqlalchemy/api.py#L3358 | 12:54 |
| agalica_ | o/ | 13:06 |
| agalica_ | Oh wait, it’s at 7 again on west coast :o | 13:07 |
| jbernard | :) | 13:07 |
| jbernard | always gets me too | 13:07 |
| agalica_ | Haha | 13:08 |
| zigo | Could someone review this and help me unfuck the CI ? https://review.opendev.org/c/openstack/cinder/+/979960 | 13:13 |
| zigo | It really is a one-liner... | 13:13 |
| zigo | eharney: ^ | 13:14 |
| jbernard | #startmeeting cinder | 14:04 |
| opendevmeet | Meeting started Wed Mar 11 14:04:58 2026 UTC and is due to finish in 60 minutes. The chair is jbernard. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:04 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:04 |
| opendevmeet | The meeting name has been set to 'cinder' | 14:04 |
| jbernard | #topic roll call | 14:05 |
| jbernard | o/ hello all | 14:05 |
| kaisers | o/ | 14:05 |
| harsh | o/ | 14:05 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-gazpacho-meetings | 14:05 |
| hvlcchao1 | o/ | 14:05 |
| simondodsley | o/ | 14:05 |
| jayaanand | o/ | 14:06 |
| hong-p[m] | o/ | 14:07 |
| Anoop_Shukla | o/ | 14:07 |
| raghavendrat | hi | 14:07 |
| agalica_ | O/ | 14:07 |
| Jyotsna | Hi | 14:08 |
| jbernard | #topic annoucements | 14:08 |
| jbernard | thanks for coming | 14:09 |
| jbernard | this week is rc1 target week | 14:09 |
| jbernard | as per https://releases.openstack.org/gazpacho/schedule.html | 14:09 |
| jbernard | our release liason, brian, has been on pto so I'll try to coordinate with him as soon as I get a chance | 14:10 |
| jbernard | lots of reviews, still a lot to do | 14:11 |
| jbernard | quick question, the vmstore author, alexey - does anyone know if they're on irc? and what their nick is? | 14:11 |
| jbernard | i thought the gate was okay today, but zigo may have stumbled on a new issue | 14:12 |
| agalica_ | I looked at that too and it looks wrong for sure. | 14:14 |
| jbernard | full name is Alexey Khodos, or possibly Eugene Ivanov | 14:15 |
| jbernard | no worries, i will send mail and/or we can address the issue directly, just wanted to see if they were around | 14:16 |
| kaisers | small q as to the rc: you requested a reno for one of my changes which i added as a follow up in order preserve the votes on the original change. Technically it absolutely makes sense to merge that into the original change (dropping the votes) but as we're close to rc i wanted to keep the votes and uploaded the follow up. If there are no changes to the original, is it ok to keep this follow up reno change or should i merge it | 14:17 |
| kaisers | either case? Change is https://review.opendev.org/c/openstack/cinder/+/979852/1 | 14:17 |
| erlon | \o late arrival | 14:19 |
| jbernard | we would want the release note to accompany the change for consistency (if we can land it) | 14:20 |
| simondodsley | or add a dependency for the remo PR? | 14:20 |
| kaisers | it is a dependency already but i understand that and will merge | 14:21 |
| kaisers | thnx | 14:21 |
| simondodsley | i meant add a depends-on in the reno patch | 14:21 |
| jbernard | ahh, i see | 14:21 |
| jbernard | i need another review for the copy-image patch, will try to find one today and convey the reno addon | 14:23 |
| nimeshdesai | Last week we merged IBM register plugin patch. So request +2s on the associated doc patch to be a part of the same release - https://review.opendev.org/c/openstack/cinder/+/974989 | 14:24 |
| kaisers | oh, ok | 14:25 |
| kaisers | thnx for that | 14:25 |
| jbernard | i thought we landed that | 14:25 |
| erlon | kaisers: My only concern was that we might end up merging one without the other. Since both were small, you could quickly merge both patches into one. | 14:25 |
| jbernard | ahh, the doc patch, done | 14:26 |
| erlon | but a 1 swoop merge for the 2 patches would work as well jbernard | 14:26 |
| nimeshdesai | There is also this legacy fix that earlier received +2s and +1s, but it had to be updated to resolve merge conflicts so reviews got reset - https://review.opendev.org/c/openstack/cinder/+/959683 | 14:26 |
| kaisers | erlon: absolutely understandable, it's just that votes are such a precious commodity that i took this slightly-off-path to save them. The request is totally understandable. | 14:26 |
| erlon | Let me know what do you think is best and I can remove my -1 | 14:27 |
| erlon | jbernard ^ | 14:27 |
| jbernard | erlon, kaisers: either is fine, ill carry previous votes if another core is in agreement | 14:27 |
| kaisers | Then let me merge this later today to have a clean slate, i'll ping you | 14:28 |
| jbernard | kaisers: ok, sounds good | 14:28 |
| erlon | regarding 922082, I posted a comment/question there | 14:28 |
| erlon | kaisers: ^ | 14:28 |
| kaisers | erlon: ya, saw that, will check it out in the same run | 14:29 |
| opendevreview | Eric Harney proposed openstack/cinder master: DNM: Set threading mode to enabled https://review.opendev.org/c/openstack/cinder/+/970168 | 14:31 |
| jbernard | that's all i have for annoucements, we can discuss whatever is needed with remaining time, or get back to reviews, etc | 14:32 |
| jbernard | #topic open discussion | 14:32 |
| Jyotsna | We have a PR for Improved handling of QOS for Alletra MP https://review.opendev.org/c/openstack/cinder/+/979115 which we are looking for +2 from reviewers. Both Zuul and all CIs have passed. Earlier this patch was part of hot cust patch https://review.opendev.org/c/openstack/cinder/+/976596 and as per the feedback from reviewrs we seperated out the PR. QOS improvement is part of same hot cust issue and requires immediate attention from reviewers | 14:33 |
| jbernard | Jyotsna: yes, that's on my current stack | 14:34 |
| jbernard | will need another core review though, if anyone has cycles ^ | 14:34 |
| Jyotsna | Thanks jbernard, sure we will wait for feedback | 14:34 |
| Jyotsna | Another PR https://review.opendev.org/c/openstack/cinder/+/972038 to Add support for active-active replication policy is a small change. Both Zuul and CIs have passed for it and we are looking for reviewers +2 and merge approval. We have addressed feedback from jbernard to add release notes.Please help us to get this patch merged. This patch can be taken on lower priority over previous patch. | 14:34 |
| jbernard | eharney: ^ we reviewed the earlier patch, you may still have context in your memory | 14:35 |
| Jyotsna | yeah sure, we really appreciate all response from reviewers helping us | 14:36 |
| agalica_ | Hoping to get this merged, but need to fix a pep8 due to a change (will do in a few): https://review.opendev.org/c/openstack/cinder/+/977285?tab=change-view-tab-header-zuul-results-summary | 14:36 |
| opendevreview | Merged openstack/cinder master: [Pure Storage] Add SafeMode support https://review.opendev.org/c/openstack/cinder/+/958569 | 14:36 |
| Anoop_Shukla | https://review.opendev.org/c/openstack/cinder/+/977911 - Added additional capabilities to ASAr2 driver - patch requires some attention. ZUUL CI somehow is failing but has been passing..also NetApp CI has passed. I have retriggered them, hopefully should get pass soon. | 14:41 |
| jbernard | ok, i don't think we're seeing consistent nfs-tempest-full failures, so that should hopefully pass | 14:43 |
| agalica_ | jbernard: Also got a it failure on the 3par driver in my patch (but not my driver) revolving around threading, so wondering if there’s a race condition bug there. Not sure what the protocol for that is since that implies it’s on master! | 14:43 |
| jbernard | agalica_: do you have a link? | 14:44 |
| jbernard | agalica_: Jyotsna may be interested in that as well | 14:44 |
| agalica_ | Jbernard Jyotsna : https://zuul.opendev.org/t/openstack/build/8779386f293344f8a57cda0b0f91c4dc | 14:45 |
| Jyotsna | ok we will check and resolve it | 14:46 |
| agalica_ | Failed py313, but passed py310 and py314 | 14:46 |
| jbernard | https://a08072a22aec7dbb4aca-289b34983b13ece36a1a19c591f4d0ce.ssl.cf1.rackcdn.com/openstack/8779386f293344f8a57cda0b0f91c4dc/testr_results.html | 14:46 |
| Anoop_Shukla | devstack-plugin-nfs-tempest-full https://zuul.opendev.org/t/openstack/build/1a51d5a9e8a84b47a6debae49797dd84 : TIMED_OUT in 3h 09m 06s | 14:48 |
| Anoop_Shukla | I see this very frequently in the recent runs | 14:48 |
| jbernard | Jyotsna: that's from the session leak patch, keep me posted | 14:49 |
| opendevreview | Merged openstack/cinder master: Dell PowerMax: fix deadlock during attach rollback https://review.opendev.org/c/openstack/cinder/+/948539 | 14:49 |
| 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 | 14:50 |
| agalica_ | Anoop: too many patches changing I think. Always happens around deadlines | 14:51 |
| jbernard | ok, last call for open discussion | 14:54 |
| zigo | Well, it's been 2 months my patch https://review.opendev.org/c/openstack/cinder/+/971710 has recieved zero patch or opinion. | 14:54 |
| jbernard | yes, | 14:55 |
| zigo | I had to rebase it twice, and it's currently affected by the CI fix I wrote earlier today. | 14:55 |
| jbernard | cores, ^ that needs review | 14:55 |
| zigo | I also need opinions about https://review.opendev.org/c/openstack/cinder/+/979641 | 14:56 |
| zigo | Normally, the db query should be filtering disabled nodes, but reality says otherwise, which is why I wrote that one (it annoyed us a lot in production). | 14:56 |
| jbernard | that we can cover in the ptg, unless you receive feedback earlier | 14:57 |
| zigo | Sure. Just wanted to highlight these 2 patches, in case someone has bandwidth to at leave give a bit of feedback. | 14:57 |
| zigo | When is PTG ? Next week or something? | 14:58 |
| jbernard | April, 3rd week i believe | 14:58 |
| zigo | Ok, thanks. | 14:58 |
| jbernard | yep, 20-24 | 14:58 |
| jbernard | alright, thank you everyone | 15:00 |
| jbernard | #endmeeting | 15:00 |
| opendevmeet | Meeting ended Wed Mar 11 15:00:39 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:00 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-03-11-14.04.html | 15:00 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-03-11-14.04.txt | 15:00 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-03-11-14.04.log.html | 15:00 |
| kaisers | thnx | 15:00 |
| opendevreview | Merged openstack/cinder master: [IBM SVf driver] Doc: Documentation updates for ibm-svf-callhome feature https://review.opendev.org/c/openstack/cinder/+/974989 | 15:08 |
| opendevreview | Fernando Ferraz proposed openstack/devstack-plugin-nfs master: [DNM] Test Glance over Cinder/NFS with NFS driver fixes https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/965409 | 15:46 |
| opendevreview | Silvan Kaiser proposed openstack/cinder master: Fix copy_image_to_volume for Quobyte driver https://review.opendev.org/c/openstack/cinder/+/922082 | 16:22 |
| opendevreview | Silvan Kaiser proposed openstack/cinder master: Report provisioning in driver stats for Quobyte driver https://review.opendev.org/c/openstack/cinder/+/920875 | 16:22 |
| opendevreview | Silvan Kaiser proposed openstack/cinder master: Quobyte: adapt to image_utils fixes https://review.opendev.org/c/openstack/cinder/+/953966 | 16:22 |
| opendevreview | Silvan Kaiser proposed openstack/cinder master: Quobyte: Fix metadata for volume from snapshot #2142477 https://review.opendev.org/c/openstack/cinder/+/978564 | 16:22 |
| opendevreview | Damian Dąbrowski proposed openstack/cinder master: scheduler: track active backup backends from service liveness https://review.opendev.org/c/openstack/cinder/+/980122 | 16:22 |
| opendevreview | Anthony Galica proposed openstack/cinder master: [Hitachi] Debug Enhancement: Introduce instrumentation for auditing http/s requests to assist with debugging (Driver, UT, or Tempest). https://review.opendev.org/c/openstack/cinder/+/979870 | 16:32 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Rename Pure driver docs to Everpure https://review.opendev.org/c/openstack/cinder/+/979365 | 19:16 |
| opendevreview | Merged openstack/os-brick stable/2025.1: [stable-only] constrain setuptools https://review.opendev.org/c/openstack/os-brick/+/978786 | 19:22 |
| opendevreview | Fernando Ferraz proposed openstack/devstack-plugin-nfs master: [DNM] Test Glance over Cinder/NFS with NFS driver fixes https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/965409 | 19:33 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!