Monday, 2016-08-15

*** bswartz has joined #openstack-meeting-cp01:02
*** bswartz has quit IRC01:13
*** TaylorHuang has joined #openstack-meeting-cp01:14
*** TaylorH has quit IRC01:17
*** gothicmindfood has quit IRC02:40
*** gouthamr has quit IRC04:33
*** bswartz has joined #openstack-meeting-cp05:07
*** ebalduf has quit IRC05:44
*** david-lyle has joined #openstack-meeting-cp06:36
*** david-lyle_ has quit IRC06:39
*** belmoreira has joined #openstack-meeting-cp06:48
*** sdake has joined #openstack-meeting-cp08:01
*** sdake has quit IRC08:15
*** sdake has joined #openstack-meeting-cp08:35
*** markvoelker has joined #openstack-meeting-cp08:41
*** sdake has quit IRC08:43
*** markvoelker has quit IRC08:45
*** sdake has joined #openstack-meeting-cp09:46
*** sdague has joined #openstack-meeting-cp10:31
*** markvoelker has joined #openstack-meeting-cp10:42
*** markvoelker has quit IRC10:47
*** sdake has quit IRC12:02
*** sdake has joined #openstack-meeting-cp12:06
*** gouthamr has joined #openstack-meeting-cp12:10
*** sigmavirus|away is now known as sigmavirus12:10
*** xarses has quit IRC12:28
*** markvoelker has joined #openstack-meeting-cp12:30
*** sdake has quit IRC12:38
*** xyang1 has joined #openstack-meeting-cp12:56
*** belmoreira has quit IRC13:31
*** belmoreira has joined #openstack-meeting-cp13:50
*** dims has quit IRC13:59
*** dims has joined #openstack-meeting-cp14:05
*** uxdanielle has joined #openstack-meeting-cp14:08
*** ebalduf has joined #openstack-meeting-cp14:09
*** sdake has joined #openstack-meeting-cp14:10
*** sdake has quit IRC14:48
*** xarses has joined #openstack-meeting-cp14:53
*** xarses has quit IRC14:59
*** tyr has joined #openstack-meeting-cp15:38
*** belmoreira has quit IRC15:38
*** Benj_ has joined #openstack-meeting-cp15:50
*** xarses has joined #openstack-meeting-cp15:57
*** dims has quit IRC16:10
*** tyr has quit IRC16:16
*** xarses has quit IRC16:32
*** dims has joined #openstack-meeting-cp16:42
*** dims has quit IRC16:47
*** dims has joined #openstack-meeting-cp16:53
ildikov#startmeeting cinder-nova-api-changes17:00
openstackMeeting started Mon Aug 15 17:00:18 2016 UTC and is due to finish in 60 minutes.  The chair is ildikov. Information about MeetBot at http://wiki.debian.org/MeetBot.17:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.17:00
*** openstack changes topic to " (Meeting topic: cinder-nova-api-changes)"17:00
openstackThe meeting name has been set to 'cinder_nova_api_changes'17:00
ildikovscottda ildikov DuncanT ameade cFouts johnthetubaguy jaypipes takashin alaski e0ne jgriffith tbarron andrearosa hemna erlon mriedem gouthamr ebalduf patrickeast smcginnis diablo_rojo gsilvis17:00
scottdahi17:00
*** sdake has joined #openstack-meeting-cp17:00
* scottda is on-time for once17:00
ildikovscottda: hi :)17:00
jgriffithildikov: :)  Thought you had a conflict17:01
ildikovI have a meeting in parallel, so if you feel like facilitating this one a bit that would be great :)17:01
jgriffithOr are you time slicing your meeting processor17:01
jgriffith:)17:01
jgriffithOk... so I'll make this sort of easy if there are no objections.17:02
patrickeastobjection!17:02
jgriffithanybody have any updates or questions they'd like to bring up?17:02
jgriffithpatrickeast: darn you!17:02
ildikovjgriffith: I'm a girl, I should be able to figure this out ;)17:02
ildikovjgriffith: any updates on your side?17:02
ildikovtempest or Nova stuff?17:03
jgriffithildikov: I have nothing new to report17:03
ildikovjgriffith: :(17:04
ildikovjgriffith: I saw you +1'd my patch, which depends on yours17:04
jgriffithildikov: turn that frown upside down17:04
ildikovI can work on adding a few more tests if you're not working on that17:04
jgriffithyes, thanks for fixing that17:04
ildikovno probs, I was playing with it earlier17:05
jgriffithildikov: so honestly once we decided this was O work it's certainly not on the front burner for me17:05
ildikovjgriffith: I'm not sure I'm following you now17:06
hemnamep17:06
*** mriedem has joined #openstack-meeting-cp17:06
* mriedem lurks17:06
jgriffithildikov: the attach/detach changes17:07
hemnaI haven't had time to test it :(17:07
jgriffithildikov: given that they're not going to go in for N17:07
hemnatoo many things coming at the same time right now.17:07
jgriffithhemna: that's how it goes... no worries17:07
mriedemyup my time is constrained this week, i'm on vacation next week and then it's n-3 after that17:07
johnthetubaguyit would be great to get a spec to write up our plans pre summit17:07
ildikovjgriffith: I thought for Cinder we can still consider it, but then I missed a few bits17:07
jgriffithI will get the latest update pushed to the review this week17:07
ildikovjgriffith: and then get things done in Nova as well17:07
ildikovjgriffith: as it's a new API in Cinder with microversions it should not break anything until no one is using it17:08
jgriffithildikov: so my point was that things for the current release are priority17:08
*** harlowja has joined #openstack-meeting-cp17:08
ildikovjgriffith: if there are other things that are urgent and no bandwidth for this that's another topic17:08
jgriffithildikov: we already all agreed (I thought) that this work was not going to go in during the N release.  Unless I horribly misunderstood something?17:08
johnthetubaguyso I have an aside here17:09
ildikovjgriffith: we might do a task list and let people pick up things if they have time17:09
jgriffithjohnthetubaguy: go for it17:09
johnthetubaguyturns out we having a very similar conversation about neutron ports and neutron attachments17:09
johnthetubaguymentioned that to jgriffith earlier17:09
johnthetubaguythere are plans to get live-migrate into a state machine during Ocata17:09
johnthetubaguyit would be great if we end up with similar patterns in the end17:10
scottdajohnthetubaguy: Is there a spec for the live-migrate state machine?17:10
johnthetubaguythere are slight differences around the shared cinder connection, and neutron needing to know the active port, but its very similar17:10
johnthetubaguyscottda: the full one I think is WIP, I have notes on the neutron thing here: https://review.openstack.org/#/c/353982/17:11
johnthetubaguyscottda: its a little bit too rough and ready to be useful yet, but just trying to capture stuff before the neutron midycle later in the week17:11
scottdaCool, thanks. #link https://review.openstack.org/#/c/353982/17:11
scottdajohnthetubaguy: sure. I'll keep an eye in it..17:12
johnthetubaguybasically, I think port bindings and volume attachments are quite similar, and during live-migrate we have two of both during the live-migration17:12
johnthetubaguyright now we only have one during the live-migration, evening though we kinda have two connections during live-migration17:12
johnthetubaguybut anyways, hopefully we converge on something similar on both sides17:13
scottda+1000 to doing things the same (as much as possible) for nova and cinder17:13
scottdas/nova/neutron17:13
johnthetubaguythere is obviously the shared problem around token expiry, etc17:13
johnthetubaguyanyways, I am trying to keep an eye on both conversations to see what is the same / different17:14
johnthetubaguyI think neutron has a little more interaction around what is active, and cinder has a little more state to share around the shared volume target stuff17:15
jgriffithjohnthetubaguy: I'll study the spec you linked and see if we can combine and get a cinder or a shared version written up17:15
johnthetubaguyso that current spec *should* be close to a shared vision, its just missing some words for it to make sense on both sides17:15
johnthetubaguyits probably worth waiting till next week, I am hopeful to have more details then17:16
ildikovjohnthetubaguy: thanks for sharing17:16
ildikovjohnthetubaguy: will this be finalised on the Neutron mid-cycle?17:17
johnthetubaguyno problem17:17
johnthetubaguyI suspect not, I am aiming to have this at the summit, ideally something cross project with us all together17:17
johnthetubaguyI am really just hoping to get vague agreement at the midcycle17:17
johnthetubaguylike to the level we have between Nova and Cinder right now17:17
ildikovah, ok17:19
ildikovalthough if there's a rough agreement that should help with figuring out which direction to look at with the Cinder items17:19
johnthetubaguy+1 hopefully17:19
ildikovfrom Cinder perspective I would guess the question is more how to use the new calls jgriffith is proposing as opposed to change and adapt those, right?17:20
scottdaI think Cinder might need additional calls for special cases like live-migrate...17:21
jgriffithildikov: my take away was to modify them most likely to have a similar pattern between the two17:21
johnthetubaguyI think it will end up being about using a similar flow, like Cinder owning attachment state, and Neutron owning binding state17:21
jgriffithscottda: I've already got that FWIW17:21
johnthetubaguyscottda: neutron needs more calls right now17:21
jgriffithscottda: anyway17:21
scottdajgriffith: Cool. I figured you'd already thought about it...17:21
johnthetubaguyjgriffith: do you have this written up now?17:22
jgriffithjohnthetubaguy: of course not :)17:22
scottdaWhat, no spec?17:22
* scottda ducks17:22
jgriffithjohnthetubaguy: the past few weeks have been..... umm... anyway17:22
jgriffithjohnthetubaguy: but like I said, I'll try and get that knocked out on planes the next week or so17:22
johnthetubaguyjgriffith: yeah, gotcha, writing that up would be cool, so we can compare whats in our heads17:23
ildikovjgriffith: your calls are pretty basic and in line with what johnthetubaguy is saying17:23
johnthetubaguyjgriffith: sounds good17:23
jgriffithildikov: first look at the spec he referenced yes I think we're aligned17:23
johnthetubaguyneutron doesn't have a reserve thing yet, but I am asking for one, etc17:23
ildikovjgriffith: I would be sad if we would need to complicate your proposal17:23
ildikovjgriffith: having some additional calls, like that safe_to_remove, etc are fine of course17:24
jgriffithildikov: I think we'll be alright17:24
johnthetubaguyyeah, if that happens we are doing it wrong17:24
jgriffithildikov: I do need to write up a proper spec and get back to finishing the code though17:24
ildikovjgriffith: and a spec would be nice to list work items, so if we can get some extra hands we can point them to the spec17:25
ildikovjohnthetubaguy: +117:25
johnthetubaguyI hope we can get some OSIC help with this, is enterprises really want stable live-migrate17:26
johnthetubaguys/is/as/17:26
ildikovyeap, that's understandable17:27
johnthetubaguyanyways, this is future stuff, did we want to cover the newton issues? did they get bashed out now?17:27
ildikovand we know it's "hacky" at a few places, so fixing it would make our lives easier as well17:28
mriedemjohnthetubaguy: the lock thing is being discussed in -cinder,17:28
mriedemi haven't been following along, but imagine there will be a summary in the ML17:28
johnthetubaguymriedem: +1 on that17:29
ildikovjohnthetubaguy: the VM actions part can be figured out when we have a good attach/detach flow17:29
ildikovjohnthetubaguy: maybe we can try to at least list them in jgriffith's spec, but it might be too much writing for no good (enough) reason17:30
johnthetubaguyildikov: so my (personal) goal post summit is to have an answer for attach, detach and live-migrate, the others "should be easy".17:30
ildikovI don't think a detailed list dough, just point out where we need eyes and then hands to fix the flow17:30
ildikovjohnthetubaguy: that's my thinking as well, attach/detach is getting close-ish to a PoC state just to see it working17:31
johnthetubaguyanyways, feels like we are done for now, maybe?17:33
scottdadone17:34
ildikovjohnthetubaguy: if you could check this chain: https://review.openstack.org/#/c/335358/ ?17:35
ildikov:)17:35
ildikovjohnthetubaguy: as it has test changes as well I would rather not keep it rebasing much as it can be painful17:36
johnthetubaguyyeah, sorry, that is on my list, just not got to it yet17:37
*** tyr has joined #openstack-meeting-cp17:38
ildikovjohnthetubaguy: ok, tnx17:40
ildikovjohnthetubaguy: the test patches already got review comments, so I hope they're fine17:40
*** tyr has quit IRC17:40
ildikovjohnthetubaguy: the remove check_attach part can be tricky, I'm not sure hemna had some time to look at it17:41
hemnaI haven't yet :(17:41
ildikovjohnthetubaguy: mriedem added a few comments, I fixed those17:41
ildikovhemna: no worries, I got the tests green now finally, I take it as a good sign :)17:42
mriedemat this point in the cycle i'm not sure how much i want to land that change17:42
ildikovhemna: although if you can take a look that would be great also, just to have more eyes on the new reserve_volume call17:42
ildikovmriedem: well, it's still a bug fix17:43
hemnaildikov, ok cool I'll try to take a look17:43
mriedemit's a pretty complicated bug fix for a long standing latent issue though17:43
hemnaI'm stacked 3 deep at the moment..17:43
mriedemand we've got 3 weeks before feature freeze and people tied up in reviewing priority blueprints still17:44
mriedemi think i'm just trying to say, this is very low priority for me right now17:44
johnthetubaguymriedem: risk / benefit wise, I think you are right17:45
ildikovmriedem: ok, I got that part, will try to annoy others as well then and we will see17:45
ildikovjohnthetubaguy: mriedem: somehow it would be great to sync efforts17:48
ildikovjohnthetubaguy: mriedem: but for now, could this land in the RC period as well or we're talking about it gets delayed until Ocata if does not land now?17:49
mriedemit would certainly not land for RC17:49
mriedemRC is like, stop ship bugs / regressions / upgrade issues17:49
mriedembut RC is october17:49
mriedemi think you mean sept17:49
mriedemidk honestly right now, i haven't put the time into reviewing it in real depth17:50
mriedemwe have a lot of other stuff going on right now17:50
mriedemtrying to close out before FF17:50
ildikovI meant the bug fixing period before the release17:51
ildikovit would be great to land the mox --> mock test changes at least as that's the bigger chunk and should not make any harm, the remove check_attach patch tests are stable with those changes17:53
ildikovbut I'll see if I can get others to check that part17:53
mriedemyeah that's maybe doable17:54
ildikovok, thanks17:55
ildikovI don't have anything else for today17:55
ildikovanything else form anyone?17:55
ildikovoh, one other thing17:56
ildikovI'm out next Monday17:56
ildikovwould there be anyone to run the meeting or let's skip?17:57
ildikovjgriffith: scottda hemna mriedem johnthetubaguy ^^17:58
mriedemskip17:59
mriedemi won't be there either way17:59
ildikovmriedem: ok, let's skip then, I'll send out a mail about it18:00
ildikovthank you all!18:01
ildikov#endmeeting18:01
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"18:01
openstackMeeting ended Mon Aug 15 18:01:09 2016 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)18:01
openstackMinutes:        http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2016/cinder_nova_api_changes.2016-08-15-17.00.html18:01
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2016/cinder_nova_api_changes.2016-08-15-17.00.txt18:01
openstackLog:            http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2016/cinder_nova_api_changes.2016-08-15-17.00.log.html18:01
*** mriedem has left #openstack-meeting-cp18:11
*** bknudson has joined #openstack-meeting-cp18:21
*** xyang1 has quit IRC18:22
*** xyang1 has joined #openstack-meeting-cp18:26
*** itisha has joined #openstack-meeting-cp18:58
*** tyr has joined #openstack-meeting-cp19:02
*** tyr has quit IRC19:03
*** tyr has joined #openstack-meeting-cp19:32
*** prateek has joined #openstack-meeting-cp19:53
*** sigmavirus is now known as sigmavirus|away20:03
*** tyr has quit IRC20:04
*** xyang1 has quit IRC20:29
*** xyang1 has joined #openstack-meeting-cp20:29
*** sdake has quit IRC20:32
*** gouthamr has quit IRC20:51
*** sdake has joined #openstack-meeting-cp20:57
*** sdake has quit IRC21:08
*** sdake has joined #openstack-meeting-cp21:17
*** tyr has joined #openstack-meeting-cp21:24
*** prateek has quit IRC21:29
*** sdake has quit IRC21:39
*** sdake has joined #openstack-meeting-cp21:54
*** harlowja has quit IRC21:55
*** xarses has joined #openstack-meeting-cp22:01
*** bswartz has quit IRC22:15
*** sdake has quit IRC22:20
*** sdake has joined #openstack-meeting-cp22:35
*** uxdanielle has quit IRC22:36
*** sdake has quit IRC22:57
*** markvoelker has quit IRC23:06
*** Benj_ has quit IRC23:07
*** xyang1 has quit IRC23:08
*** tyr has quit IRC23:11
*** xarses has quit IRC23:22
*** xarses has joined #openstack-meeting-cp23:34
*** harlowja has joined #openstack-meeting-cp23:36
*** sdague has quit IRC23:44

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!