15:01:20 <enriquetaso> #startmeeting cinder_bs
15:01:20 <opendevmeet> Meeting started Wed Sep  8 15:01:20 2021 UTC and is due to finish in 60 minutes.  The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:01:20 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:01:20 <opendevmeet> The meeting name has been set to 'cinder_bs'
15:01:26 <enriquetaso> Welcome back to the cinder bug meeting
15:01:31 <enriquetaso> Quick meeting today
15:01:39 <enriquetaso> #topic bug_1 "Upload volume to image will terminate the vm's volume attachment"
15:01:45 <eharney> comes from something like this where a variable is blank, i guess  https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/fetch-subunit-output/tasks/main.yaml#L68
15:01:52 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1942953
15:02:29 <rosmaita> eharney: that job history looks ok, hopefully you just got unlucky
15:03:29 <rosmaita> well, not hopefully, but you know what i mean
15:03:51 <eharney> i would be surprised if 1942953 was that straightforward and the whole bug is really just "oh we shouldn't call detach from copy_volume_to_image sometimes" (which is what the proposed fix does)
15:04:12 <enriquetaso> yes
15:04:21 <enriquetaso> Fix proposed to branch: master
15:04:21 <enriquetaso> #topic  https://review.opendev.org/c/openstack/cinder/+/807819
15:04:34 <enriquetaso> Maybe anyone is currently working with this and  knows if this is possible?
15:05:19 <eharney> if what is possible?
15:05:40 <enriquetaso> I didn't know that cinder allows uploading-to-image to an attached volume.
15:06:11 <enriquetaso> maybe I'm confused about this
15:07:06 <eharney> we need actual reproduction steps, i think
15:07:40 <enriquetaso> #action(enriquetaso) ask for steps and more details
15:07:42 <enriquetaso> sure
15:08:24 <enriquetaso> #topic bug_2 "Undefined CONF.quota_metadata_items is used in unit tests"
15:08:31 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1934036
15:08:43 <enriquetaso> Summary: Currently there are two places where CONF.quota_metadata_items is used in unit tests. However this parameter is not defined anywhere in cinder. It is very likely that the implementation was ported from nova and we missed removing that undefined parameter.
15:09:10 <enriquetaso> Fix proposed
15:09:11 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/798783
15:09:17 <enriquetaso> If anyone is working on tests and would like to review ^ please feel free to do so.
15:09:57 <eharney> seems straightforward enough
15:10:22 <enriquetaso> cool
15:10:35 <enriquetaso> OK, that's all I have for today's meeting
15:10:39 <enriquetaso> #topic open discussion
15:12:35 <rosmaita> i never filed a bug about quota_volumes not working in the python-cinderclient functional tests, just mentioning that so y'all ask me about it occasionally
15:13:19 <rosmaita> not sure if i'm not using devstack correctly or if it's a real problem
15:14:31 <enriquetaso> good question, I haven't either
15:15:20 <eharney> https://review.opendev.org/c/openstack/cinder/+/806010  could probably use a look before we get too far into the RC phase  (since it sort of but not really changes a config option)
15:15:20 <rosmaita> that's interesting, though, that we don't have a limit on max metadata allowed
15:15:47 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/806010
15:17:17 <enriquetaso> OK
15:18:12 <enriquetaso> ending meeting now if we don't have any more bugs for open discussion
15:18:24 <rosmaita> sounds good, thanks sofia
15:18:33 <enriquetaso> #endmeeting