14:00:14 <rosmaita> #startmeeting cinder
14:00:15 <openstack> Meeting started Wed Feb 17 14:00:14 2021 UTC and is due to finish in 60 minutes.  The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot.
14:00:16 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
14:00:18 <openstack> The meeting name has been set to 'cinder'
14:00:23 <rosmaita> #topic roll call
14:00:24 <whoami-rajat__> Hi
14:00:33 <jungleboyj> o/
14:00:36 <eharney> hey
14:01:03 <zoharm> hi!
14:01:16 <almir-okato> hi
14:01:30 <geguileo> hi! o/
14:01:54 <e0ne> hi
14:02:02 <rosmaita> looks like we've got most of the usual suspects here!
14:02:10 <rosmaita> #link https://etherpad.openstack.org/p/cinder-wallaby-meetings
14:02:21 <rosmaita> #topic announcements
14:02:39 <rosmaita> #topic announcements - festival of XS reviews
14:02:51 <rosmaita> #link http://lists.openstack.org/pipermail/openstack-discuss/2021-February/020433.html
14:03:06 <rosmaita> as we discussed at the mid-cycle last week, we're going to do a review smash on friday
14:03:19 <jungleboyj> Oh.  Thanks for the reminder.
14:03:20 <rosmaita> from 1400-1600 utc
14:03:38 <whoami-rajat__> sorry i won't be able to make it, got some work on friday so on PTO
14:03:54 <rosmaita> it won't be recorded, but you can join in for quick discussions
14:04:00 <rosmaita> whoami-rajat__: sorry you will miss it
14:04:20 <rosmaita> basic idea is we will knock out a bunch of extra-small reviews
14:04:31 <rosmaita> and possibly larger ones if things move smoothly
14:04:48 <rosmaita> everyone is welcome
14:05:04 <rosmaita> #topic announcements - upcoming deadlines
14:05:19 <rosmaita> wallaby official os-brick release in 2 weeks (thursday 4 March)
14:05:26 <rosmaita> wallaby feature freeze in 3 weeks
14:05:41 <rosmaita> feature freeze is also the time for the wallaby python-cinderclient release
14:06:00 <rosmaita> #topic announcements -assorted stuff
14:06:09 <rosmaita> gerrit launchpad integration broken
14:06:18 <rosmaita> for both bugs and blueprints, so please update manually until this gets fixed
14:06:26 <rosmaita> #link http://lists.openstack.org/pipermail/openstack-discuss/2021-February/020185.html
14:06:39 <rosmaita> also, there is a new bindep release
14:06:47 <rosmaita> for anyone interested in exploring that stuff
14:06:56 <rosmaita> #link http://lists.opendev.org/pipermail/service-announce/2021-February/000015.html
14:07:16 <rosmaita> finally, please check the "courtesy reminder" at the top of the agenda etherpad
14:07:31 <rosmaita> (if you want to be notified in #openstack-cinder before the meeting starts)
14:07:51 <rosmaita> one last thing
14:08:19 <rosmaita> we will release cinder "milestone-2" with all the new wallaby drivers
14:08:28 <rosmaita> in the hope that people will test it out
14:08:35 <jungleboyj> \o/
14:08:40 <rosmaita> anyway, that should happen later this week
14:08:54 <rosmaita> #link https://review.opendev.org/c/openstack/releases/+/776114
14:09:15 <rosmaita> ok, that's it from me ... anyone have something to announce?
14:10:06 <rosmaita> ok, guess not
14:10:16 <rosmaita> #topic cinder driver features declarations
14:10:26 <rosmaita> #link https://releases.openstack.org/wallaby/schedule.html#w-cinder-driver-features-declaration
14:10:35 <rosmaita> you can read in the above what this is about
14:10:51 <rosmaita> but briefly, features have to be merged by the time of the feature freeze in 3 weeks
14:10:58 <rosmaita> that includes drivers
14:11:18 <rosmaita> so by filing blueprints for new features, we can prioritize reviews
14:11:34 <rosmaita> bugfixes can be merged up to RC time
14:12:02 <rosmaita> i only saw 3 new blueprints for driver features:
14:12:11 <rosmaita> #link https://blueprints.launchpad.net/cinder/+spec/xtremio-ports-filtering
14:12:20 <rosmaita> #link https://blueprints.launchpad.net/cinder/+spec/netapp-ontap-min-throughput-qos-capability
14:12:26 <rosmaita> #link https://blueprints.launchpad.net/cinder/+spec/powerstore-cg-support
14:12:53 <rosmaita> but, i find launchpad blueprint searching/listing to be ... problematic
14:13:04 <sfernand> ok, I have a feel for NetApp as well
14:13:09 <rosmaita> so please check to make sure your BP is listed
14:13:13 <sfernand> few
14:13:21 <rosmaita> #link https://blueprints.launchpad.net/cinder/wallaby
14:13:31 <rosmaita> sfernand: ok
14:13:52 <rosmaita> getting these approved is not a problem, they are useful for prioritizing reviews
14:14:28 <rosmaita> anyway, if you have new driver features, please file a BP before the festival of reviews on Friday
14:14:35 <rosmaita> and i will look them over then
14:15:09 <rosmaita> don't forget that the launchpad integration with gerrit is broken
14:15:25 <rosmaita> so as soon as you have a patch up, please put it on the blueprint page so people can find it
14:15:36 <rosmaita> any questions?
14:15:44 <sfernand> ok, we are working on a bp approved for queens haha
14:15:46 <sfernand> https://blueprints.launchpad.net/cinder/+spec/netapp-flexgroup-support
14:16:02 <rosmaita> i guess we can re-target it!
14:16:26 <sfernand> that would be nice, all other bps are correctly target to wallaby
14:16:28 <sfernand> just checked
14:16:43 <rosmaita> great
14:16:56 <rosmaita> #topic wallaby features
14:17:27 <rosmaita> if you are working on a feature that was approved by spec, please make sure the launchpad BP is in this list:
14:17:39 <rosmaita> #link https://blueprints.launchpad.net/cinder/wallaby
14:18:05 <rosmaita> and if you have a bp that's missing, let me know (you can put a shout in #openstack-cinder)
14:18:28 <rosmaita> #topic os-brick
14:18:52 <rosmaita> Zohar put a request in the cinder channel earlier today
14:19:02 <rosmaita> NVMEoF monitoring & healing agent could use some feedback
14:19:10 <rosmaita> #link https://review.opendev.org/c/openstack/os-brick/+/768576
14:19:29 <rosmaita> there is a comment on PS9 where he outlines some issues he needs feedback on
14:19:52 <zoharm> thank you Brian, here is a link to the comment: https://review.opendev.org/c/openstack/os-brick/+/768576/9#message-c78aca6621c0fdb751efb56071787966da24751d
14:20:45 <rosmaita> thanks
14:21:03 <rosmaita> looking at the open patches, it looks like mostly small stuff for os-brick that's outstanding
14:21:11 <rosmaita> btw, this could use some eyes:
14:21:15 <rosmaita> #link https://review.opendev.org/c/openstack/os-brick/+/755478
14:21:42 <rosmaita> we can hit a lot of the other stuff on friday:
14:21:51 <rosmaita> #link https://review.opendev.org/q/project:openstack/os-brick+status:open+branch:master
14:23:00 <tosky> and also on !master :)
14:23:18 <rosmaita> i think rajat is going to talk about those a little later
14:23:32 <rosmaita> but !master can be released at any time
14:23:52 <rosmaita> we have a hard deadline for wallaby
14:24:01 <tosky> right
14:24:07 <whoami-rajat__> spoiler: it's in pretty good shape now
14:24:18 <rosmaita> \o/
14:24:23 <jungleboyj> :-)
14:24:33 <rosmaita> #topic cinder permissions matrix
14:25:19 <rosmaita> i've been working on documenting our policies and what they do, because it's not always evident from the sample policy.yaml file
14:25:46 <rosmaita> also, what changes could happen to support the "personas" that are implemented by default in keystone
14:26:01 <rosmaita> #link https://review.opendev.org/c/openstack/cinder/+/763306
14:26:17 <rosmaita> please look and leave comments
14:26:35 <rosmaita> it's broken up into sections, so you can review the parts of the API most of interest to you
14:26:55 <rosmaita> also, it's pretty big, and though i was very careful, there are probably errors in there
14:27:17 <whoami-rajat__> I've left a comment that might be related to reader != auditor role but correct me if my understanding is wrong on that
14:27:23 <rosmaita> you may want to look at the HTML version as easier to read:
14:27:33 <rosmaita> #link https://a91ddba9ffca8a8b5d72-1d82fafbdd07c6a56856d7b1449f73a8.ssl.cf5.rackcdn.com/763306/9/check/openstack-tox-docs/fd06b54/docs/configuration/block-storage/policy-personas.html
14:27:35 <rosmaita> whoami-rajat__: ty
14:28:38 <rosmaita> #topic cinder stable releases update
14:28:41 <rosmaita> whoami-rajat__: that's you
14:28:45 <whoami-rajat__> Hi
14:29:16 <whoami-rajat__> so in the past week, there were lots of reviews and most of the patches merged in all branches
14:29:24 <whoami-rajat__> thanks to everyone who contributed in it
14:30:01 <whoami-rajat__> the main patches i see remaining are from netapp which fernando updated today
14:30:11 <whoami-rajat__> and SVF which requires review
14:30:41 <whoami-rajat__> but the number of patches remaining is very few and we still have 1 week so i think we can release on time
14:30:48 <rosmaita> nice
14:31:08 <whoami-rajat__> again let me know if you've any patch that i missed in my tracker
14:31:14 <whoami-rajat__> #link https://etherpad.opendev.org/p/stable-releases-review-tracker-22-02-2021
14:31:25 <whoami-rajat__> that's all from me
14:31:44 <rosmaita> thank you
14:31:52 <rosmaita> well, we are moving right along today
14:31:59 <rosmaita> #topic open discussion
14:34:05 <rosmaita> well, if nobody has anything, i have a weird problem on cinderlib train branch
14:34:15 <rosmaita> #link https://review.opendev.org/c/openstack/cinderlib/+/771091
14:34:21 <jungleboyj> :-)
14:34:52 <rosmaita> the cinderlib-ceph-functional job fails right away, looks like an ansible error
14:35:06 <rosmaita> though this used to work, so i'm not sure what has changed
14:35:28 <rosmaita> https://zuul.opendev.org/t/openstack/build/9c0d03c4685842a7ba37b5db722267b4
14:38:35 <whoami-rajat__> is this the reason for failure? Failed to import the required Python library (setuptools) on centos-7-rax-ord-0022614760's Python /usr/bin/python
14:40:46 <whoami-rajat__> rosmaita:  whenever you get time, can you lift the -2 on this since it's rebased? or if you want to wait for zuul+1 https://review.opendev.org/c/openstack/cinder/+/764273
14:41:21 <rosmaita> i'll lift now
14:42:01 <rosmaita> i think that's the reason, but i would have thought ansible would know it needs setuptools
14:42:12 <whoami-rajat__> thanks! (it's the only patch remaining in cinder victoria)
14:42:28 <eharney> i rebased the patch to update to hacking 4.0.0: https://review.opendev.org/c/openstack/cinder/+/770145
14:42:42 <eharney> there is a check before that failing part that ensures that setuptools is installed, looks like something more subtle is going on..
14:43:30 <rosmaita> since this is train, i wonder whether its a weird py2/py3 thing
14:43:48 <eharney> i suspect it is
14:47:29 <whoami-rajat__> would like to ask for reviews on 2 of my patches which have been sitting there for a long time: 1) https://review.opendev.org/c/openstack/cinder/+/754397 2) https://review.opendev.org/c/openstack/cinder/+/763181
14:48:23 <eharney> yeah, we really need to get 754397 finished up
14:48:51 <rosmaita> ok, well if nobody else has anything to discuss, we can quit early and go review rajat's patches
14:48:57 <TusharTgite> hi i unable to find exact place where to put code for override the reset state request in this spec https://review.opendev.org/c/openstack/cinder/+/773985
14:49:17 <TusharTgite> can anyone tell me that
14:52:38 <eharney> is reset state still just in the "contrib" area of the api?
14:54:13 <rosmaita> looks like cinder/api/contrib/admin_actions.py
14:54:30 <eharney> yeah, trying to find a good example of other contrib code we've tweaked with microversions...
14:55:06 <TusharTgite> yes it also is in admin_action.py
14:58:42 <eharney> probably not the most helpful answer, but understanding how we added support for transferring volumes with snapshots in microversion 3.55 may be a start to see how this could be done
14:58:52 <eharney> it's a similar case of changing what we accept/reject based on microversion for contrib api code
15:00:04 <TusharTgite> ok thanks i'll take a look at this .
15:00:31 <rosmaita> ok, just noticed that we are infringing on horizon
15:00:33 <rosmaita> thanks everyone
15:00:36 <rosmaita> #endmeeting