15:03:56 <enriquetaso> #startmeeting cinder_bs
15:03:56 <opendevmeet> Meeting started Wed Jun 15 15:03:56 2022 UTC and is due to finish in 60 minutes.  The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:03:56 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:03:56 <opendevmeet> The meeting name has been set to 'cinder_bs'
15:04:04 <enriquetaso> Welcome back to the Cinder Bug Meeting.
15:04:05 <rosmaita> HappyStacker1: can't tell who you are from your nick, have you been reviewing patches yourself?
15:04:16 <enriquetaso> good question :P
15:04:22 <rosmaita> (i mean other peoples' patches)
15:04:25 <enriquetaso> We have a few bugs today, so let us get started.
15:04:32 <enriquetaso> Full list of bugs:
15:04:32 <enriquetaso> #link http://lists.openstack.org/pipermail/openstack-discuss/2022-June/029056.html
15:04:41 <HappyStacker1> thats' jp from dell rosmalta
15:05:16 <rosmaita> ok, thanks
15:05:25 <enriquetaso> :)
15:05:28 <enriquetaso> #topic [NFS/NetApp] No feedback from Nova when extending attached Volumes
15:05:33 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1978294
15:05:53 <enriquetaso> In summary, n-cpu is able to extend an attached NFS volume by itself, but c-vol is preventing this from happening when it tries to perform `qemu-img resize`.
15:06:06 <enriquetaso> This seems to be a problem not just for the NFS driver, since drivers using os-brick may fail, and Nova also handles extending the LUKS structure of attached encrypted volumes, which may also fail.
15:06:23 <enriquetaso> This bug affects Quobyte, NetApp Cinder NFS drivers as far as I know
15:06:27 <enriquetaso> No fix proposed to master yet.
15:07:04 <rosmaita> "Lucio Seki, who first worked on [2] and [3], collected some thoughts on this in an etherpad [4] together with other potential issues of the changes."
15:07:13 <rosmaita> i thought this sounded familiar
15:08:30 <enriquetaso> yes, Lucio work on this as well
15:08:35 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/739079
15:09:01 <enriquetaso> Etherpad link
15:09:04 <enriquetaso> #link https://etherpad.opendev.org/p/fix-nfs-online-extend
15:09:31 <enriquetaso> The original bug is #link https://bugs.launchpad.net/cinder/+bug/1870367
15:10:39 <enriquetaso> As far as I understood the new bug is for a related issue but we are going to track it on 1978294
15:11:13 <enriquetaso> Maybe kgube would like to add something to the bug report :)
15:12:19 <enriquetaso> Currently, the solution we have for this is to disallow extension of attached volumes for NFS & Quobyte drivers
15:12:23 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/725805/
15:13:07 <enriquetaso> OK, moving on
15:13:22 <rosmaita> yeah, that was a short term "fix"
15:13:44 <enriquetaso> do you think the second bug should be close as duplicated? rosmaita
15:14:09 <rosmaita> not sure
15:14:37 <enriquetaso> OK, i'll re review it and double check but looks like a separate bug
15:15:06 <rosmaita> i agree
15:15:15 <enriquetaso> #topic Fails to extend in-use (non LUKS v1) encrypted volumes
15:15:19 <enriquetaso> #link https://bugs.launchpad.net/os-brick/+bug/1967157
15:15:34 <enriquetaso> Summary: Patch fixing bug #1861071 resolved the issue of extending LUKS v1 volumes when nova connects them via libvirt instead of through os-brick, but nova side still fails to extend in-use volumes when they don't go through libvirt (i.e., LUKS v2).
15:15:54 <enriquetaso> geguileo, worked on the nova part of the bug that appears as Fix Released.
15:16:16 <enriquetaso> now we have a cinder patch for this
15:16:18 <enriquetaso> Fix proposed to master
15:16:18 <enriquetaso> #link https://review.opendev.org/c/openstack/os-brick/+/836059
15:18:16 <enriquetaso> OK, feel free to review ^
15:18:23 <enriquetaso> moving on
15:18:26 <enriquetaso> #topic Old-style Glance location URI sent when uploading a volume to an image
15:18:37 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1978020
15:19:09 <enriquetaso> whoami-rajat, discussed it with the reporter
15:19:25 <enriquetaso> Fix proposed to master
15:19:26 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/845178
15:20:59 <enriquetaso> So, feel free to review it as well ^
15:21:00 <rosmaita> that looks ok to me, i thought glance did a lazy-migrate of old-style URIs, but maybe that was before you could have multiple cinder glance_stores
15:22:19 <enriquetaso> good to know
15:22:27 <rosmaita> the only thing i want to check is what kind of validation we have around the store_id in a volume-type extra-specs
15:22:42 <enriquetaso> mm, not sure about that
15:22:52 <enriquetaso> maybe we need some coverage there
15:23:10 <rosmaita> i think we have something, just can't remember what
15:23:24 <rosmaita> i will look into it and leave a comment on the patch
15:23:33 <enriquetaso> thank you rosmaita !
15:23:43 <enriquetaso> OK, last two bugs:
15:23:49 <enriquetaso> #topic cinder-backup context.message_action is None on errors
15:23:57 <enriquetaso> Not sure about this one
15:24:01 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1978729
15:24:08 <enriquetaso> If a driver raises an exception another exception is thrown from https://opendev.org/openstack/cinder/src/commit/3dfc519301aeb92ed4595788450b57e0be74db12/cinder/message/api.py#L104 because AFAICT the action is None. (also checked above https://opendev.org/openstack/cinder/src/commit/3dfc519301aeb92ed4595788450b57e0be74db12/cinder/message/api.py#L101 for setting action_id.
15:25:30 <enriquetaso> looks like valid and the reporter added some logs to the case
15:26:21 <enriquetaso> I'll try to reproduce and try the possible solutions
15:26:39 <enriquetaso> #action (enriquetaso): try the fixes for ^
15:26:51 <enriquetaso> OK, last one! I'm running out of time
15:27:13 <enriquetaso> #topic [IBM Storwize] Optimize SSH calls in create replicated volume
15:27:18 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1978290
15:27:37 <enriquetaso> No fix proposed to master yet but probably IBM will proposed a fix :)
15:27:46 <enriquetaso> OK, that's all I have for today's meeting
15:28:15 <enriquetaso> thanks rosmaita
15:28:19 <enriquetaso> thanks everyone
15:28:24 <enriquetaso> #endmeeting