Friday, 2017-03-10

*** Fdaisuke has quit IRC00:14
*** yangyapeng has quit IRC00:14
*** markvoelker has quit IRC00:15
*** jose-phillips has quit IRC00:17
*** lamt has quit IRC00:19
*** crushil has joined #openstack-cinder00:26
*** Julien-zte has joined #openstack-cinder00:27
*** catintheroof has quit IRC00:29
*** mriedem has quit IRC00:29
*** jose-phillips has joined #openstack-cinder00:29
*** catintheroof has joined #openstack-cinder00:30
*** catintheroof has quit IRC00:30
*** salv-orlando has quit IRC00:30
*** crushil has quit IRC00:32
openstackgerritzhangdaolong proposed openstack/cinder master: RBD driver doesn't flatten image when clone depth > configured value.  https://review.openstack.org/40191400:33
*** ducttape_ has quit IRC00:34
*** wanghao has joined #openstack-cinder00:35
*** wanghao has quit IRC00:35
*** wanghao has joined #openstack-cinder00:35
*** jamielennox is now known as jamielennox|away00:36
*** jamielennox|away is now known as jamielennox00:37
*** catintheroof has joined #openstack-cinder00:42
*** diablo_rojo has quit IRC00:42
*** jiaopengju has joined #openstack-cinder00:43
*** gcb has quit IRC00:44
*** jose-phillips has quit IRC00:44
*** jose-phillips has joined #openstack-cinder00:53
*** diablo_rojo has joined #openstack-cinder00:53
*** jose-phillips has quit IRC00:53
*** jose-phillips has joined #openstack-cinder00:54
*** zhurong has joined #openstack-cinder00:56
openstackgerritTommyLike proposed openstack/cinder master: [Optimise] Refactor ExtraSpecsOpsTestCase with ddt.  https://review.openstack.org/44405201:07
*** crushil has joined #openstack-cinder01:09
*** jiaopengju has quit IRC01:10
openstackgerritzhangdaolong proposed openstack/cinder master: RBD driver doesn't flatten image when clone depth > configured value.  https://review.openstack.org/44405401:11
*** markvoelker has joined #openstack-cinder01:15
*** jiaopengju has joined #openstack-cinder01:16
*** yangyapeng has joined #openstack-cinder01:18
*** jiaopengju has quit IRC01:20
*** markvoelker has quit IRC01:20
*** yangyapeng has quit IRC01:22
*** mriedem has joined #openstack-cinder01:23
*** Julien-zte has quit IRC01:23
*** yangyapeng has joined #openstack-cinder01:23
*** Julien-zte has joined #openstack-cinder01:24
*** lixiaoy1 has joined #openstack-cinder01:27
*** alonma has joined #openstack-cinder01:29
*** alonma has quit IRC01:33
*** ducttape_ has joined #openstack-cinder01:34
openstackgerritzhangdaolong proposed openstack/cinder master: RBD driver doesn't flatten image when clone depth > configured value.  https://review.openstack.org/44405801:35
*** ducttape_ has quit IRC01:40
openstackgerritTommyLike proposed openstack/cinder master: [Optimise] Refactor group snapshot testcases  https://review.openstack.org/44405901:41
*** frontrunner has joined #openstack-cinder01:43
*** gouthamr has quit IRC01:48
*** gouthamr has joined #openstack-cinder01:49
*** crushil has quit IRC01:58
*** ducttape_ has joined #openstack-cinder02:05
*** Jack_Iv has joined #openstack-cinder02:06
*** Jack_Iv has quit IRC02:11
*** wanghao has quit IRC02:15
openstackgerritMerged openstack/cinder master: Set backup available after verify  https://review.openstack.org/44025502:16
*** markvoelker has joined #openstack-cinder02:17
*** kaisers_ has joined #openstack-cinder02:17
openstackgerritMasaki Hatada proposed openstack/os-brick master: remove_multipath_device() should wait until iSCSI volumes removal  https://review.openstack.org/42854402:17
*** Poornima has joined #openstack-cinder02:20
*** kaisers_ has quit IRC02:22
*** markvoelker has quit IRC02:23
*** yangyape_ has joined #openstack-cinder02:23
openstackgerritMasaki Hatada proposed openstack/os-brick master: remove_multipath_device() should wait for SCSI device removal  https://review.openstack.org/42854402:24
*** yangyapeng has quit IRC02:25
*** _ducttape_ has joined #openstack-cinder02:28
*** ducttape_ has quit IRC02:30
*** raunak has quit IRC02:37
*** wanghao has joined #openstack-cinder02:37
*** sudipto has joined #openstack-cinder02:41
*** jiaopengju has joined #openstack-cinder02:42
openstackgerritzhangdaolong proposed openstack/cinder master: RBD driver doesn't flatten image when clone depth > configured value.  https://review.openstack.org/44405802:46
*** jiaopengju has quit IRC02:52
*** dhinesh has quit IRC02:57
*** Apoorva has quit IRC02:57
*** _ducttape_ has quit IRC03:02
*** sudipto has quit IRC03:03
openstackgerritTommyLike proposed openstack/cinder-specs master: Support regexp based filter  https://review.openstack.org/44298203:06
openstackgerritzhangdaolong proposed openstack/os-brick master: The rbd_handle socket should be closed before processing ends or throwing  exception.  https://review.openstack.org/44407003:06
*** asettle has quit IRC03:07
*** frontrunner has quit IRC03:09
*** lucasxu has joined #openstack-cinder03:10
*** raunak has joined #openstack-cinder03:16
*** sdague has joined #openstack-cinder03:23
*** yangyape_ has quit IRC03:29
*** ducttape_ has joined #openstack-cinder03:30
*** alonma has joined #openstack-cinder03:30
*** lucasxu has quit IRC03:31
openstackgerritJohn Griffith proposed openstack/python-cinderclient master: Make V3 the default and fixup version reporting  https://review.openstack.org/44407603:31
jgriffithsmcginnis jungleboyj  ^^. With that I think we're good to cut a release if you guys are cool with it03:32
*** alonma has quit IRC03:34
*** yangyapeng has joined #openstack-cinder03:34
*** alonma has joined #openstack-cinder03:36
openstackgerritOpenStack Proposal Bot proposed openstack/python-brick-cinderclient-ext master: Updated from global requirements  https://review.openstack.org/44009203:37
*** links has joined #openstack-cinder03:38
*** lucasxu has joined #openstack-cinder03:40
*** alonma has quit IRC03:40
*** Poornima_K has joined #openstack-cinder03:41
*** crushil has joined #openstack-cinder03:41
*** Julien-zte has quit IRC03:41
*** Julien-zte has joined #openstack-cinder03:42
*** prashkre has joined #openstack-cinder03:44
*** Poornima has quit IRC03:44
*** ducttape_ has quit IRC03:49
*** catintheroof has quit IRC03:57
*** catintheroof has joined #openstack-cinder03:58
*** catintheroof has quit IRC03:59
*** ducttape_ has joined #openstack-cinder03:59
*** Poornima_K has quit IRC03:59
*** Poornima has joined #openstack-cinder04:00
*** salv-orlando has joined #openstack-cinder04:05
*** Jack_Iv has joined #openstack-cinder04:07
*** nicolasbock has quit IRC04:07
*** salv-orlando has quit IRC04:09
*** korean101 has quit IRC04:10
*** Jack_Iv has quit IRC04:11
*** chenhb has quit IRC04:11
*** chenhb has joined #openstack-cinder04:12
*** ducttape_ has quit IRC04:13
*** sudipto has joined #openstack-cinder04:14
*** david-lyle has quit IRC04:16
*** gcb has joined #openstack-cinder04:19
*** prashkre has quit IRC04:22
*** gkadam has joined #openstack-cinder04:22
*** lucasxu has quit IRC04:24
*** kaisers_ has joined #openstack-cinder04:33
*** sdague has quit IRC04:35
*** mdnadeem has joined #openstack-cinder04:36
*** kaisers_ has quit IRC04:38
*** crushil has quit IRC04:38
*** chhavi has joined #openstack-cinder04:39
*** psachin has joined #openstack-cinder04:42
*** salv-orlando has joined #openstack-cinder05:05
*** salv-orlando has quit IRC05:10
*** BharatK has joined #openstack-cinder05:12
*** jiaopengju has joined #openstack-cinder05:13
*** BharatK has quit IRC05:16
*** jiaopengju has quit IRC05:18
*** BharatK has joined #openstack-cinder05:18
*** BharatK has quit IRC05:20
*** markvoelker has joined #openstack-cinder05:20
*** kaisers_ has joined #openstack-cinder05:23
*** dhinesh has joined #openstack-cinder05:24
*** markvoelker has quit IRC05:25
*** kaisers_ has quit IRC05:25
*** gouthamr has quit IRC05:29
*** BharatK has joined #openstack-cinder05:29
*** jiaopengju has joined #openstack-cinder05:30
*** BharatK_ has joined #openstack-cinder05:31
*** jiaopengju has quit IRC05:35
*** jiaopengju has joined #openstack-cinder05:37
*** alonma has joined #openstack-cinder05:42
*** mriedem has quit IRC05:44
*** BharatK has left #openstack-cinder05:45
*** BharatK_ has left #openstack-cinder05:45
*** BharatK_ has quit IRC05:46
*** alonma has quit IRC05:47
*** BharatK has joined #openstack-cinder05:47
*** sandanar has joined #openstack-cinder05:48
*** BharatK has quit IRC05:48
*** BharatK_ has joined #openstack-cinder05:48
*** Jack_Iv has joined #openstack-cinder05:48
*** BharatK_ has quit IRC05:48
*** BharatK has joined #openstack-cinder05:49
*** Poornima has quit IRC05:49
openstackgerritzhangdaolong proposed openstack/os-brick master: Delete redundant code in rbd driver.  https://review.openstack.org/44407005:51
*** mtanino has joined #openstack-cinder05:57
*** prashkre has joined #openstack-cinder06:04
*** dhinesh has quit IRC06:05
*** salv-orlando has joined #openstack-cinder06:06
*** salv-orlando has quit IRC06:11
*** Shunli has joined #openstack-cinder06:12
openstackgerritjiaohaolin proposed openstack/cinder master: Inspur Cinder FC/iSCSI driver  https://review.openstack.org/43948806:13
*** crushil has joined #openstack-cinder06:14
*** ducttape_ has joined #openstack-cinder06:14
*** ducttape_ has quit IRC06:19
*** markvoelker has joined #openstack-cinder06:21
*** markstur has quit IRC06:21
*** alonmarx has joined #openstack-cinder06:21
*** markstur has joined #openstack-cinder06:22
*** diablo_rojo has quit IRC06:24
openstackgerritDinesh Bhor proposed openstack/cinder master: Remove unreachable code  https://review.openstack.org/44411006:25
*** markvoelker has quit IRC06:25
openstackgerritDinesh Bhor proposed openstack/cinder master: Remove duplicate key from dictionary  https://review.openstack.org/44411106:26
*** markstur has quit IRC06:26
openstackgerritDinesh Bhor proposed openstack/cinder master: Fix 500 error if boolean is_public is passed as string  https://review.openstack.org/44176706:27
*** alonma has joined #openstack-cinder06:29
openstackgerritzhangdaolong proposed openstack/os-brick master: the ioctx should be close when open rbd pool failed. Closes-Bug: #1671711  https://review.openstack.org/44411406:35
openstackbug 1671711 in os-brick "the ioctx should be close when open rbd pool failed." [Undecided,New] https://launchpad.net/bugs/167171106:35
*** rcernin has joined #openstack-cinder06:35
*** nkrinner_afk is now known as nkrinner06:37
*** markvoelker has joined #openstack-cinder06:44
*** BharatK has quit IRC06:46
*** BharatK has joined #openstack-cinder06:46
*** yangyapeng has quit IRC06:47
*** BharatK has quit IRC06:47
*** yangyapeng has joined #openstack-cinder06:47
*** BharatK has joined #openstack-cinder06:47
*** BharatK_ has joined #openstack-cinder06:48
*** markvoelker has quit IRC06:54
*** arne_r has joined #openstack-cinder06:55
*** alonma has quit IRC06:59
openstackgerritzhangdaolong proposed openstack/os-brick master: The rbd socket should be closed before processing ends Closes-Bug: #1671686  https://review.openstack.org/44412307:00
openstackbug 1671686 in os-brick "The rbd socket should be closed before processing ends." [Undecided,New] https://launchpad.net/bugs/1671686 - Assigned to zhangdaolong (zhangdaolong)07:00
*** alonma has joined #openstack-cinder07:00
*** alonma has quit IRC07:05
*** alonma has joined #openstack-cinder07:06
*** chhavi has quit IRC07:06
*** salv-orlando has joined #openstack-cinder07:07
*** BharatK_ has quit IRC07:07
*** raunak has quit IRC07:09
*** raunak has joined #openstack-cinder07:10
*** raunak has quit IRC07:10
*** alonma has quit IRC07:11
*** raunak has joined #openstack-cinder07:11
*** raunak has quit IRC07:11
*** raunak has joined #openstack-cinder07:12
*** salv-orlando has quit IRC07:12
*** raunak has quit IRC07:12
*** salv-orlando has joined #openstack-cinder07:12
*** raunak has joined #openstack-cinder07:12
*** raunak has quit IRC07:13
*** raunak has joined #openstack-cinder07:13
*** alonma has joined #openstack-cinder07:19
*** pgadiya has joined #openstack-cinder07:19
*** Jack_Iv has quit IRC07:20
*** Jack_Iv has joined #openstack-cinder07:20
*** salv-orl_ has joined #openstack-cinder07:23
*** alonma has quit IRC07:23
*** chhavi has joined #openstack-cinder07:25
*** salv-orlando has quit IRC07:26
*** kaisers_ has joined #openstack-cinder07:26
*** alonmarx has quit IRC07:29
*** kaisers_ has quit IRC07:30
*** xek has quit IRC07:37
*** edmondsw has joined #openstack-cinder07:39
*** tesseract has joined #openstack-cinder07:42
*** Cibo has joined #openstack-cinder07:42
*** edmondsw has quit IRC07:44
*** Cibo_ has quit IRC07:45
openstackgerritJustin A Wilson proposed openstack/cinder master: Add v3 delete volume unit test  https://review.openstack.org/44321907:46
*** Cibo has quit IRC07:46
*** ChubYann has quit IRC07:51
*** mlakat has quit IRC07:51
*** mlakat has joined #openstack-cinder07:53
*** mlakat has quit IRC07:54
*** prashkre has quit IRC07:57
*** sudipto has quit IRC08:00
*** mlakat has joined #openstack-cinder08:00
*** BharatK has quit IRC08:01
*** BharatK has joined #openstack-cinder08:01
*** yangyapeng has quit IRC08:03
*** lpetrut has joined #openstack-cinder08:03
*** wanghao has quit IRC08:07
*** wanghao has joined #openstack-cinder08:07
*** lpetrut has quit IRC08:12
*** prashkre has joined #openstack-cinder08:17
*** salv-orl_ has quit IRC08:20
*** BharatK has quit IRC08:24
*** aarefiev_afk is now known as aarefiev08:24
openstackgerritPeter Wang proposed openstack/cinder master: VNX: Add QoS support  https://review.openstack.org/44178608:31
*** BharatK has joined #openstack-cinder08:32
*** bkopilov has quit IRC08:36
*** salv-orlando has joined #openstack-cinder08:36
*** prashkre has quit IRC08:39
*** crushil has quit IRC08:48
openstackgerritchenaidong1 proposed openstack/cinder master: Replace yaml.load() with yaml.safe_load()  https://review.openstack.org/44416008:51
*** lixiaoy1 has quit IRC08:52
*** huanxuan has joined #openstack-cinder08:53
*** wanghao_ has joined #openstack-cinder08:58
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-cinder09:00
*** pgadiya has quit IRC09:02
*** wanghao has quit IRC09:02
*** BharatK_ has joined #openstack-cinder09:09
*** yangyapeng has joined #openstack-cinder09:12
*** BharatK has quit IRC09:13
*** pgadiya has joined #openstack-cinder09:17
*** sandanar has quit IRC09:17
*** Poornima has joined #openstack-cinder09:19
*** markstur has joined #openstack-cinder09:22
*** markstur has quit IRC09:26
*** alonma has joined #openstack-cinder09:27
openstackgerritMerged openstack/cinder master: Remove duplicate key from dictionary  https://review.openstack.org/44411109:27
*** Shunli has quit IRC09:27
*** swapnil-nilangek has joined #openstack-cinder09:30
*** Julien-zte has quit IRC09:30
*** wanghao_ has quit IRC09:30
openstackgerritSoffie Huang proposed openstack/cinder master: Extend Cinder driver to utilize multiple FSS storage pools  https://review.openstack.org/42246209:31
*** wanghao has joined #openstack-cinder09:31
*** alonma has quit IRC09:32
*** Poornima has quit IRC09:33
*** Poornima has joined #openstack-cinder09:33
*** Poornima has quit IRC09:33
*** wanghao has quit IRC09:35
*** pgadiya has quit IRC09:38
*** alonma has joined #openstack-cinder09:39
*** alonma has quit IRC09:44
*** sudipto has joined #openstack-cinder09:47
*** pgadiya has joined #openstack-cinder09:51
*** prashkre has joined #openstack-cinder09:51
*** chhavi has quit IRC09:52
*** pgadiya has quit IRC09:57
*** pgadiya has joined #openstack-cinder10:00
*** Poornima has joined #openstack-cinder10:00
*** asettle has joined #openstack-cinder10:03
*** edmondsw has joined #openstack-cinder10:04
*** pgadiya has quit IRC10:07
openstackgerritSoffie Huang proposed openstack/cinder master: Extend Cinder driver to utilize multiple FSS storage pools  https://review.openstack.org/42246210:09
*** edmondsw has quit IRC10:09
*** zhurong has quit IRC10:10
openstackgerritSoffie Huang proposed openstack/cinder master: Extend Cinder driver to utilize multiple FSS storage pools  https://review.openstack.org/42246210:14
*** Cibo has joined #openstack-cinder10:17
*** Cibo has quit IRC10:22
*** kaisers_ has joined #openstack-cinder10:23
*** pgadiya has joined #openstack-cinder10:23
*** kaisers_ has quit IRC10:23
*** kaisers_ has joined #openstack-cinder10:24
*** BharatK has joined #openstack-cinder10:32
*** openstackgerrit has quit IRC10:33
*** Jack_Iv has quit IRC10:33
*** BharatK has quit IRC10:33
*** Jack_Iv has joined #openstack-cinder10:33
*** huanxuan has quit IRC10:35
*** BharatK_ has quit IRC10:35
*** dalgaaf has quit IRC10:35
*** kaisers_ has quit IRC10:38
*** dalgaaf has joined #openstack-cinder10:40
*** BharatK has joined #openstack-cinder10:41
*** chhavi has joined #openstack-cinder10:46
*** jiaopengju has quit IRC10:50
*** salv-orlando has quit IRC11:02
*** salv-orlando has joined #openstack-cinder11:02
*** salv-orlando has quit IRC11:02
*** Poornima has quit IRC11:06
*** nicolasbock has joined #openstack-cinder11:07
*** markstur has joined #openstack-cinder11:10
*** salv-orlando has joined #openstack-cinder11:11
*** mdovgal has joined #openstack-cinder11:13
*** markstur has quit IRC11:15
*** Julien-zte has joined #openstack-cinder11:16
*** BharatK has quit IRC11:31
*** openstackgerrit has joined #openstack-cinder11:34
openstackgerritMate Lakat proposed openstack/cinder master: Create indexes for foreign keys  https://review.openstack.org/43731311:34
*** links has quit IRC11:37
*** BharatK has joined #openstack-cinder11:41
*** ccesario__ has joined #openstack-cinder11:41
*** swapnil-nilangek has quit IRC11:45
FransUrboIs there any documentation on Cinder drivers which specify how the API is calling functions in the driver?11:52
FransUrboI'm looking at https://docs.openstack.org/developer/cinder/devref/drivers.html#base-driver-interface, but it doesn't state the "flow" of anything.11:53
*** Jack_Iv has quit IRC11:53
*** Jack_Iv has joined #openstack-cinder11:53
*** links has joined #openstack-cinder11:54
*** BharatK has quit IRC11:58
*** chhavi has quit IRC12:07
*** chhavi has joined #openstack-cinder12:08
*** Julien-zte has quit IRC12:10
geguileoFransUrbo: I don't think we have such document12:12
geguileoFransUrbo: the call is received in cinder/api/* then calls cinder/volume/api.py that does an rpc call to cinder/volume/manager.py that call cinder/volume/driver.py or you specific driver12:13
geguileoFransUrbo: you'll have to walk the code to see the flow12:13
*** sileht has quit IRC12:17
*** e0ne has joined #openstack-cinder12:17
*** alonmarx has joined #openstack-cinder12:18
*** sileht has joined #openstack-cinder12:18
*** sileht has quit IRC12:18
FransUrbogeguileo: Ok, thanx :(.12:18
*** sileht has joined #openstack-cinder12:18
*** ducttape_ has joined #openstack-cinder12:19
*** pgadiya has quit IRC12:23
*** ducttape_ has quit IRC12:24
openstackgerritGorka Eguileor proposed openstack/python-cinderclient master: Add service cleanup command  https://review.openstack.org/36300712:26
openstackgerritGorka Eguileor proposed openstack/python-cinderclient master: Add cluster support in migration and manage  https://review.openstack.org/38750012:26
openstackgerritGorka Eguileor proposed openstack/python-cinderclient master: Add cluster support in manage listings  https://review.openstack.org/38750112:26
openstackgerritGorka Eguileor proposed openstack/python-cinderclient master: Fix pep8 errors  https://review.openstack.org/44426412:26
*** Jack_Iv_ has joined #openstack-cinder12:26
FransUrbogeguileo: Any idea why my module successfully completes initialize_connections(), but directly after that, terminate_connection() is called?12:28
FransUrbogeguileo: Any known bug in Cinder v9.0.0?12:29
*** Cibo has joined #openstack-cinder12:29
geguileoFransUrbo: it may be because you are returning something wrong in the conn_info?12:29
*** Jack_Iv has quit IRC12:30
geguileoFransUrbo: are there any error messages?12:30
geguileoin the logs12:30
FransUrboNo :(12:30
geguileoFransUrbo: not even in Nova?12:31
FransUrboOh, haven't checked there.. Hang on.12:32
geguileoFransUrbo: because the attach flow is driven by nova12:32
FransUrboI do get "Attach volume completed successfully".12:33
*** Cibo has quit IRC12:34
*** catintheroof has joined #openstack-cinder12:34
geguileoFransUrbo: you mean in the command line or in the logs?12:35
*** pgadiya has joined #openstack-cinder12:36
FransUrboIn the logs.12:37
geguileoFransUrbo: then it should be attached, unless you see the in the nova logs that it detaches it for an error after that12:37
FransUrboI'll have a look.12:38
geguileoFransUrbo: Use the request id in the logs to follow what's happening12:38
*** kaisers_ has joined #openstack-cinder12:38
FransUrboOk, I will. Thanx. Would that be in the nova-api logs?12:40
geguileoFransUrbo: not likely, because the flow is not driven there12:40
geguileoFransUrbo: you may want to merge all your logs to make it easier to read12:40
*** kaisers_ has quit IRC12:43
FransUrboWell, that might be the problem.12:43
FransUrbo2017-03-10 12:35:19 DEBUG cinder.volume.manager Attaching volume 2a37bc4c-c733-4d65-8daa-ddf28c8a80f8 to instance 6a2396d0-fa07-485c-9725-40dd85853586 at mountpoint /dev/vda on host None. attach_volume /usr/lib/python2.7/dist-packages/cinder/volume/manager.py:102312:43
FransUrboBut that instance don't exist!12:43
*** gkadam has quit IRC12:44
*** e0ne has quit IRC12:48
*** salv-orlando has quit IRC12:48
geguileoFransUrbo: well, that's weird, because Nova should have checked that before trying to do the attach12:50
*** frontrunner has joined #openstack-cinder12:55
*** bhagyashri_s has quit IRC12:58
*** markstur has joined #openstack-cinder12:59
*** salv-orlando has joined #openstack-cinder13:00
*** liverpooler has joined #openstack-cinder13:00
*** ducttape_ has joined #openstack-cinder13:00
*** sileht has quit IRC13:03
*** sileht has joined #openstack-cinder13:03
*** sileht has quit IRC13:03
*** sileht has joined #openstack-cinder13:03
*** sdague has joined #openstack-cinder13:03
*** markstur has quit IRC13:04
*** sileht has quit IRC13:08
*** sileht has joined #openstack-cinder13:08
*** sileht has quit IRC13:08
*** gcb has quit IRC13:08
*** sileht has joined #openstack-cinder13:10
smcginnisjgriffith: Testing version-list and it's just hanging for me.13:14
*** sudipto has quit IRC13:16
*** Cibo has joined #openstack-cinder13:16
*** links has quit IRC13:17
openstackgerritAdriano Freires Rosso proposed openstack/cinder master: Remove HNAS iSCSI driver  https://review.openstack.org/44428713:18
*** lpetrut has joined #openstack-cinder13:19
smcginnisjgriffith: Auth finally timed out. Config issues on my end. Nothing to see here, move along.13:23
*** salv-orl_ has joined #openstack-cinder13:23
*** edmondsw has joined #openstack-cinder13:23
*** Cibo_ has joined #openstack-cinder13:24
*** pgadiya has quit IRC13:24
*** salv-orl_ has quit IRC13:24
*** kfarr has quit IRC13:24
*** salv-orl_ has joined #openstack-cinder13:25
*** salv-orlando has quit IRC13:26
*** kaisers_ has joined #openstack-cinder13:27
*** Cibo_ has quit IRC13:28
*** wanghao has joined #openstack-cinder13:29
*** links has joined #openstack-cinder13:31
openstackgerritGorka Eguileor proposed openstack/os-brick master: Fix FC rescan for targets with single WWNN  https://review.openstack.org/43366613:31
openstackgerritGorka Eguileor proposed openstack/os-brick master: Change log level on _get_hba_channel_scsi_target  https://review.openstack.org/42474713:31
smcginnisgeguileo: Hey, if you have a moment, mind taking a look at a small driver-internal change that should make my CI happier? https://review.openstack.org/#/c/443775/113:32
*** dave-mccowan has joined #openstack-cinder13:32
* geguileo looking ^13:33
smcginnisgeguileo: Thanks!13:33
openstackgerritAdriano Freires Rosso proposed openstack/cinder master: Remove HNAS iSCSI driver  https://review.openstack.org/44428713:34
*** e0ne has joined #openstack-cinder13:38
*** links has quit IRC13:39
*** alonma has joined #openstack-cinder13:41
ildikovsmcginnis: jgriffith: it would be great to clean up the open patches and the client code too if we decide with a direction to go13:41
smcginnisildikov: Which ones do you mean?13:42
ildikovsmcginnis: jgriffith: and feel free to educate me any time how I'm supposed to use the final version in Nova :)13:42
hemnayough13:43
smcginnishemna: You're early today.13:43
hemnadifferent timezone today :)13:43
smcginnishemna: Ah, nice! :)13:44
hemna:)13:44
*** alonma has quit IRC13:45
*** kaisers_ has quit IRC13:46
*** lamt has joined #openstack-cinder13:48
*** alonmarx_ has joined #openstack-cinder13:48
*** BharatK has joined #openstack-cinder13:50
*** BharatK_ has joined #openstack-cinder13:51
ildikovsmcginnis: never mind, I see everything is on the gate now13:51
*** alonmarx has quit IRC13:51
ildikovsmcginnis: I will need the education part in this case :)13:51
ildikovsmcginnis: I will check the code once the related patches get merged and hopefully I'll be able to figure it out13:52
*** BharatK has quit IRC13:52
*** e0ne has quit IRC13:53
*** e0ne has joined #openstack-cinder13:53
*** dustins has joined #openstack-cinder13:57
*** Julien-zte has joined #openstack-cinder13:58
*** wanghao has quit IRC14:03
*** salv-orl_ has quit IRC14:08
openstackgerritIvan Kolodyazhny proposed openstack/python-cinderclient master: WIP. Fix noath support  https://review.openstack.org/42714314:08
*** mriedem has joined #openstack-cinder14:11
*** xyang1 has joined #openstack-cinder14:12
*** porrua has joined #openstack-cinder14:13
ildikovsmcginnis: jgriffith: I'm not sure about the version info here: https://review.openstack.org/#/c/444076/1/cinderclient/__init__.py or if this one is correct then about the one in api_versions.py14:13
smcginnisildikov: That's the current highest version. Let me double check that.14:15
smcginnisildikov: Ah, I see. api_versions.py needs to be updated I think.14:16
ildikovsmcginnis: ok :)14:16
smcginnisildikov: Good catch! ;)14:16
*** prashkre has quit IRC14:16
ildikovsmcginnis: I'm a consistency freak :)14:17
*** nkrinner is now known as nkrinner_afk14:17
*** dolphm has left #openstack-cinder14:17
ildikovsmcginnis: also love nit picking... :)14:17
smcginnisildikov: I'm glad you are.14:18
*** ducttape_ has quit IRC14:18
ildikovsmcginnis: good you have my back :)14:18
*** salv-orlando has joined #openstack-cinder14:18
*** salv-orlando has quit IRC14:20
*** kaisers_ has joined #openstack-cinder14:20
ildikovsmcginnis: and thanks for the reno check!14:20
smcginnisildikov: Speaking of nit picking. :D14:20
*** sudipto has joined #openstack-cinder14:21
openstackgerritJohn Griffith proposed openstack/python-cinderclient master: Make V3 the default and fixup version reporting  https://review.openstack.org/44407614:21
ildikovsmcginnis: my fat fingers are always mistyping something... :)14:22
smcginnisHah14:22
*** Dinesh_Bhor has quit IRC14:23
*** chhavi has quit IRC14:23
*** gkadam has joined #openstack-cinder14:24
*** sdague has quit IRC14:24
*** kaisers_ has quit IRC14:25
*** gouthamr has joined #openstack-cinder14:26
openstackgerritMerged openstack/python-cinderclient master: Fix discover_version  https://review.openstack.org/38562914:26
smcginnisOne down14:26
*** tommylikehu_ has joined #openstack-cinder14:26
*** e0ne has quit IRC14:27
*** mdnadeem has quit IRC14:27
*** e0ne has joined #openstack-cinder14:27
ildikov:)14:28
*** e0ne has quit IRC14:28
*** e0ne has joined #openstack-cinder14:29
*** jdillaman has joined #openstack-cinder14:29
smcginnisHaha, and now a merge conflict for the rest. jgriffith, I feel for you.14:30
*** nikeshm has joined #openstack-cinder14:31
nikeshmhi14:32
*** chhavi has joined #openstack-cinder14:37
ildikovsmcginnis: BTW why do we need get_highest_version and get_highest_client_server_version too?14:38
ildikovsmcginnis: I mean this: https://review.openstack.org/#/c/385641/4/cinderclient/api_versions.py14:38
ildikovsmcginnis: and this: https://review.openstack.org/#/c/385629/11/cinderclient/client.py14:38
smcginnisildikov: Good question.14:40
openstackgerritJohn Griffith proposed openstack/python-cinderclient master: Make V3 the default and fixup version reporting  https://review.openstack.org/44407614:40
smcginnisildikov: I think so, since those are client (library consumer) calls, vs the one that's a CLI command. But maybe a little too much overlap there?14:41
ildikovsmcginnis: I just got confused which part is for what purpose as by the end of the day we all just want a version number...14:42
openstackgerritIvan Kolodyazhny proposed openstack/python-cinderclient master: WIP. Fix noath support  https://review.openstack.org/42714314:42
smcginnisildikov: I know, seems like a pretty simple thing, right?14:43
*** cknight has joined #openstack-cinder14:43
ildikovsmcginnis: yeah, that's what I would think :)14:43
*** crushil has joined #openstack-cinder14:44
*** kaisers_ has joined #openstack-cinder14:46
ildikovsmcginnis: TBH, if it would be on me we could just use whatever version the client can offer and choose the internal flows in Nova whether that number is high enough or not14:46
*** arne_r has quit IRC14:46
ildikovs/whether/when/14:46
jgriffithyes we need both14:47
jgriffithabsolutely need both14:47
jgriffithbecause you could have a client that only supports .28... but your sever reports back it goes up to 4,98714:48
jgriffithuser needs to be able to sort that14:48
*** markstur has joined #openstack-cinder14:48
jgriffithI was doing so good pulling these together without actually logging in, but alas the rebase killed me14:48
smcginnisI see, makes sense.14:48
smcginnis:)14:48
smcginnisjgriffith: You can't escape!14:49
jgriffithI know right! :)14:49
jgriffithself-inflicted rebase problems are the worst problems :)14:49
jgriffithshit14:49
jgriffithalright, that rebase might have to wait til I get in the office.. sorry :(14:50
ildikovjgriffith: I'm not sure I follow in the sense of I would guess the client knows best what's the highest it can support and I will talk to the client14:50
jgriffithildikov sure, the client knows because you set that variable :)14:50
jgriffithand you need to tell the user somehow right?14:50
ildikovjgriffith: I set what I want and then it tells me I cannot get it or maybe I can14:51
jgriffithildikov the only way the client really *knows* if it supports the micro-version is if somebody explicitly adds support for it14:51
ildikovjgriffith: I mean now we're playing with static methods so that I can ask what I can ask for14:51
jgriffith?14:52
ildikovjgriffith: I'm guessing we're talking about two different things now :)14:52
jgriffiththat would explain my confusion :)14:52
ildikovmine too :)14:52
*** markstur has quit IRC14:53
jgriffithalright, I'll be back in a bit14:53
jgriffithcya14:53
ildikovjgriffith: anyhow, I don't want to keep you from your morning routine14:53
ildikovjgriffith: will upload the microversion part of Scott's Nova patch and then we can get back to making me understand what's going on here :)14:53
ildikovjgriffith: or just help me get the thing merged and then we can skip the my understanding part, although I would prefer to understand still :)14:54
nikeshmsmcginnis:  do we have to run v1 tests in CI14:54
smcginnisnikeshm: No, v1 is deprecated now and off by default.14:55
nikeshmfor me some v1 tests are failing14:55
*** kaisers_ has quit IRC14:55
smcginnisnikeshm: It would be good for you to understand why they are failing to know if it's a real issue that needs to be fixed, but normal CI runs don't need to run the v1 tests.14:56
*** ducttape_ has joined #openstack-cinder14:56
openstackgerritIvan Kolodyazhny proposed openstack/python-cinderclient master: Disable functional tests with multiattach  https://review.openstack.org/44434114:57
nikeshmsmcginnis: ok, http://80.179.89.186:8080/job/kaminario-dsvm-tempest-full-iscsi/2154/consoleFull, some mismatch error, but it is passing in v214:58
*** crose has joined #openstack-cinder14:58
openstackgerritMerged openstack/python-cinderclient master: Remove cinder credentials command  https://review.openstack.org/39065814:59
* e0ne wants twitter rss stream with jgriffith's comments in the code. they are awesome15:00
* scottda pokes his head in the room and looks around....15:00
*** catintheroof has quit IRC15:00
smcginnise0ne: :)15:00
smcginnisscottda!!15:00
scottdaHi15:00
*** catintheroof has joined #openstack-cinder15:01
smcginnisscottda: How's things in the land of "completely abandoning us"? :)15:01
*** lamt has quit IRC15:01
scottdaThings are bat-sh$!t crazy15:01
scottdaI stopped in for a breath of sane air15:01
ildikove0ne: +1 :)15:02
smcginnisscottda: You're just saying that to make us feel better. :D15:02
ildikovscottda: hi :)15:02
scottdaNo, I'm not incapable of flattery, but I really mean it.15:03
scottdaThis world I'm in is completely upside-down crazy15:03
scottdaildikov: Hi15:03
scottdasmcginnis: Sorry I've been so completely checked-out. I'll keep an IRC window open, if someone pings me...and hopefully get back to some reviews.15:04
scottdaI need to keep an eye on Cinder. I've had 3 talks accepted for Boston!15:04
smcginnisscottda: I'm sure you're busy with the new stuff. But we'll take whatever time you can give us.15:04
*** david-lyle has joined #openstack-cinder15:05
smcginnisscottda: I saw at least one of your talks were accepted. Are you going to be able to go to Boston?15:05
scottdaYeah, I was hoping to be heads-down and meet our delivery dates, but tops-down management keeps ramping up the scope, and not changing the schedule, so 'ef them.15:05
scottdasmcginnis: I will definitely be in Boston, one way or another.15:05
smcginnisscottda: Awesome15:06
scottdaOK, you lot get back to work . I'm off to the kids bus stop :)15:06
e0neI've got an approved talk but still didn't get budget approval :(15:06
scottdae0ne: Yes, we are talking together, with hemna :)15:06
e0ne:)15:06
hemnamep15:06
smcginnise0ne: Really? :/15:06
ildikovscottda: that's awesome! I mean the Boston part15:07
ildikovscottda: management is usually difficult... :/ :(15:07
e0nescottda, hemna, smcginnis: I hope to get approval next week15:07
smcginnise0ne: I'll keep my fingers crossed for you.15:07
e0nesmcginnis: thanks:)15:07
hemnae0ne, I was just starting to work on a patch for the brick cinderclient extension to conditionally add FC support15:07
e0nehemna: oh, great!15:08
hemnae0ne, I saw your WIP for the noauth support in the client15:08
*** lamt has joined #openstack-cinder15:08
scottdae0ne: We can talk next week either way, Tuesday or later (I'm FTO mon). I'll ping you...15:08
hemnaI'm also going to start from scratch and see what effort is needed to run Cinder from a stock ubuntu 16.0415:08
*** david-lyle_ has joined #openstack-cinder15:08
hemnahaven't done that in ages.15:08
e0nehemna: yep, it works on my env now with a keystoneauth plugin. I have to cleanup the code and fix unit tests15:08
*** david-lyle_ has quit IRC15:08
hemnaw/o checking out from source15:08
*** david-lyle has quit IRC15:08
*** lucasxu has joined #openstack-cinder15:09
e0nehemna: good idea15:09
hemnaubuntu 16.04 has mitaka bits :(15:09
hemnaI'm going to start an etherpad and take some notes that we can use in the preso.15:09
*** lamt has quit IRC15:10
e0neI do the same, but with devstack now: https://gist.github.com/e0ne/10876c05d87ee96a82e70c3a160960bf15:10
smcginnishemna: You can at least get the Ocata bits following here: https://docs.openstack.org/ocata/install-guide-ubuntu/environment-packages.html15:10
hemnae0ne, ah ok, so you still use keystone15:11
e0nehemna: it's a devstack:(15:11
*** gkadam has quit IRC15:11
e0nehemna: it can't setup cinder w/o keystone15:11
hemnayah I'd like to get it working w/o keystone15:11
geguileopatrickeast: jungleboyj pure CI is happy with it, could you review it again, please?  https://review.openstack.org/43310415:11
e0neI have to stop keystone manually15:11
hemnageguileo, sweet.  pure's multipath ci passed.  :)15:12
e0nehemna: you can try my or Gorka's patch for cinderclient with noauth15:12
geguileohemna: yeah, I was really happy about it :-)15:12
*** mtanino has quit IRC15:12
jungleboyjgeguileo:  Looking.  Hey, yay!  I am glad that open check was a legit check.15:13
*** eharney has joined #openstack-cinder15:13
* jungleboyj caught something in Gorka's code! :-)15:14
geguileojungleboyj: Yes it was a big slip on my part.  XD15:14
geguileojungleboyj: thanks for that!15:14
openstackgerritMerged openstack/cinder master: Remove unreachable code  https://review.openstack.org/44411015:14
*** chlong_ has joined #openstack-cinder15:15
jungleboyjgeguileo:  My pleasure.  We don't care if it fails though?15:15
geguileojungleboyj: we only care if they all fail15:16
geguileojungleboyj: so we have the else clause to raise the exception if we don't break out of the look on L89215:17
geguileos/look/loop15:17
jungleboyjAh, I see it now.  Thanks!  Was off the bottom of the screen.15:17
jungleboyjhemna: Should I +A?15:18
*** lucasxu has quit IRC15:20
*** chhavi has quit IRC15:20
*** shaner has quit IRC15:21
*** Jack_Iv_ has quit IRC15:24
*** shaner has joined #openstack-cinder15:25
openstackgerritIvan Kolodyazhny proposed openstack/python-cinderclient master: Remove unused and duplicated fake_client module  https://review.openstack.org/44435215:28
*** Julien-zte has quit IRC15:29
*** markstur has joined #openstack-cinder15:30
*** Jack_Iv has joined #openstack-cinder15:30
*** Julien-zte has joined #openstack-cinder15:31
*** Jack_I has joined #openstack-cinder15:31
*** lucasxu has joined #openstack-cinder15:32
*** mriedem1 has joined #openstack-cinder15:33
hemnajungleboyj, yah15:34
*** ducttape_ has quit IRC15:34
*** mriedem has quit IRC15:36
jungleboyjhemna: Done.  Merging patching on my vacation day.  :-)15:37
*** diablo_rojo has joined #openstack-cinder15:37
hemna:)15:37
openstackgerritJohn Griffith proposed openstack/python-cinderclient master: Add get_highest_version method  https://review.openstack.org/38564115:39
*** diablo_rojo has quit IRC15:41
*** Jack_Iv has quit IRC15:41
*** e0ne has quit IRC15:41
*** Jack_Iv has joined #openstack-cinder15:41
*** diablo_rojo has joined #openstack-cinder15:42
jgriffithshit15:43
*** diablo_rojo has quit IRC15:45
*** mriedem1 is now known as mriedem15:46
jungleboyjjgriffith:  What?15:47
jgriffithrebasing the convoluted willy wonka machine that scott built15:47
jungleboyj:-(15:49
hemnawilly wonka!    Does it poop out chocolate?15:50
jgriffithhemna Oompa Loompas poop chocolate, Willy just.. well, ya know15:50
*** salv-orlando has joined #openstack-cinder15:50
hemnammm chocolate15:51
*** diablo_rojo has joined #openstack-cinder15:52
karthikp_Anyone looking for a quick review : https://review.openstack.org/#/c/341184/15:54
*** alonma has joined #openstack-cinder15:54
*** salv-orlando has quit IRC15:55
jgriffithkarthikp_ ya know, I was just sitting here doing nothing thinking to myself... BOY I sure wish somebody would ping me and ask me to do a review for them15:56
jgriffithkarthikp_ unfortunately I was hoping for a very large and complicated review, so I guess you're out of luck15:56
hemnalol15:57
karthikp_jgriffith: haha okay! I forgot to be polite i guess :/15:59
*** rcernin has quit IRC15:59
jgriffithkarthikp_ it's not polite to ask for reviews :)15:59
*** alonma has quit IRC15:59
jgriffithkarthikp_ in any case I reviewed with a -216:00
jgriffithkarthikp_ I'm trying to avoid any more of these "filtering" tricks and changes16:00
jgriffithIt's sort of getting out of hand at this point.  Also, I think you can already do what you are proposing, just not via the client16:00
jgriffithfinally... I have some work I'm doing that will expose the filter mechanism that we have on the server side out to the client and replace all of these one-offs16:01
jgriffithif nothing else, the proposed changes going forward should be isolated to the client, the server, objects and DB already have a bare filter option16:02
jgriffithwe need to use that instead of continuing to bump server version and add things on the API itself16:02
jgriffithkarthikp_ make sense?16:02
karthikp_jgriffith: yes.. I remember sometime back checking with you on this....i was implementating this one: https://blueprints.launchpad.net/cinder/+spec/cinder-volume-usage-stats16:03
*** eharney has quit IRC16:03
jgriffithRight, but 18 months ago things were different :)16:03
jgriffithmy point being I think we can do what you want, but I think we can do it better :)16:04
*** chhavi has joined #openstack-cinder16:04
karthikp_jgriffith: Sure...make sense. :)16:05
jgriffithkarthikp_ check out this spec and the examples in it that do exactly what you're prposing (for the most part): https://review.openstack.org/#/c/441516/16:05
*** e0ne has joined #openstack-cinder16:07
*** tommylikehu_ has quit IRC16:07
*** e0ne has quit IRC16:08
*** ducttape_ has joined #openstack-cinder16:09
*** chenhb has quit IRC16:12
*** e0ne has joined #openstack-cinder16:12
*** chenhb has joined #openstack-cinder16:13
*** eharney has joined #openstack-cinder16:17
*** chlong_ has quit IRC16:19
openstackgerritAdriano Freires Rosso proposed openstack/cinder master: Remove HNAS iSCSI driver  https://review.openstack.org/44428716:20
rajinirPTG pictures -> https://www.flickr.com/photos/152419717@N06/albums/72157680602754246/with/32926953802/16:21
*** liverpooler has quit IRC16:22
smcginnis:)16:22
*** ducttape_ has quit IRC16:23
*** e0ne has quit IRC16:25
*** liverpooler has joined #openstack-cinder16:26
*** e0ne has joined #openstack-cinder16:27
*** mriedem1 has joined #openstack-cinder16:29
*** mriedem has quit IRC16:31
*** aarefiev is now known as aarefiev_afk16:32
*** Jack_Iv_ has joined #openstack-cinder16:33
*** Jack_Iv has quit IRC16:35
*** e0ne has quit IRC16:36
*** e0ne has joined #openstack-cinder16:37
*** e0ne_ has joined #openstack-cinder16:40
*** e0ne has quit IRC16:41
*** e0ne_ has quit IRC16:42
*** e0ne has joined #openstack-cinder16:43
*** dims_ has quit IRC16:45
*** e0ne has quit IRC16:48
openstackgerritJohn Griffith proposed openstack/python-cinderclient master: Add get_highest_version method  https://review.openstack.org/38564116:48
*** chhavi has quit IRC16:48
jgriffithThat was super painful16:49
jgriffithlet's hope it still works :)16:49
* jungleboyj hands jgriffith a beer16:49
jgriffithdrinking and merging don't mix!16:49
diablo_rojojgriffith, drinking and coding do.16:50
diablo_rojoYou should have had that beer sooner16:50
jgriffithdiablo_rojo well yeah!!!16:50
*** Cibo_ has joined #openstack-cinder16:50
smcginnisIt's just about 5 o'clock in UTC.16:52
*** e0ne has joined #openstack-cinder16:53
*** prashkre has joined #openstack-cinder16:53
jungleboyjAs my neighbor used to say, the sun is over the yard arm somewhere.16:53
*** e0ne has quit IRC16:54
diablo_rojosmcginnis, I like the way you think. Always there to help rationalize.16:55
diablo_rojojungleboyj, truth!16:55
openstackgerritMerged openstack/os-brick master: Fix iSCSI multipath rescan  https://review.openstack.org/43310416:56
*** lucasxu has quit IRC16:56
*** logan_ has joined #openstack-cinder16:56
*** logan_ is now known as Guest2958916:57
jungleboyjdiablo_rojo:  Yep!16:57
*** Jack_Iv_ has quit IRC16:57
*** e0ne has joined #openstack-cinder16:58
*** Jack_Iv has joined #openstack-cinder16:58
*** lucasxu has joined #openstack-cinder16:58
smcginnisdiablo_rojo: If there's one thing I'm good at, it's rationalization. :D16:59
*** nikeshm has quit IRC16:59
*** logan- has quit IRC16:59
openstackgerritJohn Griffith proposed openstack/python-cinderclient master: Remove duplicate get_highest_client_server_version  https://review.openstack.org/44441317:00
jungleboyjsmcginnis:  Is that a good thing.17:02
*** lucasxu has quit IRC17:02
*** e0ne has quit IRC17:02
*** Jack_Iv has quit IRC17:02
smcginnisjungleboyj: Not always. ;)17:02
jungleboyjsmcginnis:  Indeed.  Somethings should be rationalized.17:03
jungleboyjsmcginnis: And some things should not.17:03
*** Guest29589 is now known as logan-17:03
diablo_rojoBut for those things that should be rationalized, smcginnis is here to help17:04
smcginnisAnd sometimes for those things that should not.17:04
smcginnisI've never claimed to be a good influence. ;)17:04
*** Cibo_ has quit IRC17:05
jungleboyjOk, I am doing a terrible job of being on vacation.17:07
*** kaisers_ has joined #openstack-cinder17:07
*** catintheroof has quit IRC17:07
*** mriedem1 is now known as mriedem17:07
*** catintheroof has joined #openstack-cinder17:08
*** xinli has joined #openstack-cinder17:09
jungleboyjOk, stepping away from the computer.  :-)17:11
*** kaisers_ has quit IRC17:12
ildikovjgriffith: I knew I saw that freakin' method multiple times in the reviews...17:12
jgriffithildikov :)17:12
jgriffithI don't know why pep8 didn't catch that17:12
jgriffithI've grown to not like that code very much I'm afraid...17:12
ildikovjgriffith: I started to think I have either vision or mental issues...17:13
jgriffithI mean... "get_highest_client_server_version"... which is it, the client or the server?17:13
jgriffithblah blah blabh blah17:13
ildikovjgriffith: I can still have but at least it's not proven yet :)17:13
jgriffithildikov double vision FTW17:13
jgriffith2 of everything please!17:13
ildikovjgriffith: no idea about pep8, having all those micro patches up not in a chain was just too painful for me to check anything normally17:14
jgriffithildikov I'm going to look at the Nova check version thing next17:15
ildikovjgriffith: I've never liked that code :)17:15
jgriffithI think there's a better approach for that, but don't know if I care enough to propose it :)17:15
ildikovjgriffith: and I have the double vision, hence the glasses :)17:16
jgriffithildikov haha!17:16
ildikovWell I think if you have smth better this IS the time to propose it :)17:17
*** sudipto has quit IRC17:17
*** ducttape_ has joined #openstack-cinder17:20
*** mriedem is now known as mriedem_afk17:21
*** Apoorva has joined #openstack-cinder17:23
*** Apoorva has quit IRC17:24
*** Apoorva has joined #openstack-cinder17:25
*** e0ne has joined #openstack-cinder17:26
*** salv-orlando has joined #openstack-cinder17:26
*** e0ne has quit IRC17:26
*** salv-orlando has quit IRC17:27
*** salv-orlando has joined #openstack-cinder17:27
*** david-lyle has joined #openstack-cinder17:28
*** raunak has joined #openstack-cinder17:29
*** lucasxu has joined #openstack-cinder17:29
*** e0ne has joined #openstack-cinder17:29
*** Jack_I has quit IRC17:30
*** Apoorva has quit IRC17:31
*** JoseMello has joined #openstack-cinder17:31
*** yangyapeng has quit IRC17:31
*** yangyapeng has joined #openstack-cinder17:32
*** lamt has joined #openstack-cinder17:37
*** yangyapeng has quit IRC17:37
*** lpetrut has quit IRC17:45
*** psachin has quit IRC17:48
*** Apoorva has joined #openstack-cinder17:49
*** ducttape_ has quit IRC17:50
xinligeguileo: I saw your comments in 328828 / xio.py regarding volume['size']. I fix it and want to learn more: using dict. in OVO will reduce perfromance?17:51
*** alonma has joined #openstack-cinder17:56
openstackgerritJohn Griffith proposed openstack/python-cinderclient master: Make V3 the default and fixup version reporting  https://review.openstack.org/44407617:58
*** alonma has quit IRC18:01
*** e0ne has quit IRC18:02
*** alonma has joined #openstack-cinder18:03
xinligeguileo: dulek: I hit one issue in OVO that after modifying the code from volume['size'] to volume.size, the pep8 failed: https://review.openstack.org/#/c/328828/4/cinder/volume/drivers/xio.py@1168. any idea?18:05
xinligeguileo: dulek: the exception: http://paste.openstack.org/show/602299/18:06
xinligeguileo: dulek: the unit test py27 passed, but the pep8 failed18:07
*** alonma has quit IRC18:07
*** lucasxu has quit IRC18:08
*** lucasxu has joined #openstack-cinder18:15
*** alonmarx_ has quit IRC18:15
FransUrbogeguileo: Cibo_: Seems the problem is solved. There was (is!) a problem with Neutron not being able to create/setup the port correctly.18:22
*** ducttape_ has joined #openstack-cinder18:32
*** yangyapeng has joined #openstack-cinder18:33
xinlixyang1: I debugged the iisue that VMAX failed in 336092, this is the error: "Error: One or more objects are already in the requested state.", http://paste.openstack.org/show/602302/it could be the code issue, then I rerun VMAX CI on other system to reproduce it, but no response, is VMAX CI available now?18:33
*** eharney has quit IRC18:35
*** yangyapeng has quit IRC18:38
*** Jack_I has joined #openstack-cinder18:40
*** dhinesh has joined #openstack-cinder18:43
*** Jack_V has joined #openstack-cinder18:43
*** Jack_I has quit IRC18:46
*** eharney has joined #openstack-cinder18:51
*** Jack_I has joined #openstack-cinder18:52
*** Jack_V has quit IRC18:54
*** eharney_ has joined #openstack-cinder18:59
*** eharney has quit IRC19:01
*** eharney_ is now known as eharney19:02
*** crose has quit IRC19:03
*** FransUrbo has quit IRC19:07
*** mriedem_afk is now known as mriedem19:07
*** kaisers_ has joined #openstack-cinder19:08
*** mriedem has quit IRC19:11
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements  https://review.openstack.org/43205519:12
*** kaisers_ has quit IRC19:12
openstackgerritXinli Guan proposed openstack/cinder master: Fix 'driver does not honor clone/snapshot' bugs  https://review.openstack.org/33609219:13
jgriffithsmcginnis BTW. https://review.openstack.org/#/c/444465/19:18
smcginnisjgriffith: Saw your note over in Nova. Hopefully we get some feedback.19:23
*** salv-orl_ has joined #openstack-cinder19:23
*** BharatK_ has quit IRC19:24
*** sdague has joined #openstack-cinder19:24
smcginnisjgriffith: Did you see the mailing list discussion on deprecated API tempest tests?19:25
jgriffithnope, looking now19:25
smcginnishttp://lists.openstack.org/pipermail/openstack-dev/2017-March/113727.html19:26
*** salv-orlando has quit IRC19:26
*** salv-orl_ has quit IRC19:28
*** sdague has quit IRC19:29
*** markstur has quit IRC19:31
xinlihemna: could you please help me to understand how to switch dis to OVO in this case? https://review.openstack.org/#/c/296371/5/cinder/volume/drivers/hpe/hpe_lefthand_iscsi.py19:31
*** raunak has quit IRC19:32
*** raunak has joined #openstack-cinder19:33
*** yangyapeng has joined #openstack-cinder19:34
hemnaxinli, heh OVO19:37
hemnaI gave up on that19:37
hemnaas it was very very painful19:37
jgriffith@smcginnis right back at ya:  http://lists.openstack.org/pipermail/openstack-dev/2017-March/113750.html19:37
hemnawith zero to no benefit19:37
*** yangyapeng has quit IRC19:38
hemnajgriffith, because *magic*19:39
xinlihemna: the question is that geguileo insists on to fix the issue from volume['size'] to volume.size (dict --> OVO) is it simple for a single module?19:39
jgriffithhemna pretty much19:39
smcginnisjgriffith: Friday's are good days to open up a can of worms. :)19:39
hemnaxinli, well using a single volume.size vs. converting all access to OVO style are 2 different things19:40
hemnaand honestly, I can't find a reason for it now.19:40
hemnaI had a patch against the 3par and lefthand drivers that tried to convert just the volume access to OVO, and that ended up requiring converting all stuff to ovo.19:40
hemnaand it became a large and painful patch19:40
hemnaand didn't see any benefit19:41
hemnare: punt19:41
hemnabut at some point we should talk about the ovo conversion or not in drivers19:41
hemnain general and see if we plan on moving to it, or if nobody cares.19:41
jgriffith@smcginnis oh... dang, should've waited til Monday19:41
jgriffithbut I'd surely forget by then (again)19:42
smcginnisjgriffith: Hah! Nah, might get some better responses over the weekend.19:42
*** chlong_ has joined #openstack-cinder19:42
xinlihemna: my point is only for this case which geguileo has already pointed out with (-1) otherwise I could not pass,  https://review.openstack.org/#/c/296371/5/cinder/volume/drivers/hpe/hpe_lefthand_iscsi.py19:43
xinlihemna: in this case, how can I modify the scenario to create the volume with OVO instead of dict?19:44
jgriffithxinli hemna if I'm not mistaken, the point in geguileo 's comment is that the volume you have there is *already* an object19:44
jgriffithxinli hemna so just change the notation... that's all you should need to do19:44
jgriffithif manager is passing in a none object (it's not) then it wouldn't be your problem19:45
smcginnis+119:45
jgriffiths/none object/non-object/19:45
xinlijdriffith: if I change the only the syntax, it will be failed in tox -e pep8, this is the error message:19:46
jgriffithwait for it.....19:46
xinlihemna: jgriffith: @smcginnis: http://paste.openstack.org/show/602299/19:47
jgriffithxio driver?19:47
xinliyes.19:48
jgriffithso this has nothing to do with the patch you showed the link for?19:48
* jgriffith face--->desk19:48
xinlihttps://review.openstack.org/#/c/328828/19:48
*** dims has joined #openstack-cinder19:49
jgriffithxinli ok, give me just a minute please19:49
xinlijgriffith: sorry for misleading, the whole xio.py and test case have the same issue, and the issue across all the drivers' code.19:50
openstackgerritMerged openstack/python-cinderclient master: Remove duplicate get_highest_client_server_version  https://review.openstack.org/44441319:50
openstackgerritMerged openstack/python-cinderclient master: Add get_highest_version method  https://review.openstack.org/38564119:50
jgriffithlooks like the problem is the groups code is wrong19:51
jgriffithoh, no :)19:51
jgriffithIt fails because of your unit test19:52
jgriffithxinli your unit test is just doing a dict which is wrong19:52
jgriffithxinli you need to update the fake in your unit test to actually create an object19:53
jgriffithobject(s)19:53
jgriffithit does NOT fail pep8 BTW19:53
jgriffithit fails unit tests19:53
jgriffithbig difference :)19:53
jgriffithxinli does that make sense?19:54
xinlijgriffith: I will make double check.19:54
xinlijgriffith: It seems to impact a lot of drivers code19:55
xinliI tested 5 drivers, all have the same issues19:55
jgriffithxinli not sure I follow?19:55
jgriffithOh!19:55
jgriffithYou mean you updated 5 different drivers and they fail19:55
jgriffithyes, without updating the unit tests they will fail19:55
xinliyes.19:55
jgriffithxinli so we *used* to just pass dictionaries of these resources around19:56
jgriffithbut we moved everything to objects19:56
jgriffithfor code that was in *before* the move to objects they haven't been modified19:56
jgriffithBUT in an attempt to get rid of the mixed mode we're in now, we've been asking people to update their unit test fakes and the notation in their drivers when the submit changes that touch those things19:57
jgriffithxinli make sense?19:57
jgriffithxinli do you need an example of what I mean?19:57
xinlijgriffith: the one in 295480 (you are working on doesn't have the issue)19:57
xinliI need one19:57
xinliand follow th edirection19:58
xinlijgriffith, could you please use xio as example? I will modify the others, such as ProphetStor, etc19:58
jgriffithxinli well I'm not going to write code for you :). But.... here's what you need https://github.com/openstack/cinder/blob/master/cinder/tests/unit/volume/drivers/test_rbd.py#L17520:00
xinlijgriffith: thanks20:01
jgriffithxinli instead of what you have with "fake_volume = {asdfafasdfsfsadfsa.....}"20:01
jgriffiththen your unit tests will work20:01
jgriffithxinli you're welcome!  Good luck20:01
*** prashkre has quit IRC20:02
jgriffithxinli shout if you get stuck, if I'm around I can try and help20:02
xinlijgriffith: thanks a lot20:02
jgriffithquick!!  Before it breaks again!  :). https://review.openstack.org/#/c/444076/20:03
*** gouthamr has quit IRC20:04
*** JoseMello has quit IRC20:05
*** ChubYann has joined #openstack-cinder20:08
*** crushil has quit IRC20:10
*** ducttape_ has quit IRC20:13
*** dustins has quit IRC20:13
*** gouthamr has joined #openstack-cinder20:16
*** markvoelker has joined #openstack-cinder20:22
*** markstur has joined #openstack-cinder20:22
*** ducttape_ has joined #openstack-cinder20:26
eharneynice easy patch to review on friday:  https://review.openstack.org/#/c/443752/20:27
*** markstur has quit IRC20:28
*** salv-orlando has joined #openstack-cinder20:29
openstackgerritSatyajeet Shahane proposed openstack/cinder master: Adding tests for api/v3/group_specs.py  https://review.openstack.org/44449420:30
*** Apoorva has quit IRC20:31
*** salv-orlando has quit IRC20:34
*** yangyapeng has joined #openstack-cinder20:34
*** zul is now known as zulVacation20:37
*** yangyapeng has quit IRC20:39
*** jose-phillips has quit IRC20:43
*** kaisers_ has joined #openstack-cinder20:44
*** jose-phillips has joined #openstack-cinder20:45
*** kaisers_ has quit IRC20:49
openstackgerritSatyajeet Shahane proposed openstack/cinder master: Adding tests for api/v3/group_specs.py  https://review.openstack.org/44449420:52
*** lpetrut has joined #openstack-cinder20:53
*** dustins has joined #openstack-cinder20:56
*** lucasxu has quit IRC20:57
*** lucasxu has joined #openstack-cinder20:57
*** eharney has quit IRC21:07
*** ducttape_ has quit IRC21:08
*** cknight has quit IRC21:13
*** porrua has quit IRC21:15
*** xinli has quit IRC21:17
*** salv-orlando has joined #openstack-cinder21:30
*** alonmarx has joined #openstack-cinder21:32
*** alonmarx has quit IRC21:35
*** kbaegis has joined #openstack-cinder21:36
*** jose-phillips has quit IRC21:39
kbaegishttps://hastebin.com/apazoderus.ini is this config accurate for two lvm storage pools?21:39
*** Apoorva has joined #openstack-cinder21:40
*** markstur has joined #openstack-cinder21:41
*** alonmarx has joined #openstack-cinder21:48
*** jose-phillips has joined #openstack-cinder21:49
*** frontrunner has quit IRC21:50
*** lucasxu has quit IRC21:52
*** lucasxu has joined #openstack-cinder21:53
*** dave-mccowan has quit IRC21:57
*** gouthamr has quit IRC22:03
*** alonma has joined #openstack-cinder22:04
*** lpetrut has quit IRC22:06
*** lpetrut has joined #openstack-cinder22:07
*** alonma has quit IRC22:08
*** ducttape_ has joined #openstack-cinder22:08
*** alonma has joined #openstack-cinder22:11
*** lpetrut has quit IRC22:11
*** dustins has quit IRC22:12
jgriffithkbaegis yes, that should work22:13
*** Jack_I has quit IRC22:14
kbaegisjgriffith: Ty22:14
*** ducttape_ has quit IRC22:14
kbaegisThis is the error I get22:15
kbaegisjgriffith: https://hastebin.com/lasuriqupi.py22:15
*** alonma has quit IRC22:15
kbaegisI do have a vg cinder-flash-volumes, as referenced in the file22:15
kbaegisBut scheduling is failing22:15
*** eharney has joined #openstack-cinder22:16
jgriffithkbaegis can you show a `sudo lvs` for that cinder-node?22:18
jgriffithkbaegis another trick you can use is grep the c-vol log for the periodic capacity report coming back from your drivers22:19
jgriffithkbaegis making sure both of them show up and have capacity22:20
*** chlong_ has quit IRC22:21
jgriffithkbaegis you should see something like this:  https://hastebin.com/edegikulil.py for both backends22:21
jgriffithkbaegis note the line breaks don't paste, keep scrolling to the right :)22:23
*** catintheroof has quit IRC22:23
*** gouthamr has joined #openstack-cinder22:29
*** knikolla has left #openstack-cinder22:31
*** lucasxu has quit IRC22:35
*** sdague has joined #openstack-cinder22:35
*** yangyapeng has joined #openstack-cinder22:36
*** kaisers_ has joined #openstack-cinder22:40
xyang1xinli: hi22:41
*** yangyapeng has quit IRC22:41
*** sdague has quit IRC22:42
xyang1xinli: some existing code in VMAX driver that handles clone a bigger volume needs to be removed if you use the generic approach22:43
*** kaisers_ has quit IRC22:44
*** porrua has joined #openstack-cinder23:03
openstackgerritMerged openstack/cinder master: Remove HNAS iSCSI driver  https://review.openstack.org/44428723:10
*** salv-orlando has quit IRC23:18
*** nicolasbock has quit IRC23:23
*** porrua_ has joined #openstack-cinder23:23
*** alonmarx has quit IRC23:26
*** nicolasbock has joined #openstack-cinder23:27
*** porrua has quit IRC23:27
*** tesseract has quit IRC23:29
*** xyang1 has quit IRC23:33
*** edmondsw has quit IRC23:35
*** yangyapeng has joined #openstack-cinder23:36
openstackgerritchristopher.uhler proposed openstack/os-brick master: os-brick connector for Veritas HyperScale  https://review.openstack.org/44275423:37
*** edmondsw has joined #openstack-cinder23:37
*** yangyapeng has quit IRC23:41
*** e0ne has joined #openstack-cinder23:41
openstackgerritchristopher.uhler proposed openstack/os-brick master: os-brick connector for Veritas HyperScale  https://review.openstack.org/44275423:41
*** edmondsw has quit IRC23:42

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