| *** markvoelker has quit IRC | 00:02 | |
| *** ducttape_ has quit IRC | 00:12 | |
| *** tovin07 has joined #openstack-meeting-cp | 00:48 | |
| *** tovin07_ has joined #openstack-meeting-cp | 00:51 | |
| *** tovin07_ has quit IRC | 00:52 | |
| *** tovin07 has quit IRC | 00:56 | |
| *** tovin07 has joined #openstack-meeting-cp | 00:57 | |
| *** tovin07_ has joined #openstack-meeting-cp | 00:58 | |
| *** ducttape_ has joined #openstack-meeting-cp | 01:13 | |
| *** bastafidli has quit IRC | 01:16 | |
| *** zhurong has joined #openstack-meeting-cp | 01:18 | |
| *** ducttape_ has quit IRC | 01:18 | |
| *** ducttape_ has joined #openstack-meeting-cp | 01:42 | |
| *** ducttape_ has quit IRC | 01:53 | |
| *** ducttape_ has joined #openstack-meeting-cp | 02:41 | |
| *** markvoelker has joined #openstack-meeting-cp | 03:00 | |
| *** markvoelker has quit IRC | 03:05 | |
| *** ducttape_ has quit IRC | 03:05 | |
| *** ducttape_ has joined #openstack-meeting-cp | 03:31 | |
| *** markvoelker has joined #openstack-meeting-cp | 04:01 | |
| *** markvoelker has quit IRC | 04:05 | |
| *** ducttape_ has quit IRC | 04:10 | |
| *** ducttape_ has joined #openstack-meeting-cp | 04:19 | |
| *** ducttape_ has quit IRC | 04:29 | |
| *** markvoelker has joined #openstack-meeting-cp | 05:02 | |
| *** markvoelker has quit IRC | 05:07 | |
| *** prateek has joined #openstack-meeting-cp | 05:28 | |
| *** ducttape_ has joined #openstack-meeting-cp | 05:30 | |
| *** ducttape_ has quit IRC | 05:35 | |
| *** markvoelker has joined #openstack-meeting-cp | 06:02 | |
| *** lifeless_ has quit IRC | 06:02 | |
| *** lifeless has joined #openstack-meeting-cp | 06:03 | |
| *** markvoelker has quit IRC | 06:07 | |
| *** ducttape_ has joined #openstack-meeting-cp | 06:31 | |
| *** ducttape_ has quit IRC | 06:36 | |
| *** markvoelker has joined #openstack-meeting-cp | 07:03 | |
| *** markvoelker has quit IRC | 07:08 | |
| *** ducttape_ has joined #openstack-meeting-cp | 07:32 | |
| *** ducttape_ has quit IRC | 07:37 | |
| *** gouthamr has joined #openstack-meeting-cp | 07:43 | |
| *** kl_ has joined #openstack-meeting-cp | 07:50 | |
| *** kl_ has left #openstack-meeting-cp | 07:50 | |
| *** gouthamr has quit IRC | 08:00 | |
| *** markvoelker has joined #openstack-meeting-cp | 08:04 | |
| *** markvoelker has quit IRC | 08:09 | |
| *** garloff has quit IRC | 08:24 | |
| *** MarkBaker has joined #openstack-meeting-cp | 09:18 | |
| *** ducttape_ has joined #openstack-meeting-cp | 09:34 | |
| *** ducttape_ has quit IRC | 09:38 | |
| *** jamespag` is now known as jamespage | 09:53 | |
| *** zhurong has quit IRC | 10:02 | |
| *** markvoelker has joined #openstack-meeting-cp | 10:06 | |
| *** tovin07_ has quit IRC | 10:09 | |
| *** markvoelker has quit IRC | 10:10 | |
| *** MarkBaker has quit IRC | 10:21 | |
| *** ducttape_ has joined #openstack-meeting-cp | 10:35 | |
| *** ducttape_ has quit IRC | 10:39 | |
| *** ducttape_ has joined #openstack-meeting-cp | 10:50 | |
| *** markvoelker has joined #openstack-meeting-cp | 11:07 | |
| *** notmyname has quit IRC | 11:07 | |
| *** notmyname has joined #openstack-meeting-cp | 11:11 | |
| *** markvoelker has quit IRC | 11:11 | |
| *** ducttape_ has quit IRC | 11:15 | |
| *** ducttape_ has joined #openstack-meeting-cp | 11:18 | |
| *** MarkBaker has joined #openstack-meeting-cp | 11:23 | |
| *** sdague_ has joined #openstack-meeting-cp | 11:24 | |
| *** sdague_ is now known as sdague | 11:31 | |
| *** ducttape_ has quit IRC | 11:44 | |
| *** zhurong has joined #openstack-meeting-cp | 11:54 | |
| *** zhurong has quit IRC | 12:06 | |
| *** zhurong has joined #openstack-meeting-cp | 12:07 | |
| *** markvoelker has joined #openstack-meeting-cp | 12:07 | |
| *** markvoelker has quit IRC | 12:12 | |
| *** zhurong has quit IRC | 12:19 | |
| *** ducttape_ has joined #openstack-meeting-cp | 12:45 | |
| *** ducttape_ has quit IRC | 12:50 | |
| *** zhurong has joined #openstack-meeting-cp | 12:53 | |
| *** MarkBaker has quit IRC | 13:06 | |
| *** zhurong has quit IRC | 13:11 | |
| *** zhurong has joined #openstack-meeting-cp | 13:12 | |
| *** dirk has quit IRC | 13:12 | |
| *** dirk has joined #openstack-meeting-cp | 13:12 | |
| *** markvoelker has joined #openstack-meeting-cp | 13:15 | |
| *** MarkBaker has joined #openstack-meeting-cp | 13:19 | |
| *** zhurong has quit IRC | 13:26 | |
| *** zhurong has joined #openstack-meeting-cp | 13:27 | |
| *** prateek has quit IRC | 13:34 | |
| *** bastafidli has joined #openstack-meeting-cp | 13:52 | |
| *** zhurong has quit IRC | 13:53 | |
| *** xyang1 has joined #openstack-meeting-cp | 13:56 | |
| *** bastafidli has quit IRC | 14:02 | |
| *** MarkBaker has quit IRC | 14:08 | |
| *** ducttape_ has joined #openstack-meeting-cp | 14:10 | |
| *** ducttape_ has quit IRC | 14:40 | |
| *** ducttape_ has joined #openstack-meeting-cp | 14:42 | |
| *** prateek has joined #openstack-meeting-cp | 14:47 | |
| *** ducttape_ has quit IRC | 14:48 | |
| *** MarkBaker has joined #openstack-meeting-cp | 15:07 | |
| *** piet has joined #openstack-meeting-cp | 15:12 | |
| *** ducttape_ has joined #openstack-meeting-cp | 15:19 | |
| *** jaugustine has joined #openstack-meeting-cp | 15:24 | |
| *** margaret has quit IRC | 15:27 | |
| *** margaret has joined #openstack-meeting-cp | 15:27 | |
| *** samueldmq has quit IRC | 15:37 | |
| *** samueldmq has joined #openstack-meeting-cp | 15:37 | |
| *** brault_ has joined #openstack-meeting-cp | 15:45 | |
| *** brault has quit IRC | 15:47 | |
| *** prateek has quit IRC | 15:54 | |
| *** _ducttape_ has joined #openstack-meeting-cp | 15:56 | |
| *** ducttap__ has joined #openstack-meeting-cp | 15:58 | |
| *** _ducttape_ has quit IRC | 15:58 | |
| *** bastafidli has joined #openstack-meeting-cp | 15:59 | |
| *** ducttape_ has quit IRC | 15:59 | |
| *** sajeesh has joined #openstack-meeting-cp | 16:26 | |
| *** stvnoyes has joined #openstack-meeting-cp | 16:37 | |
| *** ducttap__ has quit IRC | 16:48 | |
| *** ducttape_ has joined #openstack-meeting-cp | 16:48 | |
| *** mriedem has joined #openstack-meeting-cp | 17:00 | |
| ildikov | #startmeeting cinder-nova-api-changes | 17:00 |
|---|---|---|
| openstack | Meeting started Mon Dec 12 17:00:12 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 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 17:00 |
| *** openstack changes topic to " (Meeting topic: cinder-nova-api-changes)" | 17:00 | |
| openstack | The meeting name has been set to 'cinder_nova_api_changes' | 17:00 |
| ildikov | scottda 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 | 17:00 |
| jgriffith | o/ | 17:00 |
| mriedem | o/ | 17:00 |
| DuncanT | o/ | 17:00 |
| johnthetubaguy | I am back but can't make this today again :( | 17:00 |
| johnthetubaguy | but I can lurk in two meetings at once | 17:00 |
| ildikov | johnthetubaguy: still welcome back :) | 17:01 |
| ildikov | I updated our chaotic etherpad with some info: #link https://etherpad.openstack.org/p/cinder-nova-api-changes | 17:01 |
| scottda | hi | 17:01 |
| ildikov | the Cinder side spec got merged: #link https://review.openstack.org/#/c/361512/ | 17:02 |
| ildikov | it captures the basics, like the API calls and a few more to have a foundation to start with | 17:02 |
| ildikov | code is on its way: #link https://review.openstack.org/#/q/status:open+project:openstack/cinder+branch:master+topic:attach_detach_v2 | 17:03 |
| hemna | morning | 17:03 |
| ildikov | hemna: afternoon :) | 17:03 |
| hemna | :) | 17:03 |
| ildikov | anyone has anything to add to the above announcement/items? | 17:04 |
| hemna | how far off are we getting the code out of WIP? | 17:05 |
| jgriffith | hemna depends | 17:05 |
| jgriffith | hemna need reviews (thanks ildikov by the way for the reviews over the week-end!) | 17:06 |
| jgriffith | hemna I'd like to get the attachments patch merged then push a non-wip up for the rest of the cinder changes | 17:06 |
| hemna | ok | 17:06 |
| ildikov | jgriffith: no problem (always good to have a workaholic on the team :) | 17:06 |
| hemna | I'll look at the attachments review | 17:06 |
| jgriffith | :) | 17:06 |
| ildikov | jgriffith: when do you think you can get to the cinderclient changes? | 17:07 |
| jgriffith | hemna I'm trying to figure out again how to just use the volume backref and a join to get provider id | 17:07 |
| ildikov | jgriffith: I thought to push the Nova side forward a bit, but that'll need the client to test | 17:07 |
| mriedem | well, we have other things we can do on the nova side first, | 17:08 |
| jgriffith | ildikov ok, I can bump that up today | 17:08 |
| mriedem | i.e. nova doesn't currently support cinder v3 | 17:08 |
| mriedem | that could be worked independent of any of this | 17:08 |
| *** gouthamr has joined #openstack-meeting-cp | 17:08 | |
| jgriffith | mriedem well there is that minor detail :) | 17:08 |
| mriedem | and we could enable nova to use cinder v3 in the placement CI job | 17:08 |
| mriedem | which is where we do our bleeding edge testing | 17:08 |
| ildikov | jgriffith: thanks, I'll let you know when it's worth it to reopen the Nova review | 17:09 |
| ildikov | mriedem: is there any activity on this currently? | 17:09 |
| ildikov | jgriffith: I like your Monday optimism re "minor detail" :) | 17:10 |
| mriedem | ildikov: nope | 17:10 |
| scottda | ildikov: I had put up some WIP patches to allow testing of nova -> cinder v3 | 17:10 |
| scottda | https://review.openstack.org/#/c/385682/ | 17:10 |
| mriedem | it might just be configuring nova in a job to use cinder v3 | 17:10 |
| scottda | all that ^^ needs unit tests and more love, but I *believe* it all basically worked with Cinder microversions on /v3 endpoint. It's been a while.... | 17:11 |
| ildikov | scottda: cool, tnx! | 17:11 |
| scottda | mriedem: Yeah, I think nova just needs: | 17:11 |
| scottda | Requires use of Cinder v3 endpoint by adding to nova.conf: [cinder] catalog_info = volumev3:cinderv3:publicURL | 17:11 |
| mriedem | scottda: yeah your change is still using v2 | 17:14 |
| mriedem | volumev2:cinderv2:publicURL | 17:14 |
| mriedem | is the default | 17:14 |
| *** diablo_rojo_phon has joined #openstack-meeting-cp | 17:14 | |
| ildikov | scottda: what does 'more love' mean in your above comment? I got the unit test part :) | 17:14 |
| mriedem | scott's love level was insufficient | 17:15 |
| scottda | ildikov: Ha. I think the code will need some methods to get the cinder microversion for use inside Nova code. | 17:15 |
| scottda | It's just a quick hack to allow testing. | 17:16 |
| mriedem | note that nova also still supports cinder v1 | 17:16 |
| ildikov | scottda: ok, got it, thanks | 17:16 |
| scottda | And I'm sure Nova folks will have a better idea of what they'd like to see. | 17:16 |
| mriedem | if version == '1' and not _V1_ERROR_RAISED: | 17:16 |
| mriedem | so we might want to just remove that support | 17:16 |
| mriedem | i can take that on | 17:16 |
| ildikov | mriedem: sounds good, tnx | 17:18 |
| ildikov | scottda: I will look up the Nova priority etherpad where this new Cinder API activity is tracked and add your patch there to get more eyes on it | 17:19 |
| scottda | ildikov: Well, it was a WIP and then became abandoned... | 17:19 |
| scottda | ildikov: I'll re-visit the patch and put up a real patch this week. | 17:19 |
| ildikov | scottda: bah, ok I didn't check the header :) | 17:19 |
| ildikov | scottda: I'll do the administration once that's done | 17:20 |
| ildikov | if there's nothing more to the microversions, I think last time we mainly discussed things that were in the Nova spec, like volume state changes in error scenarios, etc. | 17:23 |
| ildikov | mriedem: now that johnthetubaguy I guess the next step is push forward that spec based on the comments | 17:24 |
| ildikov | mriedem: is my assumption correct? | 17:24 |
| mriedem | john will probably need some time to ease back in from the break, and process my deluge of comments on his spec | 17:25 |
| ildikov | mriedem: :) | 17:26 |
| *** ducttape_ has left #openstack-meeting-cp | 17:26 | |
| ildikov | I think we can do the prototyping with the plain simple attach/detach flow, while the comments on the Nova spec get cleaned up | 17:27 |
| ildikov | mriedem: is there anything among the comments that we should bring up here? | 17:27 |
| mriedem | it was last monday when i reviewed it, so i'm fuzzy, but nova setting error state on the cinder resources (attachment) was my main complaint | 17:28 |
| ildikov | mriedem: I think it's currently not covered on the Cinder side, I mean in the spec or the current version of the code | 17:29 |
| ildikov | mriedem: I think we didn't have an agreement on modifying the error state part so far | 17:31 |
| ildikov | mriedem: you're on the side of not doing it, right? | 17:31 |
| mriedem | ildikov: correct | 17:32 |
| ildikov | last time beyond the Cinder basics I think this was the main item that came up | 17:32 |
| mriedem | i don't think nova, or anything, should meddle with the internal state of a resource in cinder | 17:32 |
| hemna | mriedem, +1 | 17:32 |
| jgriffith | mriedem agreed | 17:32 |
| ildikov | mriedem: +1 | 17:32 |
| ildikov | I can link in the meeting logs to the spec and then we can look into how to handle that part | 17:33 |
| ildikov | mriedem: the other thing we talked about earlier is when Nova will create a new attachment and when it should just update it | 17:34 |
| ildikov | mriedem: I think we can experiment it in the PoC patch in parallel to working on the edge cases in the spec | 17:34 |
| mriedem | i wasn't clear on that from the spec, it kind of goes both ways at times it seems | 17:34 |
| ildikov | mriedem: or if there's anything in anyone's mind then we can touch on that here too | 17:34 |
| mriedem | i thought to mimic reserve today we'd create the empty attachment in nova-api | 17:34 |
| mriedem | and then update it with host/connector details in n-cpu | 17:35 |
| jgriffith | mriedem correct | 17:35 |
| ildikov | yeap, that's the plan | 17:35 |
| mriedem | but i thought i saw something in the nova spec about creating the attachment in n-cpu, which would be racey | 17:35 |
| mriedem | but i might be confusing volume attach and BFV | 17:35 |
| mriedem | today for BFV we don't reserve the volume from n-api | 17:36 |
| ildikov | I guess questions come up when it comes to handling error cases like the instance will end up on another host by the end of the day-type of rebuild, etc. | 17:36 |
| hemna | which is odd | 17:36 |
| ildikov | mriedem: the patch for that is still on it's way :) | 17:36 |
| mriedem | for rebuilds, | 17:37 |
| ildikov | mriedem: in the sense of needs review, although I can remove a few more things from the code before told that it's the total wrong direction... :) | 17:37 |
| mriedem | we don't retry on BFV errors today | 17:37 |
| ildikov | when does Nova retry? | 17:38 |
| mriedem | https://review.openstack.org/#/c/246505/ | 17:38 |
| mriedem | when does nova retry....well.... | 17:38 |
| mriedem | depends on what's handled in the compute manager | 17:38 |
| mriedem | which is kind of a mess | 17:39 |
| ildikov | hmm, I wonder whether we can come up with anything high level regarding how we would like to handle these if things would be less messy | 17:40 |
| ildikov | or just handle this as we get there one by one | 17:40 |
| ildikov | mriedem: do you think that review will be picked up? As it's currently abandoned | 17:41 |
| mriedem | nova needs to eventually move the volume creation code to conductor, | 17:41 |
| mriedem | like we're doing with ports | 17:41 |
| mriedem | but that's a ways out | 17:41 |
| mriedem | ildikov: it's up to someone to pick up i guess, | 17:41 |
| mriedem | i don't think the submitter will | 17:41 |
| mriedem | as it's powervc and they just carry patches internally | 17:42 |
| ildikov | ok, then we keep that path in mind, but it's not a burden for now | 17:42 |
| mriedem | for the new hotness, i think nova just deletes the attachment in n-cpu on failure | 17:42 |
| mriedem | like nova should do for port deallocation between retries | 17:42 |
| jgriffith | mriedem yes, that's what I thought we agreed upon | 17:42 |
| mriedem | if you come with a volume/port, nova doesn't touch it between retries, but if nova creates it, nova should clean it up | 17:43 |
| hemna | that sounds reasonable to me | 17:44 |
| ildikov | mriedem: +1 | 17:44 |
| hemna | nova is the one who is executing the workflow | 17:44 |
| ildikov | I will reread the Nova spec with this mindset to see where we are | 17:45 |
| *** piet has quit IRC | 17:45 | |
| ildikov | although I guess mriedem captured all the hiccups in it already :) | 17:46 |
| ildikov | anything else in connection to this? | 17:47 |
| ildikov | ok, then one more regular reminder from my side | 17:48 |
| ildikov | mriedem: johnthetubaguy: pretty please, sugar, whipped cream, cherry, whatever on top give this puppy a quick review: https://review.openstack.org/#/c/335358/ :) | 17:49 |
| ildikov | mriedem: johnthetubaguy: or tell me who else is competent with the area enough to be interested in looking at it | 17:50 |
| johnthetubaguy | thats probably me | 17:50 |
| *** gouthamr has quit IRC | 17:50 | |
| *** piet has joined #openstack-meeting-cp | 17:50 | |
| ildikov | johnthetubaguy: cool, tnx, I will add you to my 'annoy people' list then :) | 17:51 |
| *** gouthamr has joined #openstack-meeting-cp | 17:52 | |
| ildikov | anything else for today to discuss? | 17:52 |
| *** MarkBaker has quit IRC | 17:52 | |
| *** gouthamr has quit IRC | 17:52 | |
| scottda | Are we meeting next week? | 17:53 |
| ildikov | scottda: I wanted to ask when to stop/resume the series, tnx for reminding :) | 17:54 |
| ildikov | I'm available next week, but we can skip if we don't have enough people around | 17:54 |
| scottda | I'll be around next week. I could go either way... | 17:54 |
| hemna | I'm around next week | 17:55 |
| ildikov | jgriffith: mriedem: johnthetubaguy: how likely will you be around next week? | 17:55 |
| mriedem | i'm around | 17:55 |
| jgriffith | me too | 17:55 |
| ildikov | ok, cool, then let's aim for a quick one next week | 17:56 |
| ildikov | and I think we can resume the meeting series on the 9th next year | 17:57 |
| ildikov | but we can decide on this next week | 17:57 |
| scottda | ildikov: Sounds good | 17:57 |
| *** openstack has joined #openstack-meeting-cp | 17:58 | |
| *** ChanServ sets mode: +o openstack | 17:58 | |
| ildikov | #endmeeting | 17:59 |
| *** piet has quit IRC | 18:01 | |
| *** bastafidli has quit IRC | 18:01 | |
| *** piet has joined #openstack-meeting-cp | 18:11 | |
| *** stvnoyes has quit IRC | 18:11 | |
| *** stvnoyes has joined #openstack-meeting-cp | 18:12 | |
| *** harlowja has joined #openstack-meeting-cp | 18:20 | |
| *** Rockyg has joined #openstack-meeting-cp | 18:50 | |
| *** Rockyg has quit IRC | 18:54 | |
| *** Rockyg has joined #openstack-meeting-cp | 18:54 | |
| *** _ducttape_ has joined #openstack-meeting-cp | 19:27 | |
| *** _ducttape_ has quit IRC | 19:39 | |
| *** ducttape_ has joined #openstack-meeting-cp | 19:40 | |
| *** diablo_rojo_phon has quit IRC | 19:40 | |
| *** gouthamr has joined #openstack-meeting-cp | 19:46 | |
| *** bastafidli has joined #openstack-meeting-cp | 19:57 | |
| *** stvnoyes has quit IRC | 20:06 | |
| *** stvnoyes has joined #openstack-meeting-cp | 20:24 | |
| *** gouthamr has quit IRC | 20:48 | |
| *** mrhillsman has quit IRC | 21:22 | |
| *** codebauss has joined #openstack-meeting-cp | 21:28 | |
| *** codebauss is now known as mrhillsman | 21:28 | |
| *** jgriffith is now known as jgriffith_AutoAw | 21:47 | |
| *** jgriffith_AutoAw is now known as jgriffith | 21:52 | |
| *** MarkBaker has joined #openstack-meeting-cp | 22:09 | |
| *** jgriffith is now known as jgriffith_AutoAw | 22:23 | |
| *** bastafidli has quit IRC | 22:27 | |
| *** jaugustine has quit IRC | 22:33 | |
| *** jgriffith_AutoAw is now known as jgriffith | 22:57 | |
| *** _ducttape_ has joined #openstack-meeting-cp | 22:59 | |
| *** ducttape_ has quit IRC | 23:02 | |
| *** _ducttape_ has quit IRC | 23:03 | |
| *** ducttape_ has joined #openstack-meeting-cp | 23:37 | |
| *** ducttape_ has quit IRC | 23:42 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!