Wednesday, 2016-07-27

*** leeantho has quit IRC00:03
*** Suyi_ has quit IRC00:04
*** rdx_matt has joined #openstack-cinder00:08
*** jamielennox is now known as jamielennox|away00:12
*** sdague has quit IRC00:16
*** gouthamr has joined #openstack-cinder00:17
*** diogogmt has quit IRC00:27
*** erlon has joined #openstack-cinder00:28
*** alonma has joined #openstack-cinder00:32
*** alonma has quit IRC00:36
*** alonma has joined #openstack-cinder00:39
*** rdx_matt has quit IRC00:41
*** laughterwym has joined #openstack-cinder00:44
*** laughterwym has quit IRC00:44
*** alonma has quit IRC00:44
*** laughterwym has joined #openstack-cinder00:44
*** laughterwym has joined #openstack-cinder00:45
*** sdake_ has quit IRC00:45
*** alonma has joined #openstack-cinder00:47
*** sdake has joined #openstack-cinder00:47
*** mtanin___ has quit IRC00:48
*** alonma has quit IRC00:51
*** ducttape_ has quit IRC00:54
*** ducttape_ has joined #openstack-cinder00:56
*** enriquetaso has quit IRC00:56
*** huyang has joined #openstack-cinder00:59
*** cknight1 has joined #openstack-cinder01:00
*** cknight has quit IRC01:02
*** sdake has quit IRC01:04
*** kaisers_ has joined #openstack-cinder01:06
*** kaisers_ has quit IRC01:11
*** yangyapeng has joined #openstack-cinder01:11
*** ducttape_ has quit IRC01:17
*** Apoorva has quit IRC01:18
*** zhongjun_ has joined #openstack-cinder01:29
*** TaylorHuang has joined #openstack-cinder01:31
*** openstackgerrit has quit IRC01:33
*** gouthamr_ has joined #openstack-cinder01:34
*** Lee1092 has joined #openstack-cinder01:34
*** TaylorH has quit IRC01:34
*** openstackgerrit has joined #openstack-cinder01:34
*** TaylorH has joined #openstack-cinder01:35
*** gouthamr has quit IRC01:37
*** rdx_matt has joined #openstack-cinder01:38
*** TaylorHuang has quit IRC01:38
*** Srinu_ has joined #openstack-cinder01:38
*** TaylorH has quit IRC01:39
*** jwcroppe has quit IRC01:42
*** yangyapeng has quit IRC01:44
*** yangyapeng has joined #openstack-cinder01:44
*** itzdilip has joined #openstack-cinder01:52
openstackgerritJiWei proposed openstack/cinder: Implement nfs manage/unmanage of a volume
*** Srinu_ has quit IRC01:58
*** laughterwym has quit IRC02:03
*** coolsvap_ has joined #openstack-cinder02:04
*** laughterwym has joined #openstack-cinder02:04
*** julim has joined #openstack-cinder02:05
openstackgerritLisaLi proposed openstack/cinder: Add backup update function (microversion)
*** rdx_matt has quit IRC02:11
*** sdake has joined #openstack-cinder02:12
*** Suyi_ has joined #openstack-cinder02:12
*** iceyao has joined #openstack-cinder02:15
*** ebalduf has joined #openstack-cinder02:16
*** bardia has quit IRC02:19
*** salv-orl_ has quit IRC02:21
*** lixiaoy1 has quit IRC02:23
*** lixiaoy1 has joined #openstack-cinder02:24
*** Gao has joined #openstack-cinder02:26
*** Lallana has quit IRC02:26
*** yangyapeng has quit IRC02:30
openstackgerritJohn Griffith proposed openstack/cinder: Move fake_constants to root test directory
*** alonma has joined #openstack-cinder02:35
*** alonma has quit IRC02:39
*** jamielennox|away is now known as jamielennox02:41
*** alonma has joined #openstack-cinder02:41
*** ebalduf has quit IRC02:45
*** alonma has quit IRC02:46
*** ducttape_ has joined #openstack-cinder02:48
*** sdake has quit IRC02:53
*** ducttape_ has quit IRC02:53
*** gouthamr_ has quit IRC02:54
*** kaisers_ has joined #openstack-cinder02:55
*** iceyao has quit IRC02:58
*** cknight1 has quit IRC02:59
*** kaisers_ has quit IRC02:59
*** alonma has joined #openstack-cinder03:00
*** cknight has joined #openstack-cinder03:01
*** iceyao has joined #openstack-cinder03:01
*** alonma has quit IRC03:04
*** alonma has joined #openstack-cinder03:07
*** rdx_matt has joined #openstack-cinder03:09
openstackgerritxiaoqin proposed openstack/cinder: Storwize: do the correct update after failover
*** alonma has quit IRC03:11
*** sdake has joined #openstack-cinder03:13
*** cknight has quit IRC03:14
*** thumpba has joined #openstack-cinder03:15
*** thumpba has quit IRC03:18
*** yangyapeng has joined #openstack-cinder03:22
*** salv-orlando has joined #openstack-cinder03:22
*** erlon has quit IRC03:26
*** salv-orlando has quit IRC03:28
*** ducttape_ has joined #openstack-cinder03:31
*** ducttape_ has quit IRC03:36
*** yangyape_ has joined #openstack-cinder03:36
*** yangyapeng has quit IRC03:39
*** smile-luobin has joined #openstack-cinder03:39
*** rdx_matt has quit IRC03:41
*** enriquetaso has joined #openstack-cinder03:44
*** enriquetaso has quit IRC03:44
*** sheel has joined #openstack-cinder03:48
*** zhongjun_ has quit IRC03:49
*** abhitechie has joined #openstack-cinder03:51
*** harlowja_at_home has joined #openstack-cinder03:52
*** zhonghua has quit IRC03:52
*** zhonghua-lee has joined #openstack-cinder03:53
*** alonma has joined #openstack-cinder03:53
*** links has joined #openstack-cinder03:54
*** alonma has quit IRC03:58
*** lpetrut has joined #openstack-cinder03:59
*** thumpba has joined #openstack-cinder04:00
*** alonma has joined #openstack-cinder04:01
*** laughterwym has quit IRC04:02
*** alonma has quit IRC04:05
*** laughterwym has joined #openstack-cinder04:05
*** thumpba has quit IRC04:06
*** kaisers_ has joined #openstack-cinder04:07
*** kaisers_ has quit IRC04:07
*** alonma has joined #openstack-cinder04:08
*** sandanar has joined #openstack-cinder04:09
*** links has quit IRC04:09
*** laughterwym has quit IRC04:10
*** links has joined #openstack-cinder04:12
*** alonma has quit IRC04:12
*** thumpba has joined #openstack-cinder04:15
*** alonma has joined #openstack-cinder04:16
*** jgregor has joined #openstack-cinder04:19
*** alonma has quit IRC04:20
*** harlowja_at_home has quit IRC04:22
*** alonma has joined #openstack-cinder04:23
*** salv-orlando has joined #openstack-cinder04:24
*** laughterwym has joined #openstack-cinder04:24
*** ManishD has joined #openstack-cinder04:26
*** jgregor1 has joined #openstack-cinder04:26
*** thumpba has quit IRC04:27
*** jgregor has quit IRC04:27
*** alonma has quit IRC04:28
*** salv-orlando has quit IRC04:28
*** laughterwym has quit IRC04:29
*** pwm has joined #openstack-cinder04:29
*** pwm has quit IRC04:30
*** lpetrut has quit IRC04:30
*** jamielennox is now known as jamielennox|away04:30
*** pwm has joined #openstack-cinder04:30
*** alonma has joined #openstack-cinder04:31
*** bks has joined #openstack-cinder04:31
*** avishay has joined #openstack-cinder04:34
*** liverpooler has quit IRC04:34
*** Poornima has joined #openstack-cinder04:35
avishaysmcginnis: hey, just saw your review on the list manageable v3 patch. the rebase was actually trivial - just change from microversion 3.7 to 3.8.04:35
*** alonma has quit IRC04:35
*** thumpba has joined #openstack-cinder04:36
*** alonma has joined #openstack-cinder04:38
*** rdx_matt has joined #openstack-cinder04:39
*** GB21 has joined #openstack-cinder04:39
*** laughterwym has joined #openstack-cinder04:39
openstackgerritxing-yang proposed openstack/cinder: Add generic volume groups
*** jgregor1 has quit IRC04:42
*** alonma has quit IRC04:42
*** sandanar has quit IRC04:44
*** alonma has joined #openstack-cinder04:46
openstackgerritxing-yang proposed openstack/cinder: Add group type and group specs
openstackgerritxing-yang proposed openstack/cinder: Add generic volume groups
*** alonma has quit IRC04:50
*** jamielennox|away is now known as jamielennox04:51
*** alonma has joined #openstack-cinder04:53
openstackgerritxing-yang proposed openstack/cinder: WIP: Add group snapshots
*** Poornima has quit IRC04:57
*** alonma has quit IRC04:57
*** ducttape_ has joined #openstack-cinder04:57
*** adisky has joined #openstack-cinder05:00
avishayif any core reviewer can +A this again I would appreciate it - keep having to rebase microversion patch...
avishayDuncanT smcginnis hemnafk ^05:01
*** ducttape_ has quit IRC05:02
*** GB21 has quit IRC05:03
*** GB21 has joined #openstack-cinder05:03
*** takedakn has joined #openstack-cinder05:04
openstackgerritVijay Ladani proposed openstack/cinder: Retry delete snapshot if snapshot is busy
openstackgerritChangBo Guo(gcb) proposed openstack/cinder: Fix wrong usage of config opiton gpfs_images_share_mode
*** pbandark has joined #openstack-cinder05:08
*** takedakn has quit IRC05:09
*** laughter_ has joined #openstack-cinder05:10
*** rdx_matt has quit IRC05:12
*** laughterwym has quit IRC05:13
*** salv-orlando has joined #openstack-cinder05:14
*** pwm has quit IRC05:15
openstackgerritChangBo Guo(gcb) proposed openstack/cinder: test: fix wrong usage of config option in test_window
*** maestropandy has joined #openstack-cinder05:21
openstackgerritSatish Venkatasubramanian proposed openstack/cinder: Add volumes/summary API support.
*** GB21 has quit IRC05:25
*** beagles has quit IRC05:26
*** GB21 has joined #openstack-cinder05:27
*** bks has quit IRC05:29
*** laughter_ has quit IRC05:31
*** ChubYann has quit IRC05:31
*** laughterwym has joined #openstack-cinder05:31
*** pgadiya has joined #openstack-cinder05:34
*** avishay has quit IRC05:43
*** alonma has joined #openstack-cinder05:47
*** GB21 has quit IRC05:49
*** GB21 has joined #openstack-cinder05:51
*** alonma has quit IRC05:52
*** sandanar has joined #openstack-cinder05:53
*** EinstCrazy has joined #openstack-cinder05:53
*** maestropandy has quit IRC05:54
*** alonma has joined #openstack-cinder05:55
*** coolsvap_ is now known as coolsvap05:56
nikeshmavishay: hi05:56
openstackgerritShay Halsband proposed openstack/cinder: XtremIO: fix the returned value of CG actions
*** EinstCrazy has quit IRC05:58
*** ducttape_ has joined #openstack-cinder05:58
*** alonma has quit IRC05:59
*** bardia has joined #openstack-cinder06:00
*** EinstCrazy has joined #openstack-cinder06:02
*** arecknag has joined #openstack-cinder06:02
*** nkrinner_afk is now known as nkrinner06:02
*** alonma has joined #openstack-cinder06:02
*** ducttape_ has quit IRC06:02
*** Suyi_ has quit IRC06:05
*** zhongjun_ has joined #openstack-cinder06:06
*** alonma has quit IRC06:07
*** laughterwym has quit IRC06:09
*** alonma has joined #openstack-cinder06:10
*** rdx_matt has joined #openstack-cinder06:10
*** EinstCra_ has joined #openstack-cinder06:12
*** laughter_ has joined #openstack-cinder06:12
*** laughte__ has joined #openstack-cinder06:14
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata
*** GB21 has quit IRC06:15
*** alonma has quit IRC06:15
*** laughterwym has joined #openstack-cinder06:15
*** GB21 has joined #openstack-cinder06:16
*** EinstCrazy has quit IRC06:16
*** sdake has quit IRC06:16
*** alonma has joined #openstack-cinder06:17
*** laughter_ has quit IRC06:18
*** laughte__ has quit IRC06:19
*** GB21 has quit IRC06:21
*** alonma has quit IRC06:22
*** alonma has joined #openstack-cinder06:25
*** salv-orl_ has joined #openstack-cinder06:28
*** alonma has quit IRC06:29
*** bardia has quit IRC06:29
*** salv-orlando has quit IRC06:31
*** liverpooler has joined #openstack-cinder06:31
openstackgerritlg.yue proposed openstack/cinder: add metadata length check to volume-manage
*** alonma has joined #openstack-cinder06:32
*** amitkqed has joined #openstack-cinder06:35
*** amitkqed has quit IRC06:36
*** alonma has quit IRC06:37
*** amitkqed has joined #openstack-cinder06:37
*** laughterwym has quit IRC06:38
*** GB21 has joined #openstack-cinder06:40
*** alonma has joined #openstack-cinder06:40
*** laughter_ has joined #openstack-cinder06:41
*** ManishD has quit IRC06:42
*** rdx_matt has quit IRC06:42
*** tesseract- has joined #openstack-cinder06:44
*** alonma has quit IRC06:44
*** sdake has joined #openstack-cinder06:45
openstackgerritVijay Ladani proposed openstack/cinder: Retry delete snapshot if snapshot is busy
openstackgerritCao ShuFeng proposed openstack/python-cinderclient: Add "start_version" and "end_version" support to  argparse
openstackgerritCao ShuFeng proposed openstack/cinder: Enhance api_microversion doc
*** rcernin has joined #openstack-cinder06:47
openstackgerritCao ShuFeng proposed openstack/python-cinderclient: Fix useless api_version of Manager class
*** maestropandy has joined #openstack-cinder06:48
*** ameade has quit IRC06:49
*** rhefner has quit IRC06:49
*** derekjhyang has quit IRC06:49
*** zhiyan has quit IRC06:49
*** briancurtin has quit IRC06:49
*** rhefner has joined #openstack-cinder06:50
*** cppforlife_ has quit IRC06:50
*** amit213 has quit IRC06:50
*** ildikov has quit IRC06:50
*** amit213 has joined #openstack-cinder06:51
*** adrianofr has quit IRC06:51
*** ildikov has joined #openstack-cinder06:51
*** derekjhyang has joined #openstack-cinder06:51
*** alyson_ has quit IRC06:52
*** ameade has joined #openstack-cinder06:52
*** zhiyan has joined #openstack-cinder06:52
*** briancurtin has joined #openstack-cinder06:52
*** cppforlife_ has joined #openstack-cinder06:52
*** thumpba has quit IRC06:53
*** adrianofr has joined #openstack-cinder06:53
*** alyson_ has joined #openstack-cinder06:53
*** nkrinner has quit IRC06:54
*** nkrinner has joined #openstack-cinder06:57
*** mjura has joined #openstack-cinder06:58
*** ducttape_ has joined #openstack-cinder06:59
*** sdake has quit IRC07:01
*** ducttape_ has quit IRC07:03
*** nkrinner has quit IRC07:05
openstackgerritSwapnil Kulkarni (coolsvap) proposed openstack/cinder: [WIP] Testing latest u-c
*** thumpba has joined #openstack-cinder07:15
ellenCould I get some reviews on this patch please?
*** rdx_matt has joined #openstack-cinder07:17
*** nkrinner has joined #openstack-cinder07:18
*** laughter_ has quit IRC07:18
*** thumpba has quit IRC07:18
*** jordanP has joined #openstack-cinder07:19
*** jordanP has quit IRC07:19
*** e0ne has joined #openstack-cinder07:20
*** JayConroy has quit IRC07:22
openstackgerritwangxiyuan proposed openstack/cinder: Use object instead of string when reset backup
*** JayConroy has joined #openstack-cinder07:23
*** salv-orlando has joined #openstack-cinder07:25
*** salv-orl_ has quit IRC07:25
*** alee_dinner has quit IRC07:26
*** laughterwym has joined #openstack-cinder07:26
*** laughter_ has joined #openstack-cinder07:28
*** laughte__ has joined #openstack-cinder07:29
*** laughterwym has quit IRC07:32
*** laughte__ has quit IRC07:32
*** laughter_ has quit IRC07:33
openstackgerritzheng yin proposed openstack/cinder: Add hacking for assertFalse
*** alonma has joined #openstack-cinder07:34
*** laughterwym has joined #openstack-cinder07:34
*** hemnafk has quit IRC07:35
*** hemnafk has joined #openstack-cinder07:35
*** mtreinish has quit IRC07:36
*** laughter_ has joined #openstack-cinder07:36
*** mtreinish has joined #openstack-cinder07:36
*** lickitysplitted has quit IRC07:37
*** lickitysplitted has joined #openstack-cinder07:37
*** maestropandy has quit IRC07:38
*** alonma has quit IRC07:39
*** laughterwym has quit IRC07:39
*** lennyb has quit IRC07:40
*** avishay has joined #openstack-cinder07:40
*** lennyb has joined #openstack-cinder07:41
*** alonma has joined #openstack-cinder07:41
openstackgerritShay Halsband proposed openstack/cinder: XtremIO: Implement update_migrated_volume
*** takedakn has joined #openstack-cinder07:42
*** alonma has quit IRC07:45
*** e0ne has quit IRC07:48
*** alonma has joined #openstack-cinder07:49
*** alonma has quit IRC07:53
*** ollie_ has quit IRC07:55
*** GB21 has quit IRC07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:00
*** ducttape_ has joined #openstack-cinder08:00
*** akapil has joined #openstack-cinder08:02
*** akapil has quit IRC08:03
*** akapil has joined #openstack-cinder08:03
*** wewe0901 has joined #openstack-cinder08:04
*** ducttape_ has quit IRC08:04
*** alonma has joined #openstack-cinder08:05
openstackgerritSatish Venkatasubramanian proposed openstack/cinder: Add volumes/summary API support.
*** guy has joined #openstack-cinder08:05
*** abhitechie has quit IRC08:07
*** abhitechie has joined #openstack-cinder08:07
*** akapil has quit IRC08:07
*** alonma has quit IRC08:09
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c
*** salv-orlando has quit IRC08:10
*** jdurgin has quit IRC08:11
*** akapil has joined #openstack-cinder08:11
*** salv-orlando has joined #openstack-cinder08:12
*** jdurgin has joined #openstack-cinder08:12
*** alonma has joined #openstack-cinder08:12
*** hoonetorg has quit IRC08:13
*** karthikp_ has quit IRC08:14
*** hoonetorg has joined #openstack-cinder08:17
*** salv-orlando has quit IRC08:17
*** avishay has quit IRC08:18
*** alonma has quit IRC08:19
*** alonma has joined #openstack-cinder08:21
*** alonma has quit IRC08:25
*** alee_dinner has joined #openstack-cinder08:25
*** alonma has joined #openstack-cinder08:28
openstackgerritGorka Eguileor proposed openstack/cinder: Fail manage operations if service is down
*** laughter_ has quit IRC08:30
*** avishay has joined #openstack-cinder08:32
*** alonma has quit IRC08:33
*** takedakn1 has joined #openstack-cinder08:33
*** takedakn has quit IRC08:33
*** takedakn1 is now known as takedakn08:33
*** alonma has joined #openstack-cinder08:36
*** e0ne has joined #openstack-cinder08:40
*** alonma has quit IRC08:40
*** lixiaoy1 has quit IRC08:42
*** alonma has joined #openstack-cinder08:43
*** yangyape_ has quit IRC08:45
*** EinstCra_ has quit IRC08:45
*** iceyao has quit IRC08:45
*** yangyapeng has joined #openstack-cinder08:46
*** iceyao has joined #openstack-cinder08:46
*** EinstCrazy has joined #openstack-cinder08:46
*** zhangjn has quit IRC08:47
*** EinstCrazy has quit IRC08:47
*** EinstCrazy has joined #openstack-cinder08:47
*** alonma has quit IRC08:48
openstackgerritGorka Eguileor proposed openstack/cinder: Refactor create, save, and destroy OVO methods
openstackgerritHuang Zhiteng proposed openstack/cinder: DO NOT MERGE: Implement Stochastic Weight handler
*** savihou has joined #openstack-cinder08:50
*** alonma has joined #openstack-cinder08:51
*** zhangjn has joined #openstack-cinder08:52
*** GB21 has joined #openstack-cinder08:53
*** alonma has quit IRC08:55
*** bks has joined #openstack-cinder08:56
*** ducttape_ has joined #openstack-cinder09:01
*** alonma has joined #openstack-cinder09:01
*** ducttape_ has quit IRC09:05
*** alonma has quit IRC09:06
*** laughterwym has joined #openstack-cinder09:07
*** avishay has quit IRC09:07
*** GB21 has quit IRC09:07
*** alonma has joined #openstack-cinder09:09
openstackgerritzheng yin proposed openstack/cinder: Add hacking for assertFalse
*** Adam__ has joined #openstack-cinder09:10
Adam__Morning! Does anyone know the impact on existing volumes if I were to enable thin provisioning / lvm_type=thin? Is there a risk of losing existing containers?09:10
*** GB21 has joined #openstack-cinder09:11
*** laughterwym has quit IRC09:11
*** sticker has quit IRC09:13
*** alonma has quit IRC09:14
*** sticker has joined #openstack-cinder09:15
*** EinstCrazy has quit IRC09:17
*** avishay has joined #openstack-cinder09:19
*** maestropandy has joined #openstack-cinder09:19
*** GB21 has quit IRC09:20
openstackgerritDinesh Bhor proposed openstack/python-cinderclient: Use self.ks_logger instead of ks_logger
*** EinstCra_ has joined #openstack-cinder09:25
Dinesh_BhorCan someone restore this patch ?
Dinesh_BhorI want to take it ahead.09:25
*** EinstCra_ has quit IRC09:27
*** iceyao has quit IRC09:27
*** iceyao has joined #openstack-cinder09:28
*** EinstCrazy has joined #openstack-cinder09:28
e0neDinesh_Bhor: I can do it but looks like community doesn't want to have it09:29
e0neDinesh_Bhor: it makes migrations code too complicated09:29
Dinesh_Bhore0ne: It will be great if you restore the patch I will push a new patch over it and discuss with community.09:30
e0neDinesh_Bhor: ok. did you see "Need to re-think it and make cross-project" comment?09:30
e0neDinesh_Bhor: patch is restrored09:31
*** nherciu has joined #openstack-cinder09:31
Dinesh_Bhore0ne: yah ..I have seen that. We are proposing this in glance as well. Lets see what comes next.09:32
Dinesh_Bhore0ne: Thank you for restoring.09:32
*** dansmith has quit IRC09:33
*** yangyapeng has quit IRC09:34
*** dansmith has joined #openstack-cinder09:35
*** dansmith is now known as Guest4106909:35
*** yangyapeng has joined #openstack-cinder09:35
openstackgerritAbhishek Kekane proposed openstack/cinder: Allow archiving deleted rows to shadow tables, for performance
openstackgerritlakshman proposed openstack/cinder: Support manage/unmanage and extra-specs in K2
*** iceyao has quit IRC09:37
e0neDinesh_Bhor: IMO, it would be great if we can do it in oslo.db09:37
Dinesh_Bhore0ne: ok, I will try to implement this suggestion.09:38
e0neDinesh_Bhor: thanks09:39
*** yangyapeng has quit IRC09:40
*** EinstCra_ has joined #openstack-cinder09:42
*** iceyao has joined #openstack-cinder09:42
*** yangyapeng has joined #openstack-cinder09:42
*** EinstCrazy has quit IRC09:43
nikeshme0ne: hi do you know how to add boolean value while setting up key value pair in volume type09:45
nikeshmcinder type-key xyz set xxx=True09:46
e0nenikeshm: did you try  True or False?09:46
nikeshmyes tried, but in cinder extra-specs-list, it is showing {'xxx': 'True'}09:47
nikeshmhow can we do {'xxx': True}09:48
*** tangchen_ has quit IRC09:50
*** yuelongguang has quit IRC09:50
openstackgerritMerged openstack/os-brick: Remove useless info logging in check_valid_device
*** alee_dinner has quit IRC09:54
*** maestropandy has quit IRC09:57
*** chlong_POffice has quit IRC09:57
*** alee_dinner has joined #openstack-cinder09:59
*** alonma has joined #openstack-cinder10:03
*** tangchen_ has joined #openstack-cinder10:04
*** anshprat has quit IRC10:05
*** hwcomcn has joined #openstack-cinder10:06
*** mvk has quit IRC10:07
*** alonma has quit IRC10:08
*** alonma has joined #openstack-cinder10:10
*** chlong_POffice has joined #openstack-cinder10:11
*** yangyapeng has quit IRC10:12
*** EinstCra_ has quit IRC10:13
*** EinstCrazy has joined #openstack-cinder10:13
*** alonma has quit IRC10:15
*** alonma has joined #openstack-cinder10:18
*** alonma has quit IRC10:22
*** alonma has joined #openstack-cinder10:25
*** maestropandy has joined #openstack-cinder10:27
*** lprice1 has quit IRC10:29
*** lprice has joined #openstack-cinder10:29
*** ociuhandu has joined #openstack-cinder10:30
*** alonma has quit IRC10:30
*** alonma has joined #openstack-cinder10:33
*** alonma has quit IRC10:37
*** mvk has joined #openstack-cinder10:38
*** alonma has joined #openstack-cinder10:40
*** takedakn has quit IRC10:45
*** alonma has quit IRC10:45
*** akapil has quit IRC10:48
*** maestropandy has quit IRC10:51
winston-dnikeshm: try 'cinder key-key xyz set xxx=<is>True'10:53
*** akapil has joined #openstack-cinder11:04
*** tpsilva has joined #openstack-cinder11:05
*** akapil has quit IRC11:09
*** salv-orlando has joined #openstack-cinder11:09
*** gouthamr has joined #openstack-cinder11:14
*** akapil has joined #openstack-cinder11:16
*** gouthamr_ has joined #openstack-cinder11:21
*** gouthamr has quit IRC11:24
*** takedakn has joined #openstack-cinder11:25
openstackgerritAlyson proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages
*** sdague has joined #openstack-cinder11:30
*** gouthamr_ is now known as gouthamr11:30
*** takedakn has quit IRC11:32
*** ociuhandu has quit IRC11:33
*** ociuhandu has joined #openstack-cinder11:35
*** cdelatte has joined #openstack-cinder11:48
*** abhitechie has quit IRC11:50
*** erlon has joined #openstack-cinder11:51
*** savihou has quit IRC11:52
*** savihou has joined #openstack-cinder11:53
*** takedakn has joined #openstack-cinder11:54
gouthamre0ne: hi! when you have time, could you please review - I had your +2/+W, but it needed rebasing and minor change at the gate.12:00
*** Lee1092 has quit IRC12:03
*** ducttape_ has joined #openstack-cinder12:03
*** ducttape_ has quit IRC12:05
*** ducttape_ has joined #openstack-cinder12:06
gouthamre0ne: thanks!12:07
e0negouthamr: you're welcome12:07
*** andymaier has joined #openstack-cinder12:12
*** savihou has quit IRC12:14
*** timcl has joined #openstack-cinder12:14
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix 'driver does not honor clone/snapshot' bugs
*** maestropandy has joined #openstack-cinder12:16
openstackgerritAlyson proposed openstack/cinder: HNAS: Add support for manage/unmanage snapshots in NFS driver
*** laughter_ has joined #openstack-cinder12:19
*** akerr has joined #openstack-cinder12:21
*** chlong_POffice has quit IRC12:21
*** ducttape_ has quit IRC12:22
*** akerr_ has joined #openstack-cinder12:23
*** sdake has joined #openstack-cinder12:23
*** alee_dinner is now known as alee12:24
*** gouthamr has quit IRC12:24
*** hwcomcn has quit IRC12:25
*** yangyapeng has joined #openstack-cinder12:25
*** akerr has quit IRC12:26
*** alonma has joined #openstack-cinder12:28
*** chlong has joined #openstack-cinder12:29
*** e0ne has quit IRC12:31
*** akerr_ is now known as akerr12:31
openstackgerritCao ShuFeng proposed openstack/python-cinderclient: Revmove extra 'u' from cli output
*** alonma has quit IRC12:33
openstackgerritCao ShuFeng proposed openstack/python-cinderclient: Revmove extra 'u' from cli output
*** beagles has joined #openstack-cinder12:35
*** gouthamr has joined #openstack-cinder12:36
*** alonma has joined #openstack-cinder12:37
*** JoseMello has joined #openstack-cinder12:38
*** EinstCrazy has quit IRC12:40
*** iceyao has quit IRC12:41
*** alonma has quit IRC12:42
*** abhitechie has joined #openstack-cinder12:46
*** pauloewerton has joined #openstack-cinder12:47
*** beagles has left #openstack-cinder12:54
*** sheel has quit IRC12:56
openstackgerritzheng yin proposed openstack/cinder: Add hacking for assertFalse
*** avishay has quit IRC13:05
*** coolsvap is now known as coolsvap_13:07
*** links has quit IRC13:08
*** rlrossit has joined #openstack-cinder13:11
*** timcl has quit IRC13:12
*** faiz89 has joined #openstack-cinder13:12
*** merooney has joined #openstack-cinder13:14
Srinupatrickeast: hi13:16
*** porrua has joined #openstack-cinder13:17
*** edmondsw has joined #openstack-cinder13:17
Srinuwhat is version of zuul, jenkins and nodepool in your CI configuration13:17
*** liverpooler has quit IRC13:17
*** jungleboyj has joined #openstack-cinder13:18
*** dustins has joined #openstack-cinder13:21
*** avishay has joined #openstack-cinder13:21
*** adisky has quit IRC13:21
openstackgerritzheng yin proposed openstack/cinder: Use assertEqual() instead of assertDictEqual()
*** akshai has joined #openstack-cinder13:21
*** gouthamr has quit IRC13:22
*** Adam__ has quit IRC13:22
*** gouthamr has joined #openstack-cinder13:23
*** pgadiya has quit IRC13:23
*** crose has joined #openstack-cinder13:24
*** xyang1 has joined #openstack-cinder13:24
*** gman-tx has joined #openstack-cinder13:25
*** timcl has joined #openstack-cinder13:26
gouthamrlooking for a +2/+W, anyone? :)
*** Yogi1 has joined #openstack-cinder13:27
smcginnisjgriffith, hemnafk: I do need to request branches for the "stable-driver" branches we discussed though.13:29
flip214what should I do if "tools/ generate" dies with "AttributeError: 'NoneType' object has no attribute 'groups'" ?13:30
smcginnisJK :)13:30
smcginnisflip214: Where did you see that?13:30
*** mriedem has joined #openstack-cinder13:31
*** david-lyle_ has joined #openstack-cinder13:31
*** david-lyle_ has quit IRC13:31
*** ebalduf has joined #openstack-cinder13:33
gouthamrsmcginnis: thank you..13:33
smcginnisgouthamr: np13:34
flip214smcginnis: running that command locally, because of
smcginnisIs gerrit extremely slow for anyone else right now?13:34
*** david-lyle has quit IRC13:34
*** ducttape_ has joined #openstack-cinder13:36
*** alonma has joined #openstack-cinder13:39
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Fix retype when storage policy is enabled
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Add volume name in vCenter to conn info
*** abhinavtechie has joined #openstack-cinder13:41
*** maestropandy has left #openstack-cinder13:42
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Cleanup of Initiator group fails
*** aagrawal has joined #openstack-cinder13:43
*** abhitechie has quit IRC13:43
*** yuriy_n17 has joined #openstack-cinder13:43
*** alonma has quit IRC13:43
*** abhinavtechie has quit IRC13:45
*** alonma has joined #openstack-cinder13:46
*** Guest41069 is now known as dansmith13:48
nikeshmwinston-d: cinder type-key xyz set xxx=<is>True -bash: is: No such file or directory13:48
*** aagrawal has quit IRC13:48
smcginnisFYI - gerrit restart coming. Brace yourselves. :)13:51
*** alonma has quit IRC13:51
-openstackstatus- NOTICE: Gerrit is being restarted now to relieve performance degradation13:51
*** alonma has joined #openstack-cinder13:54
*** takedakn has quit IRC13:55
*** jgregor has joined #openstack-cinder13:56
*** Fdaisuke has joined #openstack-cinder13:57
*** alonma has quit IRC13:58
*** avishay has quit IRC14:00
*** alonma has joined #openstack-cinder14:03
*** jwcroppe has joined #openstack-cinder14:03
*** alonma has quit IRC14:08
flip214BTW, what's the right solution if Python 2.7 "long" doesn't exist in Python 3 any more? I saw some guides telling to have a version check with "long = int"... (ouch)14:10
jgriffithsmcginnis: LOL14:10
jgriffithsmcginnis: I don't think we're going to go the branch path at this time14:10
*** alonma has joined #openstack-cinder14:10
jgriffithsmcginnis: unless others want to fight that battle, but I don't14:11
bswartznikeshm: use quotes14:11
*** aswadr_ has joined #openstack-cinder14:11
*** yuelongguang has joined #openstack-cinder14:11
smcginnisjgriffith: I know the feature branch idea got shot down.14:11
bswartzcinder type-key xyz set "xxx=<is> True"14:12
jgriffithsmcginnis: gaahhhh14:12
smcginnisBut the other one we talked about was having stable branches that were a little more open and a little longer lived for driver bug backports.14:12
jgriffithdidn't read close enough14:12
jgriffithYES please on the other branches we talked about14:12
smcginnisYeah, no worries. I figured that was the case. :)14:12
nikeshmbswartz: cinder type-key xyz set xxx='<is>True'14:12
smcginnisThat does seem to address some of the concern for wanting drivers out of tree.14:12
*** bks has quit IRC14:13
*** avishay has joined #openstack-cinder14:13
jgriffithsmcginnis: well at least the ones for people that are just worried about supporting them long term14:13
bswartzsmcginnis: who's shooting this stuff down?14:13
nikeshmbswartz: {'xyz': 'True'}14:13
jgriffithwhich is probably good14:13
jgriffithI keep trying to like gertty but I just can't get myself to do it14:14
smcginnisbswartz: The first thing we talked about for feature branches was whether that could be used for redesigning some of our cinder-nova interaction.14:14
smcginnisbswartz: But then when we synced up with nova they weren't interested in that. They just wanted to use microversions and the regular process.14:15
*** alonma has quit IRC14:15
johnthetubaguyits more we have never had a successful merge back into the tree, its probably best to just do WIP patches, unless that gets super hard for some other reason?14:16
bswartzsmcginnis: that makes sense -- the feature branch idea only came up because we thought nova wouldn't test with unmerged code14:16
johnthetubaguydepends-on has made life a lot easier with these kinds of things14:17
smcginnisjohnthetubaguy: True14:17
jgriffithsmcginnis: bswartz FWIW I say merge the cinder code, I wrote it specifically so it could be merged without impacting anything14:17
smcginnisMy only concern is the constant rebasing and version  # changing on microversions.14:18
jgriffithin theory at least :)14:18
smcginnisjgriffith: +114:18
*** cknight has joined #openstack-cinder14:18
*** alonma has joined #openstack-cinder14:18
johnthetubaguyyou can just pick a really high one, and pick the real one when you are ready to merge14:18
bswartzjgriffith: what if nova finds a bug after its merged and you have to change the API?14:18
openstackgerritPhilipp Marek proposed openstack/cinder: DRBD: Disk-options for new resources
openstackgerritPhilipp Marek proposed openstack/cinder: DRBD: Don't put clients everywhere
openstackgerritPhilipp Marek proposed openstack/cinder: DRBD: Make calculation use more bits
openstackgerritPhilipp Marek proposed openstack/cinder: DRBD: Use the ClientHelper class to reduce amount of code
openstackgerritPhilipp Marek proposed openstack/cinder: DRBD: Avoid an extra resize later on
openstackgerritPhilipp Marek proposed openstack/cinder: DRBD: Do batch delivery
openstackgerritPhilipp Marek proposed openstack/cinder: DRBD: Report an error if a snapshot has vanished
openstackgerritPhilipp Marek proposed openstack/cinder: DRBD: Avoid duplicated volumes from the error path
*** akapil_ has joined #openstack-cinder14:19
smcginnisbswartz: Version bump.14:19
jgriffithI'd rather merge it and iterate on it in tree than branches or anything else.  johnthetubaguy yeah, I like the idea of just picking some non-existent micro-version that it'll never see.  But then again I hate micro-versions... so there's that14:19
johnthetubaguyjgriffith: but the alternatives are really way more terrible14:20
jgriffithbswartz: I think you're making this more difficult than it needs to be.  This isn't a "released" thing14:20
jgriffithjohnthetubaguy: meh... perhaps14:20
jgriffithjohnthetubaguy: I still dislike them :)14:20
johnthetubaguythe experimental header is what I wanted to allow iteration in tree14:20
jgriffithjohnthetubaguy: and that's not going to change14:20
jgriffithjohnthetubaguy: bswartz smcginnis so this case is the one case that we felt the branch model would work well for14:21
jgriffithI have to admit though, the more I think about it the more I wonder "who" if anybody would be pulling that and running it other than dev teams anyway14:21
johnthetubaguyits just the merge back into master will be painful, and you can just checkout the WIP patches14:21
*** akapil has quit IRC14:22
*** rdx_matt has quit IRC14:22
jgriffithjohnthetubaguy: I get it, for sure.  Don't get me wrong, I'm not debating it14:22
*** alonma has quit IRC14:22
jgriffithjohnthetubaguy: if I had a better feel for exactly how it would make life better that might be different14:22
*** rdx_matt has joined #openstack-cinder14:22
*** rlrossit has quit IRC14:23
*** akapil_ has quit IRC14:23
johnthetubaguyI mean if a branch is better, I am all for it, but yeah, not sure it helps here14:23
*** lprice has quit IRC14:23
jgriffithjohnthetubaguy: yeah... I think we both are kinda in the same place on that.14:23
*** andymaier has quit IRC14:25
*** alonma has joined #openstack-cinder14:25
avishaythe CI instability is killing me... at the end of a 6 hour run in the gate hit the flaky unit test bug.  now did "reverify no bug" and it's now in the check queue for some reason... :-/14:26
smcginnisavishay: Not sure why it gets booted to the check queue some times. Yeah - pretty painful right now.14:27
*** liverpooler has joined #openstack-cinder14:27
*** rdx_matt has quit IRC14:27
avishaysmcginnis: bummer14:28
*** alonma has quit IRC14:30
*** alonma has joined #openstack-cinder14:33
*** diablo_rojo has joined #openstack-cinder14:33
*** alee has quit IRC14:34
*** alonma has quit IRC14:38
*** cgalan has joined #openstack-cinder14:39
*** alonma has joined #openstack-cinder14:41
*** xiexs_ has quit IRC14:41
*** ebalduf has quit IRC14:42
*** pbandark has quit IRC14:45
*** alonma has quit IRC14:46
*** xinli has joined #openstack-cinder14:46
*** bardia has joined #openstack-cinder14:46
*** alonma has joined #openstack-cinder14:48
*** abhitechie has joined #openstack-cinder14:48
*** enriquetaso has joined #openstack-cinder14:49
*** ebalduf has joined #openstack-cinder14:50
*** bardia has quit IRC14:51
smcginnis_alastor_: Can you give me that ci-status command line you used again. When you have a moment.14:51
*** alonma has quit IRC14:53
*** lprice has joined #openstack-cinder14:55
Dinesh_Bhorsmcginnis: Hi, Could you please take a look at it ?
smcginnisDinesh_Bhor: Sure, got it open in a tab.14:58
*** nkrinner has quit IRC14:58
*** coolsvap_ is now known as coolsvap14:59
_alastor_smcginnis: Sure here you go: third-party-ci-tools/monitoring/ci-status/ --all -t 2 --is-reporting --failures --number-of-reports14:59
_alastor_smcginnis: And here's my .gerritqueryrc14:59
*** jistr is now known as jistr|call15:00
Dinesh_Bhorsmcginnis: Thank you so much15:00
*** nkrinner has joined #openstack-cinder15:00
smcginnis_alastor_: Awesome, thanks.15:01
_alastor_smcginnis: Just remember to run with just the --scrape-wiki and --force flags first to make sure your cache is updated15:01
*** alonma has joined #openstack-cinder15:04
*** arecknag has quit IRC15:06
*** nkrinner has quit IRC15:08
*** alonma has quit IRC15:09
*** liverpooler has quit IRC15:10
xinli@smcginnis: I hit one issue, after pulling the master tree, and kick off "git -e py27", the unit test hang on for 30 minutes, how can I debug the issue?
xinli@smcginnis: this is just a pure unit test, only test my sandbox from master tree, no any ptach code.15:13
*** mtanino has joined #openstack-cinder15:14
*** pcaruana has quit IRC15:17
smcginnisxinli: There are issues that are being looked into with hanging unit tests. Just Ctrl-C and try again.15:17
smcginnisYay, privsep patch is on its way!
*** laughter_ has quit IRC15:18
xinli@smcginnis: got it, thanks15:19
*** laughterwym has joined #openstack-cinder15:19
*** laughterwym has quit IRC15:19
*** nkrinner has joined #openstack-cinder15:22
xinli@smcginnis: I take a look at this one:, does it relate to the unit test failure?15:22
*** iceyao has joined #openstack-cinder15:22
*** alonma has joined #openstack-cinder15:23
*** rcernin has quit IRC15:24
*** yangyapeng has quit IRC15:26
smcginnisxinli: No, that's not related.15:26
*** GB21 has joined #openstack-cinder15:27
xinli@smcginnis: now re-run the cmd: "tox -e py27"15:27
*** alonma has quit IRC15:27
*** mjura has quit IRC15:28
xinli@smcginnis: this time , it passed, how can I avoid this situation?15:30
*** alonma has joined #openstack-cinder15:31
*** iceyao has quit IRC15:32
*** yangyapeng has joined #openstack-cinder15:33
*** ebalduf has quit IRC15:34
openstackgerritMerged openstack/cinder: NetApp: Refactor unit tests
*** jistr|call is now known as jistr15:35
*** alonma has quit IRC15:35
*** ntpttr__ has joined #openstack-cinder15:35
*** ducttape_ has quit IRC15:37
*** Lee1092 has joined #openstack-cinder15:37
*** alonma has joined #openstack-cinder15:38
*** karthikp_ has joined #openstack-cinder15:39
*** bardia has joined #openstack-cinder15:39
*** rdx_matt has joined #openstack-cinder15:40
*** rdx_matt has quit IRC15:40
*** rdx_matt has joined #openstack-cinder15:40
*** faiz89 has quit IRC15:42
*** alonma has quit IRC15:43
*** david-lyle has joined #openstack-cinder15:43
*** liverpooler has joined #openstack-cinder15:43
*** diogogmt has joined #openstack-cinder15:44
*** xyang has joined #openstack-cinder15:44
*** alonma has joined #openstack-cinder15:46
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Use Live Volume for replication
*** alonma has quit IRC15:50
SwansonInfra queue is in the lowish 200s today!15:53
*** alonma has joined #openstack-cinder15:54
*** jwcroppe has quit IRC15:55
*** cgalan has quit IRC15:57
*** hemnafk is now known as hemna15:57
*** alonma has quit IRC15:58
bardiacan anyone find the failure in this:
*** leeantho has joined #openstack-cinder16:00
*** e0ne has joined #openstack-cinder16:01
*** tyrola has joined #openstack-cinder16:05
*** avishay has quit IRC16:05
*** tyrola_ has joined #openstack-cinder16:10
*** tyrola_ has quit IRC16:11
*** ducttape_ has joined #openstack-cinder16:12
*** jwcroppe has joined #openstack-cinder16:13
*** itlinux has quit IRC16:13
*** tyrola has quit IRC16:13
*** rdx_matt has quit IRC16:14
*** nkrinner is now known as nkrinner_afk16:14
*** slunkad_ has quit IRC16:15
*** sayalilunkad has joined #openstack-cinder16:17
*** cgalan has joined #openstack-cinder16:21
*** sayalilunkad has quit IRC16:25
*** Apoorva has joined #openstack-cinder16:25
*** crose has quit IRC16:26
*** david-lyle has quit IRC16:28
*** Thambirajan has joined #openstack-cinder16:32
*** Gao has quit IRC16:36
*** Lallana has joined #openstack-cinder16:37
*** sayalilunkad has joined #openstack-cinder16:38
*** mvk has quit IRC16:39
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Splitting Out Connectors from
*** Suyi has joined #openstack-cinder16:45
karthikp_Please could somebody review my patch:
openstackgerritJay Conroy proposed openstack/cinder: Fix volume upload-to-image for vhd disk-format
JayConroyCould I get reviews on my patch please? Thanks! :)16:50
*** akapil has joined #openstack-cinder16:51
*** Apoorva_ has joined #openstack-cinder16:53
*** alonma has joined #openstack-cinder16:54
*** Apoorva has quit IRC16:56
*** akapil has quit IRC16:56
hemna_alastor_, so what field(s) should I add ?16:58
*** e0ne has quit IRC16:58
*** alonma has quit IRC16:58
hemna_alastor_, so for our CI the name is "HPE Storage CI", the wiki page/name is HPE_Storage_CI, the gerrit account is 'hp-cinder-ci'17:00
_alastor_hemna: So when doing Gerrit query filters, it's really useful to have the account that posts for the CI as well as the exact job name for the CI.  Eg.  datera-ci is my account name, datera-dsvm-full is the job name17:00
hemnacan't we lookup the job name ?17:00
_alastor_hemna: That's what I was using the wiki for17:00
hemnaI'd rather not put a bunch of attributes in each drier17:00
*** nherciu has quit IRC17:00
hemnabut just exactly the required minimum field or fields.17:01
hemna1 or 217:01
_alastor_hemna: Ideally it would just be some sort of dictionary containing the necessary keys17:01
_alastor_hemna: but job name is more useful than account name because one account can post for multiple jobs17:01
hemnacan we find everything from the gerrit account id + CI name "17:01
hemnaCI_GERRIT_ID = 'hp-cinder-ci'17:01
hemnaCI_NAME = 'HPE Storage CI'17:01
hemnaaren't the job names defined in the infra somewhere ?17:02
_alastor_hemna: Not any place that isn't as out of date as the wiki :)17:02
smcginnisjob names can be whatever you want to call them.17:02
hemnasmcginnis, sure, but I'm trying to find out if they can be looked up base on gerrit account id and or CI name17:02
hemnaso we don't have to put a bunch of shit in each driver17:02
smcginnisNot that I'm aware of.17:03
hemnawhich also has to be maintained on the wiki, etc.17:03
_alastor_hemna: I agree that they should be tracked automagically somewhere by infra, but I haven't found any place with that info yet17:03
smcginnisWe can just have the script access the CI logs for each job and grep the c-vol log for to see which driver is loaded. :D17:03
* smcginnis is joking if that wasn't clear17:04
hemnathe problem is that the job names can change17:04
_alastor_smcginnis: HA, because I was having a bit of a heart attack :)17:04
hemnabased upon the internal needs of the CI network, etc17:04
hemnawe change our job names17:04
hemnabecause we move it to a new host, or array etc17:04
smcginnisHonestly though, that's probably the only consistent and accurate way to tell what any given job is testing. :/17:04
hemnawe test multiple arrays (different firmware, configurations etc)17:04
hemna_alastor_, so I suppose the job names can be put on the wiki17:06
*** faiz89 has joined #openstack-cinder17:06
_alastor_hemna: That would certainly be nice17:06
hemnathe driver has the wiki/CI name17:06
hemnathe wiki has the gerrit account name17:06
hemnaso can't we just put the driver CI wiki name ?17:07
hemnaCI_NAME = 'HPE_Storage_CI"17:07
hemnawhich is the exact name of17:07
_alastor_hemna: That's pretty close to what I currently do, but it still requires keeping the wiki up-to-date for all drivers17:07
hemnawe'd have 2 drivers that point to that CI_NAME17:08
hemnawould it make sense17:08
_alastor_hemna: We also discussed a bit of this at the mid-cycle.  Adding the mapping to the wiki would mean I could just get everything I need from there17:08
hemnato have some mapping in tree17:08
hemnaI'd rather have 1 place to keep all the shit up to date17:09
hemnaotherwise it'll always be out of sync17:09
*** rdx_matt has joined #openstack-cinder17:10
_alastor_hemna: Apparently adding a "Jobs" field to the wiki requires changing the template in infra17:10
_alastor_hemna: which I hear is a pita17:10
*** rdx_matt has quit IRC17:11
*** rdx_matt has joined #openstack-cinder17:11
_alastor_But adding it would mean I have everything I need, provided it's up-to-date17:11
hemnatheoretically, we could have a mapping in tree17:12
hemnafor all of the info17:12
hemnabut...that requires updating, etc.17:12
hemnaso you need the job names to track how often they are reporting pass/fail?17:12
*** tesseract- has quit IRC17:13
*** ebalduf has joined #openstack-cinder17:13
_alastor_hemna: I need job names so I can separate results for CIs that report multiple jobs on the same account17:13
_alastor_hemna: Otherwise they get lumped together and the results look weird17:14
*** links has joined #openstack-cinder17:15
*** sdake has quit IRC17:15
_alastor_hemna: Here's my current .gerritquerycache file, so you can see what I scrape:
DuncanTAt least if it's in tree we can see it's updated during driver addition/removal. I prefer that to the wiki (and there's talk of infra dumping the wiki anyway)17:16
*** sandanar has quit IRC17:16
_alastor_DuncanT: I agree it should be in tree, as long as it has comparable fields to the wiki17:17
DuncanTYup. Which might come back to the yaml file. We'll know when it's out of date since the tool will flag the missing CI reports17:18
patrickeast+1 for in tree, if we pick a yaml or json file with required fields we can validate changes to it too17:18
patrickeastforce all the fields to be there17:18
* patrickeast is just catching up on irc from the meeting and ealier17:19
DuncanTI can turn a gerritquerycache into yaml easily enough17:19
_alastor_DuncanT: the field that's missing from the gerritquerycache is the job field, because I had no place to get it :)17:20
*** david-lyle has joined #openstack-cinder17:20
DuncanT_alastor_: Ok, I don't mind trying to fill that in by hand once17:21
*** david-lyle has quit IRC17:23
*** david-lyle_ has joined #openstack-cinder17:23
*** david-lyle_ is now known as david-lyle17:24
*** david-lyle_ has joined #openstack-cinder17:25
erlon_alastor_: not followed the above chatting, but have you considered to have a standard to the job names?17:26
erlon_alastor_: for example, the NFs CI JOB wound need to be: NfsDriver-dsvm17:26
_alastor_erlon: I'm fine with a standard, but it seems like hemna's CI changes job name based on location?17:26
patrickeasterlon: i think trying to enforce that would be wayyyy harder than just keeping a map of what ci systems have for job names17:27
*** ebalduf has quit IRC17:27
erlon_alastor_: then would be HPE3PARFCDriver-mylocation17:27
erlonpatrickeast: but way more reliable17:28
patrickeasterlon: yea, if everyone would go change their jobs names17:28
erlonpatrickeast: we have enforced people to use CIs which has proven to be hard but possible17:28
patrickeasteh sort of17:29
*** david-lyle has quit IRC17:29
* patrickeast might just have a skewed outlook on it17:29
erlonpatrickeast: we create a list based in the names, if the maintainer does not change the name, than the Driver will be show in the removal list17:29
*** Apoorva_ has quit IRC17:30
patrickeastyea, but by when?17:30
*** Apoorva has joined #openstack-cinder17:30
patrickeastif we do it now, to give fair warning it wouldn't be till like N-3 when we yank them17:30
patrickeastwhich would probably mean early O17:31
patrickeastmaybe we do both17:31
*** xyang has quit IRC17:31
patrickeaststart forcing a naming convention17:31
patrickeast*and* start a mapping17:31
erlonpatrickeast: the removal policy its something to yet be defined, but I don't think that changing the jobs names is that hard17:31
_alastor_+1 ideal case :)17:31
*** ebalduf has joined #openstack-cinder17:31
smcginnisWe haven't even been able to get everone to adopt our consistent recheck triggers.17:32
_alastor_smcginnis: Although that's way harder to verify correctness on with how flakey the CIs are17:32
*** xyang has joined #openstack-cinder17:33
*** david-lyle_ has quit IRC17:34
*** ccesario has quit IRC17:34
*** david-lyle_ has joined #openstack-cinder17:34
erlonsmcginnis: my question is how was possible to make everyone to adopt CI, or at least have something to say its a CIs, and its not possible to change the CI names17:34
smcginnisAnd really, if we start reporting drivers for removal based on name mismatches - that will get folks to fix the names pretty darn quick.17:35
erlonsmcginnis: yeap, it work as magic, and in one whay or another, Its a way to know who its interested actively maintain theier drivers17:36
openstackgerritXinli Guan proposed openstack/cinder: Add functional tests for nested quotas
hemna_alastor_, ping17:38
erlonsmcginnis: people that don't have a reporting CI will be the one that won't give feedback17:38
hemnaso, I think what we should do is add a new field to the wiki page17:38
hemnaand that field is job names17:38
hemnaand it's a dictionary17:38
smcginnisInfra controls that template, and last time someone from Cinder tried to add something to the template they were not too happy about it.17:39
smcginnisBut there is freeform text area at the bottom that can be used for anything outside of the template.17:39
smcginnisWe could grep it out of there.17:39
hemna{'cinder': ['3par-iscsi-eos10', '3par-fc-eos10'], 'manila': ['3par-manila-eos10'], 'os-brick': ['3par-iscsi-brick-eos10']}17:39
smcginnisOr talk to infra about modifying the template and see if they would be open to it.17:39
hemnaas we have jobs for multiple projects17:39
smcginnishemna: Ooo, that would be nice.17:39
openstackgerritlakshman proposed openstack/cinder: Support manage/unmanage and extra-specs in K2
hemnaand I can update the driver code to add the CI name17:40
hemnaand modify the cinder/tools/ to output a python dict of the driver name and ci name17:41
*** david-lyle_ has quit IRC17:42
smcginnishemna: Actually, there's an idea too. If/when my patch goes through, all that will be posted in our docs:
hemnacan we get someone to update the wiki template ?17:43
smcginnisThough probably better to extract from the code than scrape the web page, now that I thought more than 2 seconds about it. :)17:43
hemnasmcginnis, that looks like it's directly from the cinder/tools/ ?17:43
smcginnishemna: Yep17:43
hemnaI was going to add a command line switch to that script17:43
hemnato output a dict instead17:43
hemnathe dict would have everything it currently outputs but the addition to the CI_NAME attribute on the driver17:44
smcginnishemna: That would be useful.17:44
hemnathat CI_NAME is the name of the 3rd party CI wiki page17:44
*** rdx_matt has quit IRC17:44
hemnaok let me hack this up and see what it looks like.17:45
smcginnisWe could probably have a few different output formatters: rst, dict, yaml17:45
hemnafor now I'll just hack the crap out of it and see if it's the direction we want to go17:45
_alastor_hemna: If I use "hp-storage-ci" instead of "hp-cinder-ci" as your gerrit account name I show your r17:45
_alastor_CI as reporting17:46
*** sdague has quit IRC17:46
hemnahp-storage-ci isn't anything though17:46
hemnathere is no association to hp-storage-ci anywhere afaik17:46
_alastor_hemna: Weird.  On reviews your CI shows up as "HPE Storage CI"17:47
hemnayah that's correct17:47
hemnathat's the CI name17:47
hemnaand the wiki page is that replacing spaces with _17:47
_alastor_hemna: The "username" in the response is "hp-storage-ci", which maps to the account used to post the comment17:52
hemnaleeantho, is our wiki page for the gerrit id out of date?17:53
hemnaleeantho, ^^^17:53
hemna_alastor_, thanks :)17:53
_alastor_hemna: np.  I was pretty perplexed as well :)17:53
*** sdague has joined #openstack-cinder17:54
*** links has quit IRC17:54
smcginnisrootwrap/privsep is merged!
hemnasmcginnis, w00t!17:55
smcginnishappy dance time17:55
* hemna is trying hard to break the world.17:56
smcginnisDang, jobs still sitting Queued for that one.17:56
smcginnisI'll watch for it.17:56
hemnais there a way to do a static class attribute method ?17:57
smcginnisGr, off to another meeting.17:57
hemnato force folks to implement it17:57
DuncanThemna: You don't need to instantiate a class to get an attribute17:58
*** mvk has joined #openstack-cinder17:58
hemnaDuncanT, yah I know17:58
DuncanTOh, to force it17:59
hemnabut I want to ensure that a driver implements the attribute17:59
hemnaand is forced to17:59
hemna(for new drivers/refactors, etc)17:59
*** ntpttr__ has quit IRC17:59
hemnaanyway, I can just add CI_NAME for now17:59
hemnaand have the generate_drivers_tool puke out some error if they don't see that attribute18:00
hemna_alastor_, ok we updated our wiki to fix the gerrit account name18:00
hemna_alastor_, the tool should probably bitch about that18:00
_alastor_hemna: True, not sure how I'd go about it though.  I could just have it complain about everything.  Like the weather in norcal at the moment :)18:02
hemnayah it's freaking hot here18:02
hemnasupposed to be 108F here in Roseville, CA today18:03
DuncanTWe;ve dropped to under 100F to past week. Only 75% humidity too18:05
hemnaDuncanT, get out the jackets!18:06
patrickeastits pretty nice here :P18:06
*** jamielennox is now known as jamielennox|away18:10
SwansonI'm not saying the Russians have a weather control machine. But if they do I think if they could turn the weather to a nice dryish 85 that would be great.18:11
jungleboyjhemna: Thankfully clouds have come over so it is only 80 in my un-airconditioned house.18:15
openstackgerritMerged openstack/cinder: Map volume/snapshot manage extensions to v3
*** ociuhandu has quit IRC18:21
openstackgerritIldiko Vancsa proposed openstack/cinder: [WIP] Add functional test to new API functions
*** zul_ has joined #openstack-cinder18:25
*** zul has quit IRC18:25
hemna_alastor_, smcginnis ^^18:27
*** abhitechie has quit IRC18:27
_alastor_hemna: Is that taken from information in tree?18:29
jungleboyjpatrickeast: Ping.18:29
hemna_alastor_, yah18:29
_alastor_hemna: it's missing a bunch of drivers like Datera :)18:30
hemnathat's only a short copy/paste18:30
_alastor_hemna: ah, gotcha.  Well, if we can get job info and gerrit account in there we'll be golden18:31
hemna_alastor_, try this
hemna(currently scrubbed the description)18:33
hemnaso that outputs a python list18:33
hemnawhich can be directly used.18:33
_alastor_hemna: I'll have to do some fuzzy matching with underscores/hyphens/spaces, but it might work18:36
hemnawhat's missing ?18:37
openstackgerritErlon R. Cruz proposed openstack/cinder: Fix 'driver does not honor clone/snapshot' bugs
erlonpots: ^~18:38
_alastor_hemna: jobname.  I need that to get results for specific jobs.  The driver itself isn't mentioned in the CI review posts18:40
potserlon: thanks!18:40
hemnaok well we need a new field on the wiki page18:40
hemnaand a dictionary18:40
hemna<hemna> {'cinder': ['3par-iscsi-eos10', '3par-fc-eos10'], 'manila': ['3par-manila-eos10'], 'os-brick': ['3par-iscsi-brick-eos10']}18:40
hemnathat would be the contents of the jobs field on the wiki18:41
_alastor_hemna: Yeah, something.  If it's a required attribute on the driver that would work too18:41
*** rdx_matt has joined #openstack-cinder18:41
hemnawe can't do that18:41
hemnajob names change frequently18:41
_alastor_hemna: Ah right, forgot18:41
*** rdx_matt has quit IRC18:41
hemnaand I don't want to force people to do code reviews to update that stuffs18:41
*** rdx_matt has joined #openstack-cinder18:42
*** alonma has joined #openstack-cinder18:42
smcginnishemna: I did verify - if you put a class method in the cinder.interface.volume_driver definitions it will cause the tox -e compliance tests to fail for drivers that don't have it defined.18:42
hemnaafter I get all of the drivers updated with a CI_NAME attribute18:42
hemnaI'll add that to the interface18:42
hemnaI have a lot of driver files to touch....18:43
*** ccesario has joined #openstack-cinder18:43
smcginnis96 of them, but the looks of it.18:43
hemnaI'm going to need everyone to review this patch18:44
*** ericksonsantos has quit IRC18:44
hemnabecause I might get the CI_NAME wrong for any given driver18:44
hemna*sigh*  EMC18:44
smcginnishemna: Well, if we start basing CI checking off of it, we'll find out pretty quick if any are wrong.18:44
hemnawtf is that18:45
hemnaI'm just guessing on this stuff18:45
*** ChubYann has joined #openstack-cinder18:45
smcginnisOpen source CoprHD driver that they still want to have a company name on. :/18:45
hemnano idea which CI is associated with it18:45
*** e0ne has joined #openstack-cinder18:45
hemnaCI_NAME = "unknown" ?18:46
hemnano clue18:46
hemnaxyang, ping18:46
*** e0ne has quit IRC18:47
smcginnisEMC CoprHD CI18:47
*** alonma has quit IRC18:47
hemna ?18:47
smcginnisI had to track it down to here:
*** smile-luobin has quit IRC18:48
smcginnisThat's the last patch it commented on.18:48
*** alonma has joined #openstack-cinder18:48
smcginnisThat's got to get fixed. It doesn't look like it's commenting on all patches.18:48
hemnaand it has no wiki page18:49
*** smile-luobin has joined #openstack-cinder18:49
hemnaor is it this one ?
hemnano idea18:49
xyang1hemna: hi18:50
hemnawhat CI is associated with the coprHD drivers ?18:50
hemnathere are 2 of them18:50
smcginnishemna: No, not that one.18:50
smcginnisThat's XtremeIO18:50
xyang1let me check18:50
smcginnisI left comments in their forum for the wiki and the reporting.18:50
hemnathose guys18:50
hemnathe and fc.py18:51
hemnaoh and in there18:51
hemnaxyang1, and the wiki url ?18:51
openstackhemna: Error: "!" is not a valid command.18:51
hemna* S M H *18:51
xyang1I created it initially for vipr, that is why18:51
*** ntpttr__ has joined #openstack-cinder18:52
xyang1before there was coprhd18:52
smcginnisxyang1: All of those EMC entries should probably be updated to actually have their names.18:52
*** avishay has joined #openstack-cinder18:52
xyang1which name is not right?18:52
hemnaI have no idea which wiki page is associated with any EMC driver18:52
hemnaso confusing18:52
*** alonma has quit IRC18:53
smcginnisxyang1: They should have the name of the CI so it's easy to read, not the cryptic account name.18:53
smcginnisEMC XXX CI18:53
avishaysmcginnis DuncanT hemna scottda_vacation etc - thanks for your help with the list manageable patches!18:53
smcginnisemc-vipr-ci = EMC CoprHD CI18:53
smcginnisavishay: ;)18:53
hemna ?18:53
xyang1I'll take a look18:54
smcginnisxyang1: Oh, wanted to point out too that the new VNX driver doesn't appear to be defining VERSION right. It doesn't show up.18:54
xyang1is it18:54
smcginnisAt least here:
hemnaI need the wiki page name for each of the 3 drivers in the coprhd dir18:54
smcginnishemna: Oh, and HPEXPFCDriver isn't either.18:55
*** faiz89 has quit IRC18:55
*** alonma has joined #openstack-cinder18:56
*** david-lyle_ has joined #openstack-cinder18:57
hemnaoh man18:57
hemnaI can't believe I took this on...18:57
smcginnisHaha! :D18:58
hemnaI really hope this is going to help18:58
smcginnishemna: I think it will. So far at least the idea seems sound.18:59
*** ebalduf has quit IRC19:00
*** alonma has quit IRC19:00
hemnaxyang1, I'll need you to review all of the EMC driver changes once I post this19:01
hemnait's hard to make sense of them19:01
*** lprice has quit IRC19:01
xyanghemna: where is your change?19:02
hemnare: once I post this19:02
hemnaI have a ways to go19:02
xyanghemna: is this a code change in cinder or wiki change?19:02
xyang@smcginnis: they defined version here:
hemnasmcginnis, I'm noticing several drivers that don't have a VERSION defined in them19:03
xyangsmcginnis: should be here?19:03
hemnaEMCVNXDriver doesn't have one19:03
hemnawas just looking at that19:03
xyangso the version was in iSCSI and FC drivers in the past.  I think that is why they added it in adapter instead of the driver file19:04
smcginnisxyang: Ah, that explains it. It's a class static variable, so the child classes don't end up with a driver.VERSION variable.19:04
xyang@smiginnis: should it be moved to
hemnahere, let me push up a WIP of what I have19:04
*** RageLtMan has joined #openstack-cinder19:04
smcginnisxyang: Not sure what you mean.19:04
RageLtManHi folks, anyone here who commits regularly? i have found a bug and understand the remedial vector, but dont want to get involved in the mess of trying to submit changes to openstack19:05
smcginnisxyang: Each driver class would need it defined.19:05
smcginnisRageLtMan: What's the issue?19:05
smcginnisYou can always file a big with the information and let someone else pick it up.19:06
xyang@smcginnis: you said you can't find the version automatically?  So it should be moved from to
*** alonma has joined #openstack-cinder19:06
RageLtManThe bug: creating an instance from an image which has hw_disk_bus=scsi set result in Cinder recording the mountpoint of the boot volume as /dev/vda in the cinder DB's volume_attachment table19:06
smcginnisxyang: Yeah, I believe so.19:06
RageLtManwhen attempting to attach a volume to the instance subsequent to the original create call, it does not record a warning or error, just puts the volume back in available state19:06
xyangsmcginnis: vnx/ has the driver definition but version is in vnx/ now.  There is only a docstring entry for version in vnx/driver.py19:06
*** david-lyle_ is now known as david-lyle19:07
RageLtManif the mountpoint in the database is manually repaired to read /dev/sda then cinder attach ops work fine19:07
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_NAME to all drivers
RageLtManso, cinder needs to properly record the mountpoint at instance creation based on the type of disk presented by the bus configured for nova19:07
*** fifieldt has quit IRC19:08
smcginnishemna: Don't we get that from Nova? ^^19:08
hemnasmcginnis, yes19:08
hemnaafaik cinder doesn't even use it anywhere19:08
avishayhemna: +119:09
smcginnishemna: Yeah, I didn't think so either.19:09
RageLtManI quickly skimmed the relevant sources, and it appears the mountpoint param passed to volume-attach is ignored19:09
xyanghemna: the CI_name is the account name?19:09
hemnaxyang, no19:09
hemnaxyang, read the commit message :)19:09
xyanghemna: smcginnis just asked me to spell them out19:09
avishayseems weird that the behavior in nova changes according to vda or sda19:09
*** haplo37__ has joined #openstack-cinder19:09
hemnaCI_NAME = "HPE_Storage_CI"19:09
hemnaxyang, the purpose of CI_NAME is to be a lookup to the wiki page19:10
smcginnisxyang: So as it is right now, CI_NAME would be account name because that's how they are defined on the wiki.19:10
*** alonma has quit IRC19:10
smcginnisBut normally it's the actual CI name, not the account name.19:10
avishaydoes cinderclient have support for microversions yet?  i see v3 but not sure how microversions work there...19:10
hemnathat's what it is19:10
hemnaavishay, yes I believe so19:10
_alastor_So if I understand it correctly:  in tree drivers --> CI_NAME --> wiki lookup --> gerrit account/jobname?19:10
hemna_alastor_, yah19:11
_alastor_hemna: kk19:11
hemna_alastor_, does that work?19:11
xyangsmcginnis: but we can't have space there though19:11
_alastor_hemna: should work just fine.  As long as by the end of it I have gerrit account and jobname19:11
hemnayou can pull that and run the cinder/tools/generate_driver_list.py19:11
hemnaand get the output19:11
avishayhemna: if i want to add support for an API that is in 3.8, how do i express that?  what if support for things before 3.8 weren't added?19:11
hemna./ --format dict19:11
hemnait'll poop out a giant list of dicts.19:11
smcginnisxyang: Check out all the other ones. The spaces just end up _19:12
xyangsmcginnis: so should I change now that hemna has submitted the patch?  they will go out of sync if I change:)19:12
hemnaavishay, I think scottda added a docs page on how to use the client w/ a specific microversion19:12
smcginnisavishay: Merged in April.19:12
openstackgerritJustin A Wilson proposed openstack/cinder: Fixed request/response parameters in doc
hemnaI think you have to ask for the version # at client instantiation19:13
hemnaanyway, _alastor_, smcginnis, xyang if you can look at that CI_NAME patch for me19:13
hemnaI have more to add19:13
hemnabut I need to grab some lunch19:13
_alastor_hemna: will do19:13
*** alonma has joined #openstack-cinder19:13
smcginnisxyang: Maybe... Would be nice to have that match the rest and have the display name and not the account name.19:13
hemnaI just want to make sure that's the direction we want to go before I go hacking up more drivers19:13
smcginnishemna: Thanks for working on that. I'll take a look soon.19:13
smcginnishemna: +119:14
hemnaadd review comments for incorrect CI_NAME values19:14
hemnaok lunch.....19:14
avishaysmcginnis hemna  thanks19:14
*** rdx_matt has quit IRC19:15
avishaysmcginnis: OK so I see the version in the client is still 3.0.  I can't just add "list manageable" and bump it to 3.8 without adding all the changes in 3.1-3.7, right?19:15
smcginnisavishay: I believe it's just an argument to specify which microversion you want.19:16
avishaysmcginnis: but if i specify 3.8 i expect 3.1-3.7 to be included as well, no?19:17
smcginnisCLI or as a library. CLI you just are making an individual call, so it should be able to just impact the one microversion you want.19:18
* smcginnis is looking for client docs19:18
*** alonma has quit IRC19:18
*** fifieldt has joined #openstack-cinder19:19
avishayso sometimes the client will say it is 3.0 and sometimes 3.8?19:19
*** xinli has quit IRC19:20
*** IlyaG has joined #openstack-cinder19:20
*** coolsvap has quit IRC19:21
smcginnisavishay: So 3.0 is the base for everything and the default if no other specified.19:24
smcginnisavishay: If I remember right, you add "--os-volume-api-version 3.8" to make that specific call to the requested microversion.19:25
smcginnisI haven't had need to really use it much, so a little rusty on that.19:25
*** sdake has joined #openstack-cinder19:25
smcginnisUnfortuntely Scott is on vacation right now, but he's done a lot as he was working on adding all that support.19:26
*** xyang has quit IRC19:26
avishaysmcginnis: yep i did that when testing v3 APIs manually and it worked fine. i just don't understand how cinderclient is supposed to work. maybe i'll just wait until scott returns...19:26
*** xyang has joined #openstack-cinder19:26
xyangsmcginnis: can you check if this is correct?
smcginnisavishay: I would consider him the authority on that at this point. ;)19:26
avishaysmcginnis: yep he certainly helped me on the server part of it19:27
xyangsmcginnis: I have to recreate the page, can't just rename it. want to make sure it is correct before changing all of them19:27
smcginnisxyang: Nice! Yeah, I think that looks better.19:27
xyangsmcginnis: ok, thanks19:27
smcginnis_alastor_: Is that what you would expect scraping the wiki? ^^19:27
*** xinli has joined #openstack-cinder19:28
*** ebalduf has joined #openstack-cinder19:29
*** Yogi1 has quit IRC19:30
avishaysmcginnis: OK I see now - e0ne pushed a patch to support the change in 3.119:32
avishaywill work off that19:33
_alastor_smcginnis: Yes, than page has everything I need other than jobname, but we're still debating what that should look like19:33
*** ccesario has quit IRC19:33
*** jungleboyj has quit IRC19:33
smcginnisavishay: Excellent19:33
smcginnis_alastor_: Cool, thanks.19:33
*** alonma has joined #openstack-cinder19:34
xinli@xyang1: I hit this issue again, all the test paseed, but the result shows error.
SwansonMe, too.19:35
*** alonma has quit IRC19:38
*** avishay has quit IRC19:39
*** ebalduf has quit IRC19:41
hemnasmcginnis, is a Dell driver ?19:41
erlonxinli: I got that too, just recheck until it passes :/19:42
*** sdake has quit IRC19:43
xinlierlon: thx.19:43
*** thiagolib has joined #openstack-cinder19:46
Swansonhemna, yes19:46
hemnamv dell/19:47
SwansonI'm not having the filthy driver in with my code.19:48
smcginnisHey now. :)19:49
*** ebalduf has joined #openstack-cinder19:50
*** ntpttr- has quit IRC19:50
smcginnisWOw, I think it's been so long since the last time the X-IO CI reported that the lastcomment script isn't even returning anything.19:50
smcginnisOr I'm screwing something up.19:50
*** ntpttr- has joined #openstack-cinder19:51
smcginnisYeah, I think I'm screwing something up.19:51
*** timcl has quit IRC19:52
hemnaHitachi drivers also missing VERSION19:52
*** sdake has joined #openstack-cinder19:53
*** jdurgin has quit IRC19:53
xyanghemna: I updated EMC CI names19:54
*** ntpttr__ has quit IRC19:54
hemnaxyang, thanks :)19:54
hemnaI'm sure half of my CI_NAME entries are wrong19:54
smcginnisAnyone know where the "official" lastcomment script is? I'm using the one here:
smcginnisBut doesn't seem like it's working right.19:54
smcginnisOr I'm really messing something up.19:55
xyanghemna: you were right.  It's just that I changed the names on wiki to be consistent with others:)19:55
*** lprice has joined #openstack-cinder19:55
patrickeastsmcginnis: it was jogo's project iirc
smcginnispatrickeast: That's it!19:56
hemnaIBM drivers missing VERSION19:57
hemnaInfortrend drivers missing VERSION19:59
*** lprice has quit IRC19:59
*** ccesario has joined #openstack-cinder20:00
smcginnisSO this is a weird one I would like to get some input on...20:00
*** porrua has quit IRC20:00
smcginnisThe DotHill drivers aren't reporting.20:01
smcginnisIf folks were around, there was some discussion on these.20:01
smcginnisThey are OEMd.20:01
smcginnisSo there IS a Lenovo driver that is basically just inherited from DotHill that is reporting CI status.20:01
smcginnisSo indirectly it is reporting CI results for that code.20:01
smcginnisBut at the time, when we agreed to allow three different drivers with one code base, we stated we needed reporting CI for all.20:02
hemnaew I see that20:02
hemnaLenovoFCDriver is just a derivation of DotHillFCDriver20:02
smcginnisOne other one too, but can't remember offhand which one.20:02
smcginnisSince been acquired by Seagate:
*** ebalduf has quit IRC20:03
smcginnisFor $694 million I'll run CI for them. ;)20:03
hemnaI'll do it for half that20:03
smcginnisHey, stay out of my way man! :P20:04
patrickeastpsh id do it for like 3 mil, plus T&M20:04
hemnafree market, bitches!20:04
smcginnisSo that's how it's going to be.20:04
hemnapatrickeast, cheapskate.  you get what you pay for!20:04
patrickeasthemna: it only has to run for 6 months at a time, then its contract renewal time XD20:04
patrickeastdon't want it working tooo smoothly20:05
smcginnisTechnically, it just has to run once every two months now.20:05
hemnaso...what to do about the lenovo ci ?20:05
smcginnisDoh! Did I write that out loud.20:05
hemnaI guess that's it ?20:06
smcginnishemna: I'm thinking leaving it for now? It's obviously not tested against the Dot Hill version of the box, but the Lenovo one is just a rebranded OEM, so kinda sorta it's being tested. :/20:07
hemnaI was trying to figure out what CI_NAME to put on it20:07
hemnanot sure about CI_NAME for lvm.py20:07
* hemna skips for now20:07
hemna300 drivers in there alone20:08
hemnaI hope they are all reporting under the same name20:08
akerrhemna: we are20:08
hemnaakerr, phew!  thanks20:08
*** Apoorva has quit IRC20:09
hemnanetapp drivers don't have VERSION either.....20:09
akerrdon't get me started20:09
akerr2 years I've been asking for a version scheme20:10
hemnaI can create one for you :P20:10
smcginnisDang, last reported 2016-03-18 20:23:25 (130 days, 23:47:11 old)20:10
smcginnisI'd say we're overdue for removal on that one.20:11
smcginnisThat was still in Mitaka20:11
*** rdx_matt has joined #openstack-cinder20:12
*** rdx_matt has quit IRC20:12
*** rdx_matt has joined #openstack-cinder20:12
*** merooney has quit IRC20:13
*** smile-luobin_ has joined #openstack-cinder20:14
*** smile-luobin has quit IRC20:14
openstackgerritSzymon Wróblewski proposed openstack/cinder: quobyte volume driver should use DLM
*** akshai has quit IRC20:15
*** Lallana has quit IRC20:17
*** cdelatte has quit IRC20:17
*** Lallana has joined #openstack-cinder20:19
*** ntpttr__ has joined #openstack-cinder20:20
*** ericksonsantos has joined #openstack-cinder20:20
*** mdenny has quit IRC20:20
openstackgerritSean McGinnis proposed openstack/cinder: Remove X-IO volume driver
*** mdenny has joined #openstack-cinder20:21
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS drivers refactoring
hemnaSynology driver20:24
hemnawas added June 23rd20:24
hemnaI don't see a wiki page20:24
smcginnishemna: You're going to need a beer after all this, aren't you?20:24
openstackgerritSzymon Wróblewski proposed openstack/cinder: Quobyte volume driver should use DLM
*** chris_morrell has quit IRC20:27
openstackgerritBen Swartzlander proposed openstack/cinder: Add stochastic weight handler to scheduler
_alastor_So if ci-status is now going to use cinder/tools/, that means it now has a dependency on cinder.  The cinder repository doesn't appear to be available as a pypi package though...20:30
*** JoseMello has quit IRC20:30
smcginnis_alastor_: Hmm, yeah, that's not good.20:30
SwansonAdded June 23rd and no Wiki page? That's the fault of our Commander in PTL. Thanks, smcginnis!20:30
smcginnisSwanson: Shut it!20:30
SwansonGonna be a bloodbath when DuncanTrump takes over this mess.20:31
*** ducttape_ has quit IRC20:31
*** aswadr_ has quit IRC20:31
_alastor_So one option is that I move the script into Cinder and just make it Cinder-specific20:31
openstackgerritSean McGinnis proposed openstack/cinder: Remove Tintri volume driver
_alastor_Or I could just add to the readme that you need to clone Cinder and develop install it into your virtualenv for it to work20:33
smcginnis_alastor_: Adds complexity, but avoids duplication.20:33
hemnaSwanson, lolz20:34
smcginnisBut we are the only ones that have the ability to generate a driver list, as far as I know.20:34
*** alonma has joined #openstack-cinder20:34
*** ntpttr__ has quit IRC20:35
*** ntpttr__ has joined #openstack-cinder20:36
_alastor_Or I could create a c-driver-ci-status service with a c-driver-ci-status-manager that will handle RPC calls to the service (Active-Active HA support pending of course)20:36
_alastor_I don't want to have to touch oslo enough to do that20:37
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file
hemnawho is ZTE ?20:37
erlonmtanino: ping20:38
mtaninoerlon: pong20:38
mtaninoerlon: hey :)20:38
*** alonma has quit IRC20:38
erlonmtanino: Hey Mitsushiro!20:38
mtaninoerlon: What's up?20:39
erlonmtanino: I know you have worked with qemu-img haven't you?20:39
*** faiz89 has joined #openstack-cinder20:39
mtaninoerlon: yes, in previous.20:39
mtaninoerlon: mu20:40
erlonmay be you can help, or know who20:40
erlonI'm trying to figure out an error on Cinder NFS Driver, I trying to create/delete snapshots from a snapshot chain. There are 3 situations:20:40
erlon1 - I create/deete the snapshots in Cinder (it uses qemu-img). It goes OK! I can delete and I can create snapshots.20:40
erlon2 - I create/delete the snapshots, in online mode (it uses libvirt). It goes Ok, as weel.20:40
erlon3 - I create the snapshots in Cinder (offline mode using qemu-img) and delete then in online mode (using libvirt), then it fails with this message:20:41
openstackgerritSean McGinnis proposed openstack/cinder: Remove Tegile volume driver
erlonlibvirtError: invalid argument: could not find image 'volume-8c58ee12-9ad0-4d71-ae29-e18399af6c8e.53a3cb87-c2fc-4e4e-988a-8ead73d753e0' in chain for '/opt/stack/data/nova/mnt/a3b4c6ddd9bf82edd4f726872be58d05/volume-8c58ee12-9ad0-4d71-ae29-e18399af6c8e20:41
erlonmtanino: any idea?20:41
*** alonma has joined #openstack-cinder20:41
mtaninoerlon: Let me see the NFS driver.20:44
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_NAME to all drivers
*** Apoorva has joined #openstack-cinder20:45
erlonmtanino: this cames from the nova side: nova/virt/libvirt/driver.py20:45
erlonmtanino: full stack error on n-cpu:
*** rdx_matt has quit IRC20:46
*** sdake has quit IRC20:46
hemnaok I think I have most of them20:46 is missing20:46
*** alonma has quit IRC20:46
hemna_alastor_, not sure what I should put for lvm.py20:46
hemnaso smcginnis didn't you say I could put a CI_NAME in the interface and it'd puke on drivers that don't have it ?20:47
openstackgerritSzymon Wróblewski proposed openstack/cinder: Scality volume driver should use DLM
_alastor_hemna: I dunno either.  I assume we have an LVM CI?20:48
hemnawell we do have CI for lvm20:48
hemnait's the standard tempest dsvm full20:48
mtaninoerlon: step 3 means that once you take a snapshot when the volume is offline(not attached) and then attach the volume to VM, then delete a snapthot, right?20:48
_alastor_Ha!  Yeah, I have no idea then.  I wasn't planning on the tool to be used for Jenkins stats.  You can just go look at jenkins :)20:49
hemna_alastor_, we have a few drivers like this though20:49
*** alonma has joined #openstack-cinder20:49
hemnathat are part of the standard check/gate20:49
hemnathat don't have a thirdparty systems wiki page20:49
erlonmtanino: yes20:49
*** ebalduf has joined #openstack-cinder20:49
hemnawould be nice if the tool worked for them as well20:49
smcginnishemna: I tested with adding a class method.20:50
smcginnishemna: Not sure if it can check that a variable is defined.20:50
hemnaoh class method20:50
smcginnishemna: Wouldn't CI_NAME = Jenkins and test name = test-dsvm-full or whatever it is?20:50
hemnaI dunno20:50
hemnathe tool would have to special case the Jenkins CI_NAME20:51
_alastor_hmmm, I don't think the name matters as long as it matches the wiki page.  Maybe just come up with a colloquial name for it like "Tempest LVM Full" or "That Shit That Always Fails For Me"20:51
*** cdelatte has joined #openstack-cinder20:51
hemnaor we create a Cinder_Jenkins wiki page w/ the list of jobs ?20:51
_alastor_hemna: I can always write a separate parser for that page.  It's not a problem20:52
hemnaup to you20:52
hemnaanyone else can push a new patch against that review to make changes20:52
_alastor_hemna: Better to have it all on one page if we can, so it's just a single page load20:52
_alastor_hemna: The longest part of doing all of this is just loading pages to scrape the wiki20:53
mtaninoerlon: I guess nova tries to remove online snapshot but acutually the snapshot is created via qemu-img at offline state so taht libvirt can't find any snapshot to remove, then you hit error?...20:53
*** alonma has quit IRC20:54
mtaninoerlon: I don't have much knowledge around remotefs drivers online/offline snapshot, just a guess.20:56
erlonmtanino: hmm, I don't know, I can be that but the error message should point that that operation were not allowed20:56
erlonmtanino: ok, no worries just asked to see if you had heard about something like that20:57
*** ducttape_ has joined #openstack-cinder20:57
erlonmtanino: but I got to this point and have no idea how to move. From that call, and on, nova uses the libvirtlib, which is C++20:58
mtaninoerlon: not python...20:59
erlonmtanino: yep20:59
mtaninoerlon: one question. if you detach the volume again, can you delete the snapshot?(this is offline snapshot delete)21:00
*** xinli has quit IRC21:01
erlonmtanino: yes, but if I have tried to delete it, attached, I have to set the status of the snapshot to avaiable21:01
mtaninoerlon: Ah,  I see21:02
erlonmtanino: it looks either a libvirt bug, or an incompatibility between qemu-img vs libvirt21:02
hemna_alastor_, smcginnis
*** ntpttr__ has quit IRC21:05
hemnacan see which ones are missing the CI_NAME21:06
mtaninoerlon: I guess "create at offline state but delete at online state" is not considered or are there drivers they can this operation?21:07
smcginnishemna: Nice21:08
hemnaso, what do we do with the drbd driver21:09
hemnaand blockdevice driver21:09
erlonmtanino: yes, all RemoteFS drivers  that implements snapshots should do that21:09
smcginnishemna: All the Jenkins tested ones?21:09
hemnasheepdog as well21:09
erlonmtanino: s/should/do/21:09
hemnaI put it as 'unknown'21:09
*** jdurgin has joined #openstack-cinder21:10
*** dustins has quit IRC21:10
erlonmtanino: don't know how they work though21:10
mtaninoerlon: unfortunately me too...21:11
*** ntpttr__ has joined #openstack-cinder21:11
openstackgerritSean McGinnis proposed openstack/cinder: Remove Scality volume driver
*** pauloewerton has quit IRC21:11
hemnaI guess that covers the sheepdog driver21:12
*** ebalduf has quit IRC21:13
hemna_alastor_, so......  CI_NAME = "Cinder_Jenkins"  ?21:13
hemnafor lvm, glusterfs, nfs, sheepdog, etc21:13
hemna(even though that Cinder_Jenkins wiki page doesn't exist)21:14
*** ntpttr__ has quit IRC21:14
hemnasmbfs as well21:14
openstackgerritSzymon Wróblewski proposed openstack/cinder: Smbfs volume driver should use DLM
mtaninoerlon: I couldn't find "create at offline state but delete at online state" test case in the tempest21:14
smcginnisIs there a driver that matches IBM NAS CI?21:14
smcginnisdiablo_rojo: Do you know? ^^21:15
hemnasmcginnis, not according to my patch :P21:15
_alastor_hemna: That works for me.  I could just report each of them as Cinder_Jenkins_XXXX21:15
smcginnisOK, didn't think so.21:15
*** zeestrat has joined #openstack-cinder21:15
erlonmtanino: yep, Im afraid that would break 'some' drivers :)21:16
hemnaI guess when jenkins doesn't exist, we can rename them to Cinder_zuul21:16
mtaninoerlon: ;)21:16
*** jamielennox|away is now known as jamielennox21:17
erlonmtanino: when I find out a way out here Ill add a test there just to see the what happens lol21:17
smcginnishemna: But isn't the reporting CI name "Jenkins"? Not sure why we can't just use that.21:17
mtaninoerlon: :)21:17
_alastor_smcginnis: It might get confusing with other projects all sharing the 3rd party wiki21:18
hemnasmcginnis, yah I didn't think we'd get away with creating a Jenkins page off of the 3rdPartySystems page21:19
smcginnisAh, OK. I see what you're saying now.21:19
smcginnisYou mean the world doesn't revolve around us!?21:19
*** faiz89 has quit IRC21:22
smcginnisOK, I think I have patches up for all non-compliant drivers.21:23
*** david-lyle has quit IRC21:25
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_NAME to all drivers
hemna   ^^21:25
smcginnisOK, I'm getting out of here and bringing my girls to Star Trek. So y'all tomorrow.21:27
hemnasmcginnis, w00t21:28
hemnahave fun21:28
mtaninosee ya!21:29
*** diablo_rojo has quit IRC21:31
hemnawtf happened to the sheepdog.py21:31
_alastor_yeah, just commented on it :)21:32
hemnal a m e21:32
*** xyang has quit IRC21:33
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_NAME to all drivers
hemnathat should be better21:34
hemnaborked blockbridge as well21:34
*** xyang has joined #openstack-cinder21:35
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_NAME to all drivers
* patrickeast is kinda late to the party21:36
patrickeastso why are we adding in strings to the wiki pages?21:36
patrickeastand not just the actual ci name?21:36
hemnathe tool is going to consume the output of the script  (
patrickeastnow depending on where you look my ci goes by: "Pure_Storage_CI", "Pure Storage CI", "pure-storage-ci", ""21:37
hemnathen use that to associate the driver to the data that's in the wiki page21:37
hemnaeveryone is different21:37
hemnawe are also going to put the job names in a new dictionary on the wiki page as well21:38
hemnaJobs {'cinder': ['3par-iscsi-eos10', '3par-fc-eos10'], 'manila': ['3par-manila-eos10'], 'os-brick': ['3par-iscsi-brick-eos10']}21:39
patrickeastyea, i guess i'm not a huge fan of the indirection21:39
hemnawell, job names change all the time21:39
hemnaat least for us21:39
patrickeastthat string in the cinder driver basically means nothing except for how mediawiki interpreted my CI name21:39
hemnaand the wiki is updateable outside of gerrit21:39
hemnathere is no other way to correctly describe where to get the info (from the wiki page)21:39
*** xyang has quit IRC21:40
hemnaI suppose I could rename the variable21:40
hemnaCI_NAME -> CI_WIKI_NAME21:40
hemnathe problem is the wiki page is the source of truth for a lot of shit21:40
patrickeastthat would at least make it less confusing, since for almost everyone it isn't actually the CI name21:40
hemnaand the data in the wiki page for each ci can change21:41
xyang1hemna: I think that's more clear, just don't make me rename the wiki page again21:41
* hemna pulls out my sed skillz21:42
xyang1hemna: patrickeast because you can't reallt rename them, you can only recreate them21:42
*** rdx_matt has joined #openstack-cinder21:42
xyang1so the old ones are still there and I don't know how to get rid of them21:43
*** rdx_matt has quit IRC21:43
patrickeastyea the wiki is kind of a pain for stuff like that21:43
*** rdx_matt has joined #openstack-cinder21:43
patrickeastwhich is why im a little bit concerned about having it be the nexus of our information with pointers from cinder to it21:43
patrickeastbut, if it works, it works21:44
*** FransUrbo has joined #openstack-cinder21:44
FransUrboHow do I specify what volume backend store to use when creating an instance w/ a volume?21:45
hemnathat look betters ?21:45
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: CI: Add CI_WIKI_NAME to all drivers
*** ketonne has joined #openstack-cinder21:48
xyang1hemna: looks better21:48
*** alonma has joined #openstack-cinder21:49
*** haplo37__ has quit IRC21:52
*** alonma has quit IRC21:54
*** jgregor has quit IRC21:54
openstackgerritAnthony Lee proposed openstack/cinder: Disabling online volume copy in HPE 3PAR drivers
*** Lee1092 has quit IRC22:03
*** ducttape_ has quit IRC22:05
*** mdenny has quit IRC22:08
*** rfinn has joined #openstack-cinder22:08
*** mdenny has joined #openstack-cinder22:09
*** IlyaG has quit IRC22:11
*** enriquetaso has quit IRC22:12
*** mriedem has quit IRC22:12
*** xyang1 has quit IRC22:13
*** salv-orlando has quit IRC22:13
*** mdenny has quit IRC22:13
*** nmadhok has joined #openstack-cinder22:15
*** rdx_matt has quit IRC22:17
*** salv-orlando has joined #openstack-cinder22:17
*** hemna is now known as hemnafk22:18
*** mdenny has joined #openstack-cinder22:19
*** david-lyle has joined #openstack-cinder22:22
*** salv-orlando has quit IRC22:23
*** edmondsw has quit IRC22:25
*** david-lyle has quit IRC22:25
*** david-lyle has joined #openstack-cinder22:25
*** FransUrbo has quit IRC22:31
*** diogogmt_ has joined #openstack-cinder22:33
*** diogogmt has quit IRC22:34
*** diogogmt_ is now known as diogogmt22:34
*** tpsilva has quit IRC22:38
*** gman-tx has quit IRC22:40
*** jwcroppe has quit IRC22:40
*** jwcroppe has joined #openstack-cinder22:41
*** leeantho has quit IRC22:41
*** IlyaG has joined #openstack-cinder22:44
*** gouthamr has quit IRC22:44
*** jwcroppe has quit IRC22:45
*** jamielennox is now known as jamielennox|away22:45
*** IlyaG has quit IRC22:49
*** david-lyle has quit IRC22:50
*** david-lyle_ has joined #openstack-cinder22:50
*** alonma has joined #openstack-cinder22:50
*** alonma has quit IRC22:54
*** diogogmt has quit IRC22:55
*** diogogmt has joined #openstack-cinder22:56
*** alonma has joined #openstack-cinder22:57
*** david-lyle_ has quit IRC23:01
*** alonma has quit IRC23:02
*** xianghui has joined #openstack-cinder23:03
*** mriedem has joined #openstack-cinder23:03
*** cknight has quit IRC23:10
*** rdx_matt has joined #openstack-cinder23:12
*** rdx_matt has quit IRC23:13
*** rdx_matt has joined #openstack-cinder23:13
*** sdake has joined #openstack-cinder23:17
patrickeastheh, thats a new one23:17
patrickeastturns out the flavor matching in nodepool uses 'in'
patrickeastso setting something up with flavor == 'devstack-node' will match a flavor called 'devstack-node-fc' in that code :(23:18
* patrickeast has gotten lucky for the last year with the order of the list matching the right ones23:18
*** rfinn has quit IRC23:24
*** sdake has quit IRC23:25
*** mtanino has quit IRC23:37
*** ducttape_ has joined #openstack-cinder23:37
*** bill_az has joined #openstack-cinder23:38
*** chlong has quit IRC23:38
*** salv-orlando has joined #openstack-cinder23:42
*** rdx_matt has quit IRC23:47
*** salv-orlando has quit IRC23:47
*** Apoorva_ has joined #openstack-cinder23:49
*** Apoorva has quit IRC23:52
*** Apoorva_ has quit IRC23:53
*** diogogmt has quit IRC23:55
*** akerr has quit IRC23:58
*** akerr has joined #openstack-cinder23:59

Generated by 2.14.0 by Marius Gedminas - find it at!