| *** diablo_rojo has joined #openstack-meeting-cp | 00:39 | |
| *** diablo_rojo has quit IRC | 01:09 | |
| *** yamahata_ has quit IRC | 02:01 | |
| *** aselius has quit IRC | 02:29 | |
| *** pewp has quit IRC | 03:00 | |
| *** pewp has joined #openstack-meeting-cp | 03:02 | |
| *** markvoelker has quit IRC | 03:21 | |
| *** markvoelker has joined #openstack-meeting-cp | 03:21 | |
| *** edmondsw has quit IRC | 03:22 | |
| *** markvoelker has quit IRC | 03:26 | |
| *** diablo_rojo has joined #openstack-meeting-cp | 03:34 | |
| *** iyamahat has joined #openstack-meeting-cp | 03:35 | |
| *** iyamahat has quit IRC | 03:36 | |
| *** iyamahat has joined #openstack-meeting-cp | 03:38 | |
| *** gouthamr has quit IRC | 03:39 | |
| *** iyamahat_ has joined #openstack-meeting-cp | 03:42 | |
| *** aselius has joined #openstack-meeting-cp | 03:45 | |
| *** iyamahat has quit IRC | 03:45 | |
| *** iyamahat has joined #openstack-meeting-cp | 03:46 | |
| *** iyamahat_ has quit IRC | 03:50 | |
| *** iyamahat_ has joined #openstack-meeting-cp | 03:54 | |
| *** iyamahat has quit IRC | 03:57 | |
| *** yamahata_ has joined #openstack-meeting-cp | 03:59 | |
| *** iyamahat__ has joined #openstack-meeting-cp | 04:22 | |
| *** iyamahat_ has quit IRC | 04:25 | |
| *** iyamahat_ has joined #openstack-meeting-cp | 04:27 | |
| *** iyamahat__ has quit IRC | 04:30 | |
| *** iyamahat_ has quit IRC | 04:40 | |
| *** iyamahat has joined #openstack-meeting-cp | 04:41 | |
| *** iyamahat has quit IRC | 04:45 | |
| *** iyamahat has joined #openstack-meeting-cp | 05:20 | |
| *** markvoelker has joined #openstack-meeting-cp | 05:28 | |
| *** markvoelker has quit IRC | 05:40 | |
| *** markvoelker_ has joined #openstack-meeting-cp | 05:44 | |
| *** markvoelker_ has quit IRC | 05:52 | |
| *** aselius has quit IRC | 05:55 | |
| *** diablo_rojo has quit IRC | 06:00 | |
| *** markvoelker has joined #openstack-meeting-cp | 06:40 | |
| *** markvoelker has quit IRC | 06:47 | |
| *** brault has joined #openstack-meeting-cp | 06:56 | |
| *** markvoelker has joined #openstack-meeting-cp | 06:59 | |
| *** markvoelker has quit IRC | 07:04 | |
| *** markvoelker has joined #openstack-meeting-cp | 07:08 | |
| *** markvoelker has quit IRC | 07:13 | |
| *** f13o has joined #openstack-meeting-cp | 07:14 | |
| *** markvoelker has joined #openstack-meeting-cp | 07:21 | |
| *** markvoelker has quit IRC | 07:29 | |
| *** markvoelker has joined #openstack-meeting-cp | 08:17 | |
| *** benj_ has quit IRC | 08:18 | |
| *** benj_ has joined #openstack-meeting-cp | 08:20 | |
| *** markvoelker has quit IRC | 08:22 | |
| *** benj_ has quit IRC | 08:25 | |
| *** benj_ has joined #openstack-meeting-cp | 08:33 | |
| *** yamahata_ has quit IRC | 08:34 | |
| *** markvoelker has joined #openstack-meeting-cp | 08:36 | |
| *** iyamahat has quit IRC | 08:37 | |
| *** markvoelker has quit IRC | 08:42 | |
| *** f13o has quit IRC | 08:47 | |
| *** markvoelker has joined #openstack-meeting-cp | 09:00 | |
| *** f13o has joined #openstack-meeting-cp | 09:02 | |
| *** markvoelker has quit IRC | 09:05 | |
| *** sdague has joined #openstack-meeting-cp | 09:44 | |
| *** f13o has quit IRC | 10:13 | |
| *** f13o has joined #openstack-meeting-cp | 10:28 | |
| *** markvoelker has joined #openstack-meeting-cp | 10:36 | |
| *** markvoelker has quit IRC | 10:41 | |
| *** markvoelker has joined #openstack-meeting-cp | 10:47 | |
| *** markvoelker has quit IRC | 10:55 | |
| *** edmondsw has joined #openstack-meeting-cp | 12:10 | |
| *** edmondsw has quit IRC | 12:38 | |
| *** f13o has quit IRC | 13:04 | |
| *** edmondsw has joined #openstack-meeting-cp | 13:11 | |
| *** stvnoyes has joined #openstack-meeting-cp | 13:18 | |
| *** f13o has joined #openstack-meeting-cp | 13:18 | |
| *** markvoelker has joined #openstack-meeting-cp | 13:22 | |
| *** gouthamr has joined #openstack-meeting-cp | 13:34 | |
| *** gouthamr has quit IRC | 13:47 | |
| *** diablo_rojo has joined #openstack-meeting-cp | 13:51 | |
| *** gouthamr has joined #openstack-meeting-cp | 13:52 | |
| *** markvoelker has quit IRC | 13:53 | |
| *** felipemonteiro has joined #openstack-meeting-cp | 14:00 | |
| *** felipemonteiro_ has joined #openstack-meeting-cp | 14:02 | |
| *** felipemonteiro has quit IRC | 14:06 | |
| *** markvoelker has joined #openstack-meeting-cp | 14:55 | |
| *** aselius has joined #openstack-meeting-cp | 15:00 | |
| *** iyamahat has joined #openstack-meeting-cp | 15:10 | |
| *** yamahata has joined #openstack-meeting-cp | 15:10 | |
| *** f13o has quit IRC | 15:19 | |
| *** mriedem has joined #openstack-meeting-cp | 15:31 | |
| ildikov | #startmeeting cinder-nova-api-changes | 16:00 |
|---|---|---|
| openstack | Meeting started Thu Jun 22 16:00:08 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 |
| ildikov | johnthetubaguy jaypipes e0ne jgriffith hemna mriedem patrickeast smcginnis diablo_rojo xyang1 raj_singh lyarwood jungleboyj stvnoyes | 16:00 |
| mriedem | o | 16:00 |
| mriedem | o/ i mean | 16:00 |
| stvnoyes | o/ | 16:00 |
| ildikov | :) | 16:00 |
| ildikov | we have a couple of Cinder guys out | 16:01 |
| ildikov | let's wait a minute to see whether we have more people to show up today | 16:01 |
| ildikov | ok, let's start and go through things quickly | 16:02 |
| * jgriffith sneaks in and hides in the back | 16:02 | |
| ildikov | stvnoyes: did you see Matt's comments on the live_migrate patch? | 16:02 |
| stvnoyes | yes, I have most of the updates done, working on the tests right now | 16:03 |
| ildikov | jgriffith: I can see you there too :) | 16:03 |
| jgriffith | so much for hiding :) | 16:03 |
| ildikov | stvnoyes: coolio | 16:03 |
| ildikov | stvnoyes: anything we should talk about here? | 16:03 |
| ildikov | stvnoyes: I updated the attach PoC to address your comments | 16:03 |
| ildikov | stvnoyes: playing with the tests, I'll have a few more rounds with that... | 16:04 |
| stvnoyes | is anyone working on this - https://bugs.launchpad.net/cinder/+bug/1692153 | 16:04 |
| openstack | Launchpad bug 1692153 in Cinder "v3 attachment connection_info formatting is different than v2" [High,New] | 16:04 |
| ildikov | stvnoyes: I didn't add the target_lun fix as it's fixed in brick and should be fix long term in Cinder | 16:04 |
| stvnoyes | that's fine, that's why i put it in as just an fyi | 16:04 |
| ildikov | jgriffith: do you or anyone in your knowledge working on that one? | 16:04 |
| jgriffith | not to my knowledge no | 16:05 |
| jgriffith | we should check with hemna | 16:05 |
| stvnoyes | i need to check what version of brick I'm running. any idea what version it's fixed in? | 16:06 |
| ildikov | I don't think brick is released yet as they wanted a few more things in last time | 16:06 |
| ildikov | I'll ask hemna | 16:06 |
| ildikov | if it's not released yet you can test by installing it from source | 16:06 |
| jgriffith | stvnoyes I don't believe it's fixed yet anyway... unless oslo object updated | 16:06 |
| jgriffith | anyway | 16:06 |
| stvnoyes | ok thx, also how about this bug - https://bugs.launchpad.net/cinder/+bug/1697526 | 16:07 |
| openstack | Launchpad bug 1697526 in Cinder "attachment_create fails on live migration with bfv instance" [Undecided,New] | 16:07 |
| ildikov | jgriffith: hemna added a quick fix in brick to address the target_lun issue and cast it to int | 16:07 |
| jgriffith | stvnoyes I can take that | 16:07 |
| stvnoyes | ok that would be great thx. | 16:08 |
| jgriffith | we'll need to figure out how we want to address the whole "multi-attach, but not really multi-attach" thing for lm | 16:08 |
| stvnoyes | i also updated this cinder review - https://review.openstack.org/#/c/472796/ - let me know if this makes more sense | 16:09 |
| stvnoyes | jgriffith - related to what you just said | 16:09 |
| ildikov | stvnoyes: I added a comment to that one recently | 16:09 |
| ildikov | jgriffith: you mean to allow creating multiple attachments to the same volume regardless they are multi-attach enabled or not? | 16:10 |
| jgriffith | ildikov that's the weird case of live-migration | 16:10 |
| jgriffith | it's actually doing two attachments, because Nova is afraid to get rid of one until the new one is made | 16:11 |
| ildikov | jgriffith: I think we said something like same instance, but another host should do the trick | 16:11 |
| jgriffith | ildikov yeah.. you mean inspecting that on the attachment-create and let it slide right? | 16:12 |
| ildikov | live_migrate is not trivial anyway because of the switch over :/ | 16:12 |
| stvnoyes | btw, that review should probably be taken over by someone on the cinder team. it might go faster that way. (https://review.openstack.org/#/c/472796/) | 16:12 |
| jgriffith | nothing is trivial anymore it seems :) | 16:12 |
| stvnoyes | i'm happy to pass it on! :-) | 16:13 |
| ildikov | stvnoyes: I planned to ping people on the channel later to figure out the strategy and fix it for lvm now | 16:13 |
| stvnoyes | ok great | 16:13 |
| ildikov | stvnoyes: :) | 16:13 |
| ildikov | jgriffith: has anything ever been? :) | 16:13 |
| jgriffith | :) | 16:14 |
| ildikov | jgriffith: so attachment_create gets the volume and the instance ids | 16:14 |
| stvnoyes | and then there's this bug - https://bugs.launchpad.net/cinder/+bug/1697526 - which is a strange one | 16:14 |
| openstack | Launchpad bug 1697526 in Cinder "attachment_create fails on live migration with bfv instance" [Undecided,New] | 16:14 |
| stvnoyes | nope. not that one | 16:15 |
| jgriffith | deja-vous | 16:15 |
| stvnoyes | this one - https://bugs.launchpad.net/cinder/+bug/1697775 | 16:15 |
| openstack | Launchpad bug 1697775 in Cinder "cinder v3 api: show hides some attachments" [Undecided,New] - Assigned to NidhiMittalHada (nidhimittal19) | 16:15 |
| stvnoyes | i guess it could be related to the cinder v3 work, but it doesn't seem like it | 16:16 |
| jgriffith | ildikov so the trick is it would also need the connector, unless we just decide same instance is safe | 16:16 |
| jgriffith | stvnoyes I think I know what's going on there btw | 16:16 |
| ildikov | jgriffith: I'm unsure whether we only need to reserve the volume or we attach it too before detaching it from the source host | 16:17 |
| jgriffith | stvnoyes did you by chance have a mix of attachments (ie created some with the old flow and some with the new)? | 16:17 |
| stvnoyes | no, i don't think so. all with the new flow. | 16:17 |
| jgriffith | The other thing is keep in mind it only shows active attachments in the volume-show | 16:18 |
| jgriffith | the one at least was "reserved" and not "attached" | 16:18 |
| jgriffith | but we can look and verify if that's what's going on or not | 16:18 |
| stvnoyes | it causes problems with the attachment_delete which is looking at attach count. | 16:18 |
| stvnoyes | so while you think there is only one attachment, in reality there are several | 16:19 |
| jgriffith | well... | 16:19 |
| * jgriffith mutters to himself | 16:19 | |
| stvnoyes | so having 'show volume' show them all is important | 16:19 |
| ildikov | can we have multiple attachments now? | 16:19 |
| jgriffith | the whole point was to NOT do counting, but if we can't get the shared connection folks to "do something" I guess we're right back where we started | 16:20 |
| stvnoyes | this is during migration, i noticed that at the end of the migration, the volume was not detaching | 16:20 |
| ildikov | jgriffith: couldn't that work for lvm though? | 16:20 |
| jgriffith | ildikov counting? | 16:20 |
| ildikov | jgriffith: as it's problematic for lvm as well in my understanding | 16:20 |
| ildikov | jgriffith: yes | 16:20 |
| jgriffith | meh | 16:20 |
| ildikov | or if not counting then what would work for that one? | 16:21 |
| jgriffith | Let's move on, I'll see if I can figure out what's up with the counts | 16:21 |
| jgriffith | ildikov there's a difference between going to the attachments table and trying to rely on what the volume-object says/thinks | 16:21 |
| ildikov | as lvm is the reference implementation we should fix it there and then tell the drivers to implement it how it's suitable for their stuff | 16:21 |
| jgriffith | and it only matters when it comes time to delete a target | 16:21 |
| jgriffith | in all other cases we shouldn't be messing with the calculation of things any more | 16:22 |
| jgriffith | ildikov +1 | 16:22 |
| ildikov | but when you detach right now it's basically deleting the target as well IIUC | 16:22 |
| jgriffith | ildikov it's calling terminate, and that's where the whole topic of "drivers figure out what they need to do" came in | 16:23 |
| jgriffith | anyway... I'll look at it | 16:23 |
| ildikov | yeah, that's where I thought to fix it | 16:23 |
| ildikov | anyway, let's move on | 16:24 |
| ildikov | I added a few tests to swap_volume | 16:24 |
| ildikov | mriedem: if you want to add any comments to that patch before wonderful China please do so :) | 16:25 |
| mriedem | probably won't happen | 16:25 |
| ildikov | jgriffith: is the Cinder side of that thing considered to be ready for review? | 16:25 |
| ildikov | jgriffith: I mean annoying Cinder people to review :) | 16:26 |
| ildikov | mriedem: thought so :( | 16:26 |
| ildikov | mriedem: but wanted to give it a try :) | 16:26 |
| jgriffith | You mean stvnoyes patch on the the terminate connection? | 16:26 |
| ildikov | jgriffith: no, the migrate_volume_completion stuff | 16:26 |
| jgriffith | ildikov yes | 16:26 |
| ildikov | this one: https://review.openstack.org/#/c/472786/ | 16:26 |
| ildikov | ok, cool, I'll add it to my annoyment-list | 16:27 |
| jgriffith | haha | 16:27 |
| ildikov | mriedem: are off for only a week? | 16:28 |
| ildikov | mriedem: also who should I annoy with the Nova side reviews besides you? | 16:28 |
| mriedem | ildikov: yeah 1 week | 16:28 |
| mriedem | but then it's the 4th of july US holiday after i get back | 16:28 |
| mriedem | so the week after is going to be weird | 16:28 |
| mriedem | ildikov: idk honestly, johnthetubaguy is still out | 16:28 |
| mriedem | the other cores are busy with their own dev things | 16:29 |
| ildikov | are you saying we're missing Pike? | 16:29 |
| mriedem | nope | 16:29 |
| ildikov | as we have roughly 4 weeks | 16:29 |
| mriedem | yup | 16:30 |
| mriedem | we have really 3 changes to get into nova right? | 16:30 |
| ildikov | and if we need one more core who kind of needs on-boarding that'll take time | 16:30 |
| mriedem | live migration, swap volume, and the attach change at the end | 16:30 |
| ildikov | yep, I believe so | 16:30 |
| ildikov | I'm playing with the attach patch now | 16:30 |
| ildikov | swap is sort of ready for review | 16:30 |
| jgriffith | swap should be pretty much ready | 16:30 |
| mriedem | dan, jay, mel and sylvain are busy with scheduler and cells v2 stuff | 16:30 |
| jgriffith | I'll look at LM | 16:31 |
| ildikov | and live_migrate will be again soon | 16:31 |
| jgriffith | kinda thought that was pretty close | 16:31 |
| stvnoyes | is the Poc going to be the change for attach, or is that going to be new work? that will need a lot of work on the test side. I can start writing tests for attach... | 16:31 |
| mriedem | stvnoyes: it's the poc | 16:31 |
| jgriffith | stvnoyes that would be AWESOME!!! | 16:31 |
| mriedem | going to be that change i think | 16:31 |
| ildikov | mriedem: what about Stephen? | 16:31 |
| mriedem | ildikov: i don't know how familiar he is with these flows | 16:31 |
| mriedem | you can certainly ask | 16:32 |
| ildikov | stvnoyes: I'm fixing tests now to make them green, but I default a lot back to the old flow | 16:32 |
| ildikov | ok I'll ask him | 16:32 |
| mriedem | we also need a grenade change | 16:32 |
| ildikov | anyone else who might be able to chime in? | 16:32 |
| stvnoyes | ok, we can talk later about it, after I get migrate back up for rv | 16:32 |
| mriedem | to have a volume attached on the old side (ocata) go through an upgrade and detach it on the new side (pike) | 16:32 |
| mriedem | for the grenade thing, just talk to sdague | 16:32 |
| ildikov | stvnoyes: yep, let's chat about it as we can work separately on that | 16:32 |
| mriedem | shouldn't be too hard to add that | 16:32 |
| ildikov | yep, doesn't sound too bad except that I don't have that much clue about grenade, but it's never too late to learn :) | 16:33 |
| ildikov | I think mainly that's all from my side today | 16:34 |
| ildikov | while mriedem is out we can work on addressing things on the Cinder side | 16:35 |
| ildikov | and I'll see who I can harass for review in Nova | 16:35 |
| ildikov | anything else from anyone we should try to address here today? | 16:36 |
| ildikov | ok, I take it as a no :) | 16:37 |
| ildikov | mriedem: safe travels to China! | 16:37 |
| ildikov | catch up with y'all separately on the different items | 16:37 |
| ildikov | have a good day! | 16:38 |
| ildikov | #endmeeting | 16:39 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 16:39 | |
| openstack | Meeting ended Thu Jun 22 16:39:01 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:39 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-06-22-16.00.html | 16:39 |
| *** mriedem has left #openstack-meeting-cp | 16:39 | |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-06-22-16.00.txt | 16:39 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-06-22-16.00.log.html | 16:39 |
| jgriffith | thanks ildikov | 16:39 |
| *** iyamahat has quit IRC | 16:59 | |
| *** yamahata has quit IRC | 17:00 | |
| *** iyamahat has joined #openstack-meeting-cp | 17:24 | |
| *** yamahata has joined #openstack-meeting-cp | 17:41 | |
| *** diablo_rojo has quit IRC | 19:06 | |
| *** bswartz has quit IRC | 19:38 | |
| *** diablo_rojo has joined #openstack-meeting-cp | 20:35 | |
| *** dmellado has quit IRC | 21:00 | |
| *** dmellado has joined #openstack-meeting-cp | 21:03 | |
| *** edmondsw has quit IRC | 21:13 | |
| *** edmondsw has joined #openstack-meeting-cp | 21:16 | |
| *** edmondsw has quit IRC | 21:21 | |
| *** gouthamr has quit IRC | 21:55 | |
| *** iyamahat has quit IRC | 22:03 | |
| *** iyamahat has joined #openstack-meeting-cp | 22:03 | |
| *** gouthamr has joined #openstack-meeting-cp | 22:16 | |
| *** felipemonteiro_ has quit IRC | 22:26 | |
| *** iyamahat has quit IRC | 22:33 | |
| *** iyamahat has joined #openstack-meeting-cp | 22:33 | |
| *** dmellado has quit IRC | 22:33 | |
| *** dmellado has joined #openstack-meeting-cp | 22:37 | |
| *** edmondsw has joined #openstack-meeting-cp | 22:49 | |
| *** dmellado has quit IRC | 23:03 | |
| *** dmellado has joined #openstack-meeting-cp | 23:06 | |
| *** dmellado has quit IRC | 23:15 | |
| *** dmellado has joined #openstack-meeting-cp | 23:18 | |
| *** diablo_rojo has quit IRC | 23:23 | |
| *** dmellado has quit IRC | 23:31 | |
| *** dmellado has joined #openstack-meeting-cp | 23:51 | |
| *** dmellado has quit IRC | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!