14:42:14 <enriquetaso> #startmeeting cinder_bs
14:42:14 <opendevmeet> Meeting started Wed Aug 17 14:42:14 2022 UTC and is due to finish in 60 minutes.  The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:42:14 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:42:14 <opendevmeet> The meeting name has been set to 'cinder_bs'
14:42:19 <enriquetaso> Hello, welcome back to the cinder bug meeting
14:42:30 <enriquetaso> Full list of bugs:
14:42:36 <enriquetaso> #link https://lists.openstack.org/pipermail/openstack-discuss/2022-August/030011.html
14:42:41 <enriquetaso> Only 3 bugs were reported! Let's see the first one:
14:42:48 <enriquetaso> #topic wrong volume_quota exception msg when extending volume_size
14:42:53 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1985962
14:43:19 <enriquetaso> Summary: When extending volume size, the quota exceeds a certain quota class and raises the wrong exception msg to the frontend. It does not consider the quota class now so the exception message below gives the wrong actual quota_size.
14:43:42 <enriquetaso> I need some help to prioritize this bug, since it's a quota bug maybe geguileo could take a quick look at it :) pls
14:44:10 <eharney> if it's just that the message is wrong, we should fix it, but not a huge issue IMO
14:45:15 <enriquetaso> i'll move it to low importance then
14:45:18 <enriquetaso> thanks eharney
14:45:24 * geguileo looks
14:47:50 <geguileo> mmmm, both messages from the LP bug look wrong to me...
14:49:58 <enriquetaso> Fix proposed to master
14:50:00 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/852945
14:52:38 <enriquetaso> OK so we need to fix the error msg geguileo.. besides that the fix ^ removes the error logging so i think it's not the right way to go
14:53:30 <eharney> it doesn't remove it, it calls process_reserve_over_quota which logs the error
14:53:35 <eharney> (whether this is all correct, i don't know)
14:53:38 <geguileo> enriquetaso: it doesn't remove it, it just uses the process_reserve_over_quota method to do it
14:53:47 <enriquetaso> oki :)
14:54:06 <geguileo> (as always eharney was faster, lucky this is not the wild west...)
14:54:14 <enriquetaso> hahaha
14:54:58 <enriquetaso> I moved the bug to low importance, please review the fix when you have time. Moving on..
14:55:15 <enriquetaso> The last two bugs are driver related and both have fix proposed to master:
14:55:32 <enriquetaso> #topic [Storwize] mkhost command failure
14:55:45 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1985065
14:56:55 <enriquetaso> Summary: attach fails Bad or unexpected response from the storage volume backend API: _get_single_iscsi_data: No node found in I/O group 0 for volume volume-4cd0ae48-30b0-41ed-af9f-646fba247e30.
14:57:14 <enriquetaso> Fix proposed to master
14:57:16 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/853126
14:57:32 <enriquetaso> Last one:
14:57:48 <enriquetaso> #topic NetApp driver is hitting the QoS policy limit due soft deletion
14:57:58 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1986658
14:59:22 <enriquetaso> Summary: Customer cannot create a new volume with Quality of Service (QoS) due the limit of QoS policy on the storage.
15:00:05 <enriquetaso> felipe_rodrigues, is working on the fix
15:00:21 <enriquetaso> Fix proposed to master (Zuul vote -1)
15:00:23 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/853450
15:01:53 <enriquetaso> That's all i have for today's meeting!
15:02:08 <enriquetaso> #topic open discussion
15:02:17 <enriquetaso> Feel free to proposed bugs now
15:02:22 <rosmaita> i have a question about https://bugs.launchpad.net/cinder/+bug/1735285
15:02:35 <rosmaita> abishop is fixing it by a new feature
15:02:55 <rosmaita> so how do we want to handle the bug?
15:02:56 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1735285
15:03:15 <rosmaita> i don't see a good Status to change it to
15:03:24 <abishop> oh, I didn't realize there's a bug against the feature (whoops!)
15:03:29 <eharney> i think that bug has another "fix" already done
15:03:45 <rosmaita> or do we want alan to add "Closes-bug: #1735285" to his patch?
15:03:50 <abishop> maybe the "fix" was to add code that rejected the xfer?
15:03:57 <rosmaita> eharney: that was declared to be a "related-bug" i think
15:03:57 <eharney> abishop: correct
15:04:21 <eharney> the bug was "you can do this and end up horribly broken", the fix was "don't allow users to do this", IIRC
15:04:35 <rosmaita> ok, i will just close it as fix released
15:05:31 <rosmaita> thanks!
15:05:55 <abishop> rosmaita: I can dredge up the patch that did that, if it helps
15:06:02 <rosmaita> i have it, don't worry
15:06:06 <rosmaita> from Queens!
15:06:13 <abishop> "what, me worry?"
15:06:39 <abishop> not sure how many people get that reference these days
15:06:44 <rosmaita> i will need to refer to you as albert from now on
15:06:52 <rosmaita> actually that would not work
15:06:53 <abishop> lol
15:06:55 <rosmaita> Alfred!
15:07:15 <enriquetaso> lol
15:07:41 <rosmaita> enriquetaso: are you familiar with Alfred E. Newman?
15:07:43 <enriquetaso> so #1735285 should be marked as released?
15:07:46 <enriquetaso> rosmaita, nop :P
15:08:24 <rosmaita> i didn't even get that right
15:08:31 <rosmaita> Alfred E. Neuman
15:08:51 <abishop> it's safe to google
15:08:54 <enriquetaso> i think i saw that boy somewhere before
15:09:07 <abishop> rosmaita and I are carbon dating ourselves
15:09:25 <rosmaita> yes, and i am proving that my memory is shot
15:10:03 <abishop> enriquetaso: "mad magazine"
15:10:07 <rosmaita> though this is still kind of relevant: https://en.wikipedia.org/wiki/Mad_(magazine)#/media/File:Mad_Magazine,_August_2017.jpg
15:11:04 <rosmaita> speaking of my memory being shot, i have a question about https://bugs.launchpad.net/cinder/+bug/1918932 also
15:11:38 <rosmaita> did we merge a change that prevents a volume transfer if the volume has snapshots, or automatically transfers the snapshots too, or did we just talk about it?
15:11:46 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1918932
15:12:33 <eharney> it's a little more complicated than that
15:12:36 <rosmaita> that bug is about the error message not being complete, but one of the parts of the message is "or be disassociated from snapshots after volume transfer"
15:13:04 <rosmaita> so it made me wonder about the current volume transfer situation
15:13:06 <eharney> we fixed transfer to transfer snapshots, but left an argument that allows people to get the old behavior (i.e. don't transfer the snapshots, and therefore basically break things)
15:13:39 <rosmaita> ok
15:13:51 <eharney> https://review.opendev.org/c/openstack/cinder/+/533564
15:13:59 <eharney> if that helps...
15:14:17 <rosmaita> yes, it does
15:14:18 <rosmaita> thanks
15:14:18 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/533564
15:14:55 <opendevreview> Merged openstack/cinder master: Veritas: Eliminate timed delays in unit tests  https://review.opendev.org/c/openstack/cinder/+/853324
15:15:00 <opendevreview> Merged openstack/cinder master: Inspur: : Eliminate timed delays in unit tests  https://review.opendev.org/c/openstack/cinder/+/853325
15:15:32 <enriquetaso> Do we need https://review.opendev.org/c/openstack/cinder/+/826608 ?
15:17:21 <opendevreview> Nahim Alves de Souza proposed openstack/cinder master: NetApp ONTAP: Implemented core functions on REST API  https://review.opendev.org/c/openstack/cinder/+/852208
15:17:44 <enriquetaso> OK, i think it would be nice to have
15:17:56 <enriquetaso> That's all I have for today's meeting
15:18:07 <enriquetaso> anything else to discuss? :)
15:19:09 <enriquetaso> guess not :)
15:19:17 <enriquetaso> thank you for joining today!
15:20:03 <enriquetaso> #endmeeting