Tuesday, 2015-07-07

*** dannywilson has quit IRC00:00
*** smoriya has joined #openstack-cinder00:01
*** annasort has quit IRC00:01
*** dims has joined #openstack-cinder00:01
*** hemna is now known as hemnafk00:03
*** laughterwym has joined #openstack-cinder00:05
*** chlong has quit IRC00:08
*** IanGovett has quit IRC00:11
*** _cjones_ has quit IRC00:11
*** r-daneel has quit IRC00:13
*** annasort has joined #openstack-cinder00:15
*** garthb has quit IRC00:16
*** tsekiyama has quit IRC00:26
*** lixiaoy1 has joined #openstack-cinder00:26
*** lixiaoy11 has joined #openstack-cinder00:35
*** heyun has joined #openstack-cinder00:36
*** lixiaoy1 has quit IRC00:37
*** Yogi1 has joined #openstack-cinder00:48
*** leeantho has quit IRC00:49
*** dims has quit IRC00:50
*** dims has joined #openstack-cinder00:58
*** Yogi1 has quit IRC00:59
*** zhenguo has joined #openstack-cinder01:03
*** daneyon has joined #openstack-cinder01:06
*** MentalRay has quit IRC01:10
*** erlon has quit IRC01:10
*** jasondotstar has quit IRC01:12
*** annashen has joined #openstack-cinder01:17
*** Longgeek has joined #openstack-cinder01:18
*** jwcroppe has quit IRC01:21
*** annashen has quit IRC01:22
*** jwcroppe has joined #openstack-cinder01:22
*** mtanino has quit IRC01:22
*** jasondotstar has joined #openstack-cinder01:25
*** jwcroppe has quit IRC01:26
openstackgerritxing-yang proposed openstack/os-brick: Add connector driver for the ScaleIO cinder driver  https://review.openstack.org/18583501:29
*** chlong has joined #openstack-cinder01:30
*** davechen1 has joined #openstack-cinder01:34
*** akerr has joined #openstack-cinder01:38
*** patrickeast has quit IRC01:38
*** akerr_ has joined #openstack-cinder01:40
openstackgerritwanghao proposed openstack/cinder: Notify the transfer volume action in cinder  https://review.openstack.org/18553101:41
*** patrickeast has joined #openstack-cinder01:41
*** akerr has quit IRC01:43
*** davechen has joined #openstack-cinder01:44
*** davechen2 has joined #openstack-cinder01:45
*** davechen1 has quit IRC01:46
*** davechen has quit IRC01:48
*** Longgeek has quit IRC01:51
openstackgerritTaoBai proposed openstack/cinder: Storwize Driver zone removing  https://review.openstack.org/19696601:55
*** patrickeast has quit IRC01:55
*** davechen has joined #openstack-cinder01:56
*** davechen2 has quit IRC01:57
*** gouthamr has joined #openstack-cinder01:59
*** gouthamr has quit IRC01:59
*** gouthamr has joined #openstack-cinder02:00
*** daneyon has quit IRC02:08
*** daneyon has joined #openstack-cinder02:08
*** patrickeast has joined #openstack-cinder02:08
*** mutoulbj has joined #openstack-cinder02:12
openstackgerritTaoBai proposed openstack/cinder: Storwize Driver zone removing  https://review.openstack.org/19696602:13
*** annashen has joined #openstack-cinder02:18
*** jskinner has joined #openstack-cinder02:22
*** annashen has quit IRC02:22
*** MentalRay has joined #openstack-cinder02:23
*** akerr_ has quit IRC02:25
*** mutoulbj_ has joined #openstack-cinder02:27
openstackgerritSean Chen proposed openstack/cinder: Tintri snapshot id  https://review.openstack.org/19882902:28
openstackgerritwanghao proposed openstack/cinder: Notify the transfer volume action in cinder  https://review.openstack.org/18553102:29
*** mutoulbj has quit IRC02:30
wanghaojungleboyj: ping02:39
*** daneyon has quit IRC02:45
*** asselin_ has joined #openstack-cinder02:45
wanghaojungleboyj: sorry, I have to update a new patch for (Notify the transfer volume action in cinder  https://review.openstack.org/185531). There are other mistakes in code I found. Well, you may have a look for this new one.02:47
*** BharatK has joined #openstack-cinder02:51
*** patrickeast_ has joined #openstack-cinder02:51
*** patrickeast has quit IRC02:51
*** patrickeast_ is now known as patrickeast02:51
openstackgerritVincent Hou proposed openstack/cinder: Volume status management during migration  https://review.openstack.org/18631202:53
*** hakimo has quit IRC02:54
*** hakimo has joined #openstack-cinder02:55
*** Longgeek has joined #openstack-cinder03:18
*** annashen has joined #openstack-cinder03:19
*** jwcroppe has joined #openstack-cinder03:21
*** annashen has quit IRC03:23
*** gouthamr_ has joined #openstack-cinder03:27
*** Longgeek has quit IRC03:29
*** gouthamr has quit IRC03:30
*** julim has joined #openstack-cinder03:31
*** MentalRay has quit IRC03:34
*** MentalRay has joined #openstack-cinder03:34
*** gouthamr_ has quit IRC03:35
openstackgerritKuo-tung Kao proposed openstack/cinder: cinder list fails with 'name' sort key  https://review.openstack.org/19861403:39
*** dims has quit IRC03:39
openstackgerritwanghao proposed openstack/cinder: Add ability to filter volumes list by volume_glance_metadata  https://review.openstack.org/14773803:44
*** kkeuning has joined #openstack-cinder03:45
openstackgerritThang Pham proposed openstack/cinder: Conversion to volume object  https://review.openstack.org/17705403:47
openstackgerritThang Pham proposed openstack/cinder: Fix saving tz aware datetimes in Versioned Objects  https://review.openstack.org/19607103:47
openstackgerritThang Pham proposed openstack/cinder: WIP: RPC and object versioning  https://review.openstack.org/18440403:47
*** rajinir has joined #openstack-cinder03:47
*** shyama has quit IRC03:51
*** rajinir has quit IRC03:55
openstackgerritThang Pham proposed openstack/cinder: Conversion to volume object  https://review.openstack.org/17705403:56
openstackgerritThang Pham proposed openstack/cinder: WIP: RPC and object versioning  https://review.openstack.org/18440403:56
openstackgerritTeruaki Ishizaki proposed openstack/cinder: Sheepdog: change snapshot and clone operation  https://review.openstack.org/19895203:57
*** anshul has joined #openstack-cinder03:58
*** kkeuning has quit IRC03:59
*** kkeuning has joined #openstack-cinder04:00
*** kkeuning has quit IRC04:05
*** davechen has quit IRC04:07
*** davechen has joined #openstack-cinder04:07
*** davechen1 has joined #openstack-cinder04:11
openstackgerritVincent Hou proposed openstack/python-cinderclient: Change cinderclient according to volume migration improvement  https://review.openstack.org/18954704:13
*** davechen has quit IRC04:13
*** yamada-h has joined #openstack-cinder04:14
*** lixiaoy11 has quit IRC04:17
*** davechen1 is now known as davechen04:17
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/18961404:17
*** julim has quit IRC04:19
*** annashen has joined #openstack-cinder04:19
*** jskinner has quit IRC04:22
*** lixiaoy1 has joined #openstack-cinder04:22
*** davechen2 has joined #openstack-cinder04:22
*** davechen has quit IRC04:23
*** annashen has quit IRC04:24
*** xyang1 has quit IRC04:24
*** laughterwym has quit IRC04:29
*** shyama has joined #openstack-cinder04:29
*** lixiaoy11 has joined #openstack-cinder04:32
*** yamada-h has quit IRC04:32
*** lixiaoy1 has quit IRC04:34
*** dims has joined #openstack-cinder04:40
*** BharatK has quit IRC04:41
*** dims_ has joined #openstack-cinder04:41
*** anshul has quit IRC04:43
*** _cjones_ has joined #openstack-cinder04:43
*** dims has quit IRC04:45
*** shausy has joined #openstack-cinder04:45
*** dims_ has quit IRC04:46
*** MentalRay has quit IRC04:55
*** _cjones_ has quit IRC04:56
*** _cjones_ has joined #openstack-cinder04:57
*** nihilifer has joined #openstack-cinder04:59
*** lpetrut has joined #openstack-cinder05:02
*** deepakcs has joined #openstack-cinder05:04
*** asselin_ has quit IRC05:05
*** deepakcs has quit IRC05:06
*** BharatK has joined #openstack-cinder05:08
*** patrickeast has quit IRC05:13
*** annashen has joined #openstack-cinder05:21
*** e0ne has joined #openstack-cinder05:22
*** dannywilson has joined #openstack-cinder05:23
*** nkrinner has joined #openstack-cinder05:24
*** deepakcs has joined #openstack-cinder05:25
*** dannywilson has quit IRC05:25
*** annashen has quit IRC05:25
*** dannywilson has joined #openstack-cinder05:26
*** deepakcs has quit IRC05:26
*** Longgeek has joined #openstack-cinder05:26
*** laughterwym has joined #openstack-cinder05:37
*** nlevinki has joined #openstack-cinder05:38
*** BharatK has quit IRC05:42
*** e0ne has quit IRC05:44
*** Trozz_ is now known as Trozz05:50
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/18961405:50
*** Trozz has left #openstack-cinder05:51
*** ig0r_ has joined #openstack-cinder05:52
*** Longgeek has quit IRC05:52
*** deepakcs has joined #openstack-cinder05:52
*** MentalRay has joined #openstack-cinder05:54
*** ig0r__ has quit IRC05:55
*** Longgeek has joined #openstack-cinder05:56
*** laughterwym_ has joined #openstack-cinder06:01
*** anshul has joined #openstack-cinder06:03
*** laughterwym has quit IRC06:04
*** MentalRay has quit IRC06:04
*** lpetrut has quit IRC06:04
*** BharatK has joined #openstack-cinder06:08
*** lpetrut has joined #openstack-cinder06:19
*** annashen has joined #openstack-cinder06:21
*** anshul has quit IRC06:25
*** _cjones_ has quit IRC06:25
*** anshul has joined #openstack-cinder06:25
*** annashen has quit IRC06:26
*** yrabl_ has joined #openstack-cinder06:32
*** yrabl_ has quit IRC06:33
*** belmoreira has joined #openstack-cinder06:43
*** fifieldt has joined #openstack-cinder06:43
*** sgotliv has quit IRC06:45
*** afazekas has joined #openstack-cinder06:50
openstackgerritSergey Vilgelm proposed openstack/cinder: Remove unused config.generator module  https://review.openstack.org/19872106:51
openstackgerritPranali Deore proposed openstack/cinder: Validate string, integer limit for input parameter  https://review.openstack.org/19496806:51
openstackgerritPeter Wang proposed openstack/cinder: Add white list support for target ports in VNX driver  https://review.openstack.org/19697906:53
*** annasort has quit IRC06:56
*** e0ne has joined #openstack-cinder06:57
*** e0ne has quit IRC07:04
*** dannywilson has quit IRC07:04
*** shausy has quit IRC07:05
*** shausy has joined #openstack-cinder07:06
openstackgerritVincent Hou proposed openstack/python-cinderclient: Change cinderclient according to volume migration improvement  https://review.openstack.org/18954707:17
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/18961407:18
*** annashen has joined #openstack-cinder07:22
*** belmoreira has quit IRC07:26
*** BharatK has quit IRC07:26
*** annashen has quit IRC07:27
*** e0ne has joined #openstack-cinder07:36
*** jistr has joined #openstack-cinder07:37
openstackgerritLisaLi proposed openstack/cinder: Fix restore point if backup base is diff-format in ceph  https://review.openstack.org/19899307:38
*** lpetrut has quit IRC07:40
*** jordanP has joined #openstack-cinder07:41
*** dims has joined #openstack-cinder07:43
*** laughterwym has joined #openstack-cinder07:45
*** yamada-h has joined #openstack-cinder07:45
*** dims has quit IRC07:48
*** lpetrut has joined #openstack-cinder07:48
*** shyama has quit IRC07:49
*** sgotliv has joined #openstack-cinder07:52
*** BharatK has joined #openstack-cinder07:54
openstackgerritTeruaki Ishizaki proposed openstack/cinder: Add missing backup/restore methods to Sheepdog driver  https://review.openstack.org/16292707:54
openstackgerritTeruaki Ishizaki proposed openstack/cinder: Sheepdog: Add class for dog command executor  https://review.openstack.org/19453207:54
*** yamada-h has quit IRC07:55
*** git-harry has quit IRC07:56
*** git-harry has joined #openstack-cinder07:56
*** bluex-pl has joined #openstack-cinder07:58
*** e0ne is now known as e0ne_08:01
*** e0ne_ is now known as e0ne08:03
openstackgerritTeruaki Ishizaki proposed openstack/cinder: Sheepdog: change create and delete operation  https://review.openstack.org/19739308:05
*** chlong has quit IRC08:13
openstackgerritLisaLi proposed openstack/cinder: Fix restore point if backup base is diff-format in ceph  https://review.openstack.org/19899308:21
*** _cjones_ has joined #openstack-cinder08:22
*** annashen has joined #openstack-cinder08:23
openstackgerritwanghao proposed openstack/cinder: Add support for force-delete backups  https://review.openstack.org/16612708:24
*** abehl has joined #openstack-cinder08:27
*** annashen has quit IRC08:28
*** shyama has joined #openstack-cinder08:29
openstackgerritVincent Hou proposed openstack/python-cinderclient: Change cinderclient according to volume migration improvement  https://review.openstack.org/18954708:31
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/18961408:31
openstackgerritVincent Hou proposed openstack/cinder: Volume status management during migration  https://review.openstack.org/18631208:32
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/18961408:35
openstackgerritwanghao proposed openstack/cinder: Add ability to filter volumes list by volume_glance_metadata  https://review.openstack.org/14773808:40
openstackgerritPradeep Sathasivam proposed openstack/cinder: Adds friendly zone name support  https://review.openstack.org/18051808:40
openstackgerritMichal Dulko proposed openstack/cinder: Service object  https://review.openstack.org/16041708:50
openstackgerritHuang Zhiteng proposed openstack/cinder: Update volume status AFTER terminate_connection is done  https://review.openstack.org/19901508:51
openstackgerritMichal Dulko proposed openstack/cinder: Service object  https://review.openstack.org/16041708:51
*** laughterwym has quit IRC08:51
*** laughterwym_ has quit IRC08:51
openstackgerritAnton Arefiev proposed openstack/cinder: Add tenant validation to quota  https://review.openstack.org/14364508:53
*** _cjones_ has quit IRC08:57
*** yamada-h has joined #openstack-cinder08:57
openstackgerritTeruaki Ishizaki proposed openstack/cinder: Sheepdog: change snapshot and clone operation  https://review.openstack.org/19895208:59
*** yamada-h has quit IRC09:00
*** yamada-h has joined #openstack-cinder09:00
*** shausy has quit IRC09:03
*** shausy has joined #openstack-cinder09:03
*** yamada-h has quit IRC09:05
*** yamada-h has joined #openstack-cinder09:05
openstackgerritwanghao proposed openstack/cinder: Notify the transfer volume action in cinder  https://review.openstack.org/18553109:05
openstackgerritSasikanth Eda proposed openstack/cinder: GPFS volume encryption-at-rest support  https://review.openstack.org/19550709:09
*** yamada-h has quit IRC09:09
wanghaoGuys, I have a question: Do we have a plan for cinder v3 API?09:10
*** e0ne is now known as e0ne_09:11
*** haypo has joined #openstack-cinder09:14
haypohi. my python 3 patches are reviewed and merged. thanks!09:14
haypoi'm still waiting for a new release of os-brick including the python 3 support09:15
openstackgerritKyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient  https://review.openstack.org/19764309:15
haypowho should i ask for this release?09:15
openstackgerritwanghao proposed openstack/cinder: Notify the transfer volume action in cinder  https://review.openstack.org/18553109:15
winston-dhaypo: ask hemnafk09:18
haypowinston-d: ok. i know that hemnafk doesn't like my change https://review.openstack.org/#/c/197624/ but we don't need it, hemnafk already wrote a patch for fix py3 support and it was merged09:20
*** boris-42 has quit IRC09:22
haypohemnafk: hi. for my python 3 patches for cinder, i need a new release of os-brick including your python 3 fix. would it be possible to release a new version of os-brick please?09:22
*** gaurangt has joined #openstack-cinder09:23
haypowinston-d: hemnafk is the maintainer, or at least the release manager, of os-brick?09:24
*** annashen has joined #openstack-cinder09:24
*** e0ne_ is now known as e0ne09:24
*** laughterwym has joined #openstack-cinder09:26
*** laughterwym_ has joined #openstack-cinder09:26
*** annashen has quit IRC09:29
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/19727309:37
*** dims has joined #openstack-cinder09:44
*** dims_ has joined #openstack-cinder09:45
haypoi just proposed https://review.openstack.org/#/c/199023/ "Make gate-os-brick-python34 voting and add gate"09:45
*** dims__ has joined #openstack-cinder09:46
openstackgerritVincent Hou proposed openstack/python-cinderclient: Change cinderclient according to volume migration improvement  https://review.openstack.org/18954709:47
openstackgerritVipin Balachandran proposed openstack/cinder: Add unit tests for cinder.api.v2.volumes  https://review.openstack.org/19863709:47
*** dims has quit IRC09:48
*** ociuhandu has joined #openstack-cinder09:49
*** dims_ has quit IRC09:50
openstackgerritVincent Hou proposed openstack/python-cinderclient: Change cinderclient according to volume migration improvement  https://review.openstack.org/18954709:50
*** dims__ has quit IRC09:50
*** smoriya has quit IRC09:53
*** lixiaoy11 has quit IRC09:55
openstackgerritDave Chen proposed openstack/cinder: set/unset volume image metadata  https://review.openstack.org/14772609:55
*** davechen2 has left #openstack-cinder09:55
openstackgerritShay Halsband proposed openstack/cinder: XtremIO Volume driver Consistency Groups support  https://review.openstack.org/18913509:55
*** e0ne is now known as e0ne_10:07
openstackgerritKyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient  https://review.openstack.org/19764310:12
*** IanGovett has joined #openstack-cinder10:14
*** ociuhandu has quit IRC10:17
*** e0ne_ is now known as e0ne10:21
*** mutoulbj_ has quit IRC10:23
*** annashen has joined #openstack-cinder10:25
*** heyun has quit IRC10:27
*** annashen has quit IRC10:30
*** sgotliv has quit IRC10:30
openstackgerritDaniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object  https://review.openstack.org/16364110:30
*** DTadrzak has quit IRC10:38
*** DTadrzak has joined #openstack-cinder10:39
*** ericksonsantos has joined #openstack-cinder10:40
*** Lee1092 has joined #openstack-cinder10:44
openstackgerritYuriy Nesenenko proposed openstack/cinder-specs: Checking the existence of volume  https://review.openstack.org/17482310:45
*** dims has joined #openstack-cinder10:46
*** dims has quit IRC10:51
*** yamada-h has joined #openstack-cinder10:59
*** ociuhandu has joined #openstack-cinder11:00
*** yamada-h has quit IRC11:04
*** timcl has joined #openstack-cinder11:04
*** lpetrut has quit IRC11:07
*** mutoulbj has joined #openstack-cinder11:08
*** dalgaaf has joined #openstack-cinder11:08
*** Longgeek has quit IRC11:08
*** sgotliv has joined #openstack-cinder11:10
*** delatte has quit IRC11:11
openstackgerritAnton Arefiev proposed openstack/cinder: Add tenant validation to quota  https://review.openstack.org/14364511:14
openstackgerritDaniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object  https://review.openstack.org/16364111:21
openstackgerritDaniel Tadrzak proposed openstack/cinder: Fix saving tz aware datetimes in Versioned Objects  https://review.openstack.org/19607111:21
*** gaurangt has left #openstack-cinder11:22
*** e0ne is now known as e0ne_11:25
*** annashen has joined #openstack-cinder11:26
*** sgotliv has quit IRC11:26
*** annashen has quit IRC11:30
*** delatte has joined #openstack-cinder11:31
*** e0ne_ is now known as e0ne11:45
*** Longgeek has joined #openstack-cinder11:45
*** tpsilva has joined #openstack-cinder11:52
*** jasondotstar has quit IRC11:57
*** markvoelker has quit IRC11:59
*** markvoelker has joined #openstack-cinder11:59
*** dims has joined #openstack-cinder12:02
*** jaypipes has joined #openstack-cinder12:02
openstackgerritVincent Hou proposed openstack/cinder: Add the notification and the migration progress indication  https://review.openstack.org/18961412:03
*** chlong has joined #openstack-cinder12:06
*** dims has quit IRC12:07
openstackgerritLin Yang proposed openstack/python-cinderclient: Improve error message when cinder quota exceeded  https://review.openstack.org/17894612:10
*** akerr has joined #openstack-cinder12:13
*** ganso_ has joined #openstack-cinder12:14
*** fabrice_ has quit IRC12:18
*** sgotliv has joined #openstack-cinder12:18
openstackgerritYuriy Nesenenko proposed openstack/cinder-specs: Checking the existence of volume  https://review.openstack.org/17482312:20
*** fabrice_ has joined #openstack-cinder12:20
openstackgerritYuriy Nesenenko proposed openstack/cinder-specs: Checking the existence of volume  https://review.openstack.org/17482312:22
*** lpetrut has joined #openstack-cinder12:26
*** annashen has joined #openstack-cinder12:27
*** bswartz has quit IRC12:28
*** annashen has quit IRC12:31
*** markvoelker has quit IRC12:31
openstackgerritChris Gacsal proposed openstack/cinder: Fix target removal in case of multi-attach  https://review.openstack.org/19840012:33
openstackgerritChris Gacsal proposed openstack/cinder: Fix target removal in case of multi-attach  https://review.openstack.org/19840012:34
*** edmondsw has joined #openstack-cinder12:38
*** kjnelson has joined #openstack-cinder12:40
*** markvoelker has joined #openstack-cinder12:40
*** MentalRay has joined #openstack-cinder12:41
*** MentalRay has quit IRC12:41
*** merooney has joined #openstack-cinder12:52
*** BharatK has quit IRC12:55
*** bswartz has joined #openstack-cinder12:56
*** tpsilva has quit IRC13:04
*** eharney has joined #openstack-cinder13:07
*** changbl has quit IRC13:08
*** jskinner has joined #openstack-cinder13:08
*** haomaiw__ has quit IRC13:08
*** mildan has joined #openstack-cinder13:09
*** jasondotstar has joined #openstack-cinder13:09
*** jasondot_ has joined #openstack-cinder13:10
*** jasondot_ has quit IRC13:13
*** annasort has joined #openstack-cinder13:13
*** jasondotstar has quit IRC13:14
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Pass volume size in GB to copy_volume function  https://review.openstack.org/19800613:14
*** jasondotstar has joined #openstack-cinder13:14
openstackgerritDaniel Tadrzak proposed openstack/cinder: CGSnapshot Object  https://review.openstack.org/19502713:18
openstackgerritDaniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object  https://review.openstack.org/16364113:18
*** Yogi1 has joined #openstack-cinder13:18
*** dims has joined #openstack-cinder13:19
*** haomaiwa_ has joined #openstack-cinder13:22
*** nihilifer has quit IRC13:23
*** BharatK has joined #openstack-cinder13:23
*** julim has joined #openstack-cinder13:24
*** Yogi1 has quit IRC13:27
*** mriedem_away is now known as mriedem13:27
*** annashen has joined #openstack-cinder13:27
*** mtanino has joined #openstack-cinder13:31
*** annashen has quit IRC13:32
*** BharatK has quit IRC13:32
*** boris-42 has joined #openstack-cinder13:33
openstackgerritVipin Balachandran proposed openstack/cinder: Add unit tests for cinder.api.v2.volumes  https://review.openstack.org/19863713:35
*** mildan has quit IRC13:35
*** e0ne is now known as e0ne_13:37
*** gouthamr has joined #openstack-cinder13:37
*** Ctina has joined #openstack-cinder13:39
*** e0ne_ is now known as e0ne13:39
*** dustins has joined #openstack-cinder13:39
*** jasondotstar has quit IRC13:40
*** Yogi1 has joined #openstack-cinder13:40
openstackgerritVipin Balachandran proposed openstack/cinder: Add unit tests for cinder.api.v2.volumes  https://review.openstack.org/19863713:41
*** deepakcs has quit IRC13:42
*** annegentle has joined #openstack-cinder13:42
*** haomaiwa_ has quit IRC13:42
openstackgerritVipin Balachandran proposed openstack/cinder: Raise BadRequest for invalid replication status  https://review.openstack.org/19864013:45
smcginnistbarron: ping13:46
smcginnistbarron: Wondering if you saw the response to your comment here: https://review.openstack.org/#/c/162927/13:47
*** haomaiwang has joined #openstack-cinder13:48
openstackgerritVipin Balachandran proposed openstack/cinder: Raise BadRequest for invalid replication status  https://review.openstack.org/19864013:48
*** merooney has quit IRC13:54
*** shausy has quit IRC13:54
*** r-daneel has joined #openstack-cinder13:56
*** abehl has quit IRC13:56
*** anshul has quit IRC13:56
*** annegentle has quit IRC13:57
*** lpetrut has quit IRC13:58
*** Ctina has quit IRC14:00
*** lpetrut has joined #openstack-cinder14:00
*** Ctina has joined #openstack-cinder14:00
*** rushil has joined #openstack-cinder14:01
tbarronsmcginnis: looking14:01
*** annegentle has joined #openstack-cinder14:02
*** rajinir has joined #openstack-cinder14:03
tbarronsmcginnis: yes, I read it, but can't say I feel qualified to assess the response.14:03
*** zhenguo has quit IRC14:03
tbarronsmcginnis: I don't yet understand why the issue happens with large stdin vs read of large buffer (e.g. large chunk) directly from a file14:04
*** bluex-pl has quit IRC14:05
*** xyang1 has joined #openstack-cinder14:05
tbarronsmcginnis: do we really know this issue won't happen with other backends if the chunk size is large?14:05
tbarrondo we know it couldn't be avoided with sheepdog if the read size was reduced?14:06
smcginnistbarron: OK, thanks. I _think_ I understand what's going on, but I don't feel fully qualified either.14:06
*** bluex-pl has joined #openstack-cinder14:06
*** merooney has joined #openstack-cinder14:06
tbarronto me, reverse-monkey patching eventlet in a particular driver feels all wrong14:06
smcginnistbarron: Yeah, I agree. Not that in the end it might not be valid, but it definitely feels wrong. :)14:07
tbarronBut my -1 is just to get attention of people who are more qualified on this issue.  Looks like it's starting to work.14:07
smcginnistbarron: Yeah, saw it when you posted. Was hoping for someone else to come along and explain it better. ;)14:07
*** annegentle has quit IRC14:07
openstackgerritSimon Dodsley proposed openstack/cinder: Fix incorrect error message to be specific to driver in use. Closes-Bug: #1469523  https://review.openstack.org/19911314:08
openstackbug 1469523 in Cinder "Incorrect error message in Pure cinder driver" [Undecided,In progress] https://launchpad.net/bugs/1469523 - Assigned to Simon Dodsley (simon-dodsley)14:08
*** geguileo has quit IRC14:09
*** rushil has quit IRC14:13
*** geguileo has joined #openstack-cinder14:14
*** rushil has joined #openstack-cinder14:19
*** asselin_ has joined #openstack-cinder14:20
openstackgerritSimon Dodsley proposed openstack/cinder: Fix incorrect error message to be specific to driver in use.  https://review.openstack.org/19911314:21
*** chlong has quit IRC14:25
*** annashen has joined #openstack-cinder14:28
*** simondodsley has joined #openstack-cinder14:29
*** annasort has quit IRC14:30
*** annegentle has joined #openstack-cinder14:32
*** annashen has quit IRC14:33
*** nlevinki has quit IRC14:35
*** rushil has quit IRC14:36
*** BharatK has joined #openstack-cinder14:37
*** Yogi1 has quit IRC14:38
*** thangp has joined #openstack-cinder14:38
openstackgerritSean McGinnis proposed openstack/cinder: Remove useless logging from unit tests  https://review.openstack.org/19772314:39
*** annegentle has quit IRC14:43
*** Yogi1 has joined #openstack-cinder14:44
*** BharatK has quit IRC14:46
openstackgerrithadi esiely proposed openstack/cinder: Store volume encryption metadata on each volume  https://review.openstack.org/15228414:47
DuncanTUndoing monkey patching looks like the wrong thing to do.... if you *really* need the unpatched version, you should probably grab it and save it somewhere with a new name14:50
*** nlevinki has joined #openstack-cinder14:50
*** nikeshm has quit IRC14:51
DuncanTtbarron: I think your instinctive dislike is spot on14:51
*** jungleboyj has joined #openstack-cinder14:52
tbarronDuncanT: speaking of instinctive dislike, https://review.openstack.org/#/c/188805/9/os_brick/remotefs/remotefs.py requires that vendor-specific (I think) pstorage and pstorage-mount commands be root-wrapped14:57
tbarronDuncanT: is that OK?14:57
tbarronsmcginnis: ^^14:57
eharneywe have lots of vendor-specific commands rootwrapped14:57
tbarroneharney: ok, is there any policy on these being open-sourced or whatever?14:58
eharneyno14:58
tbarroneharney: ok, that helps with my review.  Thanks.14:58
eharneytbarron: we already have pstorage/pstorage-mount in cinder's rootwrap conf14:59
*** BharatK has joined #openstack-cinder14:59
tbarroninstincts only go so far ...14:59
*** Yogi1 has quit IRC15:02
*** annegentle has joined #openstack-cinder15:04
*** laughterwym has quit IRC15:04
*** Yogi1 has joined #openstack-cinder15:05
*** Apoorva has joined #openstack-cinder15:06
*** topshare has joined #openstack-cinder15:08
*** topshare has quit IRC15:08
*** topshare has joined #openstack-cinder15:09
*** topshare has quit IRC15:09
*** topshare has joined #openstack-cinder15:09
DuncanTtbarron: The only rule is that they need to be properly mocked for tests, can't require them for tests to run15:11
*** rajinir has quit IRC15:11
tbarronDuncanT: kk, thanks.15:11
*** annegentle has quit IRC15:13
*** e0ne is now known as e0ne_15:16
*** dannywilson has joined #openstack-cinder15:16
*** dims has quit IRC15:16
*** nlevinki has quit IRC15:19
*** dannywilson has quit IRC15:20
*** tsekiyama has joined #openstack-cinder15:20
*** annegentle has joined #openstack-cinder15:21
*** _cjones_ has joined #openstack-cinder15:23
*** nkrinner is now known as nkrinner_afk15:23
*** e0ne_ is now known as e0ne15:26
*** dannywilson has joined #openstack-cinder15:27
*** merooney has quit IRC15:28
*** _cjones_ has quit IRC15:28
*** _cjones_ has joined #openstack-cinder15:28
*** annashen has joined #openstack-cinder15:29
openstackgerritKuo-tung Kao proposed openstack/cinder: cinder list fails with 'name' sort key  https://review.openstack.org/19861415:29
*** jdurgin1 has joined #openstack-cinder15:30
*** changbl has joined #openstack-cinder15:31
*** merooney has joined #openstack-cinder15:32
*** garthb has joined #openstack-cinder15:33
*** annasort has joined #openstack-cinder15:33
*** annashen has quit IRC15:34
openstackgerritChris Gacsal proposed openstack/cinder: Fix target removal in case of multi-attach  https://review.openstack.org/19840015:42
*** dims has joined #openstack-cinder15:43
openstackgerritChris Gacsal proposed openstack/cinder: Fix target removal in case of multi-attach  https://review.openstack.org/19840015:44
*** asselin_ has quit IRC15:46
*** BharatK has quit IRC15:49
*** jsafrane has joined #openstack-cinder15:52
*** akshai has joined #openstack-cinder15:53
*** crose has joined #openstack-cinder15:53
*** annegentle has quit IRC15:56
*** salv-orlando has joined #openstack-cinder15:58
openstackgerritAlex Meade proposed openstack/cinder: Add tracing facility for drivers  https://review.openstack.org/15743915:59
openstackgerritAlex Meade proposed openstack/cinder: NetApp ONTAP: Add debug tracing  https://review.openstack.org/19917015:59
*** jistr has quit IRC16:00
*** daneyon has joined #openstack-cinder16:00
*** BharatK has joined #openstack-cinder16:00
*** hemnafk is now known as hemna16:00
*** mgarza_ has joined #openstack-cinder16:00
*** aswadr has joined #openstack-cinder16:02
*** annegentle has joined #openstack-cinder16:03
*** laughterwym_ has quit IRC16:06
*** jistr has joined #openstack-cinder16:07
*** annegentle has quit IRC16:10
openstackgerritSimon Dodsley proposed openstack/cinder: Fix incorrect error message in Pure driver to use driver type in use  https://review.openstack.org/19911316:11
*** haomaiw__ has joined #openstack-cinder16:16
*** jsafrane has quit IRC16:18
*** haomaiwang has quit IRC16:18
*** leeantho has joined #openstack-cinder16:19
*** BharatK has quit IRC16:21
*** annashen has joined #openstack-cinder16:28
*** e0ne has quit IRC16:33
*** jistr has quit IRC16:35
*** jordanP has quit IRC16:38
openstackgerritSimon Dodsley proposed openstack/cinder: Fix error message in Pure driver with correct text  https://review.openstack.org/19911316:38
*** haypo has quit IRC16:39
*** haypo has joined #openstack-cinder16:40
*** annegentle has joined #openstack-cinder16:40
*** haypo has left #openstack-cinder16:41
*** annegentle has quit IRC16:44
*** patrickeast has joined #openstack-cinder16:45
*** anshul has joined #openstack-cinder16:45
*** Longgeek has quit IRC16:47
*** abehl has joined #openstack-cinder16:47
*** anshul has quit IRC16:47
*** Yogi1 has quit IRC16:51
*** diogogmt has joined #openstack-cinder16:57
*** bluex-pl has quit IRC16:58
*** afazekas has quit IRC17:01
*** ociuhandu has quit IRC17:04
*** lcurtis has joined #openstack-cinder17:04
*** anshul has joined #openstack-cinder17:05
openstackgerritRaunak Kumar proposed openstack/cinder: Add extra spec capability for Nimble Cinder Driver  https://review.openstack.org/19885317:06
*** aswadr has quit IRC17:07
*** aix has quit IRC17:08
*** salv-orl_ has joined #openstack-cinder17:08
openstackgerritRaunak Kumar proposed openstack/cinder: Add extra spec capability for Nimble Cinder Driver  https://review.openstack.org/19885317:08
*** e0ne has joined #openstack-cinder17:09
*** salv-orlando has quit IRC17:11
openstackgerritHuang Zhiteng proposed openstack/cinder: Update volume status AFTER terminate_connection is done  https://review.openstack.org/19901517:14
*** lpetrut has quit IRC17:16
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix update_migrate_volume() exception handling  https://review.openstack.org/19873117:17
*** lpetrut has joined #openstack-cinder17:17
*** e0ne is now known as e0ne_17:18
*** david-lyle has quit IRC17:19
*** e0ne_ is now known as e0ne17:21
*** e0ne is now known as e0ne_17:26
*** e0ne_ is now known as e0ne17:27
rlucio\17:28
*** lpetrut has quit IRC17:29
*** annegentle has joined #openstack-cinder17:30
*** e0ne is now known as e0ne_17:33
*** annegentle has quit IRC17:35
*** timcl has quit IRC17:35
*** e0ne_ is now known as e0ne17:39
*** Yogi1 has joined #openstack-cinder17:45
*** merooney has quit IRC17:47
*** Longgeek has joined #openstack-cinder17:53
*** reed_ has quit IRC17:59
*** mgarza_ has quit IRC18:00
*** julim_ has joined #openstack-cinder18:03
*** annashen has quit IRC18:03
*** Longgeek has quit IRC18:04
*** coolsvap|away is now known as coolsvap18:04
*** changbl has quit IRC18:05
*** julim has quit IRC18:05
*** jwcroppe has quit IRC18:05
*** annashen has joined #openstack-cinder18:05
*** jwcroppe has joined #openstack-cinder18:06
*** mgarza_ has joined #openstack-cinder18:06
*** lpetrut has joined #openstack-cinder18:16
*** timcl has joined #openstack-cinder18:17
*** e0ne is now known as e0ne_18:17
openstackgerritDaniel Tadrzak proposed openstack/cinder: Cleanup for SnapshotObject  https://review.openstack.org/19834118:21
*** salv-orl_ has quit IRC18:22
*** lpetrut has quit IRC18:23
*** annashen has quit IRC18:23
*** jwcroppe has quit IRC18:23
*** annashen has joined #openstack-cinder18:24
*** andymccr has quit IRC18:25
*** annashen has quit IRC18:28
openstackgerritAdriano Freires Rosso proposed openstack/cinder: Adds manage/unmanage methods for HNAS drivers  https://review.openstack.org/19799918:33
*** Apoorva has quit IRC18:33
*** lpetrut has joined #openstack-cinder18:36
*** mgarza_ has quit IRC18:38
*** e0ne_ is now known as e0ne18:39
*** abehl has quit IRC18:41
*** anshul has quit IRC18:42
*** Apoorva has joined #openstack-cinder18:42
*** lpetrut has quit IRC18:42
*** mgarza_ has joined #openstack-cinder18:45
*** salv-orlando has joined #openstack-cinder18:49
*** e0ne has quit IRC18:50
*** dims has quit IRC18:53
openstackgerritDaniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object  https://review.openstack.org/16364118:53
*** Lee1092 has quit IRC18:55
*** mriedem has quit IRC18:55
*** annegentle has joined #openstack-cinder18:56
openstackgerritSean Chen proposed openstack/cinder: Tintri snapshot id  https://review.openstack.org/19882918:57
*** dims has joined #openstack-cinder18:58
*** shyama has quit IRC19:02
*** harlowja_ has joined #openstack-cinder19:02
*** harlowja has quit IRC19:03
*** ociuhandu has joined #openstack-cinder19:05
*** Rockyg has joined #openstack-cinder19:06
*** mriedem has joined #openstack-cinder19:06
*** annashen has joined #openstack-cinder19:06
*** angela-s has joined #openstack-cinder19:11
*** mgarza_ has quit IRC19:14
*** topshare has quit IRC19:19
*** ndipanov has quit IRC19:20
*** topshare has joined #openstack-cinder19:21
*** changbl has joined #openstack-cinder19:22
*** mgarza has joined #openstack-cinder19:24
*** mriedem has quit IRC19:25
*** mriedem has joined #openstack-cinder19:29
SwansonAny cores floating around bored?  https://review.openstack.org/#/c/196818/ adds consistency groups to the dell driver.  Could use a push.19:30
*** bswartz has quit IRC19:32
*** rushil has joined #openstack-cinder19:32
*** bswartz has joined #openstack-cinder19:32
openstackgerritPatrick East proposed openstack/cinder: WIP generic image cache  https://review.openstack.org/19579519:34
patrickeastjgriffith: ^ i changed things around so it doesn’t use the higher level api’s to do stuff (except for delete), i tried out going even lower and using new driver api’s but things work much nicer imo if we use real volume db entries and stick with the manager level api’s for the create/clones19:36
patrickeastlemmie kno if that is more in line with what you were thinking19:37
Swansongeneric image cache?  This require any driver work to support?19:39
patrickeastSwanson: as-is, nope19:39
Swansonpatrickeast: You just create a volume and then snap off clones whenever you create an image on that backend?  Nice.  That sound you hear is "image caching" falling off my plate.19:44
jgriffithpatrickeast: awesome, I'll check it out19:46
*** dims has quit IRC19:48
*** david-lyle has joined #openstack-cinder19:50
mfischthingee: whats the ETA on a new cinder-client? bug #1464160 is causing some annoyances for us19:51
openstackbug 1464160 in python-cinderclient "ERROR: Bad Request (HTTP 400) with cinder 1.2.1" [Critical,Fix committed] https://launchpad.net/bugs/1464160 - Assigned to John Griffith (john-griffith)19:51
*** jwcroppe has joined #openstack-cinder19:53
*** e0ne has joined #openstack-cinder19:54
thingeemfisch: soon. I believe we just merged that today19:54
mfischyou did just merge it19:54
mfischwe have the exact issue you list in the commit message19:54
*** merooney has joined #openstack-cinder19:55
*** timcl has quit IRC19:59
thingeemfisch: excellent, sorry for the trouble there. missed this particular case with a proxy involved.19:59
thingeemfisch: I'll see about doing a tag release today.19:59
*** mgarza has quit IRC20:00
*** Longgeek has joined #openstack-cinder20:01
*** rushil has quit IRC20:02
thingeejgriffith: thanks on getting the patch through for the revert. I think it was the right call20:02
thingeeand based on the discussion from the list20:02
jgriffiththingee: yeah, unfortunately probably the best option at this point20:03
*** mgarza_ has joined #openstack-cinder20:03
thingeewe can always revisit later once the config option to help with this case has been out for some time20:03
*** chlong has joined #openstack-cinder20:06
mfischwhats the typical backwards compat requirement with clients? because this will break anyone w/o kilo right?20:07
thingeemfisch: right. in Kilo we introduced a public_endpoint config option to get around the proxy issue called public_endpoint http://docs.openstack.org/kilo/config-reference/content/cinder-conf-changes-kilo.html20:09
openstackgerritAlex Meade proposed openstack/cinder: NetApp ONTAP: Add debug tracing  https://review.openstack.org/19917020:10
openstackgerritAlex Meade proposed openstack/cinder: Add tracing facility for drivers  https://review.openstack.org/15743920:10
mfischis the client supposed to be N-1 compatible?20:10
thingeemfisch: it was a mistake initially to introduce this change without that configuration option being available in stable releases. backporting config option changes can sometimes cause confusion though since you need to be using a particular stable release of a version.20:11
thingeemfisch: I'm unaware if we ever came to a consensus on that.20:12
*** rushil has joined #openstack-cinder20:13
mfischthanks thingee I was just curious20:13
*** merooney has quit IRC20:14
*** lpetrut has joined #openstack-cinder20:17
openstackgerritJon Bernard proposed openstack/cinder: Add support for generic volume migration  https://review.openstack.org/18727020:19
*** thangp has quit IRC20:19
openstackgerritAdriano Freires Rosso proposed openstack/cinder: Adds manage/unmanage methods for HNAS drivers  https://review.openstack.org/19799920:19
openstackgerritMike Perez proposed openstack/python-cinderclient: Prep for 1.3.0  https://review.openstack.org/19924220:19
*** Ctina has quit IRC20:20
*** salv-orlando has quit IRC20:20
thingeejgriffith ^20:20
ameadegeguileo: https://review.openstack.org/#/c/157439/20:20
ameadethanks for the revs20:20
geguileoameade: np20:21
geguileoameade: Thank you for the nice feature  :)20:22
geguileoameade: Patch lgtm now20:22
*** vokt has joined #openstack-cinder20:23
ameadecool20:24
*** Longgeek has quit IRC20:24
*** jungleboyj has quit IRC20:25
*** daneyon has quit IRC20:26
*** rushil has quit IRC20:26
hemnaseems you can't use tox to run specific tests now, since the switch to ostestr20:27
hemna:(20:27
*** rushil has joined #openstack-cinder20:28
eharneyhemna: you can in a more complicated way20:28
hemna:(20:28
*** ganso_ has quit IRC20:28
hemnamakes tox useless now basically20:28
eharneyhemna: tox -e py27 -- --regex somethingsomethingtest     worked for me20:28
hemnaugh20:28
eharneybut yes, definitely a downgrade20:28
hemnaman that completely sucks20:29
eharneyi'd have to agree20:29
smcginnisCan still do ./run_tests.sh somethingsomethingtest20:29
hemnasmcginnis, I don't like run_tests.sh.   jenkins jobs and the gate use tox20:30
hemnaso I try and use what the gate uses.20:30
eharneysmcginnis: i think so20:30
hemnanot sure why we moved to ostestr20:30
hemnanova doesn't use it.20:30
smcginnishemna: I was under the impression that was the general direction all projects where going.20:31
hemnaugh, then this has to get fixed, because this is unusable20:31
smcginnisIIRC, jgriffith was the one that switched it. Could be wrong.20:31
eharneyit was related to running tests concurrently maybe, but i'm not sure if it was actually required for that or not20:31
hemnasmcginnis, yah just noticed that in the log :(20:31
hemnamtreinish, ping20:32
hemnahttp://paste.openstack.org/show/352938/20:33
hemnapoop20:33
hemnathat used to just work.20:33
e0nehemna: i use ./run_tests.sh to run specified test20:35
hemnayah I know some folks use run_tests.sh20:35
hemnabut the gate and jenkins doesn't use run_tests.sh20:35
hemnait uses tox20:35
e0nehemna: it's good point to remove run_tests.sh20:36
eharneyi don't think the folks using run_tests.sh -N would appreciate that20:37
smcginnisI use run_tests.sh too. But I could see an argument for consistency to make sure everyone is running the same thing.20:37
hemnanot sure why ostestr can't use the args as a default --regex entry20:37
hemnathen that'd just work20:37
smcginnisMaybe everyone should just use run_tests.sh. ;)20:37
eharneyif tests are failing or passing differently between tox and run_tests then we have bigger problems20:37
hemnasmcginnis, well, if the gate used run_tests.sh then sure.20:37
smcginniseharney: +120:37
eharneyso i think we should just leave good enough alone on that front20:37
hemnabut I think we should all use what jenkins and the gate uses.20:38
smcginnishemna: It's ultimately the same (so far) so should it matter?20:38
jgriffithsmcginnis: what's up?20:38
mtreinishhemna: pong20:38
eharneyhemna: i don't think that at all for my dev env20:38
smcginnisjgriffith: Just throwing you under the bus.20:38
smcginnis:)20:38
hemnahehe20:38
hemnamtreinish, so, ostestr20:38
jgriffithsmcginnis: hemna what's new?20:38
smcginnisjgriffith: Questions came up on the switch to ostestr.20:38
hemnaI'm finding the switch to ostestr to be a PITA20:38
jgriffithlittle slow/behind20:38
jgriffithhemna: why?20:38
smcginnisjgriffith: Just pointed out I thought you did it and may have more context.20:38
hemnaas tox is now not working as it used to.20:38
hemnaand not like every other project20:38
*** coolsvap is now known as coolsvap|away20:39
jgriffithhemna: should work exactly the same20:39
hemnait doesn't20:39
jgriffithtox -epy2720:39
hemnahttp://paste.openstack.org/show/352938/20:39
hemnathat's worked forever20:39
hemnauntil the switch to ostestr20:39
mtreinishhemna: just run tox -epy27 -- --regex cinder.tests.unit.test_volume.VolumeTestCase.test_create_delete_consistencygroup20:39
hemnaugh20:39
hemnathat's aweful20:39
hemnaseriously awful20:39
jgriffithhemna: why?20:40
mtreinishhemna: or to do it without discovery do tox -epy27 -- -n cinder.tests.unit.test_volume.VolumeTestCase.test_create_delete_consistencygroup20:40
smcginnishemna: Write a script to wrap it for you. Call it run_hemnas_tests.sh. :P20:40
mtreinishwhich will be what you really want it to do20:40
hemnamtreinish, because it doesn't used to be that bad20:40
jgriffithsmcginnis: or propose a revert of the change20:40
hemnamtreinish, it should work like it used to20:40
jgriffitherr... ^^ hemna20:40
hemna^^ paste url20:40
hemnabah20:40
smcginnisjgriffith: I seem to remember this was the direction other projects were going. Is that true?20:40
jgriffithsmcginnis: yes20:40
hemnaso in order to fix this, we have to come up with a hacked .sh script ?20:40
hemnawow20:40
jgriffithhemna: no, you just need to run a slightly modified command20:41
mtreinishhemna: or propose an ostestr change, it's not like the cli is done yet20:41
jgriffithor use run_tests.sh if you want20:41
jgriffithmtreinish: yeah... or that :)20:41
jgriffitheven better20:41
hemnawhy can't ostestr just work w/o that extra needed terseness ?20:41
*** jamielennox is now known as jamielennox|away20:41
jgriffithhemna: because it needs your patch to make it do so :)20:41
mtreinishhemna: because the selection logic and grammar isn't done yet so it's explicit20:41
mtreinishthat was a choice I made, because well I'm the only one who's really worked on any of this20:42
hemnadefault it to ostestr <regex>20:42
hemna:)20:42
hemnathat should fix it IMHO20:42
smcginnishttps://bugs.launchpad.net/os-testr20:42
hemna:)20:42
smcginnis:)20:42
mtreinishhemna: that has bigger implications when you have more involved selection logic then just a single test20:42
mtreinishwhich is one of the big complaints about testr's cli20:43
*** e0ne has quit IRC20:44
jdandreaQ: Is it possible to get cinder-volume to work with a remote OpenStack installation in a manner similar to Heat's Standalone Mode? I ask because this comes in handy for development of Heat in conjunction with an existing cluster, and without adversely impacting users of that cluster (e.g., testing and tracing scheduler filter plugins, for instance). Context: http://docs.openstack.org/developer/heat/getting_started/standalone.html20:44
jgriffithhemna: I'm still not following why adding this "-- --regex" is such a hardship?20:45
hemnabecause I shouldn't have to do that just to run a single test.   just like it's always been.20:45
hemnafor every project that uses tox20:46
mtreinishjgriffith: or add a -n because that's really what he's trying to do. The regex is kinda heavyweight for a single test it'll do a full discovery and then match against each test20:46
jgriffithhemna: hmm... that seems like an odd objection20:46
*** erlon has joined #openstack-cinder20:46
jgriffithmtreinish: yeah, for me I just use what works and move on, but maybe -n is better in this case20:46
hemnareally?  you want to remember which project requires -- --regex to run tox and which one doesn't ?20:46
jgriffithmtreinish: I can change20:46
hemnasmh20:46
jgriffithhemna: perhaps you should just use run_tests.sh20:47
jgriffithhemna: until I remove it that is :)20:47
hemnaanyway, I'm filing a bug and moving on.20:47
hemnaI don't use run_tests.sh20:47
jgriffithhemna: I'll mark it as invalid FYI :)20:47
hemnabecause it's not used by jenkins, nor is it used by the gate20:47
mtreinishjgriffith: well that's why I added the opt, to fast track it because people with really large test suites which are expensive to import every file were complaining the discovery takes too long20:47
hemnagreat20:47
jgriffithhemna: ahh.. but it used to be20:47
jgriffithhemna: they changed20:47
hemnanot sure why I even bother then20:47
hemnawhatever20:47
jgriffithhemna: sorry... I was kinda kidding20:48
jgriffithhemna: don't get all upset20:48
jgriffithhemna: file a bug and propose a revert if you like20:48
jgriffithhemna: that's perfectly acceptable20:48
jgriffithhemna: or I suppose we could try and speed up the changes to the other projects to get the consistency you're looking for?20:50
*** jasondotstar has joined #openstack-cinder20:50
*** annasort has quit IRC20:51
*** jamielennox|away is now known as jamielennox20:52
*** jwcroppe has quit IRC20:53
*** akerr has quit IRC20:54
*** jskinner has quit IRC20:54
eharneyjgriffith: will reverting that revert the change to test things in parallel?20:54
*** bswartz has quit IRC20:54
jgriffitheharney: yes20:54
hemnaI'm fully ok with using new tools, we should just figure out how not to break the tox usage20:54
eharneywhatever we do, let's not make it easy to introduce concurrency bugs again20:55
jgriffitheharney: +120:55
jgriffitheharney: FTR I'm ok with just adding "-- -n"20:55
jgriffitheharney: mtreinish I've already changed from "-- -regex"20:55
mtreinishjgriffith: cool, I'm not sure what the discovery overhead is for cinder's tests20:58
mtreinishbut like on nova it takes a while to import everything in the tests dir20:58
jgriffithmtreinish: suppose I could nuke my discovery and try it out20:59
mtreinishbut making it faster even marginally so can't hurt :)20:59
mtreinishjgriffith: well that's what -n does20:59
jgriffithmtreinish: faster is always better IMHO20:59
jgriffithmtreinish: oh... it nukes it for me ?20:59
jgriffithmtreinish: that's cool then, no perceptable diff that I can see21:00
mtreinishjgriffith: yep it calls directly to subunit.run with the test_id you passed in21:00
mtreinishand runs subunit-trace the output21:00
*** jdurgin2 has joined #openstack-cinder21:01
*** rushil has quit IRC21:01
*** mutoulbj has quit IRC21:01
*** mutoulbj has joined #openstack-cinder21:01
*** mutoulbj has quit IRC21:01
*** jdurgin has quit IRC21:02
openstackgerritRaunak Kumar proposed openstack/cinder: Add extra spec capability for Nimble Cinder Driver  https://review.openstack.org/19885321:03
*** jwcroppe has joined #openstack-cinder21:04
*** julim_ has quit IRC21:04
*** dustins has quit IRC21:07
*** jwcroppe_ has joined #openstack-cinder21:07
*** jungleboyj has joined #openstack-cinder21:09
*** jwcroppe has quit IRC21:09
*** daneyon has joined #openstack-cinder21:10
*** dims has joined #openstack-cinder21:13
patrickeastis it a safe assumption that all volume drivers can extend volumes?21:13
*** ociuhandu has quit IRC21:13
*** delatte has quit IRC21:14
hemnapatrickeast, don't think so :)21:14
patrickeastlame21:15
patrickeasthemna: any idea how to detect that?21:15
hemnahttps://wiki.openstack.org/wiki/CinderSupportMatrix21:15
hemnathat's a source...fwiw21:15
jgriffithpatrickeast: cd cinder/volume/drivers; grep -r -A4 "def extend"21:16
patrickeastmmm21:16
hemnalooks like a lot do21:16
*** jwcroppe has joined #openstack-cinder21:16
patrickeastwho cares about windows server?21:16
patrickeast>.>21:16
*** jasondotstar has quit IRC21:16
hemnaI don't think it's a required driver feature though21:16
patrickeastyea i see it all alone in  a abc class21:16
* jgriffith sure wishes we didn't start down the "optional" features path21:16
hemnajgriffith, +121:17
patrickeastjgriffith: so i ask, because i realized the approach i switched to for the cache wasn’t creating minimal sized volumes for the cache21:17
jgriffithpatrickeast: hemna wasn't the plan for that to just give drivers a release or two to "catch up" or something like that21:17
eharneyif we didn't, we would have deleted almost every driver by now21:17
patrickeastwhat i would like to do is always create a minimal size one and extend21:17
eharneythat seems bad21:17
hemnawasn't one of the 'features' of the ABC stuff was to be able to inspect the driver's ability of a supported feature?  (ick)21:17
jgriffitheharney: very true21:17
*** jwcroppe_ has quit IRC21:17
jgriffithhemna: so all abc buys you is that you have to "implement" something21:18
jgriffithhemna: that implementation can still be "pass"21:18
hemnableh yah that is true.21:18
jgriffithhemna: it really does nothing IMHO21:18
*** annegentle has quit IRC21:18
hemna:(21:18
hemnait's done a great job of confusing the driver class hierarchy21:18
jgriffitheharney: yeah, I know there's some logic behind it, but like many things we never really followed through21:18
jgriffithhemna: yeah, that was what I mentioned in a review a while back and it was taken kindly21:19
jgriffithhemna: I've even offered up alternate ways to accomplish the check21:19
openstackgerritPatrick East proposed openstack/cinder: WIP generic image cache  https://review.openstack.org/19579521:19
jgriffithbut alas, such is life21:19
eharneyi thought it detected missing bits at import time rather than call time21:20
jgriffitheharney: yes it does, that's the nice thing21:20
jgriffitheharney: but the impl doesn't actually have to have anything in it21:20
jgriffitheharney: and IIRC I found a couple cases where that's just what folks did21:20
jgriffitheharney: they just wrote the method with "pass" or "raise" which completey satisfies abc meta21:21
jgriffitheharney: but still doesn't do anything21:21
patrickeastso if i want to do something like this https://review.openstack.org/#/c/195795/8..9/cinder/volume/flows/manager/create_volume.py where it uses a potentially not-implemented feature of a driver21:21
eharneythen why does it use that VD base class...21:21
patrickeasthow can i protect against it blowing up?21:21
jgriffitheharney: not sure I follow... we might be on different pages21:21
patrickeastdo we need to just call the driver method and see if an exception is raised?21:21
*** salv-orlando has joined #openstack-cinder21:22
patrickeastor can i just not do that21:22
jgriffitheharney: what I was talking about is that you can put something in the base VD or wherever, which means the driver MUST have "def foo(x, y, z)"21:22
jgriffitheharney: or it will barf at load just like you said21:22
eharneyah yeah, ok21:22
jgriffitheharney: but... in the driver it could be that "def foo(x, y, z): pass"21:22
jgriffithor whatever21:23
jgriffithdoesn't mean it actually impelements/supports it21:23
*** merooney has joined #openstack-cinder21:23
jgriffithpatrickeast: we used to be able to do that21:23
jgriffithpatrickeast: if extend isn't in the enforced abc class you can still do that21:23
jgriffithpatrickeast: by "enforced abc class" I mean the "required/mandatory base class"21:24
patrickeastjgriffith: hmm ok21:24
patrickeastextend isn’t21:24
jgriffithpatrickeast: and it assumes all the drivers have implemented the inheritance of the abc metas (which they don't)21:24
patrickeastso i guess we can't21:24
jgriffithpatrickeast: so you can do the trick you just mentioned, try/catch21:24
jgriffithpatrickeast: it actually works to your advantage in this case that it isn't in that class21:25
jgriffithwell... depending on your perspective I guess21:25
patrickeasthaha yea21:25
patrickeastalright i’ll play around with some try/catches and stuff21:25
jgriffithpatrickeast: all the reasons I punted on the generic impl are slowly coming back to me :)21:28
patrickeastlol21:28
jgriffithpatrickeast: you've made it MUCH further than I did21:28
patrickeastit is definitely harder than i had first thought21:28
jgriffithpatrickeast: mucho respect21:28
patrickeasthopefully we can get something that works for everyone*21:28
patrickeast*=most people21:29
*** eharney has quit IRC21:29
*** salv-orlando has quit IRC21:29
*** salv-orlando has joined #openstack-cinder21:30
*** kjnelson has quit IRC21:30
*** dims has quit IRC21:30
patrickeastmaybe as a safety net i’ll put the extend stuff behind the cache enable flag, that way if a driver doesn’t implement extend they can just not use the cache and continue on as normal21:31
jgriffithpatrickeast: might be a descent solution21:31
*** dims has joined #openstack-cinder21:35
*** dims has quit IRC21:35
*** Guest7393 has joined #openstack-cinder21:35
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Fix missing pool name in consistency group  https://review.openstack.org/19931421:36
*** mriedem is now known as mriedem_away21:37
*** crose has quit IRC21:42
*** crose has joined #openstack-cinder21:42
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Fix missing pool name in consistency group  https://review.openstack.org/19931421:43
*** jungleboyjiPad has quit IRC21:46
*** merooney has quit IRC21:47
*** sgotliv has quit IRC21:48
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Fix missing pool name in consistency group  https://review.openstack.org/19931421:51
*** Longgeek has joined #openstack-cinder21:51
*** lpetrut has quit IRC21:54
hemna*sigh*21:54
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Fix missing pool name in consistency group  https://review.openstack.org/19931421:54
hemnathe spaminator today21:54
*** mgarza_ has quit IRC21:57
*** annegentle has joined #openstack-cinder21:57
*** Longgeek has quit IRC21:57
*** changbl has quit IRC21:59
*** mgarza has joined #openstack-cinder21:59
openstackgerritxing-yang proposed openstack/cinder: Non-disruptive backup  https://review.openstack.org/19393722:00
*** akerr has joined #openstack-cinder22:03
*** jwcroppe_ has joined #openstack-cinder22:04
*** jwcroppe has quit IRC22:06
openstackgerritxing-yang proposed openstack/cinder: Non-disruptive backup  https://review.openstack.org/19393722:07
*** akerr has quit IRC22:08
*** jungleboyj has quit IRC22:10
*** simondodsley has quit IRC22:18
*** jwcroppe_ has quit IRC22:22
*** jwcroppe has joined #openstack-cinder22:23
*** Apoorva has quit IRC22:26
*** jdurgin2 has quit IRC22:27
*** jdurgin has joined #openstack-cinder22:27
*** jwcroppe has quit IRC22:28
*** rajinir has joined #openstack-cinder22:28
*** annegentle has quit IRC22:29
openstackgerritJohn Griffith proposed openstack/cinder: Handle volume not found on zfssa volume delete  https://review.openstack.org/19933022:30
*** edmondsw has quit IRC22:33
*** daneyon has quit IRC22:34
*** angela-s has quit IRC22:36
*** rajinir has quit IRC22:37
*** lcurtis has quit IRC22:41
*** jwcroppe has joined #openstack-cinder22:44
*** andymccr has joined #openstack-cinder22:46
*** chlong has quit IRC22:46
*** IlyaG has joined #openstack-cinder22:49
*** IlyaG has quit IRC22:49
*** annashen has quit IRC22:54
*** jwcroppe has quit IRC22:56
*** Rockyg has quit IRC22:57
*** merooney has joined #openstack-cinder22:58
*** merooney has quit IRC22:58
*** merooney has joined #openstack-cinder22:59
*** r-daneel has quit IRC23:00
*** andymccr has quit IRC23:09
*** andymccr has joined #openstack-cinder23:10
*** Yogi1 has quit IRC23:11
*** annegentle has joined #openstack-cinder23:12
*** zhenguo has joined #openstack-cinder23:14
openstackgerritRyan Lucio proposed openstack/cinder: VMEM v7000: external-head storage pool and volume type selection  https://review.openstack.org/19577323:14
*** hemna is now known as hemnafk23:16
*** jungleboyj has joined #openstack-cinder23:17
*** akshai has quit IRC23:18
*** andymccr has quit IRC23:18
*** gouthamr has quit IRC23:19
*** bswartz has joined #openstack-cinder23:19
*** yamada-h has joined #openstack-cinder23:26
*** salv-orlando has quit IRC23:26
*** Apoorva has joined #openstack-cinder23:27
*** merooney has quit IRC23:28
*** krtaylor has quit IRC23:29
*** diogogmt has quit IRC23:30
*** Apoorva has quit IRC23:31
*** annegentle has quit IRC23:34
*** jungleboyj has quit IRC23:34
thingeejgriffith: I think mkoderer was open to the idea of reducing classes in reply to your abc thread http://lists.openstack.org/pipermail/openstack-dev/2015-June/067683.html23:35
thingeejgriffith: just nothing has happened past that discussion23:35
*** diogogmt has joined #openstack-cinder23:35
thingeejgriffith: I replied to thread too23:37
thingeekmartin_: ^23:37
*** krtaylor has joined #openstack-cinder23:42
*** Longgeek has joined #openstack-cinder23:44
*** yamada-h has quit IRC23:46
*** Longgeek has quit IRC23:49
*** erlon has quit IRC23:50
*** jasondotstar has joined #openstack-cinder23:51
*** IanGovett has quit IRC23:54
*** jwcroppe has joined #openstack-cinder23:57
*** laughterwym has joined #openstack-cinder23:58
*** dalgaaf has quit IRC23:58
*** vincent_hou_ has joined #openstack-cinder23:58

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!