Tuesday, 2017-07-25

*** chhavi has joined #openstack-cinder00:06
*** chhavi has quit IRC00:10
*** jwcroppe has joined #openstack-cinder00:14
openstackgerritChuck Fouts proposed openstack/cinder master: NetApp cDOT: Support generic groups for file  https://review.openstack.org/48613600:16
*** sdague has quit IRC00:21
*** wanghao has joined #openstack-cinder00:31
*** zhangguoqing has joined #openstack-cinder00:31
*** jwcroppe has quit IRC00:32
openstackgerritMerged openstack/python-cinderclient master: Add cinder create --poll  https://review.openstack.org/47591900:33
*** mtanino has quit IRC00:35
*** stakeda has joined #openstack-cinder00:39
*** itlinux has joined #openstack-cinder00:45
*** wanghao has quit IRC00:46
*** catintheroof has joined #openstack-cinder00:47
*** diablo_rojo has quit IRC00:53
*** wanghao has joined #openstack-cinder00:56
openstackgerritMerged openstack/cinder master: Fix test_scheduler_stats error module calls  https://review.openstack.org/48514101:04
*** Apoorva_ has joined #openstack-cinder01:13
*** harlowja has quit IRC01:13
*** wanghao has quit IRC01:14
*** Apoorva has quit IRC01:16
*** Apoorva_ has quit IRC01:17
*** catintheroof has quit IRC01:18
*** wanghao has joined #openstack-cinder01:27
*** fanzhang has joined #openstack-cinder01:37
*** chlong_ has joined #openstack-cinder01:45
*** chlong__ has quit IRC01:47
smcginnisAny cores around, there are still a couple unsupported driver patches out there.01:48
smcginnisI think it would be good to get those through before the milestone 3 cut.01:48
smcginnishttps://review.openstack.org/#/q/status:open+project:openstack/cinder+branch:master+topic:ci_unsupported01:48
*** Shunli has joined #openstack-cinder01:49
*** crushil has joined #openstack-cinder01:50
*** csaikia has quit IRC01:57
*** dave-mccowan has joined #openstack-cinder01:58
*** bkopilov_ has quit IRC01:59
*** bkopilov has quit IRC01:59
*** itlinux has quit IRC02:00
*** itlinux has joined #openstack-cinder02:00
*** _ix has quit IRC02:01
*** csaikia has joined #openstack-cinder02:04
*** wanghao_ has joined #openstack-cinder02:05
*** chlong__ has joined #openstack-cinder02:07
*** jwcroppe has joined #openstack-cinder02:07
*** _ix has joined #openstack-cinder02:07
*** chlong_ has quit IRC02:07
*** dave-mccowan has quit IRC02:07
*** wanghao has quit IRC02:08
*** jwcroppe has quit IRC02:12
*** _ix has quit IRC02:14
*** chlong_ has joined #openstack-cinder02:25
*** chlong__ has quit IRC02:27
openstackgerritMerged openstack/python-cinderclient master: Updated from global requirements  https://review.openstack.org/48633102:34
*** gouthamr has quit IRC02:46
*** jwcroppe has joined #openstack-cinder02:57
*** lbragsta_ has joined #openstack-cinder03:03
openstackgerritMerged openstack/cinder master: Add VNX replication group support  https://review.openstack.org/45920503:07
openstackgerritMerged openstack/cinder master: [DOC BLD FIX] Correct docstring errors in falconstor driver  https://review.openstack.org/48671503:12
openstackgerritMerged openstack/cinder master: [DOC BLD FIX] Fix docstring issues in dothill driver  https://review.openstack.org/48668803:12
openstackgerritJay Bryant proposed openstack/cinder master: [DOC BLD FIX] Fix docstring issues in ibm_storage  https://review.openstack.org/48683803:13
*** bkopilov_ has joined #openstack-cinder03:15
*** bkopilov has joined #openstack-cinder03:15
vivsoniMay i request you to please review https://review.openstack.org/#/c/477888/03:19
*** randomhack has joined #openstack-cinder03:19
*** randomhack has quit IRC03:26
*** chlong_ has quit IRC03:28
*** chlong__ has joined #openstack-cinder03:28
*** wanghao_ has quit IRC03:34
*** wanghao has joined #openstack-cinder03:35
openstackgerritMerged openstack/cinder master: [DOC BLD FIX] Correct dosctring errors in disco driver  https://review.openstack.org/48667503:37
*** diablo_rojo has joined #openstack-cinder03:45
*** lbragsta_ has quit IRC03:48
*** gkadam has joined #openstack-cinder03:50
*** armax has quit IRC03:52
*** chlong__ has quit IRC04:01
*** chlong_ has joined #openstack-cinder04:01
*** chhavi has joined #openstack-cinder04:09
*** nicolasbock has joined #openstack-cinder04:13
*** markvoelker has quit IRC04:14
*** chlong__ has joined #openstack-cinder04:25
*** chlong_ has quit IRC04:27
*** links has joined #openstack-cinder04:30
*** dikonoor has joined #openstack-cinder04:33
*** fanzhang_ has joined #openstack-cinder04:46
*** fanzhang has quit IRC04:50
*** cknight has quit IRC05:02
openstackgerritVivek Soni proposed openstack/cinder master: Volume type with encryption comparison  https://review.openstack.org/47788805:08
*** dikonoor has quit IRC05:16
*** Julien-zte has quit IRC05:19
*** Julien-zte has joined #openstack-cinder05:20
openstackgerritKendall Nelson proposed openstack/cinder master: Mock execute in unit test  https://review.openstack.org/48687005:21
*** pgadiya has joined #openstack-cinder05:22
*** jwcroppe has quit IRC05:30
*** josecastroleon has joined #openstack-cinder05:30
*** josecastroleon1 has quit IRC05:30
*** itlinux has quit IRC05:31
*** dikonoor has joined #openstack-cinder05:31
*** chlong_ has joined #openstack-cinder05:33
*** josecastroleon has quit IRC05:34
*** chlong__ has quit IRC05:34
*** psachin has joined #openstack-cinder05:36
*** randomhack has joined #openstack-cinder05:38
*** csaikia has quit IRC05:41
*** randomhack has quit IRC05:42
*** wanghao has quit IRC05:42
*** wanghao has joined #openstack-cinder05:42
*** josecastroleon has joined #openstack-cinder05:51
*** wanghao_ has joined #openstack-cinder05:59
*** wanghao_ has quit IRC06:00
*** wanghao has quit IRC06:01
*** wanghao has joined #openstack-cinder06:01
*** rcernin has joined #openstack-cinder06:10
*** markvoelker has joined #openstack-cinder06:15
*** wanghao_ has joined #openstack-cinder06:20
*** wanghao has quit IRC06:24
*** bkopilov has quit IRC06:24
*** belmoreira has joined #openstack-cinder06:43
openstackgerritwangxiyuan proposed openstack/cinder master: Add input format check for qos_spec update  https://review.openstack.org/48471806:46
openstackgerritRawan Herzallah proposed openstack/cinder master: Adding NVMEoF target for LVM driver  https://review.openstack.org/48256206:47
*** markvoelker has quit IRC06:49
openstackgerritRawan Herzallah proposed openstack/os-brick master: Adding NVMEoF for initiator CLI  https://review.openstack.org/48264206:49
openstackgerritRawan Herzallah proposed openstack/os-brick master: Adding NVMEoF for initiator CLI  https://review.openstack.org/48264206:53
openstackgerritRawan Herzallah proposed openstack/os-brick master: Adding NVMEoF for initiator CLI  https://review.openstack.org/48264206:54
*** ChubYann has quit IRC06:56
*** danpawlik has joined #openstack-cinder06:58
*** josecastroleon has quit IRC06:58
*** Poornima_K has joined #openstack-cinder06:59
*** josecastroleon has joined #openstack-cinder07:06
openstackgerritSean McGinnis proposed openstack/cinder-specs master: Revert "Add API to query backup capabilities"  https://review.openstack.org/48692107:24
*** diablo_rojo has quit IRC07:27
*** Poornima_K has quit IRC07:30
*** gkadam is now known as gkadam-afk07:31
smcginnisThis patch still needs to land to complete a Pike targeted spec: https://review.openstack.org/#/c/471541/07:32
*** isaacb has joined #openstack-cinder07:38
*** e0ne has joined #openstack-cinder07:38
*** fanzhang__ has joined #openstack-cinder07:38
*** zhangguoqing has quit IRC07:39
*** fanzhang__ has quit IRC07:41
*** fanzhang_ has quit IRC07:41
*** josecastroleon has quit IRC07:45
*** markvoelker has joined #openstack-cinder07:46
*** zhangguoqing has joined #openstack-cinder07:49
openstackgerritMerged openstack/cinder master: [DOC BLD FIX] Fix docstring issues in ibm_storage  https://review.openstack.org/48683807:57
openstackgerritMerged openstack/cinder master: [DOC BLD FIX] Fix docstring errors in Dell/EMC Unity  https://review.openstack.org/48544907:59
*** aarefiev_afk is now known as aarefiev08:05
*** josecastroleon has joined #openstack-cinder08:07
*** josecastroleon has quit IRC08:13
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Add cinder backup service initialize check  https://review.openstack.org/44651808:19
*** randomhack has joined #openstack-cinder08:19
*** markvoelker has quit IRC08:20
*** randomhack has quit IRC08:23
*** josecastroleon has joined #openstack-cinder08:24
*** johnhaan has joined #openstack-cinder08:26
*** josecastroleon has quit IRC08:29
arnewiebalckDoes Cinder support syncing quotas (like “nova-manage project quota_usage_refresh” in nova)? From what I see, it does not, but maybe I’m not looking at the correct places …08:35
openstackgerritwangxiyuan proposed openstack/cinder master: Add input format check for qos_spec update  https://review.openstack.org/48471808:36
*** _pewp_ has quit IRC08:40
*** _pewp_ has joined #openstack-cinder08:44
*** gkadam-afk is now known as gkadam08:46
*** wanghao has joined #openstack-cinder08:48
*** wanghao_ has quit IRC08:51
openstackgerritMerged openstack/cinder master: Don't verify format with autodetect after image conversion  https://review.openstack.org/48301209:12
*** markvoelker has joined #openstack-cinder09:17
*** lkuchlan has joined #openstack-cinder09:24
*** mvk has quit IRC09:25
*** Shunli has quit IRC09:29
*** chlong__ has joined #openstack-cinder09:33
*** wanghao has quit IRC09:33
*** chlong_ has quit IRC09:35
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Add cinder backup service initialize check  https://review.openstack.org/44651809:42
*** _pewp_ has quit IRC09:48
*** markvoelker has quit IRC09:49
*** _pewp_ has joined #openstack-cinder09:51
*** mvk has joined #openstack-cinder09:54
*** zhangguoqing has quit IRC09:58
*** sdague has joined #openstack-cinder09:59
openstackgerritMerged openstack/python-brick-cinderclient-ext master: Update URLs in documentation  https://review.openstack.org/48546809:59
*** stakeda has quit IRC10:00
*** openstackgerrit has quit IRC10:17
*** Poornima_K has joined #openstack-cinder10:30
*** dikonoor has quit IRC10:33
*** openstackgerrit has joined #openstack-cinder10:39
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Add cinder backup service initialize check  https://review.openstack.org/44651810:39
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Add cinder backup service initialize check  https://review.openstack.org/44651810:45
*** markvoelker has joined #openstack-cinder10:46
*** vivsoni has quit IRC10:48
*** chlong_ has joined #openstack-cinder10:52
*** dikonoor has joined #openstack-cinder10:52
*** Poornima_K has quit IRC10:52
*** vivsoni has joined #openstack-cinder10:53
*** chlong__ has quit IRC10:53
*** bkopilov_ has quit IRC10:55
*** chlong__ has joined #openstack-cinder10:57
*** zhangguoqing has joined #openstack-cinder10:57
*** chlong_ has quit IRC10:59
*** dikonoor has quit IRC10:59
*** Poornima_K has joined #openstack-cinder11:00
*** dikonoor has joined #openstack-cinder11:03
*** dinesh_ has joined #openstack-cinder11:06
*** dinesh_ has left #openstack-cinder11:06
*** rawanh has joined #openstack-cinder11:10
*** _ix has joined #openstack-cinder11:15
*** dikonoor has quit IRC11:15
*** _ix has quit IRC11:19
*** _ix has joined #openstack-cinder11:20
*** markvoelker has quit IRC11:20
openstackgerritRawan Herzallah proposed openstack/os-brick master: Adding NVMEoF for initiator CLI  https://review.openstack.org/48264211:23
openstackgerritRawan Herzallah proposed openstack/os-brick master: Adding NVMEoF for initiator CLI  https://review.openstack.org/48264211:25
*** Poornima_K has quit IRC11:28
*** dikonoor has joined #openstack-cinder11:29
*** isaacb has quit IRC11:34
*** _ix has quit IRC11:42
*** _ix has joined #openstack-cinder11:42
*** chlong__ has quit IRC11:45
*** danpawlik has quit IRC11:45
*** chlong_ has joined #openstack-cinder11:45
*** abishop has joined #openstack-cinder11:46
openstackgerritTzur Eliyahu proposed openstack/cinder master: Ibm_storage - fix delete group with replication  https://review.openstack.org/48701811:46
*** danpawlik has joined #openstack-cinder11:47
*** isaacb has joined #openstack-cinder11:48
openstackgerritNidhiMittalHada proposed openstack/python-cinderclient master: Unreadable output of upload-to-image.  https://review.openstack.org/48702011:49
*** lbragsta_ has joined #openstack-cinder11:51
*** _ix has quit IRC11:53
*** chlong_ has quit IRC11:58
*** dikonoo has joined #openstack-cinder11:59
*** dikonoor has quit IRC12:02
openstackgerritMerged openstack/cinder master: LVM: Activate LV before creating exports  https://review.openstack.org/48525612:03
*** Julien-zte has quit IRC12:10
*** Julien-zte has joined #openstack-cinder12:12
*** lbragsta_ has quit IRC12:12
*** zhangguo_ has joined #openstack-cinder12:16
*** markvoelker has joined #openstack-cinder12:17
*** zhangguoqing has quit IRC12:17
openstackgerritMerged openstack/cinder master: Api-ref: add documentation for v2/v3 backup 'os-reset_status' action  https://review.openstack.org/47742912:18
openstackgerritMerged openstack/cinder master: [Api-ref] Update quota-set APIs ref  https://review.openstack.org/47404812:19
*** zhangguo_ has quit IRC12:22
*** jwcroppe has joined #openstack-cinder12:25
*** gouthamr has joined #openstack-cinder12:26
*** markvoelker has quit IRC12:26
*** markvoelker has joined #openstack-cinder12:26
*** links has quit IRC12:27
*** MVenesio has joined #openstack-cinder12:28
*** jwcroppe has quit IRC12:30
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Add cinder backup service initialize check  https://review.openstack.org/44651812:31
*** dustins has joined #openstack-cinder12:33
*** vivsoni has quit IRC12:35
*** edmondsw has joined #openstack-cinder12:36
*** catintheroof has joined #openstack-cinder12:39
*** catintheroof has quit IRC12:39
*** catintheroof has joined #openstack-cinder12:39
*** links has joined #openstack-cinder12:44
*** pgadiya has quit IRC12:47
*** isaacb has quit IRC12:50
openstackgerritGorka Eguileor proposed openstack/python-cinderclient master: Add service cleanup command  https://review.openstack.org/36300712:55
openstackgerritGorka Eguileor proposed openstack/python-cinderclient master: Add cluster support in migration and manage  https://review.openstack.org/38750012:55
openstackgerritGorka Eguileor proposed openstack/python-cinderclient master: Add cluster support in manage listings  https://review.openstack.org/38750112:55
*** isaacb has joined #openstack-cinder12:56
*** eharney has joined #openstack-cinder12:58
*** felipemonteiro_ has joined #openstack-cinder12:59
*** jwcroppe has joined #openstack-cinder12:59
*** gkadam has quit IRC13:02
*** lucasxu has joined #openstack-cinder13:05
*** gnufied has joined #openstack-cinder13:08
*** randomhack has joined #openstack-cinder13:17
*** dikonoo has quit IRC13:23
*** breton has joined #openstack-cinder13:27
*** csaikia has joined #openstack-cinder13:28
*** chaynikasaikia has joined #openstack-cinder13:30
*** sticker_ has joined #openstack-cinder13:32
*** csaikia has quit IRC13:33
arnewiebalckDoes Cinder support syncing/refreshing quotas (like “nova-manage project quota_usage_refresh” in nova)? From what I see, it does not, but maybe I’m not looking at the correct place …13:34
*** sticker has quit IRC13:35
*** crushil has quit IRC13:35
*** crushil has joined #openstack-cinder13:35
*** chlong has joined #openstack-cinder13:37
*** bkopilov_ has joined #openstack-cinder13:42
*** armax has joined #openstack-cinder13:48
*** crushil has quit IRC13:49
*** e0ne has quit IRC13:50
*** e0ne has joined #openstack-cinder13:50
*** e0ne has quit IRC13:51
*** e0ne has joined #openstack-cinder13:51
*** e0ne has quit IRC13:52
*** e0ne has joined #openstack-cinder13:52
*** cknight has joined #openstack-cinder13:52
*** e0ne has quit IRC13:52
*** e0ne has joined #openstack-cinder13:53
*** e0ne has quit IRC13:53
*** e0ne has joined #openstack-cinder13:54
*** e0ne has quit IRC13:54
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Add cinder backup service initialize check  https://review.openstack.org/44651813:56
*** crushil has joined #openstack-cinder13:59
*** chlong has quit IRC14:03
*** rawanh has quit IRC14:04
openstackgerritTzur Eliyahu proposed openstack/cinder master: ibm_storage - fix enable replication after disable  https://review.openstack.org/48706314:06
*** Trident has quit IRC14:14
*** Trident has joined #openstack-cinder14:14
*** mtanino has joined #openstack-cinder14:24
*** kwathore has quit IRC14:24
*** kwathore_ has joined #openstack-cinder14:24
*** links has quit IRC14:26
*** eharney has quit IRC14:32
*** lucasxu has quit IRC14:38
*** psachin has quit IRC14:40
*** mgiles1 has joined #openstack-cinder14:42
*** eharney has joined #openstack-cinder14:47
*** isaacb has quit IRC14:52
*** dulek_mobile has joined #openstack-cinder14:55
*** dulek_mobile has quit IRC14:57
*** dulek_mobile has joined #openstack-cinder14:57
*** eharney has quit IRC15:03
*** lkuchlan has quit IRC15:07
*** lifeless has quit IRC15:09
*** chlong has joined #openstack-cinder15:16
*** rcernin has quit IRC15:17
*** eharney has joined #openstack-cinder15:19
*** Julien-zte has quit IRC15:19
*** lifeless has joined #openstack-cinder15:22
*** lifeless has quit IRC15:29
*** felipemonteiro__ has joined #openstack-cinder15:31
*** firebat has joined #openstack-cinder15:33
*** felipemonteiro_ has quit IRC15:34
*** jwcroppe has quit IRC15:34
*** dustins has quit IRC15:35
*** lifeless has joined #openstack-cinder15:36
*** felipemonteiro__ has quit IRC15:39
*** chaynikasaikia has quit IRC15:40
*** csaikia has joined #openstack-cinder15:40
*** lkuchlan has joined #openstack-cinder15:40
*** MVenesio has quit IRC15:42
openstackgerritOpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext master: Updated from global requirements  https://review.openstack.org/48502515:42
*** belmoreira has quit IRC15:45
*** dulek_mobile has quit IRC15:47
*** dulek_mobile has joined #openstack-cinder15:48
*** _pewp_ has quit IRC15:55
*** Apoorva has joined #openstack-cinder15:57
*** jistr is now known as jistr|mtg15:58
*** _pewp_ has joined #openstack-cinder15:58
*** imacdonn_ has joined #openstack-cinder15:59
*** edmondsw has quit IRC15:59
*** edmondsw has joined #openstack-cinder16:00
*** imacdonn has quit IRC16:01
*** lkuchlan has quit IRC16:02
*** shaner_ has joined #openstack-cinder16:03
*** edmondsw_ has joined #openstack-cinder16:03
*** dr_gogeta86_ has joined #openstack-cinder16:03
*** randomhack has quit IRC16:03
*** jwcroppe has joined #openstack-cinder16:04
*** edmondsw has quit IRC16:05
*** shaner has quit IRC16:06
*** dr_gogeta86 has quit IRC16:06
*** jessegler has joined #openstack-cinder16:07
*** edmondsw_ has quit IRC16:07
*** cburgess has quit IRC16:08
*** cburgess has joined #openstack-cinder16:09
*** yuval has quit IRC16:10
*** yuval has joined #openstack-cinder16:12
*** diliprenkila has joined #openstack-cinder16:14
*** rcernin has joined #openstack-cinder16:14
*** oomichi has quit IRC16:16
*** edmondsw has joined #openstack-cinder16:18
*** oomichi has joined #openstack-cinder16:18
*** jistr|mtg is now known as jistr16:19
smcginnisArnie, no, we don't have that one. I wonder how much effort it would be to add that.16:20
*** edmondsw has quit IRC16:22
*** edmondsw has joined #openstack-cinder16:24
eharneysmcginnis: it's been a while since i thought about that issue, but iirc there's a reasonable path to make the quota_usage info a live count that doesn't get out of sync and therefore doesn't need a tool like that16:24
*** dulek_mobile has quit IRC16:24
openstackgerritJay Bryant proposed openstack/cinder master: [DOC BLD FIX] Fix docstring errors in VNX driver  https://review.openstack.org/48665016:24
smcginniseharney: there wouldn't be too much overhead with that?16:25
eharneysmcginnis: it should be doable by just querying db tables, i would think16:26
*** lucasxu has joined #openstack-cinder16:26
eharneysmcginnis: the whole source of those problems is that we have code that tries to separately increment/decrement counts of usage... but that info should be redundant since we could just look at the volumes table to figure out what exists rather than counting it on create/delete16:27
smcginniseharney: that whole area could probably do with some attention.16:27
*** lucasxu has quit IRC16:28
*** edmondsw has quit IRC16:28
*** tobasco has quit IRC16:30
*** karthikp has quit IRC16:31
*** karthikp has joined #openstack-cinder16:33
*** tobasco has joined #openstack-cinder16:36
ericyounghello all. What is the deadline for getting in patches to the config reference?16:38
*** _ix has joined #openstack-cinder16:40
*** jmlowe has quit IRC16:42
*** edmondsw has joined #openstack-cinder16:52
*** lucasxu has joined #openstack-cinder16:54
*** _ix has quit IRC16:55
*** links has joined #openstack-cinder16:55
*** edmondsw has quit IRC16:56
openstackgerritJay Bryant proposed openstack/cinder master: [DOC BLD FIX] Fix docstring issues in Lenovo driver  https://review.openstack.org/48714216:57
*** randomhack has joined #openstack-cinder16:59
*** firebat has quit IRC17:03
*** diliprenkila has quit IRC17:05
*** randomhack has quit IRC17:06
*** mvk has quit IRC17:09
openstackgerritNishant Kumar proposed openstack/cinder master: Remove In-repo cinder tempest plugin  https://review.openstack.org/48714417:15
*** ChubYann has joined #openstack-cinder17:15
*** diliprenkila has joined #openstack-cinder17:22
openstackgerritChaynika Saikia proposed openstack/cinder master: cinder backup sets incorrect parent_id  https://review.openstack.org/48308917:24
*** diliprenkila has left #openstack-cinder17:25
*** firebat has joined #openstack-cinder17:27
*** links has quit IRC17:31
*** randomhack has joined #openstack-cinder17:36
openstackgerritChris M proposed openstack/cinder master: Remove references to dothill driver options  https://review.openstack.org/48714817:38
*** jwcroppe has quit IRC17:38
*** mvk has joined #openstack-cinder17:41
openstackgerritJay Bryant proposed openstack/cinder master: [DOC BLD FIX] Fix netapp docstring issues  https://review.openstack.org/48714917:41
*** jwcroppe has joined #openstack-cinder17:48
*** oomichi_ has joined #openstack-cinder17:51
*** oomichi has quit IRC17:52
*** oomichi_ is now known as oomichi17:52
*** harlowja has joined #openstack-cinder17:55
*** eharney has quit IRC18:05
*** jmlowe has joined #openstack-cinder18:05
openstackgerritMerged openstack/os-brick master: Get the right portal from output of iscsiadm command  https://review.openstack.org/48607718:06
*** Apoorva_ has joined #openstack-cinder18:07
*** KeithMnemonic has joined #openstack-cinder18:09
*** Apoorva has quit IRC18:10
*** rawanh has joined #openstack-cinder18:13
*** jessegler has quit IRC18:16
*** rawanh has left #openstack-cinder18:18
*** eharney has joined #openstack-cinder18:20
*** itlinux_ has joined #openstack-cinder18:46
*** jdillaman has quit IRC18:55
*** jdillaman has joined #openstack-cinder18:55
*** phealy has quit IRC18:55
*** diablo_rojo has joined #openstack-cinder18:59
*** chhavi has quit IRC19:02
*** KeithMnemonic1 has joined #openstack-cinder19:06
*** KeithMnemonic has quit IRC19:06
*** xyang1 has joined #openstack-cinder19:22
*** firebat has quit IRC19:24
*** randomhack has quit IRC19:25
*** phealy has joined #openstack-cinder19:37
*** phealy has quit IRC19:43
openstackgerritKendall Nelson proposed openstack/cinder master: Mock execute in unit test  https://review.openstack.org/48687019:43
openstackgerritChaynika Saikia proposed openstack/cinder master: cinder incremental backup with ceph fails if last one deleted  https://review.openstack.org/48667819:47
*** nicolasbock has quit IRC19:52
*** mtreinish has quit IRC19:55
*** phealy has joined #openstack-cinder19:59
*** firebat has joined #openstack-cinder20:00
*** Apoorva_ has quit IRC20:02
*** Apoorva has joined #openstack-cinder20:03
*** phealy has quit IRC20:09
*** armax has quit IRC20:10
*** mtreinish has joined #openstack-cinder20:10
*** e0ne has joined #openstack-cinder20:14
*** oomichi has quit IRC20:28
*** oomichi has joined #openstack-cinder20:28
openstackgerritJay Bryant proposed openstack/cinder master: [DOC BLD FIX] Fix docstring errors in nexenta driver  https://review.openstack.org/48719620:31
*** mgiles1 has left #openstack-cinder20:31
*** phealy has joined #openstack-cinder20:36
*** edmondsw has joined #openstack-cinder20:40
*** crushil has quit IRC20:41
*** randomhack has joined #openstack-cinder20:41
*** randomhack has quit IRC20:45
*** armax has joined #openstack-cinder20:47
*** cknight has quit IRC20:50
*** crushil has joined #openstack-cinder20:51
rajinirsmcginnis: The discard option support, will it force all volumes to have the ‘discard’ parameter to be set no matter what the cinder settings are?20:54
*** jmlowe has quit IRC20:57
rajinir: The discard option support, will it force all volumes to have the ‘discard’ parameter to be set no matter what the cinder settings are?21:00
rajinirhttps://github.com/openstack/cinder/blob/master/cinder/volume/drivers/dell_emc/sc/storagecenter_iscsi.py#L16521:00
*** gouthamr has quit IRC21:01
*** csaikia has quit IRC21:05
patrickeastrajinir: looks like for that particular driver, yes it will21:06
SwansonIt does.21:08
*** abishop has quit IRC21:09
*** lucasxu has quit IRC21:09
*** catintheroof has quit IRC21:11
*** jmlowe has joined #openstack-cinder21:13
jgriffith"force" is kind of a misleading word though don't ya think?21:13
jgriffithIt's not really 'forcing' anything, it's just making libvirt actually do what it's supposed to do and not eat the scsi command21:13
jgriffithfrankly it sould behave that way by default if you ask me21:14
jgriffithor any rational person21:14
jgriffithnot that those are mutually exclusive :)21:14
rajinirAgree.21:14
rajinirCustomer concerns21:14
rajinir;)21:14
jgriffithrajinir that I'm not a rational person?  :)21:15
jgriffithLot's of people have that concern21:15
jgriffith:)21:15
rajinirjgriffth: lol. Thanks y'all. Will pass it back.21:15
*** csaikia has joined #openstack-cinder21:16
jgriffithdiablo_rojo ping21:17
openstackgerritVipin Balachandran proposed openstack/cinder master: VMware: Add volume adapter type extra-spec option  https://review.openstack.org/46618521:21
*** e0ne has quit IRC21:23
*** gouthamr has joined #openstack-cinder21:23
openstackgerritVipin Balachandran proposed openstack/cinder master: Fix release notes link  https://review.openstack.org/48722121:28
*** eharney has quit IRC21:28
diablo_rojojgriffith, what's up?21:34
jgriffithdiablo_rojo the sky for now... although I heard it's falling21:35
jgriffithdiablo_rojo I just commented on your patch here https://review.openstack.org/#/c/486870/221:35
* diablo_rojo looks21:36
diablo_rojojgriffith, I too wish tests weren't so magical. I spent like two hours tracing through to figure out what execute was actually being used in the first place to find the ugliness up top in that file where it fake executes EVERY COMMAND POSSIBLE.21:37
jgriffithdiablo_rojo :(21:38
jgriffithsad21:38
* diablo_rojo was curious who wrote all that insanity and it was a bunch of names I didn't really recognize before 2014. The earliest I recognized were like eharney and you21:38
*** csaikia has quit IRC21:38
jgriffithsome of it was me, I'm willing to bet ddt was Eric :)21:39
diablo_rojoI feel like they should be using mock instead of the fake_execute ugliness, but thats for a different patch.21:39
jgriffiththere was something about that whole fake-execute thing, and the insistence of having an execute wrapper/helper in the object21:40
jgriffithbut it's all ancient history at this point21:40
jgriffithanyway, I was curious if you had unravelled all of that and could teach me WTF it's actually doing21:40
jgriffith:)21:40
diablo_rojojgriffith, with the fake_execute? or how it runs the tests again with the different config?21:41
jgriffithwell... the fake_execute is interesting mostly just because it was apparantly nonsense21:42
jgriffiththe fact that it was _executor and not _execute was the first clue21:42
*** mtreinish has quit IRC21:42
jgriffithbut the ddt part is the side that I was more confused on, to your point; yes... the "how the different versions are being called and where they're configs are"21:43
diablo_rojoWhen fake_execute runs the test gets caught here: https://github.com/openstack/cinder/blob/master/cinder/tests/unit/brick/test_brick_lvm.py#L15921:43
diablo_rojoand it just passes which is why I think anything that was changed in the overrode method would work21:43
diablo_rojocause it was passing every time21:43
jgriffithmake sense21:43
diablo_rojoas for the ddt part.. Separating the two runs of the tests would be ideal.21:44
diablo_rojoRather than running them back to back. The code reuse is nice, but I feel like there is a better way.21:45
jgriffithyeah21:45
jgriffithtoo much of a good thing sometimes21:46
jgriffithoh well21:46
*** mtreinish has joined #openstack-cinder21:46
*** rtjure has joined #openstack-cinder21:47
diablo_rojojgriffith, looking at the setup function ..its almost like..recursive.21:47
jgriffithyeah, I guess it's cool21:47
jgriffithjust not very friendly21:48
diablo_rojoIt goes through setup and is like if there is no config set it up this way and then it calls super with the test case21:48
diablo_rojoAgreed21:48
diablo_rojojgriffith, I didn't notice that it was running twice till this morning when I saw Jenkins had failed.21:48
jgriffithdiablo_rojo yeah, and one of the things that's kind of misleading is if you run the test by itself (ie tox -epy27 -- xxxxxx) it only runs once21:49
diablo_rojoAnd then I had to play around with ways of passing in the right options for each run, what I pushed was the cleanest thing I could come up with21:49
diablo_rojojgriffith, lol yeah that's exactly why I missed it21:50
diablo_rojoIt was getting late so I didn't do the customary running of all the tests before I pushed or I would have seen it before this morning21:50
*** Apoorva_ has joined #openstack-cinder21:51
jgriffithdiablo_rojo anyway, good for you tackling it and fixing it up21:51
diablo_rojoIt occurred to me that I want to vote in this PTL election and that  I should probably push some patches so I can actually do that :)21:51
jgriffithhaah!21:52
diablo_rojojgriffith, so if you wanna +2 me that would be super helpful ;)21:52
jgriffithoh wait.. shit21:52
*** rtjure has quit IRC21:52
diablo_rojojgriffith, lol21:52
jgriffithI'm a solid +121:52
jgriffithif eharney likes it I'll gladly +A21:52
diablo_rojojgriffith, cool :) I will go poke at him when/if he comes online.21:53
*** Apoorva has quit IRC21:54
*** gouthamr has quit IRC21:58
openstackgerritVipin Balachandran proposed openstack/cinder-specs master: Backend validation for extra spec options  https://review.openstack.org/31857621:58
*** firebat has quit IRC21:59
openstackgerritMerged openstack/cinder master: Mark ZTE driver as unsupported  https://review.openstack.org/47350522:07
*** xyang1 has quit IRC22:08
*** rtjure has joined #openstack-cinder22:17
*** cknight has joined #openstack-cinder22:34
*** mtanino has quit IRC22:41
openstackgerritMerged openstack/cinder master: Add cinder backup service initialize check  https://review.openstack.org/44651822:42
*** rcernin has quit IRC22:58
*** sdague has quit IRC23:07
*** edmondsw has quit IRC23:10
*** edmondsw has joined #openstack-cinder23:11
*** rtjure has quit IRC23:14
*** edmondsw has quit IRC23:15
*** rtjure has joined #openstack-cinder23:17
*** chlong has quit IRC23:32
*** deep-book-gk_ has joined #openstack-cinder23:33
*** deep-book-gk_ has left #openstack-cinder23:35
openstackgerritjunbo.li proposed openstack/python-cinderclient master: Update cinder.rst and shell.rst  https://review.openstack.org/47707423:36
*** jwcroppe has quit IRC23:36
*** jwcroppe has joined #openstack-cinder23:37
*** jwcroppe has quit IRC23:37
*** wxy- has joined #openstack-cinder23:46
*** wxy- has quit IRC23:48
*** sdague has joined #openstack-cinder23:50
*** DFFlanders has joined #openstack-cinder23:56

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