15:01:44 <enriquetaso> #startmeeting cinder_bs
15:01:44 <opendevmeet> Meeting started Wed Mar  2 15:01:44 2022 UTC and is due to finish in 60 minutes.  The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:01:44 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:01:44 <opendevmeet> The meeting name has been set to 'cinder_bs'
15:01:53 <enriquetaso> Welcome back to the bug meeting
15:02:01 <enriquetaso> Only two new bugs reported this period and a bug proposed for the open discussion.
15:02:26 <enriquetaso> List of bugs:
15:02:26 <enriquetaso> #link http://lists.openstack.org/pipermail/openstack-discuss/2022-March/027494.html
15:02:57 <enriquetaso> As I wrote we have a bug proposed for open discussion but I think we should discuss it first.
15:03:05 <enriquetaso> #topic Create x Update Volume Metadata in Cinder Backend should be invalid
15:03:13 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1929606
15:03:26 <enriquetaso> (rosmaita)I thought it was wishlist, but looking at the proposed patch, I think we should reject it
15:03:49 <rosmaita> enriquetaso: sorry, i lost track of time
15:03:51 <rosmaita> o/
15:03:56 <enriquetaso> \o
15:04:00 <rosmaita> looking now
15:04:10 <enriquetaso> thanks!
15:04:22 <enriquetaso> don't worry
15:04:25 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/817642
15:04:40 <rosmaita> enriquetaso: ty for bringing that up
15:04:48 <enriquetaso> Brian would you mind sharing your thoughts?
15:04:49 <rosmaita> i thought i had marked it invalid
15:05:19 <enriquetaso> i think it's still in "in progess"
15:05:25 <enriquetaso> I'll move it to invalid then
15:05:34 <enriquetaso> I haven't review it yet
15:05:39 <rosmaita> here's my reasoning
15:05:42 <rosmaita> "As I understand the bug, the problem is that client programs are using the create call to do updates, and hence the permissions appear not to be working correctly.  But if you make API calls directly for create vs. update, the policies are correctly applied."
15:06:33 <enriquetaso> cool
15:06:52 <rosmaita> i guess i didn't update the bug because there could be some discussion
15:07:46 <enriquetaso> the last comment on the upstream patch is from Feb 18
15:08:25 <enriquetaso> Maybe I can set the bp to invalid and ask the reporter to join the upstream meeting if he'd like to discuss it a bit more?
15:08:27 <rosmaita> i will respond and tell the author to bring it to the cinder meeting
15:08:31 <rosmaita> :)
15:08:35 <enriquetaso> nice :D
15:08:50 <enriquetaso> thanks rosmaita
15:09:53 <enriquetaso> Moving forward..
15:10:10 <enriquetaso> #topic [lightos] Compression handling in lightos driver while default compression enabled
15:10:10 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1962221
15:10:10 <enriquetaso> It already has a bugfix upstream, please take a look when you have time. There's a discussion about when to add a release note.
15:10:18 <enriquetaso> yuval ^
15:10:43 <enriquetaso> Moving on..
15:11:07 <enriquetaso> #topic IndexError when creating Backups
15:11:30 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1962607There's an indexError in the chunkeddriver.py that may be valid. However, I've asked for steps to reproduce and the backup driver name. Finally, The fix proposed to master doesn't look right.
15:11:39 <enriquetaso> oops
15:11:44 <eharney> i was looking at this patch this morning
15:11:46 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1962607
15:11:56 <eharney> it would presumably affect any backup driver using ChunkedDriver
15:12:37 <eharney> i'm not too sure about the proposed fix though, i'm wondering if it's papering over something going wrong elsewhere... not sure
15:13:07 <enriquetaso> I think the upstream fix need more work too..
15:13:12 <eharney> i think we need to know why the index can exceed the size of the list which isn't explained
15:13:32 <enriquetaso> OK, so, maybe I should mark it as NEW instead of Invalid
15:14:00 <eharney> i mean, it seems to have failed for them, so it's probably not invalid
15:14:38 <rosmaita> enriquetaso: about your comment on https://review.opendev.org/c/openstack/cinder/+/830538 ... this is a special case because it's a pre-release bug (the lightos driver doesn't exist officially until the Yoga release) ... so this is one case where we shouldn't require a release note
15:15:18 <enriquetaso> rosmaita, ooh! need to update my comment then..
15:15:54 <rosmaita> that's fine, in most cases we do want a release note with a driver bugfix
15:15:56 <enriquetaso> eharney, yes.. when the index exceed the size of the list.. maybe a clone of a clone?
15:16:11 <enriquetaso> need to take a deeper look
15:16:15 <eharney> i don't think this is about clones
15:16:26 <eharney> i'm going to ask them to explain it on the patch, because it might need a deeper look
15:16:33 <enriquetaso> thanks eharney
15:16:40 <enriquetaso> eharney++
15:16:42 <enriquetaso> rosmaita++
15:18:46 <enriquetaso> Moving on..
15:18:58 <enriquetaso> #topic open discussion
15:18:58 <enriquetaso> Feel free to propose bugs now.
15:19:35 <opendevreview> Luciano Lo Giudice proposed openstack/cinder master: Fix QOS computation  https://review.opendev.org/c/openstack/cinder/+/815560
15:21:02 <enriquetaso> OK, I don't have anything else today!
15:21:04 <eharney> do we plan to backport https://review.opendev.org/c/openstack/cinder/+/827860 ?
15:21:12 <eharney> i think we probably should
15:21:20 <eharney> geguileo: ^
15:21:35 <rosmaita> we talked about that last week
15:21:42 <eharney> oh
15:21:45 <rosmaita> geguileo wants it to marinate a bit in master first
15:21:53 <eharney> that works
15:22:02 <geguileo> yeah, in case I broke things horribly
15:22:17 <geguileo> I tried to test it thoroughly though
15:22:25 <eharney> should we add a target to the bug or a WIP patch or something so we don't forget about it?
15:22:42 <geguileo> eharney: great idea!
15:23:03 <geguileo> I'll add a calendar reminder just in case   XD
15:24:16 <enriquetaso> sounds good
15:24:19 <eharney> i added a Wallaby target to https://bugs.launchpad.net/cinder/+bug/1960329
15:24:33 <geguileo> eharney: thanks
15:24:54 <rosmaita> i was trying to figure out which would be the best tracking bug, that one works
15:25:04 <rosmaita> (since the patch fixes like 8 bugs)
15:27:56 <enriquetaso> \O/
15:28:10 <enriquetaso> OK, we have a plan
15:28:27 <enriquetaso> anything else?
15:28:52 <rosmaita> i just closed this old one as invalid: https://bugs.launchpad.net/cinder/+bug/1640173
15:30:25 <enriquetaso> I need to think of something for  old bugs somehow. Maybe once a month I can send a report with very old bugs?
15:31:12 <enriquetaso> run out of time
15:31:18 <enriquetaso> Thanks!!
15:31:24 <enriquetaso> #endmeeting