*** felipemonteiro has joined #openstack-meeting-cp | 01:15 | |
*** markvoelker has quit IRC | 02:08 | |
*** markvoelker has joined #openstack-meeting-cp | 02:08 | |
*** gouthamr has quit IRC | 02:47 | |
*** gouthamr has joined #openstack-meeting-cp | 02:48 | |
*** felipemonteiro has quit IRC | 03:12 | |
*** felipemonteiro has joined #openstack-meeting-cp | 03:35 | |
*** markvoelker has quit IRC | 03:56 | |
*** markvoelker has joined #openstack-meeting-cp | 03:57 | |
*** markvoelker has quit IRC | 04:01 | |
*** diablo_rojo has quit IRC | 04:34 | |
*** lamt has joined #openstack-meeting-cp | 04:43 | |
*** lamt has quit IRC | 05:17 | |
*** lamt has joined #openstack-meeting-cp | 05:29 | |
*** felipemonteiro has quit IRC | 05:37 | |
*** lamt has quit IRC | 05:53 | |
*** lamt has joined #openstack-meeting-cp | 06:12 | |
*** lamt has quit IRC | 06:26 | |
*** gouthamr has quit IRC | 06:36 | |
*** MarkBaker has joined #openstack-meeting-cp | 07:53 | |
*** MarkBaker has quit IRC | 08:18 | |
*** MarkBaker has joined #openstack-meeting-cp | 08:43 | |
*** MarkBaker has quit IRC | 08:48 | |
*** MarkBaker has joined #openstack-meeting-cp | 09:00 | |
*** MarkBaker has quit IRC | 09:17 | |
*** MarkBaker has joined #openstack-meeting-cp | 10:15 | |
*** pewp has quit IRC | 10:16 | |
*** pewp has joined #openstack-meeting-cp | 10:17 | |
*** sdague has joined #openstack-meeting-cp | 11:06 | |
*** edmondsw has joined #openstack-meeting-cp | 12:03 | |
*** markvoelker has joined #openstack-meeting-cp | 12:23 | |
*** lamt has joined #openstack-meeting-cp | 12:52 | |
*** gouthamr has joined #openstack-meeting-cp | 12:58 | |
*** MarkBaker has quit IRC | 13:20 | |
*** MarkBaker has joined #openstack-meeting-cp | 13:33 | |
*** MarkBaker has quit IRC | 13:46 | |
*** lbragstad has joined #openstack-meeting-cp | 13:48 | |
*** lamt has quit IRC | 13:52 | |
*** MarkBaker has joined #openstack-meeting-cp | 14:01 | |
*** felipemonteiro has joined #openstack-meeting-cp | 14:06 | |
*** felipemonteiro_ has joined #openstack-meeting-cp | 14:09 | |
*** felipemonteiro__ has joined #openstack-meeting-cp | 14:12 | |
*** felipemonteiro has quit IRC | 14:12 | |
*** felipemonteiro_ has quit IRC | 14:15 | |
*** diablo_rojo has joined #openstack-meeting-cp | 14:18 | |
*** jaugustine has joined #openstack-meeting-cp | 14:42 | |
*** lamt has joined #openstack-meeting-cp | 15:11 | |
*** MarkBaker has quit IRC | 16:00 | |
ildikov | #startmeeting cinder-nova-api-changes | 16:00 |
---|---|---|
openstack | Meeting started Thu May 18 16:00:41 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 | 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 |
jungleboyj | o/ | 16:00 |
ildikov | jungleboyj: hi :) | 16:01 |
stvnoyes | o/ | 16:01 |
smcginnis | ./ | 16:01 |
jungleboyj | ildikov: Hello. | 16:01 |
smcginnis | ildikov: Back home yet? | 16:01 |
ildikov | smcginnis: yep, landed yesterday afternoon | 16:02 |
jungleboyj | ildikov: Glad you made it safe. Hope it was a good trip. | 16:02 |
smcginnis | ildikov: Did you remember how to get back to your apartment? :D | 16:02 |
ildikov | smcginnis: my brain is still useless though, after Summit syndrome... | 16:02 |
hemna | mep | 16:02 |
ildikov | smcginnis: luckily taxi drivers help you out if you know the address :) | 16:02 |
jungleboyj | ildikov: ++ | 16:03 |
ildikov | jungleboyj: yep, it was alright | 16:03 |
ildikov | so let's get started | 16:03 |
ildikov | the main news from last week is that I've never had this many people approaching me about multi-attach ever before | 16:04 |
ildikov | a guy even told me they fixed up my two Nova patches and running it in production on Kilo | 16:05 |
jungleboyj | ildikov: Wow. | 16:05 |
hemna | ildikov, w00t | 16:05 |
ildikov | the conclusion is that we better get ourselves together and finally finish this thing! :) | 16:05 |
* johnthetubaguy wonders in a touch later than planned | 16:05 | |
ildikov | at least that's my conclusion :) | 16:06 |
ildikov | johnthetubaguy: no worries, I'm just sharing my experiences from last week | 16:06 |
johnthetubaguy | yup, lets make it safe before they all do it very unsafe! | 16:06 |
johnthetubaguy | honestly its the feature most people ask me about at openstack events too, at least right now | 16:06 |
ildikov | johnthetubaguy: big +1 | 16:07 |
ildikov | so that said we started to look into swap with jgriffith | 16:07 |
hemna | has anyone asked those folks what they plan on doing with it? :) | 16:08 |
hemna | and what filesystem they plan on using? | 16:08 |
stvnoyes | our customers want Oracle RAC | 16:08 |
ildikov | hemna: I don't remember what the guy said who's running this in production, but he didn't complain it didn't work | 16:09 |
ildikov | I don't think this is gonna be the first feature doing harm if you don't use it correctly | 16:10 |
hemna | yah, I'm just curious | 16:10 |
hemna | I think most folks are smart and know they can't run ext4fs on it, but some will try. | 16:10 |
ildikov | I tried to recruit everyone, so I asked them more about how much they can get involved | 16:10 |
ildikov | I will feel sorry for those ones for sure | 16:11 |
johnthetubaguy | hemna: I have this crazy clustered thingy is the usual answer I get | 16:11 |
hemna | clusterfs! | 16:11 |
hemna | *magic* | 16:11 |
johnthetubaguy | its really easy to use this feature badly, I am curious if we want a "read-only" multi-attach mode too | 16:11 |
ildikov | hemna: also there are cases where they would want to have one instance that writes on the volume and the rest just read | 16:11 |
johnthetubaguy | ildikov: not sure thats always the case, but that sounds like the most common case | 16:12 |
* hemna wonders if that even works | 16:12 | |
ildikov | johnthetubaguy: no, certainly not always | 16:12 |
johnthetubaguy | some stuff was a simple active/passive HA pair, that both needs storage | 16:12 |
hemna | I might try hacking the brick cinderclient extension and see if I can do a multiattach with cinder directly | 16:12 |
*** MarkBaker has joined #openstack-meeting-cp | 16:12 | |
hemna | to see if I can test that out at all | 16:12 |
ildikov | hemna: sounds good | 16:13 |
johnthetubaguy | hemna: there is a libvirt property you want to set on the connection, to stop the caches doing funky things | 16:13 |
johnthetubaguy | hemna: I think thats in the attach WIP patch anyways | 16:13 |
hemna | ok cool. | 16:13 |
hemna | and for baremetal ? | 16:13 |
ildikov | hemna: I can rebase my old Nova patches and enable multi-attach in them again so you could even play with Nova | 16:13 |
hemna | just has to be mounted with -o ro ? | 16:13 |
smcginnis | hemna: I think so. | 16:14 |
stvnoyes | one of our guys has ma working, sorta basic. no migrate, swap, etc. keeping the connection from getting terminated was the biggest challenge, tho it wasn't much code | 16:14 |
hemna | I'm a little wary that even mounting ext4fs in ro mode will try induce a write somehow. | 16:14 |
johnthetubaguy | are we getting distracted? | 16:14 |
hemna | anyway, ignore me. | 16:14 |
stvnoyes | this was not using the new v3 api. just a proto for now. a learning experience. | 16:14 |
johnthetubaguy | we have a bunch of patches pending with my +2 on them, the others all looked WIP right now, did I just miss them? | 16:15 |
johnthetubaguy | oh wait there was a think about ildikov looking at swap? | 16:16 |
johnthetubaguy | how did that go? | 16:16 |
ildikov | johnthetubaguy: no, you didn't | 16:16 |
stvnoyes | i'd like someone to take a look at this rv -https://review.openstack.org/#/c/463987/ - just to let me know if i'm on the right direction. | 16:16 |
ildikov | johnthetubaguy: stvnoyes works on the live migrate snippets | 16:17 |
ildikov | johnthetubaguy: with jgriffith we got to the conclusion of re-writing swap | 16:17 |
johnthetubaguy | stvnoyes: I can take a peak, there was a live-migrate one I voted on recently | 16:17 |
stvnoyes | great thx | 16:17 |
johnthetubaguy | ildikov: thats probably for the best, happy to have a new Nova API if that makes it easier, or at least, I am open to the idea | 16:18 |
ildikov | I think the idea is to let Nova do the attach and detach | 16:18 |
ildikov | and have things there in a clean way as opposed to the current call-back | 16:19 |
johnthetubaguy | Nova does do the attach and detach today | 16:19 |
johnthetubaguy | oh... API wise | 16:19 |
ildikov | there are some attach/detach calls on the Cinder side today too for some corner cases I think | 16:19 |
johnthetubaguy | I think I see what you mean now | 16:19 |
ildikov | that code is a bit confusing... | 16:20 |
johnthetubaguy | nova does detach, then does the call back? | 16:20 |
jungleboyj | Sorry, was pulled off on other things. The Read Only use case was asked for by several people at the Summit. | 16:20 |
ildikov | I think Nova does terminate | 16:20 |
*** pewp has quit IRC | 16:20 | |
johnthetubaguy | the call back does the crazy volume uuid rename nonsense | 16:20 |
ildikov | yeah | 16:20 |
johnthetubaguy | jungleboyj: I kept hearing that too | 16:20 |
ildikov | that's the main part | 16:20 |
johnthetubaguy | ildikov: I like the sound of your plan | 16:20 |
jungleboyj | johnthetubaguy: Ok, so not just me. :-) | 16:21 |
johnthetubaguy | stvnoyes: can I ask about attachment_ids in your live-migrate patch | 16:21 |
johnthetubaguy | jungleboyj: if I were an operator again, I would only enable the read-only version, basically attach a snapshot or somesuch | 16:21 |
stvnoyes | johnthetubaguy: sure | 16:22 |
ildikov | johnthetubaguy: we can move on from swap, I will sync up with jgriffith as soon as I can regarding where he is with the code changes he started | 16:22 |
johnthetubaguy | stvnoyes: we have a new attachment_id after the attach call in the setup I guess, in pre_live_migration | 16:22 |
johnthetubaguy | on the destination host, basically | 16:22 |
jungleboyj | johnthetubaguy: ++ | 16:22 |
johnthetubaguy | but when in the tidy up we do the _detach call... we need the old attachment_id | 16:22 |
johnthetubaguy | it doesn't seem like your patch is doing that | 16:22 |
johnthetubaguy | but I might be missing some magic | 16:23 |
stvnoyes | ah good catch, i'll take a look at that | 16:23 |
stvnoyes | we don't delete the first attachment until the migrate is successful, so the orig bdm is still around. but i'll verify | 16:24 |
johnthetubaguy | do we want a destination_attachment_id in the BDM? | 16:24 |
johnthetubaguy | that would make it clear | 16:24 |
johnthetubaguy | not sure what mriedman would think about that mind | 16:24 |
ildikov | I wanted to ask whether we need to store two attachment_id's | 16:24 |
johnthetubaguy | I think we have to | 16:25 |
johnthetubaguy | I mean we can go look it up, but that seems stupid | 16:25 |
ildikov | you mean storing or not storing it? | 16:25 |
stvnoyes | i would like to ask if we could get attach with v3 cinder in the code, but disabled. then as we develop these bits we can emable it in a dev env to test it through. | 16:26 |
smcginnis | stvnoyes: Like a temporary flag to switch? | 16:26 |
johnthetubaguy | stvnoyes: I think we should do that, get ildikov to rebase her patch? | 16:26 |
stvnoyes | maybe or a config option. or even disabled in source. | 16:26 |
johnthetubaguy | why not just cherry-pick the attach patch on top for testing? | 16:27 |
ildikov | johnthetubaguy: I did the rebase, just couldn't get there yet to spin it up in Devstack and test it | 16:27 |
johnthetubaguy | ildikov: what did the gate say about the rebase? | 16:27 |
ildikov | johnthetubaguy: making the unit tests work requires more work with the microversions and I certainly didn't want to get deep in that just to be able to test some stuff | 16:27 |
*** pewp has joined #openstack-meeting-cp | 16:27 | |
ildikov | johnthetubaguy: I know we will have to, I just would like to go a bit step by step here and try to get something working for real and then play with the microversion discovery and unit tests | 16:28 |
johnthetubaguy | ildikov: I was thinking the functional stuff shoudl run OK | 16:28 |
johnthetubaguy | ah, gotcha | 16:29 |
ildikov | I know, I'm evil, but just for practicality... | 16:29 |
ildikov | johnthetubaguy: I will get myself back on track with that patch and will ping people who're involved, when it seems working | 16:29 |
johnthetubaguy | all good | 16:30 |
ildikov | a ton of things have changed, since jgriffith and me made that one work for the last time... :( | 16:30 |
johnthetubaguy | stvnoyes: so I added comments on your live-migrate patch | 16:31 |
*** MarkBaker has quit IRC | 16:31 | |
johnthetubaguy | stvnoyes: do ping me for questions, I know that flow fairly well at the moment | 16:31 |
stvnoyes | thx | 16:31 |
johnthetubaguy | I think we should be creating a new attachment in the pre-live-migration stuff | 16:32 |
johnthetubaguy | before we do that first attach call | 16:32 |
ildikov | johnthetubaguy: did you add a note about whether or not store the old attachment_id in the BDM as well? | 16:32 |
johnthetubaguy | yeah, I said please store it in the BDM | 16:32 |
johnthetubaguy | I think you might be able to hide it without needing a db migration, but I would be tempted to make it a top level item and be done with it | 16:33 |
johnthetubaguy | anyways, you are totally in the correct bit of code there | 16:33 |
ildikov | cool, tnx, just would want to have mriedem's eyes on that part as well before ending up in rabbit holes | 16:35 |
johnthetubaguy | anyways, stvnoyes, I would think about being explcit about which of the two attachments you use in each case, and for each place we touch a volume in the flow, I think that will shake out more of the details | 16:36 |
johnthetubaguy | I suspect a lot of... bugs to be found in that code, FWIW | 16:36 |
smcginnis | Undocumented features. | 16:36 |
jungleboyj | smcginnis: He he he. | 16:37 |
ildikov | johnthetubaguy: I also wonder whether we can get rid of the second use of initialize_connection there | 16:37 |
johnthetubaguy | we do that twice? yeah, probably | 16:37 |
* johnthetubaguy nods at smcginnis | 16:37 | |
ildikov | I mean we should, just cannot see that far as of yet on when we get there and how that will look like exactly... :) | 16:38 |
ildikov | johnthetubaguy: yeah, it's used in that flow a couple of times | 16:38 |
johnthetubaguy | for live-migration always thinking about plumbing really | 16:38 |
johnthetubaguy | you know a hosepipe with one of those shut off values, so when you pull ou the VM you don't get water everywhere, or something | 16:39 |
johnthetubaguy | my head is a strange place at times | 16:39 |
johnthetubaguy | sounds like everyone has their next steps for the comming week | 16:40 |
johnthetubaguy | or did we miss something? | 16:40 |
stvnoyes | johnthetubaguy: kk will do | 16:40 |
ildikov | I think we should be fine | 16:40 |
ildikov | I haven't done my homework so will not finger point at others either | 16:41 |
johnthetubaguy | thats all good | 16:41 |
ildikov | thanks stvnoyes for taking care of live migrate! | 16:41 |
johnthetubaguy | its a very scary part of the code, its good to see that starting | 16:42 |
ildikov | johnthetubaguy: I will try to ping lyarwood to take a look at those patches with +2's and get them merged sooner rather than later | 16:42 |
johnthetubaguy | cool, more eyes on those would be good | 16:42 |
ildikov | agreed | 16:42 |
stvnoyes | is this rv still needed. i noticed after i posted mine. i think it | 16:42 |
johnthetubaguy | sfinucan might be able to +2 them, its worth asking him | 16:43 |
stvnoyes | s redunandant now - https://review.openstack.org/#/c/456988/ | 16:43 |
johnthetubaguy | ah, thats the one I head in my head from early in the week | 16:43 |
johnthetubaguy | yeah | 16:43 |
stvnoyes | (not typing well, hand in sling) | 16:43 |
ildikov | stvnoyes: if you cover everything in your WIP, I'll just abandon mine | 16:44 |
stvnoyes | kk | 16:44 |
johnthetubaguy | yeah, I think stvnoyes has got it covered | 16:44 |
*** MarkBaker has joined #openstack-meeting-cp | 16:46 | |
ildikov | stvnoyes: abandoned now, sorry for the confusion | 16:46 |
johnthetubaguy | cool, are we all done for now? | 16:47 |
stvnoyes | ildikov: np, i should have noticed it before | 16:47 |
ildikov | stvnoyes: it's ok, it was easy to take care of :) | 16:47 |
ildikov | I don't have more for today | 16:48 |
ildikov | the main focus is live migrate with detach, re-write swap and make attach PoC work again with Cinder v3 to be able to test | 16:48 |
ildikov | and as the last bit of the summary get the patches with +2 merged | 16:48 |
ildikov | did I miss anything? | 16:49 |
johnthetubaguy | sounds good | 16:49 |
ildikov | johnthetubaguy: cool, tnx | 16:49 |
ildikov | does anyone have questions to any of the above items or else? | 16:50 |
ildikov | ok, I think we're good for today :) | 16:51 |
ildikov | thank you all! | 16:52 |
ildikov | #endmeeting | 16:52 |
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 16:52 | |
openstack | Meeting ended Thu May 18 16:52:29 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:52 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-05-18-16.00.html | 16:52 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-05-18-16.00.txt | 16:52 |
openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-05-18-16.00.log.html | 16:52 |
*** harlowja has joined #openstack-meeting-cp | 17:30 | |
*** harlowja has quit IRC | 18:01 | |
*** harlowja has joined #openstack-meeting-cp | 18:15 | |
*** rakhmerov has quit IRC | 18:17 | |
*** IgorYozhikov has quit IRC | 18:19 | |
*** markvoelker has quit IRC | 18:19 | |
*** markvoelker has joined #openstack-meeting-cp | 18:20 | |
*** markvoelker has quit IRC | 18:24 | |
*** lamt has quit IRC | 18:39 | |
*** lamt has joined #openstack-meeting-cp | 18:42 | |
*** rakhmerov has joined #openstack-meeting-cp | 18:44 | |
*** IgorYozhikov has joined #openstack-meeting-cp | 18:47 | |
*** harlowja has quit IRC | 18:56 | |
*** harlowja has joined #openstack-meeting-cp | 19:07 | |
*** gouthamr has quit IRC | 19:37 | |
*** lamt has quit IRC | 19:49 | |
*** lamt has joined #openstack-meeting-cp | 19:51 | |
*** gouthamr has joined #openstack-meeting-cp | 19:56 | |
*** gouthamr has quit IRC | 19:57 | |
*** harlowja has quit IRC | 20:03 | |
*** rakhmerov has quit IRC | 20:20 | |
*** rakhmerov has joined #openstack-meeting-cp | 20:21 | |
*** gouthamr has joined #openstack-meeting-cp | 20:29 | |
*** harlowja has joined #openstack-meeting-cp | 20:30 | |
*** harlowja has quit IRC | 20:39 | |
*** rakhmerov has quit IRC | 20:56 | |
*** rakhmerov has joined #openstack-meeting-cp | 21:00 | |
*** sdague has quit IRC | 21:06 | |
*** jhesketh_ has joined #openstack-meeting-cp | 21:14 | |
*** wko_ has joined #openstack-meeting-cp | 21:18 | |
*** wko_ has left #openstack-meeting-cp | 21:18 | |
*** wko_ has joined #openstack-meeting-cp | 21:18 | |
*** jhesketh has quit IRC | 21:18 | |
*** knikolla has quit IRC | 21:18 | |
*** jaugustine has quit IRC | 21:30 | |
*** edmondsw_ has joined #openstack-meeting-cp | 21:36 | |
*** gouthamr has quit IRC | 21:37 | |
*** edmondsw has quit IRC | 21:38 | |
*** edmondsw_ has quit IRC | 22:05 | |
*** edmondsw has joined #openstack-meeting-cp | 22:06 | |
*** edmondsw_ has joined #openstack-meeting-cp | 22:07 | |
*** edmondsw has quit IRC | 22:10 | |
*** edmondsw_ has quit IRC | 22:11 | |
*** wko_ has left #openstack-meeting-cp | 22:14 | |
*** diablo_rojo has quit IRC | 22:27 | |
*** gouthamr has joined #openstack-meeting-cp | 23:01 | |
*** gouthamr has quit IRC | 23:13 | |
*** lamt has quit IRC | 23:14 | |
*** lamt has joined #openstack-meeting-cp | 23:14 | |
*** lamt has quit IRC | 23:22 | |
*** felipemonteiro__ has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!