Monday, 2017-10-16

*** felipemonteiro has quit IRC00:02
*** felipemonteiro_ has quit IRC00:14
*** sticker has joined #openstack-cinder00:18
*** psachin has joined #openstack-cinder00:34
*** wanghao has joined #openstack-cinder00:37
*** wanghao has quit IRC00:44
*** wanghao has joined #openstack-cinder00:44
*** sdague has quit IRC00:50
*** wanghao has quit IRC00:50
*** jwcroppe has quit IRC00:51
*** wanghao has joined #openstack-cinder00:51
*** jwcroppe has joined #openstack-cinder00:52
*** wanghao has quit IRC00:55
*** wanghao has joined #openstack-cinder00:56
*** jmlowe has quit IRC00:57
*** armax has joined #openstack-cinder00:57
*** wanghao has quit IRC00:58
*** wanghao has joined #openstack-cinder00:59
*** jmlowe has joined #openstack-cinder00:59
*** wanghao has quit IRC01:03
*** wanghao has joined #openstack-cinder01:04
*** dalgaaf has quit IRC01:05
*** wanghao has quit IRC01:08
*** yangyapeng has joined #openstack-cinder01:12
*** jmlowe has quit IRC01:12
*** wanghao has joined #openstack-cinder01:17
*** wanghao has quit IRC01:23
*** wewe0901 has joined #openstack-cinder01:25
*** wanghao has joined #openstack-cinder01:27
*** Shunli has joined #openstack-cinder01:27
*** wanghao has quit IRC01:29
*** kaisers has joined #openstack-cinder01:30
*** jmlowe has joined #openstack-cinder01:31
*** kaisers_ has quit IRC01:32
*** gkadam_ has quit IRC01:35
*** dalgaaf has joined #openstack-cinder01:39
*** yangyapeng has quit IRC01:39
*** yangyapeng has joined #openstack-cinder01:41
*** sbezverk has quit IRC01:43
*** sbezverk has joined #openstack-cinder01:44
*** dhinesh has joined #openstack-cinder01:45
*** jmlowe has quit IRC01:45
*** wanghao has joined #openstack-cinder01:56
*** wanghao has quit IRC02:01
*** lhx__ has joined #openstack-cinder02:11
*** zhaochao has joined #openstack-cinder02:12
*** masber has quit IRC02:26
*** gouthamr has quit IRC02:27
*** dalgaaf has quit IRC02:30
*** dalgaaf has joined #openstack-cinder02:31
*** dave-mccowan has quit IRC02:35
*** felipemonteiro_ has joined #openstack-cinder02:38
*** gcb has joined #openstack-cinder02:51
*** crushil has joined #openstack-cinder03:01
*** crushil has left #openstack-cinder03:04
*** crushil_ has joined #openstack-cinder03:05
*** jmlowe has joined #openstack-cinder03:06
*** gkadam_ has joined #openstack-cinder03:10
*** gkadam_ has quit IRC03:10
*** gkadam has joined #openstack-cinder03:10
*** masber has joined #openstack-cinder03:20
*** felipemonteiro_ has quit IRC03:22
*** chlong has joined #openstack-cinder03:29
*** wewe0901 has quit IRC03:35
*** edmondsw has joined #openstack-cinder03:53
*** felipemonteiro_ has joined #openstack-cinder03:55
*** edmondsw has quit IRC03:57
*** dhinesh has quit IRC04:04
*** hoonetorg has quit IRC04:06
*** wes_dillingham has quit IRC04:09
*** arnewiebalck_ has joined #openstack-cinder04:10
*** dhinesh has joined #openstack-cinder04:10
*** felipemonteiro__ has joined #openstack-cinder04:15
*** dhinesh has quit IRC04:15
openstackgerritShunei Shiono proposed openstack/cinder master: NEC driver: Never use target info in migration_status  https://review.openstack.org/51173704:17
*** felipemonteiro_ has quit IRC04:18
*** hoonetorg has joined #openstack-cinder04:22
*** felipemonteiro__ has quit IRC04:27
*** Dinesh_Bhor has joined #openstack-cinder04:42
*** lkuchlan has joined #openstack-cinder04:45
*** armaan has joined #openstack-cinder04:50
*** links has joined #openstack-cinder04:57
*** markvoelker has quit IRC04:58
*** markvoelker has joined #openstack-cinder04:58
*** crushil_ is now known as crushil05:05
*** armaan has quit IRC05:05
*** armax has quit IRC05:07
*** armax has joined #openstack-cinder05:07
*** armax has quit IRC05:07
*** armax has joined #openstack-cinder05:08
*** armax has quit IRC05:08
*** armax has joined #openstack-cinder05:09
*** armax has quit IRC05:09
*** armax has joined #openstack-cinder05:10
*** armax has quit IRC05:10
*** armax has joined #openstack-cinder05:10
*** dhinesh has joined #openstack-cinder05:11
*** armax has joined #openstack-cinder05:11
*** armax has quit IRC05:11
*** armax has joined #openstack-cinder05:12
*** armax has quit IRC05:12
*** arnewiebalck_ has quit IRC05:12
*** armax has joined #openstack-cinder05:13
*** armax has quit IRC05:13
*** armax has joined #openstack-cinder05:14
*** armax has quit IRC05:14
*** armax has joined #openstack-cinder05:14
*** armax has quit IRC05:15
*** dhinesh has quit IRC05:15
*** lkuchlan has quit IRC05:19
*** lpetrut_ has joined #openstack-cinder05:23
*** lhx__ has quit IRC05:33
*** lhx__ has joined #openstack-cinder05:34
*** kaisers1 has joined #openstack-cinder05:36
*** wanghao has joined #openstack-cinder05:37
*** zhaochao has quit IRC05:39
*** wanghao has quit IRC05:41
*** edmondsw has joined #openstack-cinder05:41
*** edmondsw has quit IRC05:45
*** wanghao has joined #openstack-cinder05:46
*** markvoelker has quit IRC05:46
*** armaan has joined #openstack-cinder05:47
*** wanghao has quit IRC05:50
*** wanghao has joined #openstack-cinder05:50
*** wanghao has quit IRC05:56
*** ircuser-1 has joined #openstack-cinder05:57
*** wanghao has joined #openstack-cinder05:58
*** ChubYann has quit IRC06:00
*** wanghao has quit IRC06:02
*** wanghao has joined #openstack-cinder06:03
*** lpetrut_ has quit IRC06:06
*** lpetrut_ has joined #openstack-cinder06:06
*** wanghao has quit IRC06:07
*** wanghao has joined #openstack-cinder06:08
evrardjphello. Has someone seen this kind of error during online data migrations or should I dig deeper? http://logs.openstack.org/37/512037/1/check/gate-openstack-ansible-openstack-ansible-ceph-ubuntu-xenial/8f2d7ee/console.html#_2017-10-14_21_26_10_49179606:08
*** lhx__ has quit IRC06:09
*** pgadiya has joined #openstack-cinder06:09
*** zhaochao has joined #openstack-cinder06:09
*** pgadiya has quit IRC06:09
*** wanghao has quit IRC06:09
*** wanghao has joined #openstack-cinder06:10
*** dhinesh has joined #openstack-cinder06:12
*** lpetrut_ has quit IRC06:13
*** vivsoni has joined #openstack-cinder06:13
*** wanghao has quit IRC06:14
*** wanghao has joined #openstack-cinder06:15
*** dhinesh has quit IRC06:16
*** wanghao has quit IRC06:21
*** wanghao has joined #openstack-cinder06:21
*** lpetrut_ has joined #openstack-cinder06:22
*** wanghao has quit IRC06:23
*** wanghao has joined #openstack-cinder06:24
*** AndyWojo has quit IRC06:24
*** zeestrat has quit IRC06:24
*** serverascode has quit IRC06:24
*** ameade has quit IRC06:24
*** cargonza has quit IRC06:24
*** wolsen has quit IRC06:24
*** zhongjun has quit IRC06:24
*** sweston has quit IRC06:24
*** aarefiev has quit IRC06:24
*** wanghao has quit IRC06:24
*** gcb has quit IRC06:24
*** DuncanT has quit IRC06:25
*** TheJulia has quit IRC06:25
*** wanghao has joined #openstack-cinder06:25
*** fmccrthy has quit IRC06:25
*** fmccrthy has joined #openstack-cinder06:26
*** wanghao has quit IRC06:26
*** AndyWojo has joined #openstack-cinder06:26
*** wolsen has joined #openstack-cinder06:26
*** zeestrat has joined #openstack-cinder06:26
*** cargonza has joined #openstack-cinder06:26
*** sweston has joined #openstack-cinder06:26
*** ameade has joined #openstack-cinder06:26
*** ying_zuo has quit IRC06:26
*** serverascode has joined #openstack-cinder06:27
*** wanghao has joined #openstack-cinder06:27
*** TheJulia has joined #openstack-cinder06:27
*** aarefiev has joined #openstack-cinder06:27
*** DuncanT has joined #openstack-cinder06:27
*** wanghao has quit IRC06:27
*** wN has quit IRC06:27
*** wanghao has joined #openstack-cinder06:27
*** openstackgerrit has quit IRC06:28
*** gcb has joined #openstack-cinder06:28
*** wN has joined #openstack-cinder06:29
*** wN has quit IRC06:29
*** wN has joined #openstack-cinder06:29
*** wanghao has quit IRC06:29
*** psachin has quit IRC06:30
*** wanghao has joined #openstack-cinder06:30
*** reedip has joined #openstack-cinder06:33
*** lkuchlan has joined #openstack-cinder06:33
reedipHi All,06:33
reedipWe are facing an issue. We have 3 Compute nodes and 1 Controller. The Controller and Compute nodes are used as Storage nodes as well. The volume created on Controller nodes can be connected to the VMs deployed. But the volumes created on Compute node cannot be connected to the VMs. The error reported is for FailedISCSIInitiator.06:35
*** wanghao has quit IRC06:35
reedipWe checked and found that iscsi is no installed on Controller but installed in Compute.06:35
*** wanghao has joined #openstack-cinder06:35
reedipCan anyone give any pointers about how to resolve the issue on the compute nodes ?06:36
*** jwcroppe has quit IRC06:36
*** jwcroppe has joined #openstack-cinder06:36
*** bkopilov_ has joined #openstack-cinder06:37
*** wanghao has quit IRC06:38
*** wanghao has joined #openstack-cinder06:39
*** AlexeyAbashkin has joined #openstack-cinder06:39
*** yangyapeng has quit IRC06:42
*** wanghao has quit IRC06:42
*** openstackgerrit has joined #openstack-cinder06:42
openstackgerritjunbo.li proposed openstack/cinder master: Api-ref: Add missing api response for volume transfer  https://review.openstack.org/50402306:42
*** wanghao has joined #openstack-cinder06:42
*** yangyapeng has joined #openstack-cinder06:42
*** psachin has joined #openstack-cinder06:46
*** wanghao has quit IRC06:46
*** psachin has quit IRC06:46
*** AlexeyAbashkin has quit IRC06:46
*** psachin has joined #openstack-cinder06:46
*** lhx__ has joined #openstack-cinder06:47
*** wanghao has joined #openstack-cinder06:47
*** wanghao has quit IRC06:52
*** wanghao has joined #openstack-cinder06:53
*** wanghao has quit IRC06:56
*** wanghao has joined #openstack-cinder06:56
*** s-shiono has joined #openstack-cinder06:57
*** pgadiya has joined #openstack-cinder06:58
*** wanghao has quit IRC06:59
*** wanghao has joined #openstack-cinder06:59
openstackgerritTommyLike proposed openstack/cinder master: Add policy documentation and sample file  https://review.openstack.org/51218707:01
*** markvoelker has joined #openstack-cinder07:02
*** wanghao has quit IRC07:04
*** wanghao has joined #openstack-cinder07:05
*** lpetrut_ has quit IRC07:10
*** wanghao has quit IRC07:11
*** wanghao has joined #openstack-cinder07:12
*** wanghao has quit IRC07:17
*** tesseract has joined #openstack-cinder07:18
*** wanghao has joined #openstack-cinder07:18
*** markvoelker has quit IRC07:18
*** markvoelker has joined #openstack-cinder07:20
*** markvoelker has quit IRC07:20
*** markvoelker has joined #openstack-cinder07:20
*** wanghao has quit IRC07:20
*** wanghao has joined #openstack-cinder07:21
*** wanghao has quit IRC07:23
*** wanghao has joined #openstack-cinder07:24
*** bkopilov has quit IRC07:27
*** bkopilov has joined #openstack-cinder07:27
*** edmondsw has joined #openstack-cinder07:29
*** wanghao has quit IRC07:29
*** wanghao has joined #openstack-cinder07:30
*** edmondsw has quit IRC07:33
*** wanghao has quit IRC07:35
*** wanghao has joined #openstack-cinder07:35
*** AlexeyAbashkin has joined #openstack-cinder07:38
*** wanghao has quit IRC07:41
*** wanghao has joined #openstack-cinder07:42
*** lpetrut_ has joined #openstack-cinder07:45
*** wanghao has quit IRC07:45
*** wanghao has joined #openstack-cinder07:45
openstackgerritwang yong proposed openstack/cinder master: Inspur Cinder FC/iSCSI driver  https://review.openstack.org/50245507:47
openstackgerritTommyLike proposed openstack/cinder master: [policy in code] Add support for volume, volume type resources  https://review.openstack.org/51142707:48
*** wanghao has quit IRC07:48
*** wanghao has joined #openstack-cinder07:51
*** lpetrut_ has quit IRC07:53
*** wanghao has quit IRC07:54
*** wanghao has joined #openstack-cinder07:54
*** wanghao has quit IRC07:58
*** wanghao has joined #openstack-cinder08:04
*** wanghao has quit IRC08:07
openstackgerritTommyLike proposed openstack/cinder master: Fetch 'zoning_mode' from 'backend_defaults' section  https://review.openstack.org/51220808:07
*** armaan has quit IRC08:08
*** armaan has joined #openstack-cinder08:08
*** belmoreira has joined #openstack-cinder08:13
*** slunkad has joined #openstack-cinder08:16
*** wanghao has joined #openstack-cinder08:18
*** wanghao has quit IRC08:21
*** wanghao has joined #openstack-cinder08:22
*** zhongjun has joined #openstack-cinder08:22
*** wanghao has quit IRC08:23
*** hoonetorg has quit IRC08:25
*** belmoreira has quit IRC08:25
*** ying_zuo has joined #openstack-cinder08:26
*** s-shiono has quit IRC08:27
*** armaan has quit IRC08:28
*** armaan has joined #openstack-cinder08:28
*** wanghao has joined #openstack-cinder08:29
*** wanghao has quit IRC08:30
*** markvoelker has quit IRC08:31
*** wanghao has joined #openstack-cinder08:33
openstackgerritJack Lu proposed openstack/os-brick master: Update comments for _validate_iface_transport  https://review.openstack.org/51221208:33
*** wanghao has quit IRC08:35
*** wanghao has joined #openstack-cinder08:35
*** wanghao has quit IRC08:39
*** wanghao has joined #openstack-cinder08:40
*** pgadiya has quit IRC08:40
*** wanghao has quit IRC08:45
*** markvoelker has joined #openstack-cinder08:45
*** wanghao has joined #openstack-cinder08:46
*** wanghao has quit IRC08:51
*** wanghao has joined #openstack-cinder08:52
*** wanghao has quit IRC08:56
*** wanghao has joined #openstack-cinder08:56
openstackgerritJack Lu proposed openstack/os-brick master: Update supported transports for iscsi connector  https://review.openstack.org/51077608:58
*** wanghao has quit IRC08:58
*** wanghao has joined #openstack-cinder08:59
*** wanghao has quit IRC09:01
*** wanghao has joined #openstack-cinder09:02
openstackgerritJack Lu proposed openstack/os-brick master: Update supported transports for iscsi connector  https://review.openstack.org/51077609:02
openstackgerritJack Lu proposed openstack/os-brick master: Update supported transports for iscsi connector  https://review.openstack.org/51077609:02
*** wanghao has quit IRC09:04
*** wanghao has joined #openstack-cinder09:05
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Implement Tiramisu feature on VMAX  https://review.openstack.org/50558109:05
*** belmoreira has joined #openstack-cinder09:05
*** belmoreira has quit IRC09:06
*** wanghao has quit IRC09:07
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Remove workload for next gen arrays  https://review.openstack.org/50707009:07
*** wanghao has joined #openstack-cinder09:07
*** e0ne has joined #openstack-cinder09:08
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Deprecate backend xml configuration  https://review.openstack.org/50884309:09
*** wanghao has quit IRC09:09
*** wanghao has joined #openstack-cinder09:10
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Add async replication support  https://review.openstack.org/50922209:11
*** wanghao has quit IRC09:13
*** belmoreira has joined #openstack-cinder09:13
*** wanghao has joined #openstack-cinder09:14
*** wanghao has quit IRC09:19
*** wanghao has joined #openstack-cinder09:19
*** wanghao has quit IRC09:22
*** wanghao has joined #openstack-cinder09:23
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Add async replication support  https://review.openstack.org/50922209:25
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Enable CHAP authentication for vmax backend  https://review.openstack.org/51061409:26
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Cannot extend volumes in a replication relationship  https://review.openstack.org/51181209:27
*** wanghao has quit IRC09:27
*** Shunli has quit IRC09:33
*** wanghao has joined #openstack-cinder09:33
openstackgerritCiara Stacke proposed openstack/cinder master: VMAX driver - Cannot extend volumes in a replication relationship  https://review.openstack.org/51181209:33
*** wanghao has quit IRC09:39
*** wanghao has joined #openstack-cinder09:40
*** belmoreira has quit IRC09:43
*** wanghao has quit IRC09:44
*** wanghao has joined #openstack-cinder09:44
*** e0ne has quit IRC09:45
*** belmoreira has joined #openstack-cinder09:46
*** wanghao has quit IRC09:46
*** Guest50022 is now known as geguileo09:47
*** geguileo is now known as Guest3654009:48
*** wanghao has joined #openstack-cinder09:52
*** kwathore_ has quit IRC09:52
*** AlexeyAbashkin has quit IRC09:53
*** AlexeyAbashkin has joined #openstack-cinder09:53
*** kwathore has joined #openstack-cinder09:54
*** wanghao has quit IRC09:54
*** mvk has quit IRC09:54
*** wanghao has joined #openstack-cinder09:54
*** wanghao has quit IRC09:57
*** wanghao has joined #openstack-cinder09:58
*** Guest36540 is now known as geguile-10:02
*** wanghao has quit IRC10:02
*** geguile- is now known as geguileo_10:03
*** wanghao has joined #openstack-cinder10:03
*** kwathore has quit IRC10:06
*** lkuchlan has quit IRC10:07
evrardjpso no one here got an issue recently with cinder-manage db online_data_migrations?10:08
evrardjpmy db sync seems to run fine10:08
*** wanghao has quit IRC10:08
evrardjpmy online_data_migrations seem to have 0 migrations (new install), but miserably fails10:08
evrardjpCRITICAL cinder [req-58d21c56-b546-4be8-b8ba-66a03b65513d - - - - -] Unhandled error: TypeError: Message objects do not support addition.10:08
evrardjpI can dig deeper10:09
*** wanghao has joined #openstack-cinder10:11
*** kwathore has joined #openstack-cinder10:11
evrardjpseems to work better if I comment out the prettytable code included here: https://github.com/openstack/cinder/commit/939fa2c0ff6527258a9b4e17be8f0f5a765eefce#diff-fcaa8b96216aed2203b811ac2e7cb85510:11
*** wanghao has quit IRC10:16
*** wanghao has joined #openstack-cinder10:16
*** sdague has joined #openstack-cinder10:17
*** wanghao has quit IRC10:18
*** wanghao has joined #openstack-cinder10:19
*** wanghao has quit IRC10:19
*** psachin has quit IRC10:31
*** openstackgerrit has quit IRC10:33
*** mvk has joined #openstack-cinder10:33
*** belmoreira has quit IRC10:37
*** lpetrut has joined #openstack-cinder10:39
*** armaan has quit IRC10:43
*** yangyapeng has quit IRC10:44
*** armaan has joined #openstack-cinder10:44
*** openstackgerrit has joined #openstack-cinder10:45
openstackgerritJean-Philippe Evrard proposed openstack/cinder master: Make stringify header work under python2.  https://review.openstack.org/51225910:45
openstackgerritJean-Philippe Evrard proposed openstack/cinder master: Make stringify header work under python2.  https://review.openstack.org/51225910:46
evrardjpI'd be happy if someone can triage the validity of this bug: https://bugs.launchpad.net/cinder/+bug/1723922 and its included.10:47
openstackLaunchpad bug 1723922 in Cinder "cinder db online_db_migration crashes" [Undecided,In progress] - Assigned to Jean-Philippe Evrard (jean-philippe-evrard)10:47
*** AlexeyAbashkin has quit IRC11:04
*** edmondsw has joined #openstack-cinder11:05
*** wanghao has joined #openstack-cinder11:08
*** dave-mccowan has joined #openstack-cinder11:09
*** edmondsw has quit IRC11:09
*** armaan has quit IRC11:11
*** wanghao has quit IRC11:11
*** armaan has joined #openstack-cinder11:11
*** ganso has joined #openstack-cinder11:11
*** wanghao has joined #openstack-cinder11:12
*** armaan has quit IRC11:12
*** armaan has joined #openstack-cinder11:13
*** wanghao has quit IRC11:16
*** wanghao has joined #openstack-cinder11:16
*** masber has quit IRC11:16
*** AlexeyAbashkin has joined #openstack-cinder11:20
*** wanghao has quit IRC11:20
*** ociuhandu has quit IRC11:22
*** wanghao has joined #openstack-cinder11:27
*** lkuchlan has joined #openstack-cinder11:27
*** Swimingly has quit IRC11:30
*** wanghao has quit IRC11:31
*** wanghao has joined #openstack-cinder11:31
*** tpsilva has joined #openstack-cinder11:33
*** sticker has quit IRC11:34
*** wanghao has quit IRC11:35
*** nicolasbock has joined #openstack-cinder11:36
*** wanghao has joined #openstack-cinder11:36
*** Swimingly has joined #openstack-cinder11:39
*** Swimingly has joined #openstack-cinder11:39
*** wanghao has quit IRC11:39
*** wanghao has joined #openstack-cinder11:40
*** wanghao has quit IRC11:45
*** wanghao has joined #openstack-cinder11:46
*** dustins has joined #openstack-cinder11:47
*** wanghao has quit IRC11:47
*** ociuhandu has joined #openstack-cinder11:49
*** yangyapeng has joined #openstack-cinder11:50
*** abishop has joined #openstack-cinder11:59
*** arnewiebalck_ has joined #openstack-cinder12:00
*** lkuchlan has quit IRC12:01
*** gcb has quit IRC12:01
*** edmondsw has joined #openstack-cinder12:09
openstackgerritzhangbailin proposed openstack/cinder master: Replace http with https for doc links in cinder  https://review.openstack.org/50024512:09
*** chlong has quit IRC12:10
*** stakeda has quit IRC12:10
*** wes_dillingham has joined #openstack-cinder12:13
*** jmlowe has quit IRC12:17
openstackgerritTommyLike proposed openstack/cinder master: Schedule the request to scheduler when creating from snapshot  https://review.openstack.org/51227612:22
*** dave-mccowan has quit IRC12:23
tommylikehujungleboyj, smcginnis  could you take a look at this patch? https://review.openstack.org/#/c/510722/  Thanks in advance !12:27
*** kaisers has quit IRC12:27
*** kaisers has joined #openstack-cinder12:28
openstackgerritzhangbailin proposed openstack/cinder master: Replace http with https for doc links in cinder  https://review.openstack.org/50024512:31
*** kaisers has quit IRC12:31
*** kaisers has joined #openstack-cinder12:31
tommylikehupatrickeast:  need your eye on this patch:) https://review.openstack.org/#/c/512208/12:34
tommylikehuwinston-d, wxy_:  it would be great if you can take a look at this one: https://review.openstack.org/#/c/512276/ , thanks!12:37
*** gkadam has quit IRC12:38
*** felipemonteiro has joined #openstack-cinder12:39
*** kaisers has quit IRC12:41
*** felipemonteiro_ has joined #openstack-cinder12:41
*** kaisers has joined #openstack-cinder12:42
*** felipemonteiro has quit IRC12:45
*** dave-mccowan has joined #openstack-cinder12:45
*** kaisers has quit IRC12:46
*** dalgaaf has quit IRC12:46
*** dalgaaf has joined #openstack-cinder12:47
evrardjpjgriffith: FYI here is the fix for i18n that you have workarouned in your patchset https://review.openstack.org/#/c/510201/4 : https://review.openstack.org/#/c/512259/12:53
evrardjp"fix"12:53
evrardjpat least you have confirmed I am not the only one, thank god.12:54
*** liverpooler has joined #openstack-cinder12:54
*** armaan has quit IRC12:54
*** armaan has joined #openstack-cinder12:55
*** mriedem has joined #openstack-cinder13:02
*** dave-mccowan has quit IRC13:06
*** kaisers has joined #openstack-cinder13:11
*** links has quit IRC13:12
*** wanghao has joined #openstack-cinder13:13
*** kaisers has quit IRC13:15
*** dave-mccowan has joined #openstack-cinder13:16
*** lhx__ has quit IRC13:16
*** lkuchlan has joined #openstack-cinder13:17
*** armaan has quit IRC13:19
*** armaan has joined #openstack-cinder13:20
*** gcb has joined #openstack-cinder13:25
*** kaisers has joined #openstack-cinder13:28
*** erlon has joined #openstack-cinder13:30
*** lbragstad has joined #openstack-cinder13:37
*** mriedem1 has joined #openstack-cinder13:43
*** crushil has quit IRC13:43
*** lhx_ has joined #openstack-cinder13:44
*** wanghao_ has joined #openstack-cinder13:44
*** mriedem has quit IRC13:48
*** wanghao has quit IRC13:48
*** slunkad has quit IRC13:48
*** slunkad has joined #openstack-cinder13:48
*** mriedem1 is now known as mriedem13:49
*** exarr has joined #openstack-cinder13:50
exarrHi all. Ocata release (Ubuntu). The install docs here: https://docs.openstack.org/ocata/install-guide-ubuntu/cinder-controller-install.html refer to elements within the package config13:51
exarrExcept they aren't present.13:51
exarr... just saying.13:51
*** gnufied has quit IRC13:51
*** psachin has joined #openstack-cinder13:53
smcginnisexarr: Which part?13:53
*** gouthamr has joined #openstack-cinder13:54
*** chlong has joined #openstack-cinder13:54
*** armax has joined #openstack-cinder13:56
exarrsmcginnis: /etc/cinder/cinder.conf13:56
exarr""in the [database] section, configure database access:""13:56
exarrThere is only the DEFAULT section in the respective config file. No commented out entries or anything.13:57
smcginnisexarr: Hmm, they must have changed the ubuntu package default cinder.conf. Last time I went throug there, it did contain all sections with commented out defaults.13:57
mriedemare there any known bugs in pike for os-brick where volume_use_multipath isn't getting passed through properly to brick for iscsci or fc connections? https://bugs.launchpad.net/os-brick/+bug/172392813:58
openstackLaunchpad bug 1723928 in OpenStack Compute (nova) "In case of volume_use_multipath=True, Nova unable to fetch CONF.libvirt.volume_use_multipath value from nova.conf" [Undecided,Incomplete]13:58
smcginnismriedem: Saw that - none that I am aware of.13:58
smcginnisFirst I heard of this issue.13:58
smcginnisI should state that more clearly - this bug was the first I had heard of multipath problems.13:59
*** crushil has joined #openstack-cinder14:01
*** gnufied has joined #openstack-cinder14:02
*** kaisers_ has joined #openstack-cinder14:04
*** swright has joined #openstack-cinder14:04
*** mchlumsky has joined #openstack-cinder14:05
*** kaisers has quit IRC14:07
*** arnewiebalck_ has quit IRC14:09
*** masber has joined #openstack-cinder14:12
*** dhinesh has joined #openstack-cinder14:13
*** arnewiebalck_ has joined #openstack-cinder14:13
*** david-lyle has joined #openstack-cinder14:14
*** dhinesh has quit IRC14:17
*** masber has quit IRC14:17
*** kaisers_ has quit IRC14:21
exarrsmcginnis: Sure. No biggie. Just mentioning it as it's different from the usual convention.14:25
*** geguileo_ is now known as geguileo14:28
*** david-lyle has quit IRC14:29
*** david-lyle has joined #openstack-cinder14:30
evrardjpjungleboyj: thanks for the recheck, but I expect we have more work to do for openstack-tox-py35 job since zuul v3?14:42
evrardjpor am I wrong?14:42
*** lkuchlan has quit IRC14:44
*** kaisers has joined #openstack-cinder14:44
*** gcb has quit IRC14:54
*** sdague has quit IRC15:07
*** sdague has joined #openstack-cinder15:07
jungleboyjevrardjp:  Well, I thought that the issues were all worked out there.  Figured I would start with a recheck and then go from there.15:08
*** arnewiebalck_ has quit IRC15:08
evrardjp:)15:10
*** dave-mccowan has quit IRC15:13
*** armax has quit IRC15:14
*** xyang1 has joined #openstack-cinder15:14
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix backup compression unit tests  https://review.openstack.org/50750915:16
openstackgerritGorka Eguileor proposed openstack/cinder master: Run backup compression on native thread  https://review.openstack.org/50751015:16
*** e0ne has joined #openstack-cinder15:17
*** catintheroof has joined #openstack-cinder15:30
*** sbezverk has quit IRC15:31
*** kaisers has quit IRC15:35
*** catintheroof has quit IRC15:38
*** catintheroof has joined #openstack-cinder15:38
openstackgerritEric Young proposed openstack/cinder master: ScaleIO Driver: Backup volume via snapshot  https://review.openstack.org/51192115:40
*** e0ne_ has joined #openstack-cinder15:42
*** e0ne_ has quit IRC15:44
*** e0ne has quit IRC15:45
*** markvoelker has quit IRC15:47
*** wanghao_ has quit IRC15:47
*** wanghao has joined #openstack-cinder15:47
*** wanghao has quit IRC15:48
*** wanghao has joined #openstack-cinder15:48
*** wanghao has quit IRC15:49
*** wanghao has joined #openstack-cinder15:49
openstackgerritMerged openstack/cinder master: Remove deprecated nova config options  https://review.openstack.org/50359615:49
*** wanghao has quit IRC15:50
*** wanghao has joined #openstack-cinder15:50
*** wanghao has quit IRC15:50
*** wanghao has joined #openstack-cinder15:51
*** e0ne has joined #openstack-cinder15:51
*** wanghao has quit IRC15:51
*** wanghao has joined #openstack-cinder15:52
*** jccates has joined #openstack-cinder15:55
*** wanghao has quit IRC15:56
*** swright has quit IRC16:00
*** jccates has quit IRC16:02
*** dave-mccowan has joined #openstack-cinder16:03
*** jccates has joined #openstack-cinder16:04
*** dhinesh has joined #openstack-cinder16:14
*** mvk has quit IRC16:14
*** gkadam has joined #openstack-cinder16:14
*** dustins has quit IRC16:15
*** armax has joined #openstack-cinder16:16
*** dhinesh has quit IRC16:18
*** markvoelker has joined #openstack-cinder16:20
*** kaisers has joined #openstack-cinder16:21
*** e0ne has quit IRC16:23
*** ntpttr_laptop has joined #openstack-cinder16:23
*** e0ne has joined #openstack-cinder16:24
*** e0ne has quit IRC16:24
*** ntpttr_laptop has quit IRC16:24
*** e0ne has joined #openstack-cinder16:25
*** kaisers has quit IRC16:26
*** e0ne has quit IRC16:26
*** e0ne has joined #openstack-cinder16:27
*** e0ne has quit IRC16:27
*** e0ne has joined #openstack-cinder16:28
*** ociuhandu has quit IRC16:28
*** e0ne has quit IRC16:29
*** e0ne has joined #openstack-cinder16:29
*** chlong has quit IRC16:29
*** e0ne has quit IRC16:29
*** sbezverk has joined #openstack-cinder16:32
*** tesseract has quit IRC16:35
*** AlexeyAbashkin has quit IRC16:36
*** lkuchlan has joined #openstack-cinder16:47
*** links has joined #openstack-cinder16:51
*** mvk has joined #openstack-cinder16:54
openstackgerritMerged openstack/cinder master: VMAX driver - documentation updates around versions  https://review.openstack.org/51088816:55
*** lpetrut has quit IRC16:59
hemna_mriedem, first I've heard of that17:04
hemna_I might be able to find time to look into that one17:05
*** chlong has joined #openstack-cinder17:11
hemna_fwiw, you can get multipath=False coming back from os-brick if multipath daemon isn't running.17:18
*** Apoorva has joined #openstack-cinder17:20
patrickeasthemna_: yea i was just looking at that, maybe our test of whether its running isn't working?17:20
hemna_the defect doesn't state that the daemon is running or not17:21
hemna_so it's hard to tell what his env is like.17:21
hemna_I just commented on the bug and marked it incomplete.17:21
hemna_the detection could be broken, but that code has been in place for a long while17:22
hemna_https://github.com/openstack/os-brick/blob/master/os_brick/initiator/linuxscsi.py#L154-L16717:23
patrickeastyea, would be weird17:23
hemna_anyway, I bet it's just not installed or running on his env.17:23
patrickeastif it was broken it would be something with the multipathd package on the system, or maybe like something with the rootwrap/exe stuff we've been shifting around the last few releases17:23
openstackgerritMonicaJoshi proposed openstack/cinder master: Message change for Policy error message change when user is not-authorized  https://review.openstack.org/51119517:25
*** dustins has joined #openstack-cinder17:26
*** cali_boxer has quit IRC17:26
*** cali_boxer has joined #openstack-cinder17:28
*** Taseer has joined #openstack-cinder17:31
TaseerHi all, how do I set the QoS Spec's consumer to 'front-end', 'back-end', or 'both', via the Python API ?17:33
*** sgu has joined #openstack-cinder17:36
sguhi17:37
sguwanted to know if infra issue is fixed?17:37
hemna_patrickeast, yah I would think so.  we'll see how he replies to the bug17:38
hemna_looks like DuncanT might be in a hurricane in Gallway ?17:39
jungleboyjsgu:  Infra issue?17:40
*** links has quit IRC17:42
*** dhinesh has joined #openstack-cinder17:42
*** dave-mccowan has quit IRC17:42
jungleboyjTaseer:  What do you mean?17:45
sgujungleboyj: yes during gerrit CI runs17:45
sgujungleboyj: last ran it on oct 11 and jenkins passed but zuul failed17:46
jungleboyjsgu:  So, they have switched to Zuulv3 and to me it looks like .... oh.17:46
sgujungleboyj: cannot figure out anything from errors what exactly caused it17:46
jungleboyjsgu:  Which patch?17:46
sgujungleboyj: Ib701edb733567831ee80a317d0e3b3945a31276017:47
Taseerjungleboyj: I mean how to do from CLI ?17:47
jungleboyjsgu:  At the time that ran it only needed to pass Jenkins.  So you are good there.  If you want to remove the -1 from Zuul you can recheck it and that should clear things up as Zuul should be working properly now.17:48
Taseerenforcing on hypervisor level and backend level17:48
jungleboyjTaseer:  I am not an expert on the QoS aspect of things but as I understand it you associate a QoS spec with your volume type.17:49
*** cali_boxer has quit IRC17:49
jungleboyjWhether the QoS is enforced by Nova or the Backend depends on what specs you set and what is supported by your backend.17:49
Taseerjungleboyj: okay, thanks.17:50
jungleboyjTaseer:  So, by default the QoS specs are enforced by Nova.  If you backend supports backend QoS then how to make that happen should be documented for your backend.17:51
*** cali_boxer has joined #openstack-cinder17:51
jungleboyjTrying to think who has done more with QoS.17:51
sgujungleboyj: thank you17:51
jungleboyjTaseer:  The guys who might know more aren't here right now.  e0ne or eharney have played around with QoS more if you  can't google your way out of things.17:52
jungleboyjsgu:  Welcome.17:52
*** wanghao has joined #openstack-cinder17:52
Taseerjungleboyj: thank you for the help. I'll try and dig in deeper myself.17:52
jungleboyjTaseer:  Sure.  Let me know what you find.  :-)17:52
Taseerjungleboyj: I definitely will.17:53
jungleboyjsgu:  I have been meaning to do a review of your driver.  I will try to do that in the next day or so.17:54
jungleboyjsgu How is the CI coming along for that?17:54
*** wanghao has quit IRC17:57
*** AlexeyAbashkin has joined #openstack-cinder17:59
*** tonygunk has joined #openstack-cinder18:00
*** dave-mccowan has joined #openstack-cinder18:00
*** psachin has quit IRC18:00
*** crushil has quit IRC18:00
*** AlexeyAbashkin has quit IRC18:03
*** ChubYann has joined #openstack-cinder18:04
*** sgu has quit IRC18:05
*** catintheroof has quit IRC18:07
*** catintheroof has joined #openstack-cinder18:08
*** catinthe_ has joined #openstack-cinder18:16
*** markvoelker has quit IRC18:16
*** markvoelker has joined #openstack-cinder18:17
*** kaisers has joined #openstack-cinder18:17
*** catintheroof has quit IRC18:18
*** markvoelker has quit IRC18:21
*** catinthe_ has quit IRC18:23
*** shaner has quit IRC18:24
*** shaner_ has joined #openstack-cinder18:24
*** lpetrut has joined #openstack-cinder18:25
*** jmlowe has joined #openstack-cinder18:32
*** Aditya has joined #openstack-cinder18:38
*** ociuhandu has joined #openstack-cinder18:40
*** shaner_ has quit IRC18:43
*** shaner has joined #openstack-cinder18:44
*** Aditya has quit IRC18:46
*** sgu has joined #openstack-cinder18:48
sgujungleboyj: It is in final stages and will be done by end of this week.18:49
jungleboyjsgu:  Ok, good news.18:49
sgujungleboyj: also how to issue recheck on just zuul tests18:49
jungleboyjsgu: Don't think you can do just zuul.  Have to do the broad recheck.18:50
sgujungleboyj: ok thank you18:51
jungleboyjWelcome.  :-)18:51
*** lhx_ has quit IRC18:52
*** mmethot has joined #openstack-cinder18:52
*** jmlowe has quit IRC18:53
sgujungleboyj: I am sorry but I think I mistakenly did recheck on one of the comments. Would that trigger another build or I should do recheck on patchset?18:55
jungleboyjI think what you did is fine.18:56
jungleboyjsgu ... You just submitted a reply with 'recheck' right?18:57
sguyes18:57
jungleboyjOk, then in a little while you should see a job for your patch number pop up here:  http://zuulv3.openstack.org/18:57
sgujungleboyj: ok18:58
*** lkuchlan has quit IRC19:03
*** edmondsw has quit IRC19:04
*** edmondsw has joined #openstack-cinder19:04
*** lkuchlan has joined #openstack-cinder19:08
*** edmondsw has quit IRC19:09
*** lkuchlan has quit IRC19:14
openstackgerritMonicaJoshi proposed openstack/cinder master: DS8000 ConsistencyGroup failed to update group with volumes  https://review.openstack.org/51121219:15
sgujungleboyj: it didn't start19:16
jgriffithwell I'm about ready to give up on these unit tests :(19:20
*** hemna_ has quit IRC19:20
jgriffithjust wrap the save call in a try/except block and forget it19:21
jgriffithand I don't know what magic nonsense was used for things like the v3 api unit tests but it's evil19:21
*** mordred has quit IRC19:31
smcginnissgu: The recheck comment can't have any text before "recheck"19:32
*** datasundae has joined #openstack-cinder19:33
jungleboyjsmcginnis:  I wondered if that had happened.19:33
datasundaeOK19:34
*** hemna_ has joined #openstack-cinder19:34
jungleboyjdatasundae:  Welcome!19:34
ildikovjgriffith: which way did you go with those tests?19:34
datasundae /message jungleboyj19:34
ildikovjgriffith: I mean with mocking19:34
jgriffithildikov: I have been trying to "do the right thing"19:34
jgriffithand go through the existing tests and mock the service_update call where it's being indirectly called19:35
jgriffithI had some success doing class-wide mock patches in setUp for things like the volume and scheduler tests19:35
jgriffithbut then I got to api/v319:35
jgriffithand there's all sorts of inheritance things linking forward and backwards (and sideways) that makes that approach fall apart a bit19:36
smcginnisjgriffith: Doesn't it just work to mock class-wide for the v2 tests that get inherited by the v3 ones?19:37
jgriffithmaybe I'll go through each test individually19:37
ildikovjgriffith: what if you mock save itself for the API tests?19:37
jgriffithsmcginnis: I thought it would19:37
jgriffithsmcginnis: I added the mock to the setUp in the V2 class but it didn't work19:37
jgriffithsmcginnis: so I also added it to the V3 class as well... but then the contrib versions failed19:38
jgriffithand some other things blew up19:38
jgriffithand I got annoyed19:38
smcginnisAh, I bet the v3 ones would have to call super(v3test, self).setUp().19:38
ildikovjgriffith: is that uploaded to Gerrit or all local at this point?19:39
smcginnisAnnoyance is understood and empathised. ;)19:39
jgriffithildikov: the mocks?  No, not in gerrit because frankly it's hideous anyway19:40
jgriffitheven if it all worked, it's terrible19:40
ildikovjgriffith: just wanted to take a look how you do the mocking in those API tests19:40
*** crushil has joined #openstack-cinder19:40
ildikovYeah, I get the disappointment, certainly sounds to be very far from rainbows and unicorns...19:42
*** mordred has joined #openstack-cinder19:42
ildikovjgriffith: is there an alternative?19:43
jgriffithildikov: there's lots of alternatives :)19:43
jgriffithtake a vacation, play bingo, nap....19:43
jgriffithall sorts of things19:43
*** sgu has quit IRC19:43
ildikovjgriffith: I knew I should've specified the question more, lol19:44
jgriffithso we could allow the column to be nullable in the service object, but that's not *right*19:44
jgriffithand honestly at some point we're going to need to address this19:44
jgriffithI guess I'll just go through tests one by one19:44
jgriffithor look at other options19:45
ildikovThe API ones you mean?19:45
jgriffithjust in general19:45
jgriffithbut i had kinda hoped that maybe folks smarter than me would take a look at this and see if they had ideas/suggestions19:45
ildikovI would say find a way to get that patch working and refactor from there19:45
jgriffithLOL... I don't even know what that means?19:46
ildikovWhich obviously leads to the fact that we will forget about refactoring19:46
ildikovWhich part did sound so unfamiliar? :)19:47
jgriffith"get that patch workign and refactor from there"19:47
jgriffithgetting the patch working means getting the unit tests working as the functional code works19:47
jgriffithunless you mean put a try/except block around the save call and hope for the best19:47
jgriffithbut that's kinda bad19:48
ildikovI know and you said that it looks terrible even where it works19:48
jgriffithwell, I guess it won't look any more terrible than what we already have :)19:48
ildikovNah, I meant to polish how the tests are running and using the mocks19:48
jgriffithwhat's one more mock.patch decorator :)19:48
ildikovI'm fine with that :)19:49
jgriffithalso FWIW a side effect of the class wide mock approach is then you have tests that are actually calling update and fail because of the mock :)19:49
ildikovUsed a lot of those in Nova :)19:49
jgriffithso I guess that just means going through each test one by one until all 198 of them are addressed... sigh19:50
ildikovjgriffith: why are the unit tests calling update?19:50
*** armax has quit IRC19:50
jgriffithildikov: because we do updates all over the place with objects19:50
jgriffiththat's how it works19:50
ildikovjgriffith: if you upload the current version I can play with it a bit this afternoon19:50
*** markvoelker has joined #openstack-cinder19:51
jgriffithit's uploaded, feel free :)19:51
*** markvoelker has quit IRC19:51
*** markvoelker has joined #openstack-cinder19:51
jgriffithI even put a comment in the patch pointing out exactly where the problem is19:51
*** wes_dillingham has quit IRC19:51
ildikovjgriffith: ok, will work from what's up there now19:51
ildikovjgriffith: I know, I think I reflected on that over the weekend :)19:52
*** wanghao has joined #openstack-cinder19:53
jgriffithI suppose I'm just being lazy and whiney about it.  I would like input from folks if there's a better way to deal with it though19:53
*** edmondsw has joined #openstack-cinder19:54
jgriffithif the answer is to just add a mock decorator in each test that needs it that's fine19:54
* jungleboyj wishes jgriffith Hadn't mentioned a nap. That sounds so good.19:54
jgriffithjust feels like there should be another way to go about it19:54
jgriffithjungleboyj: :)19:54
jungleboyjAnd my bed is so close.19:54
jgriffithjungleboyj: naps are *ALWAYS* good, but I never seem to actually take them19:54
jungleboyjjgriffith:  I love them when I take them but feel so guilty afterwards.19:55
jgriffithjungleboyj: pishhh... reall power naps are done in your chair at your desk19:55
*** shaner has quit IRC19:55
jgriffithThat's what those airplane pillow collars are for, because you know I wouldn't be caught dead on an airplane in public with one :)19:55
jungleboyjhttp://gph.is/29qftCt19:55
jgriffithjungleboyj: hehe... been there19:56
*** markvoelker_ has joined #openstack-cinder19:57
*** wanghao has quit IRC19:57
*** shaner has joined #openstack-cinder19:59
ildikovI wish I could do a power nap...20:00
*** markvoelker has quit IRC20:01
*** dustins has quit IRC20:09
*** diablo_rojo has quit IRC20:10
openstackgerritMarc proposed openstack/cinder master: Allowed to delete up to same day marked deleted records. Removed the now non-needed unit test, for non-zero days value  https://review.openstack.org/51211120:14
*** salv-orlando has joined #openstack-cinder20:16
openstackgerritMarc proposed openstack/cinder master: Allowed to delete up to same day marked deleted records  https://review.openstack.org/51211120:17
jgriffithildikov: smcginnis BTW the other option on those patches was to figure out how to setup the fake service or fake db in all of the tests to have the db.service.uuid columng and have it populated20:21
jgriffithbut it looks to me like most of that is all mocked on a case per case basis so it wasn't any better/easier to do20:21
jgriffiththe point being is that if the db service entry had the uuid column and it was set, then the object code would not attempt to do the save/update on it20:22
*** armax has joined #openstack-cinder20:24
smcginnisSeems like it would be worth doing (at some point) to fake out those things for all unit tests.20:24
smcginnisSince a unit test shouldn't really be interacting with a database.20:24
ildikovjgriffith: do you mean the fake service is not always used? Or that update is mocked on a per case basis anyway?20:24
ildikovsmcginnis: don't get jgriffith started on that :)20:24
*** bjolo has joined #openstack-cinder20:25
*** diablo_rojo has joined #openstack-cinder20:25
jgriffithildikov: what I mean is there's no global fake service for fake-db being used in all places20:28
jgriffiththere are just random mock.patch adds in places to make tests work.  That's "ok", except the thing we did with objects made it 'seemless'20:28
ildikovjgriffith: ah ok, got it20:28
jgriffithwhich is great!!  Until you try and change something in the fundamental object code :)20:29
*** tpsilva has quit IRC20:43
*** tonygunk has quit IRC20:44
*** edmondsw has quit IRC20:56
*** cali_boxer has quit IRC20:58
*** cali_boxer has joined #openstack-cinder20:58
*** kfarr has joined #openstack-cinder21:08
openstackgerritJohn Griffith proposed openstack/cinder master: Fix migration 112 to use live_data_migration API  https://review.openstack.org/51020121:15
jgriffithildikov: so if you're curious ^^21:16
jgriffithsmcginnis: you might be too21:16
jgriffithso the interesting one in there is the api test_volume_metadata21:16
jgriffithI have the class setUp adding  a mock.patch, but it's not picked up; and some of those V3 micro tests are only implemented in V2, I also put the mock there but that didn't work either.  At some point somebody has to be the base-class you would think :)21:17
*** abishop has quit IRC21:17
smcginnisWait, microversion tests implemented in v2 unit tests?21:18
jgriffithsmcginnis: that was my point :)21:19
smcginnisWell that doesn't seem right. :)21:19
smcginnisjgriffith: But you did those mocks in the v2 test setUp and the v3 inheriting ones failed?21:20
*** kfarr has quit IRC21:20
jgriffithunless it's mv 3.0 I guess?21:20
jgriffithI did21:20
jgriffithaww crap... hold on21:20
jgriffithlooking that patch doesn't have the v2 mock in it... let me make sure that doesn't work still21:20
jgriffithI was doing the binary test thing add/remove21:21
jgriffithmaybe I screwed someting up21:21
ildikovjgriffith: thank you, pretty much appreciated21:21
ildikovjgriffith: always happy to share some pain :)21:21
jgriffithsmcginnis: it's odd though I would've thought that having it in the V2 setUp would've covered v3, (at leat the v3 that inherits that class) but it didn't21:22
jgriffithildikov: well, I just figured I'm not good at articulating code sometimes so it's easier to just show it21:22
smcginnisjgriffith: Yeah, that would have been my expectation too.21:22
jgriffithsmcginnis: it's also tricky because there are two classes in the v3 volume_metadata test21:23
jgriffithand then throw in some ddt magic on top of it for good measure and I'm not necessarily sure exactly what's going on :)21:23
jgriffithbut I fully admit I'm not the sharpest tool in the shed with the unit test magic21:23
smcginnisMock works in mysterious ways too.21:23
jgriffithsmcginnis: +121:24
jgriffithit's a wonderful beast, until something goes awry :)21:24
jgriffithI'm about to yank the stupd zone-manager tests too btw21:24
smcginnisSomething wrong with them too?21:24
jgriffithor I guess spend the time to figure out why they're soooo sloowwwwwwwww21:24
smcginnisOh, right.21:25
smcginnisI could never really tell if it was them, or if it was something right after they run.21:25
smcginnisBut always seems to hang around there.21:25
smcginnisAnd awhile back when we were getting OOM errors and weird test run hangs, it always seemed to be there too.21:25
jgriffithsmcginnis: it's one of them, because I delete that dir and it's okie-dokie :)21:25
smcginnisHmm...21:26
jgriffithsmcginnis: and21:26
jgriffithcinder.tests.unit.zonemanager.test_brcd_fc_zone_driver.TestBrcdFcZoneDriver.test_add_connection_for_invalid_fabric                              129.27721:26
smcginnisYikes21:26
jgriffith129.277 seconds21:26
jgriffithcra-cra21:26
smcginnisWell that could explain why it hangs so long there.21:26
smcginnisMust be a sleep or something in there. Can't imagine it actually being busy for two minutes.21:26
jgriffithsmcginnis: so here's an example:   cinder.tests.unit.api.v3.test_volume_metadata.VolumeMetaDataTestNoMicroversion.test_update_metadata_item_keys_value_none21:27
jgriffithyeah, probably missing a patch sleep21:27
jgriffithI haven't looked21:27
jgriffithand the last thing I need is another distraction :)21:27
jgriffithsmcginnis: so taht test21:27
smcginnisRight!21:27
jgriffiththat21:27
jgriffithgrep on that21:27
*** lpetrut has quit IRC21:27
*** ganso has quit IRC21:28
jgriffithso what's weird too is that I have a mock.patch service_update in both the V2 setUp and both of the V3 setUp classes21:28
jgriffithbut it's still trying to pull from the db.  unless...21:29
*** crushil has quit IRC21:29
smcginnisDang you git blame!!21:29
jgriffithhaha21:29
jgriffithI can just explicitly mock that test, but I'd really like to understand *what* is going on here21:30
smcginnisjgriffith: It's odd. That's calling the super.setUp(), so I would expect to be able to mock that out in the v2 setUp and be fine.21:30
jgriffithand I'm also concerned that I'm making more of a mess than fixing anything21:31
jgriffithbecause the mocks in the various setUps fixes some tests but not others21:31
jgriffithso I've just got crap scattered all over the place21:31
jgriffithsmcginnis: yeah... maybe I'm just doing something incorrrectly21:31
jgriffithbut it seems it should be pretty straight forward21:32
smcginnisShould be.21:32
jgriffithit figures it would be volume_metadata that's such a mess :)21:35
smcginnisjgriffith: Haha, didn't think of that. How appropriate. :D21:35
jgriffithsmcginnis: I swear it's a curse that will haunt me forever!!!21:36
* smcginnis writes up storage-sig post on adding metadata21:36
* jgriffith throws his hands in the air and steps away from the computer21:36
* jungleboyj wants to cry while writing Jira stories.21:37
*** chlong has quit IRC21:45
jungleboyj@!h21:48
jungleboyjWait, where is pewp bot!?!21:48
jungleboyjhemna:  ^^^21:48
*** wanghao has joined #openstack-cinder21:54
*** xyang1 has quit IRC21:58
*** bjolo has quit IRC22:00
*** wanghao has quit IRC22:03
*** crushil has joined #openstack-cinder22:06
*** diablo_rojo has quit IRC22:09
*** salv-orlando has quit IRC22:14
*** mchlumsky has quit IRC22:19
*** edmondsw has joined #openstack-cinder22:25
*** masber has joined #openstack-cinder22:26
*** edmondsw has quit IRC22:29
*** felipemonteiro_ has quit IRC22:37
*** gouthamr has quit IRC22:37
*** cali_boxer has quit IRC22:43
*** wes_dillingham has joined #openstack-cinder22:43
hemna_oh noes22:48
tommylikehujungleboyj, smcginnis  : https://review.openstack.org/#/c/510722/ can you take a look? thanks!22:51
openstackgerritJohn Cates proposed openstack/cinder master: FlashSystem: Add CMMVC6045E CLI error for multi-host mapping  https://review.openstack.org/51164522:52
*** sdague has quit IRC22:56
*** lbragstad has quit IRC22:57
*** liverpooler has quit IRC22:57
*** catintheroof has joined #openstack-cinder22:58
*** gouthamr has joined #openstack-cinder22:58
*** jccates has quit IRC23:02
*** dhinesh has quit IRC23:05
*** ganso has joined #openstack-cinder23:13
*** salv-orlando has joined #openstack-cinder23:15
*** salv-orlando has quit IRC23:21
jgriffithsmcginnis: ok, got it down to 53; remaining are manual adds needed to response checks etc23:31
jgriffithshould be able to get this working tomorrow23:31
jgriffithsmcginnis: so the interesting thing about mocking those calls in the metadata tests is it needs to be done before calling super23:31
ildikovjgriffith: sounds good, thank you23:37
jgriffithildikov: good times :)23:37
ildikovjgriffith: I will try my best to fix the changes requested on the Nova side patches and I hope we can get the live_migrate bits and pieces merged this week23:37
ildikovjgriffith: fingers crossed... :)23:38
jgriffithildikov: I haven't looked at that yet23:38
ildikovjgriffith: I don't think you want to ;)23:38
ildikovjgriffith: I need to split out the refresh_conn_info changes to a separate patch and make the live_migrate one depending on it, the new attach one depends on the live migrate one already23:39
ildikovthe live_migrate patch also needs to be updated to have the attachment_update call covered in it elsewhere23:39
ildikovjgriffith: fun times is the right expression for that, right? :)23:40

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