| *** abishop has quit IRC | 00:03 | |
| *** Apoorva_ has quit IRC | 00:08 | |
| *** Apoorva has joined #openstack-cinder | 00:08 | |
| *** savvas_ has joined #openstack-cinder | 00:10 | |
| *** savvas has quit IRC | 00:11 | |
| *** ociuhandu_ has joined #openstack-cinder | 00:12 | |
| *** ociuhandu has quit IRC | 00:12 | |
| *** itlinux has quit IRC | 00:16 | |
| *** takedakn has joined #openstack-cinder | 00:18 | |
| *** takedakn has quit IRC | 00:19 | |
| *** takedakn has joined #openstack-cinder | 00:21 | |
| *** sbezverk has quit IRC | 00:25 | |
| *** sbezverk has joined #openstack-cinder | 00:26 | |
| *** jmlowe has joined #openstack-cinder | 00:27 | |
| *** bigdogstl has joined #openstack-cinder | 00:29 | |
| *** Apoorva_ has joined #openstack-cinder | 00:30 | |
| *** zhurong has joined #openstack-cinder | 00:31 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 00:31 |
|---|---|---|
| *** wanghao has joined #openstack-cinder | 00:32 | |
| *** Apoorva has quit IRC | 00:34 | |
| *** Apoorva_ has quit IRC | 00:34 | |
| *** bigdogstl has quit IRC | 00:36 | |
| *** itlinux has joined #openstack-cinder | 00:38 | |
| *** openstackgerrit has quit IRC | 00:41 | |
| *** openstackgerrit has joined #openstack-cinder | 00:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/502732 | 00:42 |
| *** wanghao has quit IRC | 00:44 | |
| *** wanghao has joined #openstack-cinder | 00:44 | |
| *** bigdogstl has joined #openstack-cinder | 00:53 | |
| *** hoonetorg has joined #openstack-cinder | 00:56 | |
| *** takedakn has quit IRC | 00:57 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 01:10 |
| *** bigdogstl has quit IRC | 01:15 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/505864 | 01:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/502732 | 01:21 |
| *** dave-mccowan has joined #openstack-cinder | 01:21 | |
| *** itlinux has quit IRC | 01:28 | |
| *** dave-mcc_ has joined #openstack-cinder | 01:34 | |
| *** dave-mccowan has quit IRC | 01:35 | |
| *** abishop has joined #openstack-cinder | 01:40 | |
| *** jmlowe has quit IRC | 01:42 | |
| *** jmlowe has joined #openstack-cinder | 01:43 | |
| *** itlinux has joined #openstack-cinder | 01:49 | |
| *** mriedem has quit IRC | 01:51 | |
| *** itlinux has quit IRC | 01:51 | |
| *** crushil has joined #openstack-cinder | 01:54 | |
| *** guyk has joined #openstack-cinder | 01:56 | |
| *** strigazi has quit IRC | 01:56 | |
| *** kaisers__ has joined #openstack-cinder | 01:57 | |
| *** abishop has quit IRC | 01:57 | |
| *** winston-d has quit IRC | 01:58 | |
| *** winston-d has joined #openstack-cinder | 01:58 | |
| *** strigazi has joined #openstack-cinder | 01:58 | |
| *** kaisers_ has quit IRC | 02:00 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 02:04 |
| *** lhx__ has joined #openstack-cinder | 02:04 | |
| *** markstur has quit IRC | 02:08 | |
| *** Shunli has joined #openstack-cinder | 02:10 | |
| *** wanghao has quit IRC | 02:13 | |
| *** crushil has quit IRC | 02:20 | |
| *** gouthamr has joined #openstack-cinder | 02:20 | |
| *** dave-mcc_ has quit IRC | 02:23 | |
| openstackgerrit | zhangyangyang proposed openstack/cinder master: Update sphinx version in test-requirements https://review.openstack.org/505919 | 02:23 |
| *** savvas_ has quit IRC | 02:24 | |
| *** savvas has joined #openstack-cinder | 02:24 | |
| *** bigdogstl has joined #openstack-cinder | 02:25 | |
| *** zhaochao has quit IRC | 02:30 | |
| *** zhaochao has joined #openstack-cinder | 02:31 | |
| *** bigdogstl has quit IRC | 02:33 | |
| openstackgerrit | Merged openstack/cinder master: Switch to use key_manager.backend https://review.openstack.org/505752 | 02:36 |
| *** wanghao has joined #openstack-cinder | 02:41 | |
| *** gkadam-away is now known as gkadam | 02:55 | |
| *** mfisch` is now known as mfisch | 02:59 | |
| *** mfisch is now known as Guest44234 | 03:00 | |
| *** lbragstad has joined #openstack-cinder | 03:00 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 03:03 |
| *** vivsoni has joined #openstack-cinder | 03:07 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/505864 | 03:12 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/502732 | 03:14 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 03:20 |
| *** lbragstad has quit IRC | 03:21 | |
| *** armax has quit IRC | 03:22 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/505864 | 03:29 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/502732 | 03:30 |
| *** bigdogstl has joined #openstack-cinder | 03:31 | |
| *** gman-tx has joined #openstack-cinder | 03:35 | |
| *** bigdogstl has quit IRC | 03:35 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 03:38 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/505864 | 03:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/502732 | 03:49 |
| *** Guest44234 is now known as mfisch | 03:52 | |
| *** mfisch is now known as Guest13446 | 03:53 | |
| *** takedakn has joined #openstack-cinder | 03:58 | |
| *** bigdogstl has joined #openstack-cinder | 04:00 | |
| *** erlon has quit IRC | 04:03 | |
| *** kbaegis has quit IRC | 04:08 | |
| *** gman-tx has quit IRC | 04:10 | |
| *** bigdogstl has quit IRC | 04:11 | |
| *** gouthamr has quit IRC | 04:12 | |
| *** takedakn has quit IRC | 04:16 | |
| *** stakeda has joined #openstack-cinder | 04:37 | |
| *** bigdogstl has joined #openstack-cinder | 04:38 | |
| *** psachin has joined #openstack-cinder | 04:41 | |
| *** bigdogstl has quit IRC | 04:43 | |
| *** links has joined #openstack-cinder | 04:43 | |
| *** wanghao_ has joined #openstack-cinder | 04:57 | |
| *** wanghao has quit IRC | 04:57 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Check for outstanding attachments during reserve https://review.openstack.org/504467 | 05:00 |
| *** itlinux has joined #openstack-cinder | 05:00 | |
| *** vivsoni has quit IRC | 05:05 | |
| *** vivsoni has joined #openstack-cinder | 05:05 | |
| *** pcaruana has joined #openstack-cinder | 05:09 | |
| *** e0ne has joined #openstack-cinder | 05:16 | |
| *** markvoelker has quit IRC | 05:16 | |
| *** markvoelker has joined #openstack-cinder | 05:17 | |
| *** e0ne has quit IRC | 05:20 | |
| *** markvoelker has quit IRC | 05:22 | |
| openstackgerrit | yixuan zhang proposed openstack/cinder master: Storwize: add NPIV support https://review.openstack.org/501128 | 05:35 |
| *** bigdogstl has joined #openstack-cinder | 05:40 | |
| *** bigdogstl has quit IRC | 05:44 | |
| *** pgadiya has joined #openstack-cinder | 05:59 | |
| *** bigdogstl has joined #openstack-cinder | 06:05 | |
| *** bigdogstl has quit IRC | 06:09 | |
| *** josecastroleon has joined #openstack-cinder | 06:27 | |
| *** rcernin has joined #openstack-cinder | 06:28 | |
| *** itlinux has quit IRC | 06:37 | |
| openstackgerrit | TommyLike proposed openstack/cinder-specs master: Support create volume from backup https://review.openstack.org/506004 | 06:38 |
| *** itlinux has joined #openstack-cinder | 06:40 | |
| *** itlinux has quit IRC | 06:40 | |
| *** jiaohaolin1 has left #openstack-cinder | 06:42 | |
| openstackgerrit | TommyLike proposed openstack/cinder-specs master: Proposal to count resources to check quota in API https://review.openstack.org/496562 | 06:56 |
| *** ChubYann has quit IRC | 07:04 | |
| *** pcaruana has quit IRC | 07:05 | |
| *** bigdogstl has joined #openstack-cinder | 07:17 | |
| *** markvoelker has joined #openstack-cinder | 07:18 | |
| *** ChanServ sets mode: +o openstack | 07:18 | |
| *** tesseract has joined #openstack-cinder | 07:19 | |
| *** bigdogstl has quit IRC | 07:22 | |
| *** pcaruana has joined #openstack-cinder | 07:23 | |
| *** alexchadin has joined #openstack-cinder | 07:29 | |
| *** zhurong has quit IRC | 07:35 | |
| *** markvoelker has quit IRC | 07:52 | |
| *** vivsoni has quit IRC | 07:53 | |
| *** vivsoni has joined #openstack-cinder | 07:53 | |
| *** gkadam has quit IRC | 07:54 | |
| *** wanghao has joined #openstack-cinder | 07:54 | |
| *** zhurong has joined #openstack-cinder | 07:55 | |
| *** wanghao_ has quit IRC | 07:58 | |
| openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Implement Tiramisu feature on VMAX https://review.openstack.org/505581 | 08:17 |
| *** bigdogstl has joined #openstack-cinder | 08:28 | |
| *** bigdogstl has quit IRC | 08:34 | |
| *** markvoelker has joined #openstack-cinder | 08:48 | |
| *** pgadiya has quit IRC | 08:53 | |
| openstackgerrit | Vivek Soni proposed openstack/cinder master: 3PAR: Cinder volume revert to snapshot support https://review.openstack.org/506072 | 08:56 |
| *** wanghao has quit IRC | 08:58 | |
| *** wanghao has joined #openstack-cinder | 08:59 | |
| *** gkadam has joined #openstack-cinder | 09:01 | |
| *** bigdogstl has joined #openstack-cinder | 09:14 | |
| *** edmondsw has joined #openstack-cinder | 09:15 | |
| *** bigdogstl has quit IRC | 09:19 | |
| *** edmondsw has quit IRC | 09:20 | |
| *** liverpooler has quit IRC | 09:21 | |
| *** markvoelker has quit IRC | 09:21 | |
| *** Shunli has quit IRC | 09:31 | |
| *** alexchadin has quit IRC | 09:34 | |
| *** alexchadin has joined #openstack-cinder | 09:35 | |
| *** alexchadin has quit IRC | 09:38 | |
| *** tesseract has quit IRC | 09:41 | |
| *** tesseract has joined #openstack-cinder | 09:41 | |
| *** dalgaaf has quit IRC | 09:43 | |
| *** dalgaaf has joined #openstack-cinder | 09:43 | |
| *** mvk has quit IRC | 09:49 | |
| *** zhurong has quit IRC | 09:55 | |
| *** alexchadin has joined #openstack-cinder | 10:09 | |
| *** alexchadin has quit IRC | 10:10 | |
| *** zhurong has joined #openstack-cinder | 10:10 | |
| *** e0ne has joined #openstack-cinder | 10:13 | |
| *** guyk has quit IRC | 10:16 | |
| *** markvoelker has joined #openstack-cinder | 10:19 | |
| *** mvk has joined #openstack-cinder | 10:19 | |
| *** stakeda has quit IRC | 10:30 | |
| openstackgerrit | Kushal Wathore proposed openstack/cinder master: HPE 3PAR: fix delete operation of replicated volume https://review.openstack.org/506102 | 10:31 |
| *** markstur has joined #openstack-cinder | 10:34 | |
| openstackgerrit | Kushal Wathore proposed openstack/cinder master: HPE 3PAR: fix delete operation of replicated volume https://review.openstack.org/506102 | 10:37 |
| *** markstur has quit IRC | 10:38 | |
| *** bigdogstl has joined #openstack-cinder | 10:38 | |
| *** bigdogstl has quit IRC | 10:45 | |
| *** lhx_ has joined #openstack-cinder | 10:51 | |
| *** markvoelker has quit IRC | 10:52 | |
| *** lhx__ has quit IRC | 10:52 | |
| *** sapd has quit IRC | 10:59 | |
| *** chenying_ has quit IRC | 10:59 | |
| *** chenying_ has joined #openstack-cinder | 11:00 | |
| *** zhaochao has quit IRC | 11:01 | |
| *** ociuhandu_ has quit IRC | 11:15 | |
| *** gkadam has quit IRC | 11:21 | |
| *** sapd has joined #openstack-cinder | 11:32 | |
| *** ociuhandu has joined #openstack-cinder | 11:33 | |
| *** kbaegis has joined #openstack-cinder | 11:36 | |
| *** sdague has joined #openstack-cinder | 11:38 | |
| *** sapd has quit IRC | 11:39 | |
| *** zhaochao has joined #openstack-cinder | 11:41 | |
| *** sapd has joined #openstack-cinder | 11:43 | |
| *** tpsilva has joined #openstack-cinder | 11:49 | |
| *** markvoelker has joined #openstack-cinder | 11:49 | |
| *** zhurong has quit IRC | 11:51 | |
| *** abishop has joined #openstack-cinder | 11:51 | |
| *** chlong has joined #openstack-cinder | 11:53 | |
| *** markvoelker has quit IRC | 11:54 | |
| *** sapd has quit IRC | 11:54 | |
| *** abishop_ has joined #openstack-cinder | 11:54 | |
| *** sapd has joined #openstack-cinder | 11:55 | |
| *** sapd has quit IRC | 11:56 | |
| *** abishop has quit IRC | 11:56 | |
| *** sapd has joined #openstack-cinder | 11:56 | |
| *** markvoelker has joined #openstack-cinder | 11:56 | |
| *** kaisers__ has quit IRC | 11:58 | |
| *** kaisers_ has joined #openstack-cinder | 11:59 | |
| *** psachin has quit IRC | 12:06 | |
| *** adrianofr has joined #openstack-cinder | 12:06 | |
| *** gkadam has joined #openstack-cinder | 12:06 | |
| *** dave-mccowan has joined #openstack-cinder | 12:09 | |
| *** bigdogstl has joined #openstack-cinder | 12:10 | |
| *** edmondsw has joined #openstack-cinder | 12:13 | |
| *** markstur has joined #openstack-cinder | 12:22 | |
| openstackgerrit | TommyLike proposed openstack/cinder-specs master: Update backup's size when backup is created https://review.openstack.org/506141 | 12:22 |
| *** ociuhandu has quit IRC | 12:25 | |
| *** adrianofr has quit IRC | 12:25 | |
| *** markstur has quit IRC | 12:27 | |
| *** mchlumsky has joined #openstack-cinder | 12:31 | |
| *** gnufied has joined #openstack-cinder | 12:32 | |
| *** MVenesio has joined #openstack-cinder | 12:34 | |
| *** erlon has joined #openstack-cinder | 12:34 | |
| *** catintheroof has joined #openstack-cinder | 12:41 | |
| *** kbaegis has quit IRC | 12:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 12:44 |
| openstackgerrit | TommyLike proposed openstack/cinder-specs master: Update backup's size when backup is created https://review.openstack.org/506141 | 12:44 |
| *** mriedem has joined #openstack-cinder | 12:46 | |
| *** bigdogstl has quit IRC | 12:46 | |
| *** ociuhandu has joined #openstack-cinder | 12:48 | |
| *** xyang1 has joined #openstack-cinder | 12:51 | |
| *** xyang has joined #openstack-cinder | 12:51 | |
| *** kaisers__ has joined #openstack-cinder | 12:51 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/505864 | 12:53 |
| *** ganso has joined #openstack-cinder | 12:54 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/502732 | 12:54 |
| *** kaisers_ has quit IRC | 12:54 | |
| *** ociuhandu has quit IRC | 12:54 | |
| *** xyang has quit IRC | 12:55 | |
| *** gkadam has quit IRC | 12:58 | |
| *** gkadam has joined #openstack-cinder | 12:59 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 12:59 |
| *** xyang has joined #openstack-cinder | 13:01 | |
| *** gkadam has quit IRC | 13:02 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/cinder master: Add default configuration files to data_files https://review.openstack.org/506155 | 13:02 |
| *** gkadam has joined #openstack-cinder | 13:02 | |
| *** xyang has quit IRC | 13:05 | |
| *** gman-tx has joined #openstack-cinder | 13:06 | |
| *** dustins has joined #openstack-cinder | 13:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/505864 | 13:09 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/502732 | 13:10 |
| *** xyang has joined #openstack-cinder | 13:13 | |
| *** ociuhandu has joined #openstack-cinder | 13:14 | |
| *** gman-tx has quit IRC | 13:14 | |
| *** lucasxu has joined #openstack-cinder | 13:16 | |
| *** xyang has quit IRC | 13:17 | |
| openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/cinder master: Add default configuration files to data_files https://review.openstack.org/506155 | 13:25 |
| *** gouthamr has joined #openstack-cinder | 13:25 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 13:27 |
| *** xyang has joined #openstack-cinder | 13:29 | |
| *** chhavi has joined #openstack-cinder | 13:31 | |
| chhavi | smcginnis: please review:https://review.openstack.org/#/c/504482/ | 13:32 |
| *** xyang has quit IRC | 13:33 | |
| *** jmlowe has quit IRC | 13:35 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/505864 | 13:36 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/502732 | 13:37 |
| *** xyang has joined #openstack-cinder | 13:40 | |
| *** xyang has quit IRC | 13:43 | |
| *** bigdogstl has joined #openstack-cinder | 13:43 | |
| *** xyang has joined #openstack-cinder | 13:43 | |
| *** xyang has quit IRC | 13:43 | |
| *** gman-tx has joined #openstack-cinder | 13:47 | |
| *** bigdogstl has quit IRC | 13:47 | |
| *** lucasxu has quit IRC | 13:54 | |
| *** yangyapeng has joined #openstack-cinder | 13:59 | |
| openstackgerrit | TommyLike proposed openstack/cinder-specs master: Support create volume from backup https://review.openstack.org/506004 | 14:02 |
| *** josecastroleon has quit IRC | 14:02 | |
| *** lucasxu has joined #openstack-cinder | 14:03 | |
| ildikov | Morning/Afternoon/Evening | 14:03 |
| ildikov | Is there anyone who could turn on the light in my head about conditional_update by any chance? | 14:03 |
| smcginnis | ildikov: geguileo is probably the expert there, but I may be able to answer some questions. | 14:04 |
| geguileo | I'm on PTO today... | 14:05 |
| ildikov | smcginnis: so I'm trying to figure out what status information gets updated there | 14:05 |
| geguileo | ildikov: in what query? | 14:05 |
| ildikov | geguileo: well, by answering here we've just started to doubt that ;) | 14:05 |
| smcginnis | :D | 14:05 |
| ildikov | like in attachment_reserve here: https://review.openstack.org/#/c/504467/5/cinder/volume/api.py | 14:06 |
| geguileo | ildikov: I wrote some documentation for it: https://docs.openstack.org/cinder/ocata/devref/api_conditional_updates.html | 14:06 |
| geguileo | ildikov: but I'll have a look | 14:06 |
| ildikov | and in beging_detaching here: https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L701 | 14:07 |
| ildikov | geguileo: thanks | 14:07 |
| smcginnis | ildikov: So the status information that gets updated is whatever you pass in. | 14:07 |
| smcginnis | ildikov: But only if it matches the expected status. | 14:07 |
| geguileo | ildikov: ok, the attachment_reserve is updating the status to reserved if status is available or downloading | 14:08 |
| smcginnis | ildikov: Kind of a safety to say update this as long as nothing else has done something since we last checked. | 14:08 |
| ildikov | smcginnis: so if we have 'status' in the volume and the attachment too then both gets updated? | 14:08 |
| ildikov | or just the volume? | 14:08 |
| smcginnis | ildikov: Just the volume I believe. | 14:08 |
| geguileo | ildikov: the call is using vref to do the update, which is the volume reference, so only the volume in the conditional update | 14:08 |
| ildikov | geguileo: smcginnis: so my question is basically whether the attachment status gets updated too or not? | 14:09 |
| ildikov | smcginnis: geguileo: hmm, that's not good | 14:09 |
| geguileo | ildikov: you are reserving a volume for attach, so there could be no attach yet, right? | 14:09 |
| ildikov | however that's what I thought as well | 14:09 |
| ildikov | geguileo: in a multi-attach case there can be attachments in attached state already | 14:10 |
| ildikov | geguileo: which we don't have enabled yet, but we're preparing the Cinder side to support it, so we need to figure these out now | 14:10 |
| geguileo | ildikov: but those wouldn't have to change on a reserve call, because they are not reserved, they are attached | 14:10 |
| smcginnis | ildikov: So it would need to also handle in-use status, but only if multiattach? | 14:11 |
| geguileo | ildikov: maybe I'm reading it wrong, let me check and see what volume_attach on the DB does | 14:11 |
| *** bigdogstl has joined #openstack-cinder | 14:12 | |
| ildikov | geguileo: no, I'm only trying to understand whether anyone updates the attachment we're modifying as we rely on the attachment's attach_status in attachment_delete | 14:12 |
| ildikov | or well, planning to rely on it | 14:12 |
| geguileo | ildikov: OK, that call also creates an attachment entry and sets it to reserved | 14:12 |
| ildikov | but I don't think that for instance begin_detaching ever touches the attachment's status information | 14:12 |
| geguileo | ildikov: so the attachment_reserve would change volume status to reserved if it's not 'in-use' or 'reserved' and then always create an attachment with reserved status | 14:13 |
| ildikov | geguileo: ok, I see now, missed the volume_attach bit | 14:13 |
| ildikov | geguileo: is there anything I'm missing for begin_detaching too: https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L701 ? | 14:14 |
| *** crushil_ has joined #openstack-cinder | 14:14 | |
| geguileo | ildikov: but the conditional update there is not good, because you could have race conditions because it's checking on the node the vref.status | 14:14 |
| *** crushil has joined #openstack-cinder | 14:14 | |
| *** armax has joined #openstack-cinder | 14:14 | |
| geguileo | ildikov: nop, that one isn't touching the attachments | 14:14 |
| *** crushil has quit IRC | 14:15 | |
| *** crushil_ is now known as crushil | 14:15 | |
| ildikov | geguileo: if you could please add a comment about the race condition to the review that would be great | 14:15 |
| geguileo | ildikov: I was logging in to do that ;-) | 14:15 |
| ildikov | geguileo: as you can see, I'm struggling at this point with just understanding what's going on :) | 14:15 |
| ildikov | geguileo: thanks much! | 14:15 |
| geguileo | ildikov: np | 14:15 |
| ildikov | geguileo: I owe you one | 14:16 |
| *** bigdogstl has quit IRC | 14:16 | |
| ildikov | geguileo: sorry for partially ruining your PTO :/ | 14:16 |
| *** Yikun has quit IRC | 14:16 | |
| geguileo | ildikov: it was an easy one, no problem | 14:16 |
| ildikov | geguileo: thank you :) | 14:19 |
| *** itlinux has joined #openstack-cinder | 14:23 | |
| *** itlinux has quit IRC | 14:23 | |
| openstackgerrit | zhangyangyang proposed openstack/python-brick-cinderclient-ext master: Cleanup test-requirements https://review.openstack.org/506212 | 14:25 |
| *** jmlowe has joined #openstack-cinder | 14:28 | |
| *** openstackgerrit has quit IRC | 14:33 | |
| *** felipemonteiro has joined #openstack-cinder | 14:38 | |
| *** felipemonteiro_ has joined #openstack-cinder | 14:39 | |
| jgriffith | Well leave it to Cinder and attachment stuff to ruin my day early on | 14:41 |
| *** itlinux has joined #openstack-cinder | 14:41 | |
| jgriffith | If you want something quick; this patch provides one direction. If you want something that's actually "more good" then there's a good deal of work that needs to be done (and yes I had always intended to do it, but I didn't think it would take 144 iterations and almost a year to get the changes in Nova). | 14:41 |
| jgriffith | AND I didn't think that the entire design of the new attachment API's would end up being nothing more than a reimplimentation of the existing garbage | 14:42 |
| *** felipemonteiro has quit IRC | 14:43 | |
| jgriffith | ildikov I'm not sure on your suggestions around the detach state updating stuff.... I mean I get what you're saying and it makes sense | 14:43 |
| *** itlinux has quit IRC | 14:43 | |
| jgriffith | ildikov BUT. it suggests that the path I took there to solve that may not be the right path at all | 14:43 |
| ildikov | jgriffith: I haven't finished my first coffee yet, so I'm not sure I got your last comment | 14:43 |
| jgriffith | it's already far to convoluted IMO | 14:44 |
| jgriffith | ildikov haha :) | 14:44 |
| * jgriffith hasn't had his either so he's grumpy | 14:44 | |
| jgriffith | https://review.openstack.org/#/c/504467/5/cinder/volume/api.py L#2048 | 14:44 |
| ildikov | jgriffith: :( | 14:45 |
| ildikov | jgriffith: I have some left if you want | 14:45 |
| jgriffith | it's probably "good" coffee too isn't it? | 14:45 |
| ildikov | jgriffith: or I can make it fresh if that helps making you less grumpy or not grumpy at all :) | 14:45 |
| ildikov | jgriffith: I don't drink bad coffee :) | 14:46 |
| jgriffith | That would likely do the trick | 14:46 |
| jgriffith | haha!! | 14:46 |
| ildikov | jgriffith: so you're more than welcome | 14:46 |
| jgriffith | might take you up on that | 14:47 |
| ildikov | we can then figure out the everything's state mess too, leave it behind and have you as a happy person again | 14:48 |
| jgriffith | The coffee would probably be sufficient | 14:49 |
| jgriffith | why ruin good coffee with talk of cinder's attachment status stuff :) | 14:49 |
| ildikov | I'm sold :) | 14:49 |
| ildikov | jgriffith: when do you want the fresh coffee to be ready then? | 14:51 |
| jgriffith | 10 minutes ago :) | 14:52 |
| *** felipemonteiro_ has quit IRC | 14:59 | |
| *** felipemonteiro_ has joined #openstack-cinder | 14:59 | |
| *** markstur has joined #openstack-cinder | 15:02 | |
| *** markstur has quit IRC | 15:02 | |
| *** markstur has joined #openstack-cinder | 15:03 | |
| _alastor_ | hey peeps! I'm back from paternity leave, what'd I miss :) | 15:05 |
| smcginnis | _alastor_: Congrats! | 15:05 |
| _alastor_ | smcginnis: thanks! | 15:06 |
| *** links has quit IRC | 15:07 | |
| _alastor_ | Obligitory photo of the kiddo in our overly complicated stroller: https://photos.app.goo.gl/1vmd8wAY8IQ3oSI02 | 15:09 |
| _alastor_ | Fun fact: Baby car seats have an expiration date. You're supposed to buy a new one when they expire | 15:10 |
| *** felipemonteiro has joined #openstack-cinder | 15:11 | |
| *** ociuhandu has quit IRC | 15:12 | |
| *** bigdogstl has joined #openstack-cinder | 15:12 | |
| *** felipemonteiro_ has quit IRC | 15:13 | |
| *** bigdogstl has quit IRC | 15:17 | |
| smcginnis | _alastor_: Nice. :) | 15:19 |
| _alastor_ | Proposal: Add child-safety features to OpenStack. It'll allow charging three times as much since nobody wants to "endanger kids" by not ponying up for it ;) | 15:21 |
| *** felipemonteiro_ has joined #openstack-cinder | 15:24 | |
| *** bigdogstl has joined #openstack-cinder | 15:25 | |
| *** felipemonteiro has quit IRC | 15:27 | |
| *** bigdogstl has quit IRC | 15:30 | |
| *** gman-tx has quit IRC | 15:32 | |
| *** felipemonteiro_ has quit IRC | 15:32 | |
| *** felipemonteiro_ has joined #openstack-cinder | 15:33 | |
| *** felipemonteiro has joined #openstack-cinder | 15:34 | |
| *** bigdogstl has joined #openstack-cinder | 15:34 | |
| *** gman-tx has joined #openstack-cinder | 15:35 | |
| *** felipemonteiro_ has quit IRC | 15:37 | |
| *** zhaochao has quit IRC | 15:38 | |
| *** bigdogstl has quit IRC | 15:39 | |
| *** e0ne has quit IRC | 15:40 | |
| *** ntpttr_laptop has joined #openstack-cinder | 15:41 | |
| *** felipemonteiro has quit IRC | 15:49 | |
| *** felipemonteiro_ has joined #openstack-cinder | 15:49 | |
| jgriffith | _alastor_ great idea... but 3x0 is still 0 :) | 15:52 |
| jgriffith | _alastor_ congratulations! And welcome back! | 15:52 |
| *** bigdogstl has joined #openstack-cinder | 15:52 | |
| *** felipemonteiro has joined #openstack-cinder | 15:53 | |
| *** felipemonteiro_ has quit IRC | 15:56 | |
| *** bigdogstl has quit IRC | 15:58 | |
| *** sbezverk has quit IRC | 15:59 | |
| *** sbezverk has joined #openstack-cinder | 16:01 | |
| *** openstackgerrit has joined #openstack-cinder | 16:10 | |
| openstackgerrit | Merged openstack/cinder master: RBD: get provisioned capacity using same connection https://review.openstack.org/500047 | 16:10 |
| *** bigdogstl has joined #openstack-cinder | 16:14 | |
| *** zhouyaguo has joined #openstack-cinder | 16:14 | |
| *** Apoorva has joined #openstack-cinder | 16:18 | |
| openstackgerrit | John Griffith proposed openstack/python-cinderclient master: Implement UserID in snapshot list response https://review.openstack.org/501897 | 16:19 |
| *** bigdogstl has quit IRC | 16:20 | |
| _alastor_ | jgriffith: thanks! | 16:22 |
| _alastor_ | jgriffith: I was thinking maybe a tiered feature set like Volume-Types, but for Children. This allows service down the line where vendors charge extra for the additional child-safety features, but still have a saftey-optional tier for those child-hating money-grubbers that don't pony up for children in their storage arrays :P | 16:24 |
| openstackgerrit | Yaguo Zhou proposed openstack/cinder master: Upgrade oslo.db and Replace DbMigrationError with DBMigrationError https://review.openstack.org/500881 | 16:26 |
| *** bigdogstl has joined #openstack-cinder | 16:33 | |
| *** zhouyaguo has quit IRC | 16:39 | |
| *** bigdogstl has quit IRC | 16:42 | |
| jgriffith | _alastor_ it would fit right in with many of the proposals as of late :) | 16:44 |
| *** itlinux has joined #openstack-cinder | 16:45 | |
| *** dustins_ has joined #openstack-cinder | 16:45 | |
| *** tesseract has quit IRC | 16:50 | |
| *** gman-tx has quit IRC | 16:52 | |
| *** rcernin has quit IRC | 16:52 | |
| *** dave-mcc_ has joined #openstack-cinder | 16:55 | |
| *** Swanson has quit IRC | 16:57 | |
| *** dave-mccowan has quit IRC | 16:58 | |
| *** mvk has quit IRC | 17:01 | |
| jungleboyj | jgriffith: Did you want to look through the MV Constants patch? https://review.openstack.org/#/c/505420/2 | 17:02 |
| jungleboyj | I went through it pretty carefully this morning and it looks ok. | 17:02 |
| jungleboyj | Or any other cores? hemna | 17:03 |
| jgriffith | jungleboyj yeah, will do, thanks! | 17:03 |
| jungleboyj | jgriffith: Ok, there is only one issue, there is one place where he used 'microversion' for the import instead of 'mv' . Not worth redoing the patch, it can be easily fixed. | 17:04 |
| *** ntpttr_laptop has quit IRC | 17:04 | |
| jungleboyj | So, go ahead and merge if if you feel good. | 17:04 |
| jgriffith | Will need some time to review it, but ack | 17:04 |
| ildikov | _alastor_: congrats! and ooooh what a cuteness is there on that photo!!! :) | 17:04 |
| jungleboyj | jgriffith: Cool. | 17:04 |
| jungleboyj | _alastor_: Welcome back! Congratulations! Wondered why we didn't see you at the PTG. | 17:06 |
| jungleboyj | _alastor_: Hope fatherhood is treating you well. There is no other experience like it. | 17:07 |
| *** Swanson has joined #openstack-cinder | 17:07 | |
| _alastor_ | jungleboyj: It's certainly been quite the ride so far. 2-hours between feedings is about what we get these days | 17:07 |
| _alastor_ | jungleboyj: we get one ~4 hour solid sleep block each day which is marvelous | 17:09 |
| *** ntpttr_laptop has joined #openstack-cinder | 17:09 | |
| *** bigdogstl has joined #openstack-cinder | 17:12 | |
| jungleboyj | _alastor_: You block all that from your memory and the next think you know they are 7 or 12 years old and you have totally new challenges. | 17:13 |
| jgriffith | smcginnis jungleboyj missed a couple in the view builders | 17:14 |
| jgriffith | and I added some comments/questions about possible follow up work | 17:15 |
| jungleboyj | jgriffith: Cool. Thanks for looking at it. | 17:15 |
| jgriffith | The only thing that warranted the -1 for me were the couple of items on the API side of things; the other stuff would be awesome to follow up with | 17:15 |
| *** bigdogstl has quit IRC | 17:17 | |
| smcginnis | jgriffith: Which view builders? | 17:17 |
| *** chenying_ has quit IRC | 17:17 | |
| jgriffith | smcginnis v3/views/groups.py and v3/views/snapshots.py | 17:17 |
| *** chenying_ has joined #openstack-cinder | 17:18 | |
| *** Aditya has joined #openstack-cinder | 17:18 | |
| jgriffith | 3.38 and 3.41 respectively | 17:18 |
| smcginnis | Huh, must have picked up after rebasing or something. I'll clean that up. | 17:18 |
| *** itlinux has quit IRC | 17:18 | |
| jgriffith | smcginnis honestly I'm rather impressed that I only found 2 :) | 17:19 |
| smcginnis | heh | 17:19 |
| * jgriffith bows to smcginnis 's attention to detail | 17:19 | |
| jungleboyj | smcginnis: I updated my vote. Can we change that one place that was inconsistent since you need to push up another patch? | 17:20 |
| smcginnis | jungleboyj: Which one? | 17:20 |
| jungleboyj | smcginnis: https://review.openstack.org/#/c/505420/2/cinder/api/contrib/volume_actions.py | 17:21 |
| smcginnis | jungleboyj: I guess if you think that matters. :D | 17:21 |
| jungleboyj | jgriffith: You want me to put your future work proposals into next week's weekly meeting to discuss? | 17:21 |
| jungleboyj | :-p | 17:21 |
| *** felipemonteiro_ has joined #openstack-cinder | 17:22 | |
| *** felipemonteiro has quit IRC | 17:22 | |
| *** felipemonteiro has joined #openstack-cinder | 17:25 | |
| smcginnis | jgriffith: Saw you comments on there. Yeah, I was originally hoping to do something clever with microversion objects that would handle a lot of things, but that got too bloaty. | 17:26 |
| smcginnis | jgriffith: But I was hoping to just use docstrings somehow to automatically document and publish microversion information. | 17:26 |
| *** felipemonteiro_ has quit IRC | 17:28 | |
| jungleboyj | smcginnis: Yeah, that would be nice to get in there somehow. | 17:28 |
| jungleboyj | Rather than having to have them go update documentation. | 17:29 |
| smcginnis | Would be ideal to have it all in one place. | 17:29 |
| jungleboyj | Yeah, would be nice if we had "Constant, number, description" in here: https://review.openstack.org/#/c/505420/2/cinder/api/microversions.py | 17:32 |
| jungleboyj | I don't have a good idea for how to do that though. | 17:33 |
| *** erlon has quit IRC | 17:37 | |
| jgriffith | smcginnis I hear ya, I wouldn't want to see it all in that one patch anyway. Like I said; what you have there is AWESOME IMO and a huge help! I just thought it would be fantastic if somebody else saw that and took it another step further | 17:38 |
| *** crushil has quit IRC | 17:38 | |
| *** crushil has joined #openstack-cinder | 17:38 | |
| *** mvk has joined #openstack-cinder | 17:41 | |
| *** gman-tx has joined #openstack-cinder | 17:44 | |
| *** diablo_rojo has joined #openstack-cinder | 17:53 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/cinder master: NetApp ONTAP: Fix reporting of provisioned_capacity_gb https://review.openstack.org/505672 | 17:55 |
| *** diablo_rojo has quit IRC | 17:57 | |
| *** diablo_rojo has joined #openstack-cinder | 17:57 | |
| *** gman-tx has quit IRC | 17:57 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Use constants for microversion values https://review.openstack.org/505420 | 17:59 |
| *** chhavi has quit IRC | 17:59 | |
| *** crushil has quit IRC | 18:06 | |
| *** bigdogstl has joined #openstack-cinder | 18:06 | |
| *** gman-tx has joined #openstack-cinder | 18:11 | |
| *** bigdogstl has quit IRC | 18:14 | |
| *** tonygunk has joined #openstack-cinder | 18:15 | |
| -openstackstatus- NOTICE: Gerrit OpenIDs have been accidentally overwritten and are in the process of being restored | 18:16 | |
| smcginnis | Hah, oops. | 18:18 |
| ildikov | time to get some ice cream then I guess | 18:19 |
| -openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being taken offline briefly to perform database repair work but should be back up shortly | 18:21 | |
| *** ChanServ changes topic to "The Gerrit service on review.openstack.org is being taken offline briefly to perform database repair work but should be back up shortly" | 18:21 | |
| *** kambiz has joined #openstack-cinder | 18:24 | |
| *** tonygunk has quit IRC | 18:25 | |
| jungleboyj | That isn't good. | 18:26 |
| jungleboyj | I wonder if someone fell for little Jimmy DropTables ? | 18:27 |
| *** crushil has joined #openstack-cinder | 18:28 | |
| *** Aditya has quit IRC | 18:29 | |
| *** ChubYann has joined #openstack-cinder | 18:30 | |
| smcginnis | ildikov: Not that it's not a good time for ice cream, but apparently only affects new logins. ;) | 18:30 |
| *** ntpttr_laptop has quit IRC | 18:44 | |
| ildikov | smcginnis: well, I couldn't load the page, and that's a good enough reason for me :) | 18:44 |
| *** armax has quit IRC | 18:44 | |
| *** armax has joined #openstack-cinder | 18:44 | |
| smcginnis | ildikov: If you're still in CO, just outside Lyon's there's a distillery that makes a coffee liquor. It goes really well over some vanilla ice cream. ;) | 18:46 |
| jungleboyj | smcginnis: Oh man, that sounds good. | 18:46 |
| ildikov | smcginnis: noted :) | 18:46 |
| smcginnis | http://www.spirithounds.com/coffee-liqueur | 18:47 |
| jungleboyj | Nice. | 18:52 |
| jungleboyj | ildikov: Don't have that before driving up Mt. Evans. ;-) | 18:52 |
| ildikov | jungleboyj: nah, I'll have it on the top | 18:54 |
| jungleboyj | There you go. :-) | 18:54 |
| ildikov | jungleboyj: makes the ride down more fun I guess ;) | 18:54 |
| *** mriedem1 has joined #openstack-cinder | 18:56 | |
| *** mriedem has quit IRC | 18:56 | |
| *** Apoorva_ has joined #openstack-cinder | 19:01 | |
| *** Apoorva has quit IRC | 19:04 | |
| *** mriedem1 is now known as mriedem | 19:07 | |
| *** bigdogstl has joined #openstack-cinder | 19:11 | |
| *** bigdogstl has quit IRC | 19:16 | |
| *** ChanServ changes topic to "The Block Storage Project | https://wiki.openstack.org/wiki/Cinder" | 19:18 | |
| -openstackstatus- NOTICE: OpenIDs for the Gerrit service have been restored from a recent backup and the service is running again; before/after table states are being analyzed now to identify any remaining cleanup needed for changes made to accounts today | 19:18 | |
| *** e0ne has joined #openstack-cinder | 19:21 | |
| openstackgerrit | Merged openstack/python-cinderclient master: Implement UserID in snapshot list response https://review.openstack.org/501897 | 19:26 |
| *** xyang1 has quit IRC | 19:26 | |
| *** e0ne has quit IRC | 19:31 | |
| *** xyang1 has joined #openstack-cinder | 19:37 | |
| *** gman-tx has quit IRC | 19:37 | |
| *** pcaruana has quit IRC | 19:41 | |
| *** crushil has quit IRC | 19:43 | |
| *** crushil has joined #openstack-cinder | 19:44 | |
| *** dave-mcc_ is now known as dave-mccowan | 19:45 | |
| *** gman-tx has joined #openstack-cinder | 19:46 | |
| *** lhx_ has quit IRC | 19:53 | |
| *** lhx__ has joined #openstack-cinder | 19:53 | |
| *** wanghao_ has joined #openstack-cinder | 20:01 | |
| *** jmlowe has quit IRC | 20:02 | |
| *** gman-tx has quit IRC | 20:03 | |
| *** wanghao has quit IRC | 20:03 | |
| *** mvk has quit IRC | 20:05 | |
| *** gman-tx has joined #openstack-cinder | 20:05 | |
| *** crushil has quit IRC | 20:09 | |
| *** crushil has joined #openstack-cinder | 20:09 | |
| *** mvk has joined #openstack-cinder | 20:17 | |
| *** armax has quit IRC | 20:18 | |
| *** ntpttr_laptop has joined #openstack-cinder | 20:22 | |
| *** crushil has quit IRC | 20:26 | |
| *** gouthamr has quit IRC | 20:28 | |
| *** dustins has quit IRC | 20:29 | |
| *** abishop_ has quit IRC | 20:43 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 20:49 |
| *** edmondsw has quit IRC | 20:51 | |
| *** gman-tx has quit IRC | 20:55 | |
| *** Apoorva_ has quit IRC | 20:55 | |
| *** Apoorva has joined #openstack-cinder | 20:55 | |
| *** edmondsw has joined #openstack-cinder | 20:57 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/505864 | 20:58 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/502732 | 20:59 |
| *** Apoorva_ has joined #openstack-cinder | 21:01 | |
| *** edmondsw has quit IRC | 21:01 | |
| *** Apoorva has quit IRC | 21:04 | |
| *** lucasxu has quit IRC | 21:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 21:06 |
| *** catintheroof has quit IRC | 21:07 | |
| *** bigdogstl has joined #openstack-cinder | 21:13 | |
| *** mchlumsky has quit IRC | 21:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/505864 | 21:15 |
| *** felipemonteiro has quit IRC | 21:16 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/502732 | 21:16 |
| *** bigdogstl has quit IRC | 21:18 | |
| *** felipemonteiro_ has joined #openstack-cinder | 21:18 | |
| *** felipemonteiro__ has joined #openstack-cinder | 21:19 | |
| *** dustins_ has quit IRC | 21:23 | |
| *** felipemonteiro_ has quit IRC | 21:23 | |
| *** armax has joined #openstack-cinder | 21:28 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/503206 | 21:34 |
| *** Apoorva_ has quit IRC | 21:34 | |
| *** Apoorva has joined #openstack-cinder | 21:35 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements https://review.openstack.org/505864 | 21:43 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient master: Updated from global requirements https://review.openstack.org/502732 | 21:44 |
| *** zhenguo has quit IRC | 21:45 | |
| *** adisky has quit IRC | 21:46 | |
| *** simondodsley has quit IRC | 21:46 | |
| *** jbryce has quit IRC | 21:46 | |
| *** enriquetaso has quit IRC | 21:47 | |
| *** evrardjp has quit IRC | 21:47 | |
| *** twouters has quit IRC | 21:47 | |
| *** dr_gogeta86 has quit IRC | 21:48 | |
| *** sweston has quit IRC | 21:48 | |
| *** simondodsley has joined #openstack-cinder | 21:48 | |
| *** adisky has joined #openstack-cinder | 21:48 | |
| *** enriquetaso has joined #openstack-cinder | 21:49 | |
| *** pkoniszewski has quit IRC | 21:49 | |
| *** MVenesio has quit IRC | 21:49 | |
| *** dalgaaf has quit IRC | 21:50 | |
| *** sweston has joined #openstack-cinder | 21:50 | |
| *** dalgaaf has joined #openstack-cinder | 21:51 | |
| *** pkoniszewski has joined #openstack-cinder | 21:51 | |
| *** tpsilva has quit IRC | 21:51 | |
| *** evrardjp has joined #openstack-cinder | 21:52 | |
| *** twouters has joined #openstack-cinder | 21:53 | |
| *** twouters has quit IRC | 21:53 | |
| *** twouters has joined #openstack-cinder | 21:53 | |
| *** dr_gogeta86 has joined #openstack-cinder | 21:53 | |
| *** dr_gogeta86 has quit IRC | 21:53 | |
| *** dr_gogeta86 has joined #openstack-cinder | 21:53 | |
| *** felipemonteiro__ has quit IRC | 21:59 | |
| *** felipemonteiro__ has joined #openstack-cinder | 22:00 | |
| *** zhenguo has joined #openstack-cinder | 22:01 | |
| *** felipemonteiro__ has quit IRC | 22:06 | |
| tommylikehu | hey jungleboyj , smcginnis could you take a look at this patch again? https://review.openstack.org/#/c/505555/ | 22:07 |
| tommylikehu | wow, it's a nice review id.... | 22:08 |
| smcginnis | ;) | 22:12 |
| *** bigdogstl has joined #openstack-cinder | 22:16 | |
| *** bigdogstl has quit IRC | 22:23 | |
| *** sdague has quit IRC | 22:34 | |
| *** mriedem has quit IRC | 22:37 | |
| *** xyang1 has quit IRC | 22:38 | |
| stvnoyes | jgriffith: Hi john, is there a recommended way for a client to determine whether an attachment is the new or old style? In my tempest test, I want to be able to tell the difference. | 22:42 |
| openstackgerrit | Kaitlin Farr proposed openstack/cinder master: Remove deprecated keymgr code https://review.openstack.org/468252 | 22:49 |
| *** bigdogstl has joined #openstack-cinder | 22:53 | |
| *** edmondsw has joined #openstack-cinder | 22:58 | |
| *** bigdogstl has quit IRC | 23:00 | |
| *** edmondsw has quit IRC | 23:02 | |
| *** bigdogstl has joined #openstack-cinder | 23:06 | |
| *** bigdogstl has quit IRC | 23:11 | |
| openstackgerrit | Kaitlin Farr proposed openstack/cinder master: Remove deprecated keymgr code https://review.openstack.org/468252 | 23:20 |
| *** jmlowe has joined #openstack-cinder | 23:35 | |
| *** gouthamr has joined #openstack-cinder | 23:40 | |
| *** markvoelker has quit IRC | 23:40 | |
| *** bigdogstl has joined #openstack-cinder | 23:41 | |
| *** jbryce has joined #openstack-cinder | 23:45 | |
| *** bigdogstl has quit IRC | 23:50 | |
| *** chhavi has joined #openstack-cinder | 23:55 | |
| *** ntpttr_laptop has quit IRC | 23:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!