15:03:16 <enriquetaso> #startmeeting cinder_bs
15:03:16 <opendevmeet> Meeting started Wed May 31 15:03:16 2023 UTC and is due to finish in 60 minutes.  The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:03:16 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:03:16 <opendevmeet> The meeting name has been set to 'cinder_bs'
15:03:21 <eharney> rosmaita: tosky: https://review.opendev.org/c/openstack/cinder/+/884896
15:03:32 <enriquetaso> Okay, full report of bugs:
15:03:33 <enriquetaso> #link https://lists.openstack.org/pipermail/openstack-discuss/2023-May/033901.html
15:03:43 <rosmaita> eharney: ack
15:04:21 <enriquetaso> #topic Creating a snapshot failed by schema validation when its description
15:04:21 <enriquetaso> contains a new line control character
15:04:32 <eharney> i analyzed this one a bit already
15:04:51 <eharney> basically we have different validation rules for descriptions of volumes and snapshots for no apparent reason
15:04:56 <eharney> one allows endlines and the other doesn't
15:04:58 <enriquetaso> maybe we can talk about the CI gate failure
15:05:06 <enriquetaso> and then continue with the bug meeting..
15:05:33 <eharney> ok
15:05:50 <enriquetaso> #topic Gate failure
15:06:04 <eharney> https://review.opendev.org/c/openstack/cinder/+/884896 should fix it and might actually make it through the gate
15:06:37 <opendevreview> Pavlo Shchelokovskyy proposed openstack/cinder master: Add noop backup API class  https://review.opendev.org/c/openstack/cinder/+/884897
15:07:05 <enriquetaso> so https://review.opendev.org/c/openstack/cinder/+/884896  is going to unblock https://review.opendev.org/c/openstack/cinder/+/884671 ?
15:07:26 <eharney> yes
15:09:49 <enriquetaso> okay
15:10:19 <enriquetaso> we need  https://review.opendev.org/c/openstack/cinder/+/884671  to unblock https://review.opendev.org/c/openstack/cinder/+/884194
15:10:27 <enriquetaso> so, please review ^^
15:10:42 <eharney> we don't need 844194 if we land the new one
15:10:57 <enriquetaso> oh.. okay!
15:12:10 <tosky> unless https://review.opendev.org/c/openstack/cinder/+/884671 happens to merge before
15:13:38 <enriquetaso> :/
15:14:00 * enriquetaso fingercrosses
15:14:31 <enriquetaso> okay, I'm going to continue with the bug meeting
15:14:49 <enriquetaso> #topic Creating a snapshot failed by schema validation when its description contains a new line control character.
15:15:00 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/2021909
15:15:03 <eharney> we have different validation rules for descriptions of volumes and snapshots for no apparent reason
15:15:09 <eharney> one allows endlines and the other doesn't
15:15:35 <eharney> we should probably fix the snapshot check to allow endlines like the volume check does
15:16:05 <enriquetaso> Okay, so it's a valid bug that needs to be fix regardless of the cinder backend
15:16:12 <eharney> yes this is just api validation
15:16:29 <enriquetaso> I'll update my comment on the bug report then
15:16:42 <enriquetaso> thanks eharney
15:17:50 <enriquetaso> @all feel free to work on the bug ^
15:17:58 <enriquetaso> moving on
15:18:08 <enriquetaso> #topic Cached images duplicated per host.
15:18:15 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/2020733
15:18:23 <enriquetaso> Cinder cache using NFS backend creates duplicate cached images per host.
15:19:06 <enriquetaso> No fix proposed to master yet. Looks like a bug on NFS backend that should be fix
15:19:22 <eharney> i'm not sure if it's an nfs driver bug or an image cache bug
15:19:33 <eharney> or if there's some reason this happens that means it isn't a bug
15:19:57 <enriquetaso> "Currently if you bring up an instance with image A on host A, first launch will cause it to download from glance and cache. On subsequent builds if instance is allocated to host A then it will refer to cache."
15:21:21 <enriquetaso> eharney, true, I'm not sure if this is a something that you can config or not
15:21:43 <eharney> abishop worked on locking around the image cache before, he might have some thoughts on this one
15:23:14 <enriquetaso> sounds good, maybe abishop can add some comments on the bug report if possible
15:23:17 <enriquetaso> thanks!
15:23:19 <enriquetaso> moving on
15:23:30 <enriquetaso> #topic HPE 3par: Unable to create clones of replicated vol.
15:23:37 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/2021941
15:23:47 <enriquetaso> Fix proposed to master:
15:23:49 <enriquetaso> #link https://review.opendev.org/c/openstack/cinder/+/884594
15:24:00 <enriquetaso> moving on
15:24:30 <enriquetaso> #topic NetApp ONTAP
15:24:49 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/2021746
15:25:21 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/2021741
15:25:30 <enriquetaso> #link https://bugs.launchpad.net/cinder/+bug/2021739
15:26:03 <enriquetaso> Some patches were merged that dont fully cover the problems they meant to solve
15:26:12 <enriquetaso> so those bug reports were open
15:26:20 <enriquetaso> no fix proposed to master yet for them
15:26:23 <enriquetaso> moving on
15:26:29 <enriquetaso> #topic Open discussion
15:26:37 <enriquetaso> any bug to discuss ?
15:26:42 <enriquetaso> for the las 4 minutes
15:26:43 <hillpd> I wanted to discuss: https://bugs.launchpad.net/cinder/+bug/2017575
15:27:17 <enriquetaso> sure, go hillpd
15:27:32 <hillpd> Passing the cinder.conf "host" value to os-brick makes sense going forward. Unfortunately, that would not be safe to backport due to how the host value gets used in existing deployments.
15:27:59 <hillpd> To address this in prior releases, we are considering adding a "[cinder-oceanstor]" config option, "use_host_fqdn" to address this in the huawei driver. Is that a reasonable direction?
15:29:45 <enriquetaso> are you looking to backport "[cinder-oceanstor]"?
15:30:02 <enriquetaso> Remember from the cinder meeting today that: we will EOL all current EM branches
15:30:02 <enriquetaso> <rosmaita> that means EOL Xena, Wallaby, Victoria, Ussuri, Train
15:30:02 <enriquetaso> <rosmaita> and we will have no EM branches until Yoga goes to EM in 5 months
15:30:02 <hillpd> yeah, we would be looking to backport that solution
15:30:20 <enriquetaso> rosmaita, whoami-rajat, eharney any thoughts on ^?
15:31:50 <eharney> i'd need to dig into it to understand the details better
15:32:26 <enriquetaso> from comment https://bugs.launchpad.net/cinder/+bug/2017575/comments/4
15:33:11 <enriquetaso> i think it would be better if you discuss this with rajat whoami-rajat (it's really late for him now but maybe tomorrow on IRC)
15:33:33 <hillpd> Okay, I can add this discussion to the bug and get their feedback.
15:34:00 <enriquetaso> if that doesnt work, please bring this to the main meeting, where more people attend
15:34:20 <enriquetaso> thank you
15:34:26 <hillpd> ah, okay -- I tried that last week and was told to raise it in the bug squad :)
15:34:33 <enriquetaso> https://etherpad.opendev.org/p/cinder-bobcat-meetings
15:34:38 <enriquetaso> oh
15:34:53 <enriquetaso> sorry about that
15:34:58 <hillpd> no worries
15:35:03 <enriquetaso> okay, I'll ask Rajat tomorrow
15:35:09 <enriquetaso> here and I'll ping you
15:35:15 <hillpd> thanks
15:35:16 <enriquetaso> I'll added to my calendar
15:35:26 <enriquetaso> what timezone are you in?
15:35:45 <hillpd> GMT-6
15:35:50 <enriquetaso> excellent
15:35:54 <enriquetaso> thanks!
15:35:56 <enriquetaso> #endmeeting