Thursday, 2016-08-11

*** akapil has joined #openstack-cinder00:00
*** akapil has quit IRC00:05
*** gouthamr has quit IRC00:08
*** Julien-zte has joined #openstack-cinder00:08
*** gouthamr has joined #openstack-cinder00:09
*** gouthamr has quit IRC00:10
*** alonma has joined #openstack-cinder00:12
*** ksumit has quit IRC00:15
*** ksumit has joined #openstack-cinder00:15
*** diogogmt has quit IRC00:16
*** IlyaG has quit IRC00:20
*** ksumit has quit IRC00:23
*** gouthamr has joined #openstack-cinder00:29
*** raunak has joined #openstack-cinder00:33
*** alonma has quit IRC00:33
*** alonma has joined #openstack-cinder00:33
*** alonma has quit IRC00:38
*** JoseMello has joined #openstack-cinder00:38
*** ksumit has joined #openstack-cinder00:39
*** ksumit has quit IRC00:48
*** oomichi has quit IRC01:00
*** mtanino has joined #openstack-cinder01:02
*** diogogmt has joined #openstack-cinder01:03
*** RaySun has joined #openstack-cinder01:04
*** cheneydc has joined #openstack-cinder01:05
*** cheneydc has quit IRC01:06
*** RaySun has quit IRC01:06
*** RaySun has joined #openstack-cinder01:07
openstackgerritwangxiyuan proposed openstack/cinder-specs: Inspection Mechanism For Capacity Limited Host  https://review.openstack.org/30886901:12
*** laughterwym has joined #openstack-cinder01:12
openstackgerritMerged openstack/cinder: Dell SC: Requests async header is invalid  https://review.openstack.org/35355101:15
*** laughterwym has quit IRC01:16
*** ebalduf has quit IRC01:19
*** alonma has joined #openstack-cinder01:22
*** yangyapeng has joined #openstack-cinder01:22
*** raunak has quit IRC01:23
*** laughterwym has joined #openstack-cinder01:26
*** alonma has quit IRC01:26
*** jwcroppe has quit IRC01:27
*** cknight has joined #openstack-cinder01:28
*** gouthamr_ has joined #openstack-cinder01:28
openstackgerritMerged openstack/cinder: Revert "Remove Tegile volume driver"  https://review.openstack.org/35255501:28
*** RaySun has quit IRC01:29
*** cknight1 has joined #openstack-cinder01:29
*** alonma has joined #openstack-cinder01:30
*** bardia has quit IRC01:30
*** jwcroppe has joined #openstack-cinder01:31
*** Lee1092 has joined #openstack-cinder01:31
*** gouthamr has quit IRC01:31
*** cknight has quit IRC01:32
*** EinstCrazy has joined #openstack-cinder01:33
*** jwcroppe has quit IRC01:33
*** alonma has quit IRC01:34
*** jwcroppe has joined #openstack-cinder01:35
*** EinstCrazy has quit IRC01:36
*** EinstCrazy has joined #openstack-cinder01:36
*** Julien-zte has quit IRC01:37
*** laughterwym has quit IRC01:37
*** laughterwym has joined #openstack-cinder01:37
*** EinstCrazy has quit IRC01:38
*** jgregor has joined #openstack-cinder01:38
*** EinstCrazy has joined #openstack-cinder01:38
*** laughterwym has quit IRC01:39
*** laughterwym has joined #openstack-cinder01:39
*** alonma has joined #openstack-cinder01:39
*** jwcroppe has quit IRC01:39
openstackgerrityuyafei proposed openstack/os-brick: Multipath device keeps old size when extending volume  https://review.openstack.org/35332501:40
*** mtanino has quit IRC01:41
*** karthik__ has quit IRC01:44
*** alonma has quit IRC01:44
*** laughterwym has quit IRC01:46
*** laughterwym has joined #openstack-cinder01:47
*** jdurgin1 has quit IRC01:47
*** alonma has joined #openstack-cinder01:48
*** lprice1 has joined #openstack-cinder01:48
*** laughterwym has quit IRC01:48
*** mriedem has quit IRC01:49
*** laughterwym has joined #openstack-cinder01:49
*** lprice has quit IRC01:50
*** mtanino has joined #openstack-cinder01:51
*** alonma has quit IRC01:52
*** akapil has joined #openstack-cinder01:54
*** mtanino has quit IRC01:57
*** chlong|mtg has joined #openstack-cinder01:57
*** jgregor has quit IRC01:58
openstackgerritMerged openstack/cinder: EMC VMAX - Oversubscription support  https://review.openstack.org/30749901:58
*** akapil has quit IRC01:59
*** Apoorva_ has quit IRC02:00
*** Apoorva has joined #openstack-cinder02:00
*** yangyape_ has joined #openstack-cinder02:01
*** yangyapeng has quit IRC02:04
*** bill_az_ has quit IRC02:04
*** Apoorva has quit IRC02:05
*** ducttape_ has joined #openstack-cinder02:18
*** alonma has joined #openstack-cinder02:23
*** alonma has quit IRC02:27
*** jungleboyj has joined #openstack-cinder02:32
*** laughterwym has quit IRC02:32
*** laughterwym has joined #openstack-cinder02:33
*** yangyapeng has joined #openstack-cinder02:36
*** jwcroppe has joined #openstack-cinder02:36
*** enriquetaso has quit IRC02:37
*** laughterwym has quit IRC02:37
*** yangyape_ has quit IRC02:38
*** harlowja_at_home has joined #openstack-cinder02:39
*** cknight1 has quit IRC02:43
*** gouthamr_ has quit IRC02:49
*** wewe0901 has quit IRC02:52
*** jwcroppe has quit IRC02:53
*** harlowja_at_home has quit IRC02:54
*** laughterwym has joined #openstack-cinder02:58
*** raunak has joined #openstack-cinder03:01
*** alonma has joined #openstack-cinder03:02
*** ducttape_ has quit IRC03:06
*** ducttape_ has joined #openstack-cinder03:06
*** alonma has quit IRC03:06
*** alonma has joined #openstack-cinder03:09
*** gouthamr has joined #openstack-cinder03:10
openstackgerritMerged openstack/cinder: Fix volume creation with no volume type  https://review.openstack.org/35366503:11
*** krotscheck_ has joined #openstack-cinder03:12
*** krotscheck has quit IRC03:13
*** krotscheck_ is now known as krotscheck03:14
*** alonma has quit IRC03:14
*** Srinu has joined #openstack-cinder03:17
*** alonma has joined #openstack-cinder03:17
*** gouthamr has quit IRC03:19
Srinupatrickeast: hi03:19
*** alonma has quit IRC03:22
*** merooney has quit IRC03:22
*** julim has joined #openstack-cinder03:26
*** alonma has joined #openstack-cinder03:26
*** alonma has quit IRC03:31
*** alonma has joined #openstack-cinder03:36
*** links has joined #openstack-cinder03:37
*** alonma has quit IRC03:40
*** laughterwym has quit IRC03:42
*** laughterwym has joined #openstack-cinder03:42
*** Lallana has quit IRC03:44
*** Lallana has joined #openstack-cinder03:44
*** mkoderer__ has quit IRC03:46
*** karthikp| has quit IRC03:46
*** jungleboyj has quit IRC03:47
*** laughterwym has quit IRC03:47
*** mfisch has quit IRC03:48
*** karthikp_ has joined #openstack-cinder03:48
*** yangyapeng has quit IRC03:49
*** yangyapeng has joined #openstack-cinder03:49
*** raunak has quit IRC03:50
openstackgerritnikesh proposed openstack/cinder: Concurrency issue in K2 iSCSI and FC Cinder drivers  https://review.openstack.org/35018203:50
*** coolsvap has joined #openstack-cinder03:51
*** aorourke has quit IRC03:52
*** mfisch has joined #openstack-cinder03:52
*** aorourke has joined #openstack-cinder03:52
*** mkoderer__ has joined #openstack-cinder03:52
*** mfisch has quit IRC03:53
*** mfisch has joined #openstack-cinder03:53
*** raunak has joined #openstack-cinder03:54
*** Thambirajan has quit IRC04:04
*** JoseMello has quit IRC04:10
*** yangyapeng has quit IRC04:15
*** laughterwym has joined #openstack-cinder04:16
*** shausy has joined #openstack-cinder04:16
*** yangyapeng has joined #openstack-cinder04:16
*** yangyapeng has quit IRC04:20
*** xinli has quit IRC04:20
*** xinli has joined #openstack-cinder04:21
*** IlyaG has joined #openstack-cinder04:26
*** julim has quit IRC04:27
*** alonma has joined #openstack-cinder04:30
*** alonma has quit IRC04:35
*** alonma has joined #openstack-cinder04:38
*** pdeore has joined #openstack-cinder04:41
*** alonma has quit IRC04:43
*** alonma has joined #openstack-cinder04:47
*** yangyapeng has joined #openstack-cinder04:48
*** jwcroppe has joined #openstack-cinder04:49
*** Thambirajan has joined #openstack-cinder04:50
*** alonma has quit IRC04:52
*** jwcroppe has quit IRC04:54
*** alonma has joined #openstack-cinder04:54
*** alonma has quit IRC04:59
*** Julien-zte has joined #openstack-cinder05:03
*** dkehn_ has quit IRC05:03
*** raunak has quit IRC05:07
*** Julien-zte has quit IRC05:07
*** gcb has joined #openstack-cinder05:11
*** lpetrut has joined #openstack-cinder05:11
*** lkuchlan has joined #openstack-cinder05:15
*** pdeore has quit IRC05:15
*** e0ne has joined #openstack-cinder05:17
*** sdake has quit IRC05:20
*** pbandark has joined #openstack-cinder05:21
*** dkehn_ has joined #openstack-cinder05:22
*** e0ne has quit IRC05:23
*** IlyaG has quit IRC05:26
*** akapil has joined #openstack-cinder05:28
*** rcernin has joined #openstack-cinder05:29
*** alonma has joined #openstack-cinder05:29
*** pdeore has joined #openstack-cinder05:30
flip214DuncanT: https://review.openstack.org/#/c/347274/ was successful now05:30
*** lpetrut has quit IRC05:31
*** akapil has quit IRC05:33
*** alonma has quit IRC05:34
*** lpetrut has joined #openstack-cinder05:34
*** ChubYann has quit IRC05:36
*** liverpooler has joined #openstack-cinder05:42
*** david-lyle_ has joined #openstack-cinder05:43
*** liverpooler is now known as yrabl|WFH05:43
*** Lee1092_ has joined #openstack-cinder05:43
*** yrabl|WFH is now known as liverpooler05:43
*** liverpooler has quit IRC05:44
*** serverascode_ has joined #openstack-cinder05:44
*** liverpooler has joined #openstack-cinder05:44
*** Lee1092 has quit IRC05:48
*** nicolasbock has quit IRC05:48
*** david-lyle has quit IRC05:48
*** aspiers has quit IRC05:48
*** bapalm has quit IRC05:48
*** mvk has quit IRC05:48
*** serverascode has quit IRC05:48
*** bapalm_ has joined #openstack-cinder05:48
*** mvk has joined #openstack-cinder05:49
*** Lee1092_ is now known as Lee109205:49
*** nicolasbock has joined #openstack-cinder05:49
*** liverpoo1er has joined #openstack-cinder05:49
*** aspiers has joined #openstack-cinder05:50
*** liverpooler has quit IRC05:50
*** liverpooler has joined #openstack-cinder05:51
*** jwcroppe has joined #openstack-cinder05:51
*** alonma has joined #openstack-cinder05:51
*** serverascode_ is now known as serverascode05:51
*** jwcroppe has quit IRC05:56
*** alonma has quit IRC05:56
*** alonma has joined #openstack-cinder06:12
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/35384706:15
*** alonma has quit IRC06:17
*** alonma has joined #openstack-cinder06:20
*** alonma has quit IRC06:24
*** alonma has joined #openstack-cinder06:27
openstackgerritJiong Liu proposed openstack/cinder: Remove debug logger translations  https://review.openstack.org/35347706:28
*** lpetrut has quit IRC06:30
*** Poornima has joined #openstack-cinder06:31
*** alonma has quit IRC06:31
*** alonma has joined #openstack-cinder06:34
openstackgerrithuanan proposed openstack/cinder: Migrate volume between backends in an async way  https://review.openstack.org/33492306:35
*** huanan has joined #openstack-cinder06:36
*** alonma has quit IRC06:38
*** alonma has joined #openstack-cinder06:41
*** pgadiya has joined #openstack-cinder06:44
*** nkrinner_afk is now known as nkrinner06:45
openstackgerritYanLin Ren proposed openstack/cinder: Volume Manage/Unmanage Support for IBM FlashSystem  https://review.openstack.org/35385606:45
*** pcaruana has joined #openstack-cinder06:46
*** alonma has quit IRC06:46
*** alonma has joined #openstack-cinder06:48
*** chenying has quit IRC06:51
*** chenying has joined #openstack-cinder06:52
*** tesseract- has joined #openstack-cinder06:53
*** alonma has quit IRC06:53
*** alonma has joined #openstack-cinder06:56
*** akapil has joined #openstack-cinder06:58
*** liverpoo1er has quit IRC06:58
*** alonma has quit IRC07:00
*** akapil has quit IRC07:02
*** alonma has joined #openstack-cinder07:03
*** alonma has quit IRC07:07
*** alonma has joined #openstack-cinder07:11
*** huanan has quit IRC07:12
*** Srinu has quit IRC07:13
*** huanan has joined #openstack-cinder07:14
*** avishay has joined #openstack-cinder07:14
*** arecknag has joined #openstack-cinder07:15
*** alonma has quit IRC07:15
*** alonma has joined #openstack-cinder07:18
*** alonma has quit IRC07:22
*** dkehn_ has quit IRC07:24
*** alonma has joined #openstack-cinder07:25
*** akapil has joined #openstack-cinder07:28
*** alonma has quit IRC07:29
*** akapil has quit IRC07:30
*** alonma has joined #openstack-cinder07:32
*** alonma has quit IRC07:37
*** akapil has joined #openstack-cinder07:38
*** dkehn_ has joined #openstack-cinder07:38
*** Thambirajan has quit IRC07:38
*** Thambirajan has joined #openstack-cinder07:38
*** alonma has joined #openstack-cinder07:40
*** lkuchlan has quit IRC07:40
*** Asaithambi has joined #openstack-cinder07:42
*** laughterwym has quit IRC07:45
*** e0ne has joined #openstack-cinder07:45
*** laughterwym has joined #openstack-cinder07:45
openstackgerritAbhilash Divakaran proposed openstack/cinder: Add CI_WIKI_NAME to Tegile volume driver  https://review.openstack.org/35388607:46
*** chlong|mtg has quit IRC07:46
*** alonma has quit IRC07:46
*** Thambirajan has quit IRC07:47
*** haritsu has joined #openstack-cinder07:47
*** akapil has quit IRC07:48
haritsuhi all, we are writing a cinder driver our storage devices. such as iscsi, fc or nfs.07:48
haritsubut i have one question07:48
haritsufor example we are using iSCSI/iSER protocols.07:49
*** alonma has joined #openstack-cinder07:49
haritsuis it possible iSCSI connection to VM's without using LVM?07:49
*** laughterwym has quit IRC07:49
openstackgerritJiWei proposed openstack/cinder: Add ability for ceph driver to report discard  https://review.openstack.org/35388807:52
*** jwcroppe has joined #openstack-cinder07:53
*** alonma has quit IRC07:53
*** akapil has joined #openstack-cinder07:54
*** alonma has joined #openstack-cinder07:56
*** jwcroppe has quit IRC07:57
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:00
*** alonma has quit IRC08:01
*** jyoti-ranjan has joined #openstack-cinder08:02
*** alonma has joined #openstack-cinder08:03
*** alonma has quit IRC08:08
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
*** alonma has joined #openstack-cinder08:11
e0neharitsu: hi. sure, why not? if you provide iscsi target, you don't need lvm08:11
openstackgerrityuyafei proposed openstack/os-brick: TargetPortalsNotFound exception message throws inaccurate  https://review.openstack.org/35389208:11
*** alonma has quit IRC08:12
*** alonma has joined #openstack-cinder08:12
*** amoralej|off is now known as amoralej08:12
openstackgerrityuyafei proposed openstack/os-brick: Wrong para makes exception message throws inaccurate  https://review.openstack.org/35389208:20
*** Asaithambi has quit IRC08:22
*** sdake has joined #openstack-cinder08:23
*** kwathore has joined #openstack-cinder08:24
*** Asaithambi has joined #openstack-cinder08:24
*** vladani has joined #openstack-cinder08:25
*** Julien-zte has joined #openstack-cinder08:29
*** akapil has quit IRC08:31
vladaniI have a query on retype volume: Is is require to have same volume_backend_name in cinder.conf while doing retype to source and destination volumes?08:31
vladaniWhat is the use of volume_backend_name in cinder.conf08:31
*** e0ne has quit IRC08:37
*** laughterwym has joined #openstack-cinder08:37
*** lkuchlan has joined #openstack-cinder08:39
*** coolsvap is now known as _coolsvap_08:41
*** laughterwym has quit IRC08:41
*** laughterwym has joined #openstack-cinder08:42
*** e0ne has joined #openstack-cinder08:43
*** Gao has joined #openstack-cinder08:44
*** Julien-zte has quit IRC08:45
*** v12aml has quit IRC08:46
*** v12aml has joined #openstack-cinder08:47
*** Lallana has quit IRC08:47
*** jwcroppe has joined #openstack-cinder08:54
*** jwcroppe has quit IRC08:58
*** bkopilov has joined #openstack-cinder09:00
*** bkopilov has quit IRC09:00
*** bkopilov has joined #openstack-cinder09:01
*** permalac has quit IRC09:02
*** rkrum has quit IRC09:02
*** Poornima has quit IRC09:03
*** amitkqed has quit IRC09:27
*** amitkqed has joined #openstack-cinder09:27
openstackgerritzhangyanxian proposed openstack/os-brick: Fix the typo in the file  https://review.openstack.org/34656809:30
*** lpetrut has joined #openstack-cinder09:30
*** yangyapeng has quit IRC09:34
*** yangyapeng has joined #openstack-cinder09:37
*** jyoti-ranjan has quit IRC09:38
*** _coolsvap_ is now known as coolsvap09:38
haritsu@vladani volume_backend_name value using on UI(horizon) and in database definition.09:48
*** jyoti-ranjan has joined #openstack-cinder09:49
*** gcb has quit IRC09:49
*** e0ne has quit IRC09:52
*** mvk has quit IRC09:52
*** v12aml has quit IRC09:52
*** mvk has joined #openstack-cinder09:52
*** v12aml has joined #openstack-cinder09:53
*** e0ne has joined #openstack-cinder09:56
*** mvk has quit IRC09:56
*** mvk has joined #openstack-cinder09:56
*** mvk is now known as mvk`afk10:01
*** alonma has quit IRC10:05
*** mvk`afk is now known as mvk10:07
*** mvk has quit IRC10:07
*** mvk has joined #openstack-cinder10:08
*** alonma has joined #openstack-cinder10:14
*** vbala has joined #openstack-cinder10:15
*** akapil has joined #openstack-cinder10:15
openstackgerritHelen Walsh proposed openstack/cinder: VMAX Driver - QoS support for the VMAX3  https://review.openstack.org/30750210:16
*** sdague has joined #openstack-cinder10:16
*** lpetrut has quit IRC10:17
*** alonma has quit IRC10:18
*** alonma has joined #openstack-cinder10:21
openstackgerritnikesh proposed openstack/cinder: Concurrency issue in K2 iSCSI and FC Cinder drivers  https://review.openstack.org/35018210:21
*** alonma has quit IRC10:25
*** alonma has joined #openstack-cinder10:25
*** EinstCrazy has quit IRC10:29
openstackgerritVipin Balachandran proposed openstack/cinder: Add CI wiki name to driver listing  https://review.openstack.org/35396510:32
*** laughterwym has quit IRC10:36
openstackgerrityuyafei proposed openstack/os-brick: Wrong para makes exception message throws inaccurate  https://review.openstack.org/35389210:40
*** rkrum has joined #openstack-cinder10:42
*** mvk is now known as mvk_afk10:42
*** mvk_afk is now known as mvk10:43
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Remove code duplication in enums  https://review.openstack.org/35397710:46
*** yangyapeng has quit IRC10:53
*** rkrum has quit IRC10:54
*** jwcroppe has joined #openstack-cinder10:55
*** dkehn_ has quit IRC10:56
*** jwcroppe has quit IRC11:00
*** JoseMello has joined #openstack-cinder11:01
*** avishay has quit IRC11:05
erlonharitsu: I didn't gt your question right, LVM is not needed to connect to the VMs11:07
openstackgerrityuyafei proposed openstack/os-brick: Wrong param makes exception message throws inaccurate  https://review.openstack.org/35389211:07
erlonharitsu: to connect to the VMs, once you have created the targets in your storage and added the volume to the targets, nova will call an external library (os-brick  - http://bit.ly/2aNBKR2) that implements several connectors (that will scan your target in the compute nodes and add the added volume).11:10
openstackgerritAlon Marx proposed openstack/cinder: Fix invalid services caused by enabled_backends  https://review.openstack.org/35369711:10
erlonharitsu: if the already available connectors in os-brick does not does everything you need, you can extend then and submit a patch in os-brick11:11
*** laughterwym has joined #openstack-cinder11:11
*** bkopilov has quit IRC11:11
nikesmvbala:  hi11:14
nikesmcan you look https://review.openstack.org/#/c/35018211:14
*** dkehn_ has joined #openstack-cinder11:15
*** thiagolib has joined #openstack-cinder11:20
nikesmhi Szymon Wróblewski:  addressed your comments on   https://review.openstack.org/#/c/350182, do not know your irc name11:20
erlonharitsu: this will help you: http://bit.ly/2aNDslx11:21
*** laughterwym has quit IRC11:25
*** laughterwym has joined #openstack-cinder11:25
*** avishay has joined #openstack-cinder11:26
*** amitkqed has quit IRC11:28
openstackgerritHelen Walsh proposed openstack/cinder: VMAX Driver - QoS support for the VMAX3  https://review.openstack.org/30750211:28
*** amitkqed has joined #openstack-cinder11:28
*** laughterwym has quit IRC11:30
openstackgerritMerged openstack/cinder: Fix volume upload-to-image for vhd disk-format  https://review.openstack.org/34794411:43
*** chlong|mtg has joined #openstack-cinder11:44
*** timcl has joined #openstack-cinder11:47
*** laughterwym has joined #openstack-cinder11:48
*** tpsilva has joined #openstack-cinder11:49
*** akapil has quit IRC11:53
*** akapil has joined #openstack-cinder11:58
*** gouthamr has joined #openstack-cinder11:59
openstackgerritIldiko Vancsa proposed openstack/cinder: Fix policy file used for testing attach  https://review.openstack.org/33969212:00
*** akapil has quit IRC12:02
*** Lee1092 has quit IRC12:03
*** akapil has joined #openstack-cinder12:07
*** mvk is now known as mvk_afk12:10
*** mvk_afk is now known as mvk12:11
openstackgerritChuck Fouts proposed openstack/cinder: NetApp: Add Consistency Group Support for NFS  https://review.openstack.org/34613512:13
*** Trident has joined #openstack-cinder12:16
*** akapil has quit IRC12:18
*** akapil has joined #openstack-cinder12:19
*** pauloewerton has joined #openstack-cinder12:22
*** mvk is now known as mvk_afk12:27
*** pdeore has quit IRC12:28
*** edmondsw has joined #openstack-cinder12:28
*** amoralej is now known as amoralej|lunch12:35
openstackgerritGoutham Pacha Ravi proposed openstack/cinder: NetApp: Add Consistency Group Support for NFS  https://review.openstack.org/34613512:37
*** lpetrut has joined #openstack-cinder12:37
*** mvk_afk has quit IRC12:40
*** mvk has joined #openstack-cinder12:40
*** lpetrut has quit IRC12:41
*** cristicalin has joined #openstack-cinder12:43
*** sdake has quit IRC12:43
*** rlrossit has joined #openstack-cinder12:43
*** lpetrut has joined #openstack-cinder12:43
*** akapil has quit IRC12:44
*** akapil has joined #openstack-cinder12:47
*** takedakn has joined #openstack-cinder12:48
*** lpetrut has quit IRC12:50
*** lpetrut has joined #openstack-cinder12:51
nikesmhi12:54
nikesmAlon Marx    there12:54
nikesmnickname?12:54
*** jwcroppe has joined #openstack-cinder12:57
*** amoralej|lunch is now known as amoralej12:58
*** mvk has quit IRC12:59
*** alonma has quit IRC13:00
*** jwcroppe has quit IRC13:01
*** jwcroppe has joined #openstack-cinder13:06
*** cristicalin has quit IRC13:06
*** xyang1 has joined #openstack-cinder13:07
*** akapil has quit IRC13:08
*** jyoti-ranjan has quit IRC13:08
*** alonma has joined #openstack-cinder13:08
*** lkuchlan has quit IRC13:09
*** mvk has joined #openstack-cinder13:11
*** Julien-zte has joined #openstack-cinder13:15
nikesmhi alonma Dinesh_Bhor alyson_13:15
openstackgerritMerged openstack/cinder: Add strict Boolean checking for volume create  https://review.openstack.org/33498913:16
*** merooney has joined #openstack-cinder13:18
*** alonma has quit IRC13:19
*** dustins has joined #openstack-cinder13:20
*** rkrum has joined #openstack-cinder13:20
*** akerr has joined #openstack-cinder13:22
*** pgadiya has quit IRC13:22
*** julim has joined #openstack-cinder13:22
*** vbala has quit IRC13:26
*** guy has quit IRC13:26
*** nikesm has quit IRC13:26
*** porrua has joined #openstack-cinder13:27
*** yangyapeng has joined #openstack-cinder13:28
*** jistr has quit IRC13:28
*** eharney has joined #openstack-cinder13:29
*** mvk has quit IRC13:29
*** jistr has joined #openstack-cinder13:30
openstackgerritnikesh proposed openstack/cinder: Concurrency issue in K2 iSCSI and FC Cinder drivers  https://review.openstack.org/35018213:30
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - iSCSI Multipath support  https://review.openstack.org/31706013:31
e0nexyang1: hi! just FYI, I'll review your patch once again including tests tomorrow morning. also, I'm going to test it too13:31
*** ducttape_ has quit IRC13:31
*** nikeshm has joined #openstack-cinder13:33
*** diogogmt has quit IRC13:33
nikeshmalonma alyson_ Dinesh_Bhor : please have a look , uploaded new patch13:34
xyang1e0ne: thanks!  I'll address your comments today.13:34
e0nexyang1: thanks13:34
*** diogogmt has joined #openstack-cinder13:34
alyson_nikeshm: ok, ill take a look :)13:34
*** mriedem has joined #openstack-cinder13:36
*** jgregor has joined #openstack-cinder13:36
openstackgerritnikesh proposed openstack/cinder: Concurrency issue in K2 iSCSI and FC Cinder drivers  https://review.openstack.org/35018213:36
*** akapil has joined #openstack-cinder13:37
nikeshmbluex: hi13:38
nikeshmaddressed yur reviews13:38
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Remove code duplication in enums  https://review.openstack.org/35397713:38
nikeshmcan you look on https://review.openstack.org/#/c/35018213:38
*** jgregor has quit IRC13:40
*** takedakn has quit IRC13:40
*** mriedem has quit IRC13:43
*** mriedem has joined #openstack-cinder13:43
*** akapil has quit IRC13:45
*** sborkows has joined #openstack-cinder13:46
*** sdake has joined #openstack-cinder13:47
openstackgerritxing-yang proposed openstack/cinder: WIP: Migrate consistency groups to groups  https://review.openstack.org/35035013:48
*** diogogmt has quit IRC13:50
*** mvk has joined #openstack-cinder13:53
*** shausy has quit IRC13:53
*** rlrossit_ has joined #openstack-cinder13:55
*** yangyape_ has joined #openstack-cinder13:56
*** diablo_rojo has joined #openstack-cinder13:56
*** rlrossit has quit IRC13:58
*** yangyapeng has quit IRC13:59
*** Asaithambi has quit IRC14:00
*** wxy has quit IRC14:01
*** jgregor has joined #openstack-cinder14:03
*** eharney has quit IRC14:04
openstackgerritMerged openstack/cinder: Remove debug logger translations  https://review.openstack.org/35347714:05
openstackgerritMerged openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/35384714:06
*** lprice1 has quit IRC14:06
*** sdake_ has joined #openstack-cinder14:07
*** ducttape_ has joined #openstack-cinder14:07
*** ducttape_ has quit IRC14:07
*** haritsu has quit IRC14:07
*** ducttape_ has joined #openstack-cinder14:08
*** sdake has quit IRC14:09
*** leeantho has joined #openstack-cinder14:10
*** pdeore has joined #openstack-cinder14:11
*** rkrum has quit IRC14:15
*** jistr is now known as jistr|call14:16
*** pdeore has quit IRC14:17
*** ebalduf has joined #openstack-cinder14:18
*** eharney has joined #openstack-cinder14:18
openstackgerritEric Harney proposed openstack/cinder: Add functional-py35 to tox  https://review.openstack.org/35352114:19
*** gaurangt1 has joined #openstack-cinder14:25
*** xyang has joined #openstack-cinder14:27
xyangeharney: hi, I have a question for you regarding how to get a bug fix into the Red Hat distro14:30
xyangeharney: There is a ScaleIO bug fix that was backported to liberty: https://bugs.launchpad.net/os-brick/+bug/1521396   A customer testing Red Hat OpenStack 8.0 ran into the bug.  How can this bug fix be picked up by 8.0?14:32
openstackLaunchpad bug 1521396 in python-os-brick (Ubuntu Wily) "[SRU] Fix iopsLimit parameter in ScaleIO connector" [Medium,Fix released] - Assigned to Edward Hope-Morley (hopem)14:32
*** chlong|mtg has quit IRC14:33
*** lprice has joined #openstack-cinder14:33
smcginnisxyang: Sorry, Liberty is too old. :P14:34
xyang@smcginnis: :)14:35
xyang@smcginnis: it was already merged in stable/liberty14:35
eharneyxyang: in a meeting, will look and get back to you in a bit14:35
xyangeharney: thanks14:35
xyang@smcginnis: it was also picked up by ubuntu.  I just don't know how Red Hat distro can pick it up as well14:36
smcginnisxyang: Yeah, I'm still just a little punchy from the ML thread. :)14:36
xyang@smcginnis: ok:)14:37
eharneyxyang: have we done a stable/liberty os-brick release?14:42
*** arecknag has quit IRC14:42
*** gaurangt1 has left #openstack-cinder14:42
xyangeharney: oh, that's a question for @smcginnis14:43
xyang@smcginnis: ?14:46
*** chlong|mtg has joined #openstack-cinder14:46
*** enriquetaso has joined #openstack-cinder14:46
*** sdake has joined #openstack-cinder14:47
*** sdake_ has quit IRC14:47
*** wN has quit IRC14:48
*** diogogmt has joined #openstack-cinder14:51
*** jdurgin1 has joined #openstack-cinder14:51
xyangeharney, @smcginnis, hemna: It seems that the last os-brick liberty release was 0.5.0: https://openstack.nimeyo.com/59780/openstack-announce-release-cinder-os-brick-release-liberty?qa_q=os-brick+release+0.5.0&show=59780#q5978014:52
*** sborkows has quit IRC14:52
xyangthat was Sept 201514:52
eharneyright, i'm not sure what the stable plans are for os-brick, are we doing an 0.5.1 at some point or no?14:53
eharney(presumably or we wouldn't be landing backports)14:53
xyangeharney: not sure.  0.6.0 is for mitaka14:54
eharneyxyang: for OSP the best plan is to file a bugzilla and we'll take it from there, but these seem like important questions regardless14:54
xyangeharney: sure14:55
*** cknight has joined #openstack-cinder14:58
*** rajinir has joined #openstack-cinder14:58
smcginniseharney, xyang: Sorry, was away. We don't (or haven't) done stable releases of libraries.15:01
smcginnisIs there a reason not to use the latest?15:01
smcginnisIt should be just like any other library. Just move forward.15:01
eharneysmcginnis: so the stable branch is just to make it easier for me to do downstream backports? :)15:01
smcginniseharney: I've asked because stable branches didn't make sense to me on os-brick or python-cinderclient.15:02
smcginnisWas told it's basically a convenience thing for the state of the code at release.15:02
smcginnisI don't think we ever need to backport to those.15:02
e0nexyang, hemna: thanks for the reviewing my patch to brick-cinderclient-ext15:02
*** dustins_ has joined #openstack-cinder15:03
xyange0ne: welcome15:03
*** dustins_ is now known as dschoenb15:03
*** Swanson has quit IRC15:03
e0nescottda: hi, are you around?15:03
*** dschoenb is now known as dustins_15:04
eharneysmcginnis: sounds like it should be done as a tag instead of a branch15:04
*** dustins has quit IRC15:05
xyangsmcginnis, eharney: should we tag the current stable/liberty?15:06
eharneyi don't think so15:06
*** karthik__ has joined #openstack-cinder15:06
smcginniseharney: That's what I thought.15:06
*** karthik__ has quit IRC15:09
xyangsmcginnis: why not backport?  there were bugs fixed in master and they were broken in earlier releases.15:09
xyangsmcginnis: so if you pick the new os-brick released later, it won't work with earlier cinder releases15:10
*** yangyapeng has joined #openstack-cinder15:11
*** yangyape_ has quit IRC15:14
*** dustins_ has quit IRC15:17
openstackgerritIvan Kolodyazhny proposed openstack/os-brick: Fix linuxrbd to work with Python 3  https://review.openstack.org/35415215:18
*** eharney has quit IRC15:19
*** karthik__ has joined #openstack-cinder15:21
*** jistr|call is now known as jistr15:21
*** tesseract- has quit IRC15:23
smcginnisxyang: Why won't the newer os-brick work?15:24
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Do not use utils.convert_str in API  https://review.openstack.org/35415615:26
xyang1smcginnis: no one has tested cinder in master with os-brick in liberty.  I think there are big changes since liberty15:26
xyang1smcginnis: or cinder in liberty with os-brick in newton15:27
e0nesmcginnis, xyang: I thought, that stable branches for libraries were introduced to be able backort and release fixes for stable verstions15:27
*** catintheroof has joined #openstack-cinder15:27
*** diablo_rojo has quit IRC15:28
*** diablo_rojo1 has joined #openstack-cinder15:28
e0nesmcginnis, xyang: http://docs.openstack.org/project-team-guide/stable-branches.html#active-maintenance15:28
smcginnisxyang1: Does the upper constraints file cap that? Otherwise the gate for liberty should pick up the latest os-brick on all backports.15:29
*** dustins_ has joined #openstack-cinder15:29
smcginnisJust like any other library.15:29
e0nehttps://github.com/openstack/requirements/blob/stable/liberty/upper-constraints.txt#L18215:29
smcginnise0ne: Not on libraries. Or at least we never have.15:29
e0nesmcginnis: from that link "This generally applies to libraries and client projects."15:30
e0nesmcginnis: I agree, that we never released fixes to stable cinderclient and os-brick. but we can15:31
smcginnisSo that upper-constraints file does cap it. We would just need to update that to allow something later than 0.6.0.15:31
smcginnisThe tricky part of doing a new stable library release is what would you version it?15:31
*** karthik__ has quit IRC15:32
smcginnisWe can't really do a 0.6.1, because that goes against our versioning convention if there is any kind of significant change.15:32
e0nesmcginnis: e.g. keystone team uses this https://github.com/openstack/python-keystoneclient/commit/195594b343a9adc35a1f7d12617d3616dda8a99e15:32
xyangeharney: does this work?  https://github.com/openstack/os-brick/commits/0.6.0/os_brick/initiator/connector.py?author=xing-yang15:32
*** karthik__ has joined #openstack-cinder15:32
xyangeharney: the fix is in 0.6.015:32
*** eharney has joined #openstack-cinder15:32
smcginnise0ne: Not sure how that relates?15:32
e0nesmcginnis: it's an example of backport to stable branch in client15:33
xyangeharney: the announcement said it is mitaka: https://openstack.nimeyo.com/67466/openstack-announce-release-cinder-os-brick-release-mitaka?qa_q=os-brick+release+0.6.0&show=67466#q6746615:33
smcginnise0ne: But do they actually release that?15:33
eharneyxyang: ok?15:34
smcginnisI think the right thing would be to propose an upper-constraints change to bump up stable/liberty os-brick version, then have the cinder backport that needs it have a Depends-on tag for that change.15:34
*** Swanson has joined #openstack-cinder15:36
e0nesmcginnis: I don't know15:36
e0nesmcginnis: technically, we're able to backport fixes and release them for liverty and mitaka only15:37
xyangeharney: I mean can OSP pick up 0.6.0 for liberty?15:37
eharneyxyang: it's more likely we'd backport the individual fix15:37
xyangeharney: ok15:38
eharneysince nobody is CIing 0.6.0+liberty15:38
smcginniseharney: Should be: https://github.com/openstack/requirements/blob/stable/liberty/upper-constraints.txt#L18215:39
e0nesmcginnis: according to http://releases.openstack.org/mitaka/index.html#python-keystoneclient, some teams release stable fixes for clients15:39
*** avishay has quit IRC15:40
e0nesmcginnis: please, ignore my link above. I didn't verify releases dates15:40
*** haplo37__ has joined #openstack-cinder15:40
eharneysmcginnis: ah, just looked at a stable/liberty patchset -- the gate job is running w/ 0.6.015:40
smcginnise0ne: There's quite a spread on version numbers there. I wonder if that's just all supported client releases?15:41
smcginnisWe have 1.5 and 1.6 there. Maybe that's just the client releases that happened during that cycle.15:41
smcginnisbswartz: ping15:42
*** tpsilva has quit IRC15:42
*** tpsilva has joined #openstack-cinder15:43
*** links has quit IRC15:43
*** karthik__ has quit IRC15:44
*** karthi___ has joined #openstack-cinder15:44
*** pbandark has quit IRC15:46
*** akapil has joined #openstack-cinder15:47
*** sdake_ has joined #openstack-cinder15:49
e0nesmcginnis: that's everythig I've found http://lists.openstack.org/pipermail/openstack-dev/2015-April/061731.html15:49
e0nesmcginnis: glance team released patch to stable version at least once15:50
*** sdake has quit IRC15:51
*** akapil has quit IRC15:53
*** akapil has joined #openstack-cinder15:54
*** Lee1092 has joined #openstack-cinder15:57
*** jwcroppe has quit IRC15:57
*** jwcroppe has joined #openstack-cinder15:58
*** jwcroppe_ has joined #openstack-cinder15:59
openstackgerritAdriano Freires Rosso proposed openstack/cinder: HNAS: Deprecating XML config file  https://review.openstack.org/29407616:00
_alastor_Does anyone know the process for deprecating config StrOpts?16:00
_alastor_I've moved some of my StrOpts into ExtraSpecs, but apparently just removing the options will break the upgrade process?16:01
*** jwcroppe has quit IRC16:03
*** raunak has joined #openstack-cinder16:04
patrickeast_alastor_: iirc there is a flag you put on the config option to mark it as deprecated, and for the next release you have to support both ways of configuring the value16:04
patrickeastwell, s/have to/might want to/16:04
*** akapil has quit IRC16:04
patrickeastat least log a big warning message saying that you are ignoring the deprecated value16:04
_alastor_Oslo.config has deprecated_for_removal as an option.  There's about six other deprecated options though, a bit confusing which one to use16:06
*** e0ne has quit IRC16:07
*** xyang has quit IRC16:08
_alastor_patrickeast: Only a couple of folks have used that option though16:08
*** jdurgin1 has quit IRC16:08
adrianofr_alastor_: I guess the deprecated_for_removal is the right one16:11
*** seldenr has joined #openstack-cinder16:11
adrianofr_alastor_: At least it's the one I'm using on my patch16:11
_alastor_adrianofr: That's the one I'm going to go with :)16:12
adrianofr_alastor_:  ;)16:12
openstackgerritMatt Smith (_alastor_) proposed openstack/cinder: Datera driver 2.2 update  https://review.openstack.org/35253316:14
*** rcernin has quit IRC16:15
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: Mask out passwords when tracing  https://review.openstack.org/35313716:17
*** IlyaG has joined #openstack-cinder16:18
*** timcl has quit IRC16:18
*** Apoorva has joined #openstack-cinder16:20
*** laughterwym has quit IRC16:21
*** laughterwym has joined #openstack-cinder16:22
*** laughterwym has quit IRC16:26
JayConroyFor new unit tests on master, mock should be used. I was wondering does this also apply to patches in mitaka or is mox also acceptable? Thanks16:28
hemnaJayConroy, do not use mox please16:29
JayConroyOk cool16:29
*** xyang has joined #openstack-cinder16:30
*** asselin__ has quit IRC16:30
*** takedakn has joined #openstack-cinder16:37
*** asselin_ has joined #openstack-cinder16:37
*** takedakn has quit IRC16:41
*** satya has joined #openstack-cinder16:46
*** baumann has joined #openstack-cinder16:46
*** timcl has joined #openstack-cinder16:47
*** karthi___ has quit IRC16:48
*** _ducttape_ has joined #openstack-cinder16:49
*** ducttape_ has quit IRC16:53
*** dustins_ is now known as dustins16:53
*** sdake_ has quit IRC16:57
*** julim has quit IRC16:58
*** bwallis has quit IRC16:58
satyaI am an intern at Intel and have been working on small bug in cinder since the past few weeks. My internship is getting over really soon, so I would really appreciate if my patch (https://review.openstack.org/#/c/350310/) could be reviewed by the cores soon. Thanks in advance!16:59
*** rcernin has joined #openstack-cinder17:02
*** rcernin has quit IRC17:05
*** rcernin has joined #openstack-cinder17:05
*** thiagolib has quit IRC17:08
*** karthik__ has joined #openstack-cinder17:09
bswartzsmcginnis: pong17:10
smcginnisbswartz: Hey!17:11
smcginnisbswartz: Was wondering, do you ever do stable releases of python-manilaclient?17:11
*** lpetrut has quit IRC17:11
bswartzso the flames on that mail thread seem to have died down, eh?17:11
bswartzsmcginnis: we reserve the right to, but I can't recall the last time we actually did17:11
hemnaI think the thread just turned into something useless17:12
hemnaand off topic17:12
smcginnisbswartz: OK, same here. The question came up and was curious.17:12
*** ksumit has joined #openstack-cinder17:12
smcginnishemna: totally17:12
ntpttr___satya: I'll pull down your patch and manually test it17:12
smcginnisNew tangential thread to that now if you haven't seen it yet.17:12
hemnacrap really?17:12
bswartzwell the stable guys have some good points, and the qa guys have some good points17:13
*** e0ne has joined #openstack-cinder17:13
nikeshmDetails: Volume 4a85ea05-4c69-4b14-9294-16518372f89b failed to reach available status (current creating) within the required time (196 s).17:13
nikeshmwhere can i change 196s17:13
nikeshmin tempest17:13
hemnawhat is the thread name ?17:13
bswartzI think we're all just dissatisfied with eachother's ability to get work done17:13
hemnatitle17:13
openstackgerritNate Potter proposed openstack/python-cinderclient: Add "start_version" and "end_version" support to  argparse  https://review.openstack.org/34012917:14
bswartz[openstack-dev] [Cinder] [stable] [all] Changing stable policy for drivers17:14
bswartzhemna: ^17:14
*** chenying has quit IRC17:14
hemnaoh I thought there was a new thread17:14
smcginnisbswartz: I'm dissatisfied with saying we'd feel better having everyting well tested, so screw end users and move your stuff somewhere else where it can't be tested.17:14
bswartzdamnit17:15
bswartz[openstack-dev] [tc][cinder] tag:follows-standard-deprecation should be removed17:15
bswartzhemna: ^17:15
bswartzfirst one was c/p error17:15
smcginnishemna: Yeah, that one.17:15
*** zhangjn has quit IRC17:15
hemnaoh :)17:15
*** chenying has joined #openstack-cinder17:15
*** charlesr has joined #openstack-cinder17:15
hemnaoh jeez17:16
*** tongli has joined #openstack-cinder17:16
*** tongli has quit IRC17:16
*** zhangjn has joined #openstack-cinder17:17
bswartzsmcginnis: my reading of mtreinish's comments are that he's worried we will lock ourselves into a suboptimal future, when he wants to build a perfect future17:17
e0nehemna: it's only 10 messages in thread yet)17:17
bswartzit's a case of his perfect being the enemy of our good17:17
hemnathe infra stuff is just a mess IMHO17:17
bswartzand it's true that tempest has gotten less bad over time, and stable branches have gotten less bad over time17:18
hemnait boils down to folks not wanting to ensure the entire infra stuff works as well as the openstack codebase itself does17:18
bswartzso maybe someday we will realize mtreinish's perfect future17:18
hemnaso, since infra can't test older stuff, we can't fix bugs in cinder drivers.17:18
hemnaand asking cinder driver developers to fix all of infra's problems is...b.s.17:18
hemnainfra doesn't tag or branch anything17:19
hemnajust the wild west17:19
hemnascrew deployers/users of openstack17:19
bswartzhemna: I think the request for help is aimed more at the distros than at us in cinder17:19
hemnait's not just distros that want this though17:20
hemnaI think this also greatly benefits driver developers17:20
hemnaat the end of the day, openstack users/admins/distros lose because we can't fix driver bugs in cinder for past releases.17:21
hemna</rant>17:21
bswartzhemna: I agree with you17:21
smcginnishemna: Totally17:21
e0nehemna: +217:21
bswartzI'm not sure I want to fan those flames anymore though17:21
bswartzpicking a fight with the stable/infra/qa guys can't end well17:22
openstackgerritGorka Eguileor proposed openstack/cinder: Refactor cinder.utils._copy_volume_with_path  https://review.openstack.org/35426917:22
openstackgerritGorka Eguileor proposed openstack/cinder: Fix exceed of dd action when copying with path  https://review.openstack.org/35427017:22
hemnaat this point, I think we should just come up with a Cinder solution that works for us.17:22
smcginnisA separate github repo?17:22
bswartzgithub would create a nightmare for access control17:23
hemnaI think unfortunately, it means we either 1) do nothing and leave the mess as is now or 2) create a separate github for stable cinder driver releases.17:23
e0nestandalone cinder! w/o openstack infra17:23
hemnaand I'll put my thingee hat on and say 3) move drivers to a new repo17:23
guitarzanhemna: "this discussion feels to me like we are letting day-to-day operational concerns define the vision and scope of openstack"17:24
* guitarzan runs away17:24
smcginnisYeah, 2 is slightly better than where we are today. Still very unfortunate that we would have to go to somewhere outside of our own environment to do it.17:24
* bswartz snaps hemna around a bit with a large trout17:24
hemnaguitarzan, well, as a driver developer the current status quo is a failure IMHO.17:24
*** yangyapeng has quit IRC17:24
hemna:)17:24
guitarzanthat was from the TC meeting about the golang swift merge, but I think it applies here17:25
nikeshmtempest.exceptions.TimeoutException: Request timed out Details: Volume 4a85ea05-4c69-4b14-9294-16518372f89b failed to reach available status (current creating) within the required time (196 s).  how can i increase this time in CI17:25
hemnatelling openstack deployers that they have to get Cinder Kilo driver fixes from N different places on the web is a disaster.17:25
geguileohemna: When you have a minute, XIO_FC is passing now  https://review.openstack.org/34959817:25
hemnageguileo, sweet :)17:26
bswartzguitarzan: that was a totally different issue, and the arguments were just people trying to be polite about telling the golang crowd to GTFO17:26
hemnageguileo, so it looks like all the reporting FC CIs are passing then.  thanks17:26
guitarzanbswartz: right, even though they're solving a real problem17:26
*** yangyapeng has joined #openstack-cinder17:26
geguileohemna: That's always a good sign ;-)17:27
bswartzguitarzan: that's besides the point17:27
geguileohemna: I actually tested the patch on a XIO env before submitting O:-)17:27
guitarzanno, that was the whole point17:27
guitarzanbut I'm not going to argue about it17:27
bswartzyeah let's not drag the swift debate into the cinder channel17:27
bswartzwe have enough flame wars to keep it plenty hot in here17:28
hemnaguitarzan, sorry I guess I'm not following.17:28
xyanggeguileo: I believe you are talking about XtremIO, not XIO?17:28
geguileoxyang: Yep, my bad  XD17:28
bswartzhemna: it's not related -- it was a reference to the TC debate about golang17:28
hemnaah ok17:28
guitarzanhemna: it's about ideology preventing people from making things better17:28
guitarzanin this case, driver bug fixes17:29
hemnakew.17:29
hemnawhich we basically can't do now.17:29
*** _ducttape_ has quit IRC17:29
hemnawell not in tree at least.17:29
*** sdake has joined #openstack-cinder17:29
hemnaI think even if we did option 3, we'd still be stuck with the same policy problem17:30
smcginnishemna: Even if we can tag it differently, I think we would just be moving the problem and adding a bunch of extra overhead.17:30
*** catintheroof has quit IRC17:30
xyanggeguileo: Ganesh kept rerunning CI because he ran it before the volume type fix was merged and then after it was merged:)17:30
*** nkrinner is now known as nkrinner_afk17:30
hemnasmcginnis, yah most likely17:31
geguileoxyang: Oh, but I saw another request after it was already passing, that's why I said that17:31
hemnait would be 1 place to hold all driver fixes though17:31
smcginnishemna: Definitely better than having them scattered all over the internets.17:32
*** yangyapeng has quit IRC17:32
hemnayah17:32
hemnaI don't know how we'd handle the access rights to deal with pull requests, etc.17:32
hemnasadness all the way around17:32
* hemna looks at our fearless leader17:33
e0necan I start offtopic and talk about os-brick and cinder newton?17:35
smcginnishemna: I was thinking of that too.17:35
e0neI need some kind of advice17:35
smcginnisI guess we would just add core (or stable core) to the repo to have rights.17:35
smcginnise0ne: Sure, what's up?17:36
*** bwallis has joined #openstack-cinder17:36
e0nesmcginnis: I'm not sure that I did everything right https://review.openstack.org/#/c/354156/ here17:37
e0nesmcginnis, hemna: I don't like copy-paste from one side17:38
e0nesmcginnis, hemna: and I don't waste  to get heeded function merged in oslo-utils for newton or even for O release17:38
smcginnise0ne: Looks right to me, but maybe we should track down Victor.17:38
e0nes/heeded/needed17:38
hemnafwiw.....https://review.openstack.org/#/c/73464/17:39
*** jgregor has quit IRC17:40
hemnaI put that patch up a few years ago when we started the driver removal debate17:41
e0nehemna: why do we need it and CIs?17:41
smcginnisI vaguely remember that. Reading through again.17:41
hemnanot sure if it's relevant now, but it basically marks drivers as certified or not.17:41
hemnaand logs something if they aren't.17:41
hemnanot really related to the bug fixing issue of past releases.17:42
*** bardia has joined #openstack-cinder17:42
hemnabut I just thought of it while reading that follows-standard-deprecation thread17:42
e0nehemna: I'm not sure that it's a good idea from the vendors' perspective17:42
e0nehemna: vendors could have own certification policy17:42
smcginnisI think with the focus on CI now we wouldn't need that.17:43
hemnaso basically, if we wanted to stop removing drivers17:43
hemnathen we just put up a patch saying the driver is now uncertified17:43
openstackgerritMerged openstack/os-brick: RemoteFsClient extend Executor  https://review.openstack.org/35184517:43
hemnaand log use at your own risk17:43
hemnafwiw17:43
smcginnisI think I'd rather do the suggestion from the ML of marking them deprecated and logging that big and loud.17:44
scottdae0ne: I had a question in https://review.openstack.org/#/c/354156/117:44
hemnathat's basically the same thing17:44
hemnawe could rearrange the driver tree possibly17:45
e0nescottda: sure17:45
smcginniscinder/volume/drivers/crappyones/* ?17:45
*** xyang has quit IRC17:45
hemnacinder/volume/drivers/tested, cinder/volume/drivers/SUCKS17:45
hemna:P17:45
smcginnis:)17:45
e0necinder/volume/drivers/contrib?17:45
hemnathen we just do a git mv17:45
hemnaor cinder/volume/drivers/deprecated17:46
hemnato make the TC happy17:46
smcginnisIf we did leave them in tree, and we did mark them deprecated, then would be pretty easy to use the driver map to change the location and have it logged.17:46
hemnayah17:47
hemnawe could get really annoying and also log a deprecation warning on every driver call17:47
hemna:P17:47
e0nescottda: https://review.openstack.org/#/c/304462/17:48
hemnae0ne, so why is it bad to use utils.convert_str ?17:48
e0nehemna: not bad in general17:48
smcginnishemna: Fill their disk with log warnings. ;)17:49
e0nehemna: but I need the same thing in os-brick17:49
nikeshmpatrickeast asselin asselin_ : hi any idea on tempest.exceptions.TimeoutException: Request timed out Details: Volume 4a85ea05-4c69-4b14-9294-16518372f89b failed to reach available status (current creating) within the required time (196 s).  how can i increase this time in CI17:49
asselin_there should be a tempest setting with that value. tempest.conf? ask in openstack-qa ?17:49
e0nehemna: I'm looking on a solution how to not duplicate the code and do not stuck with oslo libs17:49
smcginnisnikeshm: I think you should resolve the issue. If a customer hit that I don't think they would be happy.17:49
hemnae0ne, there is already some small duplicated utils in os-brick17:49
hemnae0ne, tracing, retry stuffs17:50
*** lpetrut has joined #openstack-cinder17:50
e0nehemna: hm.. so I need just to abandone the patch and feel free?17:50
hemnathe amount of duplicated stuff is small at this point.17:50
*** haplo37__ has quit IRC17:50
bswartzhemna, smcginnis: Linux leaves broken drivers in tree, and simply marks them as such. See this file: https://github.com/torvalds/linux/blob/master/MAINTAINERS17:50
hemnais there something in oslo that does this ?17:50
e0nehemna: I didn't found17:50
*** haplo37__ has joined #openstack-cinder17:50
bswartzsee lines 80-8917:51
e0nehemna: I don't think they will be happy to merge 1 function just for cinder17:51
*** haplo37- has joined #openstack-cinder17:51
nikeshmsmcginnis: correct, but as of now for passing CI 100 % i want to increase,17:51
hemnabswartz, yah17:51
e0nehemna:  here is my patch for os-brick https://review.openstack.org/#/c/354152/17:51
smcginnisnikeshm: OK, fair enough.17:51
hemnamigrating to a deprecated folder or marking the driver as unmaintained/uncertified/deprecated17:51
nikeshmsmcginnis: sometimes i am getting this17:51
hemnaI kinda still like the idea of forcing the admin to put in a conf entry in the cinder.conf to enable this known unsupported driver.17:52
hemnare: my old patch.17:52
scottdae0ne: Yes, I remember that patch to use convert_str...it looks like you are now wanting to revert that? And I understand why you are doing the revert...i17:52
hemnaforces the admin to acknowledge that they are using a possibly broken/unsupported driver.17:53
hemnare: https://review.openstack.org/#/c/73464/2/etc/cinder/cinder.conf.sample17:53
e0nescottda: hemna convinced me to abandone this patch17:53
scottdae0ne: But it looks like you missed an "else" in the revert...or am I wrong?17:53
scottdae0ne: OK17:53
openstackgerritOpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext: Updated from global requirements  https://review.openstack.org/35428417:54
e0nescottda: it was try to make less copy-paste in the code. but it failed17:54
e0nescottda: looking on my previous patch (https://review.openstack.org/#/c/304462/) and code in master https://github.com/openstack/cinder/blob/master/cinder/api/openstack/wsgi.py#L538-L540, we could face  https://bugs.launchpad.net/cinder/+bug/1550951 again17:55
openstackLaunchpad bug 1550951 in cinder (Gentoo Linux) "http header value must be a string" [Undecided,New]17:55
smcginnishemna: We could just move the drivers and not put them in the rename map.17:55
smcginnisThen they are still there, but it's very explicit for them to still use the driver.17:56
*** dkehn_ has quit IRC17:56
hemnadoes that meet the TC's requirements for keeping the tag ?17:56
smcginnishemna: No idea.17:56
bswartzsmcginnis: that would break people on upgrade and anything that breaks upgrades with a deprecation warning will probably attract attention17:57
bswartzs/with/without/17:57
hemnaI'd guess we'd have to leave them in the same place, but not start them unless the conf entry was explicitly added to enable deprecated drivers.17:57
smcginnisbswartz: yeah, you're right. That probably wouldn't fly then.17:57
hemnabswartz, +!17:57
*** ametts has joined #openstack-cinder17:57
hemnaerr 117:57
asselin_I find it ironic that last time drivers got pulled, no one from the TC objected17:57
e0nesmcginnis, hemna: I missed something. why do you want to rename or move drivers? how will it help maintainers to fix stable branches?17:57
smcginnishemna: That wouldn't work either then.17:58
hemnae0ne, separate issue17:58
bswartzasselin_: probably someone is pulling the strings this time17:58
hemnae0ne, this is part of the "[openstack-dev] [Cinder] [stable] [all] Changing stable policy for drivers​" discussion17:58
e0nehemna: what issue do you mean?17:58
e0neoh..17:58
hemnaarg!17:58
smcginnise0ne: Just the deal with being follows-deprecation-policy tag.17:58
e0ne<censoured>17:58
hemnae0ne, "[openstack-dev] [tc][cinder] tag:follows-standard-deprecation should be removed"17:58
hemnasorry that one17:58
hemnaI did the same thing that bswartz did earlier....damn copy/paste!17:59
e0neit's stupid18:00
hemna:)18:01
e0nekeep untested and propably broken code just to feed some policy18:01
openstackgerritSofia Enriquez proposed openstack/cinder: Error when manage an already-managed volume  https://review.openstack.org/35428918:01
hemnaif we really want to keep the tag, then adding in some hooks to the volume manager to not start deprecated drivers is the only way to go.18:02
hemnaI can rework that patch and put it up and annoy everyone.18:02
e0neI don't like to support untested code18:03
*** ducttape_ has joined #openstack-cinder18:03
hemnayah I don't think anyone does18:03
e0neso, personally, I would not care about the tag18:03
e0nethere are something like it in Ironic18:04
*** julim has joined #openstack-cinder18:04
e0nethey have repo with untested drivers18:04
hemnaon the other hand, if you upgrade from Mitaka to Newton and the driver you were using is now gone....18:04
hemnathat sucks for OpenStack18:04
e0neand almost nobody cares if they work18:04
hemnaand admins18:05
e0nehemna: I'm agree with you18:05
hemnaif we left them in, and just disabled starting them, the admin could put in a new conf entry in the driver section to say, no_really_enable_this_even_though_it_may_be_broken = True18:05
hemnaand the volume manager could start it and puke lots of this driver sucks warnings.18:06
hemnabut really, at some point in the future when that driver really no longer starts because it is missing a required interface method....it won't start at all.18:07
hemnaIf we had this mechanism, we could use it for the Scality driver.18:08
hemnathey are responsive to ML posts about removal18:08
bswartzhemna: I liked the idea of spamming the log with warnings because it's backwards compatible and eventually the filesystem will fill up and the admin will take a look to find out why18:08
hemnathey are just having major h/w issues that prevents them from getting their CI deployment working.18:08
hemnafwiw.18:08
e0nedid anyone proposed again to move drivers to the saparate repo and don't follow deprication policy there?18:08
hemnabswartz, that's why I liked the forced conf entry.18:09
hemnadon't start the driver unless the admin takes action (adding conf entry)18:09
bswartze0ne: a significant number of people are against moving drivers out of tree, even to another repo18:09
hemnathey'll know right away when they try and start cinder18:09
bswartzhemna: that's not backwards compatible though18:09
hemnainstead of 2 weeks later when their disk fills up18:09
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/35429318:10
*** dkehn_ has joined #openstack-cinder18:10
hemnaarguable :)18:10
e0newe can't make everyone happy18:10
hemnayah, I'm just trying to find some compromise that gets the TC off our back and keeps the tag.18:11
*** lpetrut has quit IRC18:11
e0neTC tells us that we must follow deprication policy to make operators happy, but we can't backord fixes to make operators and users happy18:11
hemnae0ne, you are talking crazy18:12
hemna:)18:12
e0ne:)18:12
e0neprobably, I need to take a rest. 9pm18:12
*** lpetrut has joined #openstack-cinder18:13
hemnae0ne, before you nap......https://review.openstack.org/#/c/353737/18:14
nikeshmhi any core get time please review https://review.openstack.org/#/c/350182  (waiting for CI to report success)18:14
hemnanikeshm, let me know when CI passes18:15
nikeshmsure18:15
e0nehemna: done18:16
hemnae0ne, thanks18:16
e0nehemna: np18:16
*** lpetrut has quit IRC18:17
*** e0ne has quit IRC18:20
*** mvk has quit IRC18:20
*** charlesr has quit IRC18:23
*** eharney has quit IRC18:32
*** ChubYann has joined #openstack-cinder18:33
*** catintheroof has joined #openstack-cinder18:34
*** chlong|mtg has quit IRC18:37
*** chlong|mtg has joined #openstack-cinder18:38
openstackgerritMerged openstack/python-brick-cinderclient-ext: Add fast8 ability to tox  https://review.openstack.org/35373718:39
*** ducttape_ has quit IRC18:44
*** ducttape_ has joined #openstack-cinder18:44
*** eharney has joined #openstack-cinder18:47
*** amoralej is now known as amoralej|off18:47
*** xyang has joined #openstack-cinder18:49
karthikp_scottda: Hi18:50
scottdakarthikp_: Hi18:51
karthikp_scottda: I wanted to check with you if in case of RIP, if the order of upgrades that operators use is capi -> csch -> c-bak -> c-vol ...is that right ?18:52
karthikp_RUP*18:52
karthikp_scottda: I was testing a few of our rup patches and wanted to perform a similar approach18:53
scottdaRUP?18:53
karthikp_scottda:  rolling upgrades sorry...mostly patches moving stuff to ovo18:54
karthikp_:D18:54
*** karthik__ has quit IRC18:54
*** karthik__ has joined #openstack-cinder18:54
scottdakarthikp_: I think we came up with an order that made sense during review of rolling upgrade specs...looking for that now...18:55
scottdakarthikp_: "As cinder-backup was strongly reworked in this release, the recommended upgrade order when executing live (rolling) upgrade is c-api->c-sch->c-vol->c-bak."18:57
scottdafrom http://docs.openstack.org/releasenotes/cinder/mitaka.html18:57
*** rlrossit_ has quit IRC18:58
karthikp_scottda: this is gold..thanks :)18:58
scottdakarthikp_: np18:58
*** mvk has joined #openstack-cinder19:04
*** fifieldt has quit IRC19:06
nikeshmhemna: CI passed19:11
nikeshmhttps://review.openstack.org/#/c/35018219:11
*** xinli has quit IRC19:12
nikeshmhi, CI is passed on https://review.openstack.org/#/c/350182  and jenkins +119:13
nikeshmi believe all addressed all reviews19:13
nikeshmand it should be OK19:13
nikeshmto let it land19:14
openstackgerritMerged openstack/os-brick: Fix FC multipath rescan  https://review.openstack.org/34959819:14
*** satya has quit IRC19:18
*** fifieldt has joined #openstack-cinder19:18
nikeshmsmcginnis xyang1 hemna: if get time lets merge https://review.openstack.org/#/c/35018219:20
*** karthik__ has quit IRC19:22
*** eharney has quit IRC19:23
nikeshmone doubt i have19:24
nikeshmhow can i see coordination lock is aquired in http://54.209.116.144/82/350182/24/check/kaminario-dsvm-tempest-full-iscsi/d5a696e/logs/screen-c-vol.txt.gz19:24
*** MAlkholm has joined #openstack-cinder19:25
*** MAlkholm is now known as Malkholm19:28
MalkholmHey there!19:28
*** catintheroof has quit IRC19:28
MalkholmI need help. Mitaka, Cinder, Ceph, multi-backend19:29
MalkholmIn what part of cinder.conf should I place backend sections (2) to have it work?19:30
MalkholmI tried to the end of file - no luck, right after the enabled_backend - works only first19:30
hemnaMalkholm, pastebin19:31
openstackgerritAlyson proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages  https://review.openstack.org/31078519:31
nikeshmsmcginnis: how to see coordination lock in c-vol.log.  http://54.209.116.144/82/350182/24/check/kaminario-dsvm-tempest-full-iscsi/d5a696e/logs/screen-c-vol.txt.gz19:32
smcginnisnikeshm: Not sure. I haven't had to do anything with that.19:32
Malkholmhemna, sorry?19:32
smcginnisMalkholm: Do you have a config you're trying to use that you could put in a pastebin so we can take a look at it?19:32
openstackgerritAlyson proposed openstack/cinder: HNAS: Add support for manage/unmanage snapshots in NFS driver  https://review.openstack.org/33399219:32
hemnaMalkholm, http://bfy.tw/7Axp19:34
*** eharney has joined #openstack-cinder19:36
*** takedakn has joined #openstack-cinder19:37
openstackgerritHelen Walsh proposed openstack/cinder: VMAX Driver - Live Migration for VMAX3  https://review.openstack.org/33077819:40
*** takedakn has quit IRC19:42
Malkholmhemna, thank you, here it is - http://paste.openstack.org/raw/554253/19:44
Malkholmdamn, it cut off19:45
hemnahehe19:46
Malkholmhttp://paste.openstack.org/raw/554254/19:46
hemnamost folks nuke all the commented out stuff19:46
hemnaMalkholm, so yah19:48
hemnaI'd clean that file up19:48
hemnayou have [DEFAULT] section variables declared after your driver sections.   that'd break things.19:48
MalkholmYeah, I've read a lot about, but have no idea where should I insert a backends sections19:50
Malkholmhttp://paste.openstack.org/raw/554256/ - without comments, pure config19:50
smcginnisMalkholm: Here's one of mine (non-Ceph) so you can see a simplified layout: http://paste.openstack.org/show/554257/19:51
hemnagoosh19:51
hemnathat config is all borked19:51
smcginnisHmm, yeah. That needs a little cleanup I think.19:51
hemnahehe yah19:51
hemnaall of your default options need to live in [DEFAULT]19:52
hemnathat can't be mixed after other sections19:52
hemnaeach section has to be clean19:52
MalkholmAm I understanding correctly: I have to remove all comments and unused things from the cinder.conf19:55
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - multi-attach failure to VMAX3 when SLO is omitted  https://review.openstack.org/30895219:55
*** e0ne has joined #openstack-cinder19:55
*** lpetrut has joined #openstack-cinder19:55
smcginnisMalkholm: No, you don't have to. But it might make it easier to get rid of comments that aren't necessary and config settings for things that aren't being used.19:55
*** lprice has quit IRC19:57
*** lprice has joined #openstack-cinder19:59
*** catintheroof has joined #openstack-cinder20:00
*** porrua has quit IRC20:02
openstackgerritIvan Kolodyazhny proposed openstack/os-brick: Fix linuxrbd to work with Python 3  https://review.openstack.org/35415220:04
*** xyang has quit IRC20:05
*** e0ne has quit IRC20:08
*** tyrola has joined #openstack-cinder20:09
*** tyrola has quit IRC20:09
*** xyang has joined #openstack-cinder20:15
*** merooney has quit IRC20:18
*** karthik__ has joined #openstack-cinder20:18
*** Apoorva has quit IRC20:20
nikeshmhemna: hi20:21
nikeshmyou said to ping once ci passed :)20:21
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Add tracing unit tests  https://review.openstack.org/35436420:21
hemnayah just haven't gotten to it yet20:22
*** akerr has quit IRC20:22
*** e0ne has joined #openstack-cinder20:23
hemnawtf is up with the VMware NSX CI20:23
hemnait fails like instantly and doesn't post logs20:23
hemnahttps://review.openstack.org/#/c/354364/20:23
hemnahttp://208.91.1.172/logs/dsvm-ext-os-brick/354364/1/15/20:23
hemnapewp!20:23
openstackgerritIvan Kolodyazhny proposed openstack/os-brick: Fix linuxrbd to work with Python 3  https://review.openstack.org/35415220:26
*** xyang1 has quit IRC20:28
*** Malkholm has quit IRC20:28
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick: Mask out passwords when tracing  https://review.openstack.org/35437020:28
nikeshmhemna: http://208.91.1.172/logs/dsvm-ext-os-brick/   upto here20:29
nikeshmonly working20:29
*** karthik__ has quit IRC20:30
nikeshm354364 is missing20:30
*** eharney has quit IRC20:30
*** karthik__ has joined #openstack-cinder20:31
*** lpetrut has quit IRC20:31
hemnaodd20:31
hemnamaybe vmware ran out of vms20:31
*** e0ne has quit IRC20:32
nikeshmhemna: this is not working http://208.91.1.172/logs/dsvm-ext-os-brick/353892/4/12/20:34
*** karthik__ has quit IRC20:34
nikeshmalso console.log20:35
hemnahttp://208.91.1.172/logs/dsvm-ext-os-brick/353892/20:35
hemnayah it's odd20:35
nikeshmis missing20:35
hemnathere is no patchset 420:35
hemnaonly 220:35
*** eharney has joined #openstack-cinder20:35
*** xyang1 has joined #openstack-cinder20:35
*** ebalduf has quit IRC20:38
*** Apoorva has joined #openstack-cinder20:40
*** timcl has quit IRC20:40
*** akapil has joined #openstack-cinder20:44
*** xyang has quit IRC20:45
*** ametts has quit IRC20:45
*** daneyon_ has joined #openstack-cinder20:47
*** xyang has joined #openstack-cinder20:48
*** sdake has quit IRC20:48
*** daneyon has quit IRC20:50
*** rlrossit has joined #openstack-cinder20:51
*** ametts has joined #openstack-cinder20:51
*** karthik__ has joined #openstack-cinder20:51
*** karthik__ has quit IRC20:52
*** eharney has quit IRC20:53
*** daneyon has joined #openstack-cinder20:53
*** xinli has joined #openstack-cinder20:54
*** lpetrut has joined #openstack-cinder20:56
*** daneyon_ has quit IRC20:56
*** ametts has quit IRC20:56
*** cknight has quit IRC20:58
*** dustins has quit IRC20:58
*** bwallis has quit IRC20:59
openstackgerritNate Potter proposed openstack/python-cinderclient: Fix useless api_version of Manager class  https://review.openstack.org/35439521:01
*** e0ne has joined #openstack-cinder21:02
*** e0ne has quit IRC21:05
*** rlrossit has quit IRC21:08
*** pauloewerton has quit IRC21:09
ntpttr___scottda: ping21:10
*** seldenr has quit IRC21:12
ntpttr___scottda: I've read all your comments on https://review.openstack.org/#/c/309283, and started trying to break up the patches into api_version fix and upload-to-image fix, but the issue isn't that one is dependent on the other, it's that they're codependent. Fixing just the api_version in manager alone exposes the flaw in upload-to-image logic and causes all of its tests to fail, and fixing just upload-to-image without fixing the api_version bug21:14
ntpttr___makes the tests fail as well, because line 127 in this file can't work without the api_version fix https://review.openstack.org/#/c/309283/26/cinderclient/v3/volumes.py21:14
ntpttr___sorry for that wall of text21:14
ntpttr___is breaking it up into two patches and adding dependencies both ways a possibility? It seems like neither would be able to merge because of a bit of a chicken and the egg problem21:15
*** bwallis has joined #openstack-cinder21:16
*** sdake has joined #openstack-cinder21:16
*** arch-nemesis has joined #openstack-cinder21:17
ntpttr___I could mock the manager.api_version call in the tests but that seems like a bit of a bandaid, and we'd have to take it out again after both patches were merged21:18
scottdantpttr___: No, you cannot have circular dependencies...21:18
ntpttr___I think merging both things at once might be the best way to do it, even if it's not ideal21:19
ntpttr___to break it up into two patches would require some hacky mocking to get tests to be happy21:19
scottdantpttr___: Or we could merge the first patch (i.e. the manager fix) with the failing upload-to-image tests skipped (and a note explaining)and then merge the second for upload-to-image with the tests un-skipped...21:20
scottdantpttr___: I'm not sure which is more of a hack :)21:21
*** coolsvap has quit IRC21:21
ntpttr___scottda: sure I could do that, is skipping a test a decorator on the function or is there another way to do it? I've never added that before21:21
scottdaI'd actually appreciate some other core having a look, and telling me if I'm just being a P.I.T.A.21:22
*** laughterwym has joined #openstack-cinder21:23
scottda@test.testtools.skip("SKIP until ISER driver is removed or fixed")21:23
scottdalike that ^^21:23
ntpttr___scottda: okay, I'll hold off on a rebase for now until we get a second opinion, but I can do it if it's the best thing for sure21:24
*** satya has joined #openstack-cinder21:24
scottdantpttr___: Thanks for the work on this (as well as Cao)21:25
ntpttr___scottda: happy to help while Cao is on break21:25
*** sgotliv has joined #openstack-cinder21:27
*** sgotliv has quit IRC21:27
*** laughterwym has quit IRC21:28
*** ebalduf has joined #openstack-cinder21:29
*** sdake has quit IRC21:33
*** sgotliv has joined #openstack-cinder21:41
*** sgotliv has quit IRC21:41
*** lprice has quit IRC21:43
*** Lee1092 has quit IRC21:43
*** ksumit has quit IRC21:45
*** jwcroppe_ has quit IRC21:46
*** jwcroppe has joined #openstack-cinder21:47
*** jwcroppe_ has joined #openstack-cinder21:49
*** JoseMello has quit IRC21:50
*** jwcroppe has quit IRC21:51
openstackgerritNate Potter proposed openstack/python-cinderclient: Add "start_version" and "end_version" support to  argparse  https://review.openstack.org/34012921:52
*** baumann has quit IRC21:53
*** jwcroppe_ has quit IRC21:54
*** haplo37__ has quit IRC21:54
*** laughterwym has joined #openstack-cinder21:55
*** mriedem has quit IRC21:58
*** laughterwym has quit IRC22:00
*** gouthamr has quit IRC22:00
*** gouthamr has joined #openstack-cinder22:01
*** xinli has quit IRC22:02
*** alyson_ has quit IRC22:03
*** edmondsw has quit IRC22:03
*** xyang has quit IRC22:05
*** gouthamr has quit IRC22:08
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Cleanup of Initiator group fails  https://review.openstack.org/34783822:11
*** xyang1 has quit IRC22:12
*** lpetrut has quit IRC22:14
*** asselin_ has quit IRC22:18
*** tbarron is now known as tbarron|gone22:19
*** ntpttr has quit IRC22:31
*** satya has quit IRC22:31
*** catintheroof has quit IRC22:35
*** ntpttr has joined #openstack-cinder22:36
*** ducttape_ has quit IRC22:37
*** takedakn has joined #openstack-cinder22:38
scottdantpttr___:  See if my latest comments make sense. IF need be, Gorka and I can arm wrestle....22:41
*** takedakn has quit IRC22:42
*** rkrum has joined #openstack-cinder22:44
ntpttr___scottda: I'll test it out a bit, I think as long as a needless exception isn't thrown and the help message reads well it'll be okay to have it return null there22:45
*** karthik__ has joined #openstack-cinder22:46
*** karthik__ has quit IRC22:47
ntpttr___I do think it sort of makes sense though, if there is no version it can be represented as any version, e.g. *.*22:48
*** jwcroppe has joined #openstack-cinder22:50
*** mriedem has joined #openstack-cinder22:53
scottdantpttr___: Yeah, that part isn't that weird. It's turning is_null() to *.* directly in the get_string() that bugs me...22:54
*** xyang has joined #openstack-cinder22:54
ntpttr___for sure, I can see that. I'll take a look at changing it once I've finished this kolla deployment I'm working on22:55
scottdantpttr___: Cool. Thanks for sticking with this.22:55
*** karthikp- has quit IRC22:59
*** jwcroppe has quit IRC22:59
DuncanTAnybody know much about the internals of the unit test loader code? I'm wondering if we can use unittest.TestLoader sortTestMethodsUsing to randomise the test running order to try to catch unintentional dependencies?23:03
*** leeantho has quit IRC23:03
*** cknight has joined #openstack-cinder23:03
*** xyang has quit IRC23:04
*** fernnest has quit IRC23:05
*** lprice has joined #openstack-cinder23:07
*** tpsilva has quit IRC23:08
*** fernnest__ has quit IRC23:10
*** merooney has joined #openstack-cinder23:13
guitarzannot me, but that is a great idea23:16
*** akapil has quit IRC23:17
*** satya has joined #openstack-cinder23:18
*** bwallis has quit IRC23:20
*** lprice1 has joined #openstack-cinder23:20
*** ducttape_ has joined #openstack-cinder23:20
*** diablo_rojo1 has quit IRC23:21
*** diablo_rojo has joined #openstack-cinder23:22
*** lprice has quit IRC23:22
*** arch-nemesis has quit IRC23:23
*** diablo_rojo has quit IRC23:23
openstackgerritMerged openstack/cinder: Concurrency issue in K2 iSCSI and FC Cinder drivers  https://review.openstack.org/35018223:24
*** kragniz1 has joined #openstack-cinder23:26
openstackgerritMerged openstack/cinder: Mask out passwords when tracing  https://review.openstack.org/35313723:27
*** kragniz1 is now known as testing123423:27
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/35429323:36
*** satya has quit IRC23:40
openstackgerritOpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext: Updated from global requirements  https://review.openstack.org/35428423:42
*** akapil has joined #openstack-cinder23:44
*** sdague has quit IRC23:55
*** Julien-zte has quit IRC23:55
*** akapil has quit IRC23:55
*** jwcroppe has joined #openstack-cinder23:56
*** bwallis has joined #openstack-cinder23:57

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