15:06:40 <enriquetaso> #startmeeting cinder_bs
15:06:40 <opendevmeet> Meeting started Wed Sep  1 15:06:40 2021 UTC and is due to finish in 60 minutes.  The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:06:40 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:06:40 <opendevmeet> The meeting name has been set to 'cinder_bs'
15:06:48 <enriquetaso> Hello, really short meeting today. Nine new bugs were reported this week.
15:06:56 <enriquetaso> Full list of reported bugs:
15:06:57 <enriquetaso> #link http://lists.openstack.org/pipermail/openstack-discuss/2021-September/024593.html
15:07:15 <enriquetaso> Six of them are driver related and have people looking over them. One is Incomplete and I'm going to talk about the last 2:
15:07:29 <whoami-rajat> eharney, do we plan to backport this fix further i.e. victoria and ussuri? https://review.opendev.org/c/openstack/cinder/+/800268
15:07:44 <enriquetaso> #topic bug_1 "Online database schema migrations to Xena always fail"
15:07:47 <rosmaita> probably to train!
15:07:52 <eharney> whoami-rajat: it would be a good idea
15:07:59 <rosmaita> enriquetaso: sorry to be interrupting
15:08:07 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1942077
15:08:07 <enriquetaso> Summary:  Wallaby deploys fine and upgrade to Xena fails with 2021-08-30 10:44:07.275 16 ERROR cinder.cmd.manage TypeError: cannot unpack non-iterable NoneType object
15:08:23 <enriquetaso> Maybe we should talked about the granade fix here
15:08:37 <enriquetaso> i don't know
15:08:56 <whoami-rajat> eharney, then yes, we need to backport the grenade patch as well
15:09:11 <enriquetaso> I'm not sure how to handle this. Because I'm not sure if we have a tool upstream for upgrading to Xena. However, I'd like to heads up this bugs.  (regarding bug/1942077)
15:09:22 <whoami-rajat> sorry sofia for interrupting
15:09:26 <eharney> enriquetaso: 1942077 looks like a potential bug in our data migrations, which is concerning
15:09:26 <rosmaita> enriquetaso: snapshot_use_quota_online_data_migration was just added very recently
15:09:39 <eharney> this should be investigated
15:09:53 <enriquetaso> I'll move it to high importance then
15:10:18 <enriquetaso> Not sure how to debug tho, can I perform this with devstack?
15:10:42 <eharney> i think the volume.migration_status.startswith() check needs a volume.migration_status is not None check added before it
15:11:05 <eharney> compare volume_use_quota_online_data_migration doing this to _volume_data_get_for_project which both examine for this "target:" string
15:11:44 <eharney> i think the latter handles migration_status being None but the former (this failure) doesn't
15:13:09 <eharney> i added a comment
15:13:28 <rosmaita> ok, thanks Eric
15:13:34 <enriquetaso> thanks for the comment, so we need some is not none check
15:13:38 <enriquetaso> before anything
15:14:06 <enriquetaso> OK.. moving on
15:14:12 <enriquetaso> #topic bug_2 "ssh exec timeout"
15:14:20 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1930267
15:14:26 <enriquetaso> I'm really lost regarding this bug, I'm not sure if I'm missing something or it's a valid bug. I guess the reporter could have added more information but I'd like to heads up on this one as well.
15:14:42 <eharney> there isn't much info on this one... is there actually a patch, since it's "In Progress"?
15:15:23 <enriquetaso> I couldn't found any
15:15:40 <eharney> https://review.opendev.org/c/openstack/cinder/+/793264
15:15:48 <eharney> https://review.opendev.org/q/project:openstack/cinder+and+message:1930267   finds it
15:16:17 <rosmaita> nice search-fu eharney
15:16:40 <rosmaita> enriquetaso: that is a handy search to know
15:16:40 <enriquetaso> oh, i filter by '-bug:<number>'
15:17:06 <enriquetaso> yes, noted. use message:1930267
15:17:25 <rosmaita> looks like eharney already -2d it a few months ago
15:17:27 <enriquetaso> https://review.opendev.org/q/project:openstack/cinder+and+bug:1930267 isn't working
15:17:36 <eharney> enriquetaso: the field is "message"
15:17:54 <eharney> rosmaita: yeah i guess i'll take another look but i think there isn't a lot to go on with this one
15:18:09 <rosmaita> doesn't sound like an emergency
15:18:17 <enriquetaso> it's not
15:18:35 <enriquetaso> we can focus on the reviews before Friday for sure
15:18:57 <enriquetaso> OK, nothing else
15:19:15 <enriquetaso> #topic open discussion
15:20:24 <enriquetaso> If you have any bugs to discuss, please feel free to add there it now
15:20:29 <rosmaita> eharney: could use your opinion on https://review.opendev.org/c/openstack/python-cinderclient/+/806817
15:20:34 <rosmaita> (sorry, not a bug)
15:21:34 <rosmaita> eharney: it would be much easier to simply make 'force' illegal for mv >= 3.66, but that kind of goes against what we did in cinder ... lmk what you think
15:21:36 <eharney> rosmaita: started looking at it, still trying to wrap my head around what's going on there
15:21:46 <enriquetaso> OK, looks like we are done for today
15:21:55 <enriquetaso> thanks!!
15:21:56 <enriquetaso> #endmeeting