16:00:00 <thingee> #startmeeting cinder
16:00:00 <openstack> Meeting started Wed Mar  4 16:00:00 2015 UTC and is due to finish in 60 minutes.  The chair is thingee. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:01 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:04 <openstack> The meeting name has been set to 'cinder'
16:00:10 <thingee> DuncanT: yes, it's now time
16:00:17 <thingee> hi everyone
16:00:20 <dulek> Hi!
16:00:21 <thangp> o/
16:00:21 <murugang-brcd> hi
16:00:22 <scottda> hey
16:00:22 <eharney> hi
16:00:23 <mkoderer> hello
16:00:25 <mitake> hi
16:00:25 <yamada-h> hi
16:00:25 <DuncanT> Hi
16:00:26 <kmartin> hello
16:00:27 <tbarron> hi
16:00:30 <gary-smith_> hi
16:00:30 <bswartz> o/
16:00:33 <kazum> hi
16:00:35 <e0ne> рш
16:00:37 <thingee> Third party ci reminder
16:00:38 <e0ne> hi!
16:00:39 <thingee> #link http://lists.openstack.org/pipermail/openstack-dev/2015-January/054614.html
16:00:42 <hemna> morning
16:00:43 <thingee> March 19th
16:01:16 <thingee> you should have a third party running way earlier than that. Expecting to have one done on March 19th is just not logical.
16:01:39 <rhe00> hi
16:01:40 <thingee> Please see me if you're driver maintainer and have no idea what I'm talking about
16:01:50 <thingee> Cinder deadlines...
16:01:59 <thingee> #link http://lists.openstack.org/pipermail/openstack-dev/2015-February/056964.html
16:02:04 <smcginnis> o/
16:02:07 <thingee> march 10th, all blueprint code must be merged.
16:02:14 <thingee> if it's not merged by then, it's not going in.
16:02:17 <thingee> no exceptions.
16:02:24 <thingee> bug fixes from there on out for Kilo
16:02:57 <thingee> I'll be hitting the reviews hard this week, and I hope others will join me
16:02:59 <thingee> #link https://launchpad.net/cinder/+milestone/kilo-3
16:03:18 <thingee> great job to all cinder contributors for this
16:03:25 <thingee> we're getting close!
16:03:28 <hemna> thingee, +1
16:03:33 <thingee> see the priorities here https://etherpad.openstack.org/p/cinder-k3-priorities
16:03:36 <thingee> of what to review
16:03:42 <asselin_> hi
16:03:48 <thingee> even if you're not core, your review matters!
16:03:50 <DuncanT> thingee: Did you see the discussion about the oslo logging namespace change? Ivan is going to update it on the 10th, we want to merge it last so it doesn't cause rebase conflicts for everybody else
16:04:27 <thingee> DuncanT: there's an agenda item about this. we can talk about it then
16:04:30 <thingee> lets get started!
16:04:31 <DuncanT> thingee: Actually, it's later on the agenda, ignore me for now
16:04:34 <thingee> #link https://wiki.openstack.org/wiki/CinderMeetings#Next_meeting
16:04:46 <thingee> #topic Vendor driver documentation changes
16:04:48 <thingee> smcginnis: hi
16:04:54 <smcginnis> thingee: Hey!
16:04:57 <thingee> #link https://review.openstack.org/#/c/133372/
16:05:04 <thingee> #link https://wiki.openstack.org/wiki/Documentation/VendorDrivers
16:05:15 <smcginnis> I just wanted a moment to point out some vendor driver documentation changes.
16:05:30 <smcginnis> See links above for the review and a new wiki page to be aware of.
16:05:45 <smcginnis> Basically the plan is to make vendor driver documentation smaller.
16:05:55 <smcginnis> Mostly config info hosted by OpenStack,
16:06:10 <smcginnis> So some vendors probably will need to work on hosting their own documentation.
16:06:20 <smcginnis> It can then be linked to from the official manuals.
16:06:28 <annegentle> thanks for bringing it up smcginnis!
16:06:31 <smcginnis> Please take a moment to go to the wiki page and update your info.
16:06:36 <smcginnis> annegentle: No problem!
16:06:48 <smcginnis> That's really it. Just wanted to make sure everyone was aware of that.
16:07:02 <smcginnis> annegentle: Please correct me if I've mis-stated anything.
16:07:03 <thingee> This is excellent
16:07:11 <annegentle> smcginnis: looks good to me
16:07:19 <annegentle> smcginnis: and yes, really want to raise awareness
16:07:41 <thingee> smcginnis: consider updating https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver
16:07:53 <smcginnis> thingee: Good point, I'll take care of that.
16:08:14 <kmartin> annegentle, would you prefer we have one line per driver or company on the vendor driver wiki?
16:08:23 <thingee> #action smcginnis to update how to contribute cinder driver doc with info on driver doc in openstack docs
16:08:24 <thingee> so much doc
16:08:31 <annegentle> kmartin: looking
16:08:51 <smcginnis> annegentle: If you could also review this section at some point that would be great: https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver#After_Your_Driver_Is_Added
16:08:53 <rhe00> what's the deadline for documentation changes?
16:09:22 <annegentle> kmartin: I like it as one line per "configurable" driver (even if the vendor's the same)
16:10:13 <annegentle> rhe00: for the Configuration Reference, the deadline is same as code, https://wiki.openstack.org/wiki/Kilo_Release_Schedule
16:10:57 <annegentle> #link http://specs.openstack.org/openstack/docs-specs/specs/kilo/move-driver-docs.html
16:11:10 <rhe00> ok, thanks
16:11:17 <thingee> thanks smcginnis and annegentle!
16:11:28 <annegentle> thankee thingee!
16:11:35 <smcginnis> :)
16:11:41 <thingee> #topic Following up modifying volume image metadata
16:11:43 <thingee> davechen: hi!
16:11:45 <davechen> hi
16:11:55 <davechen> Here is some quick update for volume image metadata modification to follow up some comments from previous meeting.
16:11:57 <thingee> spec:
16:11:59 <thingee> #link https://review.openstack.org/#/c/160147/
16:12:04 <thingee> cli:
16:12:06 <thingee> #link https://review.openstack.org/#/c/147077/
16:12:15 <davechen> Firstly, remove the snapshot related changes both in the SPEC (https://review.openstack.org/#/c/160147/)
16:12:21 <davechen> and CLI patch (https://review.openstack.org/#/c/147077/).
16:12:28 <davechen> So, only CRUD on the image metadata is proposed in the patches.
16:12:47 <jungleboyj> o/
16:12:51 <davechen> Secondly, import both role based props protection (https://review.openstack.org/#/c/160090/) and policies based props protection(https://review.openstack.org/#/c/160145/)  from Glance, and have a thorough testing recently.
16:13:11 <davechen> Props protection is based on the RBAC, basically, role based props protection judge user's permission by his role and compare his roles with predefined rule configuration files;
16:13:20 <davechen> polices based props protection parse the rules defined in configuration files and appended the rules to the basic RBAC rules and then determine the user access permission.
16:13:37 <DuncanT> davechen: Any plan at some point to make this a common library with glance?
16:13:47 <davechen> these is the most important comment I get from previous meeting.
16:14:08 <davechen> DuncanT: common library?
16:14:22 <davechen> DuncanT: not yet
16:14:48 <DuncanT> davechen: Ok, thanks. Just wondered
16:14:58 <thingee> davechen: I appreciate the work on this.
16:15:00 <davechen> DuncanT: just did these separately, but it is good idea to do in that way.
16:15:17 <davechen> so, anything else I need address?
16:15:21 <thingee> davechen: at this time we'll have to revisit things in Liberty. We're already pretty late in the Kilo cycle with the 20 targeted things in review.
16:16:02 <davechen> thingee: yeah, so is it still possible to merge the basic CRUD APIs?
16:16:41 <thingee> davechen: The problem is review resources. If I can keep focused on what's already on our plate, that would be great.
16:16:58 <thingee> davechen: plus the number of people that would be pissed that aren't getting in because of the feb 15 feature freeze deadline
16:17:00 <davechen> thingee: just check the review list, the feature is not planed in the review list even if the low priority.
16:17:15 <thingee> davechen: and march 1st code pre feature freeze deadline
16:18:07 <davechen> thingee: okay, hope some volunteer could help to review it.
16:18:48 <thingee> davechen: thanks
16:18:55 <thingee> #topic OpenStack Infra: add non-voting Sheepdog test job to cinder check pipeline
16:18:57 <thingee> kazum: hi
16:19:02 <kazum> hi
16:19:07 <thingee> I'm not sure why we're discussing this here. Not much we can do.
16:19:24 <eharney> i didn't understand hemna's objection on this one
16:19:40 <hemna> my objection was related to the fact that this was added w/o talking to the Cinder team
16:19:51 <hemna> they want to make it voting
16:19:59 <e0ne> if it is stable and non-voting, why not?
16:20:03 <DuncanT> No way is it going to be voting
16:20:12 <hemna> Their entire intention was to make it voting
16:20:12 <thingee> the commit message says never voting
16:20:18 <eharney> i thought it was aiming for voting, which would put it in line with what we are already doing for ceph/gluster
16:20:21 <hemna> and they didn't even talk to us about it.
16:20:23 <eharney> non-voting*
16:20:25 <kazum> yes, we have no plan to make it voting.
16:20:35 <eharney> so this isn't really a new scheme
16:21:01 <thingee> So I know there was some discussion about this on the ML. I really don't care if people want to add checks in the pipe queue if infra wants to work with them.
16:21:16 <DuncanT> Bringing it up at the meeting is probably polite, just so there's an opportunity to shout.
16:21:17 <smcginnis> Original commit mentioned planning to eventually vote. That was my only concern.
16:21:18 <thingee> non-voting for sure.
16:21:25 <hemna> smcginnis, +1
16:21:35 <DuncanT> I'd rather it didn't vote as jenkins, but it isn't the end of the world
16:21:38 <hemna> yup, their original messaging was to make it voting
16:21:52 <smcginnis> Any concern with this increasing gate time?
16:21:55 <thingee> hemna: it might've just been a mistake
16:21:56 <kazum> hemna: yes, sorry for confusion.
16:21:58 <hemna> so I just thought it wasn't very cool to do something like this without at least talking to the Cinder team first.
16:22:04 <thingee> and misunderstanding
16:22:09 <hemna> that's all.
16:22:19 <DuncanT> Voting as jenkins makes it look more official than the other 3rd party CIs, which I don't really like
16:22:31 <eharney> fortunately infra is waiting for Cinder folks to give feedback on the review
16:22:45 <jordanP> DuncanT, I agree with you
16:22:50 <hemna> eharney, we only found out about it because anteaya brought it up to me
16:22:51 <smcginnis> Would be great if there was a separete open source third party CI.
16:22:57 <thingee> well I think the interested parties should follow up on the review. Nothing should be voting.
16:23:01 <thingee> We've discussed this here:
16:23:01 <hemna> thankfully she gave us a heads up
16:23:03 <thingee> #link https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#When_thirdparty_CI_voting_will_be_required.3F
16:23:20 <asselin_> infra is working on creating new accounts to comment as not-jenkins
16:23:42 <thingee> hemna: can you follow up so infra feels comfortable again?
16:23:48 <hemna> thingee, sure
16:24:01 <thingee> kazum: anything else?
16:24:05 <anteaya> all I needed to see was that you knew
16:24:07 <hemna> just to be clear
16:24:09 <anteaya> so now I see
16:24:11 <anteaya> and thanks
16:24:13 <DuncanT> asselin_: Cool stuff
16:24:13 <hemna> we are ok with it non-voting ?
16:24:14 <kazum> no, thanks a lot :)
16:24:42 <thingee> hemna: right non-voting
16:24:46 <hemna> ok cool.
16:24:49 <thingee> current decision still stands at https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#When_thirdparty_CI_voting_will_be_required.3F
16:24:53 <thingee> until rediscussion
16:25:00 <jungleboyj> +2
16:25:14 <thingee> hemna, kazum: thanks
16:25:35 <thingee> #topic Porting volume drivers to use ABCmeta
16:25:40 <thingee> mkoderer: hi
16:25:41 <mkoderer> hi there :)
16:25:42 <thingee> DuncanT: hi
16:25:54 <DuncanT> Hi
16:26:01 <thingee> #link https://review.openstack.org/#/c/138999/
16:26:01 <DuncanT> mkoderer has the mike
16:26:08 <thingee> #link https://review.openstack.org/#/q/project:openstack/cinder+branch:master+topic:bp/abc-volume-drivers,n,z
16:26:09 <mkoderer> the driver change already got merged
16:26:19 <thingee> #link https://review.openstack.org/#/c/127455/
16:26:19 <mkoderer> and I just realized that the BP was in state implemented
16:26:34 <mkoderer> but in fact there is still much to do
16:26:48 <mkoderer> IMHO we should try to port all drivers to use the model
16:27:03 <hemna> mkoderer, in K ?
16:27:04 <mkoderer> so I started with: rbd, gluster, nfs and block_device
16:27:05 <e0ne> mkoderer: +1
16:27:08 <DuncanT> mkoderer ++
16:27:09 <patrickeast> in liberty right?
16:27:20 <thingee> mkoderer: would consider breaking up the blueprint then. Have the base driver class bp marked as implemented.
16:27:27 <mkoderer> I personally don't care... it's up to you guys :)
16:27:30 <DuncanT> Might as well start ASAP, it is a mostly a mechanical change
16:27:32 <hemna> patrickeast, +1
16:27:34 <thingee> mkoderer: just in interest of time left in kilo with testing
16:27:48 <hemna> I'm a bit worried making this change this late in the game
16:27:53 <thingee> hemna: +1
16:28:03 <hemna> I'd vote for doing it early in L
16:28:08 <mkoderer> so the changes as mostly simple: https://review.openstack.org/#/q/project:openstack/cinder+branch:master+topic:bp/abc-volume-drivers,n,z
16:28:21 <tbarron> hemna: +1
16:28:33 <e0ne> mkoderer: we can't verify it without CI
16:28:34 <mkoderer> ok let's to it in L
16:28:43 <tbarron> changes may be simple but soak time is needed
16:28:45 <mkoderer> e0ne: I see
16:28:51 <hemna> tbarron, yup
16:28:57 <e0ne> mkoderer: i mean 3rd party ci :(
16:28:59 <jungleboyj> Agree this shoudl wait for L.
16:29:08 <kmartin> mkoderer, yes but all the additional reviews for each driver change needs to be taken into account
16:29:14 <winston-d> +1 for L
16:29:29 <mkoderer> thingee: should I create an additional spec for that?
16:29:36 <cebruns> tbarron: +1
16:29:37 <thingee> #agreed we wait for the rest abc driver changes in L
16:29:38 <Swanson> L
16:29:38 <thingee> mkoderer: please
16:29:42 <mkoderer> thingee: k
16:29:54 <thingee> #action mkoderer to break up abc driver blueprint
16:29:57 <jungleboyj> mkoderer: Is this something you are planning to do or do driver owners need to plan to make the changes?
16:30:01 <hemna> mkoderer, I'd like to ping you about a driver.FibreChannelVD in #cinder and see if it makes sense to you
16:30:07 <thingee> #action to adjust kilo release page accordingly with blueprints
16:30:12 <mkoderer> and I added a script that extracts the driver support matrix out of ABC
16:30:13 <mkoderer> https://review.openstack.org/#/c/160346/
16:30:37 <mkoderer> jungleboyj: I would really like to have the help of the driver owners .. yes :)
16:30:43 <thingee> jungleboyj, mkoderer: I have plans to send a mass email to driver maintainers on some other changes in Cinder, so it would be great to send it out then as well.
16:30:57 <mkoderer> hemna: ok do that
16:31:11 <jungleboyj> mkoderer: Ok.  Thanks for the heads up.
16:31:14 <jungleboyj> thingee: Good idea.
16:31:26 <mkoderer> thingee: +1
16:31:26 <hemna> mkoderer, we can convert our drivers, so no problem there.
16:31:32 <hemna> just in L :)
16:31:35 <DuncanT> thingee: Can you cc me on any such mass emails. please? I invariably get asked on IRC about them the day after (since I'm in a GMT+X timezone) and I have to go digging
16:31:41 <thingee> emails about new things in Kilo that drivers can do in L will be going out soon. Want to make sure folks are informed.
16:31:43 <mkoderer> hemna: :) ook
16:31:45 <jungleboyj> mkoderer: Same here, just need to example to work from.
16:32:04 <thingee> DuncanT: sure
16:32:13 <DuncanT> Thanks
16:32:14 <thingee> I also send a copy to the mailing list though
16:32:14 <mkoderer> jungleboyj: yep I guess there are enough examples already in.. and it should be simple
16:32:17 <thingee> DuncanT: ^
16:32:27 <thingee> did te same thing with the CI email
16:32:31 <mkoderer> jungleboyj: like https://review.openstack.org/#/c/138661/
16:32:41 <DuncanT> thingee: That's where I go digging for it. Anything with my address CCd bypasses filters, that's all
16:32:49 <thingee> DuncanT: k
16:32:51 <jungleboyj> mkoderer: Cool.  Thanks!
16:32:56 <mkoderer> k folks thx
16:33:01 <thingee> mkoderer: thank you
16:33:26 <thingee> mkoderer: we all appreciate your work in this release with getting things going :)
16:33:51 <thingee> #topic Move to oslo_log is coming!
16:33:55 <mkoderer> thingee: I do my very best :D
16:33:55 <thingee> jungleboyj: hi
16:33:59 <e0ne> thingee: tha main question is: do you want to land it in Kilo?
16:34:07 <thingee> #link https://review.openstack.org/#/c/157441
16:34:18 <e0ne> when is the best time for it? before or after FF. it requires a lot of rebases
16:34:19 <jungleboyj> thangp: Hi.
16:34:28 <jungleboyj> thingee: Hi.
16:34:29 <thangp> jungleboyj: hi
16:34:37 <jungleboyj> thangp: Sorry, wrong th*
16:34:41 <thangp> lol
16:34:46 <e0ne> :)
16:34:47 <thingee> e0ne, jungleboyj: what does it buy us to rush things?
16:35:08 <jungleboyj> thingee: So, I think we want to get this landed in K.  DuncanT and hemna Were feeling that way.
16:35:28 <e0ne> thingee: personally, i've got a lot requests from our operators for it
16:35:29 <jungleboyj> thingee: Less that we need to worry about oslo.log in incubator.
16:35:34 <DuncanT> thingee: It is a good time to do it, straight after code freeze - otherwise we get lots of merge conflicts in future
16:35:37 <jungleboyj> jgriffith: Had requested it.
16:35:48 <thingee> DuncanT: +1
16:35:49 <hemna> avoiding rebase churn would be nice.
16:36:00 <hemna> I'm trying to avoid that as well with the shadow tables review
16:36:05 <e0ne> hemna: +1
16:36:09 <winston-d> e0ne: requests about what?
16:36:16 <jungleboyj> So, we would like to do it 3/10 or 3/11 to avoid disrupting the new function and get it in before we start doing a lot of fixes.
16:36:20 <tbarron> DuncanT: that's compelling
16:36:32 <thingee> well technically this is bug in Kilo https://launchpad.net/cinder/+milestone/kilo-3
16:36:34 <jungleboyj> Still get plenty of runt ime on it.
16:36:35 <e0ne> winston-d: moving to oslo.log to make logs of different components similar
16:36:40 <thingee> so I don't think anyone needs to convince me ;)
16:36:43 * DuncanT is happy to +2 it very early on the 11th
16:36:59 <e0ne> thingee: i filled a BP for it
16:37:07 <DuncanT> Just so that it doesn't stop any last minute merges on the 10th
16:37:08 <thingee> e0ne: not sure why we need one
16:37:17 <jungleboyj> thingee: +2
16:37:27 <thingee> anyways, I'm fine if we can get it early after the freeze to help avoid rebase hell
16:37:36 <jungleboyj> thingee: +2
16:37:49 <e0ne> thingee: thanks! +1 for each rebase
16:37:57 <jungleboyj> Great.  Mainly I just wanted to bring it up here so people knew it was coming and there would be a brief period of rebase hell.
16:38:03 <jungleboyj> In a week.
16:38:04 <thingee> jungleboyj: makes sense
16:38:30 <thingee> jungleboyj, e0ne: thanks for keeping up with the oslo changes.
16:38:36 <jungleboyj> thingee: Cool!  Thank you to e0ne for is work on that!
16:38:40 <thingee> jungleboyj: are we done with oslo_ changes after this?
16:38:55 <jungleboyj> e0ne: Thanks for keeping me honest wiht the changes.
16:39:08 <e0ne> jungleboy: :)
16:39:12 * thingee thinks he asks jungleboyj this once a week
16:39:30 <jungleboyj> We are mostly done with name space changes.  I have the removal of config patch I am working on to get that out of incubator.
16:39:46 <jungleboyj> I have to get the scheduler workingt.
16:39:50 <thingee> doh ok
16:39:52 <e0ne> jungleboyj: cool
16:40:14 <hemna> so how much more do we want to get in then for K?
16:40:15 <jungleboyj> Once the logging change is in and the other two I will have a couple more clean-up patches.
16:40:19 <hemna> those seem like big changes
16:40:21 <hemna> late
16:40:40 <jungleboyj> thingee: As far as oslo_ goes.  I think we are done.  Looked a bit yesterday.
16:40:40 <hemna> the oslo updates have bit us in the rear end a few times before late in the release cycles
16:40:54 <thingee> so maybe early L
16:41:01 <jungleboyj> hemna: Right, that is why I want to get these wrapped up in the next week.
16:41:05 <jungleboyj> I should be able to do so.
16:41:05 <hemna> thingee, +1
16:41:17 <hemna> ok
16:41:24 <jungleboyj> hemna: The only big change is the scheduler part.
16:41:33 <hemna> yah, that one, should wait IMHO
16:41:50 <DuncanT> hemna might have a point... big changes at this point are high risk
16:41:52 <thingee> hemna: I agree. I'm mostly not excited about oslo_log changes personally because I don't know what it's buying us to have it so late.
16:41:53 <thingee> hemna: but if majority of the community is really wanting this and we can have it early after freeze, fine
16:41:53 <thingee> hemna: re oslo_log
16:41:53 <jungleboyj> thingee: winston-d, what do you think?
16:42:11 <DuncanT> logging is a small change, just hits an annoying number of files
16:42:12 <hemna> thingee, +1
16:42:20 <jungleboyj> DuncanT: +1
16:42:23 <e0ne> DuncanT: +1
16:42:25 <thingee> jungleboyj: again I'm not sure why we want oslo_log right away. So I'm unlikely to agree with anything else being updated at this point.
16:42:30 <DuncanT> scheduler makes me more nervous
16:42:31 <winston-d> jungleboyj: the normalize support in scheduler filter?
16:42:36 <hemna> yah the log patch is fairly limited.  as long as oslo_log isn't broken itself, it should be ok.
16:42:40 <jungleboyj> winston-d: Yes.
16:42:44 <thingee> 8 days + whatever RC time for gate testing. I think we need to be careful at this point
16:42:53 <hemna> thingee, +2!!
16:43:11 <winston-d> jungleboyj: don't we have a dependenecy in olso-incubator?
16:43:27 <jungleboyj> winston-d: There is a fix I would need to get in there.
16:43:31 <winston-d> jungleboyj: to add support for 'infinite' in normalize()
16:43:47 <jungleboyj> winston-d: Right.  I am ok with waiting on that if that is ok with you.
16:43:58 <thingee> does anyone disagree on waiting for things besides oslo_log?
16:44:03 <thingee> in Kilo
16:44:19 <jungleboyj> thingee:  Let me make the following proposal:
16:44:37 <jungleboyj> 1) Get oslo_log change in for Kilo.
16:44:47 <winston-d> jungleboyj: please wait for a proper fix to accommadate 'infinite'
16:44:56 <jungleboyj> 2) Hold off on scheduler change for Liberty.
16:45:12 <e0ne> just fyi, oslo feature freaze is on March, 12th
16:45:25 <jungleboyj> 3) I move to the new config engine this week as it lets us get other cleanup in incubator done.
16:45:37 <jungleboyj> Remove files that we don't need to be carrying along.
16:45:58 <jungleboyj> Those will just be removing files we already have stopped using.
16:46:07 <jungleboyj> Then I will be done with Oslo.
16:46:15 <thingee> 1 and 2 seem fine. Need to see what 3 looks like
16:46:19 <DuncanT> Config engine change makes me a little nervous, but gate should test it fairly well
16:46:41 <jungleboyj> DuncanT: +1  It shouldn't be a huge deal.
16:46:51 <jungleboyj> I will get it coded up and see what you guys think.
16:46:51 <DuncanT> Need to follow up on any 3rd party failures though, since they are testing more of the config code than jenkins
16:46:52 <e0ne> jungleboyj: will it impact any package mainteiners?
16:47:04 <jungleboyj> e0ne:  Not if I write it properly.  :-)
16:47:20 <e0ne> jungleboyj: fair enough :)
16:47:25 <thingee> jungleboyj: anything else? we have a couple more things on the agenda
16:47:43 <jungleboyj> e0ne: DuncanT thingee I will stop work on the scheduler and move to the config work so you have something to see yet this wek.
16:47:49 <jungleboyj> We can revisit next week.
16:47:56 <thingee> jungleboyj: ok
16:47:59 <e0ne> ok
16:48:04 <thingee> #topic Replication follow up from last meeting
16:48:05 <jungleboyj> thingee: Thanks for the discussion and input!
16:48:05 <thingee> jungleboyj: hi
16:48:11 <e0ne> :)
16:48:13 <jungleboyj> thingee: He.
16:48:17 <jungleboyj> Hi.
16:48:20 <jungleboyj> :-)  *sigh*
16:48:25 <jungleboyj> I can't type today.
16:48:30 <thingee> jungleboyj: I'm going to do a hard cut on this in 6 mins. need to leave time for jbernard
16:48:31 <jungleboyj> Anyway, so I just wanted to follow up.
16:48:55 <jungleboyj> We took the discussion last week about GPFS and Storwize to heart.
16:49:02 <jungleboyj> So, this is a quick discusion.
16:49:26 <jungleboyj> We are not going to try to get any more replication changes in for this release and we are going to focus efforts on helping get V2 on track for Liberty.
16:49:41 <thingee> jungleboyj: +1
16:49:44 <jungleboyj> You all are right that that makes more sense than trying to get more changes in now.
16:50:06 <jungleboyj> Hope that we can be of better benefit to everyone by focusing our efforts that way.
16:50:14 <jungleboyj> That was all I had on that.
16:50:17 <jungleboyj> :-)
16:50:43 <thingee> I'll be providing more input on the replication stuff soon. Just in constant review mode :)
16:50:46 <thingee> for kilo
16:50:58 <jungleboyj> thingee: Same here.
16:51:07 <thingee> jungleboyj: thanks
16:51:10 <jungleboyj> We are continuing to look at the bugs out there.
16:51:26 <DuncanT> 9 minute warning
16:51:26 <thingee> jungleboyj: I've been watching, thanks :)
16:51:30 <thingee> #topic Generic volume migration consensus
16:51:33 <thingee> jbernard: hi
16:51:35 <jbernard> hey guys
16:51:35 <thingee> jgriffith: here?
16:51:42 <jungleboyj> thingee: Welcome.
16:51:53 <jbernard> my volume migraiton patch is availalbe for review https://review.openstack.org/#/c/135002/
16:51:58 <thingee> I've pinged jgriffith number of times to relook at this
16:52:03 <jbernard> winston has provided some great feedback
16:52:08 <jbernard> and we've iterated a few times
16:52:17 <jbernard> im left with two questions:
16:52:32 <jbernard> 1. Do we agree that the patch in it's current form is not good enough to merge
16:52:49 <jbernard> 2. If winston's suggested approach is implemented, is that what we want
16:53:09 <jbernard> and i guess also
16:53:19 <jbernard> 3. we need CI for this, where does that fit in
16:54:01 <jbernard> the patch as it is, adds support for rbd migration wihtout introducing regressions for previous drivers
16:54:07 <jbernard> but the encapsulation is not awesome
16:54:50 <jbernard> and in general, im looking for some agreement so i don't go off in a wrong direction
16:54:51 * DuncanT would like to look at LVM + 3rd party driver for 3rd party CI in L. Probably not required initially, but it would allow at least some migration testing
16:55:11 <jbernard> also, we can revisit next week, i dont need an answer right now
16:55:18 <jbernard> it may take time to digest it all
16:55:23 <jbernard> but i wanted to bring it up
16:55:38 <jbernard> QED
16:55:56 * DuncanT didn't like the overloaded attach that doesn't attach, for the record
16:56:18 <jbernard> DuncanT: in the alternative? or as it is currently?
16:56:25 <DuncanT> But I couldn't come up with anything better... am trying to make sense of winston's proposal now
16:56:29 <jbernard> ahh
16:56:45 <jbernard> me too, this morning has been busy
16:57:15 <winston-d> I hope avishay can sponse the review this generic vol migration patch.
16:57:31 <thingee> ok, I don't have much input on this. Honestly haven't been following, so looking to others to chime in
16:57:33 <winston-d> he's the original author of migration.
16:57:45 <thingee> 3 mins
16:58:12 <winston-d> time for opens?
16:58:33 <winston-d> I have one
16:58:44 <thingee> jbernard: thanks and sorry for the lack of discussion
16:58:49 <thingee> #topic open discussion
16:58:53 <bswartz> Next week cinder meeting is Noon Eastern/9AM Pacific thanks to DST in the USA
16:58:58 <jbernard> no worries, if anyone does have time to look, i would be grateful
16:59:02 <thingee> bswartz: +1
16:59:04 <thingee> thanks for the reminder
16:59:18 <DuncanT> Still the same time UTC, right?
16:59:23 <thingee> make sure to send your summit session proposals here! https://etherpad.openstack.org/p/cinder-liberty-proposed-sessions
16:59:27 <bswartz> yeah UTC time never changes
16:59:28 <jungleboyj> bswartz: +1  Thank you!
16:59:55 <hemna> 1 minute
17:00:08 <winston-d> bug: https://bugs.launchpad.net/cinder/+bug/1406703
17:00:10 <openstack> Launchpad bug 1406703 in Cinder "Deleting VM with an attached volume during copy-volume-to-image causes the volume remains in-use state" [Undecided,In progress] - Assigned to Abhijeet Malawade (abhijeet-malawade)
17:00:10 <thingee> #endmeeting