| opendevreview | Anthony Galica proposed openstack/cinder master: [Hitachi] Support DRS volumes in configuration https://review.opendev.org/c/openstack/cinder/+/987530 | 04:11 |
|---|---|---|
| opendevreview | Anthony Galica proposed openstack/cinder master: [Hitachi] Small documentation updates https://review.opendev.org/c/openstack/cinder/+/997765 | 04:20 |
| opendevreview | Anthony Galica proposed openstack/cinder master: [Hitachi] Small documentation updates https://review.opendev.org/c/openstack/cinder/+/997765 | 04:39 |
| opendevreview | Anthony Galica proposed openstack/cinder master: [Hitachi] Fix VCP-volume zombie issue on B20 storage https://review.opendev.org/c/openstack/cinder/+/995607 | 04:39 |
| opendevreview | Anthony Galica proposed openstack/cinder master: [Hitachi] Add support for compression capacity saving value. https://review.opendev.org/c/openstack/cinder/+/997689 | 04:40 |
| opendevreview | Anthony Galica proposed openstack/cinder master: [Hitachi] Small documentation updates https://review.opendev.org/c/openstack/cinder/+/997765 | 04:40 |
| *** jgilaber__ is now known as jgilaber | 08:16 | |
| opendevreview | Siddharth Kumar proposed openstack/cinder master: Dell PowerFlex: Improve UT coverage https://review.opendev.org/c/openstack/cinder/+/961355 | 09:23 |
| mhen | is there no meeting this week either? | 14:04 |
| jbernard | #startmeeting cinder | 14:04 |
| opendevmeet | Meeting started Wed Jul 22 14:04:42 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 | sorry im running a bit behind, school starts back in 2 weeks... i cannot wait :) | 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 Jyotsna | 14:05 |
| jbernard | #link https://etherpad.opendev.org/p/cinder-hibiscus-meetings | 14:05 |
| jbernard | #topic roll call | 14:06 |
| jbernard | o/ | 14:06 |
| mhen | o/ | 14:06 |
| sfernand | hi | 14:06 |
| hvlcchao1 | o/ | 14:06 |
| Jyotsna | o/ | 14:06 |
| fqi | o/ | 14:06 |
| mhen | some (incl. me) put agenda points in for last week but that meeting didn't happen, so should we move them to today? | 14:06 |
| jayaanand | o/ | 14:07 |
| erlon | \o | 14:07 |
| erlon | back from PTO | 14:08 |
| jbernard | mhen: yes, we could maybe just change the date | 14:09 |
| jbernard | etherpad is not loading for me... | 14:09 |
| jbernard | nm, working now | 14:10 |
| jbernard | #topic perodic job checkin | 14:11 |
| jbernard | #link https://zuul.openstack.org/builds?project=openstack%2Fpython-cinderclient&project=openstack%2Fpython-brick-cinderclient-ext&pipeline=periodic-weekly&limit=18 | 14:11 |
| jbernard | looks good | 14:11 |
| jbernard | i think rosmaita is moving this week, but he'll be happy this is working | 14:11 |
| jbernard | #topic enforce_scope in cinder (RBAC) | 14:12 |
| jbernard | gmaan needed some patches to merge so that a requirements change could unblock | 14:13 |
| jbernard | i *think* we're in good shape now | 14:13 |
| jbernard | gmaan: please correct me if im wrong | 14:14 |
| jbernard | #topic enabling image/volume resize for encrypted images | 14:14 |
| jbernard | mhen: ^ | 14:14 |
| mhen | hi | 14:14 |
| mhen | so in https://review.opendev.org/c/openstack/cinder/+/926298/comment/469d680a_7982c539/ we were asked to also take care of "resize_image()" for our image encryption implementation, which requires volume encryption | 14:15 |
| mhen | so far, we had been testing with ceph and lvm backends on DevStack, which do not use this function | 14:15 |
| mhen | we saw nfs might be affected so we set up DevStack with nfs but encrypted volumes fail due to nfs not supporting encrypted volumes at all, so we cannot test / develop with this backend | 14:15 |
| mhen | the remaining backends using this mechanism all seem proprietary: dell (nfs), vzstorage, quobyte, vmstore (nfs), ibm/gpfs, netapp (nfs) | 14:15 |
| mhen | it seems that "qemu-img resize" is called in most of the affected code paths but encryption keys are never passed to it; so we started wondering if any of those support volume encryption like this at all in the current state | 14:16 |
| jbernard | eharney: ^ you may have some familiarity | 14:16 |
| eharney | the generic nfs driver has an implementation of encrypted volumes which basically works, but i think there are still a couple of issues to sort out with it | 14:18 |
| eharney | i would not be surprised if there are gaps around resize | 14:18 |
| jbernard | for the backends, you should be able to see both the destack config and tests run, i wonder if any of them show what you're looking for in their logs | 14:18 |
| mhen | I ran into issues with nfs and asked here in this channel, this was the response: | 14:18 |
| mhen | "tosky_ mhen: encryption is not supported with NFS. I can't find the bug but it is a known issue" | 14:19 |
| mhen | i.e. "driver.copy_image_to_encrypted_volume()" failing with "error while converting raw: Cannot grow device files" | 14:19 |
| mhen | even creation of encrypted volume fails with nfs in DevStack | 14:19 |
| eharney | hmm, it shouldn't, probably something we need to look into | 14:20 |
| mhen | I simply created an encrypted volume type with nfs backend and then a volume with the basic CirrOS image (qcow2) present on DevStack. Also tried raw image version, same story. | 14:21 |
| jbernard | do any vendors here happen to know if you're testing encryption+resize? | 14:21 |
| eharney | this area has been neglected for a little bit, but filing specific bugs would be useful (especially for the generic driver) | 14:23 |
| mhen | I could file a bug for the DevStack nfs part | 14:24 |
| jayaanand | I am not aware of any limitation from NetApp | 14:24 |
| jayaanand | In-use NFS resize is having limitation from Nova side locking | 14:25 |
| jbernard | this does highlight one of our primary reasons for having vendor CI, the ability to make a core cinder change and have confidence that we aren't negatively impacting any drivers. is it safe to assume that all of teh CIs reporting on the patch look okay? | 14:25 |
| mhen | I haven't looked in a while but I don't think so. This was a discussion way back we had about this patchset. | 14:26 |
| mhen | Because we couldn't judge if it was our fault that the vendor CIs failed and back then we agreed to focus on the vanilla stuff. | 14:27 |
| mhen | (mostly because logs were inaccessible to us, amongst other issues) | 14:27 |
| mhen | anyway, we would need something we can develop against locally on a DevStack to address the review remark | 14:28 |
| mhen | blindly guessing necessary changes and waiting for vendor CIs to report back doesn't seem feasible to me | 14:29 |
| jbernard | i agree | 14:31 |
| mhen | is there anything non-proprietary aside from nfs that would be applicable? | 14:32 |
| * jbernard is looking | 14:32 | |
| mhen | (or is waiting for nfs to be fixed our only option?) | 14:32 |
| jbernard | looks like nfs is the only non-proprietary option | 14:33 |
| mhen | ok | 14:33 |
| jbernard | vendors are responsible for test coverage and providing logs | 14:34 |
| mhen | I'll file the bug report about nfs+encryption asap | 14:34 |
| jbernard | yeah, that's the right thing to do | 14:34 |
| mhen | ack | 14:34 |
| jbernard | eharney: to clarify, we should be able to reach the resize_image() code path through this patch when configured corectly? | 14:36 |
| jbernard | eharney: assuming mhen's bug gets resolved | 14:36 |
| eharney | will have to check later on | 14:37 |
| jbernard | eharney: iow, there's no theoretical limitation, there just may be additional bugs that need to be sorted | 14:37 |
| jbernard | ok | 14:38 |
| jbernard | #topic open discussion | 14:39 |
| fqi | Could you please review these patches for the "retype volume including snapshots" feature? | 14:41 |
| fqi | Cinder-spec:https://review.opendev.org/c/openstack/cinder-specs/+/989322 | 14:41 |
| fqi | We found two issues related to volume retype with migration between an LVM backend and a generic NFS backend https://bugs.launchpad.net/cinder/+bug/2160206 | 14:41 |
| fqi | Here are the patches to fix the issues. | 14:42 |
| fqi | https://review.opendev.org/c/openstack/cinder/+/996728 | 14:42 |
| fqi | https://review.opendev.org/c/openstack/cinder/+/996727 | 14:42 |
| Jyotsna | HPE Alletra MP new driver patch - https://review.opendev.org/c/openstack/cinder/+/969373 Recieved +2 and awaiting another +2 from reviewers and merge | 14:42 |
| Jyotsna | https://review.opendev.org/c/openstack/requirements/+/974322 Received +1 and waiting for +2 and merge | 14:43 |
| Jyotsna | We lost +2 from Brian Rosmaita after rework from suggested from <eharney> Now <eharney> has approved driver patch with +2 so we need Brian's +2 again | 14:45 |
| Jyotsna | As we are apporaching towards final merge we request core reviewers to provide +2 and bless new driver to make it for merge | 14:47 |
| jbernard | ack | 14:47 |
| erlon | Jyotsna: will check that one later | 14:48 |
| Jyotsna | thanks erlon | 14:48 |
| aloke_dev | InfoScale Cinder driver patch: https://review.opendev.org/c/openstack/cinder/+/986072. We have built our inhouse CI and it is up and reported latest run status on that PR and ZUUL build is clean. The PR has got +1. Waiting for feedback from core reviewers for moving this toward +2. | 14:49 |
| jbernard | ok, last call for discussion | 14:57 |
| jbernard | thank you everyone | 14:59 |
| jbernard | #endmeeting | 14:59 |
| opendevmeet | Meeting ended Wed Jul 22 14:59:25 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:59 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-07-22-14.04.html | 14:59 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-07-22-14.04.txt | 14:59 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-07-22-14.04.log.html | 14:59 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Mock sleep in Fujitsu driver unit test https://review.opendev.org/c/openstack/cinder/+/925535 | 15:02 |
| opendevreview | Sailaja Mada proposed openstack/cinder master: Support for Fibre Channel (FC) as the data plane transport for SnapMirror replication https://review.opendev.org/c/openstack/cinder/+/996599 | 15:37 |
| *** erlon3 is now known as erlon | 17:06 | |
| opendevreview | Hao Wang proposed openstack/cinder master: Support flattening volume when cloning from image https://review.opendev.org/c/openstack/cinder/+/998352 | 18:02 |
| gouthamr | jbernard: rosmaita: could you please ack/update https://review.opendev.org/c/openstack/governance/+/996563 please? | 18:58 |
| opendevreview | Merged openstack/cinder master: Replace usage of undocumented logging.WARN https://review.opendev.org/c/openstack/cinder/+/977540 | 19:40 |
| opendevreview | Merged openstack/cinder master: [Everpure]: report provisioned capacity under correct stat key https://review.opendev.org/c/openstack/cinder/+/994165 | 20:24 |
| opendevreview | Merged openstack/cinder master: Tests: Remove duplicate key in qos dict https://review.opendev.org/c/openstack/cinder/+/963568 | 20:25 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: api: add microversion for snapshot-aware retype https://review.opendev.org/c/openstack/cinder/+/994902 | 21:13 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: policy: add rule for snapshot-aware retype https://review.opendev.org/c/openstack/cinder/+/994903 | 21:13 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: api: add max_snapshots_per_migration config option https://review.opendev.org/c/openstack/cinder/+/994904 | 21:13 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: volume: preflight snapshot-aware retype https://review.opendev.org/c/openstack/cinder/+/994905 | 21:13 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: volume: block snapshot ops during retype https://review.opendev.org/c/openstack/cinder/+/994906 | 21:13 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: volume: route snapshot retype to generic migration https://review.opendev.org/c/openstack/cinder/+/994907 | 21:13 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: volume: add migrated snapshot update plumbing https://review.opendev.org/c/openstack/cinder/+/994908 | 21:13 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: volume: rebuild one snapshot into a staging destination snapshot https://review.opendev.org/c/openstack/cinder/+/994909 | 21:13 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: volume: add driver hook for snapshot retype data copy https://review.opendev.org/c/openstack/cinder/+/994910 | 21:13 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: volume: refactor migrate_volume driver path https://review.opendev.org/c/openstack/cinder/+/994911 | 21:13 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: volume: orchestrate generic snapshot retype https://review.opendev.org/c/openstack/cinder/+/994912 | 21:13 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: scheduler: require snapshot migration capable retype backends https://review.opendev.org/c/openstack/cinder/+/994913 | 21:13 |
| opendevreview | Fuli Qi proposed openstack/cinder master: WIP: lvm: support migrated snapshot rename https://review.opendev.org/c/openstack/cinder/+/994914 | 21:13 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!