*** edmondsw has quit IRC | 00:00 | |
*** diablo_rojo has quit IRC | 00:05 | |
*** diablo_rojo has joined #openstack-meeting-cp | 00:06 | |
*** felipemonteiro has joined #openstack-meeting-cp | 00:19 | |
*** felipemonteiro has quit IRC | 01:35 | |
*** edmondsw has joined #openstack-meeting-cp | 01:44 | |
*** edmondsw has quit IRC | 01:48 | |
*** markvoelker has joined #openstack-meeting-cp | 01:50 | |
*** yamahata has joined #openstack-meeting-cp | 02:18 | |
*** markvoelker has quit IRC | 02:36 | |
*** yamahata has quit IRC | 02:38 | |
*** yamahata has joined #openstack-meeting-cp | 02:39 | |
*** yamahata has quit IRC | 02:54 | |
*** felipemonteiro has joined #openstack-meeting-cp | 03:18 | |
*** harlowja_ has quit IRC | 03:21 | |
*** nhelgeson has quit IRC | 03:39 | |
*** felipemonteiro has quit IRC | 03:57 | |
*** felipemonteiro has joined #openstack-meeting-cp | 04:15 | |
*** felipemonteiro has quit IRC | 04:39 | |
*** edmondsw has joined #openstack-meeting-cp | 05:20 | |
*** edmondsw has quit IRC | 05:24 | |
*** harlowja has joined #openstack-meeting-cp | 05:29 | |
*** markvoelker has joined #openstack-meeting-cp | 05:47 | |
*** coolsvap has joined #openstack-meeting-cp | 05:47 | |
*** markvoelker has quit IRC | 06:40 | |
*** markvoelker has joined #openstack-meeting-cp | 06:42 | |
*** harlowja has quit IRC | 06:47 | |
*** edmondsw has joined #openstack-meeting-cp | 07:08 | |
*** edmondsw has quit IRC | 07:13 | |
*** dims_ has quit IRC | 07:21 | |
*** dims has joined #openstack-meeting-cp | 07:25 | |
*** edmondsw has joined #openstack-meeting-cp | 08:56 | |
*** edmondsw has quit IRC | 09:01 | |
*** markvoelker has quit IRC | 10:13 | |
*** edmondsw has joined #openstack-meeting-cp | 10:44 | |
*** edmondsw has quit IRC | 10:49 | |
*** sdague has joined #openstack-meeting-cp | 11:39 | |
*** markvoelker has joined #openstack-meeting-cp | 12:14 | |
*** edmondsw has joined #openstack-meeting-cp | 12:33 | |
*** edmondsw has quit IRC | 12:38 | |
*** markvoelker has quit IRC | 12:48 | |
*** haint has joined #openstack-meeting-cp | 12:48 | |
*** markvoelker has joined #openstack-meeting-cp | 13:44 | |
*** markvoelker has quit IRC | 14:18 | |
*** edmondsw has joined #openstack-meeting-cp | 14:21 | |
*** edmondsw_ has joined #openstack-meeting-cp | 14:24 | |
*** edmondsw has quit IRC | 14:28 | |
*** edmondsw_ is now known as edmondsw | 14:45 | |
*** zhipeng has joined #openstack-meeting-cp | 14:54 | |
*** markvoelker has joined #openstack-meeting-cp | 15:02 | |
*** markvoelker has quit IRC | 15:02 | |
*** edmondsw has quit IRC | 15:18 | |
*** markvoelker has joined #openstack-meeting-cp | 15:24 | |
*** edmondsw has joined #openstack-meeting-cp | 15:24 | |
*** mriedem has joined #openstack-meeting-cp | 15:31 | |
*** felipemonteiro has joined #openstack-meeting-cp | 15:37 | |
*** felipemonteiro_ has joined #openstack-meeting-cp | 15:38 | |
*** zhipeng has quit IRC | 15:38 | |
*** felipemonteiro has quit IRC | 15:43 | |
ildikov | #startmeeting cinder-nova-api-changes | 16:00 |
---|---|---|
openstack | Meeting 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 |
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 xyang xyang1 raj_singh lyarwood jungleboyj stvnoyes | 16:00 |
* johnthetubaguy lurks | 16:00 | |
stvnoyes | o/ | 16:00 |
ildikov | johnthetubaguy: you can lurk if you're reviewing in the meantime :) | 16:01 |
jungleboyj | @! | 16:01 |
_pewp_ | jungleboyj ( *՞ਊ՞*)ノ | 16:01 |
ildikov | so thanks to mriedem among other things we have a thorough tracking etherpad | 16:01 |
ildikov | #link https://etherpad.openstack.org/p/multi-attach-volume-queens | 16:01 |
ildikov | we have the Nova side patches almost good to go | 16:02 |
ildikov | we'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 case | 16:03 |
ildikov | I believe we agreed it's an easy fix | 16:03 |
ildikov | otherwise this is the top of the series in Nova: https://review.openstack.org/#/c/271047/ | 16:04 |
mriedem | i've also got the nova-multiattach ci job patch here https://review.openstack.org/#/c/532689/ | 16:04 |
mriedem | but annoyingly, | 16:04 |
mriedem | i can see it queued up in the experimental queue in zuul, | 16:04 |
mriedem | but it doesn't seem to run, and zuul didn't post results on it last night | 16:05 |
mriedem | i know if that's a problem in the patch, or with zuul | 16:05 |
mriedem | *don't know | 16:05 |
mriedem | i'm assuming the latter because the gate has been borked all week | 16:05 |
ildikov | mriedem: if you saw the queues they are in rough shape in general | 16:05 |
mriedem | yes the constant restarts don't help, but it is what it is, | 16:05 |
mriedem | thanks intel | 16:05 |
ildikov | yeah :/ | 16:06 |
ildikov | I did a recheck as well and then realized it won't go anywhere today... | 16:06 |
ildikov | mriedem: thanks for the CI job! | 16:06 |
mriedem | stvnoyes: were you still working on adding some tempest patches on top of https://review.openstack.org/#/c/266605/ for resize? | 16:07 |
mriedem | the two tempest patches below that are approved, so that's nice | 16:07 |
stvnoyes | yes, that's what I'm doing today | 16:07 |
mriedem | ok i'll see if the qa people really need me to split up https://review.openstack.org/#/c/266605/25 | 16:08 |
mriedem | re: cinder patches, i commented on https://review.openstack.org/#/c/531569 last night | 16:09 |
ildikov | it's not that big... | 16:09 |
mriedem | looks like jgriffith is going to update that | 16:09 |
ildikov | he 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 |
mriedem | so my comments on the policy change was, | 16:09 |
mriedem | 1. should be the same as volume create so non-admins can create a multiattach volume | 16:10 |
mriedem | otherwise the tempest patch won't work | 16:10 |
mriedem | i 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 upgrading | 16:10 |
mriedem | and 2. the nova spec said we'd have a cinder policy for disabling multiattach+bootable for the bfv case, | 16:10 |
mriedem | i think that becomes just a check in the set_bootable volume action API or whatever | 16:11 |
ildikov | yeah, we talked about the bootable case back at the time of the Nova spec | 16:11 |
mriedem | https://github.com/openstack/cinder/blob/master/cinder/api/contrib/volume_actions.py#L362 | 16:12 |
mriedem | it looks like there is no existing policy check on creating bootable volumes | 16:12 |
mriedem | but 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 policy | 16:12 |
jungleboyj | Yes, we agreed it wasn't supported for BFV. | 16:12 |
mriedem | and default the policy to allow it | 16:12 |
mriedem | jungleboyj: no we did agree we'd support bfv | 16:13 |
mriedem | jungleboyj: you might be thinking of pike when we said we wouldn't :) | 16:13 |
ildikov | jungleboyj: and we add policy to turn it off | 16:13 |
jungleboyj | mriedem: What? I thought we said no multi-attach for BFV? | 16:13 |
jungleboyj | ildikov: Oh, ok. I am just going to be quiet now. | 16:13 |
mriedem | we said nova wouldn't do any bfv checks and just leave it to cinder policy | 16:13 |
jungleboyj | You guys are the experts. | 16:13 |
mriedem | https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/cinder-volume-multi-attach.html#rest-api-impact | 16:14 |
ildikov | jungleboyj: 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 |
mriedem | The following policy rules will be added to Cinder: Enable/Disable multiattach=True Enable/Disable multiattach=True + bootable=True | 16:14 |
jungleboyj | mriedem: Ok, that makes sense. | 16:15 |
ildikov | and the bootable part is missing currently fro the proposed changes | 16:15 |
mriedem | yeah comments are in the cinder patch, so i think we're good | 16:17 |
ildikov | mriedem: do you see anything else missing? | 16:17 |
ildikov | mriedem: 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 |
mriedem | swap volume is the only thing that came to me late last night | 16:18 |
mriedem | and the policy thing | 16:18 |
mriedem | i won't know about other stuff unless / until it comes to me | 16:18 |
mriedem | i'm hoping that johnthetubaguy and gibi could get a review pass on the nova patches yet this week, | 16:18 |
ildikov | I remembered the policy one, just wasn't sure what's in the queue jgriffith hasn't uploaded yet | 16:19 |
mriedem | before i have to leave | 16:19 |
ildikov | gibi started already, will continue tomorrow | 16:19 |
* johnthetubaguy nods that he hopes to do that ASAP | 16:19 | |
ildikov | he also helps with getting some use cases from the E/// folks who need this for the PTG | 16:19 |
johnthetubaguy | bit distracted with some ironic pieces, but will carve out time fo rthat | 16:19 |
ildikov | johnthetubaguy: let me know if I should talk to your boss about priorities :) | 16:20 |
johnthetubaguy | :) | 16:20 |
ildikov | mriedem: I guess you will whip up the swap patch | 16:20 |
mriedem | yeah | 16:20 |
ildikov | cool, thanks | 16:21 |
ildikov | mriedem: 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 |
stvnoyes | as I'm adding the resize test, I'll see if I notice anything missing... | 16:22 |
mriedem | ildikov: swap volume needs to be covered as well | 16:22 |
stvnoyes | ok, I can do that next | 16:22 |
mriedem | just, whatever the todos are in the tempest patch | 16:22 |
ildikov | mriedem: stvnoyes: ok, that sounds good | 16:22 |
ildikov | jungleboyj: when is the freeze for the cinderclient? | 16:24 |
jungleboyj | Let me double check. I think we have 2 weeks. | 16:24 |
ildikov | I mean the lib freeze because of which we had to wait a month last time | 16:24 |
smcginnis | Two weeks. | 16:24 |
jungleboyj | Yeah, 1/26 | 16:24 |
ildikov | ok, sounds tight but doable | 16:24 |
smcginnis | os-brick next Thursday, python-cinderclient the following Thursday. | 16:25 |
mriedem | 1/25 is cinderclient freeze | 16:25 |
mriedem | and FF | 16:25 |
ildikov | Thursday resonated better with me, but oh well | 16:25 |
ildikov | and that's the 25th, right | 16:26 |
ildikov | this is just one of those days when there's not enough caffeine... :/ | 16:26 |
ildikov | ok, I think we're good for today? | 16:27 |
mriedem | so let's end this meeting so you can get some coffee | 16:27 |
ildikov | mriedem: +1 :) | 16:27 |
jungleboyj | ++ for coffee | 16:27 |
ildikov | so 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-queens | 16:28 |
ildikov | thanks everyone! | 16:28 |
ildikov | mriedem: have a great vacation next week! | 16:28 |
jungleboyj | ildikov: Thank you and thanks to everyone for continuing to work this! | 16:28 |
mriedem | thanks | 16:28 |
jungleboyj | mriedem: Enjoy. I hope you get warmer weather than I did in RTP. | 16:29 |
ildikov | :) | 16:29 |
mriedem | puerto vallarta | 16:29 |
mriedem | ceviches and sand | 16:29 |
jungleboyj | mriedem: Nice! | 16:29 |
ildikov | that sounds pretty awesome! :) | 16:29 |
ildikov | and with that | 16:29 |
ildikov | #endmeeting | 16:30 |
*** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings" | 16:30 | |
openstack | Meeting ended Thu Jan 11 16:30:02 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:30 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2018/cinder_nova_api_changes.2018-01-11-16.00.html | 16:30 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2018/cinder_nova_api_changes.2018-01-11-16.00.txt | 16:30 |
openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_nova_api_changes/2018/cinder_nova_api_changes.2018-01-11-16.00.log.html | 16:30 |
*** mriedem has left #openstack-meeting-cp | 16:31 | |
*** felipemonteiro_ has quit IRC | 16:58 | |
*** iyamahat has joined #openstack-meeting-cp | 17:03 | |
*** iyamahat has quit IRC | 17:11 | |
*** coolsvap has quit IRC | 17:36 | |
*** nhelgeson has joined #openstack-meeting-cp | 18:14 | |
*** iyamahat has joined #openstack-meeting-cp | 18:43 | |
*** yamahata has joined #openstack-meeting-cp | 18:53 | |
*** felipemonteiro has joined #openstack-meeting-cp | 18:55 | |
*** felipemonteiro_ has joined #openstack-meeting-cp | 18:56 | |
*** felipemonteiro has quit IRC | 19:00 | |
*** harlowja has joined #openstack-meeting-cp | 19:43 | |
*** edmondsw_ has joined #openstack-meeting-cp | 19:48 | |
*** edmonds__ has joined #openstack-meeting-cp | 19:48 | |
*** edmondsw has quit IRC | 19:52 | |
*** edmondsw_ has quit IRC | 19:52 | |
*** david-lyle has quit IRC | 21:03 | |
*** david-lyle has joined #openstack-meeting-cp | 21:04 | |
*** edmonds__ is now known as edmondsw | 21:07 | |
*** markvoelker has quit IRC | 22:49 | |
*** markvoelker has joined #openstack-meeting-cp | 22:49 | |
*** markvoelker has quit IRC | 22:54 | |
*** edmondsw has quit IRC | 22:54 | |
*** edmondsw has joined #openstack-meeting-cp | 22:55 | |
*** felipemonteiro_ has quit IRC | 22:55 | |
*** markvoelker has joined #openstack-meeting-cp | 22:56 | |
*** edmondsw has quit IRC | 22:59 | |
*** markvoelker has quit IRC | 22:59 | |
*** markvoelker has joined #openstack-meeting-cp | 23:00 | |
*** markvoelker has quit IRC | 23:05 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!