| *** chlong has joined #openstack-cinder | 00:04 | |
| *** harlowja_at_home has joined #openstack-cinder | 00:04 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Attach snapshot https://review.openstack.org/198556 | 00:07 |
|---|---|---|
| *** laughterwym has joined #openstack-cinder | 00:14 | |
| *** hflai has joined #openstack-cinder | 00:27 | |
| *** jasondotstar has quit IRC | 00:54 | |
| *** dims__ has joined #openstack-cinder | 00:55 | |
| *** dims_ has quit IRC | 00:56 | |
| *** heyun has joined #openstack-cinder | 00:58 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: Separate FlashSystem FC and iSCSI common code https://review.openstack.org/195382 | 01:05 |
| *** dims__ has quit IRC | 01:06 | |
| *** dims_ has joined #openstack-cinder | 01:07 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:10 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:11 | |
| *** lixiaoy1 has quit IRC | 01:11 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:12 | |
| *** dims_ has quit IRC | 01:15 | |
| *** blmartin has joined #openstack-cinder | 01:18 | |
| *** davechen has joined #openstack-cinder | 01:26 | |
| *** zhenguo has joined #openstack-cinder | 01:34 | |
| *** jasondotstar has joined #openstack-cinder | 01:38 | |
| *** Longgeek_ has joined #openstack-cinder | 01:46 | |
| *** gouthamr has quit IRC | 01:49 | |
| *** Longgeek has quit IRC | 01:49 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: Add white list support for target ports in VNX driver https://review.openstack.org/196979 | 01:50 |
| *** jamielennox is now known as jamielennox|away | 01:51 | |
| *** jasondotstar has quit IRC | 01:55 | |
| *** annashen has joined #openstack-cinder | 02:00 | |
| *** jamielennox|away is now known as jamielennox | 02:02 | |
| *** annashen has quit IRC | 02:05 | |
| *** annashen has joined #openstack-cinder | 02:07 | |
| *** annashen has quit IRC | 02:08 | |
| *** annashen has joined #openstack-cinder | 02:08 | |
| *** mutoulbj has joined #openstack-cinder | 02:09 | |
| *** annashen has quit IRC | 02:21 | |
| *** marzif has quit IRC | 02:22 | |
| *** annashen has joined #openstack-cinder | 02:26 | |
| *** blmartin has quit IRC | 02:30 | |
| *** coolsvap|away is now known as coolsvap | 02:36 | |
| *** bkopilov has quit IRC | 02:45 | |
| *** Longgeek_ has quit IRC | 02:47 | |
| *** Longgeek has joined #openstack-cinder | 02:47 | |
| *** annashen has quit IRC | 02:50 | |
| *** jungleboyj has quit IRC | 02:50 | |
| *** coolsvap is now known as coolsvap|away | 02:50 | |
| *** hakimo has joined #openstack-cinder | 02:51 | |
| *** hakimo_ has quit IRC | 02:55 | |
| *** BharatK has joined #openstack-cinder | 03:01 | |
| *** annashen has joined #openstack-cinder | 03:03 | |
| *** annashen has quit IRC | 03:05 | |
| *** avishay_ has quit IRC | 03:15 | |
| *** Longgeek_ has joined #openstack-cinder | 03:16 | |
| *** Longgeek has quit IRC | 03:18 | |
| *** annashen has joined #openstack-cinder | 03:23 | |
| *** annashen has quit IRC | 03:27 | |
| *** yamada-h has joined #openstack-cinder | 03:30 | |
| *** tobe has joined #openstack-cinder | 03:31 | |
| *** RuiChen has quit IRC | 03:33 | |
| *** yamada-h has quit IRC | 03:34 | |
| *** harlowja_at_home has quit IRC | 03:38 | |
| *** jamielennox is now known as jamielennox|away | 03:38 | |
| *** annashen has joined #openstack-cinder | 03:41 | |
| *** jungleboyj has joined #openstack-cinder | 03:47 | |
| *** jamielennox|away is now known as jamielennox | 03:49 | |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 03:49 |
| *** bkopilov has joined #openstack-cinder | 03:51 | |
| *** mutoulbj has quit IRC | 03:51 | |
| *** mutoulbj has joined #openstack-cinder | 03:52 | |
| *** coolsvap|away is now known as coolsvap | 03:53 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add multiple pools support for Huawei driver https://review.openstack.org/188251 | 03:55 |
| *** avishay_ has joined #openstack-cinder | 03:59 | |
| *** dims_ has joined #openstack-cinder | 04:12 | |
| *** annashen has quit IRC | 04:12 | |
| *** dims__ has joined #openstack-cinder | 04:16 | |
| *** dims_ has quit IRC | 04:18 | |
| *** dims__ has quit IRC | 04:20 | |
| *** pradipta has joined #openstack-cinder | 04:20 | |
| *** dims__ has joined #openstack-cinder | 04:21 | |
| *** dims___ has joined #openstack-cinder | 04:22 | |
| *** p_deore has joined #openstack-cinder | 04:23 | |
| *** kkeuning has joined #openstack-cinder | 04:24 | |
| *** dims__ has quit IRC | 04:26 | |
| *** dims___ has quit IRC | 04:27 | |
| *** daneyon has joined #openstack-cinder | 04:28 | |
| *** daneyon has quit IRC | 04:28 | |
| *** daneyon has joined #openstack-cinder | 04:28 | |
| *** avishay_ is now known as avishay | 04:28 | |
| *** p_deore has quit IRC | 04:33 | |
| *** coolsvap is now known as coolsvap|brb | 04:35 | |
| *** BharatK has quit IRC | 04:37 | |
| *** shyama has joined #openstack-cinder | 04:38 | |
| *** annashen has joined #openstack-cinder | 04:47 | |
| *** annashen has quit IRC | 04:49 | |
| *** tobe has quit IRC | 04:51 | |
| *** nihilifer has joined #openstack-cinder | 04:53 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Attach snapshot https://review.openstack.org/198556 | 04:53 |
| *** yamada-h has joined #openstack-cinder | 04:54 | |
| *** lpetrut has joined #openstack-cinder | 05:03 | |
| *** BharatK has joined #openstack-cinder | 05:03 | |
| *** deepakcs has joined #openstack-cinder | 05:05 | |
| *** coolsvap|brb is now known as coolsvap | 05:06 | |
| *** pradipta has quit IRC | 05:08 | |
| *** lpetrut has quit IRC | 05:17 | |
| *** nkrinner has joined #openstack-cinder | 05:23 | |
| *** ankit_ag has joined #openstack-cinder | 05:28 | |
| *** shausy has joined #openstack-cinder | 05:29 | |
| *** coolsvap is now known as coolsvap|away | 05:29 | |
| *** s0nea has joined #openstack-cinder | 05:36 | |
| *** lpetrut has joined #openstack-cinder | 05:39 | |
| *** ig0r__ has joined #openstack-cinder | 05:52 | |
| *** ig0r_ has quit IRC | 05:55 | |
| *** Nikolay_St has quit IRC | 06:03 | |
| *** pradipta has joined #openstack-cinder | 06:06 | |
| *** dims_ has joined #openstack-cinder | 06:10 | |
| *** nlevinki has joined #openstack-cinder | 06:12 | |
| *** zhenguo has quit IRC | 06:13 | |
| *** dims_ has quit IRC | 06:15 | |
| *** kkeuning has quit IRC | 06:19 | |
| *** aarefiev_home has joined #openstack-cinder | 06:20 | |
| *** aarefiev_home has quit IRC | 06:27 | |
| *** daneyon has quit IRC | 06:28 | |
| *** anshul has joined #openstack-cinder | 06:29 | |
| *** ianbrown has quit IRC | 06:35 | |
| *** tobe has joined #openstack-cinder | 06:38 | |
| *** zhenguo has joined #openstack-cinder | 06:41 | |
| openstackgerrit | Xi Yang proposed openstack/cinder: Fix getting out-of-date volume operation state issue for VNX https://review.openstack.org/197458 | 06:41 |
| *** ociuhandu has joined #openstack-cinder | 06:42 | |
| *** jwcroppe has joined #openstack-cinder | 06:43 | |
| openstackgerrit | Kuo-tung Kao proposed openstack/cinder: cinder list fails with 'name' sort key https://review.openstack.org/198614 | 06:48 |
| *** belmoreira has joined #openstack-cinder | 06:53 | |
| *** sgotliv__ has quit IRC | 06:56 | |
| *** bluex-pl has joined #openstack-cinder | 07:03 | |
| *** bluex-pl has quit IRC | 07:08 | |
| *** tobe has quit IRC | 07:08 | |
| *** dims_ has joined #openstack-cinder | 07:12 | |
| *** dims_ has quit IRC | 07:16 | |
| *** ociuhandu has quit IRC | 07:16 | |
| *** tobe has joined #openstack-cinder | 07:20 | |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Change the client according to volume migration improvement https://review.openstack.org/189547 | 07:24 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 07:24 |
| *** mikedillion has joined #openstack-cinder | 07:26 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 07:29 |
| *** mikedillion has left #openstack-cinder | 07:29 | |
| *** markus_z has joined #openstack-cinder | 07:29 | |
| *** ndipanov has joined #openstack-cinder | 07:30 | |
| *** ndipanov has quit IRC | 07:32 | |
| *** ndipanov has joined #openstack-cinder | 07:32 | |
| *** chlong has quit IRC | 07:35 | |
| *** kaisers has joined #openstack-cinder | 07:37 | |
| *** kaisers has quit IRC | 07:37 | |
| openstackgerrit | wanghao proposed openstack/cinder: Notify the transfer volume action in cinder https://review.openstack.org/185531 | 07:46 |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 07:48 |
| *** annashen has joined #openstack-cinder | 07:50 | |
| *** mutoulbj has quit IRC | 07:51 | |
| *** annashen has quit IRC | 07:55 | |
| *** mutoulbj has joined #openstack-cinder | 07:55 | |
| *** nlevinki has quit IRC | 07:59 | |
| *** lpetrut has quit IRC | 08:00 | |
| *** laughterwym_ has joined #openstack-cinder | 08:00 | |
| *** bluex-pl has joined #openstack-cinder | 08:02 | |
| *** bluex-pl has quit IRC | 08:02 | |
| *** bluex-pl has joined #openstack-cinder | 08:02 | |
| *** kaisers has joined #openstack-cinder | 08:03 | |
| *** laughterwym has quit IRC | 08:04 | |
| *** kaisers has quit IRC | 08:04 | |
| *** sgotliv has joined #openstack-cinder | 08:10 | |
| *** shausy has quit IRC | 08:11 | |
| *** shausy has joined #openstack-cinder | 08:11 | |
| *** nkrinner has quit IRC | 08:15 | |
| *** nlevinki has joined #openstack-cinder | 08:15 | |
| *** subscope has joined #openstack-cinder | 08:17 | |
| *** daneyon has joined #openstack-cinder | 08:17 | |
| *** nkrinner has joined #openstack-cinder | 08:18 | |
| *** daneyon has quit IRC | 08:22 | |
| wanghao | guys, I found this patch "Add support for force-delete backups"(https://review.openstack.org/#/c/166127/) has no more new reviews for a few days. Could we push this forward in L-2? Thanks. | 08:26 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: Move volume.api test to correct location https://review.openstack.org/198636 | 08:37 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: Add unit tests for cinder.api.v2.volumes https://review.openstack.org/198637 | 08:39 |
| *** laughterwym_ has quit IRC | 08:48 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: Raise BadRequest for invalid replication status https://review.openstack.org/198640 | 08:48 |
| *** jordanP has joined #openstack-cinder | 09:00 | |
| *** bluex-pl has quit IRC | 09:00 | |
| *** bluex-pl has joined #openstack-cinder | 09:01 | |
| *** kodoku has joined #openstack-cinder | 09:06 | |
| *** nlevinki has quit IRC | 09:07 | |
| BharatK | DuncanT, Hi, need your blessings for the patch: https://review.openstack.org/182286 | 09:08 |
| kodoku | Hi, is it possible to modify the public URL send by CINDER by API ? Because I use reverse proxy for my APIs and the url for Cinder is http://proxy/volume ==> http://controler:8776/v1 BUT Cinder return http://proxy/v1 ...... | 09:09 |
| kodoku | I now in keystone I can use public_endpoint=..... | 09:09 |
| *** yuriy_n17 has joined #openstack-cinder | 09:11 | |
| *** jamielennox is now known as jamielennox|away | 09:11 | |
| *** subscope has quit IRC | 09:14 | |
| *** subscope has joined #openstack-cinder | 09:15 | |
| *** jamielennox|away is now known as jamielennox | 09:18 | |
| *** nlevinki has joined #openstack-cinder | 09:20 | |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Change the client according to volume migration improvement https://review.openstack.org/189547 | 09:22 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 09:24 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 09:25 |
| openstackgerrit | Kuo-tung Kao proposed openstack/cinder: cinder list fails with 'name' sort key https://review.openstack.org/198614 | 09:30 |
| deepakcs | jordanP: ping | 09:34 |
| jordanP | deepakcs, ping | 09:34 |
| *** lpetrut has joined #openstack-cinder | 09:34 | |
| deepakcs | jordanP: reg.https://review.openstack.org/192736, you said u don't have a failing testcase to fix the blockCommit part, right ? | 09:35 |
| *** sb has quit IRC | 09:35 | |
| deepakcs | jordanP: so i managed to get a failing testcase, outside of tempest, i can open a LP bug, if that helps u fix the blockCommit too ? | 09:35 |
| *** strictlyb has joined #openstack-cinder | 09:36 | |
| jordanP | deepakcs, feel free to open a new bug in LP and submit a new patch. One bug in LP, one patch | 09:36 |
| deepakcs | jordanP: Hmm i have seen 1 patch fixing 2 bugs too, anyways, i will open a LP bug, will leave it to you on how u want to fix it. | 09:37 |
| kodoku | nobody can help me ? | 09:37 |
| jordanP | deepakcs, that so kind of you :) | 09:38 |
| *** ociuhandu has joined #openstack-cinder | 09:46 | |
| deepakcs | jordanP: lpetrut https://bugs.launchpad.net/nova/+bug/1471726 ( FYI ) | 09:49 |
| openstack | Launchpad bug 1471726 in OpenStack Compute (nova) "libvirt: blockCommit fails if domain is not running, for attached cinder volumes" [Undecided,New] | 09:49 |
| *** swamireddy_ has joined #openstack-cinder | 09:49 | |
| *** yamada-h has quit IRC | 09:52 | |
| *** davechen has left #openstack-cinder | 09:54 | |
| openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: Posix backup driver https://review.openstack.org/163647 | 09:54 |
| *** lixiaoy1 has quit IRC | 09:54 | |
| *** bluex-pl has quit IRC | 10:00 | |
| *** heyun has quit IRC | 10:02 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Filter cgsnapshots data on the DB side https://review.openstack.org/197163 | 10:02 |
| *** mutoulbj has quit IRC | 10:03 | |
| *** daneyon has joined #openstack-cinder | 10:06 | |
| *** laughterwym has joined #openstack-cinder | 10:06 | |
| *** dtynan has quit IRC | 10:07 | |
| *** lan_ has quit IRC | 10:07 | |
| *** dtynan has joined #openstack-cinder | 10:07 | |
| lpetrut | deepakcs: Thanks for the heads up | 10:09 |
| *** kaisers has joined #openstack-cinder | 10:11 | |
| *** daneyon has quit IRC | 10:11 | |
| *** kaisers has quit IRC | 10:11 | |
| *** marzif has joined #openstack-cinder | 10:12 | |
| *** kaisers has joined #openstack-cinder | 10:12 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Filter cgsnapshots data on the DB side https://review.openstack.org/197163 | 10:12 |
| *** lan_ has joined #openstack-cinder | 10:19 | |
| *** kodoku has quit IRC | 10:22 | |
| swamireddy_ | Hi All | 10:35 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient https://review.openstack.org/198663 | 10:36 |
| *** lan__ has joined #openstack-cinder | 10:37 | |
| swamireddy_ | Quick Q: Can we block (stop giving access to user) a specific APIs? | 10:38 |
| swamireddy_ | from cinder? | 10:38 |
| openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: GlusterFS backup driver https://review.openstack.org/182254 | 10:40 |
| *** lan_ has quit IRC | 10:40 | |
| *** bluex-pl has joined #openstack-cinder | 10:41 | |
| shyama | swamireddy_: that can be done through the policy.json file. It is more of keystone | 10:41 |
| *** Longgeek_ has quit IRC | 10:41 | |
| *** Longgeek has joined #openstack-cinder | 10:42 | |
| *** pradipta has quit IRC | 10:44 | |
| *** mutoulbj has joined #openstack-cinder | 10:46 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient https://review.openstack.org/197643 | 10:46 |
| *** dims_ has joined #openstack-cinder | 10:47 | |
| *** dims_ has quit IRC | 10:52 | |
| *** ociuhandu has quit IRC | 10:58 | |
| *** Yogi1 has joined #openstack-cinder | 10:59 | |
| *** tobe has quit IRC | 11:01 | |
| *** chlong has joined #openstack-cinder | 11:05 | |
| *** Yogi1 has quit IRC | 11:11 | |
| *** subscope has quit IRC | 11:12 | |
| swamireddy_ | shyama: OK...but with policy.json, we can set the role for the API, can not block the specific API?? | 11:17 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Pass volume size in GB to copy_volume function https://review.openstack.org/198006 | 11:17 |
| *** e0ne has joined #openstack-cinder | 11:17 | |
| geguileo | winston-d: ping - VolumeIsBusy stuff | 11:28 |
| *** IanGovett has joined #openstack-cinder | 11:33 | |
| *** subscope has joined #openstack-cinder | 11:36 | |
| *** ociuhandu has joined #openstack-cinder | 11:37 | |
| *** kodoku has joined #openstack-cinder | 11:39 | |
| *** Yogi1 has joined #openstack-cinder | 11:45 | |
| *** e0ne has quit IRC | 11:45 | |
| *** ganso_ has joined #openstack-cinder | 11:45 | |
| *** bluex-pl has quit IRC | 11:49 | |
| *** daneyon has joined #openstack-cinder | 11:55 | |
| *** e0ne has joined #openstack-cinder | 11:56 | |
| *** bill_az has joined #openstack-cinder | 11:57 | |
| *** kodoku has quit IRC | 11:58 | |
| *** daneyon has quit IRC | 12:00 | |
| *** nikeshm has joined #openstack-cinder | 12:00 | |
| *** bluex-pl has joined #openstack-cinder | 12:00 | |
| *** delatte has quit IRC | 12:01 | |
| *** delatte has joined #openstack-cinder | 12:02 | |
| *** dtynan has quit IRC | 12:02 | |
| *** dtynan has joined #openstack-cinder | 12:03 | |
| *** dims_ has joined #openstack-cinder | 12:03 | |
| *** Yogi1 has quit IRC | 12:05 | |
| *** tpsilva has joined #openstack-cinder | 12:05 | |
| *** dims_ has quit IRC | 12:08 | |
| *** bkopilov has quit IRC | 12:09 | |
| DuncanT | swamireddy_: Just put a role in policy that doesn't exist and the API is blocked | 12:11 |
| *** leakypipes is now known as jaypipes | 12:12 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: IBM storwize driver configuration item default valume change https://review.openstack.org/198692 | 12:13 |
| *** edmondsw has joined #openstack-cinder | 12:24 | |
| *** bswartz has quit IRC | 12:24 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Checking the existence of volume https://review.openstack.org/175809 | 12:27 |
| *** BharatK has quit IRC | 12:31 | |
| *** bluex-pl has quit IRC | 12:35 | |
| *** bluex-pl has joined #openstack-cinder | 12:35 | |
| *** bluex-pl has quit IRC | 12:36 | |
| *** andrey-mp has joined #openstack-cinder | 12:36 | |
| *** bluex-pl has joined #openstack-cinder | 12:36 | |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient https://review.openstack.org/197643 | 12:37 |
| *** kodoku has joined #openstack-cinder | 12:38 | |
| *** jasondotstar has joined #openstack-cinder | 12:38 | |
| kodoku | Hi, Python cinder client 1.2.2 no works with Cinder API V1 JUNO. Is it normal ? | 12:38 |
| openstackgerrit | Duncan Thomas proposed openstack/cinder: Add tenant validation to quota https://review.openstack.org/143645 | 12:41 |
| *** akerr has joined #openstack-cinder | 12:42 | |
| *** jdurgin has joined #openstack-cinder | 12:47 | |
| *** eduardo has joined #openstack-cinder | 12:48 | |
| *** ankit_ag has quit IRC | 12:51 | |
| *** mildan has joined #openstack-cinder | 12:51 | |
| *** gouthamr has joined #openstack-cinder | 12:53 | |
| *** dims_ has joined #openstack-cinder | 12:55 | |
| andrey-mp | DuncanT, if you can please review my last patchset - https://review.openstack.org/#/c/193596/ | 12:57 |
| *** dims__ has joined #openstack-cinder | 12:57 | |
| *** e0ne is now known as e0ne_ | 12:57 | |
| DuncanT | andrey-mp: Certainly | 12:58 |
| *** mildan has quit IRC | 12:58 | |
| *** avishay_ has joined #openstack-cinder | 12:59 | |
| *** avishay has quit IRC | 12:59 | |
| swamireddy_ | DuncanT: Cool.. All APIs, is the policy.json check added? Means is the all APIs function call chek the policy.jscon role checks?? | 13:00 |
| *** dims___ has joined #openstack-cinder | 13:00 | |
| *** dims_ has quit IRC | 13:00 | |
| *** dims_ has joined #openstack-cinder | 13:01 | |
| *** kodoku has quit IRC | 13:01 | |
| *** julim has joined #openstack-cinder | 13:03 | |
| *** dims__ has quit IRC | 13:03 | |
| *** dims___ has quit IRC | 13:05 | |
| *** nihilifer has quit IRC | 13:05 | |
| *** erlon has joined #openstack-cinder | 13:06 | |
| *** dims__ has joined #openstack-cinder | 13:06 | |
| *** timcl has joined #openstack-cinder | 13:07 | |
| *** bswartz has joined #openstack-cinder | 13:07 | |
| *** merooney has joined #openstack-cinder | 13:08 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 13:08 |
| *** ameade_vacation is now known as ameade | 13:08 | |
| *** xyang1 has joined #openstack-cinder | 13:09 | |
| *** dims_ has quit IRC | 13:09 | |
| *** jungleboyj has quit IRC | 13:10 | |
| *** dims__ has quit IRC | 13:11 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO support for iscsi discovery auth https://review.openstack.org/189358 | 13:20 |
| *** e0ne_ is now known as e0ne | 13:24 | |
| *** andrey-mp has quit IRC | 13:24 | |
| jgriffith | e0ne: think you missed something here: https://review.openstack.org/#/c/198006/3/cinder/image/image_utils.py | 13:25 |
| e0ne | jgriffith: looks you're right. we need unit test for this case | 13:26 |
| *** jskinner has joined #openstack-cinder | 13:27 | |
| *** rushil has joined #openstack-cinder | 13:29 | |
| *** rushil has quit IRC | 13:29 | |
| *** bkopilov has joined #openstack-cinder | 13:30 | |
| *** fabrice_ has joined #openstack-cinder | 13:31 | |
| fabrice_ | hi, can we ask question on cinderclient here ? | 13:32 |
| e0ne | fabrice_: hi. sure | 13:33 |
| *** dustins has joined #openstack-cinder | 13:34 | |
| openstackgerrit | TaoBai proposed openstack/cinder: Storwize Driver zone removing https://review.openstack.org/196966 | 13:35 |
| fabrice_ | how can we do a set-bootable in python with cinderclient ? | 13:37 |
| e0ne | jgriffith: oops. we've already have unit test and i broke it too :( | 13:37 |
| *** shausy has quit IRC | 13:38 | |
| *** chlong has quit IRC | 13:39 | |
| *** mriedem has joined #openstack-cinder | 13:39 | |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: test https://review.openstack.org/198716 | 13:39 |
| *** eharney has joined #openstack-cinder | 13:40 | |
| *** chlong has joined #openstack-cinder | 13:40 | |
| jgriffith | e0ne: :) | 13:41 |
| *** Yogi1 has joined #openstack-cinder | 13:41 | |
| jgriffith | e0ne: this is why I always cringe a little when people say "-1 need a unit test" :) | 13:42 |
| jgriffith | 98% of the time we all just change the unit test incorrectly with our patches anyway :) | 13:42 |
| jgriffith | or maybe that should be :( | 13:42 |
| *** Longgeek has quit IRC | 13:43 | |
| e0ne | jgriffith: my patch is great example how to DON'T write unit tests | 13:43 |
| jgriffith | e0ne: anyway, I didn't see anything else other than that first glance. I'll take a look again after you update | 13:43 |
| jgriffith | e0ne: LOL | 13:43 |
| jgriffith | e0ne: I have lots of those myself probably | 13:43 |
| jgriffith | e0ne: so I'm trying t a new strategy: | 13:44 |
| *** daneyon has joined #openstack-cinder | 13:44 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Remove unused config.generator module https://review.openstack.org/198721 | 13:44 |
| jgriffith | e0ne: instead of "write code... write code... fix unit tests" | 13:44 |
| jgriffith | e0ne: I'm trying "write unit tests... write code... write code..." | 13:44 |
| e0ne | jgriffith: :) | 13:44 |
| jgriffith | e0ne: it works really well but it's VERY difficult to get in the habbit | 13:45 |
| fabrice_ | myvol = cinder.volumes.create(name="test-vol", size=40) | 13:46 |
| fabrice_ | cinder.volumes.set_bootable(myvol,True) | 13:46 |
| e0ne | jgriffith: i'm agree with you | 13:47 |
| fabrice_ | Traceback (most recent call last): | 13:47 |
| fabrice_ | File "/home/fabrice/programmation/workspace/Openstack/src/cinder.py", line 18, in <module> | 13:47 |
| fabrice_ | cinder.volumes.set_bootable(myvol,True) | 13:47 |
| fabrice_ | AttributeError: 'VolumeManager' object has no attribute 'set_bootable' | 13:47 |
| e0ne | jgriffith: but for me it's really very hard do tests first if i've got more than 2-3 mocks:( | 13:47 |
| *** ctracey has quit IRC | 13:47 | |
| *** ctracey has joined #openstack-cinder | 13:48 | |
| *** merooney has quit IRC | 13:48 | |
| e0ne | fabrice_: please, use paste.openstack.org to share code and traceback | 13:48 |
| *** daneyon has quit IRC | 13:48 | |
| fabrice_ | ok | 13:49 |
| *** r-daneel has joined #openstack-cinder | 13:56 | |
| *** blmartin has joined #openstack-cinder | 13:58 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Pass volume size in GB to copy_volume function https://review.openstack.org/198006 | 13:59 |
| *** merooney has joined #openstack-cinder | 14:00 | |
| *** timcl has quit IRC | 14:00 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix update_migrate_volume() exception handling https://review.openstack.org/198731 | 14:01 |
| fabrice_ | e0ne , thanks for you answers. my prob was bad lib. next time i will use paste.openstack.org | 14:01 |
| *** timcl has joined #openstack-cinder | 14:01 | |
| *** nlevinki has quit IRC | 14:01 | |
| thingee | jgriffith: thanks for writing the example | 14:04 |
| thingee | jgriffith: for capabilities | 14:04 |
| thingee | jgriffith: I think we're aligned here. I was hoping this would include the scheduler bits since that's where I'm unfamiliar with things. | 14:05 |
| openstackgerrit | Sergey Vilgelm proposed openstack/os-brick: Remove unused oslo incubator files https://review.openstack.org/198734 | 14:05 |
| *** r-daneel has quit IRC | 14:07 | |
| *** dtynan has quit IRC | 14:07 | |
| *** deepakcs has quit IRC | 14:08 | |
| *** dtynan has joined #openstack-cinder | 14:09 | |
| *** anshul has quit IRC | 14:12 | |
| DuncanT | swamireddy_: I beleive so - if you find any missing, please file a bug. | 14:13 |
| e0ne | DuncanT: could you please take one more look on https://review.openstack.org/192683? | 14:14 |
| DuncanT | e0ne: Sure, two minutes | 14:14 |
| e0ne | DuncanT: thanks! | 14:14 |
| jgriffith | thingee: the scheduler bits are "free" | 14:15 |
| jgriffith | thingee: that's what I keep trying to say :) | 14:15 |
| jgriffith | thingee: if you put any of those things in the extra-specs without a scoping on them the scheduler is going to look at capabilities to make sure they're there | 14:16 |
| jgriffith | thingee: but it MUST be in the stats piece then | 14:16 |
| jgriffith | thingee: so I could even put "set key foo=biz" in the extra spec | 14:16 |
| jgriffith | thingee: and as long as the stats report has "foo" in it it filters | 14:17 |
| *** logan2 has quit IRC | 14:17 | |
| *** nlevinki has joined #openstack-cinder | 14:17 | |
| DuncanT | e0ne: +2 | 14:17 |
| e0ne | DuncanT: thank you | 14:18 |
| thingee | jgriffith: ok | 14:18 |
| thingee | jgriffith: I think I understand. Your example is just how the driver would go about doing it, but we'd still need the manager logic to know when to report the capabilities | 14:19 |
| thingee | like we talked about last week | 14:19 |
| jgriffith | DuncanT: e0ne winston-d re https://review.openstack.org/#/c/192683/ | 14:20 |
| jgriffith | DuncanT: e0ne winston-d ^^ Is that really beneficial? | 14:20 |
| *** r-daneel has joined #openstack-cinder | 14:20 | |
| jgriffith | thingee: I'll finish up the POC later today with everything | 14:21 |
| jgriffith | thingee: in reality you don't even need to add anything to the scheduler or manager | 14:21 |
| DuncanT | jgriffith: Keystone performs quite a bit better under it | 14:21 |
| jgriffith | thingee: if the structure is in the stats update it "just works" | 14:21 |
| e0ne | jgriffith: it's abput 20% of performance increase for keysrone. | 14:21 |
| DuncanT | jgriffith: If the code isn't horrible and somebody does a CI for it, I don't think it's a bad thing | 14:21 |
| jgriffith | DuncanT: yeah... just asking | 14:21 |
| jgriffith | DuncanT: so IMHO sometimes you can have "too many options" | 14:22 |
| thingee | jgriffith: well unless you want the manager to be enforcing. For example we said somethings like standard capabilities should default to some boolean | 14:22 |
| DuncanT | jgriffith: Though most of the performance issues I usually see are DB related, so not so sure it will help *much* - enough to see the code though, I think | 14:22 |
| jgriffith | and you end up with none of them being well-supported | 14:22 |
| e0ne | jgriffith: i believe, we'll deprecate eventelt-wsgi in a future | 14:22 |
| jgriffith | DuncanT: e0ne the statement in the write up is kinda contradictory IMO | 14:22 |
| jgriffith | Enteprises exposed via client access etc | 14:23 |
| jgriffith | That sounds like a public cloud issue, not an enterprise issue behind a firewall | 14:23 |
| DuncanT | jgriffith: Honestly, I don't buy the security line | 14:23 |
| jgriffith | DuncanT: so FTR that's what caught my attention | 14:23 |
| jgriffith | and the main reason I asked | 14:23 |
| jgriffith | DuncanT: e0ne if it's just about giving a non-eventlet and a nginx option let's just be honest about it | 14:24 |
| jgriffith | not try and justify it or boost it's importance in the name of security | 14:24 |
| DuncanT | jgriffith: That's probably a fair comment - I just want to see it exist at this point, to make a more informed decission | 14:24 |
| jgriffith | DuncanT: yeah, that's fine by me | 14:25 |
| e0ne | jgriffith: some enterprises want to use single endpoint like https://internalclour/{cinder, nova} | 14:25 |
| DuncanT | jgriffith: Security arguments are not something that I generally find all that compelling - the fact we took passwords out of debug output still annoys the hell out of me | 14:25 |
| jgriffith | e0ne: DuncanT I just don't want someobdy seeing this on the web and misinterpretting this as "a or more secure than b" | 14:25 |
| DuncanT | e0ne: Some enterprises want the moon on a stick.... | 14:25 |
| e0ne | DuncanT: :) | 14:26 |
| DuncanT | jgriffith: Hmmmm, probably worth a comment to that effect on the review at least | 14:26 |
| jgriffith | e0ne: so won't openstack-client help with that? | 14:26 |
| e0ne | jgriffith: partially, yes. it won't help with more easy https deployment | 14:28 |
| *** Adriano__ has joined #openstack-cinder | 14:28 | |
| openstackgerrit | Mike Perez proposed openstack/cinder: Remove X-IO volume driver https://review.openstack.org/198741 | 14:28 |
| thingee | jgriffith: DuncanT ^ | 14:28 |
| lpetrut | thingee: Hi. We'd like to add backup support for the Windows iSCSI and SMB drivers. Could you please take a look and maybe aprove the following two BPs? https://blueprints.launchpad.net/cinder/+spec/windows-smb-backup https://blueprints.launchpad.net/cinder/+spec/windows-iscsi-backup | 14:28 |
| jgriffith | e0ne: so I gave it a -1... but only because I'd prefer we don't pretend it's more secure or anythign like that | 14:28 |
| e0ne | jgriffith: i mean, redirect between nginx+https and eventlet+http | 14:29 |
| jgriffith | e0ne: I think it's fine to introduce another option | 14:29 |
| jgriffith | e0ne: and certainly I myself am a fan of nginx | 14:29 |
| jgriffith | e0ne: would you mind terribly just rewording it a bit for me? | 14:29 |
| jgriffith | e0ne: if not I understand, and you'll get enough votes to merge it anyway :) | 14:30 |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Change the client according to volume migration improvement https://review.openstack.org/189547 | 14:30 |
| *** lpabon has joined #openstack-cinder | 14:30 | |
| *** rushil has joined #openstack-cinder | 14:31 | |
| e0ne | jgriffith: fair enough. i'll re-write security and deployers impact sections | 14:31 |
| *** rushil has quit IRC | 14:31 | |
| jgriffith | e0ne: thanks!! | 14:31 |
| e0ne | jgriffith: thanks for review! | 14:32 |
| *** nkrinner has quit IRC | 14:33 | |
| *** timcl has quit IRC | 14:33 | |
| *** timcl has joined #openstack-cinder | 14:33 | |
| jgriffith | e0ne: you're welcome, sorry if I seem pedantic about that | 14:34 |
| e0ne | jgriffith: np. i like your comment "they saw some reference to a statement in the spec on the internet" :) | 14:34 |
| *** dustins_ has joined #openstack-cinder | 14:35 | |
| jgriffith | e0ne: :) Only because it happens. | 14:35 |
| *** stefan_amann has joined #openstack-cinder | 14:35 | |
| e0ne | :) | 14:35 |
| e0ne | it does | 14:35 |
| *** stefan_amann has quit IRC | 14:35 | |
| DuncanT | jgriffith: People who believe what they read on the internet deserve exactly what they get | 14:36 |
| *** stefan_amann has joined #openstack-cinder | 14:36 | |
| *** dustins has quit IRC | 14:38 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Attach snapshot https://review.openstack.org/198556 | 14:39 |
| *** logan2 has joined #openstack-cinder | 14:41 | |
| *** nlevinki has quit IRC | 14:42 | |
| aarefiev | DuncanT: thanks for rebase patch with tenant validation in quota, I'll fix all issue and update patch | 14:43 |
| *** Longgeek has joined #openstack-cinder | 14:43 | |
| *** rushil has joined #openstack-cinder | 14:48 | |
| *** Longgeek has quit IRC | 14:49 | |
| *** tbarron_afk is now known as tbarron | 14:49 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix properties extracting from image with glance api v2 https://review.openstack.org/169430 | 14:50 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Cinder API WSGI application under Apache/Nginx https://review.openstack.org/192683 | 14:51 |
| *** thangp has joined #openstack-cinder | 14:52 | |
| openstackgerrit | TaoBai proposed openstack/cinder: Storwize Driver zone removing Storwize driver may remove zone even when the zone is still in use by an attached volume.To fix this issue, we need only report initiator and target map when there are no volumes attached. https://review.openstack.org/196966 | 14:54 |
| *** timcl has quit IRC | 14:58 | |
| *** cbader has joined #openstack-cinder | 14:58 | |
| *** timcl has joined #openstack-cinder | 14:59 | |
| *** mtanino has joined #openstack-cinder | 15:02 | |
| *** serverascode has quit IRC | 15:05 | |
| *** serverascode has joined #openstack-cinder | 15:05 | |
| *** dims_ has joined #openstack-cinder | 15:09 | |
| *** logan2 has quit IRC | 15:11 | |
| *** dannywilson has joined #openstack-cinder | 15:12 | |
| *** dtynan has quit IRC | 15:13 | |
| *** markvoelker has quit IRC | 15:14 | |
| *** haomaiwang has quit IRC | 15:14 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 15:14 |
| *** IlyaG has joined #openstack-cinder | 15:14 | |
| *** dtynan has joined #openstack-cinder | 15:14 | |
| *** haomaiw__ has joined #openstack-cinder | 15:15 | |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Change cinderclient according to volume migration improvement https://review.openstack.org/189547 | 15:16 |
| *** logan2 has joined #openstack-cinder | 15:19 | |
| *** kashyap has joined #openstack-cinder | 15:20 | |
| swamireddy_ | DuncanT: Sure, will check and file a bug, if some thing missing... Thx | 15:21 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Port NetApp E-Series iSCSI driver to ABC model https://review.openstack.org/196225 | 15:21 |
| *** timcl has left #openstack-cinder | 15:22 | |
| *** ameade has quit IRC | 15:22 | |
| *** ameade_vacation has joined #openstack-cinder | 15:23 | |
| *** ameade_vacation is now known as ameade | 15:23 | |
| aarefiev | thingee: hi, have quick question, do we accept fixes to client v1 https://review.openstack.org/#/c/178588/ ? | 15:23 |
| thingee | aarefiev: yeah fixes are fine. just no new functionality | 15:23 |
| e0ne | thingee, aarefiev: yes for both options for cinderclient | 15:24 |
| e0ne | thingee: am i right? | 15:24 |
| aarefiev | thingee: could you look at https://review.openstack.org/#/c/178588/ , please, very small fix? | 15:25 |
| swamireddy_ | All - I have created a volume with size of 998GB (~1TB) and deleted the same. But the delete took time around 9 min. (cinder with ceph backend) | 15:25 |
| swamireddy_ | Is the volume of ~ 1 TB delete takes 9 min or so?? Is there justtification for this? | 15:26 |
| *** belmoreira has quit IRC | 15:27 | |
| *** lpetrut has quit IRC | 15:27 | |
| *** tsekiyama has joined #openstack-cinder | 15:27 | |
| *** hodos has joined #openstack-cinder | 15:27 | |
| *** jungleboyj has joined #openstack-cinder | 15:28 | |
| *** crose has joined #openstack-cinder | 15:28 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add tenant validation to quota https://review.openstack.org/143645 | 15:30 |
| smcginnis | swamireddy_: Could be that ceph takes that long to complete the operation. I would think it's backend dependent. | 15:31 |
| swamireddy_ | smcginnis: I have tried with devstack + ceph, its taking 2 or 3 min..but with icehouse cinder+ceph, its talking 9 min or so.. | 15:32 |
| e0ne | swamireddy_: ceph deletion is a slow operation | 15:32 |
| *** daneyon has joined #openstack-cinder | 15:33 | |
| *** hemnafk is now known as hemna | 15:33 | |
| *** daneyon has quit IRC | 15:33 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Service object https://review.openstack.org/160417 | 15:34 |
| e0ne | swamireddy_: http://www.sebastien-han.fr/blog/2013/12/12/rbd-image-bigger-than-your-ceph-cluster/ | 15:34 |
| e0ne | swamireddy_: it shrinks all vokume(image) chumks | 15:35 |
| e0ne | s/chumks/chunks | 15:35 |
| *** IlyaG_ has joined #openstack-cinder | 15:36 | |
| *** annegentle has joined #openstack-cinder | 15:37 | |
| *** IlyaG has quit IRC | 15:39 | |
| openstackgerrit | Kuo-tung Kao proposed openstack/cinder: cinder list fails with 'name' sort key https://review.openstack.org/198614 | 15:40 |
| *** dustins_ has quit IRC | 15:41 | |
| *** harlowja_at_home has joined #openstack-cinder | 15:44 | |
| openstackgerrit | Diem Tran proposed openstack/cinder: Fix PEP476 & format message of Oracle ZFSSA drivers https://review.openstack.org/197736 | 15:44 |
| *** merooney has quit IRC | 15:44 | |
| *** lcurtis has joined #openstack-cinder | 15:44 | |
| *** zhiyan has quit IRC | 15:47 | |
| *** zhiyan has joined #openstack-cinder | 15:47 | |
| *** rushil has quit IRC | 15:48 | |
| *** patrickeast has joined #openstack-cinder | 15:53 | |
| *** eduardo has quit IRC | 15:59 | |
| *** Longgeek has joined #openstack-cinder | 16:00 | |
| *** jskinner has quit IRC | 16:03 | |
| *** jskinner has joined #openstack-cinder | 16:04 | |
| *** rwsu has joined #openstack-cinder | 16:04 | |
| *** Longgeek has quit IRC | 16:05 | |
| *** leeantho has joined #openstack-cinder | 16:08 | |
| *** avishay_ has quit IRC | 16:10 | |
| *** markvoelker has joined #openstack-cinder | 16:10 | |
| *** stefan_amann has quit IRC | 16:11 | |
| *** laughterwym has quit IRC | 16:13 | |
| DuncanT | swamireddy_: Try creating and deleting a large block device on your ceph cluster without using cinder - cinder can never go faster than that | 16:13 |
| swamireddy_ | DuncanT: sure, will check the delete time without cinder and confirm the status. | 16:15 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix properties extracting from image with glance api v2 https://review.openstack.org/169430 | 16:16 |
| *** haypo has joined #openstack-cinder | 16:20 | |
| *** jaypipes is now known as jay-deepthought | 16:21 | |
| *** lpetrut has joined #openstack-cinder | 16:24 | |
| *** rushil has joined #openstack-cinder | 16:27 | |
| *** jwcroppe_ has joined #openstack-cinder | 16:27 | |
| *** harlowja_at_home has quit IRC | 16:28 | |
| *** merooney has joined #openstack-cinder | 16:28 | |
| *** dustins has joined #openstack-cinder | 16:28 | |
| *** yuriy_n17 has quit IRC | 16:28 | |
| *** jwcroppe has quit IRC | 16:30 | |
| *** haypo has quit IRC | 16:31 | |
| *** ociuhandu has quit IRC | 16:31 | |
| *** jwcroppe_ has quit IRC | 16:32 | |
| *** timcl has joined #openstack-cinder | 16:32 | |
| e0ne | xyang1: hi. one question inline for https://review.openstack.org/#/c/193937/16/cinder/volume/driver.py | 16:34 |
| xyang1 | e0ne: Hi | 16:36 |
| *** Yogi1 has quit IRC | 16:36 | |
| xyang1 | e0ne: Backup is running in the same process as the volume driver now | 16:38 |
| e0ne | xyang1: i mean different process: cinder-volume and cinder-backup | 16:40 |
| xyang1 | e0ne: That is not happening now without code change | 16:40 |
| *** jordanP has quit IRC | 16:41 | |
| e0ne | xyang1: i'm confused | 16:41 |
| *** eharney has quit IRC | 16:42 | |
| xyang1 | e0ne: When you start backup service, it will load the driver in its process | 16:42 |
| e0ne | xyang1: yes | 16:42 |
| e0ne | xyang1: but the same for cinder-volume service | 16:42 |
| xyang1 | e0ne: I see what you are saying | 16:43 |
| e0ne | xyang1: https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L307 and https://github.com/openstack/cinder/blob/master/cinder/backup/manager.py#L176 | 16:43 |
| xyang1 | e0ne: good point | 16:44 |
| e0ne | xyang1: thanks | 16:46 |
| e0ne | xyang1: need to go offline. bb in ~2 hours | 16:46 |
| xyang1 | e0ne: Thank you! | 16:46 |
| e0ne | xyang1: you're welcome | 16:46 |
| *** e0ne has quit IRC | 16:47 | |
| *** garthb has joined #openstack-cinder | 16:49 | |
| *** garthb_ has joined #openstack-cinder | 16:49 | |
| *** garthb_ has quit IRC | 16:50 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add deactivate step to extend_lv https://review.openstack.org/198813 | 16:50 |
| *** annegentle has quit IRC | 16:50 | |
| *** dtynan has quit IRC | 16:50 | |
| *** git-harry has quit IRC | 16:50 | |
| *** jbernard has quit IRC | 16:50 | |
| *** anshprat has quit IRC | 16:50 | |
| *** ondergetekende_ has quit IRC | 16:50 | |
| *** anshprat has joined #openstack-cinder | 16:50 | |
| *** ondergetekende has joined #openstack-cinder | 16:50 | |
| *** jbernard has joined #openstack-cinder | 16:51 | |
| *** git-harry has joined #openstack-cinder | 16:54 | |
| *** annegentle has joined #openstack-cinder | 16:54 | |
| *** dtynan has joined #openstack-cinder | 16:55 | |
| *** rushil has quit IRC | 16:55 | |
| *** eharney has joined #openstack-cinder | 16:57 | |
| *** avishay_ has joined #openstack-cinder | 16:58 | |
| *** rushil has joined #openstack-cinder | 16:59 | |
| *** daneyon has joined #openstack-cinder | 17:00 | |
| *** _cjones_ has joined #openstack-cinder | 17:00 | |
| *** Longgeek has joined #openstack-cinder | 17:01 | |
| *** avishay_ has quit IRC | 17:03 | |
| *** jordanP has joined #openstack-cinder | 17:05 | |
| *** Longgeek has quit IRC | 17:06 | |
| *** markus_z has quit IRC | 17:10 | |
| *** crose has quit IRC | 17:17 | |
| *** jordanP has quit IRC | 17:17 | |
| *** logan2 has quit IRC | 17:21 | |
| *** sgotliv has quit IRC | 17:27 | |
| *** albertom has quit IRC | 17:28 | |
| *** albertom has joined #openstack-cinder | 17:35 | |
| *** jordanP has joined #openstack-cinder | 17:35 | |
| *** amit213 has quit IRC | 17:39 | |
| *** amit213 has joined #openstack-cinder | 17:39 | |
| *** jay-deepthought is now known as jaypipes | 17:40 | |
| *** amit213 has quit IRC | 17:40 | |
| *** amit213 has joined #openstack-cinder | 17:40 | |
| *** amit213 has quit IRC | 17:41 | |
| *** amit213 has joined #openstack-cinder | 17:42 | |
| *** mutoulbj has quit IRC | 17:42 | |
| *** amit213 has quit IRC | 17:42 | |
| *** amit213 has joined #openstack-cinder | 17:42 | |
| *** mutoulbj has joined #openstack-cinder | 17:43 | |
| *** rushil has quit IRC | 17:43 | |
| *** merooney has quit IRC | 17:48 | |
| *** dustins has quit IRC | 17:48 | |
| *** delattec has joined #openstack-cinder | 17:48 | |
| *** jdurgin has quit IRC | 17:48 | |
| *** annashen has joined #openstack-cinder | 17:50 | |
| *** delatte has quit IRC | 17:50 | |
| rmstar | hi guys..need a little suggestion here. I had created a volume when my storage noe is up. But while my storage node was down, i tried to delete it. It shows that it was in state deleting, but it is really hanging. I brought the storage node up, but didn't do anything. I resetted the state and to delete again, and again it hangs on 'deleting' state. Any suggestion how i can get ride of the vo | 17:50 |
| rmstar | lume? | 17:50 |
| *** logan2 has joined #openstack-cinder | 17:50 | |
| *** delatte has joined #openstack-cinder | 17:50 | |
| jgriffith | rmstar: what backend device are you using? | 17:51 |
| openstackgerrit | Sean Chen proposed openstack/cinder: Tintri snapshot id https://review.openstack.org/198829 | 17:51 |
| *** delattec has quit IRC | 17:51 | |
| jgriffith | rmstar: when you brought the backend back online did you restart your c-vol services? You shouldn't need to but maybe the driver you have isn't properly retrying/checking | 17:51 |
| *** cdelatte has joined #openstack-cinder | 17:52 | |
| *** jwcroppe has joined #openstack-cinder | 17:53 | |
| *** merooney has joined #openstack-cinder | 17:53 | |
| *** dustins has joined #openstack-cinder | 17:53 | |
| rmstar | jgriffith: i didn't restart anything. I am relatively new to all this. I am not sure what you mean by backend device. But i can certainly restart the c-services to see if that helps | 17:53 |
| jgriffith | rmstar: well, you said storage node... guess I should assume that means you're using LVM/ref driver | 17:54 |
| *** e0ne has joined #openstack-cinder | 17:54 | |
| *** delattec has joined #openstack-cinder | 17:54 | |
| rmstar | i essentially configured the node according to the juno installation guide, and it did not tell me to set up any driver...unless i missed something :S | 17:54 |
| rmstar | just checked cinder.conf. no driver parameter | 17:55 |
| *** jasondotstar has quit IRC | 17:56 | |
| jgriffith | rmstar: k, so it's the default NP | 17:56 |
| rmstar | jgriffith: btw, what would be the default backend (for my info) | 17:57 |
| e0ne | rmstar: it will use default (LVM) driver | 17:57 |
| jgriffith | rmstar: do a "cinder service-list" cmd, make sure c-vol is running and good | 17:57 |
| rmstar | eOne: ok. thanks :) | 17:57 |
| openstackgerrit | Sean Chen proposed openstack/cinder: Remove duplicate volume.filters entry https://review.openstack.org/188899 | 17:57 |
| *** jskinner has quit IRC | 17:58 | |
| rmstar | jgriffith: yes. status enabled and state is up | 17:58 |
| *** cdelatte has quit IRC | 17:58 | |
| *** delatte has quit IRC | 17:58 | |
| *** jordanP has quit IRC | 17:58 | |
| *** cdelatte has joined #openstack-cinder | 17:58 | |
| *** jskinner has joined #openstack-cinder | 17:58 | |
| jgriffith | rmstar: on your volume node, do "sudo lvs" seee if your volume is there | 17:58 |
| rmstar | jgriffith: yes. the volume's there. | 18:01 |
| *** delatte has joined #openstack-cinder | 18:01 | |
| jgriffith | rmstar: odd.. so you mentioned you reset state succesfully, then try "cinder delete" again and it hangs? | 18:02 |
| jgriffith | rmstar: what's in your logs? Check c-sch log for ERROR or TRACE | 18:02 |
| jgriffith | rmstar: perhaps c-vol as well | 18:02 |
| jgriffith | Ohh.... one other thing :) | 18:02 |
| jgriffith | how big is the volume? | 18:02 |
| jgriffith | and did you set secure_delete = False? | 18:03 |
| *** zhenguo has quit IRC | 18:03 | |
| rmstar | jgriffith: i did not set the secure_delete parameter. The volume is 10G. I will double check the logs. | 18:04 |
| *** e0ne is now known as e0ne_ | 18:04 | |
| *** rushil has joined #openstack-cinder | 18:04 | |
| jgriffith | rmstar: so if you don't set "volume_clear=none" in your conf file it does a dd of zeros to every block on the dev | 18:04 |
| jgriffith | rmstar: which can take a long time and be rather problematic on some systems | 18:04 |
| *** delattec has quit IRC | 18:05 | |
| *** cdelatte has quit IRC | 18:05 | |
| openstackgerrit | Sean Chen proposed openstack/cinder: Tintri snapshot id https://review.openstack.org/198829 | 18:06 |
| rmstar | jgriffith: i did not set volume_clear=none in my config. | 18:07 |
| rmstar | jgriffith: just checked the logs. No errors. | 18:07 |
| *** med_` is now known as med_ | 18:07 | |
| *** med_ has joined #openstack-cinder | 18:07 | |
| *** jasondotstar has joined #openstack-cinder | 18:08 | |
| jgriffith | rmstar: did the volume go to error_deleting status? | 18:09 |
| jgriffith | sorry, I'm sure you mentioned that but I forget :) | 18:09 |
| rmstar | jgriffith: np. just stuck on 'deleting' i can just wait to see if it goes to error or if it actually works. :) | 18:09 |
| rmstar | jgriffith: not to confuse things, but i also tried to create a 1G volume when the storage was down, and it is stuck in 'creating' status. | 18:10 |
| jgriffith | rmstar: so I think you're just stuck waiting for the secure delete | 18:10 |
| jgriffith | rmstar: well, yeah, it's not very smart about some things | 18:11 |
| jgriffith | rmstar: can you create volumes now? | 18:11 |
| *** IlyaG_ has quit IRC | 18:12 | |
| rmstar | jgriffith: it's not a problem. I know there are still lotsa scenarios we need to iron out. I thought i check with you guys to make sure i'm not overlooking anything before deeming this as a bug. | 18:12 |
| rmstar | jgriffith: let me try to create | 18:12 |
| rmstar | jgriffith: yes. creation successful. | 18:14 |
| rmstar | jgriffith: also able to delete | 18:14 |
| jgriffith | cool | 18:15 |
| *** anshul has joined #openstack-cinder | 18:15 | |
| *** daneyon_ has joined #openstack-cinder | 18:15 | |
| *** lpetrut has quit IRC | 18:16 | |
| rmstar | jgriffith: i will try to recreate the probelm. If it is recreatable, i will add this to the bug list :) | 18:16 |
| rmstar | jgriffith: thanks for your help :) | 18:16 |
| jgriffith | rmstar: sure | 18:17 |
| smcginnis | jgriffith: Got a sec? Wondering if you could give some historical background on the "integrated" unit tests. | 18:18 |
| *** daneyon has quit IRC | 18:18 | |
| smcginnis | jgriffith: As in - what the heck are they and are they even needed. :) | 18:19 |
| *** dims_ has quit IRC | 18:20 | |
| jgriffith | smcginnis: frankly I *HATE* them | 18:20 |
| smcginnis | jgriffith: I'm starting to share the dislike. ;) | 18:20 |
| jgriffith | smcginnis: but, thingee introduced those as I remember, so you need to talk to him about it :) | 18:21 |
| hemna | "integrated" unit tests? | 18:21 |
| jgriffith | actually, those may be there from original fork | 18:22 |
| smcginnis | Are they supposed to be something in between a unit test and a tempest test? I just don't understand them really. | 18:22 |
| jgriffith | looking at github | 18:22 |
| smcginnis | jgriffith: Yeah, most appear so. | 18:22 |
| jgriffith | Theyr'e crap is what they are :) | 18:22 |
| smcginnis | hemna: cinder/tests/unit/integrated | 18:22 |
| jgriffith | smcginnis: they should go away and be replaced with the functional tests | 18:22 |
| jgriffith | which nobody has picked up any work on | 18:23 |
| * jgriffith looks away :) | 18:23 | |
| smcginnis | jgriffith: So my question now is... are they of any use, and should I just submit a patch to blow away the whole directory? :D | 18:23 |
| jgriffith | smcginnis: I suspect that you'll get push back if there isn't something to replace them | 18:24 |
| *** e0ne_ is now known as e0ne | 18:24 | |
| smcginnis | jgriffith: I'll add it to my list of things to bring up at the midcycle I guess. | 18:25 |
| *** bluex-pl has quit IRC | 18:25 | |
| hemna | smcginnis, looks like nothing is mocked out, so does it require a running Cinder to have these tests work? | 18:25 |
| smcginnis | hemna: That was my understanding. | 18:25 |
| hemna | and isn't that what tempest is supposed to test ? | 18:25 |
| * hemna is confused | 18:25 | |
| smcginnis | hemna: Which confuses me - yeah, why not tempest. | 18:25 |
| * hemna shrugs | 18:25 | |
| jgriffith | hemna: it runs client calls | 18:26 |
| smcginnis | I was assuming maybe these were a pre-tempest attempt to do full integration testing. | 18:26 |
| jgriffith | to the api... anyway | 18:26 |
| *** lpetrut has joined #openstack-cinder | 18:26 | |
| hemna | it calls the api | 18:26 |
| smcginnis | I guess to ask another way - is there anything we would lose here by deleting these that are not covered by tempest? | 18:27 |
| hemna | I don't see client calls, I presume self.api = cinder/volume/api.py ? | 18:27 |
| *** Yogi1 has joined #openstack-cinder | 18:27 | |
| e0ne | smcginnis: imo, we need to create functional tests instead of "integrated" | 18:27 |
| smcginnis | e0ne: What should that be then that is different than tempest? | 18:28 |
| *** annashen has quit IRC | 18:28 | |
| hemna | isn't tempest's charter is to make sure the REST API's work ? | 18:28 |
| smcginnis | e0ne: Just trying to understand the need that this is filling. | 18:28 |
| jgriffith | Ok.. one more try: | 18:28 |
| hemna | that seems different than ensuring cinder's internal stuff does X | 18:28 |
| e0ne | smcginnis: it was on Paris summit | 18:28 |
| jgriffith | Those are API tests that mock and use a bunch of fixtures to call the API | 18:28 |
| jgriffith | they are a PITA to maintain | 18:28 |
| jgriffith | which I suspect is why smcginnis is asking | 18:28 |
| jgriffith | IMO they are kinda lame | 18:29 |
| e0ne | smcginnis: to move some tests inside projects instead of tempest | 18:29 |
| jgriffith | and should be replaced by cinder/tests/functional/ | 18:29 |
| *** thangp is now known as thangp_afk | 18:29 | |
| hemna | jgriffith, +1 | 18:29 |
| e0ne | jgriffith: +1 | 18:29 |
| jgriffith | removing them is fine, but they have to have equivalents | 18:29 |
| jgriffith | and although I strated the functional work it still needs "finished" | 18:29 |
| jgriffith | inparticular it needs a gate job | 18:29 |
| jgriffith | and then obviously the test suite needs grown | 18:30 |
| *** blmartin has quit IRC | 18:30 | |
| jgriffith | I can work on that again here in a while, in the meantime as much as I don't care about those tests I would say they shouldn't just be blindly removed | 18:30 |
| jgriffith | smcginnis: if you need help figuring out why your change is breaking them maybe we can help | 18:30 |
| e0ne | jgriffith: thanks for reminder:). i was going to send mail to openstack-dev to invite people to writr functional tests for cinderclient:) | 18:30 |
| jgriffith | a few of us have been through the pain of that before | 18:30 |
| jgriffith | e0ne: :) | 18:31 |
| smcginnis | smcginnis: Thanks, nothing of mine breaking. Just cleaning some things up and don't understand them. | 18:31 |
| smcginnis | Oops | 18:31 |
| smcginnis | jgriffith: Related to https://review.openstack.org/#/c/197723/ | 18:31 |
| jgriffith | we should test that and bounce anybody that puts a LOG back in there FWIW | 18:33 |
| smcginnis | jgriffith: Hacking check? | 18:34 |
| jgriffith | smcginnis: as much as I hate myself for even saying it "yes" | 18:35 |
| smcginnis | Hehe | 18:35 |
| jgriffith | :) | 18:35 |
| *** lpetrut has quit IRC | 18:36 | |
| smcginnis | jgriffith: Problem is we have lines like: block_base.LOG.error.assert_called_once_with(msg) | 18:36 |
| *** anshul has quit IRC | 18:37 | |
| smcginnis | jgriffith: So just looking for "LOG.[error|exception|warn|debug]" would have false hits. | 18:37 |
| smcginnis | Not that it couldn't be done... | 18:37 |
| jgriffith | smcginnis: yeah, maybe we could just all be more strict in review | 18:38 |
| jgriffith | smcginnis: and frankly those tests that a log message was called are AWFUL!!!! | 18:38 |
| jgriffith | I HATE them and they should die a slow and painful death | 18:38 |
| *** annashen has joined #openstack-cinder | 18:38 | |
| hemna | jgriffith, +1 | 18:38 |
| smcginnis | ;) | 18:38 |
| hemna | die die die! | 18:38 |
| jgriffith | IMO anything that imports LOG in the unit tests dir could basicly be bansished | 18:39 |
| *** annashen has quit IRC | 18:40 | |
| e0ne | jgriffith: +2. checking log messages in tests should go away | 18:41 |
| smcginnis | e0ne: +1. The whole thing of checking the text passed for log messages is another one I would like to see go away. | 18:45 |
| e0ne | smcginnis: sometimes, i want to drop error message checking too:) | 18:45 |
| *** jdurgin has joined #openstack-cinder | 18:45 | |
| openstackgerrit | Jon Bernard proposed openstack/cinder: Add support for generic volume migration https://review.openstack.org/187270 | 18:48 |
| *** Rockyg has joined #openstack-cinder | 18:55 | |
| patrickeast | jgriffith: hey, got a min? i am looking into less ridiculous ways to make the generic image cache thing work with non-public images… i have an idea but want to bounce it off someone | 18:55 |
| openstackgerrit | Raunak Kumar proposed openstack/cinder: Implements: blueprint nimble-add-extra-specs-support https://review.openstack.org/198853 | 18:56 |
| *** annashen has joined #openstack-cinder | 18:57 | |
| *** DericHorn-HP has joined #openstack-cinder | 18:58 | |
| *** jskinner has quit IRC | 18:59 | |
| jungleboyj | jgriffith: You still around? | 19:00 |
| *** jskinner has joined #openstack-cinder | 19:00 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Remove unused config.generator module https://review.openstack.org/198721 | 19:00 |
| *** Longgeek has joined #openstack-cinder | 19:02 | |
| *** e0ne has quit IRC | 19:06 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Add support for consistency groups https://review.openstack.org/196818 | 19:14 |
| *** reed has quit IRC | 19:18 | |
| *** reed has joined #openstack-cinder | 19:18 | |
| *** Yogi11 has joined #openstack-cinder | 19:25 | |
| *** e0ne has joined #openstack-cinder | 19:26 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Remove useless logging from unit tests https://review.openstack.org/197723 | 19:26 |
| *** Yogi1 has quit IRC | 19:27 | |
| *** Longgeek has quit IRC | 19:27 | |
| *** gouthamr has quit IRC | 19:29 | |
| *** Yogi1 has joined #openstack-cinder | 19:29 | |
| *** gouthamr has joined #openstack-cinder | 19:30 | |
| *** mutoulbj has quit IRC | 19:30 | |
| *** Yogi11 has quit IRC | 19:31 | |
| *** dims_ has joined #openstack-cinder | 19:36 | |
| *** annashen has quit IRC | 19:37 | |
| *** lpetrut has joined #openstack-cinder | 19:38 | |
| *** annashen has joined #openstack-cinder | 19:38 | |
| *** annashen has quit IRC | 19:42 | |
| *** e0ne has quit IRC | 19:47 | |
| *** rushil has quit IRC | 19:49 | |
| *** rushil has joined #openstack-cinder | 19:49 | |
| *** anshul has joined #openstack-cinder | 19:55 | |
| *** bswartz has quit IRC | 19:55 | |
| *** anshul has quit IRC | 19:59 | |
| *** jasondotstar has quit IRC | 20:00 | |
| *** Yogi1 has quit IRC | 20:00 | |
| *** DericHorn-HP has quit IRC | 20:02 | |
| *** Yogi1 has joined #openstack-cinder | 20:03 | |
| *** timcl has quit IRC | 20:03 | |
| *** DericHorn-HP has joined #openstack-cinder | 20:03 | |
| *** sgotliv has joined #openstack-cinder | 20:05 | |
| jgriffith | jungleboyj: back.. what's up? | 20:06 |
| *** annashen has joined #openstack-cinder | 20:07 | |
| jungleboyj | jgriffith: Howdy! Just wanted to check how things are progressing with the code for replication. | 20:07 |
| jgriffith | jungleboyj: yeah, should have a patch up later this week or next week if I get side-tracked again :( | 20:07 |
| jungleboyj | jgriffith: Ok, didn't want to be a pest, but have been getting questions about it. | 20:08 |
| jgriffith | No worries | 20:08 |
| jungleboyj | Anything I can do to help right now? | 20:08 |
| jgriffith | Not really, unless you want to take it | 20:08 |
| *** dims_ has quit IRC | 20:17 | |
| jgriffith | patrickeast: so what are you thinking? | 20:21 |
| patrickeast | jgriffith: so, the problem is that when creating the volumes owned by the internal tenant we need one request context for volume type things in cinder, but the original request context for the image related stuff | 20:23 |
| patrickeast | jgriffith: to test it out and make sure it mostly works, i piped through a new image_context parameter all the way through and put it in the glance client calls, but it requires changing the driver api for copy_image_to_volume, which is implemented by a ton of folks… and i dont really want to mess with it | 20:24 |
| *** akerr has quit IRC | 20:24 | |
| *** Longgeek has joined #openstack-cinder | 20:24 | |
| patrickeast | jgriffith: i’m thinking of storing the context for the image related stuff in the image service client thing https://github.com/openstack/cinder/blob/master/cinder/image/glance.py#L197 and then breaking the pattern we have now to instead not pass in one for each call made on it, but to instead use whatever was given as part of its __init__() | 20:25 |
| patrickeast | jgriffith: its slightly less messy, but changes code in a somewhat subtle way… which is also not so good | 20:26 |
| jungleboyj | jgriffith: No, want to see what you come up with and then can work with you from there. | 20:26 |
| *** rushil has quit IRC | 20:26 | |
| jungleboyj | Thanks for the update. | 20:26 |
| patrickeast | jgriffith: i’m also not sure how attached we are to the ‘pass context into everything’ pattern vs store context on objects that get passed around | 20:26 |
| *** annegentle has quit IRC | 20:27 | |
| jungleboyj | Back online in a bit. | 20:28 |
| jgriffith | patrickeast: that's a good question | 20:29 |
| jgriffith | patrickeast: the only objection I'd have to embedding context is inconsistency | 20:29 |
| *** rushil has joined #openstack-cinder | 20:29 | |
| jgriffith | patrickeast: also it's prone to unauthorized access IMO | 20:29 |
| jgriffith | patrickeast: it's one of those cases where explicit seems like a good idea | 20:29 |
| jgriffith | patrickeast: so what about my suggestion before... | 20:31 |
| jgriffith | patrickeast: just leaving it sort of like how I have it, but use a db entry to store info about it? | 20:31 |
| jgriffith | patrickeast: alleviate the whole context thing | 20:31 |
| jgriffith | patrickeast: because the db entry is basicly "public" | 20:31 |
| jgriffith | patrickeast: or "admin" but doesn't matter | 20:32 |
| jgriffith | patrickeast: because it's internal only | 20:32 |
| jgriffith | patrickeast: not part of list or anything | 20:32 |
| *** jungleboyj has quit IRC | 20:32 | |
| jgriffith | patrickeast: honestly the more I looked at it the more I think using the create--->clone might be more restrictive than it is helpful | 20:33 |
| jgriffith | but I could be wrong | 20:33 |
| jgriffith | I just think it's tricky depending on whether somebody does optimized cloning or not | 20:33 |
| jgriffith | or of course if they use snpashots vs volumes etc etc | 20:33 |
| jgriffith | patrickeast: certainly not saying we can't make it work | 20:34 |
| jgriffith | patrickeast: just throwing out a thought | 20:34 |
| jgriffith | patrickeast: the other thing is introduce a new method might be good | 20:34 |
| jgriffith | patrickeast: so that a driver can override it | 20:34 |
| jgriffith | patrickeast: a general one exists and can be used, but can be over-ridden as well | 20:34 |
| jgriffith | patrickeast: not sure if what I'm saying is making any sense or not | 20:34 |
| patrickeast | jgriffith: yep, sry, someone just called me, just catching up on irc | 20:36 |
| *** simondodsley has joined #openstack-cinder | 20:37 | |
| patrickeast | jgriffith: so i like the idea of making it do less, and only using the cache db to track them | 20:38 |
| patrickeast | jgriffith: i think one of the alternative approaches we talked about a while ago was very similar, adding a new driver method like create_from_cache_image or something like that | 20:39 |
| thingee | jgriffith, smcginnis I don't remember introducing cinder.tests.unit.integrated | 20:39 |
| smcginnis | thingee: Yeah, I think we figured out they were original. | 20:39 |
| smcginnis | Most have copyright to Justin Santa Barbra | 20:40 |
| patrickeast | jgriffith: im trying to remember exactly why i moved away from that approach, i think the benefit of doing it from a little big higher level where it is now is that the cache entries are tracked as just normal volumes and that it can use any driver’s create from source functionality | 20:41 |
| patrickeast | jgriffith: but maybe you are right, and that approach is making it too restrictive | 20:42 |
| *** IlyaG has joined #openstack-cinder | 20:43 | |
| *** IlyaG has quit IRC | 20:43 | |
| patrickeast | jgriffith: i’ll maybe take a few hours today and play around with not using the internal tenant, high level cinder apis, and try just some new driver api’s… see if the tradeoffs make sense | 20:45 |
| *** Rockyg has quit IRC | 20:45 | |
| *** Longgeek has quit IRC | 20:50 | |
| jgriffith | patrickeast: So maybe later today/this-evening we can look at it together? | 20:56 |
| jgriffith | patrickeast: I think there's probably some cool things we could make happen there | 20:56 |
| patrickeast | jgriffith: sounds good | 20:57 |
| *** thangp_afk is now known as thangp | 21:00 | |
| *** eharney has quit IRC | 21:01 | |
| *** ociuhandu has joined #openstack-cinder | 21:01 | |
| *** lpabon has quit IRC | 21:03 | |
| *** DericHorn-HP has quit IRC | 21:05 | |
| *** annashen has quit IRC | 21:05 | |
| *** annegentle has joined #openstack-cinder | 21:05 | |
| *** annashen has joined #openstack-cinder | 21:06 | |
| *** jungleboyj has joined #openstack-cinder | 21:06 | |
| *** simondodsley has quit IRC | 21:06 | |
| *** lpetrut has quit IRC | 21:08 | |
| *** dims_ has joined #openstack-cinder | 21:09 | |
| *** annasort has joined #openstack-cinder | 21:10 | |
| *** annashen has quit IRC | 21:10 | |
| *** dims_ has quit IRC | 21:14 | |
| *** annashen has joined #openstack-cinder | 21:14 | |
| *** thangp has quit IRC | 21:14 | |
| annasort | Hi! I could not find any cinder code that is using this policy rule in /etc/cinder/policy.json: "volume:services": "rule:admin_api". There is "volume_extension:services": "rule:admin_api" that is used by cinder/api/contrib/services.py. Is anything actually using the first rule or can it be deleted? | 21:14 |
| *** dims has joined #openstack-cinder | 21:16 | |
| *** rushil has quit IRC | 21:20 | |
| edmondsw | jungleboyj ^^ you know anything here, or who to direct that to? I don't see anything using that either | 21:24 |
| *** ociuhandu has quit IRC | 21:25 | |
| *** jasondotstar has joined #openstack-cinder | 21:25 | |
| jungleboyj | edmondsw and annasort I thought that was only in our internal code base and that it might be something PowerVC specific. | 21:26 |
| jungleboyj | Had copied Gerald McBrearty to see if he knew anything. | 21:26 |
| annasort | jungleboyj: you are not confusing this with last week's question about check_attach/detach cinder? | 21:28 |
| edmondsw | jungleboyj no, https://github.com/openstack/cinder/blob/master/etc/cinder/policy.json#L55 | 21:28 |
| *** akerr has joined #openstack-cinder | 21:28 | |
| jungleboyj | annasort: Oh, sorry, I am confused. | 21:29 |
| jungleboyj | annasort: If it doesn't appear to be used I would propose a patch to remove it and get some eyes on that to see if it concerns anyone. | 21:31 |
| annasort | ok, I will open a bug, thanks | 21:31 |
| jungleboyj | If you don't see it being used anywhere I would guess that it isn't being used anymore and its removal was just forgotten. | 21:31 |
| annasort | it does not appear to be used but I might be missing something | 21:31 |
| *** dustins has quit IRC | 21:31 | |
| annasort | ok | 21:32 |
| jungleboyj | :-) It wouldn't be the first time that has happened. | 21:32 |
| annasort | :-) | 21:32 |
| *** akerr has quit IRC | 21:34 | |
| *** julim has quit IRC | 21:34 | |
| *** Yogi1 has quit IRC | 21:36 | |
| *** ianbrown has joined #openstack-cinder | 21:46 | |
| *** ianbrown has quit IRC | 21:46 | |
| *** Longgeek has joined #openstack-cinder | 21:47 | |
| *** merooney has quit IRC | 21:49 | |
| *** tpsilva has quit IRC | 21:55 | |
| *** DericHorn-HP has joined #openstack-cinder | 21:55 | |
| *** MentalRay has joined #openstack-cinder | 22:00 | |
| *** hodos has quit IRC | 22:02 | |
| *** Longgeek has quit IRC | 22:06 | |
| *** daneyon has joined #openstack-cinder | 22:09 | |
| *** bill_az has quit IRC | 22:10 | |
| *** daneyon_ has quit IRC | 22:12 | |
| *** gouthamr has quit IRC | 22:13 | |
| *** jungleboyj has quit IRC | 22:16 | |
| *** jasondotstar has quit IRC | 22:17 | |
| *** sparr has quit IRC | 22:18 | |
| *** sparr has joined #openstack-cinder | 22:18 | |
| *** mriedem is now known as mriedem_away | 22:23 | |
| *** bswartz has joined #openstack-cinder | 22:27 | |
| *** jasondotstar has joined #openstack-cinder | 22:37 | |
| *** annegentle has quit IRC | 22:40 | |
| *** annegentle has joined #openstack-cinder | 22:44 | |
| *** edmondsw has quit IRC | 22:47 | |
| openstackgerrit | Anish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm. https://review.openstack.org/193451 | 22:49 |
| *** chlong_ has joined #openstack-cinder | 22:50 | |
| *** chlong_ has quit IRC | 22:50 | |
| anish | tsekiyama: handled as requested ^ | 22:50 |
| *** chlong has quit IRC | 22:51 | |
| *** chlong has joined #openstack-cinder | 22:51 | |
| tsekiyama | anish: thanks, I'll try it again. | 22:51 |
| *** dims has quit IRC | 22:54 | |
| *** DericHorn-HP has quit IRC | 22:58 | |
| *** lcurtis has quit IRC | 22:59 | |
| openstackgerrit | Raunak Kumar proposed openstack/cinder: Implements: blueprint nimble-add-extra-specs-support https://review.openstack.org/198853 | 23:00 |
| *** jskinner has quit IRC | 23:01 | |
| *** annegentle has quit IRC | 23:02 | |
| *** daneyon has quit IRC | 23:05 | |
| *** jamielennox is now known as jamielennox|away | 23:11 | |
| *** reed has quit IRC | 23:15 | |
| *** reed_ has joined #openstack-cinder | 23:18 | |
| *** annegentle has joined #openstack-cinder | 23:18 | |
| *** ganso_ has quit IRC | 23:18 | |
| *** jaypipes has quit IRC | 23:18 | |
| *** annegentle has quit IRC | 23:20 | |
| *** jamielennox|away is now known as jamielennox | 23:22 | |
| *** Longgeek has joined #openstack-cinder | 23:32 | |
| *** annashen has quit IRC | 23:45 | |
| *** annashen has joined #openstack-cinder | 23:46 | |
| *** Longgeek has quit IRC | 23:50 | |
| *** annashen has quit IRC | 23:51 | |
| *** Yogi1 has joined #openstack-cinder | 23:52 | |
| *** Yogi1 has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!