Thursday, 2017-05-18

*** felipemonteiro has joined #openstack-meeting-cp01:15
*** markvoelker has quit IRC02:08
*** markvoelker has joined #openstack-meeting-cp02:08
*** gouthamr has quit IRC02:47
*** gouthamr has joined #openstack-meeting-cp02:48
*** felipemonteiro has quit IRC03:12
*** felipemonteiro has joined #openstack-meeting-cp03:35
*** markvoelker has quit IRC03:56
*** markvoelker has joined #openstack-meeting-cp03:57
*** markvoelker has quit IRC04:01
*** diablo_rojo has quit IRC04:34
*** lamt has joined #openstack-meeting-cp04:43
*** lamt has quit IRC05:17
*** lamt has joined #openstack-meeting-cp05:29
*** felipemonteiro has quit IRC05:37
*** lamt has quit IRC05:53
*** lamt has joined #openstack-meeting-cp06:12
*** lamt has quit IRC06:26
*** gouthamr has quit IRC06:36
*** MarkBaker has joined #openstack-meeting-cp07:53
*** MarkBaker has quit IRC08:18
*** MarkBaker has joined #openstack-meeting-cp08:43
*** MarkBaker has quit IRC08:48
*** MarkBaker has joined #openstack-meeting-cp09:00
*** MarkBaker has quit IRC09:17
*** MarkBaker has joined #openstack-meeting-cp10:15
*** pewp has quit IRC10:16
*** pewp has joined #openstack-meeting-cp10:17
*** sdague has joined #openstack-meeting-cp11:06
*** edmondsw has joined #openstack-meeting-cp12:03
*** markvoelker has joined #openstack-meeting-cp12:23
*** lamt has joined #openstack-meeting-cp12:52
*** gouthamr has joined #openstack-meeting-cp12:58
*** MarkBaker has quit IRC13:20
*** MarkBaker has joined #openstack-meeting-cp13:33
*** MarkBaker has quit IRC13:46
*** lbragstad has joined #openstack-meeting-cp13:48
*** lamt has quit IRC13:52
*** MarkBaker has joined #openstack-meeting-cp14:01
*** felipemonteiro has joined #openstack-meeting-cp14:06
*** felipemonteiro_ has joined #openstack-meeting-cp14:09
*** felipemonteiro__ has joined #openstack-meeting-cp14:12
*** felipemonteiro has quit IRC14:12
*** felipemonteiro_ has quit IRC14:15
*** diablo_rojo has joined #openstack-meeting-cp14:18
*** jaugustine has joined #openstack-meeting-cp14:42
*** lamt has joined #openstack-meeting-cp15:11
*** MarkBaker has quit IRC16:00
ildikov#startmeeting cinder-nova-api-changes16:00
openstackMeeting 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.16:00
*** openstack changes topic to " (Meeting topic: cinder-nova-api-changes)"16:00
openstackThe meeting name has been set to 'cinder_nova_api_changes'16:00
ildikovDuncanT 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 jungleboyj16:00
jungleboyjo/16:00
ildikovjungleboyj: hi :)16:01
stvnoyeso/16:01
smcginnis./16:01
jungleboyjildikov: Hello.16:01
smcginnisildikov: Back home yet?16:01
ildikovsmcginnis: yep, landed yesterday afternoon16:02
jungleboyjildikov: Glad you made it safe.  Hope it was a good trip.16:02
smcginnisildikov: Did you remember how to get back to your apartment? :D16:02
ildikovsmcginnis: my brain is still useless though, after Summit syndrome...16:02
hemnamep16:02
ildikovsmcginnis: luckily taxi drivers help you out if you know the address :)16:02
jungleboyjildikov: ++16:03
ildikovjungleboyj: yep, it was alright16:03
ildikovso let's get started16:03
ildikovthe main news from last week is that I've never had this many people approaching me about multi-attach ever before16:04
ildikova guy even told me they fixed up my two Nova patches and running it in production on Kilo16:05
jungleboyjildikov:  Wow.16:05
hemnaildikov, w00t16:05
ildikovthe conclusion is that we better get ourselves together and finally finish this thing! :)16:05
* johnthetubaguy wonders in a touch later than planned16:05
ildikovat least that's my conclusion :)16:06
ildikovjohnthetubaguy: no worries, I'm just sharing my experiences from last week16:06
johnthetubaguyyup, lets make it safe before they all do it very unsafe!16:06
johnthetubaguyhonestly its the feature most people ask me about at openstack events too, at least right now16:06
ildikovjohnthetubaguy: big +116:07
ildikovso that said we started to look into swap with jgriffith16:07
hemnahas anyone asked those folks what they plan on doing with it? :)16:08
hemnaand what filesystem they plan on using?16:08
stvnoyesour customers want Oracle RAC16:08
ildikovhemna: I don't remember what the guy said who's running this in production, but he didn't complain it didn't work16:09
ildikovI don't think this is gonna be the first feature doing harm if you don't use it correctly16:10
hemnayah, I'm just curious16:10
hemnaI think most folks are smart and know they can't run ext4fs on it, but some will try.16:10
ildikovI tried to recruit everyone, so I asked them more about how much they can get involved16:10
ildikovI will feel sorry for those ones for sure16:11
johnthetubaguyhemna: I have this crazy clustered thingy is the usual answer I get16:11
hemnaclusterfs!16:11
hemna*magic*16:11
johnthetubaguyits really easy to use this feature badly, I am curious if we want a "read-only" multi-attach mode too16:11
ildikovhemna: also there are cases where they would want to have one instance that writes on the volume and the rest just read16:11
johnthetubaguyildikov: not sure thats always the case, but that sounds like the most common case16:12
* hemna wonders if that even works16:12
ildikovjohnthetubaguy: no, certainly not always16:12
johnthetubaguysome stuff was a simple active/passive HA pair, that both needs storage16:12
hemnaI might try hacking the brick cinderclient extension and see if I can do a multiattach with cinder directly16:12
*** MarkBaker has joined #openstack-meeting-cp16:12
hemnato see if I can test that out at all16:12
ildikovhemna: sounds good16:13
johnthetubaguyhemna: there is a libvirt property you want to set on the connection, to stop the caches doing funky things16:13
johnthetubaguyhemna: I think thats in the attach WIP patch anyways16:13
hemnaok cool.16:13
hemnaand for baremetal ?16:13
ildikovhemna: I can rebase my old Nova patches and enable multi-attach in them again so you could even play with Nova16:13
hemnajust has to be mounted with -o ro ?16:13
smcginnishemna: I think so.16:14
stvnoyesone 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 code16:14
hemnaI'm a little wary that even mounting ext4fs in ro mode will try induce a write somehow.16:14
johnthetubaguyare we getting distracted?16:14
hemnaanyway, ignore me.16:14
stvnoyesthis was not using the new v3 api. just a proto for now. a learning experience.16:14
johnthetubaguywe 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
johnthetubaguyoh wait there was a think about ildikov looking at swap?16:16
johnthetubaguyhow did that go?16:16
ildikovjohnthetubaguy: no, you didn't16:16
stvnoyesi'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
ildikovjohnthetubaguy: stvnoyes works on the live migrate snippets16:17
ildikovjohnthetubaguy: with jgriffith we got to the conclusion of re-writing swap16:17
johnthetubaguystvnoyes: I can take a peak, there was a live-migrate one I voted on recently16:17
stvnoyesgreat thx16:17
johnthetubaguyildikov: 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 idea16:18
ildikovI think the idea is to let Nova do the attach and detach16:18
ildikovand have things there in a clean way as opposed to the current call-back16:19
johnthetubaguyNova does do the attach and detach today16:19
johnthetubaguyoh... API wise16:19
ildikovthere are some attach/detach calls on the Cinder side today too for some corner cases I think16:19
johnthetubaguyI think I see what you mean now16:19
ildikovthat code is a bit confusing...16:20
johnthetubaguynova does detach, then does the call back?16:20
jungleboyjSorry, was pulled off on other things.  The Read Only use case was asked for by several people at the Summit.16:20
ildikovI think Nova does terminate16:20
*** pewp has quit IRC16:20
johnthetubaguythe call back does the crazy volume uuid rename nonsense16:20
ildikovyeah16:20
johnthetubaguyjungleboyj: I kept hearing that too16:20
ildikovthat's the main part16:20
johnthetubaguyildikov: I like the sound of your plan16:20
jungleboyjjohnthetubaguy:  Ok, so not just me.  :-)16:21
johnthetubaguystvnoyes: can I ask about attachment_ids in your live-migrate patch16:21
johnthetubaguyjungleboyj: if I were an operator again, I would only enable the read-only version, basically attach a snapshot or somesuch16:21
stvnoyesjohnthetubaguy: sure16:22
ildikovjohnthetubaguy: 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 started16:22
johnthetubaguystvnoyes: we have a new attachment_id after the attach call in the setup I guess, in pre_live_migration16:22
johnthetubaguyon the destination host, basically16:22
jungleboyjjohnthetubaguy: ++16:22
johnthetubaguybut when in the tidy up we do the _detach call... we need the old attachment_id16:22
johnthetubaguyit doesn't seem like your patch is doing that16:22
johnthetubaguybut I might be missing some magic16:23
stvnoyesah good catch, i'll take a look at that16:23
stvnoyeswe don't delete the first attachment until the migrate is successful, so the orig bdm is still around. but i'll verify16:24
johnthetubaguydo we want a destination_attachment_id in the BDM?16:24
johnthetubaguythat would make it clear16:24
johnthetubaguynot sure what mriedman would think about that mind16:24
ildikovI wanted to ask whether we need to store two attachment_id's16:24
johnthetubaguyI think we have to16:25
johnthetubaguyI mean we can go look it up, but that seems stupid16:25
ildikovyou mean storing or not storing it?16:25
stvnoyesi 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
smcginnisstvnoyes: Like a temporary flag to switch?16:26
johnthetubaguystvnoyes: I think we should do that, get ildikov to rebase her patch?16:26
stvnoyesmaybe or a config option. or even disabled in source.16:26
johnthetubaguywhy not just cherry-pick the attach patch on top for testing?16:27
ildikovjohnthetubaguy: I did the rebase, just couldn't get there yet to spin it up in Devstack and test it16:27
johnthetubaguyildikov: what did the gate say about the rebase?16:27
ildikovjohnthetubaguy: 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 stuff16:27
*** pewp has joined #openstack-meeting-cp16:27
ildikovjohnthetubaguy: 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 tests16:28
johnthetubaguyildikov: I was thinking the functional stuff shoudl run OK16:28
johnthetubaguyah, gotcha16:29
ildikovI know, I'm evil, but just for practicality...16:29
ildikovjohnthetubaguy: I will get myself back on track with that patch and will ping people who're involved, when it seems working16:29
johnthetubaguyall good16:30
ildikova ton of things have changed, since jgriffith and me made that one work for the last time... :(16:30
johnthetubaguystvnoyes: so I added comments on your live-migrate patch16:31
*** MarkBaker has quit IRC16:31
johnthetubaguystvnoyes: do ping me for questions, I know that flow fairly well at the moment16:31
stvnoyesthx16:31
johnthetubaguyI think we should be creating a new attachment in the pre-live-migration stuff16:32
johnthetubaguybefore we do that first attach call16:32
ildikovjohnthetubaguy: did you add a note about whether or not store the old attachment_id in the BDM as well?16:32
johnthetubaguyyeah, I said please store it in the BDM16:32
johnthetubaguyI 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 it16:33
johnthetubaguyanyways, you are totally in the correct bit of code there16:33
ildikovcool, tnx, just would want to have mriedem's eyes on that part as well before ending up in rabbit holes16:35
johnthetubaguyanyways, 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 details16:36
johnthetubaguyI suspect a lot of... bugs to be found in that code, FWIW16:36
smcginnisUndocumented features.16:36
jungleboyjsmcginnis:   He he he.16:37
ildikovjohnthetubaguy: I also wonder whether we can get rid of the second use of initialize_connection there16:37
johnthetubaguywe do that twice? yeah, probably16:37
* johnthetubaguy nods at smcginnis16:37
ildikovI 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
ildikovjohnthetubaguy: yeah, it's used in that flow a couple of times16:38
johnthetubaguyfor live-migration always thinking about plumbing really16:38
johnthetubaguyyou know a hosepipe with one of those shut off values, so when you pull ou the VM you don't get water everywhere, or something16:39
johnthetubaguymy head is a strange place at times16:39
johnthetubaguysounds like everyone has their next steps for the comming week16:40
johnthetubaguyor did we miss something?16:40
stvnoyesjohnthetubaguy: kk will do16:40
ildikovI think we should be fine16:40
ildikovI haven't done my homework so will not finger point at others either16:41
johnthetubaguythats all good16:41
ildikovthanks stvnoyes for taking care of live migrate!16:41
johnthetubaguyits a very scary part of the code, its good to see that starting16:42
ildikovjohnthetubaguy: I will try to ping lyarwood to take a look at those patches with +2's and get them merged sooner rather than later16:42
johnthetubaguycool, more eyes on those would be good16:42
ildikovagreed16:42
stvnoyesis this rv still needed. i noticed after i posted mine. i think it16:42
johnthetubaguysfinucan might be able to +2 them, its worth asking him16:43
stvnoyess redunandant now - https://review.openstack.org/#/c/456988/16:43
johnthetubaguyah, thats the one I head in my head from early in the week16:43
johnthetubaguyyeah16:43
stvnoyes(not typing well, hand in sling)16:43
ildikovstvnoyes: if you cover everything in your WIP, I'll just abandon mine16:44
stvnoyeskk16:44
johnthetubaguyyeah, I think stvnoyes has got it covered16:44
*** MarkBaker has joined #openstack-meeting-cp16:46
ildikovstvnoyes: abandoned now, sorry for the confusion16:46
johnthetubaguycool, are we all done for now?16:47
stvnoyesildikov: np, i should have noticed it before16:47
ildikovstvnoyes: it's ok, it was easy to take care of :)16:47
ildikovI don't have more for today16:48
ildikovthe main focus is live migrate with detach, re-write swap and make attach PoC work again with Cinder v3 to be able to test16:48
ildikovand as the last bit of the summary get the patches with +2 merged16:48
ildikovdid I miss anything?16:49
johnthetubaguysounds good16:49
ildikovjohnthetubaguy: cool, tnx16:49
ildikovdoes anyone have questions to any of the above items or else?16:50
ildikovok, I think we're good for today :)16:51
ildikovthank you all!16:52
ildikov#endmeeting16:52
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"16:52
openstackMeeting ended Thu May 18 16:52:29 2017 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:52
openstackMinutes:        http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-05-18-16.00.html16:52
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-05-18-16.00.txt16:52
openstackLog:            http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-05-18-16.00.log.html16:52
*** harlowja has joined #openstack-meeting-cp17:30
*** harlowja has quit IRC18:01
*** harlowja has joined #openstack-meeting-cp18:15
*** rakhmerov has quit IRC18:17
*** IgorYozhikov has quit IRC18:19
*** markvoelker has quit IRC18:19
*** markvoelker has joined #openstack-meeting-cp18:20
*** markvoelker has quit IRC18:24
*** lamt has quit IRC18:39
*** lamt has joined #openstack-meeting-cp18:42
*** rakhmerov has joined #openstack-meeting-cp18:44
*** IgorYozhikov has joined #openstack-meeting-cp18:47
*** harlowja has quit IRC18:56
*** harlowja has joined #openstack-meeting-cp19:07
*** gouthamr has quit IRC19:37
*** lamt has quit IRC19:49
*** lamt has joined #openstack-meeting-cp19:51
*** gouthamr has joined #openstack-meeting-cp19:56
*** gouthamr has quit IRC19:57
*** harlowja has quit IRC20:03
*** rakhmerov has quit IRC20:20
*** rakhmerov has joined #openstack-meeting-cp20:21
*** gouthamr has joined #openstack-meeting-cp20:29
*** harlowja has joined #openstack-meeting-cp20:30
*** harlowja has quit IRC20:39
*** rakhmerov has quit IRC20:56
*** rakhmerov has joined #openstack-meeting-cp21:00
*** sdague has quit IRC21:06
*** jhesketh_ has joined #openstack-meeting-cp21:14
*** wko_ has joined #openstack-meeting-cp21:18
*** wko_ has left #openstack-meeting-cp21:18
*** wko_ has joined #openstack-meeting-cp21:18
*** jhesketh has quit IRC21:18
*** knikolla has quit IRC21:18
*** jaugustine has quit IRC21:30
*** edmondsw_ has joined #openstack-meeting-cp21:36
*** gouthamr has quit IRC21:37
*** edmondsw has quit IRC21:38
*** edmondsw_ has quit IRC22:05
*** edmondsw has joined #openstack-meeting-cp22:06
*** edmondsw_ has joined #openstack-meeting-cp22:07
*** edmondsw has quit IRC22:10
*** edmondsw_ has quit IRC22:11
*** wko_ has left #openstack-meeting-cp22:14
*** diablo_rojo has quit IRC22:27
*** gouthamr has joined #openstack-meeting-cp23:01
*** gouthamr has quit IRC23:13
*** lamt has quit IRC23:14
*** lamt has joined #openstack-meeting-cp23:14
*** lamt has quit IRC23:22
*** felipemonteiro__ has quit IRC23:59

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!