| *** yamahata has joined #openstack-meeting-cp | 01:33 | |
| *** yamahata has quit IRC | 01:55 | |
| *** aselius has quit IRC | 02:18 | |
| *** felipemonteiro_ has joined #openstack-meeting-cp | 02:20 | |
| *** zerick has quit IRC | 02:26 | |
| *** zerick has joined #openstack-meeting-cp | 02:27 | |
| *** zerick has quit IRC | 02:43 | |
| *** zerick has joined #openstack-meeting-cp | 02:43 | |
| *** nhelgeson has quit IRC | 03:03 | |
| *** gouthamr has quit IRC | 03:25 | |
| *** iyamahat has joined #openstack-meeting-cp | 03:54 | |
| *** felipemonteiro_ has quit IRC | 04:01 | |
| *** markvoelker has quit IRC | 04:23 | |
| *** markvoelker has joined #openstack-meeting-cp | 04:55 | |
| *** yamahata has joined #openstack-meeting-cp | 05:12 | |
| *** markvoelker has quit IRC | 05:24 | |
| *** markvoelker has joined #openstack-meeting-cp | 05:24 | |
| *** markvoelker has quit IRC | 05:28 | |
| *** nikhil has quit IRC | 05:45 | |
| *** iyamahat has quit IRC | 07:04 | |
| *** markvoelker has joined #openstack-meeting-cp | 07:25 | |
| *** zhipeng has joined #openstack-meeting-cp | 07:53 | |
| *** markvoelker has quit IRC | 07:59 | |
| *** MarkBaker has joined #openstack-meeting-cp | 08:13 | |
| *** zhipeng has quit IRC | 08:18 | |
| *** david-lyle has quit IRC | 08:24 | |
| *** david-lyle has joined #openstack-meeting-cp | 08:25 | |
| *** MarkBaker has quit IRC | 08:27 | |
| *** iyamahat has joined #openstack-meeting-cp | 08:53 | |
| *** yamahata has quit IRC | 08:56 | |
| *** markvoelker has joined #openstack-meeting-cp | 08:57 | |
| *** MarkBaker has joined #openstack-meeting-cp | 09:07 | |
| *** iyamahat has quit IRC | 09:10 | |
| *** david-lyle has quit IRC | 09:26 | |
| *** dklyle has joined #openstack-meeting-cp | 09:26 | |
| *** markvoelker has quit IRC | 09:29 | |
| *** sdague has joined #openstack-meeting-cp | 10:03 | |
| *** markvoelker has joined #openstack-meeting-cp | 10:26 | |
| *** MarkBaker has quit IRC | 10:42 | |
| *** markvoelker has quit IRC | 10:59 | |
| *** MarkBaker has joined #openstack-meeting-cp | 11:27 | |
| *** edmondsw has joined #openstack-meeting-cp | 11:50 | |
| *** markvoelker has joined #openstack-meeting-cp | 11:57 | |
| *** markvoelker has quit IRC | 12:29 | |
| *** MarkBaker has quit IRC | 12:34 | |
| *** markvoelker has joined #openstack-meeting-cp | 12:40 | |
| *** MarkBaker has joined #openstack-meeting-cp | 13:30 | |
| *** gouthamr has joined #openstack-meeting-cp | 13:36 | |
| *** felipemonteiro_ has joined #openstack-meeting-cp | 13:40 | |
| *** dklyle has quit IRC | 14:15 | |
| *** nikhil has joined #openstack-meeting-cp | 14:41 | |
| *** scottda_ has quit IRC | 14:44 | |
| *** david-lyle has joined #openstack-meeting-cp | 14:48 | |
| *** felipemonteiro__ has joined #openstack-meeting-cp | 14:52 | |
| *** felipemonteiro_ has quit IRC | 14:55 | |
| *** mriedem has joined #openstack-meeting-cp | 15:30 | |
| *** felipemonteiro_ has joined #openstack-meeting-cp | 15:49 | |
| *** felipemonteiro__ has quit IRC | 15:52 | |
| *** nhelgeson has joined #openstack-meeting-cp | 15:52 | |
| ildikov | #startmeeting cinder-nova-api-changes | 16:00 |
|---|---|---|
| openstack | Meeting started Thu Oct 26 16:00:02 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 |
| stvnoyes | o/ | 16:00 |
| jungleboyj | @! | 16:00 |
| _pewp_ | jungleboyj (✧∇✧)╯ | 16:00 |
| * smcginnis head explodes | 16:01 | |
| ildikov | hi All :) | 16:01 |
| ildikov | smcginnis: after one minute? :) | 16:01 |
| jgriffith | o/ | 16:01 |
| jungleboyj | No smcginnis Nooo!!!!!! | 16:01 |
| ildikov | ok, let's start | 16:02 |
| ildikov | so we had a milestone last week by merging an updated version of the multi-attach spec | 16:03 |
| jungleboyj | Yay! | 16:03 |
| ildikov | special thanks to mriedem for the last round of cleanups on it! | 16:03 |
| ildikov | and all of you for contributing to figuring it out again | 16:03 |
| ildikov | we talked about capturing some of the policy bits and pieces and other relevant parts on the Cinder side | 16:05 |
| ildikov | I will consult with jgriffith on this | 16:05 |
| mriedem | note there were 2 todos in the spec | 16:05 |
| mriedem | i've got a test up for one of those | 16:05 |
| mriedem | https://review.openstack.org/#/c/515426/ | 16:06 |
| mriedem | to see what happens if you try to attach an attached volume to the same instance with the new v3 attach flow | 16:06 |
| mriedem | regardless of the multiattach flag on the volume | 16:06 |
| mriedem | i have a feeling cinder will be cool with that | 16:06 |
| mriedem | but not sure | 16:06 |
| mriedem | maybe not though - nova will still call os-reserve i think | 16:07 |
| mriedem | and looking at the db conditional update code in cinder, that might block it | 16:07 |
| mriedem | it should | 16:07 |
| ildikov | it won't call that with the new flow | 16:07 |
| mriedem | ok that would be a problem then | 16:07 |
| mriedem | but, let's see what happens | 16:08 |
| mriedem | the other todo in the spec was determining if nova would require a microversion for the change to support multiattach, and i think we would, | 16:08 |
| mriedem | and we have an example to follow (2.49 for tagged attached) | 16:08 |
| ildikov | I was emotionally prepared for the latter | 16:09 |
| ildikov | I guess we should have a spec update too | 16:09 |
| mriedem | we can do that later | 16:09 |
| mriedem | just fyi | 16:09 |
| ildikov | unless someone has objections to the idea | 16:10 |
| ildikov | sounds good! | 16:10 |
| mriedem | otherwise it looks like next up is https://review.openstack.org/#/c/514853/ and the backend_id and shared_targets stuff on the cinder side | 16:10 |
| mriedem | i need to re-review ^ the latest | 16:10 |
| ildikov | yep, jgriffith has just uploaded a fixed version | 16:10 |
| ildikov | and the new attach patch is in a good shape too | 16:11 |
| jgriffith | I'll be rewriting the shared_targets changes now that the online migration stuff is there | 16:11 |
| jgriffith | mriedem: dansmith thanks for the help on that BTW | 16:11 |
| mriedem | np | 16:12 |
| mriedem | one little comment inline | 16:12 |
| mriedem | otherwise lgtm | 16:12 |
| dansmith | yessir | 16:12 |
| jgriffith | mriedem: yeah, not following; but lemme run it and see | 16:12 |
| mriedem | jgriffith: you run the migration once with a limit of 2 and you have 3 total | 16:13 |
| mriedem | just saying, | 16:13 |
| jgriffith | mriedem: assert passed | 16:13 |
| jgriffith | right | 16:13 |
| mriedem | run it again with count=2 and assert total 1 and updated 1 | 16:13 |
| mriedem | so run the migratoin twice | 16:13 |
| jgriffith | mriedem: total is the total number of entries in the table | 16:13 |
| jgriffith | Oh | 16:13 |
| jgriffith | got ya | 16:13 |
| mriedem | total w/o a uuid set | 16:13 |
| jgriffith | yes | 16:13 |
| mriedem | cool | 16:13 |
| mriedem | and then find an intern to write some fixtures for you guys :) | 16:14 |
| jgriffith | :) | 16:14 |
| mriedem | if you're doing a contributor thing at the summit, might be something to put in a list of stuff people can work on | 16:14 |
| ildikov | lol :) | 16:14 |
| mriedem | with examples | 16:14 |
| mriedem | cleaning up the low level db api mocks i mean | 16:14 |
| ildikov | there are occasions to point people to tasks like this | 16:15 |
| ildikov | both the training and the on boarding room, so I will let jungleboyj and smcginnis to give it some extra thoughts :) | 16:16 |
| ildikov | ok, so the online migration is on track and I would expect that the shared_targets patch will be an easier bit to fix up | 16:16 |
| jungleboyj | ildikov: Good thought. | 16:16 |
| ildikov | but jgriffith can correct me if I'm overly ambitious :) | 16:17 |
| jgriffith | ildikov: should be | 16:17 |
| ildikov | jgriffith: coolio | 16:17 |
| jgriffith | but then again I thought fixing up the UUID one was going to be *simple* so... who knows :) | 16:17 |
| ildikov | jgriffith: it's almost weekend here, so give it a bit more positivity :) | 16:18 |
| ildikov | johnthetubaguy: mriedem: any chance you can look at the new attach patch? | 16:18 |
| jgriffith | That trick never works :) | 16:18 |
| mriedem | not right now | 16:18 |
| ildikov | jgriffith: you're so cruel to me today :) | 16:19 |
| mriedem | i'm waiting on those tempest test results too | 16:19 |
| ildikov | ok, let's see those and then if any Nova cores could take a look at that patch that would be great | 16:19 |
| ildikov | so we can have some progress in parallel | 16:19 |
| ildikov | to the margin, I also put the libvirt patch for multi-attach on top of the new attach patch | 16:20 |
| ildikov | I know we're not there yet, however once we are that one should be an easy one to land | 16:21 |
| ildikov | it's also small, so a quick view is also appreciated to see whether the concept used there is acceptable or not | 16:22 |
| ildikov | that's what I mainly have for today | 16:23 |
| stvnoyes | I noticed that tempest doesn't allow for multiple servers being validatable. A floating ip issue. So I was going to update tempest to allow multiple servers to be validatable so we can check multiattachments from inside the vms. It will be useful when we start writing MA tempest tests. Seem reasonable? | 16:23 |
| ildikov | I would love to see the new attach patch landed as soon as possible | 16:23 |
| mriedem | stvnoyes: probably ask the qa team | 16:24 |
| mriedem | bring it up in a weekly meeting maybe, or just ask around in the -qa channel | 16:24 |
| mriedem | andreaf or mtreinish | 16:24 |
| ildikov | stvnoyes: I like your proactivity :) | 16:24 |
| stvnoyes | ok will do | 16:24 |
| ildikov | stvnoyes: tnx | 16:24 |
| ildikov | ok, anything else from anyone for today? | 16:26 |
| mriedem | nope | 16:26 |
| mtreinish | stvnoyes: I'm pretty sure you can create 2 servers that you can ssh into in tempest, there are definitely tests for neutron stuff doing that | 16:27 |
| mtreinish | we can talk about it later in -qa | 16:27 |
| stvnoyes | mtreinish: ok, thanks | 16:27 |
| ildikov | mtreinish: sounds good, thanks! | 16:27 |
| ildikov | then this is it for today | 16:28 |
| ildikov | let's keep in touch on the tempest patch and get the Cinder bits and the new attach patch landed as soon as we can | 16:28 |
| ildikov | oh, one more thing | 16:29 |
| ildikov | does next week work for the most of us or is it "travel day" already? | 16:29 |
| * jungleboyj will be over the ocean somewhere. | 16:30 | |
| mriedem | won't work for me | 16:30 |
| mriedem | cancel next week | 16:30 |
| ildikov | ok, cancel then | 16:30 |
| ildikov | so don't forget about our forum session: https://www.openstack.org/summit/sydney-2017/summit-schedule/events/20457/cindernova-cross-project-session-on-multi-attach?BackURL=https%3A%2F%2Fwww.openstack.org%2Fsummit%2Fsydney-2017%2Fsummit-schedule%2Fglobal-search%3Ft%3Dmulti-attach%23eventid%3D20457 | 16:30 |
| ildikov | the point is to collect feedback on how people intend to use the thing with regards to the follow up plans with touched on earlier | 16:31 |
| ildikov | please make it there if you can so we can answer questions and ensure we cover all the aspects we were discussing earlier | 16:32 |
| *** iyamahat has joined #openstack-meeting-cp | 16:32 | |
| ildikov | we can sync up on the Forum session on the project channels or the ML in case needed | 16:33 |
| mriedem | about that, | 16:33 |
| mriedem | who is seeding the agenda? | 16:33 |
| mriedem | like, what questions are going to be asked? assuming policy and r/w r/o stuff | 16:33 |
| mriedem | maybe some background on the sticky parts in the spec | 16:33 |
| mriedem | like boot from volume | 16:33 |
| jungleboyj | mriedem: Good question. | 16:33 |
| mriedem | the schedule has jay's face on it so i'm deferring to him | 16:34 |
| mriedem | who will likely defer to ildikov | 16:34 |
| ildikov | mriedem: those two were the main questions/concerns | 16:34 |
| jungleboyj | mriedem: Ouch ... but you are kind-of right. | 16:35 |
| jungleboyj | So, what we talked about last time: | 16:35 |
| ildikov | mriedem: beyond that we can share current limitations on libvirt | 16:35 |
| jungleboyj | How do people want to use multi-attach? | 16:36 |
| jungleboyj | What are the expectations for the functionality? | 16:36 |
| mriedem | i'm not sure i'd ask that, | 16:36 |
| mriedem | with 40 minutes, | 16:36 |
| ildikov | and depending on the audience raise the flag for people on the Cinder back end side | 16:36 |
| mriedem | i'd be as specific as possible | 16:36 |
| jungleboyj | mriedem: Ok ... | 16:36 |
| mriedem | like, this is what we're going to do in queens | 16:36 |
| mriedem | these are the limitations | 16:36 |
| mriedem | is that cool with people (yes/no) | 16:36 |
| ildikov | I would rather tell what is the bare minimum people can expect and recruit as many people as possible to test it out | 16:37 |
| mriedem | then if there is time, get into whatever future stuff you want to find out | 16:37 |
| jungleboyj | ildikov: Ok. That makes sense. | 16:37 |
| mriedem | asking "what do you want?" will be a mes | 16:37 |
| mriedem | *mess | 16:37 |
| jungleboyj | So, we tell them ... | 16:37 |
| ildikov | mriedem: +1, I think we're on that same page | 16:37 |
| mriedem | "i want to pass volume type to nova!" | 16:37 |
| jungleboyj | We support one r/w volume and other volues are r/o. | 16:38 |
| jungleboyj | Sorry, attachments. | 16:38 |
| jungleboyj | We are not supporting boot from volume with multi-attach. | 16:38 |
| ildikov | we support all of them being r/w and make people aware that they can screw things up if they are not careful enough very easily | 16:38 |
| ildikov | and then wish good luck and smile :) | 16:38 |
| jungleboyj | ildikov: Did we agree to that? | 16:39 |
| jungleboyj | ildikov: Or are you messing with me? | 16:39 |
| ildikov | jungleboyj: we didn't figure out the R/O part when I last checked, but I might be missing something here | 16:39 |
| jungleboyj | jgriffith: mriedem ? | 16:40 |
| mriedem | jungleboyj: you need to re-read the nova spec | 16:40 |
| jgriffith | :) | 16:40 |
| jungleboyj | mriedem: Saw that coming. | 16:40 |
| * jungleboyj head explodes | 16:40 | |
| jungleboyj | Ok, so, I am going to go read the Cinder and Nova Specs and put together an etherpad for you guys to approve. | 16:41 |
| ildikov | jungleboyj: I can take the burden of MC-ing that session if you want | 16:42 |
| jungleboyj | ildikov: That would be good. :-) People will be nicer to you. | 16:42 |
| ildikov | jungleboyj: and thanks for the etherpad in advance :) | 16:42 |
| ildikov | jungleboyj: haha, not 100% sure about that, but we will see :) | 16:43 |
| jungleboyj | ildikov: Yep, I think that is a fair split of work. Will get me caught up. | 16:43 |
| ildikov | jungleboyj: +1, thanks | 16:43 |
| ildikov | mriedem: any remaining concerns to the session? | 16:43 |
| ildikov | or anyone else? | 16:44 |
| mriedem | no | 16:45 |
| ildikov | cool | 16:45 |
| ildikov | then I think we're now good for today | 16:45 |
| ildikov | please review the two Cinder patches and the new attach patch in Nova before the Summit | 16:46 |
| ildikov | that would at least make me very happy, I know it doesn't matter that much, but still | 16:46 |
| ildikov | and safe travels for next week and see you soon!!! | 16:46 |
| ildikov | thanks everyone! | 16:46 |
| ildikov | #endmeeting | 16:47 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 16:47 | |
| openstack | Meeting ended Thu Oct 26 16:47:09 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:47 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-10-26-16.00.html | 16:47 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-10-26-16.00.txt | 16:47 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-10-26-16.00.log.html | 16:47 |
| jungleboyj | Thank you. | 16:47 |
| johnthetubaguy | ildikov: sorry lots track of time, I will try hit the new attach patch soonish | 17:13 |
| ildikov | johnthetubaguy: no worries and greatly appreciated if you could review that one | 17:14 |
| ildikov | johnthetubaguy: we also cancelled the meeting for next week as many will be traveling at that time already | 17:15 |
| johnthetubaguy | ildikov: ah, good call | 17:15 |
| *** mriedem has left #openstack-meeting-cp | 17:16 | |
| johnthetubaguy | ildikov: I an not there sadly, waiting for baby arrival! | 17:16 |
| ildikov | johnthetubaguy: I hope everything will be alright! | 17:16 |
| *** yamahata has joined #openstack-meeting-cp | 17:18 | |
| ildikov | johnthetubaguy: we will miss you | 17:19 |
| *** iyamahat has quit IRC | 17:19 | |
| ildikov | johnthetubaguy: but it's a pretty nice, happy and exciting reason to miss a Summit :) | 17:20 |
| ildikov | johnthetubaguy: as for the session, if you have anything in mind we haven't talked about during the meeting on what we should ask or mention please let me know | 17:21 |
| johnthetubaguy | ildikov: yeah, a good exciting reason :) I will read back and let you know | 17:22 |
| ildikov | johnthetubaguy: or tell Stig that he has a homework and send him to the session to represent those points :) | 17:22 |
| johnthetubaguy | hehe, yeah | 17:23 |
| johnthetubaguy | ildikov: is that what I think it is? https://review.openstack.org/#/c/267587/39 | 17:23 |
| ildikov | johnthetubaguy: it's the libvirt bit for multi-attach :) | 17:28 |
| ildikov | johnthetubaguy: rebased and ready for comments | 17:29 |
| johnthetubaguy | I thought it might be, there is a good excuse for me to get reviewing on that chain, something for the morning | 17:29 |
| ildikov | glad if that patch helps with prioritizing the review of the chain :) | 17:34 |
| *** yamahata has quit IRC | 17:49 | |
| *** iyamahat has joined #openstack-meeting-cp | 18:06 | |
| jgriffith | -/ws 16 | 18:17 |
| *** yamahata has joined #openstack-meeting-cp | 18:22 | |
| *** kbyrne has quit IRC | 19:27 | |
| *** kbyrne has joined #openstack-meeting-cp | 19:30 | |
| *** harlowja has quit IRC | 19:46 | |
| *** aselius has joined #openstack-meeting-cp | 20:15 | |
| *** harlowja has joined #openstack-meeting-cp | 20:33 | |
| *** andreaf has joined #openstack-meeting-cp | 21:40 | |
| *** gouthamr has quit IRC | 21:59 | |
| *** felipemonteiro_ has quit IRC | 22:24 | |
| *** nhelgeson has quit IRC | 23:02 | |
| *** sdague has quit IRC | 23:33 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!