| *** ducttape_ has joined #openstack-meeting-cp | 00:01 | |
| *** sdague has quit IRC | 00:02 | |
| *** lamt has quit IRC | 00:26 | |
| *** lamt has joined #openstack-meeting-cp | 00:27 | |
| *** lamt has quit IRC | 00:31 | |
| *** ducttape_ has quit IRC | 01:22 | |
| *** david-lyle has joined #openstack-meeting-cp | 02:06 | |
| *** gouthamr has quit IRC | 02:16 | |
| *** kberger has quit IRC | 02:33 | |
| *** kberger has joined #openstack-meeting-cp | 02:33 | |
| *** ducttape_ has joined #openstack-meeting-cp | 02:38 | |
| *** diablo_rojo_phon has quit IRC | 02:50 | |
| *** eeiden has quit IRC | 02:59 | |
| *** eeiden has joined #openstack-meeting-cp | 03:11 | |
| *** ricolin has joined #openstack-meeting-cp | 03:17 | |
| *** antwash has joined #openstack-meeting-cp | 03:20 | |
| *** antwash has left #openstack-meeting-cp | 03:21 | |
| *** eeiden has quit IRC | 03:29 | |
| *** eeiden has joined #openstack-meeting-cp | 03:30 | |
| *** ducttape_ has quit IRC | 03:44 | |
| *** ducttape_ has joined #openstack-meeting-cp | 03:44 | |
| *** diablo_rojo has quit IRC | 03:53 | |
| *** ducttape_ has quit IRC | 03:56 | |
| *** ducttape_ has joined #openstack-meeting-cp | 04:55 | |
| *** cartik has joined #openstack-meeting-cp | 05:01 | |
| *** cartik has quit IRC | 05:44 | |
| *** cartik has joined #openstack-meeting-cp | 05:50 | |
| *** ducttape_ has quit IRC | 06:02 | |
| *** MarkBaker has quit IRC | 06:22 | |
| *** cartik has quit IRC | 06:26 | |
| *** cartik has joined #openstack-meeting-cp | 06:27 | |
| *** TheJulia_ has joined #openstack-meeting-cp | 06:29 | |
| *** patrickeast_ has joined #openstack-meeting-cp | 06:29 | |
| *** ediardo_ has joined #openstack-meeting-cp | 06:30 | |
| *** kberger has quit IRC | 06:32 | |
| *** ediardo has quit IRC | 06:33 | |
| *** TheJulia has quit IRC | 06:33 | |
| *** patrickeast has quit IRC | 06:33 | |
| *** ediardo_ is now known as ediardo | 06:33 | |
| *** TheJulia_ is now known as TheJulia | 06:33 | |
| *** patrickeast_ is now known as patrickeast | 06:33 | |
| *** ducttape_ has joined #openstack-meeting-cp | 07:03 | |
| *** ruan_20 has quit IRC | 07:08 | |
| *** MarkBaker has joined #openstack-meeting-cp | 07:22 | |
| *** MarkBaker has quit IRC | 07:30 | |
| *** ducttape_ has quit IRC | 08:14 | |
| *** cartik has quit IRC | 08:25 | |
| *** MarkBaker has joined #openstack-meeting-cp | 09:59 | |
| *** MarkBaker has quit IRC | 10:01 | |
| *** david-lyle has quit IRC | 10:02 | |
| *** ducttape_ has joined #openstack-meeting-cp | 10:11 | |
| *** MarkBaker has joined #openstack-meeting-cp | 10:17 | |
| *** ducttape_ has quit IRC | 10:17 | |
| *** ricolin has quit IRC | 10:30 | |
| *** MarkBaker has quit IRC | 11:37 | |
| *** ducttape_ has joined #openstack-meeting-cp | 12:13 | |
| *** ducttape_ has quit IRC | 12:18 | |
| *** sdague has joined #openstack-meeting-cp | 12:45 | |
| *** brault|a_ has joined #openstack-meeting-cp | 12:56 | |
| *** brault|__ has joined #openstack-meeting-cp | 12:58 | |
| *** brault|away has quit IRC | 12:58 | |
| *** ducttape_ has joined #openstack-meeting-cp | 13:02 | |
| *** brault|__ has quit IRC | 13:02 | |
| *** brault|a_ has quit IRC | 13:04 | |
| *** brault|away has joined #openstack-meeting-cp | 13:04 | |
| *** brault|a_ has joined #openstack-meeting-cp | 13:06 | |
| *** brault|__ has joined #openstack-meeting-cp | 13:08 | |
| *** brault|away has quit IRC | 13:08 | |
| *** brault|a_ has quit IRC | 13:11 | |
| *** brault|away has joined #openstack-meeting-cp | 13:11 | |
| *** brault|__ has quit IRC | 13:12 | |
| *** brault|away has quit IRC | 13:15 | |
| *** MarkBaker has joined #openstack-meeting-cp | 13:19 | |
| *** brault|away has joined #openstack-meeting-cp | 13:23 | |
| *** brault|away has quit IRC | 13:28 | |
| *** ducttape_ has quit IRC | 13:36 | |
| *** kberger has joined #openstack-meeting-cp | 13:45 | |
| *** gouthamr has joined #openstack-meeting-cp | 13:56 | |
| *** ducttape_ has joined #openstack-meeting-cp | 14:04 | |
| *** lamt has joined #openstack-meeting-cp | 14:25 | |
| *** lamt has quit IRC | 14:26 | |
| *** brault|away has joined #openstack-meeting-cp | 14:31 | |
| *** brault|away has quit IRC | 14:35 | |
| *** brault|away has joined #openstack-meeting-cp | 14:36 | |
| *** brault|a_ has joined #openstack-meeting-cp | 14:38 | |
| *** brault|away has quit IRC | 14:40 | |
| *** brault|away has joined #openstack-meeting-cp | 14:40 | |
| *** brault|__ has joined #openstack-meeting-cp | 14:42 | |
| *** brault|a_ has quit IRC | 14:42 | |
| *** brault|a_ has joined #openstack-meeting-cp | 14:45 | |
| *** brault|away has quit IRC | 14:45 | |
| *** brault|__ has quit IRC | 14:47 | |
| *** brault|away has joined #openstack-meeting-cp | 14:49 | |
| *** brault|a_ has quit IRC | 14:49 | |
| *** brault|away has quit IRC | 14:54 | |
| *** brault|away has joined #openstack-meeting-cp | 14:55 | |
| *** brault|away has quit IRC | 15:01 | |
| *** brault|away has joined #openstack-meeting-cp | 15:12 | |
| *** rarcea has joined #openstack-meeting-cp | 15:33 | |
| *** edtubill has joined #openstack-meeting-cp | 15:44 | |
| *** rarcea_ has joined #openstack-meeting-cp | 16:13 | |
| *** stvnoyes has quit IRC | 16:15 | |
| *** stvnoyes has joined #openstack-meeting-cp | 16:16 | |
| *** rarcea has quit IRC | 16:16 | |
| *** david-lyle has joined #openstack-meeting-cp | 16:40 | |
| *** MarkBaker has quit IRC | 16:48 | |
| *** breitz has joined #openstack-meeting-cp | 16:58 | |
| *** mriedem1 has joined #openstack-meeting-cp | 16:59 | |
| ildikov | #startmeeting cinder-nova-api-changes | 17:00 |
|---|---|---|
| openstack | Meeting started Thu Feb 9 17:00:07 2017 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 |
| *** david-lyle has quit IRC | 17:00 | |
| ildikov | scottda 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 |
| scottda | hey | 17:00 |
| smcginnis | o/ | 17:00 |
| ildikov | hi :) | 17:00 |
| breitz | o/ | 17:01 |
| * johnthetubaguy lurks | 17:01 | |
| ildikov | let's wait a bit before starting | 17:01 |
| mriedem1 | o/ | 17:01 |
| hemna | mep | 17:02 |
| ildikov | ok, let's deep dive in the today's meeting | 17:03 |
| smcginnis | jgriffith: metadata metadata metadata | 17:03 |
| ildikov | I would like to touch on the items we might be able to get done before the PTG and also PTG planning as mriedem1 was suggesting too earlier today | 17:04 |
| *** mriedem1 is now known as mriedem | 17:04 | |
| ildikov | we have a patch up for review in devstack-gate that will allow us to start testing Cinder V3 in the placement API job in Nova | 17:04 |
| mriedem | link? | 17:05 |
| ildikov | #link https://review.openstack.org/#/c/326585/ | 17:05 |
| mriedem | oh that | 17:05 |
| ildikov | mriedem: yep, this is the dependency one on the infra side | 17:06 |
| mriedem | yeah i was expecting to see some DNM hack patch to do some immediate testing | 17:06 |
| ildikov | mriedem: I think we don't have anything further prepared yet. Do you think we should try to put together something before this one gets merged? | 17:06 |
| mriedem | i'm also wondering what the project-config change looks like, because we could stack that on top of this in d-g to hack it out for now | 17:06 |
| mriedem | because then the cinder v3 in nova patch can depend on that for testing | 17:07 |
| jgriffith | smcginnis POOF@ | 17:07 |
| smcginnis | jgriffith: ;) | 17:07 |
| mriedem | sdague: is localrc_set in devstack-tools? | 17:08 |
| ildikov | mriedem: I would guess we need to set the catalog_info to V3 at some point | 17:08 |
| sdague | mriedem: currently, it's using a devstack shell function | 17:08 |
| ildikov | mriedem: scottda and jgriffith were playing with Tempest tests more recently | 17:08 |
| sdague | but the theory is once we're running everything through that | 17:09 |
| sdague | we can flip that to dstools | 17:09 |
| sdague | so it will be doing raw local.conf sets | 17:09 |
| smcginnis | breitz got the tests to pass as well. | 17:09 |
| sdague | also, https://review.openstack.org/#/c/430368/ for local.conf pass through from job definitions | 17:09 |
| mriedem | ok i'll need my hand held on that one | 17:11 |
| mriedem | we'll add the local_conf builder to the placement job definition | 17:11 |
| ildikov | smcginnis: breitz: great news! :) | 17:11 |
| mriedem | but then i'm not sure what after that | 17:11 |
| ildikov | mriedem: you mean from config perspective? | 17:12 |
| mriedem | yes | 17:12 |
| mriedem | well, from job template in p-c | 17:12 |
| mriedem | i'll ask in the review | 17:12 |
| *** ducttape_ has quit IRC | 17:12 | |
| ildikov | sdague: thanks for the update | 17:12 |
| ildikov | mriedem: I'm not sure at this point either, I couldn't even successfully point Nova to Cinder V3 yet in my local Tempest tests... :/ | 17:13 |
| scottda | ildikov: I'd be glad to help with that. | 17:14 |
| mriedem | sdague: left my question on usage in https://review.openstack.org/#/c/430368/ | 17:14 |
| scottda | ildikov: I just used cinder v3 in nova.conf... | 17:14 |
| scottda | ildikov: And ran tempest.scenario | 17:14 |
| mriedem | maybe that can be worked into the commit message or a comment on how to use the builder or something | 17:14 |
| ildikov | scottda: I added that there too, but according to the logs it didn't pick that up for some reason as the URL wasn't V3 | 17:14 |
| mriedem | so what are the tempest tests? | 17:15 |
| mriedem | this is just normal tempest with cinder v3 right? | 17:15 |
| ildikov | scottda: I will double check the config | 17:15 |
| scottda | mriedem: Yes, just tempest/scenario tests | 17:15 |
| ildikov | mriedem: as a first step I think we would like to test V3 overall | 17:16 |
| ildikov | mriedem: as we don't have anything in Nova yet to use the new attach/detach API that will be a next step | 17:16 |
| smcginnis | ildikov: +1 | 17:17 |
| smcginnis | Then we need to figure out the best way to handle the workflow branching based on the available microversion I think. | 17:17 |
| mriedem | so, https://review.openstack.org/#/c/420201/ | 17:17 |
| mriedem | can't we create a devstack DNM patch that configures nova for cinder v3 and then depend on that nova change to test it out? | 17:17 |
| *** lamt has joined #openstack-meeting-cp | 17:18 | |
| mriedem | or stack a nova patch on top that depends on a devstack patch that configures nova for cinder v3? | 17:18 |
| mriedem | or is this being tested some other way? | 17:18 |
| scottda | mriedem: Sure, I'll add a DNM patch as a dependency | 17:18 |
| mriedem | also, https://review.openstack.org/#/c/425785/ is unreleased so i don't know how you're testing it without using LIBS_FROM_GIT | 17:18 |
| scottda | mriedem: It's not tested in the gate, just manually. | 17:19 |
| mriedem | so locally | 17:19 |
| mriedem | ok | 17:19 |
| mriedem | i can also poke at that | 17:19 |
| mriedem | so i think a good goal by the PTG is have this at least working with some DNM patch dependencies since we don't have the local.conf stuff in yet | 17:20 |
| mriedem | "this" meaning the nova cinder v3 patch | 17:20 |
| scottda | mriedem: The lingering question that smcginnis pointed out it how to get cinder api version info and store/use in nova code. | 17:20 |
| scottda | s/it/is | 17:20 |
| scottda | One example being that ^^^ POC | 17:20 |
| smcginnis | I think we can hack a proof of concept, but it will be up to the nova team to decide how best to integrate that. | 17:21 |
| mriedem | we store it in the client i think | 17:21 |
| ildikov | just wanted to ask whether we have a good understanding on how to handle the Cinder microversions in Nova? | 17:21 |
| mriedem | that's what we do for glance | 17:21 |
| mriedem | well glance is different since it's config-based right now | 17:21 |
| mriedem | so you're either using glance v1 or v2 (v2 at this point) | 17:21 |
| mriedem | and we have a client for each | 17:21 |
| mriedem | i think glance v2 has something like microversions, but nova only ever cares about base v2 | 17:22 |
| mriedem | so i imagine we'll need to handle cinder like how we handle microversions in our placement client | 17:22 |
| mriedem | for our placement client we don't do version discovery today, we just handle the 404 if request fails and fallback to older behavior | 17:23 |
| mriedem | at some point we need to do version discovery so we don't need to dump errors in the logs | 17:23 |
| mriedem | but e.g. https://github.com/openstack/nova/blob/21e36a2c4c280ff3b1d42b7bee16ffe3f235281d/nova/scheduler/client/report.py#L205 | 17:23 |
| mriedem | https://github.com/openstack/nova/blob/21e36a2c4c280ff3b1d42b7bee16ffe3f235281d/nova/scheduler/client/report.py#L287 | 17:24 |
| mriedem | that's a method in our client that's doing a 1.1 GET request | 17:24 |
| mriedem | if we did version discovery we'd know if we could make that request or not and not log errors on failure | 17:24 |
| mriedem | same for 1.4 https://github.com/openstack/nova/blob/21e36a2c4c280ff3b1d42b7bee16ffe3f235281d/nova/scheduler/client/report.py#L257 | 17:25 |
| johnthetubaguy | we could just hard depend on the v3 version we need right now? | 17:25 |
| johnthetubaguy | maybe thats what you are saying here | 17:25 |
| mriedem | the code that's using that stuff is handling a fallback scenario in ocata | 17:25 |
| *** brault|away has quit IRC | 17:25 | |
| sdague | mriedem: replied on expected usage | 17:25 |
| ildikov | I think the direction with the Cinder client now is to get the max version supported | 17:25 |
| mriedem | ildikov: cinder client in nova, or python-cinderclient/ | 17:25 |
| mriedem | ? | 17:25 |
| scottda | ildikov: That's not hard-coded in the client.. | 17:25 |
| mriedem | whew | 17:25 |
| *** brault|away has joined #openstack-meeting-cp | 17:25 | |
| scottda | ildikov: There's a patch to do that in the CLI | 17:25 |
| ildikov | scottda: I didn't mean it's hardcoded | 17:26 |
| mriedem | the python api binding user of cinderclient needs to opt in | 17:26 |
| mriedem | which is nova in this case | 17:26 |
| mriedem | so i think when we setup the client, like in scott's patch, we get the version and build the Client object that way, and when we do that, we know the highest supported v3 version, right? | 17:26 |
| scottda | mriedem: Yes | 17:27 |
| mriedem | and we know the specific microversions that we need when making requests, 3.27 it looks like | 17:27 |
| johnthetubaguy | so the base v3 micro-version will not work with the current Nova code, thats the thing that surprized me I guess. | 17:27 |
| scottda | mriedem: And then use a method or Global to refer to something like IS_NEW_API_SUPPORTED | 17:27 |
| mriedem | johnthetubaguy: it's not backward compatible? | 17:27 |
| johnthetubaguy | I believe its not | 17:27 |
| ildikov | yep, exactly, so if the microversion is higher than 3.27 we use the new attach/detach flow | 17:27 |
| ildikov | if not than we fall back to the old one | 17:27 |
| scottda | johnthetubaguy: What do you mean? | 17:27 |
| mriedem | scottda: smcginnis: why isn't v3 compatible with v2? | 17:27 |
| scottda | I believe it is | 17:27 |
| scottda | IF I use cinder v3 in nova.conf, old api methods work | 17:28 |
| johnthetubaguy | oh, I thought we said the old binding API would fail on the base v3 version? | 17:28 |
| smcginnis | Yes, base v3 is completely compatible with v2. | 17:28 |
| scottda | and tempest.scenario passes | 17:28 |
| johnthetubaguy | OK, so I was clearly miss understanding what we said before, my mistake | 17:28 |
| ildikov | scottda: what was the solution to the Tempest failures you saw earlier with V3? | 17:28 |
| smcginnis | It's just tricky once we start adding in microversion support and having to do different flows based on that. | 17:28 |
| ildikov | scottda: I'm just curious | 17:28 |
| johnthetubaguy | so we can just follow the usual micro-version patterns it would seem. | 17:29 |
| scottda | ildikov: It was a combination of too much concurency with too small a VOLUME_BACKING_SIZE | 17:29 |
| scottda | and failed for v2 as well as v3 | 17:29 |
| mriedem | right, if we can do 3.27, we do that, if not we fallback to v3 (the old attach/detach) | 17:29 |
| scottda | nothing to do with API version | 17:29 |
| ildikov | scottda: ah, ok | 17:29 |
| *** brault|away has quit IRC | 17:29 | |
| mriedem | ^ is essentially what i was saying we're doing with placement today | 17:29 |
| johnthetubaguy | mriedem: +1 that makes sense to me | 17:30 |
| mriedem | we allow a fallback in ocata even though placement is required, | 17:30 |
| mriedem | however, | 17:30 |
| smcginnis | mriedem: +1 | 17:30 |
| mriedem | in pike i think we plan on dropping the fallback | 17:30 |
| mriedem | so we're going to require a minimum placement microversion in pike, like how the Ironic virt driver requires a minimum Ironic microversion for their code | 17:30 |
| johnthetubaguy | yeah, after a few cycles we drop the fallback, like we did with glance v1 | 17:30 |
| mriedem | johnthetubaguy: yup | 17:30 |
| ildikov | mriedem: although we ask for the version in the current PoC as opposed to fall back in case of errors | 17:30 |
| mriedem | ildikov: we're going to have to fallback to the old way if 3.27 isn't available | 17:31 |
| ildikov | mriedem: yep, we will | 17:31 |
| mriedem | i wouldn't be surprised if v3 itself won't be available | 17:31 |
| mriedem | i.e. all of our grenade jobs | 17:31 |
| mriedem | not available / not configured to use | 17:31 |
| johnthetubaguy | yeah | 17:31 |
| mriedem | because of the endpoint shenanigans you guys love to do | 17:31 |
| scottda | I'm trying to fix that... | 17:32 |
| scottda | We're discussing at the PTG | 17:32 |
| ildikov | mriedem: :) | 17:32 |
| scottda | Consistent Versioned Endpoints. | 17:32 |
| mriedem | scottda: like drop v3 and do it all in v2.x ? | 17:32 |
| scottda | Unfortunately, not possible to schedule a time and place for such discussions... | 17:32 |
| mriedem | or volume endpoint just means, the thing you want | 17:32 |
| scottda | not sure | 17:32 |
| mriedem | scottda: you're not supposed to ask about scheduling things at the ptg | 17:32 |
| mriedem | it will just work | 17:32 |
| scottda | not sure what is possible, without breaking consumers of the catalog | 17:32 |
| scottda | ha | 17:33 |
| mriedem | it would be a monday or tuesday talk, | 17:33 |
| scottda | yes | 17:33 |
| mriedem | probably arch wg or something like that, idk | 17:33 |
| mriedem | there are 20 wg's | 17:33 |
| scottda | yeah, actually on the arch wg etherpad ATM | 17:33 |
| mriedem | api wg | 17:33 |
| mriedem | ok, | 17:33 |
| mriedem | so first things first, | 17:33 |
| mriedem | we get the cinder v3 in nova change passing some testing so we know things are cool | 17:33 |
| mriedem | and worry about the microversion / fallback dance after that | 17:34 |
| mriedem | when we actually need to start requesting 3.27 | 17:34 |
| scottda | #action scottda will post DNM patch for nova to use cinder v3 | 17:34 |
| ildikov | I like this scenario | 17:35 |
| scottda | I think regular tempest/scenario running will then test cinder v3 | 17:35 |
| mriedem | actually isn't https://review.openstack.org/#/c/420201/ self-testing? | 17:35 |
| mriedem | since you're hard-coding the default config? | 17:35 |
| scottda | yah, actually it is. | 17:36 |
| mriedem | and tempest jobs are failing... | 17:36 |
| ildikov | it should be, yes | 17:36 |
| scottda | yeah, I'll have a look at the failures more closely | 17:36 |
| scottda | mriedem: Which tests run tempest/scenario for nova? | 17:37 |
| mriedem | lots | 17:37 |
| scottda | neutron-full? | 17:37 |
| scottda | ok | 17:37 |
| mriedem | test_volume_boot_pattern | 17:37 |
| mriedem | is a big one | 17:37 |
| ildikov | not just Tempest, even pep8 is failing there, so might be something other than V3 | 17:37 |
| scottda | yeah, sorry, I meant which jenkins job | 17:37 |
| mriedem | http://logs.openstack.org/01/420201/7/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/68f7c49/console.html.gz#_2017-02-01_00_55_26_080245 | 17:37 |
| mriedem | ^ is a failure for a known fixed issue | 17:38 |
| mriedem | just recheck your change | 17:38 |
| mriedem | i rechecked it | 17:38 |
| scottda | OK, I'll clean up test failures. That patch is a bit neglected. | 17:38 |
| scottda | http://logs.openstack.org/01/420201/7/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/68f7c49/console.html.gz#_2017-02-01_00_46_19_145600 | 17:39 |
| scottda | Seems to be passing | 17:39 |
| scottda | test_volume_boot_pattern | 17:39 |
| *** brault|away has joined #openstack-meeting-cp | 17:42 | |
| smcginnis | mriedem: Did we want to try to set a Nova-Cinder PTG time? | 17:43 |
| mriedem | i guess i never appreciated that we reconstruct the cinderclient on every call | 17:43 |
| smcginnis | Or just trust that it will all come together? | 17:43 |
| mriedem | smcginnis: we want to schedule it | 17:43 |
| scottda | mriedem: I hate that we do that as well | 17:43 |
| mriedem | so wed or thurs | 17:43 |
| smcginnis | I have no set schedule for those days, so really whatever you would like to do, we can schedule around it. | 17:44 |
| mriedem | smcginnis: wed or thurs morning work for you? 2 hour block? | 17:44 |
| mriedem | 10-noon on wed? | 17:44 |
| mriedem | or we could do it wed afternoon | 17:44 |
| smcginnis | I was going to say 9, but yeah, 10 is probably better so everyone gets some coffee in first. | 17:44 |
| mriedem | let the teams flush out whatever else on the first vertical team morning | 17:44 |
| scottda | 10-noon wed will be pretty much first session of the week | 17:44 |
| mriedem | right | 17:45 |
| mriedem | so maybe thursday is better | 17:45 |
| smcginnis | Morning is probably better than after lunch. | 17:45 |
| mriedem | let's do thursday then | 17:45 |
| smcginnis | Thrusday at 10 | 17:45 |
| mriedem | done | 17:45 |
| ildikov | +1 | 17:45 |
| smcginnis | I'll add it to our etherpad. | 17:45 |
| scottda | #info cinder and nova will meet Thurs at 10:00 at the PTG | 17:45 |
| scottda | cinder people in nova room, and nova people in the cinder room | 17:45 |
| scottda | :) | 17:45 |
| johnthetubaguy | oh, its different auth token for every call, as its the user token, so its a different client, I guess | 17:45 |
| ildikov | scottda: :) | 17:46 |
| smcginnis | scottda: The jokes on you - we'll all just be in one big room all week. :P | 17:46 |
| scottda | johnthetubaguy: Yes, but some novaclient calls (like attach) require 4 cinderclient calls. That could use only one instnace of cinderclient | 17:46 |
| scottda | smcginnis: Ha. The Big Tent | 17:47 |
| johnthetubaguy | scottda: oh, in that sense, good point | 17:47 |
| smcginnis | scottda: LOL | 17:47 |
| mriedem | johnthetubaguy: yeah i guess we do the same for neutron | 17:47 |
| * smcginnis pictures a circus tent in a big parking lot in ATL. | 17:47 | |
| mriedem | we don't do that for the placement (scheduler report client) | 17:47 |
| ildikov | smcginnis: LOL | 17:47 |
| smcginnis | OK, so devstack patch in flight for long term, scottda to recheck to get passing v3 results in the short term. | 17:49 |
| hemna | no sliders | 17:49 |
| smcginnis | Proof of concept will be worked for nova side of calling microversions. | 17:49 |
| ildikov | we can use the Cinder-Nova etherpad to draft a few items to talk about at the PTG, along with the Nova spec | 17:49 |
| smcginnis | And possible cleanup of multiple sessions. | 17:49 |
| smcginnis | Anything else we need to figure out today? | 17:49 |
| smcginnis | ildikov: +1 | 17:49 |
| ildikov | mriedem: the remove check_attach patch was hitting a non-related error, Jenkins looks good on it now | 17:50 |
| ildikov | mriedem: johnthetubaguy: what do I need to do to get that thing finally done? :) | 17:50 |
| johnthetubaguy | I should clean up that nova spec | 17:50 |
| johnthetubaguy | or did we have a new one now? | 17:50 |
| smcginnis | johnthetubaguy: Oh right. Not that I've seen. | 17:51 |
| johnthetubaguy | I can get that tidied up | 17:51 |
| ildikov | johnthetubaguy: I don't think we have a new one for Nova | 17:51 |
| johnthetubaguy | the spec | 17:51 |
| mriedem | johnthetubaguy: it's still yours | 17:51 |
| ildikov | johnthetubaguy: we need to get the current one in shape based on the comments from mriedem and others | 17:51 |
| smcginnis | We've at least finalized the Cinder side, so hopefully that helps with the nova spec. | 17:52 |
| ildikov | we also have a PoC patch: https://review.openstack.org/#/c/330285/ | 17:52 |
| ildikov | jgriffith: made the attach/detach parts working | 17:53 |
| smcginnis | #link https://review.openstack.org/#/c/330285/ Nova PoC for new attach calls | 17:53 |
| *** david-lyle has joined #openstack-meeting-cp | 17:53 | |
| ildikov | still have many things to figure out, but at least we have a starting point | 17:53 |
| ildikov | smcginnis: thanks :) | 17:53 |
| smcginnis | ildikov: Just trying to make the meeting logs look pretty. ;) | 17:53 |
| * ildikov is thankful for that :) | 17:54 | |
| smcginnis | OK, I gotta run. Folks on the lunch train are starting to look angry. :) | 17:54 |
| ildikov | johnthetubaguy: so you can look into the PoC above too to see what we hacked so far to make things work | 17:54 |
| ildikov | smcginnis: I think we're mainly good for today | 17:55 |
| ildikov | smcginnis: enjoy lunch :) | 17:55 |
| johnthetubaguy | ildikov: gives me a good excuse to fix that | 17:55 |
| johnthetubaguy | well, I mean look at those PoC patches, both ways | 17:55 |
| ildikov | mriedem: johnthetubaguy: the PoC depends on the Remove check_attach patch as it seemed reasonable to order things | 17:56 |
| * johnthetubaguy nods | 17:56 | |
| ildikov | and rebasing is a nightmare, so I would love to see that merged as it's considered to be a bugfix | 17:56 |
| johnthetubaguy | we are free to merge that now, I think, give we are open for pike | 17:57 |
| ildikov | mriedem: johnthetubaguy: so pretty please review it :) | 17:57 |
| mriedem | umm, | 17:57 |
| ildikov | mriedem: johnthetubaguy: or tell me who from the cores would willing to look into that change | 17:57 |
| mriedem | johnthetubaguy: i'd hold off until we release pike | 17:57 |
| johnthetubaguy | yeah, its going to take a few hours to get my head back into that again | 17:58 |
| mriedem | in case we need to bump the service version for anything in the next week | 17:58 |
| johnthetubaguy | mriedem: yeah, thats true, service version | 17:58 |
| mriedem | s/pike/ocata/ | 17:58 |
| johnthetubaguy | hehe, yeah ocata | 17:58 |
| ildikov | mriedem: ;) | 17:58 |
| mriedem | so basically can't land that until we cut rc2 | 17:58 |
| mriedem | which is 2/16 at the latest | 17:58 |
| johnthetubaguy | yeah | 17:59 |
| ildikov | mriedem: johnthetubaguy: if you could give it a review run earlier rather than later that would still be good | 17:59 |
| johnthetubaguy | we could be ready to +W in the PTG session, I suppose | 17:59 |
| ildikov | so any issues it might has could be fixed by the time it can get merged | 17:59 |
| ildikov | ok, please review it, so I can fix the issues, I don't want to spend time on this at the PTG | 18:00 |
| ildikov | it should be more straightforward than that | 18:00 |
| ildikov | ok, we're out of time and I think we also have plans | 18:01 |
| ildikov | we can double check the progress next week and maybe try to add items to the PTG session agenda so we surely talk about all the items we see important now | 18:02 |
| ildikov | anything else from anyone today? | 18:02 |
| ildikov | alright, thanks everyone | 18:03 |
| ildikov | #endmeeting | 18:04 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 18:04 | |
| openstack | Meeting ended Thu Feb 9 18:04:29 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 18:04 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-02-09-17.00.html | 18:04 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-02-09-17.00.txt | 18:04 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-02-09-17.00.log.html | 18:04 |
| *** ducttape_ has joined #openstack-meeting-cp | 18:04 | |
| *** brault|away has quit IRC | 18:07 | |
| *** david-lyle has quit IRC | 18:45 | |
| *** diablo_rojo has joined #openstack-meeting-cp | 18:47 | |
| *** rocky_g has joined #openstack-meeting-cp | 18:48 | |
| *** ducttape_ has quit IRC | 18:52 | |
| *** mriedem has left #openstack-meeting-cp | 18:55 | |
| *** rocky_g has quit IRC | 19:00 | |
| *** david-lyle has joined #openstack-meeting-cp | 19:15 | |
| *** gouthamr has quit IRC | 19:23 | |
| *** david-lyle has quit IRC | 19:31 | |
| *** rocky_g has joined #openstack-meeting-cp | 19:34 | |
| *** david-lyle has joined #openstack-meeting-cp | 19:34 | |
| *** ducttape_ has joined #openstack-meeting-cp | 19:34 | |
| *** david-lyle has quit IRC | 19:40 | |
| *** david-lyle has joined #openstack-meeting-cp | 19:42 | |
| *** rocky_g has quit IRC | 19:42 | |
| *** rocky_g has joined #openstack-meeting-cp | 19:43 | |
| *** rocky_g has quit IRC | 19:47 | |
| *** rarcea_ has quit IRC | 19:47 | |
| *** rarcea_ has joined #openstack-meeting-cp | 19:48 | |
| *** gouthamr has joined #openstack-meeting-cp | 19:52 | |
| *** rdopiera has joined #openstack-meeting-cp | 19:55 | |
| *** rarcea_ has quit IRC | 20:01 | |
| *** rocky_g has joined #openstack-meeting-cp | 20:04 | |
| *** david-lyle has quit IRC | 20:06 | |
| *** brault|away has joined #openstack-meeting-cp | 20:07 | |
| *** brault|away has quit IRC | 20:12 | |
| *** harlowja_ has joined #openstack-meeting-cp | 20:35 | |
| *** lamt has quit IRC | 20:36 | |
| *** harlowja has quit IRC | 20:37 | |
| *** lamt has joined #openstack-meeting-cp | 20:40 | |
| *** ducttape_ has quit IRC | 20:50 | |
| *** diablo_rojo has quit IRC | 20:52 | |
| *** ducttape_ has joined #openstack-meeting-cp | 20:55 | |
| *** rocky_g has quit IRC | 21:09 | |
| *** gouthamr has quit IRC | 21:38 | |
| *** ducttape_ has quit IRC | 21:40 | |
| *** david-lyle has joined #openstack-meeting-cp | 21:53 | |
| *** harlowja_ has quit IRC | 21:58 | |
| *** david-lyle has quit IRC | 22:03 | |
| *** david-lyle has joined #openstack-meeting-cp | 22:04 | |
| *** harlowja has joined #openstack-meeting-cp | 22:05 | |
| *** ducttape_ has joined #openstack-meeting-cp | 22:10 | |
| *** gouthamr has joined #openstack-meeting-cp | 22:16 | |
| *** david-lyle has quit IRC | 22:20 | |
| *** edtubill has quit IRC | 22:22 | |
| *** david-lyle has joined #openstack-meeting-cp | 22:28 | |
| *** david-lyle has quit IRC | 22:50 | |
| *** david-lyle has joined #openstack-meeting-cp | 22:50 | |
| *** david-lyle_ has joined #openstack-meeting-cp | 23:06 | |
| *** david-lyle has quit IRC | 23:06 | |
| *** david-lyle_ has quit IRC | 23:09 | |
| *** cebreidian has joined #openstack-meeting-cp | 23:10 | |
| *** lamt has quit IRC | 23:19 | |
| *** david-lyle_ has joined #openstack-meeting-cp | 23:33 | |
| *** david-lyle_ has quit IRC | 23:39 | |
| *** david-lyle_ has joined #openstack-meeting-cp | 23:41 | |
| *** david-lyle_ has quit IRC | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!