Thursday, 2018-01-11

*** edmondsw has quit IRC00:00
*** diablo_rojo has quit IRC00:05
*** diablo_rojo has joined #openstack-meeting-cp00:06
*** felipemonteiro has joined #openstack-meeting-cp00:19
*** felipemonteiro has quit IRC01:35
*** edmondsw has joined #openstack-meeting-cp01:44
*** edmondsw has quit IRC01:48
*** markvoelker has joined #openstack-meeting-cp01:50
*** yamahata has joined #openstack-meeting-cp02:18
*** markvoelker has quit IRC02:36
*** yamahata has quit IRC02:38
*** yamahata has joined #openstack-meeting-cp02:39
*** yamahata has quit IRC02:54
*** felipemonteiro has joined #openstack-meeting-cp03:18
*** harlowja_ has quit IRC03:21
*** nhelgeson has quit IRC03:39
*** felipemonteiro has quit IRC03:57
*** felipemonteiro has joined #openstack-meeting-cp04:15
*** felipemonteiro has quit IRC04:39
*** edmondsw has joined #openstack-meeting-cp05:20
*** edmondsw has quit IRC05:24
*** harlowja has joined #openstack-meeting-cp05:29
*** markvoelker has joined #openstack-meeting-cp05:47
*** coolsvap has joined #openstack-meeting-cp05:47
*** markvoelker has quit IRC06:40
*** markvoelker has joined #openstack-meeting-cp06:42
*** harlowja has quit IRC06:47
*** edmondsw has joined #openstack-meeting-cp07:08
*** edmondsw has quit IRC07:13
*** dims_ has quit IRC07:21
*** dims has joined #openstack-meeting-cp07:25
*** edmondsw has joined #openstack-meeting-cp08:56
*** edmondsw has quit IRC09:01
*** markvoelker has quit IRC10:13
*** edmondsw has joined #openstack-meeting-cp10:44
*** edmondsw has quit IRC10:49
*** sdague has joined #openstack-meeting-cp11:39
*** markvoelker has joined #openstack-meeting-cp12:14
*** edmondsw has joined #openstack-meeting-cp12:33
*** edmondsw has quit IRC12:38
*** markvoelker has quit IRC12:48
*** haint has joined #openstack-meeting-cp12:48
*** markvoelker has joined #openstack-meeting-cp13:44
*** markvoelker has quit IRC14:18
*** edmondsw has joined #openstack-meeting-cp14:21
*** edmondsw_ has joined #openstack-meeting-cp14:24
*** edmondsw has quit IRC14:28
*** edmondsw_ is now known as edmondsw14:45
*** zhipeng has joined #openstack-meeting-cp14:54
*** markvoelker has joined #openstack-meeting-cp15:02
*** markvoelker has quit IRC15:02
*** edmondsw has quit IRC15:18
*** markvoelker has joined #openstack-meeting-cp15:24
*** edmondsw has joined #openstack-meeting-cp15:24
*** mriedem has joined #openstack-meeting-cp15:31
*** felipemonteiro has joined #openstack-meeting-cp15:37
*** felipemonteiro_ has joined #openstack-meeting-cp15:38
*** zhipeng has quit IRC15:38
*** felipemonteiro has quit IRC15:43
ildikov#startmeeting cinder-nova-api-changes16:00
openstackMeeting started Thu Jan 11 16:00:06 2018 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 xyang xyang1 raj_singh lyarwood jungleboyj stvnoyes16:00
* johnthetubaguy lurks16:00
stvnoyeso/16:00
ildikovjohnthetubaguy: you can lurk if you're reviewing in the meantime :)16:01
jungleboyj@!16:01
_pewp_jungleboyj ( *՞ਊ՞*)ノ16:01
ildikovso thanks to mriedem among other things we have a thorough tracking etherpad16:01
ildikov#link https://etherpad.openstack.org/p/multi-attach-volume-queens16:01
ildikovwe have the Nova side patches almost good to go16:02
ildikovwe've just talked about swap in the Nova channel as that does not use the attach code in block_device.py hence we need to pass the multiattach info separately in that case16:03
ildikovI believe we agreed it's an easy fix16:03
ildikovotherwise this is the top of the series in Nova: https://review.openstack.org/#/c/271047/16:04
mriedemi've also got the nova-multiattach ci job patch here https://review.openstack.org/#/c/532689/16:04
mriedembut annoyingly,16:04
mriedemi can see it queued up in the experimental queue in zuul,16:04
mriedembut it doesn't seem to run, and zuul didn't post results on it last night16:05
mriedemi know if that's a problem in the patch, or with zuul16:05
mriedem*don't know16:05
mriedemi'm assuming the latter because the gate has been borked all week16:05
ildikovmriedem: if you saw the queues they are in rough shape in general16:05
mriedemyes the constant restarts don't help, but it is what it is,16:05
mriedemthanks intel16:05
ildikovyeah :/16:06
ildikovI did a recheck as well and then realized it won't go anywhere today...16:06
ildikovmriedem: thanks for the CI job!16:06
mriedemstvnoyes: were you still working on adding some tempest patches on top of https://review.openstack.org/#/c/266605/ for resize?16:07
mriedemthe two tempest patches below that are approved, so that's nice16:07
stvnoyesyes, that's what I'm doing today16:07
mriedemok i'll see if the qa people really need me to split up https://review.openstack.org/#/c/266605/2516:08
mriedemre: cinder patches, i commented on https://review.openstack.org/#/c/531569 last night16:09
ildikovit's not that big...16:09
mriedemlooks like jgriffith is going to update that16:09
ildikovhe has a couple of related patches: jgriffith is working on a series of Cinder changes as well: https://review.openstack.org/#/q/topic:bp/multi-attach-v3-attach+(status:open+OR+status:merged)16:09
mriedemso my comments on the policy change was,16:09
mriedem1. should be the same as volume create so non-admins can create a multiattach volume16:10
mriedemotherwise the tempest patch won't work16:10
mriedemi mean we could hack devstack, but it just seems like it should be open to start and ops can disable if they don't want to support it when upgrading16:10
mriedemand 2. the nova spec said we'd have a cinder policy for disabling multiattach+bootable for the bfv case,16:10
mriedemi think that becomes just a check in the set_bootable volume action API or whatever16:11
ildikovyeah, we talked about the bootable case back at the time of the Nova spec16:11
mriedemhttps://github.com/openstack/cinder/blob/master/cinder/api/contrib/volume_actions.py#L36216:12
mriedemit looks like there is no existing policy check on creating bootable volumes16:12
mriedembut it would be easy to add a policy check there for just, if you're setting bootable=True and volume is multiattach=True, check the policy16:12
jungleboyjYes, we agreed it wasn't supported for BFV.16:12
mriedemand default the policy to allow it16:12
mriedemjungleboyj: no we did agree we'd support bfv16:13
mriedemjungleboyj: you might be thinking of pike when we said we wouldn't :)16:13
ildikovjungleboyj: and we add policy to turn it off16:13
jungleboyjmriedem:  What?  I thought we said no multi-attach for BFV?16:13
jungleboyjildikov: Oh, ok.  I am just going to be quiet now.16:13
mriedemwe said nova wouldn't do any bfv checks and just leave it to cinder policy16:13
jungleboyjYou guys are the experts.16:13
mriedemhttps://specs.openstack.org/openstack/nova-specs/specs/queens/approved/cinder-volume-multi-attach.html#rest-api-impact16:14
ildikovjungleboyj: mriedem is right, we said if someone wants to properly shoot themselves in the foot, we gladly support them :)16:14
mriedem"When we enable the feature we will have a ‘multiattach’ policy to enable or disable the operation entirely on the Cinder side as noted above. Read/Only policy is a future work item and out of the scope of this spec."16:14
mriedemThe following policy rules will be added to Cinder:  Enable/Disable multiattach=True Enable/Disable multiattach=True + bootable=True16:14
jungleboyjmriedem:  Ok, that makes sense.16:15
ildikovand the bootable part is missing currently fro the proposed changes16:15
mriedemyeah comments are in the cinder patch, so i think we're good16:17
ildikovmriedem: do you see anything else missing?16:17
ildikovmriedem: or I should continue begging for reviews next week and fix whatever needs to be fixed, while you're out of office and workaholism?16:18
mriedemswap volume is the only thing that came to me late last night16:18
mriedemand the policy thing16:18
mriedemi won't know about other stuff unless / until it comes to me16:18
mriedemi'm hoping that johnthetubaguy and gibi could get a review pass on the nova patches yet this week,16:18
ildikovI remembered the policy one, just wasn't sure what's in the queue jgriffith hasn't uploaded yet16:19
mriedembefore i have to leave16:19
ildikovgibi started already, will continue tomorrow16:19
* johnthetubaguy nods that he hopes to do that ASAP16:19
ildikovhe also helps with getting some use cases from the E/// folks who need this for the PTG16:19
johnthetubaguybit distracted with some ironic pieces, but will carve out time fo rthat16:19
ildikovjohnthetubaguy: let me know if I should talk to your boss about priorities :)16:20
johnthetubaguy:)16:20
ildikovmriedem: I guess you will whip up the swap patch16:20
mriedemyeah16:20
ildikovcool, thanks16:21
ildikovmriedem: as for Tempest tests, are we good on that front including what stvnoyes is working on or there are more cases you would like to see covered?16:21
stvnoyesas I'm adding the resize test, I'll see if I notice anything missing...16:22
mriedemildikov: swap volume needs to be covered as well16:22
stvnoyesok, I can do that next16:22
mriedemjust, whatever the todos are in the tempest patch16:22
ildikovmriedem: stvnoyes: ok, that sounds good16:22
ildikovjungleboyj: when is the freeze for the cinderclient?16:24
jungleboyjLet me double check.  I think we have 2 weeks.16:24
ildikovI mean the lib freeze because of which we had to wait a month last time16:24
smcginnisTwo weeks.16:24
jungleboyjYeah, 1/2616:24
ildikovok, sounds tight but doable16:24
smcginnisos-brick next Thursday, python-cinderclient the following Thursday.16:25
mriedem1/25 is cinderclient freeze16:25
mriedemand FF16:25
ildikovThursday resonated better with me, but oh well16:25
ildikovand that's the 25th, right16:26
ildikovthis is just one of those days when there's not enough caffeine... :/16:26
ildikovok, I think we're good for today?16:27
mriedemso let's end this meeting so you can get some coffee16:27
ildikovmriedem: +1 :)16:27
jungleboyj++ for coffee16:27
ildikovso if anyone finds anything concerning keep in touch on the channel plus here's the tracking etherpad: https://etherpad.openstack.org/p/multi-attach-volume-queens16:28
ildikovthanks everyone!16:28
ildikovmriedem: have a great vacation next week!16:28
jungleboyjildikov:  Thank you and thanks to everyone for continuing to work this!16:28
mriedemthanks16:28
jungleboyjmriedem:  Enjoy.  I hope you get warmer weather than I did in RTP.16:29
ildikov:)16:29
mriedempuerto vallarta16:29
mriedemceviches and sand16:29
jungleboyjmriedem: Nice!16:29
ildikovthat sounds pretty awesome! :)16:29
ildikovand with that16:29
ildikov#endmeeting16:30
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings"16:30
openstackMeeting ended Thu Jan 11 16:30:02 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:30
openstackMinutes:        http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2018/cinder_nova_api_changes.2018-01-11-16.00.html16:30
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2018/cinder_nova_api_changes.2018-01-11-16.00.txt16:30
openstackLog:            http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2018/cinder_nova_api_changes.2018-01-11-16.00.log.html16:30
*** mriedem has left #openstack-meeting-cp16:31
*** felipemonteiro_ has quit IRC16:58
*** iyamahat has joined #openstack-meeting-cp17:03
*** iyamahat has quit IRC17:11
*** coolsvap has quit IRC17:36
*** nhelgeson has joined #openstack-meeting-cp18:14
*** iyamahat has joined #openstack-meeting-cp18:43
*** yamahata has joined #openstack-meeting-cp18:53
*** felipemonteiro has joined #openstack-meeting-cp18:55
*** felipemonteiro_ has joined #openstack-meeting-cp18:56
*** felipemonteiro has quit IRC19:00
*** harlowja has joined #openstack-meeting-cp19:43
*** edmondsw_ has joined #openstack-meeting-cp19:48
*** edmonds__ has joined #openstack-meeting-cp19:48
*** edmondsw has quit IRC19:52
*** edmondsw_ has quit IRC19:52
*** david-lyle has quit IRC21:03
*** david-lyle has joined #openstack-meeting-cp21:04
*** edmonds__ is now known as edmondsw21:07
*** markvoelker has quit IRC22:49
*** markvoelker has joined #openstack-meeting-cp22:49
*** markvoelker has quit IRC22:54
*** edmondsw has quit IRC22:54
*** edmondsw has joined #openstack-meeting-cp22:55
*** felipemonteiro_ has quit IRC22:55
*** markvoelker has joined #openstack-meeting-cp22:56
*** edmondsw has quit IRC22:59
*** markvoelker has quit IRC22:59
*** markvoelker has joined #openstack-meeting-cp23:00
*** markvoelker has quit IRC23:05

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