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