| *** cknight has quit IRC | 00:05 | |
| *** lamt has quit IRC | 00:06 | |
| *** lamt has joined #openstack-cinder | 00:08 | |
| *** lamt has quit IRC | 00:10 | |
| *** lamt has joined #openstack-cinder | 00:11 | |
| *** lamt has quit IRC | 00:11 | |
| *** crushil has quit IRC | 00:12 | |
| *** chris_morrell has quit IRC | 00:13 | |
| *** wanghao has joined #openstack-cinder | 00:15 | |
| *** winston-d_ has joined #openstack-cinder | 00:37 | |
| *** zhurong has joined #openstack-cinder | 00:40 | |
| *** Apoorva_ has quit IRC | 00:42 | |
| *** Apoorva has joined #openstack-cinder | 00:43 | |
| *** zhangguoqing has joined #openstack-cinder | 00:51 | |
| *** zhangguoqing has quit IRC | 00:51 | |
| *** zhangguoqing has joined #openstack-cinder | 00:51 | |
| *** chhavi has joined #openstack-cinder | 00:54 | |
| *** cknight has joined #openstack-cinder | 00:55 | |
| *** huanxuan has joined #openstack-cinder | 00:57 | |
| *** zengyingzhe__ has quit IRC | 01:00 | |
| *** zengyingzhe__ has joined #openstack-cinder | 01:00 | |
| *** zhangguoqing has quit IRC | 01:02 | |
| *** crushil has joined #openstack-cinder | 01:07 | |
| *** wanghao has quit IRC | 01:10 | |
| *** Jack_Iv has joined #openstack-cinder | 01:10 | |
| *** wanghao has joined #openstack-cinder | 01:11 | |
| *** Jack_Iv has quit IRC | 01:14 | |
| *** chhavi has quit IRC | 01:17 | |
| *** yangyapeng has joined #openstack-cinder | 01:17 | |
| *** liverpooler has quit IRC | 01:19 | |
| *** lucasxu has joined #openstack-cinder | 01:24 | |
| *** gcb has joined #openstack-cinder | 01:32 | |
| *** zhangguoqing has joined #openstack-cinder | 01:32 | |
| *** kaisers_ has joined #openstack-cinder | 01:36 | |
| *** lucasxu has quit IRC | 01:38 | |
| *** kaisers has quit IRC | 01:39 | |
| *** lucasxu has joined #openstack-cinder | 01:39 | |
| *** kaisers_ has quit IRC | 01:42 | |
| *** wanghao has quit IRC | 01:43 | |
| *** wanghao has joined #openstack-cinder | 01:44 | |
| *** lucasxu has quit IRC | 01:46 | |
| *** lucasxu has joined #openstack-cinder | 01:46 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: [BugFix] Add method policy in attachment APIs https://review.openstack.org/455101 | 01:47 |
|---|---|---|
| *** dillaman has quit IRC | 01:49 | |
| *** s-shiono_ has joined #openstack-cinder | 01:54 | |
| *** s-shiono has quit IRC | 01:57 | |
| *** wanghao has quit IRC | 01:58 | |
| *** dillaman has joined #openstack-cinder | 02:04 | |
| *** jamielennox is now known as jamielennox|away | 02:07 | |
| *** Apoorva_ has joined #openstack-cinder | 02:09 | |
| *** Apoorva has quit IRC | 02:13 | |
| *** Apoorva_ has quit IRC | 02:14 | |
| *** wanghao has joined #openstack-cinder | 02:15 | |
| tommylikehu | hey jgriffith ildikov , I post answer to your concerns here, I reckon the choice between | 02:17 |
| tommylikehu | 'volume:attachment_create' and 'attachment_create' is decided by where (which file) we place | 02:17 |
| tommylikehu | the policy enforcement, we now place that at the volume/api.py file. | 02:17 |
| tommylikehu | attachment_create/attachment:create | 02:17 |
| *** crushil has quit IRC | 02:20 | |
| *** lucasxu has quit IRC | 02:22 | |
| *** wanghao has quit IRC | 02:25 | |
| *** Julien-zte has joined #openstack-cinder | 02:26 | |
| *** mtanino has joined #openstack-cinder | 02:26 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Revert "Huawei: disable certificate verification" https://review.openstack.org/455878 | 02:26 |
| *** zhonghua has quit IRC | 02:29 | |
| *** crushil has joined #openstack-cinder | 02:29 | |
| *** zhonghua has joined #openstack-cinder | 02:30 | |
| *** mriedem1 has joined #openstack-cinder | 02:30 | |
| *** mriedem has quit IRC | 02:32 | |
| ildikov | tommylikehu: ah, so it's based on where the policy_enforcement calls are | 02:33 |
| ildikov | tommylikehu: I didn't know that | 02:33 |
| tommylikehu | yes, | 02:34 |
| tommylikehu | ildikov: and about the difference between create and update/delete I am not sure | 02:34 |
| *** rkmrHonjo has joined #openstack-cinder | 02:34 | |
| tommylikehu | we should not prevent user creating this resources? | 02:35 |
| *** mriedem1 is now known as mriedem | 02:36 | |
| *** kaisers has joined #openstack-cinder | 02:37 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: [BugFix] Add method policy in attachment APIs https://review.openstack.org/455101 | 02:40 |
| ildikov | tommylikehu: BTW what's the default policy? | 02:40 |
| tommylikehu | I am not sure, smcginnis leave that one empty? | 02:41 |
| *** cknight has quit IRC | 02:42 | |
| tommylikehu | ildikov: if we wanna let the administrator make the choice we should apply that enforcement and left the value in policy.json empty such as "resource:action":"" | 02:42 |
| *** Julien-zte has quit IRC | 02:43 | |
| ildikov | tommylikehu: so with the attachment_create whatever applies to the volume will be enforced anyhow within that call | 02:43 |
| ildikov | tommylikehu: so might not matter that much anyhow | 02:43 |
| *** Julien-zte has joined #openstack-cinder | 02:43 | |
| smcginnis | tommylikehu: Yeah, probably fine to leave that one empty. | 02:44 |
| *** liverpooler has joined #openstack-cinder | 02:44 | |
| *** gouthamr has quit IRC | 02:44 | |
| ildikov | smcginnis: cool, tnx for confirming | 02:45 |
| ildikov | tommylikehu: now I only miss the negative tests from that patch :) | 02:46 |
| *** Julien-zte has quit IRC | 02:47 | |
| *** Julien-zte has joined #openstack-cinder | 02:48 | |
| tommylikehu | ildikov: what do you mean by negative testcase? | 02:49 |
| tommylikehu | testcase for the missing of content in policy.json? | 02:49 |
| ildikov | tommylikehu: no, the attachment operations invoked by someone who does not fulfill the policy requirements | 02:50 |
| tommylikehu | the new added testcases would fail if you try to remove the corresponding line in test/policy.json | 02:51 |
| tommylikehu | ildikov: Is that what you want? | 02:52 |
| tommylikehu | they will fail too if you try to upgrade the required role from user to administrator, balabala | 02:53 |
| *** zhangguoqing has quit IRC | 02:54 | |
| ildikov | tommylikehu: so it's admin or owner right now, right? | 02:55 |
| *** Julien-zte has quit IRC | 02:56 | |
| ildikov | tommylikehu: so if someone tries to perform the actions without being either admin or owner that should fail too | 02:56 |
| *** Julien-zte has joined #openstack-cinder | 02:56 | |
| *** zhangguoqing has joined #openstack-cinder | 02:56 | |
| tommylikehu | ildikov: yes, that's why I add the use_admin_context parameter at the delete and update testcases | 02:57 |
| ildikov | tommylikehu: I just wanted to see the test that does not have it | 02:58 |
| *** zhangguoqing has quit IRC | 02:58 | |
| *** Apoorva has joined #openstack-cinder | 02:58 | |
| *** zhangguoqing has joined #openstack-cinder | 02:58 | |
| tommylikehu | ildikov: so add a testcase which tries to remove the line in the policy.json and then fire a update operation? | 02:59 |
| ildikov | tommylikehu: no, add a non-admin-context testcase | 03:00 |
| *** zhangguoqing has quit IRC | 03:00 | |
| tommylikehu | ildikov well, that make sense | 03:01 |
| smcginnis | So mock out check_policy to behave as if the user does not comply with the policy. | 03:01 |
| *** zhangguoqing has joined #openstack-cinder | 03:02 | |
| *** chlong has joined #openstack-cinder | 03:03 | |
| *** chlong_ has joined #openstack-cinder | 03:03 | |
| *** nicolasbock has quit IRC | 03:03 | |
| ildikov | tommylikehu: ok, cool | 03:03 |
| tommylikehu | smcginnis: maybe we can use a non-admin role to perform the admin rule required operation and then assert the raise of exception? | 03:04 |
| tommylikehu | mock means we are not sure that rule exists at the policy.json file | 03:04 |
| tommylikehu | really exists | 03:04 |
| tommylikehu | and that's why we could have that bug and while testcases run fatanstic | 03:05 |
| ildikov | tommylikehu: +1 | 03:07 |
| *** Julien-zte has quit IRC | 03:07 | |
| *** Julien-zte has joined #openstack-cinder | 03:08 | |
| *** lamt has joined #openstack-cinder | 03:14 | |
| *** dustins has quit IRC | 03:16 | |
| *** Julien-zte has quit IRC | 03:17 | |
| *** Julien-zte has joined #openstack-cinder | 03:18 | |
| *** markstur has joined #openstack-cinder | 03:19 | |
| *** jamielennox|away is now known as jamielennox | 03:23 | |
| *** Apoorva has quit IRC | 03:24 | |
| *** Julien-zte has quit IRC | 03:26 | |
| *** Julien-zte has joined #openstack-cinder | 03:27 | |
| *** kaisers has quit IRC | 03:27 | |
| *** Julien-zte has quit IRC | 03:31 | |
| *** Julien-zte has joined #openstack-cinder | 03:31 | |
| *** gkadam has joined #openstack-cinder | 03:34 | |
| *** lamt has quit IRC | 03:43 | |
| *** crushil has quit IRC | 03:44 | |
| openstackgerrit | Peter Wang proposed openstack/cinder master: VNX: Add QoS support https://review.openstack.org/441786 | 03:44 |
| *** Julien-zte has quit IRC | 03:44 | |
| *** Julien-zte has joined #openstack-cinder | 03:45 | |
| *** mriedem has quit IRC | 03:48 | |
| openstackgerrit | XieYingYun proposed openstack/cinder master: Add Apache License Content in index.rst https://review.openstack.org/455180 | 03:51 |
| *** Julien-zte has quit IRC | 03:51 | |
| *** Julien-zte has joined #openstack-cinder | 03:52 | |
| *** lamt has joined #openstack-cinder | 03:56 | |
| *** dave-mccowan has joined #openstack-cinder | 03:57 | |
| *** diablo_rojo has joined #openstack-cinder | 03:57 | |
| *** Julien-zte has quit IRC | 03:58 | |
| *** rcernin has joined #openstack-cinder | 03:59 | |
| *** rcernin is now known as rcernin|wfh|mtg | 03:59 | |
| *** Julien-zte has joined #openstack-cinder | 03:59 | |
| *** links has joined #openstack-cinder | 04:03 | |
| *** zhurong has quit IRC | 04:06 | |
| *** lamt has quit IRC | 04:08 | |
| *** markstur has quit IRC | 04:08 | |
| *** lamt has joined #openstack-cinder | 04:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/455905 | 04:10 |
| *** kaisers has joined #openstack-cinder | 04:12 | |
| *** Julien-zte has quit IRC | 04:13 | |
| *** Julien-zte has joined #openstack-cinder | 04:13 | |
| *** Julien-zte has quit IRC | 04:14 | |
| *** Julien-zte has joined #openstack-cinder | 04:14 | |
| *** kaisers has quit IRC | 04:15 | |
| *** dave-mccowan has quit IRC | 04:16 | |
| *** dave-mccowan has joined #openstack-cinder | 04:16 | |
| *** crushil has joined #openstack-cinder | 04:17 | |
| *** mtanino has quit IRC | 04:18 | |
| *** kaisers has joined #openstack-cinder | 04:18 | |
| *** lamt has quit IRC | 04:19 | |
| *** kaisers has quit IRC | 04:21 | |
| *** Poornima has joined #openstack-cinder | 04:26 | |
| *** rkmrHonjo_ has joined #openstack-cinder | 04:30 | |
| *** rkmrHonjo has quit IRC | 04:33 | |
| *** zhurong has joined #openstack-cinder | 04:34 | |
| *** psachin has joined #openstack-cinder | 04:35 | |
| *** Julien-zte has quit IRC | 04:36 | |
| *** Poornima has quit IRC | 04:36 | |
| *** lucasxu has joined #openstack-cinder | 04:36 | |
| *** Poornima has joined #openstack-cinder | 04:38 | |
| *** Julien-zte has joined #openstack-cinder | 04:40 | |
| *** lamt has joined #openstack-cinder | 04:44 | |
| *** kaisers has joined #openstack-cinder | 04:46 | |
| *** Julien-zte has quit IRC | 04:47 | |
| *** crushil has quit IRC | 04:48 | |
| *** Julien-zte has joined #openstack-cinder | 04:48 | |
| *** dave-mccowan has quit IRC | 04:50 | |
| *** kaisers has quit IRC | 04:51 | |
| *** lamt has quit IRC | 04:52 | |
| *** kaisers has joined #openstack-cinder | 04:52 | |
| *** Julien-zte has quit IRC | 04:53 | |
| *** lamt has joined #openstack-cinder | 04:54 | |
| *** Julien-zte has joined #openstack-cinder | 04:55 | |
| *** kaisers has quit IRC | 04:57 | |
| *** adisky_ has joined #openstack-cinder | 04:57 | |
| *** crushil has joined #openstack-cinder | 05:01 | |
| *** chlong_ has quit IRC | 05:03 | |
| *** markstur has joined #openstack-cinder | 05:03 | |
| *** chlong has quit IRC | 05:03 | |
| *** markstur has quit IRC | 05:07 | |
| *** lamt has quit IRC | 05:08 | |
| *** chhavi has joined #openstack-cinder | 05:10 | |
| *** Jack_Iv has joined #openstack-cinder | 05:11 | |
| *** rcernin|wfh|mtg has quit IRC | 05:11 | |
| *** Julien-zte has quit IRC | 05:12 | |
| *** Julien-zte has joined #openstack-cinder | 05:14 | |
| *** Jack_Iv has quit IRC | 05:15 | |
| *** yangyape_ has joined #openstack-cinder | 05:16 | |
| *** crushil has quit IRC | 05:16 | |
| *** yangyapeng has quit IRC | 05:17 | |
| *** zhonghua has quit IRC | 05:17 | |
| *** zhonghua has joined #openstack-cinder | 05:18 | |
| *** rcernin has joined #openstack-cinder | 05:20 | |
| dr_gogeta86 | xyang1 thank you i'll try it soon | 05:21 |
| *** Julien-zte has quit IRC | 05:30 | |
| *** Julien-zte has joined #openstack-cinder | 05:31 | |
| *** Jack_Iv has joined #openstack-cinder | 05:38 | |
| *** lucasxu has quit IRC | 05:40 | |
| *** diablo_rojo has quit IRC | 05:41 | |
| *** ChubYann has quit IRC | 05:44 | |
| *** Julien-zte has quit IRC | 05:46 | |
| *** Julien-zte has joined #openstack-cinder | 05:47 | |
| *** alonma has joined #openstack-cinder | 05:47 | |
| openstackgerrit | Merged openstack/cinder master: HPE Lefthand: add CG capability to generic groups https://review.openstack.org/451348 | 05:47 |
| *** lamt has joined #openstack-cinder | 05:49 | |
| *** kaisers has joined #openstack-cinder | 05:53 | |
| *** Julien-zte has quit IRC | 05:57 | |
| *** Julien-zte has joined #openstack-cinder | 05:57 | |
| *** lamt has quit IRC | 05:58 | |
| *** alonma has quit IRC | 05:59 | |
| *** chhavi has quit IRC | 06:02 | |
| *** Dinesh_Bhor has quit IRC | 06:04 | |
| *** Julien-zte has quit IRC | 06:06 | |
| *** Julien-zte has joined #openstack-cinder | 06:07 | |
| *** ociuhandu has quit IRC | 06:08 | |
| *** Julien-zte has quit IRC | 06:11 | |
| *** Julien-zte has joined #openstack-cinder | 06:12 | |
| *** Julien-zte has quit IRC | 06:15 | |
| *** Julien-zte has joined #openstack-cinder | 06:16 | |
| *** Elaine_wu has quit IRC | 06:18 | |
| *** Elaine_wu has joined #openstack-cinder | 06:19 | |
| *** chhavi has joined #openstack-cinder | 06:20 | |
| *** Julien-zte has quit IRC | 06:20 | |
| *** Julien-zte has joined #openstack-cinder | 06:21 | |
| *** Jack_Iv has quit IRC | 06:25 | |
| *** e0ne has joined #openstack-cinder | 06:30 | |
| *** pcaruana has joined #openstack-cinder | 06:30 | |
| openstackgerrit | zhangsong proposed openstack/cinder master: Support shared lock https://review.openstack.org/456072 | 06:31 |
| *** Julien-zte has quit IRC | 06:31 | |
| *** Julien-zte has joined #openstack-cinder | 06:32 | |
| *** Julien-zte has quit IRC | 06:34 | |
| *** alonma has joined #openstack-cinder | 06:34 | |
| *** Julien-zte has joined #openstack-cinder | 06:34 | |
| *** e0ne has quit IRC | 06:35 | |
| *** Jack_Iv has joined #openstack-cinder | 06:37 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 06:38 | |
| *** Jack_Iv has quit IRC | 06:42 | |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder master: Adds a bugfix release note regarding backing up with the Quobytedriver https://review.openstack.org/456075 | 06:47 |
| *** belmoreira has joined #openstack-cinder | 06:49 | |
| *** tesseract has joined #openstack-cinder | 06:49 | |
| *** Julien-zte has quit IRC | 06:54 | |
| *** Kimmo_ has quit IRC | 06:54 | |
| *** jistr has quit IRC | 06:54 | |
| *** ntt has quit IRC | 06:55 | |
| *** Lehtu_ has quit IRC | 06:55 | |
| *** thurloat has quit IRC | 06:55 | |
| *** sage_ has quit IRC | 06:55 | |
| *** MarkMielke has quit IRC | 06:55 | |
| *** sage_ has joined #openstack-cinder | 06:55 | |
| *** jistr has joined #openstack-cinder | 06:56 | |
| *** thurloat has joined #openstack-cinder | 06:56 | |
| *** Lehtu has joined #openstack-cinder | 06:56 | |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: WIP: Add open-iscsi manual scan support https://review.openstack.org/455394 | 06:56 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: WIP: Refactor iSCSI connect https://review.openstack.org/455393 | 06:56 |
| *** MarkMielke has joined #openstack-cinder | 06:57 | |
| *** Julien-zte has joined #openstack-cinder | 06:57 | |
| *** ntt has joined #openstack-cinder | 06:59 | |
| *** rcernin has quit IRC | 06:59 | |
| *** Jack_Iv has joined #openstack-cinder | 07:00 | |
| *** Jack_Iv has quit IRC | 07:01 | |
| openstackgerrit | Swapnil Nilangekar proposed openstack/cinder master: HPE 3PAR: CG capability in GVG for HPE-3PAR drivers. https://review.openstack.org/455141 | 07:02 |
| *** pgadiya has joined #openstack-cinder | 07:02 | |
| *** Jack_Iv has joined #openstack-cinder | 07:03 | |
| *** Jack_Iv has quit IRC | 07:03 | |
| *** Jack_Iv has joined #openstack-cinder | 07:03 | |
| *** Julien-zte has quit IRC | 07:06 | |
| *** Julien-zte has joined #openstack-cinder | 07:07 | |
| openstackgerrit | Swapnil Nilangekar proposed openstack/cinder master: HPE 3PAR: CG capability in GVG for HPE-3PAR drivers. https://review.openstack.org/455141 | 07:10 |
| *** chhavi has quit IRC | 07:12 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder-specs master: Support metadata for backup resource https://review.openstack.org/456088 | 07:13 |
| *** markstur has joined #openstack-cinder | 07:14 | |
| *** rcernin has joined #openstack-cinder | 07:16 | |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: WIP: Add open-iscsi manual scan support https://review.openstack.org/455394 | 07:16 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: WIP: Refactor iSCSI connect https://review.openstack.org/455393 | 07:16 |
| *** kaisers has quit IRC | 07:17 | |
| *** Julien-zte has quit IRC | 07:18 | |
| *** markstur has quit IRC | 07:18 | |
| *** Julien-zte has joined #openstack-cinder | 07:19 | |
| *** zhangguoqing has quit IRC | 07:22 | |
| *** chhavi has joined #openstack-cinder | 07:26 | |
| *** alonma has quit IRC | 07:27 | |
| *** alonma has joined #openstack-cinder | 07:27 | |
| *** alonma has quit IRC | 07:27 | |
| *** alonma has joined #openstack-cinder | 07:28 | |
| *** zhangguoqing has joined #openstack-cinder | 07:30 | |
| *** rcernin has quit IRC | 07:37 | |
| *** rcernin_ has joined #openstack-cinder | 07:37 | |
| *** rcernin_ has quit IRC | 07:38 | |
| *** Julien-zte has quit IRC | 07:38 | |
| *** rcernin_ has joined #openstack-cinder | 07:38 | |
| *** Julien-zte has joined #openstack-cinder | 07:38 | |
| *** Julien-zte has quit IRC | 07:45 | |
| *** chhavi has quit IRC | 07:45 | |
| *** Julien-zte has joined #openstack-cinder | 07:45 | |
| *** Julien-zte has quit IRC | 07:50 | |
| *** Julien-zte has joined #openstack-cinder | 07:50 | |
| *** e0ne has joined #openstack-cinder | 07:51 | |
| *** chhavi has joined #openstack-cinder | 07:52 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| *** e0ne has quit IRC | 08:00 | |
| *** e0ne has joined #openstack-cinder | 08:01 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: [BugFix] Add method policy in attachment APIs https://review.openstack.org/455101 | 08:01 |
| openstackgerrit | Satish Venkatasubramanian proposed openstack/cinder master: Add snapshots/summary API support https://review.openstack.org/407123 | 08:06 |
| *** Julien-zte has quit IRC | 08:07 | |
| *** Julien-zte has joined #openstack-cinder | 08:08 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: [api-ref] Update volume metadata key api https://review.openstack.org/456107 | 08:10 |
| *** Julien-zte has quit IRC | 08:12 | |
| *** Administrator_ has quit IRC | 08:12 | |
| *** Administrator_ has joined #openstack-cinder | 08:12 | |
| *** Julien-zte has joined #openstack-cinder | 08:12 | |
| *** kaisers has joined #openstack-cinder | 08:13 | |
| *** Julien-zte has quit IRC | 08:15 | |
| *** Julien-zte has joined #openstack-cinder | 08:15 | |
| *** yangyape_ has quit IRC | 08:16 | |
| *** yangyapeng has joined #openstack-cinder | 08:18 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: [BugFix] Add 'connection_info' to attachment object https://review.openstack.org/455131 | 08:19 |
| *** Julien-zte has quit IRC | 08:19 | |
| *** Julien-zte has joined #openstack-cinder | 08:19 | |
| *** dmellado has joined #openstack-cinder | 08:23 | |
| *** geguileo has joined #openstack-cinder | 08:23 | |
| *** aarefiev_afk is now known as aarefiev | 08:24 | |
| *** Jack_Iv has quit IRC | 08:26 | |
| *** Julien-zte has quit IRC | 08:26 | |
| *** Jack_Iv has joined #openstack-cinder | 08:26 | |
| *** Julien-zte has joined #openstack-cinder | 08:26 | |
| *** s-shiono_ has quit IRC | 08:28 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: [api-ref] Update volume metadata key api https://review.openstack.org/456107 | 08:28 |
| *** Julien-zte has quit IRC | 08:31 | |
| *** Julien-zte has joined #openstack-cinder | 08:31 | |
| *** links has quit IRC | 08:35 | |
| openstackgerrit | Chris MacNaughton proposed openstack/os-brick master: Include identity information in rbd commands https://review.openstack.org/453072 | 08:36 |
| *** Julien-zte has quit IRC | 08:44 | |
| *** Julien-zte has joined #openstack-cinder | 08:45 | |
| kaisers1 | jungleboy: lpetrut: I added an agenda point for today regarding the remotefs driver CIs issue, pls feel free to kick it if no longer applicable at that time. If still applicable i'd be happy to get things back rolling. I probably won't be able to join the meeting... | 08:47 |
| *** Julien-zte has quit IRC | 08:47 | |
| *** Julien-zte has joined #openstack-cinder | 08:48 | |
| *** Julien-zte has quit IRC | 08:51 | |
| *** Julien-zte has joined #openstack-cinder | 08:52 | |
| *** links has joined #openstack-cinder | 08:52 | |
| openstackgerrit | wangxiyuan proposed openstack/python-cinderclient master: Support list_volume for group show https://review.openstack.org/432179 | 08:53 |
| *** Kimmo_ has joined #openstack-cinder | 08:54 | |
| *** Poornima has quit IRC | 08:55 | |
| *** ociuhandu has joined #openstack-cinder | 08:55 | |
| openstackgerrit | wangxiyuan proposed openstack/python-cinderclient master: Support list_volume for group show https://review.openstack.org/432179 | 09:03 |
| *** Julien-zte has quit IRC | 09:05 | |
| *** Julien-zte has joined #openstack-cinder | 09:05 | |
| *** Julien-zte has quit IRC | 09:07 | |
| *** Julien-zte has joined #openstack-cinder | 09:08 | |
| *** wN_ has quit IRC | 09:11 | |
| *** lpetrut has joined #openstack-cinder | 09:12 | |
| *** wN_ has joined #openstack-cinder | 09:14 | |
| *** Julien-zte has quit IRC | 09:19 | |
| *** Julien-zte has joined #openstack-cinder | 09:20 | |
| *** links has quit IRC | 09:20 | |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder master: Disables xattrs on Quobyte driver mounts https://review.openstack.org/435328 | 09:21 |
| openstackgerrit | Balazs Gibizer proposed openstack/cinder master: Improve notification short-circuit https://review.openstack.org/442016 | 09:24 |
| *** Julien-zte has quit IRC | 09:24 | |
| *** Julien-zte has joined #openstack-cinder | 09:25 | |
| *** Julien-zte has quit IRC | 09:26 | |
| *** Julien-zte has joined #openstack-cinder | 09:27 | |
| *** Julien-zte has quit IRC | 09:28 | |
| openstackgerrit | zhangsong proposed openstack/cinder master: Support shared lock https://review.openstack.org/456072 | 09:28 |
| *** Julien-zte has joined #openstack-cinder | 09:28 | |
| *** gkadam is now known as gkadam-afk | 09:30 | |
| *** links has joined #openstack-cinder | 09:33 | |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder master: Reverts quobyte_volume_url option back to type StrOpt https://review.openstack.org/449553 | 09:33 |
| *** Julien-zte has quit IRC | 09:33 | |
| *** rkmrHonjo_ has quit IRC | 09:41 | |
| *** abhitechie has joined #openstack-cinder | 09:43 | |
| *** dalgaaf has quit IRC | 09:44 | |
| *** abhinavtechie has joined #openstack-cinder | 09:44 | |
| *** dalgaaf has joined #openstack-cinder | 09:46 | |
| *** abhitechie has quit IRC | 09:48 | |
| *** kaisers has quit IRC | 09:50 | |
| *** chhavi has quit IRC | 09:57 | |
| *** alonma has quit IRC | 10:12 | |
| *** alonma has joined #openstack-cinder | 10:13 | |
| *** alonma has quit IRC | 10:17 | |
| *** sdague has joined #openstack-cinder | 10:22 | |
| *** Poornima_K has joined #openstack-cinder | 10:24 | |
| *** alonma has joined #openstack-cinder | 10:25 | |
| *** abhinavtechie has quit IRC | 10:25 | |
| *** nicolasbock has joined #openstack-cinder | 10:33 | |
| *** abhitechie has joined #openstack-cinder | 10:37 | |
| *** kaisers has joined #openstack-cinder | 10:46 | |
| *** markstur has joined #openstack-cinder | 10:51 | |
| *** markstur has quit IRC | 10:55 | |
| *** abhitechie has quit IRC | 11:06 | |
| *** abhitechie has joined #openstack-cinder | 11:11 | |
| *** abhitechie has quit IRC | 11:13 | |
| *** abhitechie has joined #openstack-cinder | 11:14 | |
| *** abhinavtechie has joined #openstack-cinder | 11:16 | |
| *** abhitechie has quit IRC | 11:17 | |
| *** abhinavtechie has quit IRC | 11:17 | |
| *** abhinavtechie has joined #openstack-cinder | 11:17 | |
| *** abhinavtechie has quit IRC | 11:20 | |
| *** abhinavtechie has joined #openstack-cinder | 11:20 | |
| *** liverpooler has quit IRC | 11:20 | |
| openstackgerrit | Isaac Beckman proposed openstack/cinder master: XIV: switch to generic volume group https://review.openstack.org/448159 | 11:21 |
| *** zhurong has quit IRC | 11:26 | |
| *** abishop has joined #openstack-cinder | 11:31 | |
| *** mvk has quit IRC | 11:31 | |
| Dinesh_Bhor | smcginnis, jungleboyj: Hi, it will be great if you review these two fixes whenever you get time: https://review.openstack.org/#/c/452051/ https://review.openstack.org/#/c/454562/ | 11:33 |
| *** zhangguoqing has quit IRC | 11:41 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/455905 | 11:41 |
| *** gkadam-afk is now known as gkadam | 11:42 | |
| *** psachin has quit IRC | 11:51 | |
| *** chlong_ has joined #openstack-cinder | 11:53 | |
| *** chlong has joined #openstack-cinder | 11:53 | |
| *** chlong_ has quit IRC | 11:54 | |
| *** abhinavtechie has quit IRC | 11:56 | |
| *** huanxuan has quit IRC | 11:58 | |
| *** Zhongjun__ has joined #openstack-cinder | 12:02 | |
| *** mvk has joined #openstack-cinder | 12:03 | |
| *** zhonghua has quit IRC | 12:03 | |
| *** zhonghua has joined #openstack-cinder | 12:03 | |
| *** psachin has joined #openstack-cinder | 12:08 | |
| *** alonma has quit IRC | 12:08 | |
| *** alonma has joined #openstack-cinder | 12:09 | |
| *** edmondsw has joined #openstack-cinder | 12:13 | |
| *** alonma has quit IRC | 12:13 | |
| *** gouthamr has joined #openstack-cinder | 12:14 | |
| *** alonma has joined #openstack-cinder | 12:15 | |
| *** jonesn has joined #openstack-cinder | 12:15 | |
| *** Poornima_K has quit IRC | 12:16 | |
| *** alonma has quit IRC | 12:19 | |
| *** liverpooler has joined #openstack-cinder | 12:19 | |
| *** haplo37 has quit IRC | 12:25 | |
| *** alonma has joined #openstack-cinder | 12:26 | |
| *** zhonghua has quit IRC | 12:27 | |
| openstackgerrit | Isaac Beckman proposed openstack/cinder master: XIV: switch to generic volume group https://review.openstack.org/448159 | 12:28 |
| *** zhonghua has joined #openstack-cinder | 12:28 | |
| *** alonma has quit IRC | 12:30 | |
| *** mriedem has joined #openstack-cinder | 12:31 | |
| *** pgadiya has quit IRC | 12:31 | |
| *** alonma has joined #openstack-cinder | 12:31 | |
| *** winston-d_ has quit IRC | 12:33 | |
| *** kaisers has quit IRC | 12:33 | |
| *** kaisers has joined #openstack-cinder | 12:33 | |
| *** Jack_Iv has quit IRC | 12:35 | |
| *** Jack_Iv has joined #openstack-cinder | 12:36 | |
| *** alonma has quit IRC | 12:36 | |
| *** alonma has joined #openstack-cinder | 12:36 | |
| *** chhavi has joined #openstack-cinder | 12:37 | |
| *** markvoelker has joined #openstack-cinder | 12:37 | |
| *** haplo37_ has joined #openstack-cinder | 12:38 | |
| *** Jack_Iv has quit IRC | 12:40 | |
| *** alonma has quit IRC | 12:41 | |
| *** Jack_Iv has joined #openstack-cinder | 12:43 | |
| *** eharney has joined #openstack-cinder | 12:43 | |
| tommylikehu | jgriffith, ildikov, jungleboyj , xyang, please take a look at this patch again :) https://review.openstack.org/#/c/455101/ | 12:46 |
| *** lamt has joined #openstack-cinder | 12:47 | |
| *** lamt has quit IRC | 12:48 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Manage/Unmanage performance fix https://review.openstack.org/456209 | 12:52 |
| *** techstepper has joined #openstack-cinder | 12:54 | |
| *** techstepper has quit IRC | 12:56 | |
| *** lamt has joined #openstack-cinder | 12:57 | |
| *** crushil has joined #openstack-cinder | 13:03 | |
| *** gkadam has quit IRC | 13:05 | |
| *** psachin has quit IRC | 13:06 | |
| *** lamt has quit IRC | 13:07 | |
| *** porrua has joined #openstack-cinder | 13:10 | |
| *** Julien-zte has joined #openstack-cinder | 13:11 | |
| *** lamt has joined #openstack-cinder | 13:16 | |
| *** links has quit IRC | 13:17 | |
| *** xyang1 has joined #openstack-cinder | 13:18 | |
| *** Julien-zte has quit IRC | 13:21 | |
| *** erlon has joined #openstack-cinder | 13:22 | |
| *** Julien-zte has joined #openstack-cinder | 13:23 | |
| *** waj334 has quit IRC | 13:24 | |
| *** psachin has joined #openstack-cinder | 13:27 | |
| *** catintheroof has joined #openstack-cinder | 13:27 | |
| *** mtanino has joined #openstack-cinder | 13:32 | |
| *** markvoelker has quit IRC | 13:37 | |
| *** markvoelker has joined #openstack-cinder | 13:40 | |
| *** Julien-zte has quit IRC | 13:40 | |
| *** xyang has joined #openstack-cinder | 13:41 | |
| *** Julien-zte has joined #openstack-cinder | 13:42 | |
| *** mriedem has left #openstack-cinder | 13:44 | |
| *** mriedem has joined #openstack-cinder | 13:44 | |
| *** Julien-zte has quit IRC | 13:45 | |
| openstackgerrit | TommyLike proposed openstack/cinder-specs master: Use 'LIKE' operator to filter some resource https://review.openstack.org/442982 | 13:46 |
| *** zhurong has joined #openstack-cinder | 13:46 | |
| tommylikehu | ping xyang , xyang1 | 13:47 |
| xyang | tommylikehu: hi | 13:47 |
| *** Julien-zte has joined #openstack-cinder | 13:47 | |
| tommylikehu | I added some response to your comment on this patch: https://review.openstack.org/#/c/455101/ | 13:48 |
| *** Julien-zte has quit IRC | 13:48 | |
| tommylikehu | xyang: could you take a look? | 13:48 |
| xyang | tommylikehu: sure | 13:48 |
| tommylikehu | xyang: thanks | 13:48 |
| *** Julien-zte has joined #openstack-cinder | 13:48 | |
| *** lamt has quit IRC | 13:49 | |
| *** dustins has joined #openstack-cinder | 13:52 | |
| *** lamt has joined #openstack-cinder | 13:53 | |
| *** Julien-zte has quit IRC | 13:54 | |
| *** Julien-zte has joined #openstack-cinder | 13:55 | |
| openstackgerrit | Nicholas Jones proposed openstack/cinder master: Use cryptography instead of pycrypto https://review.openstack.org/452318 | 13:56 |
| *** tongli has joined #openstack-cinder | 13:57 | |
| *** ksumit has joined #openstack-cinder | 13:58 | |
| *** ksumit has quit IRC | 13:58 | |
| *** zhurong_ has joined #openstack-cinder | 13:59 | |
| *** zhurong has quit IRC | 14:01 | |
| *** ksumit has joined #openstack-cinder | 14:02 | |
| *** alonma has joined #openstack-cinder | 14:05 | |
| *** lucasxu has joined #openstack-cinder | 14:10 | |
| *** david-lyle has quit IRC | 14:15 | |
| *** kaisers has quit IRC | 14:20 | |
| *** lamt has quit IRC | 14:21 | |
| *** Julien-zte has quit IRC | 14:27 | |
| *** markstur has joined #openstack-cinder | 14:27 | |
| *** Julien-zte has joined #openstack-cinder | 14:30 | |
| *** alonma has quit IRC | 14:30 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Check "kernel_id", "ramdisk_id" for Glance v2 only https://review.openstack.org/439412 | 14:32 |
| *** markstur has quit IRC | 14:32 | |
| *** pck has quit IRC | 14:33 | |
| *** links has joined #openstack-cinder | 14:33 | |
| *** links has quit IRC | 14:36 | |
| *** eharney has quit IRC | 14:40 | |
| *** lamt has joined #openstack-cinder | 14:42 | |
| *** crose has joined #openstack-cinder | 14:43 | |
| *** eharney has joined #openstack-cinder | 14:45 | |
| *** Julien-zte has quit IRC | 14:46 | |
| *** wxy| has joined #openstack-cinder | 14:47 | |
| *** Julien-zte has joined #openstack-cinder | 14:48 | |
| *** pck has joined #openstack-cinder | 14:48 | |
| *** chhavi has quit IRC | 14:50 | |
| *** Julien-zte has quit IRC | 14:51 | |
| *** Julien-zte has joined #openstack-cinder | 14:52 | |
| *** Julien-zte has quit IRC | 14:55 | |
| *** ntpttr__ has joined #openstack-cinder | 14:55 | |
| *** zhurong_ has quit IRC | 14:57 | |
| *** lucasxu has quit IRC | 14:57 | |
| *** Julien-zte has joined #openstack-cinder | 14:57 | |
| *** mkoderer has joined #openstack-cinder | 14:59 | |
| *** chris_morrell has joined #openstack-cinder | 15:00 | |
| mkoderer | xyang: DuncanT: jgriffith: hemna: hi - can we get a review on: https://review.openstack.org/#/c/437313/ | 15:00 |
| *** Julien-zte has quit IRC | 15:00 | |
| xyang | mkoderer: sure, I'll take a look | 15:01 |
| *** ntpttr__ is now known as ntpttr_laptop | 15:01 | |
| *** Julien-zte has joined #openstack-cinder | 15:01 | |
| *** Jack_Iv has quit IRC | 15:02 | |
| *** rcernin_ has quit IRC | 15:04 | |
| *** diablo_rojo has joined #openstack-cinder | 15:05 | |
| *** alonma has joined #openstack-cinder | 15:05 | |
| *** Julien-zte has quit IRC | 15:06 | |
| *** Julien-zte has joined #openstack-cinder | 15:07 | |
| *** ntpttr__ has joined #openstack-cinder | 15:07 | |
| *** ntpttr_laptop has quit IRC | 15:08 | |
| *** Jack_Iv has joined #openstack-cinder | 15:09 | |
| *** Julien-zte has quit IRC | 15:10 | |
| *** Apoorva has joined #openstack-cinder | 15:10 | |
| *** Julien-zte has joined #openstack-cinder | 15:11 | |
| *** Apoorva has quit IRC | 15:13 | |
| *** Jack_Iv has quit IRC | 15:14 | |
| *** lucasxu has joined #openstack-cinder | 15:18 | |
| *** lamt has quit IRC | 15:20 | |
| *** chris_morrell has quit IRC | 15:21 | |
| *** chris_morrell has joined #openstack-cinder | 15:24 | |
| *** lamt has joined #openstack-cinder | 15:27 | |
| *** Julien-zte has quit IRC | 15:28 | |
| *** Julien-zte has joined #openstack-cinder | 15:28 | |
| *** belmoreira has quit IRC | 15:32 | |
| *** Julien-zte has quit IRC | 15:33 | |
| *** Julien-zte has joined #openstack-cinder | 15:34 | |
| *** Julien-zte has quit IRC | 15:37 | |
| *** Julien-zte has joined #openstack-cinder | 15:38 | |
| *** Jack_Iv has joined #openstack-cinder | 15:42 | |
| *** cknight has joined #openstack-cinder | 15:48 | |
| *** Julien-zte has quit IRC | 15:49 | |
| *** Julien-zte has joined #openstack-cinder | 15:50 | |
| *** lamt has quit IRC | 15:53 | |
| *** tesseract has quit IRC | 15:58 | |
| *** kfarr has joined #openstack-cinder | 15:59 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Glance: attach volume encryption key id to image https://review.openstack.org/453342 | 15:59 |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Create volumes from encrypted images https://review.openstack.org/453343 | 15:59 |
| *** Julien-zte has quit IRC | 16:01 | |
| *** Julien-zte has joined #openstack-cinder | 16:02 | |
| *** mlakat has quit IRC | 16:03 | |
| *** Julien-zte has quit IRC | 16:04 | |
| *** Julien-zte has joined #openstack-cinder | 16:05 | |
| *** aarefiev is now known as aarefiev_afk | 16:06 | |
| *** Julien-zte has quit IRC | 16:08 | |
| *** chris_morrell has quit IRC | 16:09 | |
| *** Julien-zte has joined #openstack-cinder | 16:09 | |
| *** KeithMnemonic has joined #openstack-cinder | 16:12 | |
| *** Julien-zte has quit IRC | 16:12 | |
| *** Julien-zte has joined #openstack-cinder | 16:14 | |
| *** alonma has quit IRC | 16:14 | |
| *** mvk has quit IRC | 16:18 | |
| *** Julien-zte has quit IRC | 16:18 | |
| *** adisky_ has quit IRC | 16:19 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Pre-zoned port group fix https://review.openstack.org/456285 | 16:23 |
| *** lamt has joined #openstack-cinder | 16:27 | |
| *** cknight has quit IRC | 16:32 | |
| *** lpetrut has quit IRC | 16:38 | |
| *** chris_morrell has joined #openstack-cinder | 16:39 | |
| *** david-lyle has joined #openstack-cinder | 16:45 | |
| *** dillaman has quit IRC | 16:47 | |
| *** ociuhandu has quit IRC | 16:47 | |
| *** wxy| has quit IRC | 16:48 | |
| *** ntpttr__ has quit IRC | 16:48 | |
| rajinir | swanson: smcginnis: Can you create volume in backend2 from a snapshot taken in backend2 and vice-versa? This is dellsc | 16:49 |
| *** e0ne has quit IRC | 16:49 | |
| *** psachin has quit IRC | 16:49 | |
| openstackgerrit | Nicholas Jones proposed openstack/cinder master: Use cryptography instead of pycrypto https://review.openstack.org/452318 | 16:51 |
| Swanson | rajinir, You can create a volume from a snapshot. Or are you referring to creating a volume in backend 2 from a snapshot taken in backend 1? The driver wouldn't handle that. Does cinder? | 16:51 |
| rajinir | swanson: yes | 16:51 |
| *** tongli has quit IRC | 16:52 | |
| rajinir | swanson: smcginnis: What's the best practice to migrate volumes? | 16:52 |
| rajinir | swanson: smcginnis: Can you create volume in backend2 from a snapshot taken in backend1 and vice-versa? This is dellss. Both backends 1 and 2 are of same type | 16:53 |
| smcginnis | rajinir: You can do a cinder retype with migration policy of on-demand. | 16:55 |
| Swanson | rajinir, smcginnis that | 16:55 |
| smcginnis | rajinir: It will create a new volume on the second backend, then copy all data over from the source to the new destination. | 16:55 |
| *** ntpttr has joined #openstack-cinder | 16:55 | |
| rajinir | smcginnis> ok, how about snapshots? | 16:56 |
| smcginnis | rajinir: If it's a large volume, then it may actually be more efficient to use the DSM interface to replicate over the volume and create a new volume from the snapshot, then use the manage existing feature to bring it into Cinder. | 16:56 |
| smcginnis | rajinir: But that depends on the situation. | 16:56 |
| rajinir | smcginnis: Can you migrate snapshots? | 16:56 |
| smcginnis | rajinir: Not via Cinder. | 16:56 |
| *** mtanino has quit IRC | 16:57 | |
| rajinir | smcginnis: Cinder doesn't support "Creating volume in backend2 from a snapshot taken in backend1 and vice-versa?" | 16:57 |
| smcginnis | rajinir: Well, you can create a volume from snapshot on backend1, then migrate that over to backend2. | 16:57 |
| smcginnis | rajinir: But you can't "migrate this volume and all of its snapshots" between backends. | 16:58 |
| rajinir | smcginnis> Thanks, its our top secret customer again who won't take not supported as an answer ;) | 16:59 |
| mriedem | jungleboyj: i just realized you replied to http://lists.openstack.org/pipermail/openstack-dev/2017-April/115292.html directly | 16:59 |
| mriedem | off the list | 16:59 |
| mriedem | was that intentional? | 16:59 |
| smcginnis | rajinir: Ug, good luck with that. :) | 17:00 |
| *** dillaman has joined #openstack-cinder | 17:00 | |
| smcginnis | mriedem: He misses you. | 17:00 |
| mriedem | i'm not sure that's true | 17:01 |
| *** ksumit has quit IRC | 17:02 | |
| *** psachin has joined #openstack-cinder | 17:06 | |
| *** ntpttr__ has joined #openstack-cinder | 17:07 | |
| *** ntpttr has quit IRC | 17:07 | |
| *** psachin_ has joined #openstack-cinder | 17:11 | |
| jungleboyj | mriedem: no. That was accidental. Sorry. | 17:13 |
| *** psachin_ has quit IRC | 17:14 | |
| *** lamt has quit IRC | 17:17 | |
| *** psachin_ has joined #openstack-cinder | 17:19 | |
| jonesn | smcginnis: I'm not confident that Synology CI is going to pass. I've been having a hard time distinguishing between 'normal' failure, and 'I caused this' failure. | 17:21 |
| *** psachin_ has quit IRC | 17:21 | |
| *** psachin_ has joined #openstack-cinder | 17:22 | |
| *** psachin_ has quit IRC | 17:23 | |
| *** chris_morrell has quit IRC | 17:33 | |
| *** psachin has quit IRC | 17:34 | |
| *** xyang has quit IRC | 17:35 | |
| *** jdillaman has quit IRC | 17:35 | |
| *** dillaman has quit IRC | 17:35 | |
| *** chris_morrell has joined #openstack-cinder | 17:38 | |
| *** chris_morrell has quit IRC | 17:42 | |
| *** jdillaman has joined #openstack-cinder | 17:47 | |
| *** dillaman has joined #openstack-cinder | 17:49 | |
| *** ociuhandu has joined #openstack-cinder | 17:50 | |
| Swanson | Daily review beg: https://review.openstack.org/#/c/443364/8 | 17:53 |
| *** crushil has quit IRC | 17:55 | |
| *** ntpttr__ has quit IRC | 17:56 | |
| *** ntpttr has joined #openstack-cinder | 17:59 | |
| *** Apoorva has joined #openstack-cinder | 18:02 | |
| *** crushil has joined #openstack-cinder | 18:02 | |
| *** Apoorva has quit IRC | 18:02 | |
| *** ChubYann has joined #openstack-cinder | 18:03 | |
| *** chhavi has joined #openstack-cinder | 18:03 | |
| *** Apoorva has joined #openstack-cinder | 18:03 | |
| *** ociuhandu has quit IRC | 18:05 | |
| *** ociuhandu has joined #openstack-cinder | 18:05 | |
| *** ociuhandu has joined #openstack-cinder | 18:05 | |
| *** lamt has joined #openstack-cinder | 18:13 | |
| *** ociuhandu has quit IRC | 18:17 | |
| *** ociuhandu has joined #openstack-cinder | 18:17 | |
| *** ociuhandu has joined #openstack-cinder | 18:18 | |
| *** Jack_Iv has quit IRC | 18:20 | |
| *** Jack_Iv has joined #openstack-cinder | 18:21 | |
| *** Swanson has quit IRC | 18:23 | |
| *** Swanson has joined #openstack-cinder | 18:25 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: qemu_img_info: Don't autodetect source format https://review.openstack.org/453341 | 18:25 |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Glance: attach volume encryption key id to image https://review.openstack.org/453342 | 18:25 |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Create volumes from encrypted images https://review.openstack.org/453343 | 18:25 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: WIP: Add open-iscsi manual scan support https://review.openstack.org/455394 | 18:26 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: WIP: Refactor iSCSI disconnect https://review.openstack.org/455392 | 18:26 |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick master: WIP: Refactor iSCSI connect https://review.openstack.org/455393 | 18:26 |
| *** markstur has joined #openstack-cinder | 18:28 | |
| smcginnis | jonesn: I'll keep my fingers crossed. | 18:30 |
| *** ntpttr has quit IRC | 18:30 | |
| smcginnis | jonesn: We can hopefully tell from the logs if any failures are related to your change. | 18:30 |
| *** xyang has joined #openstack-cinder | 18:32 | |
| *** markstur has quit IRC | 18:32 | |
| *** cknight has joined #openstack-cinder | 18:32 | |
| *** chhavi has quit IRC | 18:35 | |
| *** crose has quit IRC | 18:37 | |
| *** Zhongjun__ has quit IRC | 18:41 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder master: Dell EMC SC: Raise on _init_volume create_snapshot failure https://review.openstack.org/456353 | 18:44 |
| *** e0ne has joined #openstack-cinder | 18:45 | |
| openstackgerrit | Doug Hellmann proposed openstack/cinder master: move driver requirements to extras in setup.cfg https://review.openstack.org/456357 | 18:49 |
| *** xyang has quit IRC | 18:51 | |
| *** xyang has joined #openstack-cinder | 18:54 | |
| mriedem | is the "volume:extend" policy rule ever actually used anywhere? | 18:54 |
| mriedem | or is that a bug? | 18:54 |
| *** chris_morrell has joined #openstack-cinder | 18:54 | |
| mriedem | because i don't see it checked in the volume extend volume action api | 18:54 |
| mriedem | mgagne: ^ | 18:55 |
| mriedem | mgagne: i replied to the ML with some thoughts, but i think we're close to the same page | 18:55 |
| mriedem | the only volume action i see that checks policy is os-volume_upload_image | 18:56 |
| mriedem | https://bugs.launchpad.net/cinder/+bug/1682211 | 18:58 |
| openstack | Launchpad bug 1682211 in Cinder "The volume action "os-extend" API does not check policy" [Undecided,New] | 18:58 |
| eharney | it's supposed to be checked here: https://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/api.py?h=10.0.1#n1271 | 18:59 |
| mriedem | eharney: that seems like the wrong place, but i don't see it checked regardless | 19:00 |
| mriedem | looks like a lot of the policy auth checks happen in the actual wsgi controller code | 19:00 |
| mriedem | like for creating volume type encryptors | 19:00 |
| mriedem | https://git.openstack.org/cgit/openstack/cinder/tree/cinder/api/contrib/volume_type_encryption.py#n94 | 19:00 |
| eharney | @wrap_check_policy at least attempts to check it, just by reading it. I haven't tested to see if it works. | 19:00 |
| mgagne | mriedem: yes, we are on the same page | 19:01 |
| mriedem | eharney: oh maybe | 19:01 |
| *** xyang has quit IRC | 19:01 | |
| mriedem | eharney: ah yeah you're right | 19:01 |
| mriedem | it does the same weird thing where the name of the method is the policy | 19:01 |
| mriedem | whew | 19:02 |
| mriedem | ok, so for extending an attached volume we'd just need a specific policy auth check on a specific rule | 19:02 |
| *** mtanino has joined #openstack-cinder | 19:02 | |
| mriedem | unless you added a new method to volume api or something, e.g. def extend_attached | 19:02 |
| mriedem | anyway, details | 19:03 |
| *** xyang has joined #openstack-cinder | 19:03 | |
| mgagne | mriedem: could be implicitly done by the @wsgi.action decorator | 19:04 |
| mriedem | that's for something else | 19:04 |
| mgagne | mriedem: sure but couldn't it also do policy check? | 19:05 |
| mriedem | i'd just do policy checks like everything else does today | 19:05 |
| *** dustins has quit IRC | 19:05 | |
| mgagne | mriedem: oh sure, I don't disagree that everybody would benefit from consistency =) | 19:05 |
| mriedem | with @wrap_check_policy or whatever, or a specific check in cinder.volume.api.extend | 19:05 |
| mgagne | mriedem: https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1265 | 19:07 |
| mgagne | right, forgot to read above | 19:07 |
| mgagne | eharney: found it already | 19:07 |
| mgagne | at least it's not at the database api layer with implicit required admin context :D | 19:08 |
| *** e0ne has quit IRC | 19:08 | |
| *** mtanino has quit IRC | 19:09 | |
| *** xyang has quit IRC | 19:10 | |
| mriedem | heh yeah | 19:10 |
| mriedem | we removed that from nova v2.1 several releases ago | 19:10 |
| *** dustins has joined #openstack-cinder | 19:11 | |
| *** ntpttr has joined #openstack-cinder | 19:16 | |
| *** Jack_Iv has quit IRC | 19:16 | |
| *** lucasxu has quit IRC | 19:18 | |
| *** ntpttr has quit IRC | 19:18 | |
| *** ntpttr has joined #openstack-cinder | 19:18 | |
| *** lucasxu has joined #openstack-cinder | 19:20 | |
| *** lamt has quit IRC | 19:22 | |
| *** xyang has joined #openstack-cinder | 19:23 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder master: Dell EMC SC: Raise on _init_volume create_snapshot failure https://review.openstack.org/456353 | 19:24 |
| openstackgerrit | Tom Swanson proposed openstack/cinder master: Dell EMC SC: Raise on _init_volume create_snapshot failure https://review.openstack.org/456353 | 19:34 |
| *** edmondsw_ has joined #openstack-cinder | 19:34 | |
| *** xyang has quit IRC | 19:35 | |
| *** xyang has joined #openstack-cinder | 19:36 | |
| *** edmondsw_ has quit IRC | 19:40 | |
| *** chris_morrell has quit IRC | 19:43 | |
| *** cknight has quit IRC | 19:44 | |
| *** porrua has quit IRC | 20:01 | |
| openstackgerrit | Mathieu Gagné proposed openstack/cinder-specs master: Allow an attached volume to be extended https://review.openstack.org/453286 | 20:01 |
| *** lamt has joined #openstack-cinder | 20:02 | |
| openstackgerrit | Merged openstack/cinder master: Updated from global requirements https://review.openstack.org/455905 | 20:06 |
| openstackgerrit | Merged openstack/cinder master: Add unit test for brick/local_dev/lvm.py: create_volume https://review.openstack.org/450372 | 20:07 |
| *** alonma has joined #openstack-cinder | 20:19 | |
| *** jose-phillips has joined #openstack-cinder | 20:19 | |
| *** xyang has quit IRC | 20:20 | |
| *** e0ne has joined #openstack-cinder | 20:21 | |
| *** xyang has joined #openstack-cinder | 20:28 | |
| *** e0ne has quit IRC | 20:33 | |
| *** liverpooler has quit IRC | 20:53 | |
| *** dustins has quit IRC | 20:54 | |
| *** jonesn has quit IRC | 20:57 | |
| *** crushil has quit IRC | 21:00 | |
| *** ociuhandu has quit IRC | 21:01 | |
| *** alonma has quit IRC | 21:02 | |
| *** eharney has quit IRC | 21:05 | |
| *** sdague has quit IRC | 21:07 | |
| *** ntpttr has quit IRC | 21:08 | |
| *** ntpttr has joined #openstack-cinder | 21:08 | |
| *** lucasxu has quit IRC | 21:09 | |
| *** Apoorva_ has joined #openstack-cinder | 21:09 | |
| *** MarkMielke has quit IRC | 21:09 | |
| *** MarkMielke has joined #openstack-cinder | 21:09 | |
| *** Apoorva has quit IRC | 21:13 | |
| *** lucasxu has joined #openstack-cinder | 21:13 | |
| *** Jack_Iv has joined #openstack-cinder | 21:16 | |
| *** abishop has quit IRC | 21:17 | |
| *** eharney has joined #openstack-cinder | 21:18 | |
| *** gouthamr has quit IRC | 21:21 | |
| *** Jack_Iv has quit IRC | 21:21 | |
| Swanson | Easy patch... https://review.openstack.org/#/c/456353/ | 21:23 |
| *** crushil has joined #openstack-cinder | 21:24 | |
| -openstackstatus- NOTICE: Restarting Gerrit for our weekly memory leak cleanup. | 21:25 | |
| *** kfarr has quit IRC | 21:30 | |
| *** lucasxu has quit IRC | 21:31 | |
| *** edmondsw has quit IRC | 21:33 | |
| *** edmondsw has joined #openstack-cinder | 21:34 | |
| *** crushil has quit IRC | 21:37 | |
| *** edmondsw has quit IRC | 21:38 | |
| *** ociuhandu has joined #openstack-cinder | 21:41 | |
| *** pcaruana has quit IRC | 21:51 | |
| *** ntpttr has quit IRC | 21:55 | |
| *** xyang has quit IRC | 21:55 | |
| *** xyang has joined #openstack-cinder | 21:58 | |
| *** lamt has quit IRC | 21:59 | |
| *** ntpttr has joined #openstack-cinder | 22:02 | |
| *** lamt has joined #openstack-cinder | 22:04 | |
| *** markstur has joined #openstack-cinder | 22:04 | |
| *** Zhongjun__ has joined #openstack-cinder | 22:06 | |
| *** Apoorva_ has quit IRC | 22:07 | |
| *** Apoorva has joined #openstack-cinder | 22:07 | |
| *** markstur has quit IRC | 22:09 | |
| *** edmondsw has joined #openstack-cinder | 22:10 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Glance: attach volume encryption key id to image https://review.openstack.org/453342 | 22:13 |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Create volumes from encrypted images https://review.openstack.org/453343 | 22:13 |
| *** edmondsw has quit IRC | 22:14 | |
| *** xyang has quit IRC | 22:18 | |
| *** eharney has quit IRC | 22:18 | |
| openstackgerrit | Merged openstack/cinder master: Fix metadata api's raises TypeError https://review.openstack.org/452051 | 22:23 |
| *** lamt has quit IRC | 22:47 | |
| *** catintheroof has quit IRC | 22:48 | |
| *** abishop has joined #openstack-cinder | 22:50 | |
| *** lamt has joined #openstack-cinder | 22:59 | |
| *** xyang1 has quit IRC | 23:02 | |
| *** crushil has joined #openstack-cinder | 23:04 | |
| *** Apoorva_ has joined #openstack-cinder | 23:04 | |
| *** Apoorva has quit IRC | 23:07 | |
| *** lamt has quit IRC | 23:08 | |
| *** ntpttr has quit IRC | 23:08 | |
| *** ntpttr__ has joined #openstack-cinder | 23:08 | |
| *** lamt has joined #openstack-cinder | 23:12 | |
| *** lamt has quit IRC | 23:12 | |
| *** lamt has joined #openstack-cinder | 23:13 | |
| *** lamt has quit IRC | 23:13 | |
| *** lamt has joined #openstack-cinder | 23:16 | |
| *** lamt has quit IRC | 23:17 | |
| *** erlon has quit IRC | 23:35 | |
| *** eharney has joined #openstack-cinder | 23:35 | |
| *** Apoorva_ has quit IRC | 23:36 | |
| *** Apoorva has joined #openstack-cinder | 23:37 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Create volumes from encrypted images https://review.openstack.org/453343 | 23:46 |
| *** s-shiono has joined #openstack-cinder | 23:48 | |
| *** s-shiono_ has joined #openstack-cinder | 23:49 | |
| *** s-shiono has quit IRC | 23:52 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!