16:00:17 <smcginnis> #startmeeting Cinder
16:00:19 <openstack> Meeting started Wed Aug 23 16:00:17 2017 UTC and is due to finish in 60 minutes.  The chair is smcginnis. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:20 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:23 <openstack> The meeting name has been set to 'cinder'
16:00:25 <smcginnis> ping dulek duncant eharney geguileo winston-d e0ne jungleboyj jgriffith thingee smcginnis hemna xyang1 tbarron scottda erlon jbernard _alastor_ bluex karthikp_ patrickeast dongwenjuan JaniceLee cFouts Thelo vivekd adrianofr mtanino karlamrhein diablo_rojo jay.xu jgregor lhx_ rajinir wilson-l reduxio wanghao thrawn01 chris_morrell watanabe.isao,tommylikehu mdovgal ildikov wxy viks ketonne
16:00:31 <smcginnis> abishop sivn
16:00:32 <e0ne> hi
16:00:34 <xyang1> Hi
16:00:35 <_alastor_> o/
16:00:36 <scottda> hey
16:00:40 <hemna> hi
16:00:47 <diablo_rojo_phon> Hello :)
16:00:49 <wxy-> hi
16:01:02 <abishop> o/
16:01:06 <patrickeast> Hi
16:01:13 <evrardjp> o/
16:02:01 <prometheanfire> o/
16:02:26 <Swanson> Hi.
16:02:31 <smcginnis> OK, looks like probably enough to get going.
16:02:36 <smcginnis> #topic Announcements
16:02:43 <smcginnis> The end is near!
16:02:54 <smcginnis> Still
16:03:19 <smcginnis> We will need to cut a Cinder RC2 today, or early tomorrow at the latest.
16:03:26 * DuncanT passes Jay a sandwich board
16:03:40 <tbarron> hi
16:03:42 <hemna> any bugs we need to land ?
16:03:57 <smcginnis> hemna: Nothing I see out there right now.
16:03:58 <smcginnis> https://review.openstack.org/#/q/project:openstack/cinder+branch:stable/pike+status:open
16:04:09 <smcginnis> So if you have something, speak now or forever hold your peace.
16:04:26 <smcginnis> Or wait until the first stable release. :)
16:04:40 <prometheanfire> that's at the ptg iirc
16:04:57 <smcginnis> Speaking of which...
16:05:15 <smcginnis> Normal reminders, register for the PTG:
16:05:17 <smcginnis> #link https://etherpad.openstack.org/p/cinder-ptg-queens Planning etherpad for PTG
16:05:22 <smcginnis> Oops
16:05:37 <smcginnis> ACtually, first add topics and your name to the PTG planning etherpad. ^^
16:05:45 <smcginnis> #link https://www.eventbrite.com/e/project-teams-gathering-denver-2017-tickets-33219389087 PTG registration
16:05:50 <smcginnis> Then register ^
16:05:51 <lhx__> hi
16:06:30 <smcginnis> The topic etherpad is starting to get filled up, which is great.
16:06:30 <diablo_rojo_phon> Going to do team photos at the PTG again so sign up :)
16:06:41 <smcginnis> diablo_rojo_phon: Is there a sign up for that yet?
16:06:48 <evrardjp> diablo_rojo_phon: good to know.
16:07:06 <diablo_rojo_phon> smginnis yes, I will send it out again but I put it in the 4 wk out update
16:07:31 <diablo_rojo_phon> I have to send it to the new PTLS but I was hoping the old would communicate to the new as well ;)
16:07:42 <smcginnis> diablo_rojo_phon: OK, great.
16:07:48 <jungleboyj> @!
16:07:48 <_pewp_> jungleboyj (。Ő▽Ő。)ノ゙
16:07:53 <smcginnis> I'll let jungleboyj worry about then. :)
16:07:58 <jungleboyj> What?
16:08:04 <prometheanfire> diablo_rojo_phon: I haven't heard :P
16:08:25 <diablo_rojo_phon> Also, those that need ptgbot power need to poke me in the ptg channel so I can hook them up.
16:08:52 <evrardjp> :)
16:08:53 <diablo_rojo_phon> prometheanfire: evrardjp I'll send something out to the new PTLS today or tomorrow.
16:09:02 <prometheanfire> k
16:09:17 <smcginnis> jungleboyj: No "I have the power!" from giphy? :)
16:09:43 <jungleboyj> smcginnis: Too many things going on to play.  ;-)
16:09:50 <evrardjp> haha
16:09:53 <smcginnis> Off your game today.
16:10:05 <jungleboyj> smcginnis:  :-p
16:10:44 <smcginnis> On that note, next week is final GA for Pike. jungleboyj and I will figure out who's running the meeting next week, but I suppose that's a good time for hand off.
16:11:10 * jgriffith hands jungleboyj a beer
16:11:19 <jgriffith> hand off complete... oh wait
16:11:24 <smcginnis> ;)
16:11:30 <smcginnis> #topic Video interviews at the PTG
16:11:40 * smcginnis starts slacking off
16:11:45 * jungleboyj chugs the beer.
16:12:06 * jungleboyj gets the feeling that smcginnis  is all too happy to put me in charge.  ;-)
16:12:12 <smcginnis> #link http://lists.openstack.org/pipermail/openstack-dev/2017-August/121369.html
16:12:24 <jungleboyj> So, I think this is where I take over.  How is that for timing?
16:12:31 <smcginnis> ;)
16:12:55 <jungleboyj> So, there was an e-mail to the mailing list from Rich Bowen, RDO Community Liaison asking for volunteers to do video interviews.
16:13:12 <jungleboyj> Looking for updates on what happened in Pike and to highlight any cross project work that happened.
16:13:30 <jungleboyj> I think Sean and I could do a project update.
16:14:10 <jungleboyj> From a cross project standpoint wondering if jgriffith and ildikov  would be willing to talk about the work with Cinder/Nova APIs ?
16:14:32 <ildikov> jungleboyj: like we would have any other choice :)
16:14:41 <e0ne> :)
16:14:59 <jungleboyj> ildikov:  Well, you do.  :-)
16:15:09 <ildikov> jungleboyj: I think Nova folks could use an update too
16:15:20 <jungleboyj> Just letting you know the opportunity is there.
16:15:32 <ildikov> jungleboyj: I mean besides the 1.5 people who's reviewing
16:15:44 <smcginnis> mriedem loves giving updates.
16:15:49 <jungleboyj> We had also put on the agenda to do an overview of what has happened during Pike during the PTG.
16:15:51 <ildikov> jungleboyj: sure thing, thank you
16:15:59 <ildikov> smcginnis: :)
16:16:15 <jungleboyj> jgriffith:  Has suddenly gotten quiet.
16:16:37 <smcginnis> I think they just wanted 1-3 people for those updates. I nominate jungleboyj and ildikov.
16:16:38 <jungleboyj> :-)
16:16:50 <jungleboyj> smcginnis:  :-p
16:17:02 <smcginnis> jgriffith and I will sit back and have a beer and make faces behind the camera.
16:17:11 <jgriffith> +1
16:17:20 * jungleboyj laughs
16:17:25 <jungleboyj> What have I gotten myself into.
16:17:25 <ildikov> smcginnis: jgriffith has to be there, I can't be blamed for the whole thing :)
16:17:40 <jungleboyj> I wanted to make faces at ildikov and jgriffith
16:17:46 <jgriffith> jungleboyj ohhh..... you have no idea!! moua-moua-hahah
16:17:51 <smcginnis> ildikov: Well, if he's not there then it makes it easy to blame all the problems on him. :)
16:18:03 <ildikov> jungleboyj: be the boss and do that :)
16:18:09 <jgriffith> smcginnis nice try, I perfected that trick remember :)
16:18:09 <evrardjp> smcginnis: I can only imagine how release "collaboration" interview session will go then :p
16:18:21 <ildikov> smcginnis: you know how people associate things to faces...
16:18:22 <jungleboyj> *smh*
16:18:41 <jungleboyj> Ok, I guess I am going to be the boss and say that Sean and I will do a general update.
16:19:20 <jungleboyj> ildikov: Lets talk offline and figure out the right people to do an update on Nova/Cinder and what all should be done at the PTG with regards to that and make that happen.
16:19:40 <jungleboyj> I also need to coordinate with mriedem on the when we are going to do a joint Nova/Cinder session.
16:19:55 <smcginnis> jungleboyj: Reminds me, should also coordinate with mriedem on a joint cinder/nova timeslot.
16:20:00 <ildikov> jungleboyj: sure, I will repeat multi-attach every 10 seconds and someone else will do the update and we're done, awesomeness :)
16:20:21 <jungleboyj> Anyone else have anything related to Cinder they think we would want to share?
16:20:25 <jungleboyj> smcginnis:  Jinx.
16:20:30 <prometheanfire> https://review.openstack.org/#/c/496284/
16:20:30 <jungleboyj> You lagging up there?
16:20:44 <smcginnis> Yeah, I think so.
16:20:50 <jungleboyj> prometheanfire:  Hold on.
16:21:02 <smcginnis> jungleboyj: Next topic?
16:21:14 <jungleboyj> Yeah, I think we are done with that one.
16:21:23 <jungleboyj> At least have a plan.
16:21:25 <smcginnis> #topic Oslo cleaning up deprecated stuff
16:21:32 <jungleboyj> And it is me again.
16:21:52 <jungleboyj> So, Oslo is going to be working during queens to clean up things that they have deprecated.
16:22:03 <jungleboyj> #link http://lists.openstack.org/pipermail/openstack-dev/2017-August/121345.html
16:22:18 <jungleboyj> I haven't had a chance to look and see what impact that has on Cinder.
16:22:40 <jungleboyj> Sharing here for awareness.  If there is anyone bored and looking to get commits I would appreciate help looking at that.
16:22:55 <jungleboyj> If not, I will try to find some time to work on that during the release.
16:23:03 <smcginnis> jungleboyj: I've started looking at a few deprecation warning. I can try to spend a little time on it.
16:23:16 <jgriffith> that's a pretty long list
16:23:26 <jungleboyj> jgriffith:  Yeah, that part makes me a little nervous.
16:23:31 <e0ne> smcginnis: I can help you with it
16:23:40 <smcginnis> e0ne: ++
16:23:40 * e0ne likes to remove code
16:23:43 <jungleboyj> e0ne: smcginnis  Awesome, thank you.
16:23:49 <jgriffith> grep to the rescue
16:24:13 <smcginnis> e0ne: Me too. :)
16:24:32 <smcginnis> jungleboyj: Move on?
16:24:32 <e0ne> :)
16:24:41 <lhx__> smcginnis, plus me ;)
16:24:52 <jungleboyj> Yep, I think so.  Appreciate help we can get there.
16:25:04 <smcginnis> Let's see how many lines of code we can remove in Queens. :)
16:25:15 <hemna> replication.....
16:25:15 <jungleboyj> smcginnis:  Removal only Fridays?
16:25:17 <hemna> w00t!
16:25:19 <jgriffith> all of them!!!
16:25:27 <smcginnis> jgriffith: Hah! :)
16:25:45 <smcginnis> #topic cinder-manage output
16:25:55 <smcginnis> prometheanfire: Not on the agenda, but I know you wanted to discuss this.
16:25:59 <hemna> replication should die in a fire.
16:26:01 <smcginnis> #link https://review.openstack.org/#/c/496284/
16:26:02 <evrardjp> a whole topic!
16:26:11 <evrardjp> we are lucky out there
16:26:16 <smcginnis> :)
16:26:44 <evrardjp> looks like the review are not overwhelmingly negative, so I guess if we fix we are good to go...
16:26:50 <jungleboyj> hemna:  Comes in from left field.
16:26:55 <prometheanfire> :D
16:27:25 <prometheanfire> ya, I was more curious how far back that can be backported, as a user facing change I'm not expecting much
16:27:25 <evrardjp> so I guess it's a thank you ... and a question
16:27:40 <evrardjp> prometheanfire: what a SYNC!
16:27:45 <smcginnis> prometheanfire, evrardjp: So the tl;dr is the output is currently truncated making it hard to match up output in scripting>
16:27:46 <prometheanfire> evrardjp: :P
16:28:02 <prometheanfire> impossible in some cases
16:28:03 <evrardjp> yup.
16:28:40 <jungleboyj> prometheanfire:  I don't know that this is really a bug so I don't see it having backport potential.
16:28:45 <smcginnis> Hmm, that does change the result returned. I wonder if that would be considered an API change?
16:28:52 <eharney> i think it is a bug
16:29:03 * smcginnis would rather not microversion it, but needs to bring it up at least
16:29:03 <prometheanfire> smcginnis: that's kinda what I was wondering
16:29:21 <jungleboyj> Oy.
16:29:23 <eharney> microversion?  this is cinder-manage
16:29:32 <jungleboyj> eharney:  Yeah.
16:29:37 <eharney> or is it not?
16:29:38 <smcginnis> Oh, it's a cmd. Yeah, nevermind.
16:29:51 <scottda> Could someone possible be using the output as it is now, in a script? That's the only important question...
16:30:06 <eharney> that's not a specific enough question
16:30:19 <prometheanfire> I doubt it
16:30:27 <scottda> Could we break someone by changing this?
16:30:28 <prometheanfire> I don't think the output is useful as is
16:30:32 <smcginnis> More specific - is fixing this for this case going to screw anyone else? :)
16:30:33 <scottda> I agree.
16:30:36 <eharney> it has the same output after the fix, it's just not truncating the hostname, right?
16:30:45 <jgriffith> eharney yes
16:30:47 <prometheanfire> someone could be logging status via it though
16:30:56 <prometheanfire> so that'd change values
16:31:01 <jgriffith> I'm not sure about backporting output changes to cinder-manage though
16:31:10 <jungleboyj> smcginnis:  THat is my concern.
16:31:13 <jgriffith> besides, they could just use "cinder service-list"
16:31:25 <jgriffith> that call in cinder-manage should probably be deprecated IMO
16:31:28 <prometheanfire> ya, but cinder-manage is more direct, should be faster
16:31:38 <jgriffith> prometheanfire ?
16:31:48 <prometheanfire> cinder-manage is also useful if running offline right?
16:31:55 <evrardjp> jgriffith: inconsistency between cinder service-list and cinder-manage is a concern
16:32:00 <prometheanfire> that's where we ran into it
16:32:13 <jgriffith> prometheanfire can you define "running offline"
16:32:16 <jgriffith> I don't know what that means
16:32:20 <evrardjp> but I think the use cases that lead to noticing it are very much low.
16:32:20 <prometheanfire> cinder-api is stopped
16:32:21 <jgriffith> sorry :(
16:33:09 <smcginnis> I suppose it can be considered a bug, just not really a bug directly affecting Cinder, but users of cinder-manage.
16:33:14 <evrardjp> I meant there is not many use cases. In our case, for example, it's very specific. And we can workaround it by checking in the DB. But that's ugly. We were just looking into a more user friendly way forward
16:33:18 <smcginnis> eharney: Any distro related concerns about backporting it?
16:33:37 <prometheanfire> not from gentoo
16:33:43 <smcginnis> evrardjp: I'd rather not have an answer that involves going directly to the db. :/
16:33:44 <evrardjp> haha prometheanfire ;)
16:33:48 <smcginnis> prometheanfire: :)
16:33:53 <prometheanfire> :P
16:33:56 <evrardjp> smcginnis: agreed with you there.
16:34:03 <eharney> is the "vs" example in the commit message correct?
16:34:20 <evrardjp> smcginnis: but I also understand the risks of touching stable code, so it's all about tradeoffs and risks.
16:34:26 <prometheanfire> evrardjp: should be able to confirm
16:34:31 <evrardjp> in other words: no my call!
16:34:34 <eharney> i.e. is it supposed to be including the "@lvm" ?
16:34:54 <evrardjp> no it's FQDN@something vs hostname@something
16:35:06 <evrardjp> IIRC
16:35:36 <hemna> I just ran both cinder service-list and cinder-manage service list  and both output foo@bar for c-vol
16:36:00 <jgriffith> that's added for multi-backend
16:36:02 <smcginnis> This only happens if it is set up with a FQDN, right?
16:36:08 <evrardjp> yes.
16:36:10 <jgriffith> smcginnis yes
16:36:39 <jgriffith> I'm not inclined to support a backport of this FWIW just in order to offer up ability to query and offline service
16:36:43 <prometheanfire> my vs from ocata here
16:36:44 <prometheanfire> | cinder-volume    | storage.mthode.org@lvmdriver-1 | nova | enabled | up    | 2017-08-23T16:36:26.345706 | -               |
16:36:50 <prometheanfire> cinder-volume    storage                              nova             enabled    :-)   2017-08-23 16:36:36.351278 3.10         1.21
16:36:55 <jgriffith> I'm also not inclined to continue to have both versions of the same cmd FWIW
16:37:17 <jgriffith> prometheanfire that what I get as well on current master
16:37:19 <evrardjp> well they don't operate the same way IIRC
16:37:27 <jgriffith> evrardjp nope, they don't
16:37:28 <smcginnis> I think whether we get rid of cinder-manage is another discussion we should probably have, but while we do have it, we should make sure the output matches.
16:37:33 <evrardjp> but I understand the position, and I have fine with that.
16:37:41 <evrardjp> thanks for the consideration!
16:37:43 <jgriffith> smcginnis okie dokie
16:37:46 <smcginnis> But I'll leave that up to our Queen PTL. Er, I mean our Queens PTL. :{
16:37:49 <smcginnis> :P
16:37:53 <jgriffith> haha
16:37:54 <prometheanfire> jgriffith: I think having it in both is good as cinder-manage does not need cinder-api to be running
16:38:04 <jungleboyj> *SMH* again
16:38:05 <prometheanfire> as an operator that's useful
16:38:27 <jgriffith> prometheanfire Yeah, I get it; but I wish operators would speak up a bit more before we introduce madness :)
16:38:30 <evrardjp> prometheanfire: definitely more convenient than checking in the DB ;)
16:38:33 <jungleboyj> I am not in a hurry to remove cinder-manage.
16:38:43 <prometheanfire> :D
16:38:52 <jungleboyj> I do agree that what we get out of cinder-manage and cinder service-list should be consistent.
16:38:58 <jgriffith> jungleboyj we can table that. but to be clear I'm not recommending complete removal of cinder-manage
16:39:16 <prometheanfire> k, so anything else to discuss about this?
16:39:33 <smcginnis> We should maybe discuss cinder-manage at the PTG.
16:39:34 <evrardjp> for this bug, probably not :)
16:39:41 <jungleboyj> jgriffith:  Just removing the ability to show the running services?
16:39:43 * DuncanT wishes cinder service list (and cinder manage) listed API services...
16:39:44 <smcginnis> Let's leave any opinions on the review.
16:39:47 <jgriffith> if you have multiple cinder nodes with the same name but on different domains you're kinda asking for trouble IMO but whatever
16:39:50 <jungleboyj> smcginnis: ++
16:40:01 <evrardjp> smcginnis: agreed
16:40:17 <smcginnis> DuncanT: There's something you can work on. :]
16:40:32 <smcginnis> Thanks evrardjp and prometheanfire
16:40:36 <jgriffith> it does
16:40:37 <jgriffith> sigh
16:40:38 * jgriffith shuts up
16:40:43 <evrardjp> thanks for having us!
16:40:53 <smcginnis> jgriffith: I thought so.
16:40:59 <smcginnis> #topic Open discussion
16:41:17 <smcginnis> I did want to bring up one thing I learned about reno usage.
16:41:47 <smcginnis> It might be changed, but as it is right now reno will scan through and pick up any changes to release notes between tags and show it based on that.
16:42:10 <smcginnis> So if an Ocata release note is updated to fix a type or URL in pike, it will end up showing up in the pike release notes.
16:42:36 <smcginnis> So proper handling for that situation is to edit the release note directly in the stable/ocata branch.
16:42:45 <prometheanfire> has dhellmann been told (or a bug been made)?
16:43:17 <smcginnis> prometheanfire: He's thinking of changing the way reno handles that, but for now I have a couple patches for the reno usage docs and the project team guide.
16:43:42 <smcginnis> Apparently it's a little tricky to handle in reno, so that may take a little time.
16:44:20 <prometheanfire> perhaps a version override header or something
16:45:07 <smcginnis> There is an ignore-note directive that can be added to the release rst file to explicitly tell it to not display specific release notes in the output.
16:45:26 <smcginnis> That's the workaround for the cases where the notes have already been edited.
16:45:43 <smcginnis> But just wanted to raise it here so cores are aware of it in reviewing patches.
16:46:02 <smcginnis> Try not to let release notes updates through on master if it is for a stable release.
16:46:06 <jungleboyj> smcginnis:  Thanks for pointing that out.  That is good to know.
16:46:22 <smcginnis> It was news to me, so I figured it would be good to get out there.
16:46:40 <lhx__> smcginnis, any patch to see about this?
16:47:25 <smcginnis> Reno one was approved: https://review.openstack.org/496391
16:47:34 <lhx__> thanks
16:47:40 <smcginnis> And update to project team guide is pending: https://review.openstack.org/496318
16:48:03 <jungleboyj> smcginnis:  There was a request to add notes to the documentation page about Reno.  I can include that info as well.  :-)
16:48:37 <smcginnis> jungleboyj: Perfect! Probably good to have some basic info and reference over to the reno guide in case that changes at all.
16:48:46 <smcginnis> Anything else?
16:48:51 <jungleboyj> smcginnis:  ++
16:49:47 <smcginnis> Oh, we should probably also get ocata patches through before that transitions stable phases.
16:49:50 <smcginnis> https://review.openstack.org/#/q/project:openstack/cinder+branch:stable/ocata+status:open
16:50:14 <jungleboyj> smcginnis:  Good note.  I think I looked a week ago or so and there wasn't much sitting around.
16:50:23 <jungleboyj> Good reminder to people to get anything proposed.
16:50:46 <smcginnis> ++
16:51:00 <smcginnis> OK, I think that's all I've got. Anything else?
16:51:10 <smcginnis> Bueller?
16:51:35 <smcginnis> Cool, thanks everyone.
16:51:48 <Guest35133> thanks!
16:51:52 <evrardjp> thanks
16:51:53 <prometheanfire> o/
16:51:56 <jungleboyj> Thanks!
16:51:58 <jungleboyj> @!
16:51:58 <_pewp_> jungleboyj ( ・ω・)ノ
16:52:01 <lhx__> thanks!
16:52:14 <smcginnis> #endmeeting