14:00:02 <whoami-rajat> #startmeeting cinder
14:00:02 <opendevmeet> Meeting started Wed May 11 14:00:02 2022 UTC and is due to finish in 60 minutes.  The chair is whoami-rajat. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:02 <opendevmeet> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:02 <opendevmeet> The meeting name has been set to 'cinder'
14:00:02 <yuval> hey
14:00:08 <whoami-rajat> #topic roll call
14:00:36 <yuval> #help
14:00:59 <fabiooliveira> hi o/
14:01:12 <simondodsley> o/
14:01:45 <whoami-rajat> #link https://etherpad.openstack.org/p/cinder-zed-meetings
14:01:59 <enriquetaso> hi!
14:02:01 <felipe_rodrigues> hi
14:02:01 <jungleboyj> o/
14:02:20 <rosmaita> o/
14:02:28 <caiquemello[m]> hi
14:03:11 <whoami-rajat> now i see enough people so guess we can get started
14:03:29 <whoami-rajat> #topic announcements
14:03:36 <TusjarTgite> Hi
14:03:42 <whoami-rajat> first, Cinder mid-cycle-1 coming up (Add your topics!)
14:03:55 <whoami-rajat> As decided few weeks ago, the date for Midcycle-1 is decided to be 1st June which is Wednesday from 1400-1600 UTC
14:04:26 <whoami-rajat> #link https://etherpad.opendev.org/p/cinder-zed-midcycles
14:04:43 <whoami-rajat> So add your topics to the following etherpad, I've already added the PTG topics that were not covered due to time shortage at the PTG
14:05:15 <whoami-rajat> that includes 3 of your topics rosmaita
14:05:30 <rosmaita> ack
14:05:52 <whoami-rajat> just letting you know in case!
14:05:57 <whoami-rajat> moving on
14:06:01 <whoami-rajat> second, Cinder festival of XS reviews overlapping with Red Hat recharge day
14:06:11 <whoami-rajat> RedHat has a company holiday coming up that overlaps with our festival of XS reviews i.e. on Friday 20th May
14:06:22 <whoami-rajat> Most of our team members (that are from RedHat) won't be able to join festival of XS reviews due to this holiday
14:06:45 <whoami-rajat> the current proposal is, should we shift the festival of XS reviews to the next friday like we did before? i.e. 20th May -> 27th May
14:06:51 <whoami-rajat> if not, we can do a poll for it
14:07:10 <whoami-rajat> again, apologies, i wasn't able to attend the last one, and thanks rosmaita for conducting it!
14:07:19 <rosmaita> :)
14:07:43 <whoami-rajat> so suggestions?
14:08:30 * whoami-rajat hears crickets
14:08:48 <rosmaita> i can do either 20 or 27
14:09:00 <fabiooliveira> I see no problem in moving it
14:09:03 <whoami-rajat> we can decide in next meeting as well but better if we finalized today so people are aware
14:09:54 <caiquemello[m]> no problem moving it too
14:10:10 <whoami-rajat> ok, I will ask again next meeting but looks like the momentum is for moving it
14:10:20 <whoami-rajat> so for now, the new date is 27th
14:10:39 <whoami-rajat> thanks everyone for the feedback
14:10:55 <whoami-rajat> let's move on to topics
14:11:19 <whoami-rajat> #topic Introduce service token support for Swift backup driver
14:11:26 <whoami-rajat> tobias-urdin, that's you
14:11:29 <tobias-urdin> o/
14:11:56 <tobias-urdin> so there is a proposed patch to add support for trusts to the swift backup driver, but that seems more like a workaround
14:12:17 <whoami-rajat> #link https://bugs.launchpad.net/cinder/+bug/1298135/comments/20
14:12:18 <whoami-rajat> #link https://review.opendev.org/c/openstack/cinder/+/840289
14:12:41 <tobias-urdin> so i proposed a patch to support passing a X-Service-Token header with a service token handled by cinder to swift instead
14:13:26 <tobias-urdin> now, this should solve the problem in that the receiving service (swift) can allow the X-Auth-Token even if it's expired
14:13:40 <tobias-urdin> allowing long running processes, same as done with glance/nova integration
14:13:49 <rosmaita> i like this way better than trying to do the trusts stuff
14:14:07 <tobias-urdin> the bad thing is that ceph radosgw's implementation of keystone auth didn't support it yet, so i had to propose support for that there as well
14:14:31 <tobias-urdin> but openstack swift should already support it as far as i can tell
14:15:03 <tobias-urdin> i added a more brief explaination in the launchpad bug, just wanted to bring some attention to it
14:15:37 <tobias-urdin> implementation wise i favoured just keeping the service credentials within cinder and not pass it down to python-swiftclient, just pass the header it should send
14:15:56 <tobias-urdin> let me know what you think about it :)
14:16:26 <tobias-urdin> sed /brief/in-depth/g prob the correct phrasing there
14:17:41 <whoami-rajat> sounds good, interested folks can take a look at the patches and comment provided by tobias-urdin
14:17:56 <whoami-rajat> and by interested i mean the cinder team ^
14:18:24 <whoami-rajat> any other comment/query on the same?
14:19:08 <whoami-rajat> looks like not, so review the patch mentioned above
14:19:19 <whoami-rajat> and moving on to the next topic
14:19:25 <whoami-rajat> #topic Followup to "Move install_command constraints to deps"
14:19:28 <whoami-rajat> rosmaita, that's you
14:19:30 <tobias-urdin> thanks for your time
14:19:44 <whoami-rajat> np, thank you tobias-urdin
14:19:50 <rosmaita> just following up from last meeting
14:19:57 <rosmaita> looks like what we're doing is fine
14:20:16 <rosmaita> you can look at the discussion on the glance patch if you are interested in details
14:20:29 <rosmaita> #link https://review.opendev.org/c/openstack/glance/+/839786
14:20:32 <rosmaita> that's all!
14:20:46 <whoami-rajat> ack, thanks for bringing it up, i will followup with the discussion
14:20:54 <geguileo> rosmaita: thanks for the confirmation :-)
14:20:59 <rosmaita> :D
14:21:26 <whoami-rajat> moving on
14:21:29 <whoami-rajat> #topic Perparing for stable releases of Wallaby, Xena, and Yoga
14:21:42 <whoami-rajat> looks like jbernard is not around
14:21:46 <whoami-rajat> so i will cover up for him
14:21:58 <whoami-rajat> we released victoria final release
14:22:14 <whoami-rajat> but it has some patches that are unreleased in more recent stable branches i.e. yoga, xena, wallaby
14:22:27 <whoami-rajat> so we decided to do a quick followup release to keep the branches in sync
14:22:49 <whoami-rajat> we've some open patches in few branches and Jon and I have confirmed (but other's can look at other branches if we missed something)
14:23:00 <whoami-rajat> all of them are in the cinder repo
14:23:14 <whoami-rajat> so if those get merged soon, we can do a stable release
14:23:37 <whoami-rajat> so stable cores can take a look -- links in etherpad
14:24:43 <whoami-rajat> that's all for this topic
14:25:18 <whoami-rajat> and i don't see anymore topics
14:25:34 <whoami-rajat> so we can move to open discussion
14:25:40 <whoami-rajat> #topic open discussion
14:26:09 <rosmaita> I want to ask for reviews again for a Yoga "Known Issue", namely "Don't destroy existing backup by mistake on import"
14:26:22 <rosmaita> i mean, we really don't want to destroy existing backups by mistake
14:26:31 <rosmaita> #link https://review.opendev.org/c/openstack/cinder/+/839451
14:26:41 <enriquetaso> ack!
14:26:44 <rosmaita> (though apparently i am a minority view on that one)
14:27:09 <whoami-rajat> i will take a look
14:27:11 <enriquetaso> I have a simple tempest test https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/823719 for Deleting Volumes Which are consumed as a basis for others that we might want as well!
14:28:18 <enriquetaso> Also feel free to review https://review.opendev.org/c/openstack/cinder/+/777031 :)
14:28:24 <enriquetaso> already have one +2
14:29:32 <rosmaita> yes, let's review enriquetaso's https://review.opendev.org/c/openstack/cinder/+/777031
14:29:39 <rosmaita> it's been open for over a year
14:29:46 <rosmaita> let's get it out of her life so she can move on
14:30:37 <whoami-rajat> I've added the links to the etherpad so people that are not present and reading the etherpad later can take a look as well
14:30:53 <rosmaita> thanks!
14:31:09 <whoami-rajat> np
14:32:49 <dwhite4> Hello, I recently submitted some driver changes if anyone gets a chance to take a look: https://review.opendev.org/c/openstack/cinder/+/840516
14:34:37 <rosmaita> geguileo should like that one, i think cinderlib is the primary consumer of get_driver_options
14:34:48 <whoami-rajat> dwhite4, ack, added your link as well to the etherpad
14:34:57 <dwhite4> thanks!
14:35:17 <geguileo> rosmaita: iirc there is also some installers using it
14:35:24 <geguileo> rosmaita: hemna ^
14:36:12 <rosmaita> makes sense for installers, i hadn't thought of that
14:36:14 <geguileo> but yes, I like those patches  lol
14:36:19 <rosmaita> all the more reason for the response to be accurate
14:36:41 <simondodsley> can we get reviews for this as well: https://review.opendev.org/c/openstack/cinder/+/832130
14:37:00 <simondodsley> Just needs a +W
14:37:37 <whoami-rajat> If we don't have anything else to discuss, we can close early and give the remaining time to review the patches mentioned above
14:37:48 <whoami-rajat> all are added to L#116 of the etherpad https://etherpad.opendev.org/p/cinder-zed-meetings
14:37:59 <rosmaita> simondodsley: what's the status of the Purity NVMe-RoCE driver?
14:38:15 <simondodsley> We are struggling with Software Factory
14:38:21 <rosmaita> :(
14:38:31 <simondodsley> I know - its a complete nightmare
14:39:00 <simondodsley> Can't even get the iSCSI and FC drivers to work correctly. Currently experiencing Neutron issues
14:39:16 <simondodsley> something weird with devstack
14:39:43 <rosmaita> that sounds ... unpleasant
14:39:48 <simondodsley> We have an NVMe-Fc driver ready as well now, but need the CI
14:39:54 <jungleboyj> :-(
14:40:11 <simondodsley> something to do with `geneve`
14:40:13 * geguileo jumps of joy at the thought of NVMe-FC
14:40:17 <simondodsley> no idea at the moment
14:43:30 <whoami-rajat> ending meeting early as nothing else to discuss and we can continue in #openstack-cinder
14:43:47 <whoami-rajat> thanks everyone, please take a look at the review requests!
14:43:49 <whoami-rajat> #endmeeting