| *** thingee_ has quit IRC | 00:00 | |
| *** baojg has quit IRC | 00:02 | |
| *** itlinux has quit IRC | 00:06 | |
| mriedem | there are some volume group tests that have been failing randomly since they were added a day or so ago https://bugs.launchpad.net/tempest/+bug/1709739 | 00:12 |
|---|---|---|
| openstack | Launchpad bug 1709739 in Cinder ""Invalid volume: Volume is not local to this node" in neutron multinode job (pike)" [Undecided,Confirmed] | 00:12 |
| mriedem | 68 hits in 2 days | 00:12 |
| *** baojg has joined #openstack-cinder | 00:14 | |
| *** baojg has quit IRC | 00:14 | |
| *** baojg has joined #openstack-cinder | 00:15 | |
| *** baojg has quit IRC | 00:15 | |
| *** k2so is now known as sdake | 00:16 | |
| *** edmondsw has joined #openstack-cinder | 00:17 | |
| *** yangyapeng has quit IRC | 00:18 | |
| *** sbezverk has quit IRC | 00:21 | |
| *** edmondsw has quit IRC | 00:22 | |
| openstackgerrit | Merged openstack/cinder master: Revert "Mark ETERNUS drivers as unsupported" https://review.openstack.org/492133 | 00:34 |
| *** zhurong has joined #openstack-cinder | 00:51 | |
| *** markvoelker has joined #openstack-cinder | 01:03 | |
| *** itlinux has joined #openstack-cinder | 01:04 | |
| *** Shunli has joined #openstack-cinder | 01:04 | |
| *** Apoorva_ has quit IRC | 01:09 | |
| *** baojg has joined #openstack-cinder | 01:16 | |
| *** yangyapeng has joined #openstack-cinder | 01:21 | |
| *** baojg has quit IRC | 01:24 | |
| *** PsionTheory has quit IRC | 01:35 | |
| *** edmondsw has joined #openstack-cinder | 02:06 | |
| *** edmondsw has quit IRC | 02:10 | |
| *** hemna__ has joined #openstack-cinder | 02:11 | |
| *** zsli_ has joined #openstack-cinder | 02:11 | |
| *** yangyape_ has joined #openstack-cinder | 02:11 | |
| *** hemna_ has quit IRC | 02:11 | |
| *** diablo_rojo has quit IRC | 02:17 | |
| *** yangyapeng has quit IRC | 02:18 | |
| *** Shunli has quit IRC | 02:18 | |
| *** tomh has quit IRC | 02:18 | |
| *** tomh has joined #openstack-cinder | 02:18 | |
| *** jistr has quit IRC | 02:18 | |
| *** geguileo has quit IRC | 02:18 | |
| *** openstackgerrit has quit IRC | 02:21 | |
| *** armax has quit IRC | 02:21 | |
| *** lifeless_ has joined #openstack-cinder | 02:21 | |
| *** baojg has joined #openstack-cinder | 02:21 | |
| *** armax has joined #openstack-cinder | 02:21 | |
| *** jistr has joined #openstack-cinder | 02:23 | |
| *** tomh_ has joined #openstack-cinder | 02:23 | |
| *** baojg has quit IRC | 02:26 | |
| *** geguileo has joined #openstack-cinder | 02:27 | |
| *** wanghao has quit IRC | 02:27 | |
| *** wanghao has joined #openstack-cinder | 02:28 | |
| *** lifeless has quit IRC | 02:28 | |
| *** tomh has quit IRC | 02:28 | |
| *** openstackgerrit has joined #openstack-cinder | 02:36 | |
| openstackgerrit | Yu Zhang proposed openstack/cinder master: Storwize:Update document for Pike release https://review.openstack.org/492053 | 02:36 |
| openstackgerrit | OpenStack Release Bot proposed openstack/cinder master: Update reno for stable/pike https://review.openstack.org/492777 | 02:53 |
| openstackgerrit | Yu Zhang proposed openstack/cinder master: Storwize:Update document for Pike release https://review.openstack.org/492053 | 03:00 |
| *** dave-mccowan has quit IRC | 03:06 | |
| *** nicolasbock has joined #openstack-cinder | 03:21 | |
| *** mriedem has quit IRC | 03:22 | |
| *** sbezverk has joined #openstack-cinder | 03:24 | |
| *** mriedem has joined #openstack-cinder | 03:27 | |
| *** psachin has joined #openstack-cinder | 03:41 | |
| *** psachin has quit IRC | 03:43 | |
| *** itlinux_ has joined #openstack-cinder | 03:43 | |
| openstackgerrit | yfzhao proposed openstack/cinder master: Fix wrong links https://review.openstack.org/492790 | 03:44 |
| *** psachin has joined #openstack-cinder | 03:51 | |
| *** yangyape_ has quit IRC | 03:51 | |
| *** edmondsw has joined #openstack-cinder | 03:54 | |
| *** fredli__ has joined #openstack-cinder | 03:56 | |
| *** mvk has joined #openstack-cinder | 03:57 | |
| *** edmondsw has quit IRC | 03:58 | |
| *** chhavi has joined #openstack-cinder | 04:00 | |
| *** itlinux_ has quit IRC | 04:03 | |
| *** sbezverk has quit IRC | 04:07 | |
| *** chaynikasaikia has joined #openstack-cinder | 04:09 | |
| openstackgerrit | yfzhao proposed openstack/cinder master: Fix wrong links https://review.openstack.org/492790 | 04:10 |
| *** itlinux_ has joined #openstack-cinder | 04:16 | |
| *** itlinux_ has quit IRC | 04:19 | |
| *** chaynikasaikia has quit IRC | 04:19 | |
| *** itlinux_ has joined #openstack-cinder | 04:19 | |
| *** itlinux_ has quit IRC | 04:23 | |
| *** baojg has joined #openstack-cinder | 04:23 | |
| *** baojg has quit IRC | 04:30 | |
| *** adisky__ has joined #openstack-cinder | 04:31 | |
| *** markstur has quit IRC | 04:32 | |
| *** chaynikasaikia has joined #openstack-cinder | 04:43 | |
| *** chaynikasaikia has left #openstack-cinder | 04:43 | |
| *** abhitechie has joined #openstack-cinder | 04:44 | |
| *** abhitechie has quit IRC | 04:45 | |
| *** abhitechie has joined #openstack-cinder | 04:45 | |
| *** savvas has joined #openstack-cinder | 04:50 | |
| *** savvas has joined #openstack-cinder | 04:51 | |
| *** Poornima has joined #openstack-cinder | 04:51 | |
| *** Poornima_K has joined #openstack-cinder | 05:17 | |
| *** Poornima has quit IRC | 05:18 | |
| *** savvas has quit IRC | 05:19 | |
| *** markstur has joined #openstack-cinder | 05:35 | |
| *** markstur has quit IRC | 05:40 | |
| *** Poornima_K has quit IRC | 05:40 | |
| *** Poornima has joined #openstack-cinder | 05:40 | |
| openstackgerrit | Merged openstack/cinder master: Update reno for stable/pike https://review.openstack.org/492777 | 05:42 |
| *** edmondsw has joined #openstack-cinder | 05:42 | |
| *** edmondsw has quit IRC | 05:46 | |
| *** armax has quit IRC | 05:51 | |
| *** lpetrut has joined #openstack-cinder | 05:52 | |
| *** armax has joined #openstack-cinder | 05:52 | |
| *** armax has quit IRC | 05:52 | |
| *** armax has joined #openstack-cinder | 05:52 | |
| *** armax has quit IRC | 05:53 | |
| *** armax has joined #openstack-cinder | 05:53 | |
| *** armax has quit IRC | 05:54 | |
| *** armax has joined #openstack-cinder | 05:54 | |
| *** armax has quit IRC | 05:54 | |
| *** itlinux has quit IRC | 05:55 | |
| *** armax has joined #openstack-cinder | 05:55 | |
| *** armax has joined #openstack-cinder | 05:56 | |
| *** armax has quit IRC | 05:56 | |
| *** armax has joined #openstack-cinder | 05:57 | |
| *** itlinux has joined #openstack-cinder | 05:57 | |
| *** armax has joined #openstack-cinder | 05:57 | |
| *** armax has quit IRC | 05:58 | |
| *** fredli__ has quit IRC | 06:05 | |
| *** _pewp_ has quit IRC | 06:13 | |
| *** _pewp_ has joined #openstack-cinder | 06:14 | |
| *** tesseract has joined #openstack-cinder | 06:16 | |
| *** rcernin has joined #openstack-cinder | 06:22 | |
| *** ChubYann has quit IRC | 06:26 | |
| *** baojg has joined #openstack-cinder | 06:27 | |
| *** itlinux has quit IRC | 06:28 | |
| *** savvas has joined #openstack-cinder | 06:31 | |
| *** rtjure has joined #openstack-cinder | 06:32 | |
| *** baojg has quit IRC | 06:34 | |
| *** savvas has quit IRC | 06:35 | |
| *** Poornima has quit IRC | 06:37 | |
| *** libin_ has joined #openstack-cinder | 06:43 | |
| *** Poornima has joined #openstack-cinder | 06:52 | |
| *** lpetrut has quit IRC | 06:52 | |
| *** pgadiya has joined #openstack-cinder | 07:03 | |
| *** aarefiev_afk is now known as aarefiev | 07:08 | |
| *** Poornima has quit IRC | 07:10 | |
| *** saphi has joined #openstack-cinder | 07:13 | |
| *** baojg has joined #openstack-cinder | 07:30 | |
| *** baojg has quit IRC | 07:37 | |
| *** e0ne has joined #openstack-cinder | 07:42 | |
| *** Poornima has joined #openstack-cinder | 07:52 | |
| *** openstackgerrit has quit IRC | 08:02 | |
| *** mvk has quit IRC | 08:14 | |
| *** openstackgerrit has joined #openstack-cinder | 08:20 | |
| openstackgerrit | Yingzhe Zeng proposed openstack/cinder master: Fix to get correct hypermetro remote lun ID https://review.openstack.org/492854 | 08:20 |
| *** mvk has joined #openstack-cinder | 08:21 | |
| *** yangyapeng has joined #openstack-cinder | 08:30 | |
| *** Lee1092 has joined #openstack-cinder | 08:32 | |
| openstackgerrit | yixuan zhang proposed openstack/cinder master: Storwize: add hyperswap volume support https://review.openstack.org/492414 | 08:33 |
| *** gfidente has joined #openstack-cinder | 08:40 | |
| *** gfidente has left #openstack-cinder | 08:40 | |
| *** wanghao has quit IRC | 08:40 | |
| *** wanghao has joined #openstack-cinder | 08:41 | |
| *** wanghao_ has joined #openstack-cinder | 08:44 | |
| *** wanghao has quit IRC | 08:48 | |
| openstackgerrit | yfzhao proposed openstack/python-cinderclient master: Fix wrong links https://review.openstack.org/492913 | 09:03 |
| *** yangyapeng has quit IRC | 09:03 | |
| *** wanghao has joined #openstack-cinder | 09:09 | |
| *** chhavi has quit IRC | 09:10 | |
| *** wanghao_ has quit IRC | 09:12 | |
| *** markvoelker has quit IRC | 09:17 | |
| *** wanghao has quit IRC | 09:24 | |
| *** wanghao has joined #openstack-cinder | 09:26 | |
| *** mriedem has quit IRC | 09:30 | |
| *** baojg has joined #openstack-cinder | 09:34 | |
| *** zsli_ has quit IRC | 09:37 | |
| *** baojg has quit IRC | 09:40 | |
| *** dalgaaf has quit IRC | 09:42 | |
| *** dalgaaf has joined #openstack-cinder | 09:44 | |
| *** sdague has joined #openstack-cinder | 10:00 | |
| *** chhavi has joined #openstack-cinder | 10:00 | |
| *** pgadiya has quit IRC | 10:01 | |
| *** yangyapeng has joined #openstack-cinder | 10:04 | |
| *** sapd has joined #openstack-cinder | 10:07 | |
| *** ociuhandu has quit IRC | 10:08 | |
| *** yangyapeng has quit IRC | 10:09 | |
| *** pgadiya has joined #openstack-cinder | 10:18 | |
| *** markvoelker has joined #openstack-cinder | 10:18 | |
| *** alkhodos has quit IRC | 10:19 | |
| *** gkadam has joined #openstack-cinder | 10:20 | |
| *** psachin has quit IRC | 10:22 | |
| *** markvoelker has quit IRC | 10:23 | |
| *** wanghao has quit IRC | 10:23 | |
| *** markstur has joined #openstack-cinder | 10:31 | |
| *** psachin has joined #openstack-cinder | 10:35 | |
| *** markstur has quit IRC | 10:36 | |
| *** markvoelker has joined #openstack-cinder | 10:39 | |
| *** markvoelker_ has joined #openstack-cinder | 10:44 | |
| *** markvoelker has quit IRC | 10:44 | |
| *** markvoelker_ has quit IRC | 10:44 | |
| *** markvoelker has joined #openstack-cinder | 10:45 | |
| *** saphi has quit IRC | 10:59 | |
| *** zhurong has quit IRC | 11:00 | |
| *** yangyapeng has joined #openstack-cinder | 11:05 | |
| *** mriedem has joined #openstack-cinder | 11:07 | |
| *** edmondsw has joined #openstack-cinder | 11:10 | |
| *** yangyapeng has quit IRC | 11:10 | |
| *** sapd has quit IRC | 11:14 | |
| *** dalgaaf has quit IRC | 11:24 | |
| *** e0ne has quit IRC | 11:24 | |
| *** sapd has joined #openstack-cinder | 11:28 | |
| *** dalgaaf has joined #openstack-cinder | 11:29 | |
| *** sbezverk has joined #openstack-cinder | 11:42 | |
| *** liverpooler has joined #openstack-cinder | 11:45 | |
| *** abhitechie has quit IRC | 11:47 | |
| *** sapd has quit IRC | 11:48 | |
| *** dikonoor has joined #openstack-cinder | 11:49 | |
| *** sbezverk has quit IRC | 11:50 | |
| *** e0ne has joined #openstack-cinder | 11:51 | |
| *** yuchenghui has joined #openstack-cinder | 11:52 | |
| *** yuchenghui has quit IRC | 11:58 | |
| *** sapd has joined #openstack-cinder | 12:01 | |
| *** yuchenghui has joined #openstack-cinder | 12:01 | |
| *** _heena_py_ has joined #openstack-cinder | 12:03 | |
| *** yuchenghui has quit IRC | 12:07 | |
| *** markvoelker_ has joined #openstack-cinder | 12:10 | |
| *** markvoelker has quit IRC | 12:13 | |
| *** dikonoor has quit IRC | 12:15 | |
| *** dikonoor has joined #openstack-cinder | 12:16 | |
| *** pgadiya has quit IRC | 12:16 | |
| *** phealy has joined #openstack-cinder | 12:16 | |
| *** zeestrat has quit IRC | 12:19 | |
| *** TheJulia has quit IRC | 12:19 | |
| *** guyr-infinidat has quit IRC | 12:19 | |
| *** TheJulia has joined #openstack-cinder | 12:19 | |
| *** guyr-infinidat has joined #openstack-cinder | 12:19 | |
| *** gouthamr has joined #openstack-cinder | 12:19 | |
| *** zeestrat has joined #openstack-cinder | 12:20 | |
| *** lpetrut has joined #openstack-cinder | 12:22 | |
| *** rtjure has quit IRC | 12:23 | |
| *** dave-mccowan has joined #openstack-cinder | 12:25 | |
| _heena_py_ | Hi | 12:25 |
| _heena_py_ | does anyone know how to get volume attachments in python | 12:25 |
| _heena_py_ | ? | 12:25 |
| *** libin_ has quit IRC | 12:25 | |
| *** rtjure has joined #openstack-cinder | 12:26 | |
| *** libin_ has joined #openstack-cinder | 12:26 | |
| *** xyang1 has joined #openstack-cinder | 12:29 | |
| *** catintheroof has joined #openstack-cinder | 12:30 | |
| *** gkadam has quit IRC | 12:37 | |
| *** abishop has joined #openstack-cinder | 12:39 | |
| *** dustins has joined #openstack-cinder | 12:54 | |
| *** savvas has joined #openstack-cinder | 13:01 | |
| *** baojg has joined #openstack-cinder | 13:03 | |
| *** yangyapeng has joined #openstack-cinder | 13:07 | |
| *** Poornima has quit IRC | 13:07 | |
| *** MVenesio has joined #openstack-cinder | 13:07 | |
| *** sbezverk has joined #openstack-cinder | 13:09 | |
| TheJulia | _heena_py_: I think it it might help if you elaborate context, because attachment is slightly overloaded as it is used in several places so your question could mean a couple different things. | 13:11 |
| *** dikonoo has joined #openstack-cinder | 13:11 | |
| *** yangyapeng has quit IRC | 13:11 | |
| *** MVenesio has quit IRC | 13:13 | |
| *** MVenesio has joined #openstack-cinder | 13:13 | |
| *** josecastroleon has joined #openstack-cinder | 13:15 | |
| *** dikonoor has quit IRC | 13:15 | |
| *** randomhack has joined #openstack-cinder | 13:18 | |
| *** savvas has quit IRC | 13:21 | |
| *** bhagyashris has quit IRC | 13:22 | |
| *** savvas has joined #openstack-cinder | 13:22 | |
| *** randomhack has quit IRC | 13:23 | |
| *** randomhack has joined #openstack-cinder | 13:24 | |
| openstackgerrit | Merged openstack/cinder master: Fix wrong links https://review.openstack.org/492790 | 13:27 |
| *** Lee1092 has quit IRC | 13:28 | |
| *** takedakn has joined #openstack-cinder | 13:46 | |
| *** jmlowe_ has joined #openstack-cinder | 13:47 | |
| *** jmlowe has quit IRC | 13:49 | |
| *** _pewp_ has quit IRC | 13:52 | |
| scottda | _heena_py_: If you are using current HEAD of master, you can use V3 attachment APIs: https://docs.openstack.org/cinder/latest/contributor/api/cinder.api.v3.attachments.html | 13:52 |
| *** markvoelker_ has quit IRC | 13:52 | |
| scottda | IF you are not on current master, You have no API for attachments available. | 13:52 |
| *** cknight has joined #openstack-cinder | 13:54 | |
| *** lucasxu has joined #openstack-cinder | 13:54 | |
| *** felipemonteiro has joined #openstack-cinder | 13:55 | |
| *** _pewp_ has joined #openstack-cinder | 13:55 | |
| jgriffith | stvnoyes ildikov smcginnis morning update :) | 13:58 |
| jgriffith | digging through things again last night the problem is actually pretty simple... and it's my own fault for being naive :) | 13:58 |
| ildikov | jgriffith: so what is it exactly? | 13:59 |
| jgriffith | there is some potential to fix up the connection info (attachment_update) and that's worth doing... but the root cause of the various failures is a race between connect and detach | 13:59 |
| ildikov | jgriffith: pretty simple is of course music to my ears :) | 13:59 |
| jgriffith | ildikov so the problem is that since we immediately toggle the volume to in-use when cinder gets the attachment-update call; all of our tests and every make assumptions based on that | 14:00 |
| *** randomhack has quit IRC | 14:00 | |
| ildikov | jgriffith: not every test is building on that info as I saw | 14:00 |
| jgriffith | so what we're seeing with things like the hanging devices, is because a detach is issues WHILE os-brick is still actually setting up the connection | 14:00 |
| *** alkhodos has joined #openstack-cinder | 14:01 | |
| ildikov | jgriffith: although I guess if we fix connection info we will not see the iscsi issues and not losing performance on it | 14:01 |
| jgriffith | ildikov 'not every test is building on that info as I saw' ? I'm unsure what that means? | 14:01 |
| ildikov | that no matter what Cinder sets for the volume if no one checks whether or not it is 'in-use' before cleaning up | 14:01 |
| jgriffith | ildikov oh... well not really though | 14:02 |
| jgriffith | because the tests pretty much always wait for appropriate status changes to do "the next thing" | 14:02 |
| jgriffith | so there's something else pretty dangerous about the way I have it right now with the auto-set thing.... | 14:02 |
| ildikov | so I haven't checked the old flow, maybe I should but I would guess brick is not struggling that much with creating the actual connection | 14:03 |
| jgriffith | all of our tests assume that when that status is toggled that means nova is done and it succesfully made the connection | 14:03 |
| jgriffith | ildikov I think the timing issues with brick and the retries may have been related to the connection_info data, but regardless it's something we should deal with anyway for busy systems etc | 14:03 |
| jgriffith | what I was going to mention about the "dangerous" thing is it's quite possible for a full test suite to run and everything pass without ever making a single iscsi connection | 14:04 |
| jgriffith | well, not quite "every test" but a good portion of them | 14:04 |
| jgriffith | so that's sort of crappy... | 14:05 |
| jgriffith | or good depending on your point of view :) | 14:05 |
| jgriffith | if you just want tests to pass it's fantastic!! :) | 14:05 |
| jgriffith | so my proposal, and what I think we should do.... | 14:05 |
| ildikov | no, my main concern is that I don't want this to be slower than the old flow was as that's a problem | 14:06 |
| *** mvk_ has joined #openstack-cinder | 14:06 | |
| ildikov | and yeah, attach_volume in tempest looks for 'in-use' | 14:06 |
| jgriffith | ildikov oh.. it won't be after this is dealt with | 14:06 |
| jgriffith | at least I don't think it will be | 14:06 |
| ildikov | but still, my bigger concern is the connection_info data and all those weird iscsiadm warnings/errors | 14:06 |
| jgriffith | so as much as I like the consolidated 2 step attach thing, we are going to need a 3'rd thing | 14:06 |
| jgriffith | ildikov those are gone for me now | 14:07 |
| ildikov | ok | 14:07 |
| ildikov | so we will need a third call from Nova that the connection is done | 14:07 |
| ildikov | right? | 14:07 |
| jgriffith | yeah, probably so.... | 14:07 |
| ildikov | which I guess means a version bump | 14:07 |
| *** yangyapeng has joined #openstack-cinder | 14:08 | |
| *** mvk has quit IRC | 14:08 | |
| *** markstur has joined #openstack-cinder | 14:08 | |
| jgriffith | which also means I might as well fix the responses while we're at it | 14:08 |
| jgriffith | The only other option is to add internal tracking of that in Nova | 14:08 |
| ildikov | internal tracking? | 14:08 |
| jgriffith | something like a status column in the bdm table | 14:08 |
| jgriffith | bdm.attach_status = detached|attaching|attached | 14:09 |
| ildikov | I would call it connection_status | 14:09 |
| jgriffith | that way when nova gets a detach request, it looks at that and decides if it should wait, continue etc | 14:09 |
| ildikov | otherwise it's a hard sell why we need attach_status on both sides | 14:09 |
| jgriffith | Yeah, I'm not particular about the name | 14:09 |
| ildikov | if we want to decouple Cinder a bit, then it's not a terribly bad option | 14:10 |
| ildikov | even if I really hate touching the BDM | 14:10 |
| jgriffith | ildikov yeah, it has pros and cons | 14:10 |
| ildikov | which one are you leaning towards? | 14:11 |
| jgriffith | mriedem may have some thoughts about that which could make the decision easy (like "no way") | 14:11 |
| ildikov | jgriffith: true :) | 14:11 |
| jgriffith | well in terms of not rocking the boat and not introducing new problems, I'm inclined to add the third call to cinder | 14:11 |
| jgriffith | or add it as a form of attachment_update | 14:12 |
| *** yangyapeng has quit IRC | 14:12 | |
| jgriffith | so one thing I thought of was that attachment_update would take a flag and set to "attaching" instead of "in-use" if that flag was set | 14:12 |
| *** markstur has quit IRC | 14:12 | |
| ildikov | I would like to avoid flags if possible | 14:12 |
| jgriffith | then attachment_update (or a new call altogether) could also be extended to be used for notification of connection complete | 14:13 |
| jgriffith | meh, flags/args are ok if theyr'e not overused and they make sense | 14:13 |
| jgriffith | IMHO at least | 14:13 |
| jgriffith | but yeah, I'm open | 14:13 |
| ildikov | yeah, they all start as one simple extra function and then turn into monsters :) | 14:14 |
| jgriffith | LOL.. fair enough | 14:14 |
| ildikov | anyway, maybe I just overcomplicate it :) | 14:17 |
| *** markstur has joined #openstack-cinder | 14:20 | |
| jgriffith | It's funny, we rat holed on the whole "multiple calls to brick for a shared connection" thing so much I kinda missed the most obvious and blatant thing of all | 14:20 |
| ildikov | well, I think we talked about it at a certain point a half year ago or so, but then let it go | 14:22 |
| jgriffith | probably.. this had gone on far too long :) | 14:22 |
| jgriffith | s/had/has/ | 14:22 |
| jgriffith | ok... well maybe I'll code up the changes and see how things go | 14:22 |
| ildikov | sounds good, I'll test it when you got something | 14:23 |
| jgriffith | ok, sounds good | 14:23 |
| *** psachin has quit IRC | 14:24 | |
| *** markstur has quit IRC | 14:24 | |
| ildikov | in the meantime hopefully mriedem let's us know his preference on where to track things | 14:26 |
| *** mdavidson has quit IRC | 14:26 | |
| jgriffith | ildikov yeah, I think I know the answer... and I think it's better to reflect it in cinder but we'll see | 14:27 |
| *** armax has joined #openstack-cinder | 14:31 | |
| *** takedakn has quit IRC | 14:32 | |
| *** itlinux has joined #openstack-cinder | 14:34 | |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: SMBFS: enable thin provisioning support flag https://review.openstack.org/484424 | 14:34 |
| *** mdavidson has joined #openstack-cinder | 14:35 | |
| *** Julien-zte has quit IRC | 14:36 | |
| *** yangyapeng has joined #openstack-cinder | 14:44 | |
| mriedem | i haven't read scrollback, but | 14:49 |
| mriedem | ""bdm.attach_status = detached|attaching|attached" | 14:50 |
| mriedem | isn't that the completely wrong way to be going? | 14:50 |
| mriedem | i thought that's why we removed the check_attach and check_detach stuff, | 14:50 |
| mriedem | to stop coupling nova state tracking with cinder internal state | 14:50 |
| mriedem | we need another call now on detach? | 14:52 |
| mriedem | or on attach? | 14:52 |
| mriedem | seems we're going back to the existing v2 APIs :) reserve, initialize_connection, attach | 14:52 |
| jgriffith | mriedem yep, pretty much | 14:52 |
| mriedem | now we'll have attachment_create (reserve), attachment_update (initailize_connection), super-final-done-with-connect (attach) | 14:52 |
| jgriffith | mriedem si | 14:53 |
| mriedem | what does that buy us over what we had before? | 14:53 |
| jgriffith | mriedem just it's own API and the attachment-id | 14:53 |
| jgriffith | which it occured to me last night we likely could've done anyway | 14:53 |
| mriedem | but, does this still make multi-attach support more possible? | 14:54 |
| *** crushil has joined #openstack-cinder | 14:54 | |
| jgriffith | mriedem absolutely, yes... the attachment object is what makes that possible | 14:54 |
| mriedem | in the new apis rather than bolting that onto the old apis? | 14:54 |
| mriedem | ok | 14:54 |
| mriedem | just need to keep reminding me of the justification here :) | 14:54 |
| jgriffith | the old api is kind of a mess at this point, so that would be no good | 14:54 |
| *** sbezverk has quit IRC | 14:54 | |
| jgriffith | mriedem yeah... it's unfortunate; but there is still value | 14:55 |
| *** felipemonteiro_ has joined #openstack-cinder | 14:55 | |
| jgriffith | mriedem it is more simple, and it does allow some things to be more robust in the future | 14:55 |
| *** felipemonteiro has quit IRC | 14:55 | |
| jgriffith | most of all, no more guessing about whether we can or can't disconnect something | 14:55 |
| *** sbezverk has joined #openstack-cinder | 14:55 | |
| jgriffith | well... except this particular little race condition :) | 14:56 |
| jgriffith | I'd like to figure out a different way to solve it; but I don't want to rewrite all of our tests etc to do so | 14:56 |
| *** rcernin has quit IRC | 14:57 | |
| jgriffith | mriedem the only other thing is if we put a lock in nova while it's doing the os-brick connect, and that keeps the detach from coming in while that's in process? | 14:57 |
| jgriffith | but I think that would be troublesome | 14:57 |
| *** thingee_ has joined #openstack-cinder | 14:58 | |
| jgriffith | smcginnis jungleboyj then there's the whole snapshot-attach.. GRRRRRR | 14:59 |
| * jungleboyj needs to read the backlog for context apparently. | 15:00 | |
| *** dikonoo has quit IRC | 15:02 | |
| jungleboyj | jgriffith: Is snapshot attach more complicated? | 15:03 |
| jgriffith | jungleboyj nah.... but this: https://github.com/openstack/cinder/commit/ea7d0860c5deaa345f1d6f7b6a348751222da3d9#diff-724c53b8cd580ae533b4fb301df7c3e9 | 15:04 |
| *** markstur has joined #openstack-cinder | 15:04 | |
| jgriffith | jungleboyj we probably should've forced usage of the new API or waited IMO | 15:04 |
| jgriffith | jungleboyj now it's one more thing to go back and change | 15:04 |
| jgriffith | not a huge deal, especially since this is really just an internal backup thing | 15:05 |
| mriedem | jgriffith: the only way we can do a lock like that in nova is via db | 15:05 |
| mriedem | if it's a shared connection i mean | 15:05 |
| mriedem | not really sure though, johnthetubaguy and mdbooth went over this quite a bit when writing the spec | 15:05 |
| mriedem | but i didn't think a process lock on nova-compute would be sufficient | 15:06 |
| jgriffith | mriedem yeah... which goes back to the bdm entry again.. and honestly that isn't a great answer anyway. Yeah... johnthetubaguy was definitely thinking about it a lot | 15:06 |
| jgriffith | mriedem and yes, I think you're correct; it would help, but I don't think it's good enough | 15:06 |
| jgriffith | and provides a great opportunity for "stuck in xxxxx" states | 15:06 |
| mriedem | i don't want to mention etcd either, even though i just did | 15:06 |
| jgriffith | which nobody wants to go back to | 15:07 |
| jgriffith | mriedem I'm ignoring that line | 15:07 |
| jgriffith | : | 15:07 |
| jgriffith | :) | 15:07 |
| jgriffith | so I don't like the extra cinder call but honestly I think it's the right answer. | 15:08 |
| jgriffith | I'm coding it up now and then will get a bunch of testing on it and make sure there's not yet another problem | 15:08 |
| jgriffith | if there is... well then I'll just drink heavily | 15:09 |
| *** savvas has quit IRC | 15:13 | |
| *** baojg_ has joined #openstack-cinder | 15:15 | |
| *** Dinesh_Bhor has quit IRC | 15:15 | |
| *** dklyle has joined #openstack-cinder | 15:15 | |
| *** david-lyle has quit IRC | 15:15 | |
| *** baojg has quit IRC | 15:16 | |
| openstackgerrit | Merged openstack/python-cinderclient master: Fix wrong links https://review.openstack.org/492913 | 15:18 |
| jungleboyj | It will be good when we have this whole API cleanup done. One step at a time though. | 15:22 |
| jungleboyj | jgriffith: I will need to spend some time at the PTG with you better understanding what has been fixed and where we are at. | 15:23 |
| jgriffith | jungleboyj for sure, no problem | 15:23 |
| *** mvk has joined #openstack-cinder | 15:24 | |
| jgriffith | jungleboyj depending on time, might even be worth reviewing the whole thing as a team | 15:24 |
| jungleboyj | jgriffith: Cool, don't want to be a pest. Ildiko has also been doing such a good job of leading I haven't worried much. | 15:24 |
| jungleboyj | jgriffith: Ok, I didn't want to ask for that if everyone felt up to date, but doing a whole team review would be awesome. | 15:25 |
| jungleboyj | I can put it on the agenda and see what everyone thinks. | 15:25 |
| jgriffith | yeah, ildikov is our fearless leader on this one :) | 15:25 |
| *** mvk_ has quit IRC | 15:25 | |
| *** e0ne has quit IRC | 15:27 | |
| *** dikonoo has joined #openstack-cinder | 15:29 | |
| *** mvk has quit IRC | 15:29 | |
| stvnoyes | jgriffith,mriedem: since the overlapping of detach too quickly following an attach is a problem, is there a way for nova to know an attach's connect is not complete and throw out of a premature detach (and vice versa)? The client can do that if they use events. but we can't rely on the client doing that. | 15:29 |
| *** mvk has joined #openstack-cinder | 15:29 | |
| jgriffith | stvnoyes I looked briefly at that, but couldn't figure out how to do it without state changes in nova-db; but admittedly there could be another way | 15:31 |
| jgriffith | stvnoyes I just put a delay in nova/compute/api:detach for 5 seconds... that works too :) | 15:31 |
| stvnoyes | yeah, I found that a delay in the test worked too; very clean ;-) (not) | 15:32 |
| jgriffith | :) | 15:32 |
| jgriffith | yeah, I stole your idea and just made it global :) | 15:32 |
| jgriffith | considered hiding it in a patch :) | 15:32 |
| stvnoyes | you're only supposed to steal my good ideas! | 15:33 |
| jgriffith | stvnoyes I don't know... it was looking like a pretty good idea at the time :) | 15:33 |
| *** dikonoo has quit IRC | 15:33 | |
| *** ociuhandu has joined #openstack-cinder | 15:34 | |
| *** wN has joined #openstack-cinder | 15:41 | |
| *** wN has quit IRC | 15:41 | |
| *** wN has joined #openstack-cinder | 15:41 | |
| *** dikonoo has joined #openstack-cinder | 15:42 | |
| *** itlinux has quit IRC | 15:46 | |
| *** aarefiev is now known as aarefiev_afk | 15:47 | |
| *** savvas has joined #openstack-cinder | 15:48 | |
| *** Apoorva has joined #openstack-cinder | 15:58 | |
| *** dikonoo has quit IRC | 15:59 | |
| *** ntpttr_laptop has joined #openstack-cinder | 16:01 | |
| *** armax has quit IRC | 16:04 | |
| *** sbezverk has quit IRC | 16:05 | |
| *** lucasxu has quit IRC | 16:07 | |
| jungleboyj | Gotta love software held together with sleeps. :-) | 16:07 |
| *** ociuhandu has quit IRC | 16:10 | |
| *** dikonoo has joined #openstack-cinder | 16:17 | |
| *** lpetrut has quit IRC | 16:18 | |
| *** markvoelker has joined #openstack-cinder | 16:25 | |
| *** itlinux has joined #openstack-cinder | 16:33 | |
| *** jmlowe_ has quit IRC | 16:34 | |
| ildikov | jgriffith: mriedem: sorry, was afk a bit | 16:36 |
| ildikov | jgriffith: mriedem: while I'm not happy about the new call, sounds like the right thing to do | 16:36 |
| *** psachin has joined #openstack-cinder | 16:36 | |
| ildikov | jungleboyj: I think a team review would be very useful as we need to figure out what multi-attach brings into the picture too and it would be great having people aware of the base API before we dig deeper into that | 16:38 |
| *** savvas_ has joined #openstack-cinder | 16:39 | |
| *** tesseract has quit IRC | 16:41 | |
| *** savvas has quit IRC | 16:41 | |
| jungleboyj | ildikov: Good. Glad you like that plan. I have added it to the Agenda. | 16:41 |
| ildikov | jungleboyj: great, thank you | 16:42 |
| ildikov | jungleboyj: how packed the agenda is? | 16:42 |
| jungleboyj | ildikov: Not particularly packed right now. There are a few items that will take longer discussion, but we definitely have room for more topics. | 16:43 |
| ildikov | jungleboyj: I might add an optional multi-attach item hoping that we can sort out the remaining issues with the new API rather quickly | 16:43 |
| jungleboyj | ildikov: Great idea! | 16:43 |
| ildikov | jungleboyj: sounds good | 16:43 |
| ildikov | jungleboyj: do you have the etherpad link handy? | 16:43 |
| jungleboyj | ildikov: Yep. | 16:43 |
| jungleboyj | ildikov: https://etherpad.openstack.org/p/cinder-ptg-queens | 16:44 |
| ildikov | jungleboyj: thank you | 16:44 |
| *** dklyle has quit IRC | 16:44 | |
| jungleboyj | Welcome. | 16:44 |
| *** dklyle has joined #openstack-cinder | 16:45 | |
| *** jmlowe has joined #openstack-cinder | 16:56 | |
| *** markvoelker has quit IRC | 16:59 | |
| *** Apoorva_ has joined #openstack-cinder | 17:01 | |
| *** e0ne has joined #openstack-cinder | 17:02 | |
| *** Apoorva has quit IRC | 17:04 | |
| *** e0ne has quit IRC | 17:05 | |
| *** e0ne has joined #openstack-cinder | 17:07 | |
| *** psachin has quit IRC | 17:08 | |
| *** e0ne_ has joined #openstack-cinder | 17:09 | |
| *** Swanson has quit IRC | 17:12 | |
| *** e0ne has quit IRC | 17:12 | |
| *** abhitechie has joined #openstack-cinder | 17:13 | |
| *** abhitechie has quit IRC | 17:14 | |
| *** felipemonteiro_ has quit IRC | 17:14 | |
| *** abhitechie has joined #openstack-cinder | 17:15 | |
| *** mvk has quit IRC | 17:17 | |
| *** bradbeam has joined #openstack-cinder | 17:17 | |
| *** bradbeam has left #openstack-cinder | 17:20 | |
| *** dikonoor has joined #openstack-cinder | 17:21 | |
| *** diablo_rojo has joined #openstack-cinder | 17:22 | |
| *** ntpttr_laptop has quit IRC | 17:23 | |
| *** dikonoo has quit IRC | 17:24 | |
| *** vivsoni_ has quit IRC | 17:28 | |
| *** savvas_ has quit IRC | 17:29 | |
| *** savvas has joined #openstack-cinder | 17:29 | |
| *** abhitechie has quit IRC | 17:32 | |
| *** abhitechie has joined #openstack-cinder | 17:33 | |
| *** simondodsley has joined #openstack-cinder | 17:33 | |
| simondodsley | Anyone got decent documentation on 'cinder manage'? There seems to be a lack of good info out there. Wanted to check before I started delving into the code to work out how to use it correctly. | 17:34 |
| jgriffith | stvnoyes ildikov prelim patches here: https://gist.github.com/j-griffith/4f815fe79e07fd53230b17aded419b50 | 17:35 |
| jgriffith | would love any feedback if you see something bad in there | 17:35 |
| *** e0ne_ has quit IRC | 17:35 | |
| jgriffith | simondodsley god be with you | 17:36 |
| *** abhitechie has quit IRC | 17:37 | |
| *** abhitechie has joined #openstack-cinder | 17:37 | |
| simondodsley | jgriffith I know - looking for the easy option first :) | 17:37 |
| patrickeast | simondodsley: what? that obscure jira reference and suggestion to use "cinder help manage" wasn't enough? :P | 17:38 |
| jgriffith | simondodsley I think this about as good as it gets: https://github.com/openstack/cinder/blob/master/doc/source/man/cinder-manage.rst | 17:39 |
| jgriffith | but I'm sure you already saw that | 17:39 |
| jgriffith | doh! | 17:39 |
| jgriffith | hahaah | 17:39 |
| patrickeast | hah yea | 17:39 |
| *** crushil has quit IRC | 17:39 | |
| jgriffith | wrong cinder-manage!!!! | 17:39 |
| jgriffith | LOL | 17:39 |
| simondodsley | going to say... | 17:39 |
| patrickeast | thats why i seriously regret the naming, its impossible to search for "cinder manage" help | 17:39 |
| jgriffith | https://community.netapp.com/t5/OpenStack-Discussions/Cinder-manage-command/td-p/127839 | 17:40 |
| patrickeast | iirc it did get a new name with osc too... although i don't know what it is | 17:40 |
| simondodsley | @patrickeast: not the most helpful JIRA | 17:40 |
| *** abhitechie has quit IRC | 17:40 | |
| *** abhitechie has joined #openstack-cinder | 17:40 | |
| patrickeast | simondodsley: theres a comment about halfway down where i show how to use it | 17:41 |
| jgriffith | of course at this point I'm resorting to google so I have no value add | 17:41 |
| jgriffith | avishay is your target | 17:41 |
| simondodsley | would be nice if the actual openstack docs talked about this, not something from NetApp | 17:41 |
| jgriffith | simondodsley I don't disagree.... but hey... we ARE OpenStack :) | 17:41 |
| patrickeast | there is one in the admin guide about managing snapshots | 17:41 |
| patrickeast | pretty close | 17:41 |
| *** e0ne has joined #openstack-cinder | 17:42 | |
| simondodsley | patrickeast: at least that's vaguely correct. The docs we have on our support site reference the old Kilo syntax... I had to remove them this morning | 17:42 |
| patrickeast | yea, theres a bug open for that somewhere... | 17:43 |
| ildikov | jgriffith: looks good overall | 17:43 |
| jgriffith | ildikov amazingly, so far it even seems to work :) | 17:43 |
| ildikov | jgriffith: even better :) | 17:43 |
| *** e0ne has quit IRC | 17:43 | |
| jgriffith | hope I didn't just jinx it | 17:44 |
| patrickeast | i don't know how to update them though... we should just move it to https://docs.openstack.org/cinder/latest/configuration/block-storage/drivers/pure-storage-driver.html | 17:44 |
| *** ntpttr_laptop has joined #openstack-cinder | 17:44 | |
| patrickeast | simondodsley: ^^ | 17:44 |
| jgriffith | so basically we did a whole lot of work to reimplement the old attach flow :) | 17:44 |
| ildikov | jgriffith: by seeing the console output I thought you're kinda certain :) | 17:44 |
| jgriffith | ildikov that was my manual testing; now it's the moment of truth | 17:45 |
| ildikov | jgriffith: tempest, yaaay :) | 17:46 |
| *** abhinavtechie has joined #openstack-cinder | 17:51 | |
| *** abhitechie has quit IRC | 17:51 | |
| *** crose has joined #openstack-cinder | 17:53 | |
| *** markvoelker has joined #openstack-cinder | 17:55 | |
| *** Apoorva_ has quit IRC | 18:03 | |
| *** crushil has joined #openstack-cinder | 18:03 | |
| *** Apoorva has joined #openstack-cinder | 18:03 | |
| *** thingee_ has quit IRC | 18:07 | |
| *** e0ne has joined #openstack-cinder | 18:12 | |
| *** e0ne has quit IRC | 18:12 | |
| *** sdague has quit IRC | 18:17 | |
| *** sdague has joined #openstack-cinder | 18:17 | |
| *** savvas has quit IRC | 18:19 | |
| *** dobson has quit IRC | 18:23 | |
| *** wN_ has quit IRC | 18:23 | |
| *** dobson has joined #openstack-cinder | 18:26 | |
| ildikov | jgriffith: smcginnis: jungleboyj: we should get the cinderclient changes for what we agreed above in too before we cut the next release | 18:27 |
| *** markvoelker has quit IRC | 18:29 | |
| *** wN_ has joined #openstack-cinder | 18:29 | |
| *** abhinavtechie has quit IRC | 18:29 | |
| *** abhitechie has joined #openstack-cinder | 18:29 | |
| *** savvas has joined #openstack-cinder | 18:31 | |
| *** savvas has quit IRC | 18:31 | |
| *** savvas has joined #openstack-cinder | 18:32 | |
| *** diablo_rojo has quit IRC | 18:33 | |
| *** diablo_rojo has joined #openstack-cinder | 18:33 | |
| jungleboyj | Ildikov sounds good. | 18:38 |
| jgriffith | ildikov ok... we look pretty good now. All tests passing, no performance hit and clean logs! | 18:40 |
| ildikov | jgriffith: sounds awesome!!! | 18:40 |
| ildikov | jgriffith: thank you! | 18:40 |
| jgriffith | guess I'll start making official versions of the cinder patches..... only remaining question is "to translate, or not-to translate" :) | 18:40 |
| ildikov | jgriffith: that's a tough one | 18:41 |
| *** nicolasbock has quit IRC | 18:43 | |
| *** abhitechie has quit IRC | 18:45 | |
| ildikov | jgriffith: I kinda lean towards switching back to the original version | 18:45 |
| *** crose has quit IRC | 18:45 | |
| jgriffith | ildikov it's certainly easier | 18:46 |
| jgriffith | ok... I'll try that | 18:46 |
| ildikov | jgriffith: but if we're certain that the old structure doesn't have a purpose what we have now is a good start to remove it | 18:46 |
| *** crushil has quit IRC | 19:04 | |
| diablo_rojo | hemna, jungleboyj I am writing an article for superuser about how people can follow cinder's ongoings at the ptg so other projects can read it and do similar things. Any interest in helping/reading over a draft early next week? | 19:07 |
| jungleboyj | diablo_rojo: Sure, I can do that. | 19:07 |
| diablo_rojo | jungleboyj, cool. I will have a draft for you on monday to add to/make comments on etc. | 19:07 |
| diablo_rojo | I want to have it done by Wednesdayish so it can get posted before the end of the week. | 19:08 |
| jungleboyj | diablo_rojo: Sounds good. | 19:08 |
| jungleboyj | diablo_rojo: Ok, shouldn't be a problem. | 19:08 |
| diablo_rojo | jungleboyj, cool :) Good way to get Cinder in the limelight a bit and teach other projects how we do things virtually for those that can't attend. | 19:09 |
| jungleboyj | diablo_rojo: Oh yeah. That would be good for us to share. Should talk about how we are trying to make our notes and action items more actionable after the PTG as well. | 19:11 |
| jungleboyj | diablo_rojo: Get me a draft and I will share my thoughts. | 19:11 |
| diablo_rojo | Good idea. I'm just working on an intro and outline right now and then I can share it with you. | 19:11 |
| diablo_rojo | jungleboyj ^^ | 19:12 |
| *** crushil has joined #openstack-cinder | 19:12 | |
| jungleboyj | diablo_rojo: Cool. | 19:12 |
| *** ntpttr_laptop has quit IRC | 19:18 | |
| *** geguileo is now known as geguileo_PTO | 19:19 | |
| *** ChubYann has joined #openstack-cinder | 19:23 | |
| *** lpetrut has joined #openstack-cinder | 19:25 | |
| *** markvoelker has joined #openstack-cinder | 19:27 | |
| *** Apoorva has quit IRC | 19:32 | |
| *** adisky__ has quit IRC | 19:39 | |
| *** ntpttr_laptop has joined #openstack-cinder | 19:45 | |
| diablo_rojo | jungleboyj, I shared it with you. | 19:49 |
| jungleboyj | diablo_rojo: Thanks. I see the request. Will take a look soon. | 19:52 |
| diablo_rojo | jungleboyj, its a bit more than an outline :) I got writing and couldn't stop. | 19:55 |
| jungleboyj | diablo_rojo: Looks like a good start. I will look when I don't have so much going on. | 19:58 |
| *** markvoelker has quit IRC | 19:59 | |
| diablo_rojo | jungleboyj, thanks, sounds good. | 20:00 |
| *** wN_ has quit IRC | 20:01 | |
| *** lpetrut has quit IRC | 20:01 | |
| *** lpetrut has joined #openstack-cinder | 20:02 | |
| *** wN_ has joined #openstack-cinder | 20:04 | |
| *** gouthamr has quit IRC | 20:14 | |
| *** MVenesio has quit IRC | 20:14 | |
| *** catintheroof has quit IRC | 20:15 | |
| *** Apoorva has joined #openstack-cinder | 20:16 | |
| *** gouthamr has joined #openstack-cinder | 20:33 | |
| *** jmlowe has quit IRC | 20:45 | |
| *** itlinux has quit IRC | 20:46 | |
| *** jmlowe has joined #openstack-cinder | 20:47 | |
| *** dikonoor has quit IRC | 20:51 | |
| *** cknight has quit IRC | 20:53 | |
| *** chhavi has quit IRC | 20:53 | |
| *** jmlowe has quit IRC | 20:56 | |
| *** markvoelker has joined #openstack-cinder | 20:56 | |
| *** abishop has quit IRC | 21:00 | |
| *** Apoorva_ has joined #openstack-cinder | 21:00 | |
| *** dustins has quit IRC | 21:00 | |
| *** edmondsw has quit IRC | 21:01 | |
| *** jistr is now known as jistr|off | 21:02 | |
| *** Apoorva has quit IRC | 21:04 | |
| *** crushil has quit IRC | 21:06 | |
| *** dave-mccowan has quit IRC | 21:12 | |
| *** crushil has joined #openstack-cinder | 21:14 | |
| *** markvoelker has quit IRC | 21:30 | |
| *** MVenesio has joined #openstack-cinder | 21:33 | |
| *** baojg has joined #openstack-cinder | 21:37 | |
| *** baojg_ has quit IRC | 21:37 | |
| *** Apoorva_ has quit IRC | 21:38 | |
| *** Apoorva has joined #openstack-cinder | 21:38 | |
| *** ntpttr_laptop has quit IRC | 21:40 | |
| *** xyang1 has quit IRC | 21:54 | |
| *** markvoelker has joined #openstack-cinder | 22:10 | |
| *** ntpttr_laptop has joined #openstack-cinder | 22:16 | |
| *** lbragstad has quit IRC | 22:17 | |
| *** MVenesio_ has joined #openstack-cinder | 22:18 | |
| *** MVenesio has quit IRC | 22:22 | |
| *** lpetrut has quit IRC | 22:23 | |
| *** markvoelker has quit IRC | 22:33 | |
| *** MVenesio_ has quit IRC | 22:51 | |
| openstackgerrit | Chaynika Saikia proposed openstack/cinder master: cinder incremental backup with ceph fails if last one deleted https://review.openstack.org/486678 | 22:53 |
| *** sdague has quit IRC | 23:09 | |
| smcginnis | jgriffith: Finally home and online. Reading scrollback. Not loving another call, but that's great if it's all sorted out and working now. | 23:17 |
| smcginnis | simondodsley: OSC renamed manage to "adopt" and "abandon" IIRC. | 23:17 |
| openstackgerrit | John Griffith proposed openstack/cinder master: Make attachment_update set status to attaching not https://review.openstack.org/493262 | 23:35 |
| smcginnis | jgriffith: Is "attaching not" a new status for not attached? :D | 23:36 |
| jgriffith | smcginnis LOL.. you got me | 23:39 |
| smcginnis | hehe | 23:39 |
| jgriffith | I suppose I should fix that :) | 23:40 |
| smcginnis | Eh, maybe wait and see if you need to do another update? | 23:40 |
| openstackgerrit | John Griffith proposed openstack/cinder master: Make attachment_update set status to attaching https://review.openstack.org/493262 | 23:40 |
| jgriffith | smcginnis already done... and no more updates!!! | 23:41 |
| jgriffith | :) | 23:41 |
| smcginnis | jgriffith: +1 let's hope! | 23:41 |
| jgriffith | client on the way | 23:41 |
| smcginnis | Sweet | 23:41 |
| jgriffith | we can talk Monday about backport, etc :( | 23:41 |
| jgriffith | I don't know how that works with api micro-v's | 23:41 |
| jgriffith | if at all | 23:41 |
| openstackgerrit | John Griffith proposed openstack/python-cinderclient master: Add an attachment_complete API call https://review.openstack.org/493263 | 23:42 |
| smcginnis | Yeah... not sure if that's really an option at this point. :/ | 23:42 |
| jgriffith | Ok... fair enough | 23:42 |
| smcginnis | Maybe in RC2. | 23:42 |
| jgriffith | on that note, I gotta run; I'll catch ya later. | 23:42 |
| smcginnis | Have a good weekend. | 23:42 |
| jgriffith | Thanks!!! You too!! | 23:42 |
| *** zzzeek has quit IRC | 23:43 | |
| *** zzzeek has joined #openstack-cinder | 23:44 | |
| *** Apoorva_ has joined #openstack-cinder | 23:50 | |
| *** Apoorva has quit IRC | 23:54 | |
| *** Apoorva_ has quit IRC | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!