16:00:20 <jungleboyj> #startmeeting Cinder
16:00:21 <openstack> Meeting started Wed Sep  4 16:00:20 2019 UTC and is due to finish in 60 minutes.  The chair is jungleboyj. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:22 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:24 <openstack> The meeting name has been set to 'cinder'
16:00:29 <jungleboyj> courtesy ping:  jungleboyj whoami-rajat rajinir lseki carloss pots woojay erlon geguileo eharney rosmaita enriquetaso e0ne smcginnis davidsha walshh_ xyang hemna _hemna tosky sfernand
16:00:30 <smcginnis> o/
16:00:38 <geguileo> hi! o/
16:00:46 <walshh_> hi
16:00:47 <carloss> hi :)
16:00:57 <e0ne> hi
16:00:57 <jungleboyj> @!
16:00:58 <_pewp_> jungleboyj (^▽^)/ ʸᵉᔆᵎ
16:01:03 <hemna> doink
16:01:07 <woojay> hi
16:01:11 <rosmaita> o/
16:01:11 <hemna> @!
16:01:11 <_pewp_> hemna ( ･ω･)ﾉ
16:01:24 <Roamer`> hi
16:01:25 <hemna> oh how we miss _pewp_ in #openstack-cinder
16:01:34 <lseki> hi
16:01:45 <xyang> Hi
16:01:54 * jungleboyj wonders why we can't pewp in Cinder
16:01:57 <hemna> xyang signting!
16:02:04 * jungleboyj looks at smcginnis
16:02:08 <xyang> :)
16:02:18 <jungleboyj> hemna:  Who is that?  ;-)
16:02:22 <hemna> hehe
16:03:20 <davidsha> o/
16:03:25 <jungleboyj> Wow, we have a pretty good turnout today.
16:03:40 <whoami-rajat> Hi
16:03:57 <jungleboyj> #topic announcements
16:03:58 <sfernand> o/
16:04:18 <jungleboyj> First, congratulations to rosmaita  on the Cinder PTL election!
16:04:30 <hemna> w00t
16:04:36 <whoami-rajat> Congratulations Brian
16:04:42 <rosmaita> thanks
16:04:50 <e0ne> my congratulations, rosmaita!
16:05:01 <e0ne> jungleboyj: thanks for all your hard work as PTL
16:05:07 <xyang> Congrats Brian
16:05:14 <hemna> finally passing the torch.
16:05:19 <jungleboyj> https://gph.is/1aR7NWq
16:05:34 <xyang> Thanks jungleboyj
16:05:36 <rosmaita> congrats to jungleboyj on election to the TC!
16:05:40 <jungleboyj> Yes, thank you team for the opportunity to lead for the last 4 releases.
16:06:03 <jungleboyj> Been a great experience.  I am excited, though, to have a new perspective for Cinder.
16:06:17 <jungleboyj> So thank you rosmaita
16:06:32 <jungleboyj> rosmaita:  Thank you.  Not much of an election ... but hey ...
16:06:35 <rosmaita> well, i am glad that you are committed to continuing to work on cinder :)
16:06:55 * jungleboyj runs for the door.  ;-)
16:07:16 <geguileo> rofl
16:07:58 <jungleboyj> No, I am not going anywhere and I am hoping on the TC I might be able to bring attention to some of the things we discussed at the Mid-cycle with regards to vendor fixes, etc.
16:08:22 <rosmaita> jungleboyj: ++
16:09:06 <hemna> cinder is unique in the openstack landscape because of it's dependence on vendor drivers.
16:09:15 <jungleboyj> So, it looks like the next announcements are yours rosmaita
16:09:24 <jungleboyj> hemna:  True.
16:09:39 <rosmaita> i put up a patch for the os-brick Train release
16:09:56 <jungleboyj> rosmaita:  Awesome.  Thank you.
16:10:03 <rosmaita> #link https://review.opendev.org/679889
16:10:16 <rosmaita> i think we got in all the important stuff
16:10:36 <jungleboyj> JP has a question in your patch.
16:10:40 <rosmaita> next up is python-cinderclient release next week
16:10:56 <hemna> looks good, doesn't look like there is anything in the pipe for os-brick
16:11:04 <rosmaita> jungleboyj: i think he answered it himself
16:11:20 <jungleboyj> Oh, yes he did.  I see that now.  :-)
16:11:36 <rosmaita> so cincerclient will be a major version bump, primary thing is to get everything reviewed
16:11:55 <rosmaita> #link https://review.opendev.org/#/q/status:open+project:openstack/python-cinderclient+branch:master
16:12:06 <rosmaita> a bunch of things are in the gate now, so there's not a lot left
16:12:26 <rosmaita> also, next week is the cinder feature freeze
16:12:32 <jungleboyj> That is good.  Will look through these again today.
16:12:52 <hemna> there is one patch against master
16:13:00 <hemna> https://review.opendev.org/#/c/678523/
16:13:57 <rosmaita> that looks worth getting into the release
16:14:00 <jungleboyj> What do you mean one against master?  Oh, one that isn't the major_bump tag?
16:14:10 <hemna> that one needs some attention
16:14:15 <hemna> before we cut the client I Think
16:14:20 <jungleboyj> Yeah and I was ok with that, just needed UT.
16:14:34 <whoami-rajat> i asked for reviews quite a lot last week :)
16:14:43 <jungleboyj> whoami-rajat:  Did.
16:15:14 <rosmaita> that's all from me
16:15:36 <jungleboyj> Ok.  Thank you.
16:15:48 <hemna> ew global
16:16:03 <jungleboyj> hemna:  ?
16:16:11 <hemna> https://review.opendev.org/#/c/678523/4/cinderclient/v3/shell.py
16:16:15 <smcginnis> Yeah, not liking the global either.
16:16:15 <hemna> uses global
16:16:25 <whoami-rajat> i know :(
16:17:12 <jungleboyj> hmmm, global filters.  Wonder if that could help climate change?
16:17:52 <jungleboyj> Ok, so looks like hemna  and whoami-rajat  Will need to talk that through.
16:18:14 <_pewp_> hemna ╭∩╮◕ل͜◕)╭∩╮  ︵┻┻
16:18:14 <_pewp_> hemna ╭∩╮◕ل͜◕)╭∩╮  ︵┻┻
16:18:15 <_pewp_> hemna ╭∩╮◕ل͜◕)╭∩╮  ︵┻┻
16:18:16 <_pewp_> hemna ╭∩╮◕ل͜◕)╭∩╮  ︵┻┻
16:18:17 <_pewp_> hemna ╭∩╮◕ل͜◕)╭∩╮  ︵┻┻
16:18:18 <hemna> omg
16:18:18 <_pewp_> ...
16:18:19 <_pewp_> ...
16:18:19 <_pewp_> ...
16:18:20 <hemna> die
16:18:31 <jungleboyj> What happened there?
16:18:32 <smcginnis> Hah!
16:18:37 <Roamer`> it's alive!
16:18:48 <jungleboyj> Thanks.  I needed a laugh.
16:18:52 <whoami-rajat> it's like an argparse thing that passes multiple values but one value at a time to the action method, couldn't think of anything other than global var to append all the values at that time.
16:18:58 <senrique_> o/
16:19:03 <jungleboyj> #topic Reenable the StorPool driver after the CI was brought back up on 2019-08-03:
16:19:31 <davidsha> Is that meant to be the 3rd of September or 3rd of August?
16:19:36 <Roamer`> so, yeah, I guess the topic says it all: after a lapse of a couple of months (really sorry about that!) we brought the StorPool CI back up, Python 3.7, Zuul v3 and everything
16:19:48 <Roamer`> ...oops.... it was meant to be September, sorry.......
16:19:55 <rosmaita> \o/
16:19:57 <hemna> Roamer`: awesome!
16:20:05 <smcginnis> Thanks for getting that done.
16:20:32 <Roamer`> it now runs a very slightly customized version of tempest-and-cinderlib-run
16:20:35 <jungleboyj> Roamer`:  Indeed.  Thank you for addressing that.
16:21:13 <jungleboyj> I will take a look at your patch and vote.
16:21:14 <Roamer`> it has a small patch for the cinderlib test since our thin provisioning driver does not report the space change after a volume is created, I'll polish that up a bit and submit it for review
16:22:56 <Roamer`> BTW an unrelated question: are people aware of the fact that devstack currently has an interesting issue when installing Python 3 development libraries after they have already been installed by another project?  like something uses Python 3 and brings in os-brick, then devstack comes to os-brick itself and says "I'll install it for Python 2 just in case something else needs it", everything that
16:23:02 <Roamer`> it depends on is installed for Python 2, *including scripts*, then devstack says "I'll install it for Pyth... oh, sorry, it's already installed for Python 3" and any dependencies that installed scripts in /usr/local/bin or wherever now have these scripts say #!python2
16:23:25 <Roamer`> so when something tries to run, say, unit2, it tries to start with Python 2 and cannot find the Python 3 modules
16:23:49 <Roamer`> I'll think about this some more and possibly start a discussion in -qa or something, but I guess this is slightly off-topic for today's meeting here :)
16:23:53 <hemna> hrmm
16:24:02 <jungleboyj> Hmm, that sounds like a devstack bug.
16:24:12 <hemna> Roamer`: do you have logs for that?
16:24:26 <hemna> I'm curious why os-brick itself wants python2
16:24:45 <jungleboyj> Roamer`:  You don't need to run all the tempest tests.  Can do just the volume ones, but if you want to run them all ...
16:24:51 <jungleboyj> I think things look good.
16:25:12 <Roamer`> hemna, it's not os-brick itself, it's devstack: when the shell function that installs a Python *development* library (not an end project) is called, it (the devstack function) says "I'll install the library for Python 2, too, just in case"
16:25:34 <hemna> oh maybe os-brick needs an update to tox.ini
16:27:04 <tosky> Roamer`: uh, have seen the new run-cinderlib-tests role now exposed by cinderlib.git?
16:27:54 <Roamer`> jungleboyj, we're not running them all, just the volume tests... I took the regex from somewhere, can't remember where right now, but it does limit them
16:28:34 <Roamer`> tosky, yeah, I've seen it, I need some more work to get it to work with our driver (I can't remember right now, there was something that needed a bit more configuration, but I may have fixed it in the meantime in another way)
16:29:09 <jungleboyj> Roamer`:  This looks like all of them to me:  https://spfactory.storpool.com/logs/85/679785/1/check/cinder-storpool-tempest/7c79011/testr_results.html.gz
16:29:33 <jungleboyj> I commented on the patch.  Either need to add some things there or just revert my original patch.
16:29:51 <jungleboyj> It hasn't released so that is safe to do.
16:30:35 <jungleboyj> Anyway, any more discussion there?
16:31:08 <rosmaita> no, happy to see the ci running again
16:31:18 <jungleboyj> rosmaita: ++
16:31:21 <Roamer`> jungleboyj, hm, that's interesting, doesn't it get limited by https://spfactory.storpool.com/logs/85/679785/1/check/cinder-storpool-tempest/7c79011/job-output.txt.gz#_2019-09-03_16_08_56_469522
16:31:32 <Roamer`> but, sure, no more discussion needed, thanks y'all!
16:32:57 <jungleboyj> Depends on how you are executing tempest.  Anyway ... not a big deal.
16:33:07 <jungleboyj> Something for you to look into.
16:33:50 <jungleboyj> So, that was all we had on the agenda.
16:33:53 <Roamer`> yeah, sure, I wiil, thank you
16:33:58 <jungleboyj> #topic Open Discussion
16:34:25 <jungleboyj> Anyone have other topics to discuss.
16:34:37 <zhengMa> Hi team, we are requesting more comments for 'Leverage hw accelerator in image compression'
16:34:44 <zhengMa> #link https://review.opendev.org/#/c/668825
16:34:51 <zhengMa> Thanks!!
16:35:28 <jungleboyj> Ok.  See that you addressed my comments.  Will need to go look.
16:35:32 <jungleboyj> Thanks!
16:36:03 <zhengMa> jungleboyj: thank you
16:36:08 <jungleboyj> Welcome.
16:36:12 <whoami-rajat> Since the cinderclient and cinder feature freeze are on the same day wanted to request some reviews here too
16:36:12 <whoami-rajat> https://review.opendev.org/#/c/639180/
16:36:31 <jungleboyj> I think we need to get eharney  and rosmaita  to look at zhengMa 's patch as well.
16:36:32 <rosmaita> smcginnis just reminded me that the python-brick-cinderclient-ext train release is next week also -- looks like nothing to review, but just in case anyone wants to add something, you need to do it right away!
16:37:25 <jungleboyj> That has been quiet for a while.
16:37:49 <rosmaita> yes, looks all ready to go, has 13 changes since last release, but no open reviews
16:38:43 <rosmaita> jungleboyj: you might want to +2 priority whoami-rajat's https://review.opendev.org/#/c/639180/
16:39:35 <rosmaita> (and i did see your note about zhengMa's patch, i will review it)
16:39:54 <zhengMa> rosmaita: thank you
16:40:10 <smcginnis> jungleboyj: Should probably check to make sure the client has everything needed for service updates. At least matching the mv bumps.
16:40:14 <whoami-rajat> Thanks rosmaita  :)
16:40:25 <jungleboyj> rosmaita:  Done.
16:40:25 <smcginnis> And we may want to make sure Eric's autonegotiate patch has merged, if it hasn't yet.
16:40:36 <jungleboyj> smcginnis:  That does need to get merged.
16:41:47 <jungleboyj> Hmm, yeah, we should make sure that we don't get into a situation where we mess up the MVs for the release.
16:43:04 <jungleboyj> Anything else for today?
16:44:23 <jungleboyj> Ok.  I will take silence as a no.
16:44:31 <rosmaita> does anyone have any plans for python-brick-cinderclient-ext ?
16:44:40 <rosmaita> if not, i will put up a release patch today
16:44:42 <jungleboyj> I do not.
16:44:47 <rosmaita> no sense waiting for no reason
16:45:13 <jungleboyj> rosmaita: I am guessing that that is safe to do.
16:45:19 <rosmaita> well, i guess the deadline is technically next week, so maybe i will put it up with -W
16:45:49 <rosmaita> and release on monday
16:46:00 <jungleboyj> Makes sense.
16:46:19 <rosmaita> smcginnis: is that ok, or does it annoy the release team?
16:47:36 <jungleboyj> @!j
16:47:36 <_pewp_> jungleboyj (._.) ~ ︵ ┻━┻
16:48:09 <jungleboyj> rosmaita:  I think it should be fine.
16:48:12 <smcginnis> rosmaita: That should be fine.
16:48:16 <rosmaita> cool
16:48:18 <jungleboyj> Jix
16:48:20 <jungleboyj> Jinx
16:48:53 <jungleboyj> Ok.  I am going to call it a meeting.
16:49:05 <jungleboyj> Thank you all for attending.
16:49:11 <jungleboyj> #endmeeting