Thursday, 2017-07-20

*** edmondsw has quit IRC00:03
*** edmondsw has joined #openstack-meeting-cp00:37
*** edmondsw has quit IRC00:41
*** xyang1 has quit IRC01:12
*** diablo_rojo has quit IRC01:26
*** edmondsw has joined #openstack-meeting-cp01:31
*** harlowja has quit IRC01:32
*** edmondsw has quit IRC01:35
*** edmondsw has joined #openstack-meeting-cp01:51
*** edmondsw has quit IRC01:56
*** edmondsw has joined #openstack-meeting-cp02:12
*** edmondsw has quit IRC02:16
*** iyamahat has quit IRC02:17
*** yamahata has quit IRC02:17
*** harlowja has joined #openstack-meeting-cp02:52
*** edmondsw has joined #openstack-meeting-cp02:54
*** edmondsw has quit IRC02:58
*** nhelgeson has quit IRC02:59
*** yamahata has joined #openstack-meeting-cp03:08
*** markvoelker has quit IRC03:09
*** edmondsw has joined #openstack-meeting-cp03:10
*** edmondsw has quit IRC03:14
*** edmondsw has joined #openstack-meeting-cp03:25
*** edmondsw has quit IRC03:30
*** amrith has quit IRC03:45
*** amrith has joined #openstack-meeting-cp03:46
*** Guest85080 has joined #openstack-meeting-cp04:02
*** edmondsw has joined #openstack-meeting-cp04:05
*** Rockyg has joined #openstack-meeting-cp04:07
*** edmondsw has quit IRC04:09
*** amrith is now known as Guest7583404:12
*** Guest85080 is now known as amrith04:12
*** amrith- has joined #openstack-meeting-cp04:12
*** amrith has joined #openstack-meeting-cp04:12
*** amrith has quit IRC04:13
*** amrith- is now known as amrith04:13
*** amrith- has joined #openstack-meeting-cp04:13
*** amrith is now known as Guest3066404:13
*** Guest30664 has quit IRC04:13
*** amrith- is now known as amrith04:13
*** amrith- has joined #openstack-meeting-cp04:14
*** amrith has quit IRC04:15
*** amrith- is now known as amrith04:16
*** harlowja has quit IRC04:18
*** edmondsw has joined #openstack-meeting-cp04:20
*** edmondsw has quit IRC04:25
*** lbragstad has quit IRC04:25
*** harlowja has joined #openstack-meeting-cp04:30
*** edmondsw has joined #openstack-meeting-cp04:36
*** edmondsw has quit IRC04:40
*** amrith has quit IRC04:52
*** amrith has joined #openstack-meeting-cp04:53
*** markvoelker has joined #openstack-meeting-cp05:10
*** diablo_rojo has joined #openstack-meeting-cp05:16
*** harlowja has quit IRC05:33
*** edmondsw has joined #openstack-meeting-cp05:43
*** gouthamr has quit IRC05:43
*** markvoelker has quit IRC05:44
*** edmondsw has quit IRC05:47
*** MarkBaker has quit IRC05:52
*** edmondsw has joined #openstack-meeting-cp05:59
*** markvoelker has joined #openstack-meeting-cp06:01
*** edmondsw has quit IRC06:03
*** markvoelker has quit IRC06:06
*** edmondsw has joined #openstack-meeting-cp06:46
*** MarkBaker has joined #openstack-meeting-cp06:49
*** edmondsw has quit IRC06:51
*** markvoelker has joined #openstack-meeting-cp07:02
*** diablo_rojo has quit IRC07:07
*** rarcea has joined #openstack-meeting-cp07:11
*** rarcea has quit IRC07:20
*** Rockyg has quit IRC07:24
*** rarcea has joined #openstack-meeting-cp07:27
*** edmondsw has joined #openstack-meeting-cp07:30
*** edmondsw has quit IRC07:34
*** markvoelker has quit IRC07:36
*** edmondsw has joined #openstack-meeting-cp07:50
*** edmondsw has quit IRC07:55
*** edmondsw has joined #openstack-meeting-cp08:03
*** edmondsw has quit IRC08:07
*** markvoelker has joined #openstack-meeting-cp08:33
*** rarcea has quit IRC08:34
*** rarcea has joined #openstack-meeting-cp08:34
*** rarcea has quit IRC08:35
*** rarcea has joined #openstack-meeting-cp08:35
*** edmondsw has joined #openstack-meeting-cp08:39
*** edmondsw has quit IRC08:43
*** edmondsw has joined #openstack-meeting-cp08:51
*** edmondsw has quit IRC08:55
*** MarkBaker has quit IRC08:56
*** coolsvap has joined #openstack-meeting-cp09:05
*** markvoelker has quit IRC09:07
*** MarkBaker has joined #openstack-meeting-cp09:09
*** yamahata has quit IRC09:51
*** iyamahat has joined #openstack-meeting-cp09:56
*** iyamahat has quit IRC09:56
*** markvoelker has joined #openstack-meeting-cp10:03
*** edmondsw has joined #openstack-meeting-cp10:24
*** edmondsw has quit IRC10:28
*** sdague has joined #openstack-meeting-cp10:34
*** markvoelker has quit IRC10:37
*** edmondsw has joined #openstack-meeting-cp10:52
*** edmondsw has quit IRC10:56
*** edmondsw has joined #openstack-meeting-cp11:04
*** edmondsw has quit IRC11:09
*** markvoelker has joined #openstack-meeting-cp11:11
*** edmondsw has joined #openstack-meeting-cp11:24
*** edmondsw has quit IRC11:28
*** edmondsw has joined #openstack-meeting-cp12:25
*** markvoelker has quit IRC12:40
*** markvoelker has joined #openstack-meeting-cp12:52
*** xyang1 has joined #openstack-meeting-cp13:12
*** gouthamr has joined #openstack-meeting-cp13:17
*** MarkBaker has quit IRC13:23
*** lbragstad has joined #openstack-meeting-cp13:24
*** felipemonteiro has joined #openstack-meeting-cp13:43
*** felipemonteiro_ has joined #openstack-meeting-cp13:44
*** felipemonteiro has quit IRC13:49
*** MarkBaker has joined #openstack-meeting-cp13:49
*** MarkBaker has quit IRC13:55
*** MarkBaker has joined #openstack-meeting-cp14:16
*** nhelgeson has joined #openstack-meeting-cp15:00
*** MarkBaker has quit IRC15:08
*** felipemonteiro_ has quit IRC15:11
*** MarkBaker has joined #openstack-meeting-cp15:12
*** Rockyg has joined #openstack-meeting-cp15:38
*** felipemonteiro has joined #openstack-meeting-cp15:43
*** felipemonteiro_ has joined #openstack-meeting-cp15:44
*** felipemonteiro has quit IRC15:48
*** coolsvap has quit IRC15:54
*** MarkBaker has quit IRC15:55
*** mgiles has joined #openstack-meeting-cp15:57
*** mriedem has joined #openstack-meeting-cp15:59
ildikov#startmeeting cinder-nova-api-changes16:00
openstackMeeting 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
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
mriedemo/16:00
ildikovjohnthetubaguy jaypipes e0ne jgriffith hemna mriedem patrickeast smcginnis diablo_rojo xyang1 raj_singh lyarwood jungleboyj stvnoyes16:00
stvnoyeso/16:00
jungleboyjo/16:00
xyang1Hi16:00
mgileso/16:00
_pewp_hemna ( ´ ▽ ` )ノ16:00
ildikovhi :)16:01
ildikovLet's dive in16:01
ildikovthe current status is here: https://review.openstack.org/#/q/topic:bp/cinder-new-attach-apis16:01
ildikovwe have three patches in Nova to merge16:02
ildikovand a cinderclient issue to sort out16:02
jungleboyj@!16:02
_pewp_jungleboyj |。・ω・|ノ16:02
jungleboyjOh, I didn't know he was in here.  Nice.  :-)16:02
mriedemi 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
mriedemthen i can drop my DNM test patch and we'll still have the testing enforced16:02
ildikovmriedem: sure, I didn't add it because of your DNM patch16:03
mriedemi'll leave a reminder16:03
ildikovwill do after the meeting16:03
ildikovstvnoyes: I saw you put the live_migrate patch to W -116:04
ildikovstvnoyes: what's the issue with that one?16:04
stvnoyesi found an problem in the error path that I've fixed, and I'm updating the unit test now16:05
stvnoyesshould have an update this afternoon16:05
ildikovstvnoyes: ok, sounds good16:06
mriedemstvnoyes: ping me when it's updated16:06
stvnoyesok16:06
mriedemi'll go through that this afternoon16:06
ildikovstvnoyes: anything else to mention about that one here?16:06
stvnoyesno, but we will need that client patch tho.16:06
ildikovso the cinderclient patch we need is this one: https://review.openstack.org/#/c/485198/16:07
ildikovwhich needs this one to land to get merged: https://review.openstack.org/#/c/485197/16:07
stvnoyesi have another nova change in the queue to change a comment that will be incorrect after that cinder change.16:07
ildikovjungleboyj: hemna: if you have time to take a look ^^16:08
mriedemif those cinderclient changes merge today we'll also want to get a release out16:08
hemnaok16:08
ildikovstvnoyes: we need  a client release and version bump for that though16:08
ildikovmriedem: +116:09
hemnaildikov, on their way now.16:09
ildikovhemna: awesome, thanks!16:09
stvnoyesildikov: do you mean for the comment change?16:09
jungleboyjildikov: I will look too.16:09
ildikovmriedem: I will ping smcginnis once these landed16:09
jungleboyjildikov:  ++16:09
jungleboyjhemna:  Ah, good job.16:10
ildikovmriedem: is there anything we could do to get reviews on the remaining two Nova patches?16:10
ildikovmriedem: or basically how do you see the way forward here?16:10
mriedemi plan on focusing on those this afternoon16:10
mriedemthe live migration one has had review from stephenfin16:11
mriedemso one it's updated he might be good with it16:11
mriedem*once16:11
ildikovmriedem: sounds good, I will keep an eye on them16:11
mriedemi think once i'm good with the api change at the top,16:11
mriedemthen i can ask sdague to help review it16:11
ildikovmriedem: yeah, I hoped if you find it ok too then we kinda get there finally with that one16:11
stvnoyesmriedem: i will let you know when I've updated the migration review16:12
ildikovmriedem: ok, that sounds good16:12
mriedemwhich nova change depends on the cinderclient patch?16:12
mriedemor is it just in general?16:12
ildikovmriedem: the API one as that's what's checking the microversion16:12
stvnoyeswithout it, attach is not using the new 3.27 flow16:12
ildikovmriedem: so from testing perspective all of them16:12
mriedemok, that will also likely depend on https://review.openstack.org/#/c/485198/ or the g-r min bump for it16:12
mriedemok16:12
mriedemi'll make another note for that16:12
mriedemstvnoyes: you get the gold star for flushing these out16:13
ildikovyep, exactly, we need those in place16:13
ildikovmriedem: +216:13
ildikovstvnoyes does an awesome job!16:13
ildikovok, 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 up16:14
stvnoyesjust doing my job :-)16:14
*** diablo_rojo has joined #openstack-meeting-cp16:15
ildikovstvnoyes: my statement still stands :)16:15
ildikovmriedem: does my above short summary look ok?16:15
mriedemyes16:16
ildikovcool16:16
ildikovthen I think we're good for today, unless there's anyone with further topics to discuss16:17
ildikovlet's focus on execution16:17
mriedemone issue,16:18
mriedemthe cinderclient change16:18
mriedemnova calls that method get_highest_client_server_version16:18
mriedemso it's going to return a string instead of a float now yes?16:18
ildikovyes16:18
mriedemalthough, i think we're ok16:18
mriedemnova casts to a string16:18
mriedemmax_api_version = cinder_api_versions.APIVersion(str(max_api_version))16:18
ildikovyep, exactly16:18
mriedemcool16:18
ildikovthat's what stvnoyes was mentioning in the sense of adding a comment about it and maybe even remove that cast16:18
stvnoyesi am not going to remove the cast16:19
mriedemyeah don't worry about it16:19
stvnoyesjust change the comment16:19
ildikovbut from operational perspective it still works16:19
ildikovstvnoyes: cool, sounds good16:19
mriedemolder clients would return a float, but whatever16:19
mriedemi wouldn't stress over this16:19
ildikovyeah, true16:19
ildikovcool, so no stress then :)16:20
ildikovanything else from anyone?16:20
mriedemnope16:20
stvnoyesi'm good.16:20
ildikovsounds good16:20
ildikovthanks guys!16:20
jungleboyjThanks for all the work on this.16:21
ildikovlet's keep in sync during this week and next and let's get things done :)16:21
ildikovsee you All around :)16:21
ildikovjungleboyj: +116:21
ildikov#endmeeting16:21
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"16:21
openstackMeeting 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-cp16:21
openstackMinutes:        http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-07-20-16.00.html16:21
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-07-20-16.00.txt16:21
openstackLog:            http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2017/cinder_nova_api_changes.2017-07-20-16.00.log.html16:21
*** david-lyle has quit IRC16:52
*** david-lyle has joined #openstack-meeting-cp16:52
*** felipemonteiro_ has quit IRC17:12
*** felipemonteiro has joined #openstack-meeting-cp17:12
*** felipemonteiro_ has joined #openstack-meeting-cp17:22
*** harlowja has joined #openstack-meeting-cp17:23
*** david-lyle has quit IRC17:26
*** felipemonteiro has quit IRC17:26
*** harlowja has quit IRC17:27
*** breton has quit IRC17:42
*** breton has joined #openstack-meeting-cp17:43
*** yamahata has joined #openstack-meeting-cp17:53
*** david-lyle has joined #openstack-meeting-cp18:03
*** harlowja has joined #openstack-meeting-cp18:06
*** yamahata has quit IRC18:11
*** Rockyg has quit IRC18:12
*** yamahata has joined #openstack-meeting-cp18:47
*** _pewp_ has quit IRC19:51
*** openstack has joined #openstack-meeting-cp19:58
*** ChanServ sets mode: +o openstack19:58
*** _pewp_ has joined #openstack-meeting-cp20:04
*** iyamahat has joined #openstack-meeting-cp20:07
*** iyamahat_ has joined #openstack-meeting-cp20:09
*** iyamahat has quit IRC20:11
*** gouthamr has quit IRC20:14
*** gouthamr has joined #openstack-meeting-cp20:28
*** _pewp_ has quit IRC20:31
*** mgiles has quit IRC20:55
*** gouthamr has quit IRC21:26
*** xyang1 has quit IRC21:56
*** rarcea has quit IRC22:00
*** felipemonteiro_ has quit IRC22:01
*** iyamahat__ has joined #openstack-meeting-cp22:02
*** iyamahat_ has quit IRC22:04
*** iyamahat_ has joined #openstack-meeting-cp22:42
*** iyamahat__ has quit IRC22:44
*** edmondsw has quit IRC23:04
*** edmondsw_ has joined #openstack-meeting-cp23:07
*** gouthamr has joined #openstack-meeting-cp23:08
*** edmondsw_ has quit IRC23:11
*** edmondsw has joined #openstack-meeting-cp23:29
*** edmondsw has quit IRC23:33
*** david-lyle has quit IRC23:54

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