| *** gouthamr has joined #openstack-meeting-cp | 00:03 | |
| *** diablo_rojo has quit IRC | 00:21 | |
| *** ducttape_ has joined #openstack-meeting-cp | 00:34 | |
| *** harlowja has quit IRC | 00:39 | |
| *** ducttape_ has quit IRC | 00:46 | |
| *** ediardo has quit IRC | 00:52 | |
| *** ducttape_ has joined #openstack-meeting-cp | 00:55 | |
| *** ducttape_ has quit IRC | 01:00 | |
| *** ducttape_ has joined #openstack-meeting-cp | 01:09 | |
| *** ducttape_ has quit IRC | 01:25 | |
| *** harlowja has joined #openstack-meeting-cp | 01:26 | |
| *** harlowja has quit IRC | 01:26 | |
| *** ducttape_ has joined #openstack-meeting-cp | 01:34 | |
| *** ediardo has joined #openstack-meeting-cp | 01:34 | |
| *** ducttape_ has quit IRC | 01:37 | |
| *** ducttape_ has joined #openstack-meeting-cp | 02:02 | |
| *** ducttape_ has quit IRC | 02:06 | |
| *** ducttape_ has joined #openstack-meeting-cp | 02:08 | |
| *** ducttape_ has quit IRC | 02:25 | |
| *** ducttape_ has joined #openstack-meeting-cp | 03:11 | |
| *** lamt has joined #openstack-meeting-cp | 03:13 | |
| *** ducttape_ has quit IRC | 03:30 | |
| *** scottda has quit IRC | 03:35 | |
| *** sdague has quit IRC | 03:37 | |
| *** rderose_ has joined #openstack-meeting-cp | 03:48 | |
| *** rderose has quit IRC | 03:51 | |
| *** ducttape_ has joined #openstack-meeting-cp | 04:28 | |
| *** ducttape_ has quit IRC | 04:33 | |
| *** ediardo has quit IRC | 04:42 | |
| *** rderose_ has quit IRC | 04:44 | |
| *** lamt has quit IRC | 04:45 | |
| *** gouthamr has quit IRC | 04:47 | |
| *** ducttape_ has joined #openstack-meeting-cp | 05:58 | |
| *** ducttape_ has quit IRC | 06:02 | |
| *** markvoelker has quit IRC | 08:26 | |
| *** markvoelker has joined #openstack-meeting-cp | 10:27 | |
| *** markvoelker has quit IRC | 10:31 | |
| *** sdague has joined #openstack-meeting-cp | 11:05 | |
| *** scottda has joined #openstack-meeting-cp | 12:00 | |
| *** ducttape_ has joined #openstack-meeting-cp | 12:00 | |
| *** ducttape_ has quit IRC | 12:01 | |
| *** ducttape_ has joined #openstack-meeting-cp | 12:02 | |
| *** markvoelker has joined #openstack-meeting-cp | 12:28 | |
| *** ducttape_ has quit IRC | 12:32 | |
| *** markvoelker has quit IRC | 12:32 | |
| *** gouthamr has joined #openstack-meeting-cp | 12:36 | |
| *** ducttape_ has joined #openstack-meeting-cp | 12:39 | |
| *** lamt has joined #openstack-meeting-cp | 12:41 | |
| *** sdague has quit IRC | 13:03 | |
| *** ducttape_ has quit IRC | 13:05 | |
| *** knangia has quit IRC | 13:21 | |
| *** ducttape_ has joined #openstack-meeting-cp | 13:36 | |
| *** sdague has joined #openstack-meeting-cp | 13:47 | |
| *** ducttape_ has quit IRC | 13:48 | |
| *** lamt has quit IRC | 14:00 | |
| *** ducttape_ has joined #openstack-meeting-cp | 14:03 | |
| *** ducttape_ has quit IRC | 14:05 | |
| *** ducttape_ has joined #openstack-meeting-cp | 14:05 | |
| *** melwitt has quit IRC | 14:05 | |
| *** melwitt has joined #openstack-meeting-cp | 14:05 | |
| *** ducttape_ has quit IRC | 14:05 | |
| *** ducttape_ has joined #openstack-meeting-cp | 14:06 | |
| *** melwitt is now known as Guest48910 | 14:06 | |
| *** ducttape_ has quit IRC | 14:06 | |
| *** ducttape_ has joined #openstack-meeting-cp | 14:07 | |
| *** lbragstad has quit IRC | 14:07 | |
| *** ducttape_ has quit IRC | 14:07 | |
| *** lbragstad has joined #openstack-meeting-cp | 14:08 | |
| *** patrickeast has quit IRC | 14:13 | |
| *** patrickeast has joined #openstack-meeting-cp | 14:16 | |
| *** asingh_ has quit IRC | 14:16 | |
| *** asingh_ has joined #openstack-meeting-cp | 14:17 | |
| *** jaugustine has joined #openstack-meeting-cp | 14:20 | |
| *** markvoelker has joined #openstack-meeting-cp | 14:29 | |
| *** markvoelker has quit IRC | 14:35 | |
| *** diablo_rojo has joined #openstack-meeting-cp | 14:40 | |
| *** ducttape_ has joined #openstack-meeting-cp | 14:41 | |
| *** Guest48910 is now known as melwitt | 14:42 | |
| *** knangia has joined #openstack-meeting-cp | 14:48 | |
| *** markvoelker has joined #openstack-meeting-cp | 14:59 | |
| *** ducttape_ has quit IRC | 15:06 | |
| *** ducttape_ has joined #openstack-meeting-cp | 15:12 | |
| *** ducttape_ has quit IRC | 15:13 | |
| *** sdague has quit IRC | 15:45 | |
| *** mriedem has joined #openstack-meeting-cp | 15:55 | |
| ildikov | #startmeeting cinder-nova-api-changes | 16:00 |
|---|---|---|
| openstack | Meeting started Thu Apr 6 16:00:08 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 |
| * johnthetubaguy lurks with intent | 16:00 | |
| jgriffith | o/ | 16:00 |
| mriedem | o/ | 16:00 |
| ildikov | we have a few folks on vacation this week, so we might not get more people | 16:01 |
| ildikov | we have the detach refactor patch merged now and have the detach PoC up for review using the new detach flow: https://review.openstack.org/#/c/438750/ | 16:02 |
| ildikov | lyarwood has one more refactor patch up to centralize the terminate and detach calls: https://review.openstack.org/#/c/452181/ | 16:03 |
| ildikov | my first question here would be whether that's the direction we want or not? | 16:03 |
| mriedem | no | 16:05 |
| mriedem | well, | 16:05 |
| mriedem | nova.compute.api has never called nova.virt.block_device before | 16:05 |
| mriedem | that's a layer violation | 16:05 |
| mriedem | nova.virt has always been tied more to nova.compute.!api | 16:05 |
| mriedem | even though i see we have nova.virt.hardware in nova.compute.api, gros | 16:05 |
| mriedem | *gross | 16:05 |
| *** mgiles has joined #openstack-meeting-cp | 16:05 | |
| mriedem | idk, at this point i wouldn't hold up the new detach patch for the refactor | 16:06 |
| jgriffith | mriedem that's what I needed to hear ;) | 16:07 |
| mriedem | ideally long term it's all centralized | 16:07 |
| mriedem | because the copy/paste is not fun | 16:07 |
| ildikov | mriedem: fine with me too, at least we don't have to co-maintain a patch chain again which makes me happy at this point :) | 16:07 |
| ildikov | mriedem: +1 | 16:07 |
| mriedem | i feel like we're missing some things in the unit tests on the detach patch | 16:08 |
| jgriffith | mriedem in all honesty there's a bunch of things I'll need to add to that | 16:09 |
| mriedem | heh, yeah, because the tests are all changed to set attachment_id=None | 16:09 |
| jgriffith | unit tests included | 16:09 |
| jgriffith | I've just been out of the loop the past couple weeks | 16:09 |
| mriedem | that's fine | 16:10 |
| ildikov | mriedem: yeah, as a first step we tried to keep the test suite clean and get a general ok to the direction and then finish | 16:10 |
| mriedem | i'll comment when i do a full review | 16:10 |
| ildikov | mriedem: cool, tnx! | 16:10 |
| ildikov | I think we're fine with detach for now unless there's something that seems to be problematic and should be discussed here | 16:11 |
| mriedem | yeah i think that's the goal right now, the detach patch, | 16:12 |
| mriedem | anything else we need to talk about? | 16:12 |
| mriedem | there is something that came up in the spec review, | 16:12 |
| mriedem | if you're not tracking it, | 16:12 |
| mriedem | which is changes to grenade | 16:12 |
| mriedem | to make a volume attachment on the old side and preserve that through to the new side and then detach | 16:13 |
| mriedem | that doesn't exist today | 16:13 |
| mriedem | so that's up for anyone to pick up if they want | 16:13 |
| ildikov | I don't think we went this deep so far on the meetings | 16:14 |
| mriedem | i'm happy to end the meeting :) | 16:14 |
| ildikov | sorry, I'm just thinking about the grenade case | 16:14 |
| mriedem | we're nowhere near needing that yet | 16:14 |
| mriedem | it's just something that could be done in parallel | 16:15 |
| ildikov | ok, then we're on the same page :) | 16:15 |
| ildikov | as until we kind of keep the old attachments and detach them with the old flow we should be ok | 16:15 |
| ildikov | another thing around detach is the remove check_detach patch: https://review.openstack.org/#/c/446671/ | 16:16 |
| ildikov | johnthetubaguy has concerns with that one | 16:17 |
| johnthetubaguy | has he | 16:17 |
| ildikov | johnthetubaguy: you don't have anymore after finding the BDM checks? | 16:17 |
| johnthetubaguy | I am less worried | 16:18 |
| johnthetubaguy | just want to make sure we don't change the API behaviour | 16:18 |
| ildikov | but you're still worried :) | 16:18 |
| johnthetubaguy | around error handling | 16:18 |
| ildikov | I have some extra raise InvalidVolume lines in the patch for that one | 16:19 |
| ildikov | like how we needed those for removing check_attach due to the error translations | 16:20 |
| ildikov | but I can extend the commit message with info if that helps | 16:21 |
| ildikov | johnthetubaguy: ^ ? | 16:23 |
| johnthetubaguy | sorry, distracted | 16:26 |
| johnthetubaguy | I think more context in the commit message would be good | 16:26 |
| ildikov | ok, I will add more info there | 16:27 |
| ildikov | another one that just hangs is the Cinder v2 API deprecation: https://review.openstack.org/#/c/444253/ | 16:28 |
| ildikov | johnthetubaguy: mriedem: ^ | 16:28 |
| ildikov | marginal, but at least small and trivial | 16:29 |
| johnthetubaguy | is that all in the etherpad now? | 16:29 |
| ildikov | johnthetubaguy: I think I added them, but will check | 16:29 |
| mriedem | hmm, i thought we had some plan around when we would deprecate cinder v2 usage | 16:29 |
| mriedem | but i'd have to check my notes | 16:29 |
| mriedem | http://lists.openstack.org/pipermail/openstack-dev/2017-February/113063.html | 16:30 |
| mriedem | "Outside of the session, Sean Dague suggested that Nova actually deprecate it's usage of Cinder v2 starting in Pike too, and make Cinder v3 required for Nova in Queens." | 16:30 |
| mriedem | however, | 16:30 |
| mriedem | that side steps the issue of upgrades | 16:30 |
| ildikov | johnthetubaguy: added the deprecation patch to the etherpad now, check_detach was there already | 16:30 |
| mriedem | and old attachments | 16:30 |
| mriedem | so if we deprecate cinder v2 usage in pike, | 16:31 |
| mriedem | and drop it in queens, | 16:31 |
| mriedem | that means we have to have an upgrade plan to migrate old v2-created attachments to the v3 style | 16:31 |
| mriedem | which we punted on at the ptg | 16:31 |
| mriedem | and said we'd tackle that in queens | 16:31 |
| ildikov | mriedem: but v3 is the same as v2, dropping v2 does not mean we go up to 3.27 immediately | 16:31 |
| ildikov | or I'm missing something | 16:32 |
| mriedem | that's true i guess | 16:32 |
| mriedem | we can still call detach/terminate_connection with v3 | 16:32 |
| mriedem | on the v3.0 microversion yes? | 16:32 |
| ildikov | so until we don't add the microversion detection Nova just uses base v3, which is basically v2 | 16:32 |
| ildikov | mriedem: yes | 16:32 |
| mriedem | alright i feel better about that then | 16:32 |
| ildikov | mriedem: Nova is already configured to use v3 | 16:32 |
| mriedem | i know | 16:32 |
| mriedem | i just worry | 16:32 |
| mriedem | always | 16:33 |
| mriedem | worried | 16:33 |
| ildikov | the catalog_info patch merged and it's the old flow everywhere the old way | 16:33 |
| ildikov | mriedem: you can be relaxed about this one, piece of cake really | 16:33 |
| mriedem | ok i'll take a look | 16:34 |
| ildikov | so don't make me worry about you're worrying too much :) | 16:34 |
| ildikov | mriedem: cool, tnx | 16:34 |
| ildikov | this is mainly it from my side for today | 16:35 |
| mriedem | let's end it | 16:36 |
| ildikov | ok :) | 16:36 |
| ildikov | thanks everyone, let's look into the detach PoC until the next meeting and try to get that done | 16:37 |
| ildikov | I will be in Korea next week, no idea what time the meeting is there, but will try my best | 16:37 |
| ildikov | thank you all! | 16:38 |
| ildikov | have a good day! | 16:38 |
| ildikov | #endmeeting | 16:39 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 16:39 | |
| openstack | Meeting ended Thu Apr 6 16:39:03 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:39 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-04-06-16.00.html | 16:39 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-04-06-16.00.txt | 16:39 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-04-06-16.00.log.html | 16:39 |
| *** mgiles has left #openstack-meeting-cp | 16:40 | |
| *** mriedem has left #openstack-meeting-cp | 16:45 | |
| *** harlowja has joined #openstack-meeting-cp | 17:23 | |
| *** sdague has joined #openstack-meeting-cp | 18:34 | |
| *** Rockyg has joined #openstack-meeting-cp | 18:47 | |
| *** Rockyg has quit IRC | 19:19 | |
| *** openstack has joined #openstack-meeting-cp | 19:45 | |
| *** ChanServ sets mode: +o openstack | 19:45 | |
| robcresswell | #startmeeting keystone_horizon | 20:01 |
| openstack | Meeting started Thu Apr 6 20:01:37 2017 UTC and is due to finish in 60 minutes. The chair is robcresswell. Information about MeetBot at http://wiki.debian.org/MeetBot. | 20:01 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 20:01 |
| *** openstack changes topic to " (Meeting topic: keystone_horizon)" | 20:01 | |
| openstack | The meeting name has been set to 'keystone_horizon' | 20:01 |
| robcresswell | o/ | 20:02 |
| robcresswell | Anyone about for meeting? | 20:02 |
| *** harlowja has quit IRC | 20:03 | |
| *** scottda has quit IRC | 20:05 | |
| *** rockyg has joined #openstack-meeting-cp | 20:05 | |
| david-lyle | o/ | 20:12 |
| robcresswell | \o/ | 20:13 |
| robcresswell | david-lyle: Think it might be worth postponing :) | 20:14 |
| *** zerick has quit IRC | 20:21 | |
| *** zerick has joined #openstack-meeting-cp | 20:21 | |
| david-lyle | yeah, seems that way | 20:24 |
| david-lyle | robcresswell, close the meeting | 20:24 |
| robcresswell | #endmeeting | 20:25 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 20:25 | |
| openstack | Meeting ended Thu Apr 6 20:25:46 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 20:25 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/keystone_horizon/2017/keystone_horizon.2017-04-06-20.01.html | 20:25 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/keystone_horizon/2017/keystone_horizon.2017-04-06-20.01.txt | 20:25 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/keystone_horizon/2017/keystone_horizon.2017-04-06-20.01.log.html | 20:25 |
| *** david-lyle has quit IRC | 20:44 | |
| *** harlowja has joined #openstack-meeting-cp | 21:03 | |
| *** rockyg has quit IRC | 21:05 | |
| *** notmorgan is now known as morgan | 21:08 | |
| *** lamt has joined #openstack-meeting-cp | 21:11 | |
| *** gouthamr has quit IRC | 21:11 | |
| *** gouthamr has joined #openstack-meeting-cp | 21:36 | |
| *** openstack has joined #openstack-meeting-cp | 22:01 | |
| *** ChanServ sets mode: +o openstack | 22:01 | |
| *** jaugustine has quit IRC | 22:48 | |
| *** markvoelker has quit IRC | 23:37 | |
| *** david-lyle has joined #openstack-meeting-cp | 23:48 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!