| *** TaylorH has joined #openstack-meeting-cp | 00:29 | |
| *** TaylorHuang has quit IRC | 00:30 | |
| *** sdake has joined #openstack-meeting-cp | 00:43 | |
| *** sdake has quit IRC | 01:29 | |
| *** sheel has joined #openstack-meeting-cp | 03:15 | |
| *** coolsvap has joined #openstack-meeting-cp | 03:18 | |
| *** coolsvap has quit IRC | 03:40 | |
| *** coolsvap has joined #openstack-meeting-cp | 03:47 | |
| *** coolsvap has quit IRC | 03:52 | |
| *** coolsvap has joined #openstack-meeting-cp | 03:56 | |
| *** markvoelker has joined #openstack-meeting-cp | 04:18 | |
| *** markvoelker has quit IRC | 04:22 | |
| *** coolsvap has quit IRC | 05:12 | |
| *** markvoelker has joined #openstack-meeting-cp | 06:18 | |
| *** markvoelker has quit IRC | 06:23 | |
| *** Niham has joined #openstack-meeting-cp | 06:49 | |
| *** Niham has quit IRC | 07:07 | |
| *** belmoreira has joined #openstack-meeting-cp | 07:43 | |
| *** markvoelker has joined #openstack-meeting-cp | 08:19 | |
| *** markvoelker has quit IRC | 08:24 | |
| *** Niham has joined #openstack-meeting-cp | 09:35 | |
| *** jokke__ is now known as jokke_ | 10:07 | |
| *** markvoelker has joined #openstack-meeting-cp | 10:20 | |
| *** markvoelker has quit IRC | 10:24 | |
| *** coolsvap has joined #openstack-meeting-cp | 10:33 | |
| *** sdague has joined #openstack-meeting-cp | 11:19 | |
| *** markvoelker has joined #openstack-meeting-cp | 12:12 | |
| *** Niham has quit IRC | 12:16 | |
| *** sdague has quit IRC | 12:19 | |
| *** bswartz has quit IRC | 12:26 | |
| *** coolsvap has quit IRC | 12:26 | |
| *** hemna has quit IRC | 12:33 | |
| *** hemna has joined #openstack-meeting-cp | 12:34 | |
| *** sdague has joined #openstack-meeting-cp | 13:06 | |
| *** belmoreira has quit IRC | 13:25 | |
| *** bswartz has joined #openstack-meeting-cp | 13:26 | |
| *** belmoreira has joined #openstack-meeting-cp | 13:30 | |
| *** _sigmavirus24 is now known as sigmavirus24 | 14:00 | |
| *** sigmavirus24 has joined #openstack-meeting-cp | 14:00 | |
| *** sheel has quit IRC | 14:05 | |
| *** ChanServ sets mode: +o openstack | 14:13 | |
| *** belmoreira has quit IRC | 14:46 | |
| *** belmoreira has joined #openstack-meeting-cp | 14:47 | |
| *** uxdanielle has joined #openstack-meeting-cp | 14:57 | |
| *** belmoreira has quit IRC | 15:30 | |
| *** tyr_ has joined #openstack-meeting-cp | 15:39 | |
| *** mriedem has joined #openstack-meeting-cp | 16:58 | |
| ildikov | #startmeeting cinder-nova-api-changes | 17:00 |
|---|---|---|
| openstack | Meeting started Mon Jun 20 17:00:15 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 | 17:00 |
| mriedem | o/ | 17:00 |
| hemna | yough | 17:00 |
| DuncanT | Hi | 17:00 |
| ildikov | hi all :) | 17:01 |
| ildikov | for today I would like to check on the refactoring and testing work | 17:02 |
| ildikov | does anyone have any further topics for today? | 17:02 |
| mriedem | just a reminder that nova's non-priority bp freeze is 6/30 | 17:03 |
| mriedem | i just reviewed jgriffith's nova change | 17:04 |
| ildikov | mriedem: I know, although as the above mentioned items are dependencies I'm less sure we're gonna make it | 17:04 |
| mriedem | https://review.openstack.org/#/c/330285/ | 17:04 |
| mriedem | comments are inline. the dependent cinder change needs a rebase. | 17:04 |
| mriedem | i didn't check previous patch sets in jenkins to see if they passed a tempest run | 17:04 |
| ildikov | mriedem: what's up with those three items that ended up high prio items but they were expected to get finished before the deadline? | 17:04 |
| mriedem | glance v2 is the only priority item that nova has completed already | 17:05 |
| hemna | haven't had a chance to look at the nova patch yet | 17:05 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/priorities/newton-priorities.html | 17:05 |
| * smcginnis is here but not really here | 17:05 | |
| mriedem | so, the nova change didn't ever pass jenkins, | 17:05 |
| mriedem | zuul failed to merge the series for the job run | 17:06 |
| mriedem | which is something i was hitting last week with my large get-me-a-network series, | 17:06 |
| mriedem | rechecking in order usually gets it going, but the cinder dependency for the new APIs is a legit merge conflict | 17:06 |
| mriedem | which is this one https://review.openstack.org/#/c/327408/ | 17:06 |
| hemna | the cross project deps never seem to work from what I've seen | 17:07 |
| ildikov | mriedem: can multiattach come up and considered as some extent high priority? | 17:07 |
| mriedem | i'd rather not get into that right now | 17:07 |
| hemna | ildikov, I think multi-attach is dead unless we get the new cinder API in place no? | 17:07 |
| ildikov | I have an issue with my Devstack env, I wanted to try the patches in action | 17:07 |
| ildikov | hemna: right, I meant it with its dependencies | 17:07 |
| mriedem | so for jgriffith's series we obviously need the api change rebased | 17:08 |
| mriedem | then we can recheck the nova change and see if it passes a jenkins/tempest run | 17:08 |
| hemna | oops yah the cinder patch is merge conflict | 17:08 |
| mriedem | if jgriffith isn't around, can someone do that? | 17:08 |
| hemna | I can ping him today | 17:08 |
| smcginnis | I reminded him. | 17:08 |
| hemna | and see if he's around | 17:08 |
| mriedem | looks like there were some questions in ps3 on the cinder patch that went unanswered also | 17:09 |
| jgriffith | mriedem: I'm back :) | 17:09 |
| jgriffith | mriedem: I'll get on it right now... thanks smcginnis | 17:09 |
| smcginnis | jgriffith: Hey John! | 17:09 |
| jgriffith | smcginnis: 0/ | 17:09 |
| ildikov | jgriffith: we were discussing the Cinder API change patches and got to the merge conflict just now | 17:10 |
| *** diablo_rojo has joined #openstack-meeting-cp | 17:11 | |
| mriedem | right, so next steps are just getting that rebased and have a ci run through it | 17:12 |
| mriedem | jgriffith: i have comments in the nova change too | 17:12 |
| jgriffith | mriedem: :) Yeah, I just sent you a thank you for that on dev channel | 17:12 |
| jgriffith | mriedem: I'll address those items and rebase later today | 17:12 |
| jgriffith | mriedem: Thank you!!!! | 17:12 |
| mriedem | sure :) | 17:12 |
| jgriffith | I've been hoping for some feedback | 17:12 |
| mriedem | ildikov: what's next? | 17:13 |
| * mriedem is hungry | 17:13 | |
| ildikov | I don't know whether we have scottda around to see where we are with the tempest tests | 17:13 |
| mriedem | https://review.openstack.org/#/c/326681/ | 17:14 |
| smcginnis | I got the impression he made some progress with that, but don't know the current state. | 17:14 |
| hemna | jenkins isn't happy w/ it | 17:14 |
| mriedem | has 2 deps | 17:15 |
| ildikov | yeah I saw both | 17:15 |
| mriedem | the devstack one has some +1s https://review.openstack.org/#/c/325895/ | 17:15 |
| mriedem | the other is a project-config change for a new job... | 17:16 |
| mriedem | i'm not totally sure why a new job is needed for multibackend | 17:16 |
| mriedem | can't cinder run normally in the gate with 2 lvm backends as a default in devstack? | 17:17 |
| hemna | looks like the job config is setting up 2 lvm backends | 17:18 |
| hemna | which isn't the default afaik | 17:18 |
| mriedem | can you get the list of backends via the cinder API? | 17:19 |
| mriedem | so it doesn't need to be configured in tempest? | 17:19 |
| hemna | service-list might give you the backends | 17:20 |
| hemna | or pools | 17:20 |
| mriedem | so he's using this backend_names variable in tempest which already exists | 17:20 |
| mriedem | i wonder what jobs are testing that today | 17:21 |
| hemna | http://paste.openstack.org/show/520656/ | 17:21 |
| hemna | doesn't give the backend_name though, just the host | 17:21 |
| mriedem | regular gate doesn't test multi-backend http://logs.openstack.org/73/328673/1/check/gate-tempest-dsvm-full/d2e8a67/console.html.gz#_2016-06-12_13_28_29_392068 | 17:22 |
| hemna | yah that has to be enabled in tempest conf | 17:22 |
| hemna | and specificaly configured afaik | 17:22 |
| mriedem | i just wonder what existing job, if any, is testing this | 17:23 |
| mriedem | so that we don't need https://review.openstack.org/#/c/330678/ | 17:23 |
| mriedem | so let's start getting some action items | 17:24 |
| mriedem | #action jgriffith to cleanup and rebase nova/cinder create/remove attachment API series | 17:24 |
| mriedem | #action scottda et al to figure out if there is an existing CI job that uses cinder multi-backend | 17:24 |
| hemna | sounds good | 17:25 |
| mriedem | -1 on the devstack change https://review.openstack.org/#/c/325895/ | 17:26 |
| mriedem | they are changing some existing behavior | 17:26 |
| mriedem | which if there is a job that relies on those, it's going to break them | 17:26 |
| hemna | I don't think there is a cinder api for fetching the backend name (cinder.conf section name) | 17:27 |
| mriedem | unfortunate | 17:28 |
| hemna | you'd have to extract it from the host name I think | 17:28 |
| mriedem | so a rookie question, | 17:28 |
| mriedem | for retype (volume migration), | 17:28 |
| mriedem | can you have a single backend cinder on 2 hosts and migrate between them? or does it really have to be 2 different cinder backend drivers for the migration/retype? | 17:29 |
| mriedem | and those can live on the same host | 17:29 |
| hemna | a single cinder backend that has 2 different drivers? | 17:29 |
| hemna | you can have volume_backend_name=foo | 17:30 |
| hemna | with 2 different drivers | 17:30 |
| hemna | so yes in that case | 17:30 |
| hemna | 2 different solidfire arrays, 2 different driver instances, both with the same volume_backend_name defined | 17:31 |
| hemna | you could migrate between them | 17:31 |
| hemna | and retype for that matter | 17:31 |
| mriedem | but if i have an lvm and an rbd, | 17:32 |
| mriedem | then 2 different backends? | 17:32 |
| hemna | sure | 17:33 |
| hemna | the part that's confusing is there is a conf entry called "volume_backend_name" | 17:34 |
| hemna | and some people confuse that with the section name in cinder.conf | 17:34 |
| hemna | [3parfc] | 17:34 |
| hemna | driver=.... | 17:34 |
| hemna | volume_backend_name=3parfc (or foo) | 17:34 |
| hemna | the host is created from the section name | 17:34 |
| hemna | the volume types use the volume_backend_name value | 17:35 |
| hemna | and cinder.conf enabled_backends=[list of section names] | 17:36 |
| mriedem | does gate-tempest-dsvm-full-drbd-devstack ever run on anything? | 17:37 |
| hemna | not sure if that helps or answers your question | 17:37 |
| mriedem | that sets CINDER_ENABLED_BACKENDS in devstack | 17:37 |
| hemna | http://logs.openstack.org/85/327285/12/check/gate-tempest-dsvm-full-drbd-devstack-nv/6e0e031/ | 17:38 |
| hemna | I think that's a different test though | 17:38 |
| mriedem | heh, and doesn't run the tests we care about http://logs.openstack.org/85/327285/12/check/gate-tempest-dsvm-full-drbd-devstack-nv/6e0e031/console.html#_2016-06-17_18_41_04_809894 | 17:39 |
| mriedem | even though CINDER_ENABLED_BACKENDS=drbd:drbdmanage is in http://logs.openstack.org/85/327285/12/check/gate-tempest-dsvm-full-drbd-devstack-nv/6e0e031/logs/localrc.txt.gz | 17:40 |
| mriedem | anyway, this is a rabbit hole | 17:40 |
| mriedem | i've posted comments in scott's changes | 17:40 |
| mriedem | what he's really adding is an lvm multibackend job | 17:40 |
| mriedem | to cinder's experimental queue | 17:41 |
| hemna | sure | 17:41 |
| hemna | we can circle back with scottda when he's around | 17:41 |
| mriedem | there is a way to test this before the project-config change is merged | 17:42 |
| mriedem | with a one off devstack change | 17:42 |
| mriedem | that depends on the tempest change | 17:42 |
| mriedem | i don't know if i have the time to tinker with that today though | 17:43 |
| mriedem | anyway, anything else for today? | 17:44 |
| mriedem | i'd like to end early | 17:44 |
| mriedem | ildikov: ^? | 17:44 |
| hemna | I don't have anything | 17:44 |
| ildikov | mriedem: hemna thanks | 17:45 |
| hemna | I haven't had time to look at the check_attach patchset in a week | 17:45 |
| ildikov | I don't have anything more for today | 17:45 |
| hemna | I'd like to get to that soon | 17:45 |
| hemna | but I am training my replacements this week | 17:45 |
| hemna | :( | 17:45 |
| mriedem | boo | 17:45 |
| hemna | and I still don't have a new job. bleh | 17:45 |
| ildikov | hemna: lemme know if you need any help with that | 17:45 |
| ildikov | hemna: I have free time this week | 17:45 |
| mriedem | hemna: tell those a-holes to rebase your change | 17:45 |
| hemna | :) | 17:45 |
| hemna | they don't even know what CI is | 17:45 |
| hemna | *sigh* | 17:45 |
| ildikov | :( :) | 17:45 |
| mriedem | ildikov: have you started looking at what the nova changes would look like after jgriffith's changes? | 17:47 |
| mriedem | with the new APIs? | 17:47 |
| mriedem | i mean for multiattach | 17:47 |
| ildikov | we need to clean up check_attach as well along with the API changes | 17:47 |
| mriedem | because if the plan is to have multiattach build on that, you probably need to start POCing that code | 17:47 |
| smcginnis | hemna: :( | 17:47 |
| ildikov | besides that I think there are mostly the multiattach related changes, like the shareable flag and the version checks, etc | 17:48 |
| ildikov | mriedem: I think we need to refactor the VM actions as well, like live migrate, shelve offload, etc | 17:48 |
| mriedem | ildikov: as part of multiattach? | 17:49 |
| mriedem | or just in general? | 17:49 |
| ildikov | mriedem: johnthetubaguy walked us through those a short while ago and I think we might look into those as well | 17:49 |
| ildikov | mriedem: if we don't want to disable those for multiattach volumes, then I think before | 17:49 |
| ildikov | mriedem: but I might over worry this | 17:50 |
| mriedem | i guess my point is, there are <10 working days left in the schedule before the bp freeze, so i think it's going to be important to figure out the work order of what needs to happen | 17:51 |
| mriedem | and making priorities on what can get done | 17:51 |
| ildikov | mriedem: also I don't think we have too many options besides building on this refactoring work | 17:51 |
| mriedem | i don't want to start talking about exceptions when we don't have a clear vision on a plan | 17:51 |
| ildikov | mriedem: can the refactoring go on after the deadline? | 17:52 |
| mriedem | sure, but then does that mean that for newton, refactoring is the priority and goal | 17:52 |
| mriedem | hemna's change for check_attach could be considered a bug fix | 17:52 |
| mriedem | so i don't have a problem with that | 17:52 |
| hemna | coolio | 17:53 |
| mriedem | i'm pretty sure the nova core team isn't going to be comfortable with mass refactorings of the nova/cinder API interaction after the freeze | 17:53 |
| ildikov | cool, I think we touched on this earlier as well | 17:53 |
| mriedem | it can certainly be hacked on, but that doesn't mean it gets a pass | 17:53 |
| ildikov | mriedem: is there any chance to discuss it on one of the upcoming meetings maybe? | 17:54 |
| *** piet_ has joined #openstack-meeting-cp | 17:54 | |
| mriedem | which upcoming meeting? | 17:54 |
| ildikov | I think this reafctoring work has much value in general not just for multiattach | 17:54 |
| ildikov | I meant a Nova team meeting | 17:54 |
| mriedem | i agree, but that doesn't mean we can just drop it in in n-3 | 17:54 |
| mriedem | feel free to bring it up in a nova meeting | 17:55 |
| ildikov | or just bring it up on the channel when people around | 17:55 |
| hemna | I think we'd have some momentum if the refactoring patches would pass jenkins and show the stuff works | 17:55 |
| ildikov | ok, cool, I will sync up with jgriffith and then bring up the topic | 17:55 |
| ildikov | hemna: +1, agreed | 17:56 |
| hemna | do we have working live migration tests ? | 17:56 |
| mriedem | hemna: for which thing? or just in general? | 17:56 |
| mriedem | basic block migration and i think voume-backed live migration were passing in the experimental queue job last week once we disabled nfs and ceph from that job config | 17:57 |
| hemna | just in general | 17:57 |
| mriedem | live migration meeting is tomorrow, we were going to come back to see how that job was doing | 17:57 |
| mriedem | as in, if the basic config is working, we were going to add the job as non-voting in the nova check queue | 17:57 |
| hemna | ok | 17:58 |
| mriedem | well we know it works, but at least with trusty versions of libvirt there were pretty sporadic failures | 17:58 |
| hemna | I think that job needs to work to tests the refactoring patches | 17:58 |
| mriedem | the job uses xenial now | 17:58 |
| mriedem | hemna: yeah agreed | 17:58 |
| hemna | I'd like to see 3rd party CI using that as well | 17:58 |
| hemna | so we can confirm some segment of the cinder drivers work w/ live migration | 17:59 |
| hemna | but ceph, lvm should work | 17:59 |
| hemna | since those are the reference cinder backends | 17:59 |
| ildikov | hemna: yeap, we will need automated tests as well, not just manual verification | 17:59 |
| mriedem | yeah, we just have to fix the job, some setup broke once moving to xenial | 17:59 |
| mriedem | tdurakov fixed the nfs setup but we kept it disabled for now | 18:00 |
| mriedem | anyway, time is up | 18:00 |
| ildikov | yeap, we have the actions points, let's move forward with those | 18:00 |
| hemna | coolio | 18:00 |
| hemna | thanks guys | 18:00 |
| ildikov | anything urgent we haven't touched today? | 18:00 |
| ildikov | thanks all! | 18:01 |
| *** mriedem has left #openstack-meeting-cp | 18:01 | |
| ildikov | talk in a week the latest | 18:01 |
| ildikov | #endmeeting | 18:01 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 18:01 | |
| openstack | Meeting ended Mon Jun 20 18:01:36 2016 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 18:01 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2016/cinder_nova_api_changes.2016-06-20-17.00.html | 18:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2016/cinder_nova_api_changes.2016-06-20-17.00.txt | 18:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2016/cinder_nova_api_changes.2016-06-20-17.00.log.html | 18:01 |
| *** diablo_rojo has quit IRC | 18:10 | |
| *** diablo_rojo has joined #openstack-meeting-cp | 18:11 | |
| *** tyr_ has quit IRC | 18:25 | |
| *** tyr_ has joined #openstack-meeting-cp | 18:26 | |
| *** piet_ has quit IRC | 19:09 | |
| *** piet has joined #openstack-meeting-cp | 19:15 | |
| *** piet has quit IRC | 19:27 | |
| *** tyr_ has quit IRC | 19:40 | |
| *** tyr_ has joined #openstack-meeting-cp | 19:40 | |
| *** xyang1 has joined #openstack-meeting-cp | 20:03 | |
| *** tyr_ has quit IRC | 20:06 | |
| *** rockyg has joined #openstack-meeting-cp | 21:02 | |
| tonyb | #startmeeting stable | 21:03 |
| openstack | Meeting started Mon Jun 20 21:03:06 2016 UTC and is due to finish in 60 minutes. The chair is tonyb. Information about MeetBot at http://wiki.debian.org/MeetBot. | 21:03 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 21:03 |
| *** openstack changes topic to " (Meeting topic: stable)" | 21:03 | |
| openstack | The meeting name has been set to 'stable' | 21:03 |
| tonyb | #topic roll call | 21:03 |
| *** openstack changes topic to "roll call (Meeting topic: stable)" | 21:03 | |
| tonyb | who's here? | 21:03 |
| sigmavirus24 | tonyb: this is -cp? | 21:03 |
| sigmavirus24 | not -4 | 21:03 |
| tonyb | Gah! | 21:04 |
| tonyb | #endmeeting | 21:04 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 21:04 | |
| openstack | Meeting ended Mon Jun 20 21:04:13 2016 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 21:04 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/stable/2016/stable.2016-06-20-21.03.html | 21:04 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/stable/2016/stable.2016-06-20-21.03.txt | 21:04 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/stable/2016/stable.2016-06-20-21.03.log.html | 21:04 |
| *** diablo_rojo has quit IRC | 21:45 | |
| *** sigmavirus24 is now known as sigmavirus24_awa | 22:04 | |
| *** hemna is now known as hemnafk | 22:34 | |
| *** uxdanielle has quit IRC | 22:47 | |
| *** sdague has quit IRC | 23:00 | |
| *** rockyg has quit IRC | 23:02 | |
| *** sdake has joined #openstack-meeting-cp | 23:38 | |
| *** sdake_ has joined #openstack-meeting-cp | 23:42 | |
| *** sdake has quit IRC | 23:43 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!