Thursday, 2017-03-30

*** joanna__ has quit IRC00:03
*** ducttape_ has joined #openstack-cinder00:05
*** ntpttr_ has quit IRC00:05
*** ntpttr_ has joined #openstack-cinder00:06
*** Apoorva_ has quit IRC00:07
*** Apoorva has joined #openstack-cinder00:08
*** ducttape_ has quit IRC00:11
*** wanghao has joined #openstack-cinder00:12
*** ntpttr_ has quit IRC00:13
*** ntpttr_ has joined #openstack-cinder00:14
openstackgerritMerged openstack/os-brick master: Fix unittest run on s390x host  https://review.openstack.org/39998800:14
*** cebruns has quit IRC00:16
*** cebruns has joined #openstack-cinder00:17
*** ntpttr_ has quit IRC00:18
*** ntpttr_ has joined #openstack-cinder00:20
*** ntpttr_ has quit IRC00:24
*** dikonoor has joined #openstack-cinder00:25
*** ntpttr_ has joined #openstack-cinder00:26
*** mtanino_ has joined #openstack-cinder00:29
*** Julien-zte has joined #openstack-cinder00:29
openstackgerritOpenStack Proposal Bot proposed openstack/os-brick master: Updated from global requirements  https://review.openstack.org/45104700:29
*** mtanino has quit IRC00:30
*** mriedem has joined #openstack-cinder00:32
*** viks has joined #openstack-cinder00:34
*** ntpttr_ has quit IRC00:35
*** ntpttr_ has joined #openstack-cinder00:36
*** ntpttr_ has quit IRC00:43
*** ntpttr_ has joined #openstack-cinder00:44
*** Julien-zte has quit IRC00:45
*** david-lyle_ has joined #openstack-cinder00:45
*** gcb has quit IRC00:46
*** crushil has joined #openstack-cinder00:46
*** david-lyle has quit IRC00:48
*** mriedem has quit IRC00:49
viks:)00:49
*** diablo_rojo has joined #openstack-cinder00:49
openstackgerritEmily Hugenbruch proposed openstack/cinder master: Separate out routine for getting qemu_img_info  https://review.openstack.org/44888700:50
*** zhangguoqing has joined #openstack-cinder00:54
*** yangyapeng has joined #openstack-cinder00:56
*** jiaopengju has joined #openstack-cinder01:15
*** Apoorva has quit IRC01:16
*** dikonoor has quit IRC01:16
*** raunak has quit IRC01:17
*** ntpttr_ has quit IRC01:17
*** ntpttr_ has joined #openstack-cinder01:19
*** castulo has quit IRC01:21
*** ntpttr_ has quit IRC01:23
*** ntpttr_ has joined #openstack-cinder01:25
*** castulo has joined #openstack-cinder01:26
*** mtanino_ has quit IRC01:28
*** zhonghua has quit IRC01:28
*** edmondsw has quit IRC01:29
*** zhonghua has joined #openstack-cinder01:30
*** huanxuan has joined #openstack-cinder01:32
*** kaisers__ has joined #openstack-cinder01:33
*** ntpttr_ has quit IRC01:33
*** david-lyle__ has joined #openstack-cinder01:34
*** Julien-zte has joined #openstack-cinder01:34
*** ntpttr_ has joined #openstack-cinder01:35
*** kaisers_ has quit IRC01:35
*** david-lyle_ has quit IRC01:37
*** ntpttr_ has quit IRC01:39
*** ntpttr_ has joined #openstack-cinder01:41
openstackgerritTommyLike proposed openstack/python-cinderclient master: [BugFix] Make 'instance_id' required in attachment-create CLI.  https://review.openstack.org/44985801:41
openstackgerritliucheng proposed openstack/cinder master: Huawei: disable certificate verification  https://review.openstack.org/44787301:45
*** crushil has quit IRC01:45
*** ntpttr_ has quit IRC01:45
*** ntpttr_ has joined #openstack-cinder01:46
*** chlong has joined #openstack-cinder01:47
*** oomichi has quit IRC01:48
*** ntpttr_ has quit IRC01:51
*** oomichi has joined #openstack-cinder01:52
*** ntpttr_ has joined #openstack-cinder01:53
*** gcb has joined #openstack-cinder01:53
*** viks has quit IRC01:57
*** ntpttr_ has quit IRC01:57
*** ntpttr_ has joined #openstack-cinder01:59
*** ntpttr_ has quit IRC02:03
*** ntpttr_ has joined #openstack-cinder02:04
openstackgerritXieYingYun proposed openstack/cinder master: Fix some format error in docstrings  https://review.openstack.org/44990802:07
*** ntpttr_ has quit IRC02:08
*** tommylikehu_ has joined #openstack-cinder02:09
*** ntpttr_ has joined #openstack-cinder02:11
*** tommylikehu_ has quit IRC02:12
*** jichen has joined #openstack-cinder02:13
*** ntpttr_ has quit IRC02:15
*** chenhb has quit IRC02:15
*** chenhb has joined #openstack-cinder02:16
*** ntpttr_ has joined #openstack-cinder02:17
*** ntpttr_ has quit IRC02:21
*** ntpttr_ has joined #openstack-cinder02:23
*** yangyapeng has quit IRC02:24
*** ntpttr_ has quit IRC02:27
*** ntpttr_ has joined #openstack-cinder02:28
*** ntpttr has quit IRC02:31
*** wxy_ has joined #openstack-cinder02:34
*** ntpttr_ has quit IRC02:35
openstackgerritXieYingYun proposed openstack/cinder master: Remove unnecessary setUp function in testcase  https://review.openstack.org/45162602:36
*** ntpttr has joined #openstack-cinder02:37
*** yangyapeng has joined #openstack-cinder02:40
*** ntpttr has quit IRC02:41
*** ntpttr has joined #openstack-cinder02:43
*** ntpttr_ has joined #openstack-cinder02:44
*** ntpttr has quit IRC02:49
*** ntpttr has joined #openstack-cinder02:51
*** jichen has quit IRC02:54
*** ntpttr has quit IRC02:55
*** ntpttr has joined #openstack-cinder02:56
*** ntpttr has quit IRC03:01
*** ntpttr has joined #openstack-cinder03:02
*** gkadam has joined #openstack-cinder03:06
*** oomichi has quit IRC03:08
*** ntpttr has quit IRC03:09
*** ntpttr has joined #openstack-cinder03:10
*** oomichi has joined #openstack-cinder03:12
*** diablo_rojo has quit IRC03:16
*** cknight has joined #openstack-cinder03:16
openstackgerritXieYingYun proposed openstack/cinder master: Remove unnecessary setUp function in testcase  https://review.openstack.org/45162603:17
*** dikonoor has joined #openstack-cinder03:19
openstackgerritMerged openstack/os-brick master: Updated from global requirements  https://review.openstack.org/45104703:20
*** ntpttr has quit IRC03:20
*** ntpttr has joined #openstack-cinder03:23
*** dave-mccowan has quit IRC03:27
*** ntpttr has quit IRC03:27
*** yangyapeng has quit IRC03:27
*** yangyapeng has joined #openstack-cinder03:28
*** ntpttr has joined #openstack-cinder03:28
*** crushil has joined #openstack-cinder03:28
*** ntpttr has quit IRC03:32
*** ntpttr has joined #openstack-cinder03:35
*** ntpttr has quit IRC03:41
openstackgerrityuhui proposed openstack/cinder master: [Trivial]Fix some incorrect annotations  https://review.openstack.org/43011603:42
*** ntpttr has joined #openstack-cinder03:42
*** Julien-zte has quit IRC03:43
*** Julien-zte has joined #openstack-cinder03:46
*** ntpttr has quit IRC03:51
*** ntpttr has joined #openstack-cinder03:52
*** gouthamr has quit IRC03:55
*** chris_morrell has joined #openstack-cinder03:57
*** ntpttr has quit IRC03:59
*** monjoshi has joined #openstack-cinder03:59
*** ntpttr has joined #openstack-cinder04:01
*** chris_morrell has quit IRC04:02
*** cknight has quit IRC04:04
*** Fdaisuke has quit IRC04:05
*** psachin has joined #openstack-cinder04:11
*** jhova has quit IRC04:23
*** links has joined #openstack-cinder04:26
*** zhangguoqing has quit IRC04:26
*** dobson has quit IRC04:36
*** dobson has joined #openstack-cinder04:40
*** Apoorva has joined #openstack-cinder04:45
*** zhangguoqing has joined #openstack-cinder04:48
*** Apoorva has quit IRC04:56
*** pdeore has joined #openstack-cinder04:57
*** Apoorva has joined #openstack-cinder04:57
*** pdeore has quit IRC05:03
*** Apoorva has quit IRC05:05
*** oomichi has quit IRC05:09
*** tommylikehu_ has joined #openstack-cinder05:11
*** oomichi has joined #openstack-cinder05:12
*** monjoshi has quit IRC05:13
*** wN has quit IRC05:13
*** dalvarez has quit IRC05:14
*** dmellado has quit IRC05:14
*** chlong has quit IRC05:14
*** pots has quit IRC05:15
*** geguileo has quit IRC05:15
*** supamatt has quit IRC05:15
*** ChubYann has quit IRC05:15
*** tommylikehu_ has quit IRC05:16
*** geguileo has joined #openstack-cinder05:16
*** supamatt has joined #openstack-cinder05:16
*** dmellado has joined #openstack-cinder05:16
*** ChubYann has joined #openstack-cinder05:16
*** wN has joined #openstack-cinder05:17
*** wN has quit IRC05:17
*** wN has joined #openstack-cinder05:17
*** oomichi has quit IRC05:18
*** dalvarez has joined #openstack-cinder05:20
*** oomichi has joined #openstack-cinder05:23
*** Poornima has joined #openstack-cinder05:27
*** adisky_ has joined #openstack-cinder05:27
*** pgadiya has joined #openstack-cinder05:28
*** nkrinner_afk is now known as nkrinner05:30
*** pdeore has joined #openstack-cinder05:30
*** zhonghua has quit IRC05:35
*** chenying__ has quit IRC05:36
*** zhonghua has joined #openstack-cinder05:36
*** chenying__ has joined #openstack-cinder05:37
*** rcernin has joined #openstack-cinder05:45
*** ntpttr has quit IRC05:49
*** ntpttr has joined #openstack-cinder05:50
*** ntpttr has quit IRC05:55
*** monjoshi has joined #openstack-cinder05:55
*** ntpttr has joined #openstack-cinder05:56
*** ntpttr has quit IRC06:01
*** ntpttr has joined #openstack-cinder06:02
*** ntpttr has quit IRC06:07
*** ntpttr has joined #openstack-cinder06:08
*** ChubYann has quit IRC06:14
*** zhurong has joined #openstack-cinder06:15
*** ntpttr has quit IRC06:21
*** ntpttr has joined #openstack-cinder06:22
*** isaacb has joined #openstack-cinder06:25
*** ntpttr has quit IRC06:27
*** ntpttr has joined #openstack-cinder06:28
*** alonma has quit IRC06:30
*** gkadam is now known as gkadam-afk06:31
*** deepak_jon has joined #openstack-cinder06:33
*** ntpttr has quit IRC06:35
*** ntpttr has joined #openstack-cinder06:36
*** zhurong has quit IRC06:38
*** zhongjun_ has quit IRC06:42
*** ntpttr has quit IRC06:43
*** zhongjun_ has joined #openstack-cinder06:43
*** ntpttr has joined #openstack-cinder06:43
*** haplo37- has quit IRC06:45
*** lkuchlan has joined #openstack-cinder06:45
*** pcaruana has joined #openstack-cinder06:47
*** bhagyashris has quit IRC06:50
*** ntpttr has quit IRC06:53
*** haplo37_ has joined #openstack-cinder06:53
*** ntpttr has joined #openstack-cinder06:55
*** raunak has joined #openstack-cinder06:57
*** Dinesh_Bhor has quit IRC07:07
*** Dinesh_Bhor has joined #openstack-cinder07:07
*** dineshbhor has joined #openstack-cinder07:09
*** dineshbhor has quit IRC07:09
*** deepak_jon has quit IRC07:09
*** ntpttr has quit IRC07:09
*** deepak_jon has joined #openstack-cinder07:09
*** ntpttr has joined #openstack-cinder07:10
*** tommylikehu_ has joined #openstack-cinder07:13
*** ntpttr has quit IRC07:15
*** tesseract has joined #openstack-cinder07:16
*** ntpttr has joined #openstack-cinder07:17
*** tommylikehu_ has quit IRC07:17
*** kukacz has quit IRC07:17
*** kukacz has joined #openstack-cinder07:18
*** zhonghua has quit IRC07:20
*** zhonghua has joined #openstack-cinder07:22
*** alonma has joined #openstack-cinder07:23
*** Dinesh_Bhor has quit IRC07:24
*** Dinesh_Bhor has joined #openstack-cinder07:25
*** ntpttr has quit IRC07:28
*** ntpttr has joined #openstack-cinder07:28
*** Dinesh_Bhor has quit IRC07:30
*** Dinesh_Bhor has joined #openstack-cinder07:32
*** zhurong has joined #openstack-cinder07:39
*** ntpttr has quit IRC07:39
*** Dinesh_Bhor has quit IRC07:40
*** ntpttr has joined #openstack-cinder07:40
*** Dinesh_Bhor has joined #openstack-cinder07:43
*** pdeore has quit IRC07:45
*** ntpttr has quit IRC07:45
*** raunak has quit IRC07:45
*** aarefiev_afk is now known as aarefiev07:46
*** ntpttr has joined #openstack-cinder07:46
*** lpetrut has joined #openstack-cinder07:50
*** gkadam_ has joined #openstack-cinder07:52
*** gkadam-afk has quit IRC07:52
*** gkadam_ is now known as gkadam07:52
*** zhangguoqing has quit IRC07:57
*** wuyanjun has joined #openstack-cinder07:57
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-cinder08:02
*** zhangguoqing has joined #openstack-cinder08:02
*** openstackgerrit has quit IRC08:03
*** zhangguoqing has quit IRC08:08
*** zhangguoqing has joined #openstack-cinder08:12
*** zhangguo_ has joined #openstack-cinder08:18
*** zhangguoqing has quit IRC08:18
*** yangyapeng has quit IRC08:19
*** yangyapeng has joined #openstack-cinder08:20
*** pdeore has joined #openstack-cinder08:21
*** zhangguo_ has quit IRC08:22
*** openstackgerrit has joined #openstack-cinder08:23
openstackgerritxiaoqin proposed openstack/cinder master: Storwize: add CG capability to generic groups  https://review.openstack.org/44480208:23
*** yangyapeng has quit IRC08:24
*** haplo37_ has quit IRC08:28
*** haplo37_ has joined #openstack-cinder08:31
*** alonma has quit IRC08:31
*** alonma has joined #openstack-cinder08:32
*** alonma has quit IRC08:33
*** alonma has joined #openstack-cinder08:33
*** yulijie has joined #openstack-cinder08:33
*** mlakat has quit IRC08:35
*** mlakat has joined #openstack-cinder08:35
*** mlakat has quit IRC08:36
*** mlakat has joined #openstack-cinder08:36
*** pkoniszewski has left #openstack-cinder08:37
*** ntpttr has quit IRC08:46
*** ntpttr has joined #openstack-cinder08:47
*** ntpttr has quit IRC08:51
*** ntpttr has joined #openstack-cinder08:53
*** pgadiya has quit IRC09:01
*** zhurong has quit IRC09:01
*** alonma has quit IRC09:02
*** alonma has joined #openstack-cinder09:05
*** ducttape_ has joined #openstack-cinder09:07
*** ntpttr has quit IRC09:09
*** ntpttr has joined #openstack-cinder09:11
*** ducttape_ has quit IRC09:12
*** pbandark has joined #openstack-cinder09:14
*** pgadiya has joined #openstack-cinder09:16
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX driver - Live Migration is dropping connection  https://review.openstack.org/45043009:16
*** pdeore has quit IRC09:18
*** ntpttr has quit IRC09:19
*** deepak_jon has quit IRC09:20
*** ntpttr has joined #openstack-cinder09:21
*** deepak_jon has joined #openstack-cinder09:22
*** pgadiya has quit IRC09:22
*** wN_ has quit IRC09:24
*** tommylikehu[m] has quit IRC09:24
*** ntpttr has quit IRC09:26
*** tommylikehu[m] has joined #openstack-cinder09:26
*** ntpttr has joined #openstack-cinder09:27
*** pooja_jadhav has joined #openstack-cinder09:28
*** deepak_jon has quit IRC09:31
*** deepak_jon has joined #openstack-cinder09:31
*** huanxuan has quit IRC09:34
*** pgadiya has joined #openstack-cinder09:34
*** ntpttr has quit IRC09:35
*** ntpttr has joined #openstack-cinder09:37
*** _dalgaaf has joined #openstack-cinder09:37
*** Julien-zte has quit IRC09:38
*** dalgaaf has quit IRC09:40
*** ntpttr has quit IRC09:41
*** sdague has joined #openstack-cinder09:41
*** ntpttr has joined #openstack-cinder09:43
*** e0ne has joined #openstack-cinder09:45
*** wN_ has joined #openstack-cinder09:48
*** Dinesh_Bhor has quit IRC09:49
*** ntpttr has quit IRC09:51
*** ntpttr has joined #openstack-cinder09:53
*** pdeore has joined #openstack-cinder09:53
*** ntpttr has quit IRC09:59
*** ntpttr has joined #openstack-cinder10:01
*** yangyapeng has joined #openstack-cinder10:02
*** ntpttr has quit IRC10:05
*** ntpttr has joined #openstack-cinder10:06
*** mlakat has quit IRC10:06
*** jkraj has joined #openstack-cinder10:07
*** ntpttr has quit IRC10:10
*** yangyapeng has quit IRC10:11
*** Dinesh_Bhor has joined #openstack-cinder10:12
*** ntpttr has joined #openstack-cinder10:13
*** zhangguoqing has joined #openstack-cinder10:13
*** tommylikehu_ has joined #openstack-cinder10:15
*** yangyapeng has joined #openstack-cinder10:17
*** mvk has quit IRC10:19
*** tommylikehu_ has quit IRC10:20
*** monjoshi has quit IRC10:21
*** ntpttr has quit IRC10:21
*** edmondsw has joined #openstack-cinder10:21
*** ntpttr has joined #openstack-cinder10:23
*** edmondsw has quit IRC10:26
*** ntpttr has quit IRC10:27
*** zhangguo_ has joined #openstack-cinder10:29
*** ntpttr has joined #openstack-cinder10:29
*** zhangguoqing has quit IRC10:29
*** ntpttr has quit IRC10:36
*** ntpttr has joined #openstack-cinder10:37
*** ducttape_ has joined #openstack-cinder10:38
*** kwathore has quit IRC10:41
*** ntpttr has quit IRC10:41
*** ntpttr has joined #openstack-cinder10:43
*** ducttape_ has quit IRC10:43
*** ntpttr has quit IRC10:47
*** ntpttr has joined #openstack-cinder10:49
openstackgerritIvan Kolodyazhny proposed openstack/os-brick master: Move vzstorage related code out of RemoteFsClient  https://review.openstack.org/45175610:54
*** tbarron is now known as tbarron|out10:54
*** ntpttr has quit IRC10:58
*** ntpttr has joined #openstack-cinder10:59
*** ntpttr has quit IRC11:05
*** ntpttr has joined #openstack-cinder11:07
*** ntpttr has quit IRC11:11
openstackgerritTommyLike proposed openstack/cinder-specs master: Better user messages  https://review.openstack.org/45176111:11
*** ntpttr has joined #openstack-cinder11:13
*** ntpttr has quit IRC11:18
*** ntpttr has joined #openstack-cinder11:19
*** monjoshi has joined #openstack-cinder11:22
*** ntpttr has quit IRC11:23
*** gcb has quit IRC11:24
*** ntpttr has joined #openstack-cinder11:25
*** jwcroppe has joined #openstack-cinder11:27
*** deepak_jon has quit IRC11:27
*** deepak_jon has joined #openstack-cinder11:27
*** ntpttr has quit IRC11:32
*** monjoshi_ has joined #openstack-cinder11:32
*** ntpttr has joined #openstack-cinder11:33
*** yangyapeng has quit IRC11:33
*** monjoshi has quit IRC11:35
*** kwathore has joined #openstack-cinder11:36
openstackgerritKushal Wathore proposed openstack/cinder master: HPE 3PAR: Handle manage and unmanage hosts present  https://review.openstack.org/40912411:38
*** ntpttr has quit IRC11:40
*** pdeore has quit IRC11:41
*** ntpttr has joined #openstack-cinder11:41
*** yangyapeng has joined #openstack-cinder11:43
*** gcb has joined #openstack-cinder11:44
*** abishop has joined #openstack-cinder11:44
*** kaisers has quit IRC11:45
*** pgadiya has quit IRC11:47
*** ericyoung has joined #openstack-cinder11:47
*** ntpttr has quit IRC11:50
*** ntpttr has joined #openstack-cinder11:51
*** deepak_jon has quit IRC11:54
*** deepak_jon has joined #openstack-cinder11:54
*** zhangguo_ has quit IRC11:56
*** ntpttr has quit IRC11:57
*** ntpttr has joined #openstack-cinder11:59
*** mvk has joined #openstack-cinder11:59
*** zhangguoqing has joined #openstack-cinder12:00
*** kaisers has joined #openstack-cinder12:00
openstackgerritKushal Wathore proposed openstack/cinder master: HPE 3PAR: Handle manage and unmanage hosts present  https://review.openstack.org/40912412:01
*** ntpttr has quit IRC12:03
*** dikonoo has joined #openstack-cinder12:05
*** jiaopengju has quit IRC12:05
openstackgerritliucheng proposed openstack/cinder master: Huawei: Record and check Snapshot wwn.  https://review.openstack.org/44838312:06
*** zhangguoqing has quit IRC12:06
*** zhangguoqing has joined #openstack-cinder12:07
*** ntpttr has joined #openstack-cinder12:10
*** ntpttr has quit IRC12:14
*** mlakat has joined #openstack-cinder12:15
*** dave-mccowan has joined #openstack-cinder12:17
*** ntpttr has joined #openstack-cinder12:17
*** zhangguoqing has quit IRC12:18
*** zhangguoqing has joined #openstack-cinder12:19
*** catintheroof has joined #openstack-cinder12:21
*** ntpttr has quit IRC12:21
*** edmondsw has joined #openstack-cinder12:22
*** ntpttr has joined #openstack-cinder12:22
*** zhangguo_ has joined #openstack-cinder12:24
*** ntpttr has quit IRC12:26
*** zhangguoqing has quit IRC12:27
*** ntpttr has joined #openstack-cinder12:29
yuvalHey, can any core please take a final look at https://review.openstack.org/#/c/451211/ ?    It fixes an error caused by https://review.openstack.org/#/c/442241/ - exception when trying to create a volume from a snapshot whose originating volume is attached12:31
*** gkadam has quit IRC12:32
*** ducttape_ has joined #openstack-cinder12:35
openstackgerritTommyLike proposed openstack/cinder-specs master: Explicit user messages  https://review.openstack.org/45176112:37
*** pbandark is now known as pbandark`brb12:39
*** ntpttr has quit IRC12:39
*** ntpttr has joined #openstack-cinder12:41
*** yangyapeng has quit IRC12:43
openstackgerrithuyang proposed openstack/cinder master: Add CG capability to generic groups in Huawei driver  https://review.openstack.org/44902912:44
*** liverpooler has quit IRC12:47
*** liverpooler has joined #openstack-cinder12:47
*** ntpttr has quit IRC12:49
*** ntpttr has joined #openstack-cinder12:51
*** jwcroppe has quit IRC12:53
*** links has quit IRC12:54
*** ntpttr has quit IRC12:55
*** pbandark has joined #openstack-cinder12:55
*** pbandark`brb has quit IRC12:56
*** ntpttr has joined #openstack-cinder12:57
*** gouthamr has joined #openstack-cinder13:01
*** ntpttr has quit IRC13:05
*** lamt has joined #openstack-cinder13:06
*** pdeore has joined #openstack-cinder13:07
*** ntpttr has joined #openstack-cinder13:07
*** yangyapeng has joined #openstack-cinder13:10
*** ntpttr has quit IRC13:14
*** ntpttr has joined #openstack-cinder13:15
*** tommylikehu_ has joined #openstack-cinder13:17
*** eharney has joined #openstack-cinder13:18
*** lamt has quit IRC13:21
*** tommylikehu_ has quit IRC13:22
*** ntpttr has quit IRC13:22
*** ntpttr has joined #openstack-cinder13:23
openstackgerritCarl Pecinovsky proposed openstack/cinder master: Add an optional db session argument to cinder.db.sqlalchemy.api. quota_allocated_get_all_by_project(). Then in quota_reserve() pass the session that is fetched for that method so that a new session does not have to be made. This can avoid DB timeouts and   https://review.openstack.org/45182113:26
*** Julien-zte has joined #openstack-cinder13:27
*** dustins has joined #openstack-cinder13:28
*** xinli has joined #openstack-cinder13:29
*** ntpttr has quit IRC13:29
*** ntpttr has joined #openstack-cinder13:31
*** chlong has joined #openstack-cinder13:34
*** zhonghua has quit IRC13:35
*** zhonghua has joined #openstack-cinder13:37
openstackgerritwanghao proposed openstack/cinder master: Support for HostAddress opt  https://review.openstack.org/45182413:39
*** wanghao has quit IRC13:40
*** chenying__ has quit IRC13:41
*** chenying__ has joined #openstack-cinder13:41
xinli@smcginnis: one question in 328828 jenkins build failure, the patch passed all the test cases, but hit time out in "gate-cinder-python35", is there any issue in test environmnet/13:43
*** ntpttr has quit IRC13:43
*** lucasxu has joined #openstack-cinder13:43
*** ntpttr has joined #openstack-cinder13:45
smcginnisxinli: Could be, that doesn't sound like it would be related to your change in any way. I think you can just recheck it.13:49
*** ntpttr has quit IRC13:49
xinli@smcginnis: thx13:49
*** ntpttr has joined #openstack-cinder13:51
*** tommylikehu_ has joined #openstack-cinder13:51
*** tommylik_ has joined #openstack-cinder13:52
*** tommylikehu_ has quit IRC13:53
*** isaacb_ has joined #openstack-cinder13:55
*** ntpttr has quit IRC13:56
*** ntpttr has joined #openstack-cinder13:57
*** isaacb has quit IRC13:58
*** crose has joined #openstack-cinder13:59
*** alonma has quit IRC14:00
*** lkuchlan has quit IRC14:03
*** alonma has joined #openstack-cinder14:03
*** mriedem has joined #openstack-cinder14:06
tommylik_hey ildikov, hope I understand your idea correctly: https://review.openstack.org/#/c/449858/, https://review.openstack.org/#/c/451252/14:07
openstackgerritAdriano Freires Rosso proposed openstack/cinder master: HNAS: Mark NFS driver as unsupported  https://review.openstack.org/45184014:07
*** ntpttr has quit IRC14:08
*** ntpttr has joined #openstack-cinder14:09
*** Poornima has quit IRC14:09
*** eharney has quit IRC14:09
ildikovtommylik_: hi14:10
tommylik_ildikov: hi14:10
ildikovtommylik_: I think jgriffith was suggesting to have server-id as opposed to server-uuid14:10
tommylik_ok14:11
ildikovtommylik_: so we don't lock users into a cage especially as we do not really care whether it's a uuid or not14:11
ildikovjgriffith: please correct me if I got it wrong which can easily happen :)14:12
tommylik_ildikov: ok, I will fix that14:12
openstackgerritTommyLike proposed openstack/cinder-specs master: Explicit user messages  https://review.openstack.org/45176114:13
lpetrutHi, I have the following two Cinder SMB driver related blueprints:14:15
lpetruthttps://blueprints.launchpad.net/cinder/+spec/smbfs-nova-assisted-volume-snapshots14:15
lpetruthttps://blueprints.launchpad.net/cinder/+spec/remotefs-pools-support14:15
lpetrutcould someone please approve them? the code is up for review as well.14:15
lpetrutwe're basically enabling nova assisted volume snapshots for the SMB driver, as well as reporting each share as a pool so that the scheduler is aware of them14:15
*** chenhb has quit IRC14:16
smcginnislpetrut: Will do that now.14:16
lpetrutsmcginnis: thank you14:16
*** chenhb has joined #openstack-cinder14:16
*** lkuchlan has joined #openstack-cinder14:18
*** zhangguo_ has quit IRC14:18
ildikovtommylik_: I think we could also fix the REST API too now14:19
*** ntpttr has quit IRC14:19
*** lamt has joined #openstack-cinder14:19
*** tommylik_ has quit IRC14:19
ildikovtommylik_: so the external interfaces would be consistent and internally we can figure out what to do with the code14:19
*** tommylikehu_ has joined #openstack-cinder14:20
ildikovtommylikehu_: what do you think? ^14:20
*** mriedem has quit IRC14:20
*** ntpttr has joined #openstack-cinder14:21
tommylikehu_ildikov: sure, I could handle that, one question, if we change the attribute name should we update the nova side?14:24
*** eharney has joined #openstack-cinder14:25
*** ntpttr has quit IRC14:25
*** yulijie has quit IRC14:26
*** yulijie has joined #openstack-cinder14:27
*** ntpttr has joined #openstack-cinder14:27
openstackgerritTommyLike proposed openstack/python-cinderclient master: [BugFix] Make 'instance_id' required in attachment-create CLI.  https://review.openstack.org/44985814:27
ildikovtommylikehu_: we only have PoC code for Nova up for review at this point, so that's not an issue14:28
*** ntpttr has quit IRC14:31
openstackgerritMerged openstack/cinder master: Remove unused CG code  https://review.openstack.org/44601814:32
tommylikehu_ildikov: If that is true, maybe we can unify the 'connection_info' and 'connector' too?14:32
*** pdeore has quit IRC14:32
*** ntpttr has joined #openstack-cinder14:33
openstackgerritMerged openstack/cinder master: Fix error status check when create backup  https://review.openstack.org/45121114:33
ildikovtommylikehu_: I'm not the expert on what kinds of information packages we have in the sense of who passes what to os-brick and what they get back and what we call those... :)14:34
*** Julien-zte has quit IRC14:35
*** ntpttr has quit IRC14:37
tommylikehu_ildikov: I am only talking about unifying the request and response parameter. So I think it's less envolved and easy for other components to cooperate.14:38
*** ntpttr has joined #openstack-cinder14:39
*** yangyapeng has quit IRC14:39
ildikovtommylikehu_: sure, if you have the way forward already I would say go ahead14:40
ildikovtommylikehu_: better now than when we start using it14:40
tommylikehu_ildikov: thanks14:40
ildikovtommylikehu_: np :)14:41
*** isaacb_ has quit IRC14:41
*** dikonoo has quit IRC14:43
*** dikonoor has quit IRC14:44
*** eharney has quit IRC14:44
*** lkuchlan has quit IRC14:45
*** ntpttr has quit IRC14:45
*** diablo_rojo has joined #openstack-cinder14:46
*** ntpttr has joined #openstack-cinder14:47
*** isaacb_ has joined #openstack-cinder14:47
*** yangyapeng has joined #openstack-cinder14:49
*** nkrinner is now known as nkrinner_afk14:49
*** markstur has joined #openstack-cinder14:50
diablo_rojogeguileo, smcginnis jungleboyj around?14:52
smcginnisdiablo_rojo: More oblong.14:54
diablo_rojoI got an email in the wee hours of the morning from a jun zhong (I think a manila person?) That said that oslo.service already handles the sighup like we want for dynamic config- or thought we wanted cause it is already there and works.14:55
smcginnisdiablo_rojo: Oh good.14:55
smcginnisdiablo_rojo: So we just need to tie in to that?14:55
diablo_rojosmcginnis, so..if its already there and we didn't do it.. what should I do with the spec?14:55
diablo_rojosmcginnis, it already works, I dont think we need to do anything else14:56
smcginnisdiablo_rojo: Really? There's nothing more that we need to do to handle that correctly?14:56
diablo_rojosmcginnis, I believe so? I need to do more testing, and it would be great if others could too, but I think this is now the third person that says its already a thing14:57
diablo_rojoAnd its an outside source :)14:57
*** xyang has joined #openstack-cinder14:57
*** pdeore has joined #openstack-cinder14:57
smcginnisdiablo_rojo: Well cool. If it's just a matter of testing to validate it, that should be a lot less work. ;)14:57
*** lkuchlan has joined #openstack-cinder14:57
*** alonma has quit IRC14:58
*** ntpttr has quit IRC14:58
diablo_rojosmcginnis, truth. So, what do we need to do with the spec then? I cleaned up the formatting things already.14:58
smcginnisdiablo_rojo: Let's hold on that until we know nothing further is needed I guess.14:58
diablo_rojoIt has two -1's but not for anything that is actually wrong.14:58
tommylikehu_diablo_rojo: So you are going to handle this implementation during Pike?14:58
diablo_rojotommylikehu_, are you the one that sent me the email?14:59
*** ntpttr has joined #openstack-cinder14:59
*** alonma has joined #openstack-cinder14:59
*** alonma has quit IRC14:59
diablo_rojotommylikehu_, if it already works, I dont understand what needs to be implemented.14:59
tommylikehu_diablo_rojo: I am a workmate of zhongjun, she is going to bring that feature into manila14:59
diablo_rojotommylikehu_, Ah got it :) The pieces are fitting together.14:59
*** eharney has joined #openstack-cinder15:00
*** alonma has joined #openstack-cinder15:00
*** rcernin has quit IRC15:00
tommylikehu_diablo_rojo: we need to investigate, I think she will show up in about 5 minutes15:00
zhongjun_tommylikehu_: me?15:00
tommylikehu_diablo_rojo: manila week meeting is on going and she has a topic15:00
diablo_rojozhongjun_, Hello :)15:01
tommylikehu_zhongjun_: nice you are here15:01
zhongjun_diablo_rojo: hi :)15:01
zhongjun_I am in manila meeting15:01
diablo_rojozhongjun_, so you tested what cinder has and it works according to the spec? Or it just works inherently?15:01
tommylikehu_diablo_rojo: she could be busy with manila meetings at present :)15:03
diablo_rojotommylikehu_, I figured :) I can wait :)15:04
*** mlakat has quit IRC15:05
*** jwcroppe has joined #openstack-cinder15:07
*** pdeore has quit IRC15:08
*** pdeore has joined #openstack-cinder15:09
*** ntpttr has quit IRC15:10
*** ebalduf has quit IRC15:11
jungleboyjdiablo_rojo:  I had tried it on a box here and didn't get the expected behavior but I could have been doing it wrong.15:11
*** ntpttr has joined #openstack-cinder15:11
jungleboyjdiablo_rojo:  So we need to test and then document what the behavior is at a minimum.15:11
*** isaacb__ has joined #openstack-cinder15:11
*** ebalduf has joined #openstack-cinder15:12
diablo_rojojungleboyj, agreed. I dont currently have things set up to test so I am collecting other peoples reports. What happened when you tried?15:13
diablo_rojoAlso what did you try exactly15:13
*** cknight has joined #openstack-cinder15:13
jungleboyjdiablo_rojo:  I had a box running the volume service.  I update the config file to change logging level or something simple and then I sent a sighup to the volume process.  The process reported it had seen the signal and then waited for the children to shutdown.15:14
jungleboyjThen it exited.  That was it.15:14
*** mriedem has joined #openstack-cinder15:15
*** isaacb_ has quit IRC15:15
*** ntpttr has quit IRC15:15
diablo_rojojungleboyj, interesting, from what I understand zhongjun_ actually had it work so we will see what she says when she gets out of the Manila meeting.15:16
jungleboyjdiablo_rojo:  Ok, the question is whether the process is supposed to just exit or if it is expected that once it has drained the children it should automatically restart.15:16
jungleboyjIf it is expected that systemd or something is going to externally restart it, then maybe it is working.15:17
*** ntpttr has joined #openstack-cinder15:17
smcginnisProbably can't rely on systemd.15:17
openstackgerritMerged openstack/os-brick master: Move vzstorage related code out of RemoteFsClient  https://review.openstack.org/45175615:18
*** lkuchlan has quit IRC15:18
*** deepak_jon has quit IRC15:20
diablo_rojoDo people trust/ is there a way to make it restart without something external?15:20
*** tommylikehu_ has quit IRC15:23
*** alonma has quit IRC15:23
*** alonma has joined #openstack-cinder15:25
*** alonma has quit IRC15:29
isaacb__xyang: Can you please review again https://review.openstack.org/#/c/448159/15:33
raroraIn regards to the bandit gate, the patchset (https://review.openstack.org/#/c/446124/2) that added the tox job has been merged. What are the next steps in order to actually integrate it as a non-voting job?15:34
*** pdeore has quit IRC15:34
openstackgerritMerged openstack/cinder master: Fix some format error in docstrings  https://review.openstack.org/44990815:34
eharneyrarora: it will need to be added to project-config, does the job need to just launch a tox env?15:36
jungleboyjdiablo_rojo:  I think that is part of what we need to determine.15:36
raroraeharney: I'm not really sure, is there a way to test that? I know you can run the bandit-baseline command by itself and the tox job basically just runs the command for us15:37
diablo_rojojungleboyj, ha ha yes, that is why I am asking. Curious about what people think.15:38
eharneyrarora: well what commands need to run for you to determine success/failure?15:38
jungleboyjWell, Sean15:38
eharneydiablo_rojo: wasn't the goal to have SIGHUP just reload the conf on the running service?15:39
jungleboyj's input was probably not.  Would need a deployer's input like eharney15:39
raroraeharney: bandit-baseline -r cinder -x tests -ll -ii -n515:39
jungleboyjeharney:  Right.15:39
raroraeharney: that will run bandit-baseline on cinder excluding tests and does medium confidence/severity and shows at most 5 lines for each issue15:39
diablo_rojoeharney, yes, but how realistic is that? Is that something you think would actually work and not break a bunch of things?15:40
*** jkraj has quit IRC15:40
xyangisaacb_: sure, I'll take a look later15:40
smcginniseharney: +115:41
eharneydiablo_rojo: I think it's realistic and is the right goal -- I saw some issues when I experimented with this long ago but it should be doable15:41
smcginnisrarora: You should be able to take a look at how the pylint jobs are set up.15:41
eharneydiablo_rojo: doing anything else is going to be kind of weird, better to behave like a "standard" service15:41
diablo_rojoeharney, Okay cool. Just trying to get the lay of the land here.15:42
eharneyrarora: let me look at this for a minute15:43
raroraeharney: was talking to jonesn and he said he believes that yes it just needs to launch a a tox env15:43
raroraeharney: ok no worries and no rush15:44
eharneyrarora: will there be similar jobs for other projects?15:44
rarorasmcginnis: ok, will take a look at the pylint stuff15:44
*** e0ne has quit IRC15:44
raroraeharney: yes I would imagine I can reach out to some people I know on Keystone as they have a bandit gate15:44
eharneyrarora: what's the name of that job?15:50
*** aarefiev is now known as aarefiev_afk15:51
*** yangyapeng has quit IRC15:52
raroraeharney: bandit-baseline15:52
*** isaacb__ has quit IRC15:56
jungleboyjOk, manila meeting is done.15:56
jungleboyjeharney:  So, what do you consider a 'standard service' then?15:57
zhongjun_diablo_rojo, jungleboyj: ^ ^, I just use signal-handling in oslo_service .   https://docs.openstack.org/developer/oslo.service/usage.html#signal-handling15:58
* jungleboyj is reading15:59
zhongjun_When I send SIGHUP  to manila service, the  def reset(self):  fuction will recieve it.16:00
*** raunak has joined #openstack-cinder16:00
*** yangyapeng has joined #openstack-cinder16:00
zhongjun_s/recieve/receive16:01
jungleboyjzhongjun_:  Does the manila service end when you do that?16:01
*** alonma has joined #openstack-cinder16:02
diablo_rojoAccording to the oslo docs it sounds like it will restart16:02
diablo_rojo'On receiving SIGHUP configuration files are reloaded and a service is being reset and started again. Then all child workers are gracefully stopped using SIGTERM and workers with new configuration are spawned. Thus, SIGHUP can be used for changing config options on the go.'16:02
*** pbandark has quit IRC16:02
jungleboyjdiablo_rojo: That is how I read that too.  :-)16:02
zhongjun_jungleboyj: I just do a simple test, manila service is still work when I do that16:02
jungleboyjzhongjun_:  Even in devstack?16:02
zhongjun_I just send to manila thread16:03
zhongjun_kill SIGHUP 1358516:03
jungleboyjzhongjun_:  It reloads the configuration file and continues on.16:04
*** joanna__ has joined #openstack-cinder16:04
*** jordanP_ has joined #openstack-cinder16:05
zhongjun_jungleboyj: I didn't write the code about reloads the configuration file, but I think it will be OK, if we write code in reset() function,  and continues on16:05
jungleboyjzhongjun_:  I have a manila running right now.  So does this work for any of the processes there?16:07
zhongjun_I think it work for the processes  inherit  service.ServiceBase  class16:10
jungleboyjThe share and api services just end when I send a SIGHUP.16:11
jungleboyjzhongjun_:  ^^16:11
zhongjun_which signal hand do you catch?16:15
zhongjun_INFO oslo_service.service [-] Caught SIGHUP16:16
jungleboyjLet me see what it caought.16:16
*** alonma has quit IRC16:17
openstackgerritMerged openstack/cinder master: Support for HostAddress opt  https://review.openstack.org/45182416:18
zhongjun_jungleboyj: sorry, I need to go to sleep, If you have some things, you could leave message to me.16:19
jungleboyjOk.  I am not seeing any message about the signal being caught.16:19
jungleboyjWill look closer.16:19
zhongjun_log like this:    INFO oslo_service.service [-] Child caught SIGHUP16:20
*** mriedem has quit IRC16:22
jungleboyjYeah, that isn't happening in my devstack, it just exits.16:23
*** pcaruana has quit IRC16:23
*** psachin has quit IRC16:24
*** tommylikehu1 has joined #openstack-cinder16:25
*** tommylikehu has quit IRC16:27
*** tommylikehu1 is now known as tommylikehu16:27
openstackgerritMerged openstack/cinder master: ScaleIO: Fixing support for SIO 1.3x  https://review.openstack.org/44867316:29
*** chenhb has quit IRC16:31
*** chenhb has joined #openstack-cinder16:31
*** kaisers__ has quit IRC16:34
*** kaisers_ has joined #openstack-cinder16:36
*** jonesn has joined #openstack-cinder16:36
*** markstur_ has joined #openstack-cinder16:36
*** markstur has quit IRC16:40
openstackgerritSatyajeet Shahane proposed openstack/cinder master: Add new column to the 'transfers' tables  https://review.openstack.org/45190116:40
openstackgerritSatyajeet Shahane proposed openstack/cinder master: [WIP] Update 'transfer' table queries  https://review.openstack.org/45190216:40
*** tesseract has quit IRC16:40
*** alonma has joined #openstack-cinder16:40
*** alonma has joined #openstack-cinder16:40
*** kaisers_ has quit IRC16:41
*** xyang has quit IRC16:44
*** xyang has joined #openstack-cinder16:45
*** lpetrut has quit IRC16:46
*** adrianofr has joined #openstack-cinder16:47
*** ntpttr has quit IRC16:51
*** xyang has quit IRC16:52
eharneyjungleboyj: standard being that SIGHUP reloads config16:52
*** lucasxu has quit IRC16:53
*** ntpttr has joined #openstack-cinder16:54
*** lucasxu has joined #openstack-cinder16:56
*** Apoorva has joined #openstack-cinder17:00
openstackgerritMerged openstack/cinder master: HNAS: Mark NFS driver as unsupported  https://review.openstack.org/45184017:01
*** tomh has joined #openstack-cinder17:01
*** tomh has quit IRC17:01
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being restarted briefly to relieve performance issues, and should return to service again momentarily.17:02
*** Apoorva has quit IRC17:03
*** Apoorva has joined #openstack-cinder17:04
openstackgerritTom Swanson proposed openstack/cinder master: Dell EMC SC: Support generic groups  https://review.openstack.org/44336417:05
*** xyang has joined #openstack-cinder17:07
*** ntpttr has quit IRC17:07
*** ntpttr has joined #openstack-cinder17:09
*** arunkant has quit IRC17:13
*** ntpttr has quit IRC17:16
*** ntpttr has joined #openstack-cinder17:17
jungleboyjeharney:  Ok, so that isn't what is happening.17:19
*** ChubYann has joined #openstack-cinder17:20
jungleboyjdiablo_rojo:  I don't think we have yet utilized service.ServiceBase to enable this.  The only place I see service.ServiceBase being used is for the WSGIService .17:21
eharneyjungleboyj: it's not expected to be happening yet, right?17:21
jungleboyjWell, there have been conflicting reports there.  :-)17:21
eharneydid anyone implement it..?17:21
jungleboyjI think we are circling in on the answer.17:21
*** xyang has quit IRC17:23
diablo_rojoeharney, sounds like Manila uses it and we should be.17:23
diablo_rojojungleboyj, should be a relatively small change right?17:23
jungleboyjdiablo_rojo:  I am not sure that manila does because I don't see it in the code I have pulled down.17:24
jungleboyjdiablo_rojo:  Anyway, so I think we need to change up cinder/service.py to use service.ServiceBase to get the functionality.17:24
diablo_rojojungleboyj, huh, then I wonder how it was working for zhongjun_17:24
diablo_rojojungleboyj, Okay I can modify the spec to say that.17:25
eharney?  we already use service.Service in service.py, which is based on ServiceBase17:26
*** ntpttr has quit IRC17:26
jungleboyjeharney:  Ok ...17:26
jungleboyjSo we must not be using it right then?17:27
*** ntpttr has joined #openstack-cinder17:27
jungleboyjOr I am not testing it properly.17:27
diablo_rojoLooks like ServiceBase isnt used for the class service in service.py17:27
diablo_rojojust for the WSGI17:27
eharneySIGHUP calls def reset(), which for us ends up calling reset() in the manager17:28
hemnahttps://github.com/openstack/cinder/blob/master/cinder/service.py#L11817:28
hemnawe already extend service.Service17:28
jungleboyjYeah, that is  different from Manila .17:28
eharneyso don't we just have to change reset() in our code to do what we want w/ config files?17:28
diablo_rojoIf we use service.ServiceBase there the functionality should work?17:29
hemnawe already use service.Service17:29
eharneywe already are using ServiceBase, because service.Service is based on it.17:29
hemnause the code Luke....17:29
hemnahttps://github.com/openstack/oslo.service/blob/master/oslo_service/service.py#L64417:29
diablo_rojoservice doesn't even look like a word anymore lol17:30
diablo_rojoeharney, then yes? I think we just need to change the restart method..17:30
*** david-lyle__ is now known as david-lyle17:30
eharneydiablo_rojo: isn't the restart method already "reload" by default?17:31
*** xyang has joined #openstack-cinder17:32
diablo_rojohttps://github.com/openstack/cinder/blob/master/cinder/service.py#L52817:33
diablo_rojoLet me look17:34
*** ntpttr has quit IRC17:35
*** chlong has quit IRC17:36
diablo_rojoeharney, yeah it looks like it17:36
diablo_rojohttps://github.com/openstack/oslo.service/blob/master/oslo_service/service.py#L23817:36
*** ntpttr has joined #openstack-cinder17:37
openstackgerritAdriano Freires Rosso proposed openstack/cinder master: Hitachi: Mark HBSD and VSP as unsupported  https://review.openstack.org/45192317:39
eharneydiablo_rojo: so what's the expectation there?17:40
*** ntpttr has quit IRC17:44
openstackgerritxing-yang proposed openstack/cinder master: Tiramisu: replication group support  https://review.openstack.org/35222817:44
*** ntpttr has joined #openstack-cinder17:45
diablo_rojoeharney, You should be able to change a config option send the sighup and the services should restart with the new configuration. From what I am seeing, that should work already. Some people have said it does and others say it doesnt. I'm at a loss :)17:46
eharneydiablo_rojo: my starting assumption was that we would need to do some work to make that actually function  (not based on much other than guessing, though)17:47
openstackgerritxing-yang proposed openstack/cinder master: WIP: Fix backup temp snapshot path on remote node  https://review.openstack.org/31943417:47
smcginnisLooks like it just resets the version pins in the manager: https://github.com/openstack/cinder/blob/master/cinder/manager.py#L13717:48
smcginnisWouldn't we need code to explicitly restart?17:48
eharneysmcginnis: yeah, but the method that calls that method also calls up the stack with super(), and it's not clear what all happens up there17:48
eharneyi think you are right though that there is some integration that just isn't done there17:49
smcginnisWell this comment in oslo.service sure makes it sound like it should just do what we want: """Reload config files and restart service.17:49
eharneysure, but even if it reloads the conf file, is it going to know how to propagate that through our multi-backend self.configuration bits?17:50
eharneyit seems unlikely that all of that will just happen17:51
smcginnisThis looks like it should be restarting things: https://github.com/openstack/oslo.service/blob/master/oslo_service/service.py#L70317:51
smcginnisBut I've never had to dig into how this area of the code really works, so this is just a high level assumption.17:51
eharneyyeah, same issue here17:52
*** ntpttr has quit IRC17:52
diablo_rojoI would think the super call in the cinder restart would go to the restart in oslo.service17:52
diablo_rojoBased on my extensive knowledge of computer science and logic ;)17:52
smcginnisI can't test right now, but sounds like we need someone to actually run through this and step through in a debugger to see what's really happening if it is not doing what we think it should.17:52
diablo_rojoBut apparently thats not the case17:52
*** ntpttr has joined #openstack-cinder17:53
jungleboyjeharney: smcginnis  Right now if I take a devstack and send a SIGHUP it just cleans up the children and shuts down.17:53
diablo_rojojungleboyj, can you see where it goes instead of to the oslo.service restart?17:54
openstackgerritMerged openstack/cinder master: Add an optional db session argument to cinder.db.sqlalchemy.api. quota_allocated_get_all_by_project(). Then in quota_reserve() pass the session that is fetched for that method so that a new session does not have to be made. This can avoid DB timeouts and   https://review.openstack.org/45182117:55
*** kaisers_ has joined #openstack-cinder17:56
smcginnisHow did that commit message get through?17:56
*** xyang has quit IRC17:56
*** chris_morrell has joined #openstack-cinder17:56
*** ameade_ has joined #openstack-cinder17:56
eharneylol17:56
diablo_rojosmcginnis, LOL I think that is a little longer than 50 chars17:56
*** lpetrut has joined #openstack-cinder17:57
*** SamYaple_ has joined #openstack-cinder17:57
jungleboyjsmcginnis: Ooops?17:57
* jungleboyj is laughing histerically.17:58
jungleboyjgerrit made it look like it was just one line.17:58
smcginnisSo much for "git log --pretty=oneline" being easily parsable.17:58
jungleboyjShit.17:59
jungleboyjWe should revert that and fix it?17:59
*** ameade has quit IRC17:59
*** ekhugen_alt has quit IRC17:59
*** SamYaple has quit IRC17:59
*** BlackDex has quit IRC17:59
diablo_rojoEven that first line was 64 :)17:59
smcginnisNot sure if it's worth it. Though I'm tempted. :)17:59
*** ntpttr has quit IRC18:00
*** ekhugen_alt has joined #openstack-cinder18:00
openstackgerritLee Yarwood proposed openstack/os-brick master: WIP encryptors: Add missing noop constant  https://review.openstack.org/45193018:00
openstackgerritLee Yarwood proposed openstack/os-brick master: WIP encryptors: block out of tree encryptors  https://review.openstack.org/45193118:00
jungleboyjAnyway, I just SIGHUPed my volume process and it definitely exited.  oslo_service.service reports that the parent process unexpected dies.18:00
jungleboyjdiablo_rojo:  So, something is broken there ^^18:01
jungleboyjsmcginnis: I need to run for a bit.  I F'ed up that commit.  I will fix it later.18:01
*** lpetrut has quit IRC18:01
*** ameade_ is now known as ameade18:01
*** ntpttr has joined #openstack-cinder18:01
diablo_rojoWell exiting is a good first step :)18:01
jungleboyjsmcginnis:  It was a much needed laugh.18:01
diablo_rojoJust gotta restart now18:01
*** kaisers_ has quit IRC18:01
jungleboyjdiablo_rojo:  Ok.  get devstacking18:02
eharneyjungleboyj: there were some related finds in this bug that you may want to read -- devstack services may not act like other services : https://bugs.launchpad.net/cinder/+bug/154899218:02
openstackLaunchpad bug 1548992 in Cinder "SIGHUP causes problems with the LVM driver" [Undecided,Invalid] - Assigned to Eric Harney (eharney)18:02
eharney(why, i don't know)18:02
*** chlong has joined #openstack-cinder18:02
smcginniseharney: So we might need an actual full deployment to test this right.18:02
eharneythe claim there is that since devstack runs the service as a foreground process, it may do something different from normally deployed services18:02
*** SamYaple_ has quit IRC18:03
smcginniseharney: Ah, I could see that being the case.18:03
eharneysmcginnis: i guess it would make sense if you were running the service in a terminal, and closed the terminal, etc...18:03
jungleboyjI was afraid it might be something like that.18:03
*** ntpttr has quit IRC18:06
eharneyjungleboyj: https://git.openstack.org/cgit/openstack/oslo.service/tree/oslo_service/service.py?h=1.21.0#n59418:06
eharneyjungleboyj: is_sighup_and_daemon makes it do exactly what you describe18:06
diablo_rojodevstack is a special flower18:06
*** ntpttr has joined #openstack-cinder18:07
eharneywow, apparently devstack can run services w/ systemd: https://git.openstack.org/cgit/openstack-dev/devstack/tree/SYSTEMD.rst18:08
*** mvk has quit IRC18:10
smcginnisHmm, interesting.18:11
hemnaeharney, is it all or nothing?18:11
smcginnisAnd all you need is USE_SYSTEMD=True18:11
hemnascreen still makes a lot of sense for project developers18:11
hemnainstead of tailing logs18:11
*** chlong has quit IRC18:11
*** ntpttr has quit IRC18:12
eharneyyou could spawn up a bunch of screen windows running journalctl -f and get the same thing18:12
eharneywell, except for pdb18:12
*** xinli has quit IRC18:12
hemnayah, but CTRL-C, up arrow, return18:12
hemnais easy to restart c-vol18:12
hemnavs needing another terminal and restarting18:13
hemnaanyway18:13
*** ntpttr has joined #openstack-cinder18:13
hemnaif you could put all services minus cinder-* into systemd that'd be cool.18:13
hemnawell.....maybe n-cpu too18:13
oomichihi, can I get some feedback on https://review.openstack.org/#/c/448733 ?18:15
*** ntpttr has quit IRC18:18
*** ntpttr has joined #openstack-cinder18:18
*** xinli has joined #openstack-cinder18:20
*** ntpttr has quit IRC18:22
*** ntpttr has joined #openstack-cinder18:23
*** ntpttr has quit IRC18:27
*** ntpttr has joined #openstack-cinder18:29
*** ntpttr has quit IRC18:34
*** ntpttr has joined #openstack-cinder18:34
*** crose has quit IRC18:36
*** ntpttr has quit IRC18:39
*** ntpttr has joined #openstack-cinder18:41
*** mvk has joined #openstack-cinder18:43
*** ntpttr has quit IRC18:46
*** ntpttr has joined #openstack-cinder18:47
*** adisky_ has quit IRC18:49
*** joanna__ has quit IRC18:57
*** chlong has joined #openstack-cinder19:00
*** joanna__ has joined #openstack-cinder19:04
*** xyang has joined #openstack-cinder19:05
*** xyang has quit IRC19:06
*** chlong has quit IRC19:07
*** joanna__ has quit IRC19:08
*** xyang has joined #openstack-cinder19:12
*** xyang1 has joined #openstack-cinder19:13
*** pjm6 has quit IRC19:14
*** xyang has quit IRC19:15
openstackgerritMerged openstack/cinder master: Remove XML description from media-types  https://review.openstack.org/44873319:16
*** pjm6 has joined #openstack-cinder19:17
*** joanna__ has joined #openstack-cinder19:17
*** joanna__ has quit IRC19:20
*** xyang has joined #openstack-cinder19:24
*** dillaman has joined #openstack-cinder19:24
*** ntpttr has quit IRC19:25
*** lpetrut has joined #openstack-cinder19:27
*** xyang has quit IRC19:27
*** xyang has joined #openstack-cinder19:27
*** ntpttr has joined #openstack-cinder19:27
*** joanna__ has joined #openstack-cinder19:31
*** ntpttr has quit IRC19:32
*** ntpttr has joined #openstack-cinder19:33
*** eharney has quit IRC19:34
*** joanna__ has quit IRC19:34
*** ntpttr has quit IRC19:40
*** ntpttr has joined #openstack-cinder19:41
*** mvk has quit IRC19:42
*** lpetrut has quit IRC19:49
*** eharney has joined #openstack-cinder19:49
*** joanna__ has joined #openstack-cinder19:50
*** mriedem has joined #openstack-cinder19:52
*** mvk has joined #openstack-cinder19:55
*** joanna__ has quit IRC19:55
*** joanna__ has joined #openstack-cinder19:59
*** viks has joined #openstack-cinder20:02
*** mriedem has quit IRC20:04
*** ntpttr has quit IRC20:04
*** mriedem has joined #openstack-cinder20:04
*** ntpttr has joined #openstack-cinder20:05
*** e0ne has joined #openstack-cinder20:07
*** mriedem has quit IRC20:07
*** mriedem has joined #openstack-cinder20:09
*** jordanP_ has quit IRC20:10
*** xyang has quit IRC20:11
*** e0ne has quit IRC20:11
openstackgerritAdriano Freires Rosso proposed openstack/cinder master: Hitachi: Mark HBSD and VSP as unsupported  https://review.openstack.org/45192320:11
*** xyang has joined #openstack-cinder20:12
*** ntpttr has quit IRC20:12
*** ntpttr has joined #openstack-cinder20:13
*** cknight has quit IRC20:14
*** alkhodos has joined #openstack-cinder20:14
alkhodosHi everyone, can some reviewers please take a look at this patch? https://review.openstack.org/#/c/424671/ CI is happy and it's been 10 days since latest patchset.20:16
*** ntpttr has quit IRC20:18
*** adrianofr has quit IRC20:19
*** eharney has quit IRC20:19
*** ntpttr has joined #openstack-cinder20:19
*** xyang has quit IRC20:21
*** mriedem has quit IRC20:22
*** mriedem has joined #openstack-cinder20:22
*** mriedem has quit IRC20:23
*** xyang has joined #openstack-cinder20:23
*** mriedem has joined #openstack-cinder20:23
*** ntpttr has quit IRC20:24
*** ntpttr has joined #openstack-cinder20:25
*** ntpttr has quit IRC20:29
*** xyang has quit IRC20:30
*** mriedem has quit IRC20:30
*** mriedem has joined #openstack-cinder20:31
*** mriedem has quit IRC20:31
*** ntpttr has joined #openstack-cinder20:31
*** mriedem has joined #openstack-cinder20:32
*** xyang has joined #openstack-cinder20:32
*** raunak_ has joined #openstack-cinder20:32
*** raunak has quit IRC20:33
*** raunak_ is now known as raunak20:33
openstackgerritMonicaJoshi proposed openstack/cinder master: Fix for Group API update to include check policy  https://review.openstack.org/45197420:33
*** mriedem has quit IRC20:34
*** xyang has quit IRC20:34
*** mriedem has joined #openstack-cinder20:34
*** ntpttr has quit IRC20:36
*** chlong has joined #openstack-cinder20:36
*** xyang has joined #openstack-cinder20:37
*** crose has joined #openstack-cinder20:37
*** asselin has joined #openstack-cinder20:37
*** ntpttr has joined #openstack-cinder20:37
openstackgerritAleksey Ruban proposed openstack/cinder master: NexentaStor5: Added ZFS cleanup  https://review.openstack.org/42467120:38
*** asselin_ has quit IRC20:39
*** ducttape_ has quit IRC20:40
*** xyang has quit IRC20:41
*** ntpttr has quit IRC20:42
*** ntpttr has joined #openstack-cinder20:43
*** mriedem has quit IRC20:44
*** mriedem has joined #openstack-cinder20:45
*** xyang has joined #openstack-cinder20:46
*** dustins has quit IRC20:50
*** ntpttr has quit IRC20:50
*** ntpttr has joined #openstack-cinder20:50
*** mriedem has quit IRC20:52
openstackgerritAleksey Ruban proposed openstack/cinder master: NexentaStor5: Added ZFS cleanup  https://review.openstack.org/42467120:53
*** mriedem has joined #openstack-cinder20:53
*** ntpttr has quit IRC20:55
*** crushil has quit IRC20:56
*** ntpttr has joined #openstack-cinder20:57
*** xyang has quit IRC21:00
*** ntpttr has quit IRC21:02
*** catintheroof has quit IRC21:03
*** ntpttr has joined #openstack-cinder21:03
*** asselin has quit IRC21:03
*** xyang has joined #openstack-cinder21:03
*** asselin has joined #openstack-cinder21:04
*** chlong has quit IRC21:09
*** ntpttr has quit IRC21:10
*** dansmith is now known as danased21:11
*** ntpttr has joined #openstack-cinder21:11
*** abishop has quit IRC21:11
*** jonesn has quit IRC21:12
*** danased is now known as dansmith21:12
*** ntpttr has quit IRC21:16
*** ebalduf has quit IRC21:16
*** ntpttr has joined #openstack-cinder21:17
*** joanna__ has quit IRC21:19
*** ntpttr has quit IRC21:22
*** joanna__ has joined #openstack-cinder21:23
*** ntpttr has joined #openstack-cinder21:23
*** ebalduf has joined #openstack-cinder21:24
openstackgerritEmily Hugenbruch proposed openstack/cinder master: Separate out routine for getting qemu_img_info  https://review.openstack.org/44888721:25
*** yangyapeng has quit IRC21:25
*** ntpttr has quit IRC21:28
*** yangyapeng has joined #openstack-cinder21:29
*** ntpttr has joined #openstack-cinder21:29
*** edmondsw has quit IRC21:30
*** rarora has quit IRC21:30
*** xyang has quit IRC21:31
*** edmondsw has joined #openstack-cinder21:31
*** ebalduf has quit IRC21:31
*** crose has quit IRC21:32
*** mriedem has quit IRC21:32
*** dave-mccowan has quit IRC21:32
*** mriedem has joined #openstack-cinder21:32
*** mriedem has quit IRC21:33
*** cknight has joined #openstack-cinder21:33
*** mriedem has joined #openstack-cinder21:33
*** ntpttr has quit IRC21:34
*** edmondsw has quit IRC21:35
*** xyang has joined #openstack-cinder21:35
*** ntpttr has joined #openstack-cinder21:35
*** mriedem has quit IRC21:36
*** mriedem has joined #openstack-cinder21:37
*** mriedem has quit IRC21:40
*** viks has quit IRC21:40
*** ntpttr has quit IRC21:40
*** mriedem has joined #openstack-cinder21:40
*** ebalduf has joined #openstack-cinder21:41
*** ntpttr has joined #openstack-cinder21:41
*** mriedem has quit IRC21:43
*** mriedem has joined #openstack-cinder21:43
*** ntpttr has quit IRC21:50
*** ntpttr has joined #openstack-cinder21:51
*** lucasxu has quit IRC21:54
*** xyang has quit IRC21:54
*** rarora has joined #openstack-cinder21:55
*** ducttape_ has joined #openstack-cinder21:58
*** monjoshi_ has quit IRC21:59
*** ntpttr has quit IRC22:00
*** ntpttr has joined #openstack-cinder22:01
*** harlowja has quit IRC22:02
*** mriedem has quit IRC22:02
*** xyang1 has quit IRC22:04
*** ntpttr has quit IRC22:06
*** ntpttr has joined #openstack-cinder22:07
openstackgerritKarthik Prabhu Vinod proposed openstack/cinder master: Generate support matrix using driver interfaces  https://review.openstack.org/37116922:08
*** dillaman has quit IRC22:11
*** ntpttr has quit IRC22:12
*** ntpttr has joined #openstack-cinder22:13
*** xinli has quit IRC22:15
*** ntpttr has quit IRC22:18
*** ntpttr has joined #openstack-cinder22:19
*** diablo_rojo has quit IRC22:20
*** ntpttr has quit IRC22:24
*** ntpttr has joined #openstack-cinder22:25
openstackgerritTom Swanson proposed openstack/cinder master: Dell EMC SC: Support generic groups  https://review.openstack.org/44336422:26
*** ntpttr has quit IRC22:30
*** gouthamr has quit IRC22:30
*** cknight has quit IRC22:31
*** ntpttr has joined #openstack-cinder22:31
*** crushil has joined #openstack-cinder22:35
*** lamt has quit IRC22:35
*** ntpttr has quit IRC22:38
*** ntpttr has joined #openstack-cinder22:39
openstackgerritSzymon Wr√≥blewski proposed openstack/cinder-specs master: Add DB object version counting  https://review.openstack.org/42994722:41
*** ntpttr has quit IRC22:44
*** ntpttr has joined #openstack-cinder22:45
*** crushil has quit IRC22:47
karthikpsmcginnis: o/ ..got a minute22:48
openstackgerritJay Bryant proposed openstack/cinder master: Revert "Add an optional db session argument to cinder.db.sqlalchemy.api."  https://review.openstack.org/45200222:49
*** ntpttr has quit IRC22:50
*** tommylikehu_ has joined #openstack-cinder22:51
*** ntpttr has joined #openstack-cinder22:51
*** gouthamr has joined #openstack-cinder22:52
tommylikehu_hey smcginnis, jungleboyj, could you take a look at this patch? https://review.openstack.org/#/c/449858/22:52
*** tommylikehu_ has quit IRC22:52
*** tommylikehu_ has joined #openstack-cinder22:53
*** tommylikehu_ has quit IRC22:55
*** chris_morrell has quit IRC22:55
*** ntpttr has quit IRC22:56
*** jwcroppe has quit IRC22:57
*** ntpttr has joined #openstack-cinder22:57
*** chris_morrell has joined #openstack-cinder23:02
jungleboyjtommylikehu_  Looking23:03
*** ntpttr has quit IRC23:04
*** ntpttr has joined #openstack-cinder23:05
*** ntpttr has quit IRC23:10
*** ntpttr has joined #openstack-cinder23:10
*** chris_morrell has quit IRC23:12
*** ntpttr has quit IRC23:14
*** ntpttr has joined #openstack-cinder23:15
*** ntpttr has quit IRC23:20
*** ntpttr has joined #openstack-cinder23:21
*** harlowja has joined #openstack-cinder23:24
*** joanna__ has quit IRC23:25
*** stakeda has quit IRC23:28
*** ntpttr has quit IRC23:28
*** ntpttr has joined #openstack-cinder23:29
*** edmondsw has joined #openstack-cinder23:31
*** chris_morrell has joined #openstack-cinder23:33
*** ntpttr has quit IRC23:34
*** edmondsw has quit IRC23:35
*** ntpttr has joined #openstack-cinder23:35
*** ntpttr has quit IRC23:40
*** ntpttr has joined #openstack-cinder23:41
*** openstack has joined #openstack-cinder23:45
*** ntpttr has quit IRC23:46
*** ntpttr has joined #openstack-cinder23:46
*** crose has quit IRC23:47
*** chris_morrell has quit IRC23:49
openstackgerritJay Bryant proposed openstack/cinder master: Add opt db session arg to quota_allocated_get_all_by_project  https://review.openstack.org/45201223:50
*** abishop has joined #openstack-cinder23:50
*** ntpttr has quit IRC23:50
*** ntpttr has joined #openstack-cinder23:51
*** sdague has quit IRC23:55
*** ntpttr has quit IRC23:56
*** ntpttr has joined #openstack-cinder23:57

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