| *** lamt has quit IRC | 00:06 | |
| *** dhellmann has quit IRC | 00:35 | |
| *** dhellmann has joined #openstack-meeting-cp | 00:36 | |
| *** david-lyle has joined #openstack-meeting-cp | 01:16 | |
| *** david-lyle has quit IRC | 01:34 | |
| *** markvoelker has quit IRC | 02:40 | |
| *** david-lyle has joined #openstack-meeting-cp | 02:53 | |
| *** homerp_ has joined #openstack-meeting-cp | 03:10 | |
| *** kbyrne_ has joined #openstack-meeting-cp | 03:10 | |
| *** homerp has quit IRC | 03:11 | |
| *** kbyrne has quit IRC | 03:11 | |
| *** kbyrne_ is now known as kbyrne | 03:11 | |
| *** knangia has joined #openstack-meeting-cp | 03:12 | |
| *** markvoelker has joined #openstack-meeting-cp | 03:44 | |
| *** gouthamr has quit IRC | 03:58 | |
| *** rarcea has joined #openstack-meeting-cp | 05:59 | |
| *** brault has quit IRC | 06:19 | |
| *** rarcea has quit IRC | 08:11 | |
| *** rarcea has joined #openstack-meeting-cp | 08:21 | |
| *** brault has joined #openstack-meeting-cp | 08:24 | |
| *** knangia has quit IRC | 08:43 | |
| *** knangia has joined #openstack-meeting-cp | 08:43 | |
| *** markvoelker has quit IRC | 09:25 | |
| *** rarcea has quit IRC | 09:34 | |
| *** rarcea has joined #openstack-meeting-cp | 09:35 | |
| *** sdague has joined #openstack-meeting-cp | 10:11 | |
| *** markvoelker has joined #openstack-meeting-cp | 10:26 | |
| *** markvoelker has quit IRC | 10:33 | |
| *** MarkBaker has joined #openstack-meeting-cp | 10:37 | |
| *** wxy has quit IRC | 10:48 | |
| *** brault_ has joined #openstack-meeting-cp | 10:48 | |
| *** wxy has joined #openstack-meeting-cp | 10:48 | |
| *** brault has quit IRC | 10:49 | |
| *** lamt has joined #openstack-meeting-cp | 11:17 | |
| *** lamt has quit IRC | 11:21 | |
| *** MarkBaker has quit IRC | 11:58 | |
| *** lamt has joined #openstack-meeting-cp | 12:32 | |
| *** brault has joined #openstack-meeting-cp | 12:56 | |
| *** brault_ has quit IRC | 12:57 | |
| *** markvoelker has joined #openstack-meeting-cp | 13:23 | |
| *** gouthamr has joined #openstack-meeting-cp | 13:38 | |
| *** jaugustine has joined #openstack-meeting-cp | 14:01 | |
| *** brault has quit IRC | 14:06 | |
| *** markvoelker has quit IRC | 14:31 | |
| *** stvnoyes has quit IRC | 14:38 | |
| *** stvnoyes has joined #openstack-meeting-cp | 14:40 | |
| *** lamt has quit IRC | 14:43 | |
| *** markvoelker has joined #openstack-meeting-cp | 14:53 | |
| *** brault has joined #openstack-meeting-cp | 15:19 | |
| *** stvnoyes has quit IRC | 15:22 | |
| *** stvnoyes has joined #openstack-meeting-cp | 15:23 | |
| *** brault has quit IRC | 15:24 | |
| *** jaugustine has quit IRC | 15:40 | |
| *** jaugustine has joined #openstack-meeting-cp | 15:50 | |
| *** mriedem has joined #openstack-meeting-cp | 15:56 | |
| ildikov | #startmeeting cinder-nova-api-changes | 16:00 |
|---|---|---|
| openstack | Meeting started Thu May 4 16:00:01 2017 UTC and is due to finish in 60 minutes. The chair is ildikov. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| *** openstack changes topic to " (Meeting topic: cinder-nova-api-changes)" | 16:00 | |
| openstack | The meeting name has been set to 'cinder_nova_api_changes' | 16:00 |
| mriedem | o/ | 16:00 |
| jungleboyj | o/ | 16:00 |
| ildikov | DuncanT ameade cFouts johnthetubaguy jaypipes takashin alaski e0ne jgriffith tbarron andrearosa hemna erlon mriedem gouthamr ebalduf patrickeast smcginnis diablo_rojo gsilvis xyang1 raj_singh lyarwood breitz jungleboyj | 16:00 |
| stvnoyes | o/ | 16:00 |
| hemna | \0 | 16:00 |
| hemna | @! | 16:00 |
| pewp | hemna (╯°□°)╯︵ ┻━┻ | 16:00 |
| ildikov | hemna: already? :) | 16:00 |
| hemna | yah, why not :P | 16:00 |
| jungleboyj | @!h | 16:00 |
| pewp | jungleboyj (/ .□.) ︵╰(゜Д゜)╯︵ /(.□. ) | 16:00 |
| jungleboyj | It is a good way to start any meeting. | 16:01 |
| ildikov | pre-Summit meeting, I get it... :) | 16:01 |
| ildikov | ok, let's start then | 16:01 |
| ildikov | I see mriedem added a few comments to shutdown_instance | 16:02 |
| *** mgiles has joined #openstack-meeting-cp | 16:02 | |
| stvnoyes | saw those, starting on that | 16:02 |
| ildikov | I think local_cleanup is also ready for review: https://review.openstack.org/#/c/456851/ | 16:02 |
| ildikov | stvnoyes: am I right? ^ | 16:03 |
| stvnoyes | yes | 16:03 |
| ildikov | mriedem: so please add comments to that one too ^^ :) | 16:04 |
| mriedem | already doing it | 16:04 |
| ildikov | we started to discuss migrate_volume_completion with the Cinder team and in my understanding to conclusion is to look into how to use the new calls with it | 16:04 |
| ildikov | so we plan to keep the same call back method concept and have it covered in the new API | 16:05 |
| ildikov | at least my expectation is that it's more clear that way, but maybe we can update the existing call easily | 16:06 |
| ildikov | e0ne will help looking into the current version and figure out the new API (compatible) solution | 16:06 |
| ildikov | so in this sense swap_volume in Nova is on hold currently until the Cinder side is done/fixed | 16:07 |
| ildikov | in the meantime stvnoyes is looking into live migrate to see what the challenges are there | 16:07 |
| mriedem | so what do you mean by the 'new' api in cinder? | 16:08 |
| ildikov | mriedem: the new attach/detach API | 16:08 |
| mriedem | because if migrate_volume_completion doesn't handle this for 3.27, then cinder needs a new microversion for that | 16:08 |
| smcginnis | mriedem: We still need to determine if we need a new API call or not. | 16:08 |
| mriedem | ok | 16:09 |
| mriedem | like, nova might not need to call migrate_volume_completion at all? | 16:09 |
| mriedem | or you plan on doing something in 3.27 maybe to handle it if it's a new-style attachment? | 16:09 |
| mriedem | like as a bug fix? | 16:09 |
| smcginnis | Bug fix. | 16:09 |
| ildikov | is there an option that does not need new microversion? | 16:10 |
| ildikov | just for my understanding :) | 16:10 |
| smcginnis | ildikov: If we can make the current call handle the new attachment correctly, then we don't need a microversion bump. | 16:11 |
| smcginnis | But if we need to add a new API, or somehow change the current one, then that will need it. | 16:11 |
| hemna | smcginnis, +1 | 16:11 |
| smcginnis | But that's the part I am not clear on - which of those we need to do. | 16:11 |
| smcginnis | Down side being with NOT doing a microversion bump, there will be no way externally to know if the version of cinder present can handle this or not. | 16:12 |
| ildikov | smcginnis: I just wanted to ask whether that's a bug fix to backport | 16:13 |
| smcginnis | ildikov: If we can fix the current method, then definitely we will get that backported. | 16:13 |
| * johnthetubaguy wonders in late and hides at the back | 16:14 | |
| ildikov | smcginnis: ok, cool, I think that was my concern | 16:14 |
| ildikov | johnthetubaguy: hi :) | 16:14 |
| ildikov | johnthetubaguy: we discussed swap_volume and migrate_volume_completion briefly | 16:15 |
| johnthetubaguy | yeah, just scrolling back | 16:15 |
| ildikov | johnthetubaguy: let us know if you have further questions or comments | 16:15 |
| johnthetubaguy | smcginnis: do a microversion bump to just signal the new stuff is there? | 16:15 |
| smcginnis | johnthetubaguy: We could. But then we can't really backport that. | 16:16 |
| johnthetubaguy | downside is we probably would then no want to trigger the new flow until that microversion is there | 16:16 |
| smcginnis | +1 | 16:16 |
| johnthetubaguy | yeah, I guess we just said the same thing in two different ways | 16:16 |
| smcginnis | :) | 16:16 |
| johnthetubaguy | why would we need/want to backport it? | 16:16 |
| smcginnis | If we can do it as a bug fix and get it backported, I don't think we would have too big of a window until it's out there. | 16:17 |
| ildikov | if we do a version bump, then we need to check that higher version when we add attach | 16:17 |
| johnthetubaguy | ildikov: yep | 16:17 |
| smcginnis | Just to allow Ocata Cinder with Pike Nova. But then again, maybe we don't care and just make it the new required level. | 16:17 |
| ildikov | or we do what smcginnis says | 16:17 |
| johnthetubaguy | the silent backport sounds way too much like cheating in a way we will break things | 16:18 |
| johnthetubaguy | smcginnis: so Nova just uses the old flow if cinder is too old, so its OK I think | 16:18 |
| johnthetubaguy | I mean, its not idea... but life sucks sometimes | 16:18 |
| smcginnis | johnthetubaguy: Yeah. Was just hoping we had all the pieces in place to not have to do that. | 16:18 |
| smcginnis | But maybe not. | 16:18 |
| johnthetubaguy | so cleanest option is a new API we pass the old and new attachment ids too? | 16:18 |
| smcginnis | First we need to figure out this code and understand what we actually need to do. | 16:19 |
| smcginnis | I have not touched that, and most who have recently are gone. | 16:19 |
| mriedem | yeah before we rathole, | 16:19 |
| smcginnis | So we just need to refresh our knowledge a bit. | 16:19 |
| mriedem | let's figure out what actually needs to happen on the cinder side for migrate_volume_completion | 16:19 |
| * johnthetubaguy things mriedem saw we typing | 16:19 | |
| johnthetubaguy | thinks^ | 16:19 |
| smcginnis | mriedem: +1 | 16:19 |
| johnthetubaguy | me^ | 16:19 |
| mriedem | if it's a request body change or something, it's definitely microversion bump | 16:19 |
| ildikov | e0ne is not around, but offered to help by having some knowledge about the current code | 16:19 |
| mriedem | so until we sort that out, let's move on | 16:19 |
| smcginnis | +1 | 16:20 |
| johnthetubaguy | so migrate_volume_completion let get clear on what it does | 16:20 |
| ildikov | yep, that's the plan | 16:20 |
| mriedem | also note that we should be able to attach/detach with 3.27, | 16:20 |
| mriedem | but maybe not swap volume until a later microversion or something; nova as client code can determine that | 16:20 |
| johnthetubaguy | I think... it makes volume B look like volume A, we currently pass in A and B, but that only happens if the user called cinder to migrate volume A | 16:20 |
| johnthetubaguy | if the user called nova to swap A and B, non that really happens, I am not sure if the cinder call actually does anything in that case | 16:21 |
| johnthetubaguy | mriedem: I kinda like the one high water mark, if I am honest, makes life less complicated (please upgrade to make swap volume work with your newly attached volumes, etc) | 16:22 |
| smcginnis | Certainly safer. | 16:22 |
| mriedem | sure, just stating options | 16:22 |
| mriedem | but let's move on - todo is on cinder to figure out what needs to happen | 16:22 |
| ildikov | mriedem: so that means if a volume is attached in the new way then you cannot swap that if you don't have the high enough microversion, if we go down that path | 16:22 |
| ildikov | #action Cinder team to figure out how to handle migrate_volume_completion on the Cinder side | 16:24 |
| jungleboyj | smcginnis: ildikov We are planning to get together with e0ne to talk about this next week. Right? | 16:24 |
| ildikov | I'm ready to move on | 16:24 |
| smcginnis | Let's move on. | 16:24 |
| ildikov | so as jungleboyj brought it up | 16:24 |
| smcginnis | jungleboyj: Yes | 16:24 |
| jungleboyj | :-) | 16:24 |
| ildikov | do we want a small sub-team gathering next week? | 16:24 |
| johnthetubaguy | at this point I have free lunch times and break times | 16:25 |
| ildikov | like mriedem johnthetubaguy would you be interested to join? | 16:25 |
| mriedem | my schedule is full | 16:25 |
| smcginnis | johnthetubaguy: There's always between 11pm and 6am. :) | 16:25 |
| johnthetubaguy | there is the Nova and Cinder forum session, we could get onto this if we "have spare time" | 16:26 |
| mriedem | smcginnis: you and i are in customer meetings at those times | 16:26 |
| smcginnis | mriedem: Sadly true | 16:26 |
| mriedem | there isn't a nova/cinder forum session, | 16:26 |
| mriedem | the one you're thinking of is specifically about using cinder for ephemeral | 16:26 |
| mriedem | and volume affinity | 16:26 |
| johnthetubaguy | yeah, thats the one | 16:26 |
| mriedem | those aren't this though | 16:26 |
| johnthetubaguy | oh, so thats like 2 hours of material already | 16:26 |
| johnthetubaguy | yeah, no spare time there | 16:26 |
| ildikov | yeah, we could reserve a slot in a hacking room | 16:27 |
| jungleboyj | :-( | 16:27 |
| mriedem | there are spill over rooms on thursday afternoon, | 16:27 |
| smcginnis | Maybe let's just try to get a hacking room time reserved, then whoever can make it can make it. | 16:27 |
| mriedem | ok | 16:27 |
| ildikov | which I hope we can do with a few Cinder guys | 16:27 |
| mriedem | i'm totally happy if the cinder team can get together and figure out all of the problems | 16:28 |
| ildikov | smcginnis: yep, I will sync up with e0ne and try to figure out when (if) jgriffith is available | 16:28 |
| ildikov | mriedem: :) | 16:28 |
| smcginnis | +1 | 16:28 |
| jungleboyj | :-) | 16:28 |
| ildikov | ok, we can take finding a slot offline | 16:29 |
| ildikov | stvnoyes: is there anything for live migrate to talk about here today? | 16:29 |
| ildikov | I take it as a no :) | 16:31 |
| ildikov | I don't have additional topics for today, I don't think it would worth going into other things before get swap and migrate figured out | 16:31 |
| ildikov | obviously no meeting next week | 16:31 |
| mriedem | i'm +2 on the local delete patch, | 16:31 |
| mriedem | and the shutdown_instance one should be simple to address my -1 | 16:31 |
| mriedem | just decouple that refactor | 16:32 |
| ildikov | mriedem: I saw, thank you | 16:32 |
| ildikov | johnthetubaguy: if you have a minute plz check the local_cleanup patch | 16:32 |
| johnthetubaguy | I will try look at that | 16:32 |
| ildikov | johnthetubaguy: it's a pretty small one at least | 16:32 |
| ildikov | johnthetubaguy: thank you | 16:32 |
| mriedem | i can update the review priorities etherpad too | 16:32 |
| ildikov | mriedem: sounds good, thanks | 16:33 |
| ildikov | is it ok to have the next meeting the week after the summit? | 16:33 |
| smcginnis | ildikov: Works for me. | 16:33 |
| mriedem | sure | 16:34 |
| jungleboyj | ildikov: Yep. | 16:34 |
| ildikov | cool, then I'll send out a mail to the ML that next week is cancelled and hopefully we will have some meet for swap to discuss two weeks from today | 16:34 |
| ildikov | anything else from anyone for today? | 16:35 |
| mriedem | oh btw | 16:35 |
| mriedem | https://review.openstack.org/#/c/459113/ was updated | 16:35 |
| mriedem | that's the attachment_update method | 16:35 |
| mriedem | so should be good for review again | 16:35 |
| *** wxy has quit IRC | 16:35 | |
| *** knangia has quit IRC | 16:35 | |
| ildikov | mriedem: awesome, thank you! | 16:36 |
| ildikov | mriedem: I'll check after the meeting | 16:36 |
| johnthetubaguy | yeah, me too | 16:36 |
| ildikov | if nothing else, then thanks everyone for today | 16:37 |
| ildikov | safe travels! | 16:37 |
| smcginnis | See you all in Boston. | 16:37 |
| ildikov | and see you next week! :) | 16:37 |
| jungleboyj | See you next week. Safe travels! | 16:37 |
| ildikov | literally :) | 16:37 |
| johnthetubaguy | +1 | 16:37 |
| ildikov | #endmeeting | 16:38 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 16:38 | |
| openstack | Meeting ended Thu May 4 16:38:43 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:38 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-05-04-16.00.html | 16:38 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-05-04-16.00.txt | 16:38 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-05-04-16.00.log.html | 16:38 |
| *** mgiles has quit IRC | 16:38 | |
| *** mriedem has left #openstack-meeting-cp | 16:40 | |
| *** stvnoyes has quit IRC | 16:45 | |
| *** wxy has joined #openstack-meeting-cp | 16:46 | |
| *** knangia has joined #openstack-meeting-cp | 16:46 | |
| *** stvnoyes has joined #openstack-meeting-cp | 16:59 | |
| *** rarcea has quit IRC | 17:26 | |
| *** harlowja has quit IRC | 18:19 | |
| *** Guest92102 is now known as redrobot | 18:42 | |
| *** gouthamr has quit IRC | 19:14 | |
| *** gouthamr has joined #openstack-meeting-cp | 19:24 | |
| *** brault has joined #openstack-meeting-cp | 19:40 | |
| *** brault_ has joined #openstack-meeting-cp | 19:45 | |
| *** harlowja has joined #openstack-meeting-cp | 19:48 | |
| *** brault has quit IRC | 19:48 | |
| *** lbragstad_alt has joined #openstack-meeting-cp | 20:02 | |
| *** gouthamr has quit IRC | 20:56 | |
| *** brault_ has quit IRC | 21:03 | |
| *** gouthamr has joined #openstack-meeting-cp | 21:26 | |
| *** lbragstad has quit IRC | 21:45 | |
| *** lamt has joined #openstack-meeting-cp | 22:15 | |
| *** lamt has quit IRC | 22:27 | |
| *** jaugustine has quit IRC | 22:36 | |
| *** gouthamr has quit IRC | 22:55 | |
| *** brault has joined #openstack-meeting-cp | 23:04 | |
| *** brault has quit IRC | 23:09 | |
| *** sdague has quit IRC | 23:20 | |
| *** lamt has joined #openstack-meeting-cp | 23:27 | |
| *** lamt has quit IRC | 23:39 | |
| *** lamt has joined #openstack-meeting-cp | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!