15:01:49 <enriquetaso> #startmeeting cinder_bs
15:01:49 <opendevmeet> Meeting started Wed Jul 20 15:01:49 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:49 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:01:49 <opendevmeet> The meeting name has been set to 'cinder_bs'
15:02:04 <enriquetaso> Welcome back to the Cinder Bug Meeting
15:02:17 <enriquetaso> Cinder has 6 new bugs reported. Mostly low bugs:
15:02:26 <enriquetaso> #topic Infinidat
15:02:38 <enriquetaso> Cinder has two bugs regarding Infinidat and one of them it's kind of a big bug:
15:02:46 <enriquetaso> "Infinidat Cinder driver multi-attach feature is broken"
15:02:51 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1982350
15:03:00 <enriquetaso> I think many drivers have made the same mistake in their multiattach implementations. This is a sign that we need a tempest test that actually tests multiattach
15:03:12 <rosmaita> enriquetaso: ++
15:03:15 <enriquetaso> eharney ^
15:03:16 <eharney> i tossed up https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/850526  but it probably doesn't work yet
15:03:34 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/850526
15:03:44 <enriquetaso> thanks eric!
15:03:45 <enriquetaso> Please keep an eye on this test so we can merge it as soon as possible.
15:04:36 <enriquetaso> maybe infinidat team would like to add some comments to this ? :)
15:05:08 <enriquetaso> The second one is
15:05:13 <enriquetaso> "Infinidat Cinder driver ignores driver_use_ssl option"
15:05:19 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1981982
15:05:26 <enriquetaso> It has a fix proposed to master, feel free to review it!
15:06:50 <rosmaita> ok
15:06:56 <eharney> infinidat also has https://review.opendev.org/c/openstack/cinder/+/849022 up which needs review, maybe someone looking at these reviews should hit all of them together
15:08:33 <enriquetaso> sounds good
15:08:50 <enriquetaso> Next, we have 3 generic-NFS and remotefs related bugs:
15:09:04 <enriquetaso> #topic  NFS Backup driver doesn't remove empty directories on backup deletion.
15:09:06 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1982032
15:09:15 <enriquetaso> whoami-rajat: Using cinder backup with NFS backend, everything works fine, except that when we delete a backup the actual "content" of the backup gets deleted but not the directory structure on the NFS share
15:09:32 <enriquetaso> Fix proposed to master:
15:09:34 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/804603
15:09:49 <eharney> i called out some issues w/ race conditions on the patch, need to go look at rajat's updates and see if i can find any further concerns
15:09:59 <whoami-rajat> yeah, I've revised with some of eharney's suggestions
15:10:17 <whoami-rajat> thanks eharney
15:12:24 <enriquetaso> thanks
15:12:25 <enriquetaso> #topic [NFS] Nova raises an error on the server to resize command
15:12:34 <enriquetaso> https://bugs.launchpad.net/cinder/+bug/1981562
15:12:39 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1981562
15:12:50 <enriquetaso> I have not been able to look into it more, so I am not sure if there is anything to do from the cinder side. I have a devstack+nfs so I'll try this out today.
15:14:19 <eharney> yeah, we need to determine if this is doing something that is not intended to be supported
15:14:23 <eharney> a closer look would be good
15:15:18 <enriquetaso> #action (enriquetaso) take a closer look and reproduce bug1981562
15:15:22 <enriquetaso> The last one is incomplete and waiting from the reporter side but I'd like to mention it
15:15:32 <enriquetaso> #topic RemoteFs: Unexcepted error while running the command.
15:15:39 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1981961
15:16:10 <eharney> we should be able to understand this one just by looking at the code
15:16:22 <eharney> does the nfs driver call mkdir via rootwrap?  if so it needs to be in the volume.filters file
15:18:00 <enriquetaso> OK, so it's not a incomplete bug
15:18:19 <enriquetaso> it's a valid bug
15:18:23 <eharney> i think only os-brick calls mkdir, so this is probably not the right fix, but unclear what is going on
15:18:49 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/850142
15:19:55 <enriquetaso> OK, maybe we can discuss it later in this week?
15:20:08 <eharney> sure
15:20:48 <eharney> it would repro on generic nfs, but may depend on how they have it configured
15:20:59 <eharney> obviously our gate job is setup in a way where mkdir works
15:21:00 <enriquetaso> I'm not completely aware of what the fix want's to achieve, but since I have the nfs devstack maybe I can check this as well
15:21:15 <enriquetaso> makes sense
15:21:33 <enriquetaso> thanks Eric
15:21:44 <eharney> yes, mkdir being run as root needs to be done, but the code that does that is in brick, and there is a privsep layer involved there instead of just rootwrap etc, need to peel back the onion a bit
15:22:28 <enriquetaso> okay
15:22:48 <enriquetaso> eharney++
15:22:49 <enriquetaso> OK, moving one. The last bug of the day is from Storwize
15:22:55 <enriquetaso> #topic [storwize] Driver Initialization error w.r.t default portset
15:23:01 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/1982078
15:23:07 <enriquetaso> Driver initialization failed with the below error:
15:23:16 <enriquetaso> ERROR cinder.volume.manager cinder.exception.InvalidInput: Invalid input received: The storage device does not support iSCSI. Please configure the device to support iSCSI or switch to a driver using a different protocol.
15:23:23 <enriquetaso> There's a fix proposed to master
15:23:31 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/850310
15:24:10 <enriquetaso> #topic open discussion
15:24:20 <enriquetaso> Please feel free to mention bugs now
15:24:53 <eharney> https://bugs.launchpad.net/cinder/+bug/1973228 is one that could use some review
15:25:46 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/842237
15:26:04 <enriquetaso> Please Cinder team review ^
15:26:08 <enriquetaso> i'll do it
15:26:34 <enriquetaso> it's a medium size bug
15:28:21 <dwhite44> There is a tempest bug affecting some CIs, maybe relevant if someone is digging into the multi-attach bug from earlier: https://bugs.launchpad.net/tempest/+bug/1980816
15:30:15 <eharney> i don't know enough about tempest to really understand that bug, but good to know about
15:31:08 <enriquetaso> oh, thanks for sharing that dwhite44 i'll keep an eye on bug1980816
15:31:23 <enriquetaso> thanks everyone!
15:31:42 <enriquetaso> hello?
15:31:46 <enriquetaso> #endmeeting