*** edmondsw has quit IRC | 00:03 | |
*** edmondsw has joined #openstack-meeting-cp | 00:37 | |
*** edmondsw has quit IRC | 00:41 | |
*** xyang1 has quit IRC | 01:12 | |
*** diablo_rojo has quit IRC | 01:26 | |
*** edmondsw has joined #openstack-meeting-cp | 01:31 | |
*** harlowja has quit IRC | 01:32 | |
*** edmondsw has quit IRC | 01:35 | |
*** edmondsw has joined #openstack-meeting-cp | 01:51 | |
*** edmondsw has quit IRC | 01:56 | |
*** edmondsw has joined #openstack-meeting-cp | 02:12 | |
*** edmondsw has quit IRC | 02:16 | |
*** iyamahat has quit IRC | 02:17 | |
*** yamahata has quit IRC | 02:17 | |
*** harlowja has joined #openstack-meeting-cp | 02:52 | |
*** edmondsw has joined #openstack-meeting-cp | 02:54 | |
*** edmondsw has quit IRC | 02:58 | |
*** nhelgeson has quit IRC | 02:59 | |
*** yamahata has joined #openstack-meeting-cp | 03:08 | |
*** markvoelker has quit IRC | 03:09 | |
*** edmondsw has joined #openstack-meeting-cp | 03:10 | |
*** edmondsw has quit IRC | 03:14 | |
*** edmondsw has joined #openstack-meeting-cp | 03:25 | |
*** edmondsw has quit IRC | 03:30 | |
*** amrith has quit IRC | 03:45 | |
*** amrith has joined #openstack-meeting-cp | 03:46 | |
*** Guest85080 has joined #openstack-meeting-cp | 04:02 | |
*** edmondsw has joined #openstack-meeting-cp | 04:05 | |
*** Rockyg has joined #openstack-meeting-cp | 04:07 | |
*** edmondsw has quit IRC | 04:09 | |
*** amrith is now known as Guest75834 | 04:12 | |
*** Guest85080 is now known as amrith | 04:12 | |
*** amrith- has joined #openstack-meeting-cp | 04:12 | |
*** amrith has joined #openstack-meeting-cp | 04:12 | |
*** amrith has quit IRC | 04:13 | |
*** amrith- is now known as amrith | 04:13 | |
*** amrith- has joined #openstack-meeting-cp | 04:13 | |
*** amrith is now known as Guest30664 | 04:13 | |
*** Guest30664 has quit IRC | 04:13 | |
*** amrith- is now known as amrith | 04:13 | |
*** amrith- has joined #openstack-meeting-cp | 04:14 | |
*** amrith has quit IRC | 04:15 | |
*** amrith- is now known as amrith | 04:16 | |
*** harlowja has quit IRC | 04:18 | |
*** edmondsw has joined #openstack-meeting-cp | 04:20 | |
*** edmondsw has quit IRC | 04:25 | |
*** lbragstad has quit IRC | 04:25 | |
*** harlowja has joined #openstack-meeting-cp | 04:30 | |
*** edmondsw has joined #openstack-meeting-cp | 04:36 | |
*** edmondsw has quit IRC | 04:40 | |
*** amrith has quit IRC | 04:52 | |
*** amrith has joined #openstack-meeting-cp | 04:53 | |
*** markvoelker has joined #openstack-meeting-cp | 05:10 | |
*** diablo_rojo has joined #openstack-meeting-cp | 05:16 | |
*** harlowja has quit IRC | 05:33 | |
*** edmondsw has joined #openstack-meeting-cp | 05:43 | |
*** gouthamr has quit IRC | 05:43 | |
*** markvoelker has quit IRC | 05:44 | |
*** edmondsw has quit IRC | 05:47 | |
*** MarkBaker has quit IRC | 05:52 | |
*** edmondsw has joined #openstack-meeting-cp | 05:59 | |
*** markvoelker has joined #openstack-meeting-cp | 06:01 | |
*** edmondsw has quit IRC | 06:03 | |
*** markvoelker has quit IRC | 06:06 | |
*** edmondsw has joined #openstack-meeting-cp | 06:46 | |
*** MarkBaker has joined #openstack-meeting-cp | 06:49 | |
*** edmondsw has quit IRC | 06:51 | |
*** markvoelker has joined #openstack-meeting-cp | 07:02 | |
*** diablo_rojo has quit IRC | 07:07 | |
*** rarcea has joined #openstack-meeting-cp | 07:11 | |
*** rarcea has quit IRC | 07:20 | |
*** Rockyg has quit IRC | 07:24 | |
*** rarcea has joined #openstack-meeting-cp | 07:27 | |
*** edmondsw has joined #openstack-meeting-cp | 07:30 | |
*** edmondsw has quit IRC | 07:34 | |
*** markvoelker has quit IRC | 07:36 | |
*** edmondsw has joined #openstack-meeting-cp | 07:50 | |
*** edmondsw has quit IRC | 07:55 | |
*** edmondsw has joined #openstack-meeting-cp | 08:03 | |
*** edmondsw has quit IRC | 08:07 | |
*** markvoelker has joined #openstack-meeting-cp | 08:33 | |
*** rarcea has quit IRC | 08:34 | |
*** rarcea has joined #openstack-meeting-cp | 08:34 | |
*** rarcea has quit IRC | 08:35 | |
*** rarcea has joined #openstack-meeting-cp | 08:35 | |
*** edmondsw has joined #openstack-meeting-cp | 08:39 | |
*** edmondsw has quit IRC | 08:43 | |
*** edmondsw has joined #openstack-meeting-cp | 08:51 | |
*** edmondsw has quit IRC | 08:55 | |
*** MarkBaker has quit IRC | 08:56 | |
*** coolsvap has joined #openstack-meeting-cp | 09:05 | |
*** markvoelker has quit IRC | 09:07 | |
*** MarkBaker has joined #openstack-meeting-cp | 09:09 | |
*** yamahata has quit IRC | 09:51 | |
*** iyamahat has joined #openstack-meeting-cp | 09:56 | |
*** iyamahat has quit IRC | 09:56 | |
*** markvoelker has joined #openstack-meeting-cp | 10:03 | |
*** edmondsw has joined #openstack-meeting-cp | 10:24 | |
*** edmondsw has quit IRC | 10:28 | |
*** sdague has joined #openstack-meeting-cp | 10:34 | |
*** markvoelker has quit IRC | 10:37 | |
*** edmondsw has joined #openstack-meeting-cp | 10:52 | |
*** edmondsw has quit IRC | 10:56 | |
*** edmondsw has joined #openstack-meeting-cp | 11:04 | |
*** edmondsw has quit IRC | 11:09 | |
*** markvoelker has joined #openstack-meeting-cp | 11:11 | |
*** edmondsw has joined #openstack-meeting-cp | 11:24 | |
*** edmondsw has quit IRC | 11:28 | |
*** edmondsw has joined #openstack-meeting-cp | 12:25 | |
*** markvoelker has quit IRC | 12:40 | |
*** markvoelker has joined #openstack-meeting-cp | 12:52 | |
*** xyang1 has joined #openstack-meeting-cp | 13:12 | |
*** gouthamr has joined #openstack-meeting-cp | 13:17 | |
*** MarkBaker has quit IRC | 13:23 | |
*** lbragstad has joined #openstack-meeting-cp | 13:24 | |
*** felipemonteiro has joined #openstack-meeting-cp | 13:43 | |
*** felipemonteiro_ has joined #openstack-meeting-cp | 13:44 | |
*** felipemonteiro has quit IRC | 13:49 | |
*** MarkBaker has joined #openstack-meeting-cp | 13:49 | |
*** MarkBaker has quit IRC | 13:55 | |
*** MarkBaker has joined #openstack-meeting-cp | 14:16 | |
*** nhelgeson has joined #openstack-meeting-cp | 15:00 | |
*** MarkBaker has quit IRC | 15:08 | |
*** felipemonteiro_ has quit IRC | 15:11 | |
*** MarkBaker has joined #openstack-meeting-cp | 15:12 | |
*** Rockyg has joined #openstack-meeting-cp | 15:38 | |
*** felipemonteiro has joined #openstack-meeting-cp | 15:43 | |
*** felipemonteiro_ has joined #openstack-meeting-cp | 15:44 | |
*** felipemonteiro has quit IRC | 15:48 | |
*** coolsvap has quit IRC | 15:54 | |
*** MarkBaker has quit IRC | 15:55 | |
*** mgiles has joined #openstack-meeting-cp | 15:57 | |
*** mriedem has joined #openstack-meeting-cp | 15:59 | |
ildikov | #startmeeting cinder-nova-api-changes | 16:00 |
---|---|---|
openstack | Meeting started Thu Jul 20 16:00:05 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 |
mriedem | o/ | 16:00 |
ildikov | johnthetubaguy jaypipes e0ne jgriffith hemna mriedem patrickeast smcginnis diablo_rojo xyang1 raj_singh lyarwood jungleboyj stvnoyes | 16:00 |
stvnoyes | o/ | 16:00 |
jungleboyj | o/ | 16:00 |
xyang1 | Hi | 16:00 |
mgiles | o/ | 16:00 |
_pewp_ | hemna ( ´ ▽ ` )ノ | 16:00 |
ildikov | hi :) | 16:01 |
ildikov | Let's dive in | 16:01 |
ildikov | the current status is here: https://review.openstack.org/#/q/topic:bp/cinder-new-attach-apis | 16:01 |
ildikov | we have three patches in Nova to merge | 16:02 |
ildikov | and a cinderclient issue to sort out | 16:02 |
jungleboyj | @! | 16:02 |
_pewp_ | jungleboyj |。・ω・|ノ | 16:02 |
jungleboyj | Oh, I didn't know he was in here. Nice. :-) | 16:02 |
mriedem | i think it would be a good idea if the top level api change patch depends on this grenade change https://review.openstack.org/#/c/484469/ | 16:02 |
mriedem | then i can drop my DNM test patch and we'll still have the testing enforced | 16:02 |
ildikov | mriedem: sure, I didn't add it because of your DNM patch | 16:03 |
mriedem | i'll leave a reminder | 16:03 |
ildikov | will do after the meeting | 16:03 |
ildikov | stvnoyes: I saw you put the live_migrate patch to W -1 | 16:04 |
ildikov | stvnoyes: what's the issue with that one? | 16:04 |
stvnoyes | i found an problem in the error path that I've fixed, and I'm updating the unit test now | 16:05 |
stvnoyes | should have an update this afternoon | 16:05 |
ildikov | stvnoyes: ok, sounds good | 16:06 |
mriedem | stvnoyes: ping me when it's updated | 16:06 |
stvnoyes | ok | 16:06 |
mriedem | i'll go through that this afternoon | 16:06 |
ildikov | stvnoyes: anything else to mention about that one here? | 16:06 |
stvnoyes | no, but we will need that client patch tho. | 16:06 |
ildikov | so the cinderclient patch we need is this one: https://review.openstack.org/#/c/485198/ | 16:07 |
ildikov | which needs this one to land to get merged: https://review.openstack.org/#/c/485197/ | 16:07 |
stvnoyes | i have another nova change in the queue to change a comment that will be incorrect after that cinder change. | 16:07 |
ildikov | jungleboyj: hemna: if you have time to take a look ^^ | 16:08 |
mriedem | if those cinderclient changes merge today we'll also want to get a release out | 16:08 |
hemna | ok | 16:08 |
ildikov | stvnoyes: we need a client release and version bump for that though | 16:08 |
ildikov | mriedem: +1 | 16:09 |
hemna | ildikov, on their way now. | 16:09 |
ildikov | hemna: awesome, thanks! | 16:09 |
stvnoyes | ildikov: do you mean for the comment change? | 16:09 |
jungleboyj | ildikov: I will look too. | 16:09 |
ildikov | mriedem: I will ping smcginnis once these landed | 16:09 |
jungleboyj | ildikov: ++ | 16:09 |
jungleboyj | hemna: Ah, good job. | 16:10 |
ildikov | mriedem: is there anything we could do to get reviews on the remaining two Nova patches? | 16:10 |
ildikov | mriedem: or basically how do you see the way forward here? | 16:10 |
mriedem | i plan on focusing on those this afternoon | 16:10 |
mriedem | the live migration one has had review from stephenfin | 16:11 |
mriedem | so one it's updated he might be good with it | 16:11 |
mriedem | *once | 16:11 |
ildikov | mriedem: sounds good, I will keep an eye on them | 16:11 |
mriedem | i think once i'm good with the api change at the top, | 16:11 |
mriedem | then i can ask sdague to help review it | 16:11 |
ildikov | mriedem: yeah, I hoped if you find it ok too then we kinda get there finally with that one | 16:11 |
stvnoyes | mriedem: i will let you know when I've updated the migration review | 16:12 |
ildikov | mriedem: ok, that sounds good | 16:12 |
mriedem | which nova change depends on the cinderclient patch? | 16:12 |
mriedem | or is it just in general? | 16:12 |
ildikov | mriedem: the API one as that's what's checking the microversion | 16:12 |
stvnoyes | without it, attach is not using the new 3.27 flow | 16:12 |
ildikov | mriedem: so from testing perspective all of them | 16:12 |
mriedem | ok, that will also likely depend on https://review.openstack.org/#/c/485198/ or the g-r min bump for it | 16:12 |
mriedem | ok | 16:12 |
mriedem | i'll make another note for that | 16:12 |
mriedem | stvnoyes: you get the gold star for flushing these out | 16:13 |
ildikov | yep, exactly, we need those in place | 16:13 |
ildikov | mriedem: +2 | 16:13 |
ildikov | stvnoyes does an awesome job! | 16:13 |
ildikov | ok, I think we have a solid plan in the sense of get the cinderclient part sorted out, add the depends-on tag to the grenade test to the API patch and get reviews in once live_migration is fixed up | 16:14 |
stvnoyes | just doing my job :-) | 16:14 |
*** diablo_rojo has joined #openstack-meeting-cp | 16:15 | |
ildikov | stvnoyes: my statement still stands :) | 16:15 |
ildikov | mriedem: does my above short summary look ok? | 16:15 |
mriedem | yes | 16:16 |
ildikov | cool | 16:16 |
ildikov | then I think we're good for today, unless there's anyone with further topics to discuss | 16:17 |
ildikov | let's focus on execution | 16:17 |
mriedem | one issue, | 16:18 |
mriedem | the cinderclient change | 16:18 |
mriedem | nova calls that method get_highest_client_server_version | 16:18 |
mriedem | so it's going to return a string instead of a float now yes? | 16:18 |
ildikov | yes | 16:18 |
mriedem | although, i think we're ok | 16:18 |
mriedem | nova casts to a string | 16:18 |
mriedem | max_api_version = cinder_api_versions.APIVersion(str(max_api_version)) | 16:18 |
ildikov | yep, exactly | 16:18 |
mriedem | cool | 16:18 |
ildikov | that's what stvnoyes was mentioning in the sense of adding a comment about it and maybe even remove that cast | 16:18 |
stvnoyes | i am not going to remove the cast | 16:19 |
mriedem | yeah don't worry about it | 16:19 |
stvnoyes | just change the comment | 16:19 |
ildikov | but from operational perspective it still works | 16:19 |
ildikov | stvnoyes: cool, sounds good | 16:19 |
mriedem | older clients would return a float, but whatever | 16:19 |
mriedem | i wouldn't stress over this | 16:19 |
ildikov | yeah, true | 16:19 |
ildikov | cool, so no stress then :) | 16:20 |
ildikov | anything else from anyone? | 16:20 |
mriedem | nope | 16:20 |
stvnoyes | i'm good. | 16:20 |
ildikov | sounds good | 16:20 |
ildikov | thanks guys! | 16:20 |
jungleboyj | Thanks for all the work on this. | 16:21 |
ildikov | let's keep in sync during this week and next and let's get things done :) | 16:21 |
ildikov | see you All around :) | 16:21 |
ildikov | jungleboyj: +1 | 16:21 |
ildikov | #endmeeting | 16:21 |
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 16:21 | |
openstack | Meeting ended Thu Jul 20 16:21:47 2017 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:21 |
*** mriedem has left #openstack-meeting-cp | 16:21 | |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-07-20-16.00.html | 16:21 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-07-20-16.00.txt | 16:21 |
openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-07-20-16.00.log.html | 16:21 |
*** david-lyle has quit IRC | 16:52 | |
*** david-lyle has joined #openstack-meeting-cp | 16:52 | |
*** felipemonteiro_ has quit IRC | 17:12 | |
*** felipemonteiro has joined #openstack-meeting-cp | 17:12 | |
*** felipemonteiro_ has joined #openstack-meeting-cp | 17:22 | |
*** harlowja has joined #openstack-meeting-cp | 17:23 | |
*** david-lyle has quit IRC | 17:26 | |
*** felipemonteiro has quit IRC | 17:26 | |
*** harlowja has quit IRC | 17:27 | |
*** breton has quit IRC | 17:42 | |
*** breton has joined #openstack-meeting-cp | 17:43 | |
*** yamahata has joined #openstack-meeting-cp | 17:53 | |
*** david-lyle has joined #openstack-meeting-cp | 18:03 | |
*** harlowja has joined #openstack-meeting-cp | 18:06 | |
*** yamahata has quit IRC | 18:11 | |
*** Rockyg has quit IRC | 18:12 | |
*** yamahata has joined #openstack-meeting-cp | 18:47 | |
*** _pewp_ has quit IRC | 19:51 | |
*** openstack has joined #openstack-meeting-cp | 19:58 | |
*** ChanServ sets mode: +o openstack | 19:58 | |
*** _pewp_ has joined #openstack-meeting-cp | 20:04 | |
*** iyamahat has joined #openstack-meeting-cp | 20:07 | |
*** iyamahat_ has joined #openstack-meeting-cp | 20:09 | |
*** iyamahat has quit IRC | 20:11 | |
*** gouthamr has quit IRC | 20:14 | |
*** gouthamr has joined #openstack-meeting-cp | 20:28 | |
*** _pewp_ has quit IRC | 20:31 | |
*** mgiles has quit IRC | 20:55 | |
*** gouthamr has quit IRC | 21:26 | |
*** xyang1 has quit IRC | 21:56 | |
*** rarcea has quit IRC | 22:00 | |
*** felipemonteiro_ has quit IRC | 22:01 | |
*** iyamahat__ has joined #openstack-meeting-cp | 22:02 | |
*** iyamahat_ has quit IRC | 22:04 | |
*** iyamahat_ has joined #openstack-meeting-cp | 22:42 | |
*** iyamahat__ has quit IRC | 22:44 | |
*** edmondsw has quit IRC | 23:04 | |
*** edmondsw_ has joined #openstack-meeting-cp | 23:07 | |
*** gouthamr has joined #openstack-meeting-cp | 23:08 | |
*** edmondsw_ has quit IRC | 23:11 | |
*** edmondsw has joined #openstack-meeting-cp | 23:29 | |
*** edmondsw has quit IRC | 23:33 | |
*** david-lyle has quit IRC | 23:54 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!