*** felipemonteiro has quit IRC | 00:02 | |
*** felipemonteiro_ has quit IRC | 00:14 | |
*** sticker has joined #openstack-cinder | 00:18 | |
*** psachin has joined #openstack-cinder | 00:34 | |
*** wanghao has joined #openstack-cinder | 00:37 | |
*** wanghao has quit IRC | 00:44 | |
*** wanghao has joined #openstack-cinder | 00:44 | |
*** sdague has quit IRC | 00:50 | |
*** wanghao has quit IRC | 00:50 | |
*** jwcroppe has quit IRC | 00:51 | |
*** wanghao has joined #openstack-cinder | 00:51 | |
*** jwcroppe has joined #openstack-cinder | 00:52 | |
*** wanghao has quit IRC | 00:55 | |
*** wanghao has joined #openstack-cinder | 00:56 | |
*** jmlowe has quit IRC | 00:57 | |
*** armax has joined #openstack-cinder | 00:57 | |
*** wanghao has quit IRC | 00:58 | |
*** wanghao has joined #openstack-cinder | 00:59 | |
*** jmlowe has joined #openstack-cinder | 00:59 | |
*** wanghao has quit IRC | 01:03 | |
*** wanghao has joined #openstack-cinder | 01:04 | |
*** dalgaaf has quit IRC | 01:05 | |
*** wanghao has quit IRC | 01:08 | |
*** yangyapeng has joined #openstack-cinder | 01:12 | |
*** jmlowe has quit IRC | 01:12 | |
*** wanghao has joined #openstack-cinder | 01:17 | |
*** wanghao has quit IRC | 01:23 | |
*** wewe0901 has joined #openstack-cinder | 01:25 | |
*** wanghao has joined #openstack-cinder | 01:27 | |
*** Shunli has joined #openstack-cinder | 01:27 | |
*** wanghao has quit IRC | 01:29 | |
*** kaisers has joined #openstack-cinder | 01:30 | |
*** jmlowe has joined #openstack-cinder | 01:31 | |
*** kaisers_ has quit IRC | 01:32 | |
*** gkadam_ has quit IRC | 01:35 | |
*** dalgaaf has joined #openstack-cinder | 01:39 | |
*** yangyapeng has quit IRC | 01:39 | |
*** yangyapeng has joined #openstack-cinder | 01:41 | |
*** sbezverk has quit IRC | 01:43 | |
*** sbezverk has joined #openstack-cinder | 01:44 | |
*** dhinesh has joined #openstack-cinder | 01:45 | |
*** jmlowe has quit IRC | 01:45 | |
*** wanghao has joined #openstack-cinder | 01:56 | |
*** wanghao has quit IRC | 02:01 | |
*** lhx__ has joined #openstack-cinder | 02:11 | |
*** zhaochao has joined #openstack-cinder | 02:12 | |
*** masber has quit IRC | 02:26 | |
*** gouthamr has quit IRC | 02:27 | |
*** dalgaaf has quit IRC | 02:30 | |
*** dalgaaf has joined #openstack-cinder | 02:31 | |
*** dave-mccowan has quit IRC | 02:35 | |
*** felipemonteiro_ has joined #openstack-cinder | 02:38 | |
*** gcb has joined #openstack-cinder | 02:51 | |
*** crushil has joined #openstack-cinder | 03:01 | |
*** crushil has left #openstack-cinder | 03:04 | |
*** crushil_ has joined #openstack-cinder | 03:05 | |
*** jmlowe has joined #openstack-cinder | 03:06 | |
*** gkadam_ has joined #openstack-cinder | 03:10 | |
*** gkadam_ has quit IRC | 03:10 | |
*** gkadam has joined #openstack-cinder | 03:10 | |
*** masber has joined #openstack-cinder | 03:20 | |
*** felipemonteiro_ has quit IRC | 03:22 | |
*** chlong has joined #openstack-cinder | 03:29 | |
*** wewe0901 has quit IRC | 03:35 | |
*** edmondsw has joined #openstack-cinder | 03:53 | |
*** felipemonteiro_ has joined #openstack-cinder | 03:55 | |
*** edmondsw has quit IRC | 03:57 | |
*** dhinesh has quit IRC | 04:04 | |
*** hoonetorg has quit IRC | 04:06 | |
*** wes_dillingham has quit IRC | 04:09 | |
*** arnewiebalck_ has joined #openstack-cinder | 04:10 | |
*** dhinesh has joined #openstack-cinder | 04:10 | |
*** felipemonteiro__ has joined #openstack-cinder | 04:15 | |
*** dhinesh has quit IRC | 04:15 | |
openstackgerrit | Shunei Shiono proposed openstack/cinder master: NEC driver: Never use target info in migration_status https://review.openstack.org/511737 | 04:17 |
---|---|---|
*** felipemonteiro_ has quit IRC | 04:18 | |
*** hoonetorg has joined #openstack-cinder | 04:22 | |
*** felipemonteiro__ has quit IRC | 04:27 | |
*** Dinesh_Bhor has joined #openstack-cinder | 04:42 | |
*** lkuchlan has joined #openstack-cinder | 04:45 | |
*** armaan has joined #openstack-cinder | 04:50 | |
*** links has joined #openstack-cinder | 04:57 | |
*** markvoelker has quit IRC | 04:58 | |
*** markvoelker has joined #openstack-cinder | 04:58 | |
*** crushil_ is now known as crushil | 05:05 | |
*** armaan has quit IRC | 05:05 | |
*** armax has quit IRC | 05:07 | |
*** armax has joined #openstack-cinder | 05:07 | |
*** armax has quit IRC | 05:07 | |
*** armax has joined #openstack-cinder | 05:08 | |
*** armax has quit IRC | 05:08 | |
*** armax has joined #openstack-cinder | 05:09 | |
*** armax has quit IRC | 05:09 | |
*** armax has joined #openstack-cinder | 05:10 | |
*** armax has quit IRC | 05:10 | |
*** armax has joined #openstack-cinder | 05:10 | |
*** dhinesh has joined #openstack-cinder | 05:11 | |
*** armax has joined #openstack-cinder | 05:11 | |
*** armax has quit IRC | 05:11 | |
*** armax has joined #openstack-cinder | 05:12 | |
*** armax has quit IRC | 05:12 | |
*** arnewiebalck_ has quit IRC | 05:12 | |
*** armax has joined #openstack-cinder | 05:13 | |
*** armax has quit IRC | 05:13 | |
*** armax has joined #openstack-cinder | 05:14 | |
*** armax has quit IRC | 05:14 | |
*** armax has joined #openstack-cinder | 05:14 | |
*** armax has quit IRC | 05:15 | |
*** dhinesh has quit IRC | 05:15 | |
*** lkuchlan has quit IRC | 05:19 | |
*** lpetrut_ has joined #openstack-cinder | 05:23 | |
*** lhx__ has quit IRC | 05:33 | |
*** lhx__ has joined #openstack-cinder | 05:34 | |
*** kaisers1 has joined #openstack-cinder | 05:36 | |
*** wanghao has joined #openstack-cinder | 05:37 | |
*** zhaochao has quit IRC | 05:39 | |
*** wanghao has quit IRC | 05:41 | |
*** edmondsw has joined #openstack-cinder | 05:41 | |
*** edmondsw has quit IRC | 05:45 | |
*** wanghao has joined #openstack-cinder | 05:46 | |
*** markvoelker has quit IRC | 05:46 | |
*** armaan has joined #openstack-cinder | 05:47 | |
*** wanghao has quit IRC | 05:50 | |
*** wanghao has joined #openstack-cinder | 05:50 | |
*** wanghao has quit IRC | 05:56 | |
*** ircuser-1 has joined #openstack-cinder | 05:57 | |
*** wanghao has joined #openstack-cinder | 05:58 | |
*** ChubYann has quit IRC | 06:00 | |
*** wanghao has quit IRC | 06:02 | |
*** wanghao has joined #openstack-cinder | 06:03 | |
*** lpetrut_ has quit IRC | 06:06 | |
*** lpetrut_ has joined #openstack-cinder | 06:06 | |
*** wanghao has quit IRC | 06:07 | |
*** wanghao has joined #openstack-cinder | 06:08 | |
evrardjp | hello. 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_491796 | 06:08 |
*** lhx__ has quit IRC | 06:09 | |
*** pgadiya has joined #openstack-cinder | 06:09 | |
*** zhaochao has joined #openstack-cinder | 06:09 | |
*** pgadiya has quit IRC | 06:09 | |
*** wanghao has quit IRC | 06:09 | |
*** wanghao has joined #openstack-cinder | 06:10 | |
*** dhinesh has joined #openstack-cinder | 06:12 | |
*** lpetrut_ has quit IRC | 06:13 | |
*** vivsoni has joined #openstack-cinder | 06:13 | |
*** wanghao has quit IRC | 06:14 | |
*** wanghao has joined #openstack-cinder | 06:15 | |
*** dhinesh has quit IRC | 06:16 | |
*** wanghao has quit IRC | 06:21 | |
*** wanghao has joined #openstack-cinder | 06:21 | |
*** lpetrut_ has joined #openstack-cinder | 06:22 | |
*** wanghao has quit IRC | 06:23 | |
*** wanghao has joined #openstack-cinder | 06:24 | |
*** AndyWojo has quit IRC | 06:24 | |
*** zeestrat has quit IRC | 06:24 | |
*** serverascode has quit IRC | 06:24 | |
*** ameade has quit IRC | 06:24 | |
*** cargonza has quit IRC | 06:24 | |
*** wolsen has quit IRC | 06:24 | |
*** zhongjun has quit IRC | 06:24 | |
*** sweston has quit IRC | 06:24 | |
*** aarefiev has quit IRC | 06:24 | |
*** wanghao has quit IRC | 06:24 | |
*** gcb has quit IRC | 06:24 | |
*** DuncanT has quit IRC | 06:25 | |
*** TheJulia has quit IRC | 06:25 | |
*** wanghao has joined #openstack-cinder | 06:25 | |
*** fmccrthy has quit IRC | 06:25 | |
*** fmccrthy has joined #openstack-cinder | 06:26 | |
*** wanghao has quit IRC | 06:26 | |
*** AndyWojo has joined #openstack-cinder | 06:26 | |
*** wolsen has joined #openstack-cinder | 06:26 | |
*** zeestrat has joined #openstack-cinder | 06:26 | |
*** cargonza has joined #openstack-cinder | 06:26 | |
*** sweston has joined #openstack-cinder | 06:26 | |
*** ameade has joined #openstack-cinder | 06:26 | |
*** ying_zuo has quit IRC | 06:26 | |
*** serverascode has joined #openstack-cinder | 06:27 | |
*** wanghao has joined #openstack-cinder | 06:27 | |
*** TheJulia has joined #openstack-cinder | 06:27 | |
*** aarefiev has joined #openstack-cinder | 06:27 | |
*** DuncanT has joined #openstack-cinder | 06:27 | |
*** wanghao has quit IRC | 06:27 | |
*** wN has quit IRC | 06:27 | |
*** wanghao has joined #openstack-cinder | 06:27 | |
*** openstackgerrit has quit IRC | 06:28 | |
*** gcb has joined #openstack-cinder | 06:28 | |
*** wN has joined #openstack-cinder | 06:29 | |
*** wN has quit IRC | 06:29 | |
*** wN has joined #openstack-cinder | 06:29 | |
*** wanghao has quit IRC | 06:29 | |
*** psachin has quit IRC | 06:30 | |
*** wanghao has joined #openstack-cinder | 06:30 | |
*** reedip has joined #openstack-cinder | 06:33 | |
*** lkuchlan has joined #openstack-cinder | 06:33 | |
reedip | Hi All, | 06:33 |
reedip | We 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 IRC | 06:35 | |
reedip | We checked and found that iscsi is no installed on Controller but installed in Compute. | 06:35 |
*** wanghao has joined #openstack-cinder | 06:35 | |
reedip | Can anyone give any pointers about how to resolve the issue on the compute nodes ? | 06:36 |
*** jwcroppe has quit IRC | 06:36 | |
*** jwcroppe has joined #openstack-cinder | 06:36 | |
*** bkopilov_ has joined #openstack-cinder | 06:37 | |
*** wanghao has quit IRC | 06:38 | |
*** wanghao has joined #openstack-cinder | 06:39 | |
*** AlexeyAbashkin has joined #openstack-cinder | 06:39 | |
*** yangyapeng has quit IRC | 06:42 | |
*** wanghao has quit IRC | 06:42 | |
*** openstackgerrit has joined #openstack-cinder | 06:42 | |
openstackgerrit | junbo.li proposed openstack/cinder master: Api-ref: Add missing api response for volume transfer https://review.openstack.org/504023 | 06:42 |
*** wanghao has joined #openstack-cinder | 06:42 | |
*** yangyapeng has joined #openstack-cinder | 06:42 | |
*** psachin has joined #openstack-cinder | 06:46 | |
*** wanghao has quit IRC | 06:46 | |
*** psachin has quit IRC | 06:46 | |
*** AlexeyAbashkin has quit IRC | 06:46 | |
*** psachin has joined #openstack-cinder | 06:46 | |
*** lhx__ has joined #openstack-cinder | 06:47 | |
*** wanghao has joined #openstack-cinder | 06:47 | |
*** wanghao has quit IRC | 06:52 | |
*** wanghao has joined #openstack-cinder | 06:53 | |
*** wanghao has quit IRC | 06:56 | |
*** wanghao has joined #openstack-cinder | 06:56 | |
*** s-shiono has joined #openstack-cinder | 06:57 | |
*** pgadiya has joined #openstack-cinder | 06:58 | |
*** wanghao has quit IRC | 06:59 | |
*** wanghao has joined #openstack-cinder | 06:59 | |
openstackgerrit | TommyLike proposed openstack/cinder master: Add policy documentation and sample file https://review.openstack.org/512187 | 07:01 |
*** markvoelker has joined #openstack-cinder | 07:02 | |
*** wanghao has quit IRC | 07:04 | |
*** wanghao has joined #openstack-cinder | 07:05 | |
*** lpetrut_ has quit IRC | 07:10 | |
*** wanghao has quit IRC | 07:11 | |
*** wanghao has joined #openstack-cinder | 07:12 | |
*** wanghao has quit IRC | 07:17 | |
*** tesseract has joined #openstack-cinder | 07:18 | |
*** wanghao has joined #openstack-cinder | 07:18 | |
*** markvoelker has quit IRC | 07:18 | |
*** markvoelker has joined #openstack-cinder | 07:20 | |
*** markvoelker has quit IRC | 07:20 | |
*** markvoelker has joined #openstack-cinder | 07:20 | |
*** wanghao has quit IRC | 07:20 | |
*** wanghao has joined #openstack-cinder | 07:21 | |
*** wanghao has quit IRC | 07:23 | |
*** wanghao has joined #openstack-cinder | 07:24 | |
*** bkopilov has quit IRC | 07:27 | |
*** bkopilov has joined #openstack-cinder | 07:27 | |
*** edmondsw has joined #openstack-cinder | 07:29 | |
*** wanghao has quit IRC | 07:29 | |
*** wanghao has joined #openstack-cinder | 07:30 | |
*** edmondsw has quit IRC | 07:33 | |
*** wanghao has quit IRC | 07:35 | |
*** wanghao has joined #openstack-cinder | 07:35 | |
*** AlexeyAbashkin has joined #openstack-cinder | 07:38 | |
*** wanghao has quit IRC | 07:41 | |
*** wanghao has joined #openstack-cinder | 07:42 | |
*** lpetrut_ has joined #openstack-cinder | 07:45 | |
*** wanghao has quit IRC | 07:45 | |
*** wanghao has joined #openstack-cinder | 07:45 | |
openstackgerrit | wang yong proposed openstack/cinder master: Inspur Cinder FC/iSCSI driver https://review.openstack.org/502455 | 07:47 |
openstackgerrit | TommyLike proposed openstack/cinder master: [policy in code] Add support for volume, volume type resources https://review.openstack.org/511427 | 07:48 |
*** wanghao has quit IRC | 07:48 | |
*** wanghao has joined #openstack-cinder | 07:51 | |
*** lpetrut_ has quit IRC | 07:53 | |
*** wanghao has quit IRC | 07:54 | |
*** wanghao has joined #openstack-cinder | 07:54 | |
*** wanghao has quit IRC | 07:58 | |
*** wanghao has joined #openstack-cinder | 08:04 | |
*** wanghao has quit IRC | 08:07 | |
openstackgerrit | TommyLike proposed openstack/cinder master: Fetch 'zoning_mode' from 'backend_defaults' section https://review.openstack.org/512208 | 08:07 |
*** armaan has quit IRC | 08:08 | |
*** armaan has joined #openstack-cinder | 08:08 | |
*** belmoreira has joined #openstack-cinder | 08:13 | |
*** slunkad has joined #openstack-cinder | 08:16 | |
*** wanghao has joined #openstack-cinder | 08:18 | |
*** wanghao has quit IRC | 08:21 | |
*** wanghao has joined #openstack-cinder | 08:22 | |
*** zhongjun has joined #openstack-cinder | 08:22 | |
*** wanghao has quit IRC | 08:23 | |
*** hoonetorg has quit IRC | 08:25 | |
*** belmoreira has quit IRC | 08:25 | |
*** ying_zuo has joined #openstack-cinder | 08:26 | |
*** s-shiono has quit IRC | 08:27 | |
*** armaan has quit IRC | 08:28 | |
*** armaan has joined #openstack-cinder | 08:28 | |
*** wanghao has joined #openstack-cinder | 08:29 | |
*** wanghao has quit IRC | 08:30 | |
*** markvoelker has quit IRC | 08:31 | |
*** wanghao has joined #openstack-cinder | 08:33 | |
openstackgerrit | Jack Lu proposed openstack/os-brick master: Update comments for _validate_iface_transport https://review.openstack.org/512212 | 08:33 |
*** wanghao has quit IRC | 08:35 | |
*** wanghao has joined #openstack-cinder | 08:35 | |
*** wanghao has quit IRC | 08:39 | |
*** wanghao has joined #openstack-cinder | 08:40 | |
*** pgadiya has quit IRC | 08:40 | |
*** wanghao has quit IRC | 08:45 | |
*** markvoelker has joined #openstack-cinder | 08:45 | |
*** wanghao has joined #openstack-cinder | 08:46 | |
*** wanghao has quit IRC | 08:51 | |
*** wanghao has joined #openstack-cinder | 08:52 | |
*** wanghao has quit IRC | 08:56 | |
*** wanghao has joined #openstack-cinder | 08:56 | |
openstackgerrit | Jack Lu proposed openstack/os-brick master: Update supported transports for iscsi connector https://review.openstack.org/510776 | 08:58 |
*** wanghao has quit IRC | 08:58 | |
*** wanghao has joined #openstack-cinder | 08:59 | |
*** wanghao has quit IRC | 09:01 | |
*** wanghao has joined #openstack-cinder | 09:02 | |
openstackgerrit | Jack Lu proposed openstack/os-brick master: Update supported transports for iscsi connector https://review.openstack.org/510776 | 09:02 |
openstackgerrit | Jack Lu proposed openstack/os-brick master: Update supported transports for iscsi connector https://review.openstack.org/510776 | 09:02 |
*** wanghao has quit IRC | 09:04 | |
*** wanghao has joined #openstack-cinder | 09:05 | |
openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Implement Tiramisu feature on VMAX https://review.openstack.org/505581 | 09:05 |
*** belmoreira has joined #openstack-cinder | 09:05 | |
*** belmoreira has quit IRC | 09:06 | |
*** wanghao has quit IRC | 09:07 | |
openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Remove workload for next gen arrays https://review.openstack.org/507070 | 09:07 |
*** wanghao has joined #openstack-cinder | 09:07 | |
*** e0ne has joined #openstack-cinder | 09:08 | |
openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Deprecate backend xml configuration https://review.openstack.org/508843 | 09:09 |
*** wanghao has quit IRC | 09:09 | |
*** wanghao has joined #openstack-cinder | 09:10 | |
openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Add async replication support https://review.openstack.org/509222 | 09:11 |
*** wanghao has quit IRC | 09:13 | |
*** belmoreira has joined #openstack-cinder | 09:13 | |
*** wanghao has joined #openstack-cinder | 09:14 | |
*** wanghao has quit IRC | 09:19 | |
*** wanghao has joined #openstack-cinder | 09:19 | |
*** wanghao has quit IRC | 09:22 | |
*** wanghao has joined #openstack-cinder | 09:23 | |
openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Add async replication support https://review.openstack.org/509222 | 09:25 |
openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Enable CHAP authentication for vmax backend https://review.openstack.org/510614 | 09:26 |
openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Cannot extend volumes in a replication relationship https://review.openstack.org/511812 | 09:27 |
*** wanghao has quit IRC | 09:27 | |
*** Shunli has quit IRC | 09:33 | |
*** wanghao has joined #openstack-cinder | 09:33 | |
openstackgerrit | Ciara Stacke proposed openstack/cinder master: VMAX driver - Cannot extend volumes in a replication relationship https://review.openstack.org/511812 | 09:33 |
*** wanghao has quit IRC | 09:39 | |
*** wanghao has joined #openstack-cinder | 09:40 | |
*** belmoreira has quit IRC | 09:43 | |
*** wanghao has quit IRC | 09:44 | |
*** wanghao has joined #openstack-cinder | 09:44 | |
*** e0ne has quit IRC | 09:45 | |
*** belmoreira has joined #openstack-cinder | 09:46 | |
*** wanghao has quit IRC | 09:46 | |
*** Guest50022 is now known as geguileo | 09:47 | |
*** geguileo is now known as Guest36540 | 09:48 | |
*** wanghao has joined #openstack-cinder | 09:52 | |
*** kwathore_ has quit IRC | 09:52 | |
*** AlexeyAbashkin has quit IRC | 09:53 | |
*** AlexeyAbashkin has joined #openstack-cinder | 09:53 | |
*** kwathore has joined #openstack-cinder | 09:54 | |
*** wanghao has quit IRC | 09:54 | |
*** mvk has quit IRC | 09:54 | |
*** wanghao has joined #openstack-cinder | 09:54 | |
*** wanghao has quit IRC | 09:57 | |
*** wanghao has joined #openstack-cinder | 09:58 | |
*** Guest36540 is now known as geguile- | 10:02 | |
*** wanghao has quit IRC | 10:02 | |
*** geguile- is now known as geguileo_ | 10:03 | |
*** wanghao has joined #openstack-cinder | 10:03 | |
*** kwathore has quit IRC | 10:06 | |
*** lkuchlan has quit IRC | 10:07 | |
evrardjp | so no one here got an issue recently with cinder-manage db online_data_migrations? | 10:08 |
evrardjp | my db sync seems to run fine | 10:08 |
*** wanghao has quit IRC | 10:08 | |
evrardjp | my online_data_migrations seem to have 0 migrations (new install), but miserably fails | 10:08 |
evrardjp | CRITICAL cinder [req-58d21c56-b546-4be8-b8ba-66a03b65513d - - - - -] Unhandled error: TypeError: Message objects do not support addition. | 10:08 |
evrardjp | I can dig deeper | 10:09 |
*** wanghao has joined #openstack-cinder | 10:11 | |
*** kwathore has joined #openstack-cinder | 10:11 | |
evrardjp | seems to work better if I comment out the prettytable code included here: https://github.com/openstack/cinder/commit/939fa2c0ff6527258a9b4e17be8f0f5a765eefce#diff-fcaa8b96216aed2203b811ac2e7cb855 | 10:11 |
*** wanghao has quit IRC | 10:16 | |
*** wanghao has joined #openstack-cinder | 10:16 | |
*** sdague has joined #openstack-cinder | 10:17 | |
*** wanghao has quit IRC | 10:18 | |
*** wanghao has joined #openstack-cinder | 10:19 | |
*** wanghao has quit IRC | 10:19 | |
*** psachin has quit IRC | 10:31 | |
*** openstackgerrit has quit IRC | 10:33 | |
*** mvk has joined #openstack-cinder | 10:33 | |
*** belmoreira has quit IRC | 10:37 | |
*** lpetrut has joined #openstack-cinder | 10:39 | |
*** armaan has quit IRC | 10:43 | |
*** yangyapeng has quit IRC | 10:44 | |
*** armaan has joined #openstack-cinder | 10:44 | |
*** openstackgerrit has joined #openstack-cinder | 10:45 | |
openstackgerrit | Jean-Philippe Evrard proposed openstack/cinder master: Make stringify header work under python2. https://review.openstack.org/512259 | 10:45 |
openstackgerrit | Jean-Philippe Evrard proposed openstack/cinder master: Make stringify header work under python2. https://review.openstack.org/512259 | 10:46 |
evrardjp | I'd be happy if someone can triage the validity of this bug: https://bugs.launchpad.net/cinder/+bug/1723922 and its included. | 10:47 |
openstack | Launchpad 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 IRC | 11:04 | |
*** edmondsw has joined #openstack-cinder | 11:05 | |
*** wanghao has joined #openstack-cinder | 11:08 | |
*** dave-mccowan has joined #openstack-cinder | 11:09 | |
*** edmondsw has quit IRC | 11:09 | |
*** armaan has quit IRC | 11:11 | |
*** wanghao has quit IRC | 11:11 | |
*** armaan has joined #openstack-cinder | 11:11 | |
*** ganso has joined #openstack-cinder | 11:11 | |
*** wanghao has joined #openstack-cinder | 11:12 | |
*** armaan has quit IRC | 11:12 | |
*** armaan has joined #openstack-cinder | 11:13 | |
*** wanghao has quit IRC | 11:16 | |
*** wanghao has joined #openstack-cinder | 11:16 | |
*** masber has quit IRC | 11:16 | |
*** AlexeyAbashkin has joined #openstack-cinder | 11:20 | |
*** wanghao has quit IRC | 11:20 | |
*** ociuhandu has quit IRC | 11:22 | |
*** wanghao has joined #openstack-cinder | 11:27 | |
*** lkuchlan has joined #openstack-cinder | 11:27 | |
*** Swimingly has quit IRC | 11:30 | |
*** wanghao has quit IRC | 11:31 | |
*** wanghao has joined #openstack-cinder | 11:31 | |
*** tpsilva has joined #openstack-cinder | 11:33 | |
*** sticker has quit IRC | 11:34 | |
*** wanghao has quit IRC | 11:35 | |
*** nicolasbock has joined #openstack-cinder | 11:36 | |
*** wanghao has joined #openstack-cinder | 11:36 | |
*** Swimingly has joined #openstack-cinder | 11:39 | |
*** Swimingly has joined #openstack-cinder | 11:39 | |
*** wanghao has quit IRC | 11:39 | |
*** wanghao has joined #openstack-cinder | 11:40 | |
*** wanghao has quit IRC | 11:45 | |
*** wanghao has joined #openstack-cinder | 11:46 | |
*** dustins has joined #openstack-cinder | 11:47 | |
*** wanghao has quit IRC | 11:47 | |
*** ociuhandu has joined #openstack-cinder | 11:49 | |
*** yangyapeng has joined #openstack-cinder | 11:50 | |
*** abishop has joined #openstack-cinder | 11:59 | |
*** arnewiebalck_ has joined #openstack-cinder | 12:00 | |
*** lkuchlan has quit IRC | 12:01 | |
*** gcb has quit IRC | 12:01 | |
*** edmondsw has joined #openstack-cinder | 12:09 | |
openstackgerrit | zhangbailin proposed openstack/cinder master: Replace http with https for doc links in cinder https://review.openstack.org/500245 | 12:09 |
*** chlong has quit IRC | 12:10 | |
*** stakeda has quit IRC | 12:10 | |
*** wes_dillingham has joined #openstack-cinder | 12:13 | |
*** jmlowe has quit IRC | 12:17 | |
openstackgerrit | TommyLike proposed openstack/cinder master: Schedule the request to scheduler when creating from snapshot https://review.openstack.org/512276 | 12:22 |
*** dave-mccowan has quit IRC | 12:23 | |
tommylikehu | jungleboyj, smcginnis could you take a look at this patch? https://review.openstack.org/#/c/510722/ Thanks in advance ! | 12:27 |
*** kaisers has quit IRC | 12:27 | |
*** kaisers has joined #openstack-cinder | 12:28 | |
openstackgerrit | zhangbailin proposed openstack/cinder master: Replace http with https for doc links in cinder https://review.openstack.org/500245 | 12:31 |
*** kaisers has quit IRC | 12:31 | |
*** kaisers has joined #openstack-cinder | 12:31 | |
tommylikehu | patrickeast: need your eye on this patch:) https://review.openstack.org/#/c/512208/ | 12:34 |
tommylikehu | winston-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 IRC | 12:38 | |
*** felipemonteiro has joined #openstack-cinder | 12:39 | |
*** kaisers has quit IRC | 12:41 | |
*** felipemonteiro_ has joined #openstack-cinder | 12:41 | |
*** kaisers has joined #openstack-cinder | 12:42 | |
*** felipemonteiro has quit IRC | 12:45 | |
*** dave-mccowan has joined #openstack-cinder | 12:45 | |
*** kaisers has quit IRC | 12:46 | |
*** dalgaaf has quit IRC | 12:46 | |
*** dalgaaf has joined #openstack-cinder | 12:47 | |
evrardjp | jgriffith: 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 |
evrardjp | at least you have confirmed I am not the only one, thank god. | 12:54 |
*** liverpooler has joined #openstack-cinder | 12:54 | |
*** armaan has quit IRC | 12:54 | |
*** armaan has joined #openstack-cinder | 12:55 | |
*** mriedem has joined #openstack-cinder | 13:02 | |
*** dave-mccowan has quit IRC | 13:06 | |
*** kaisers has joined #openstack-cinder | 13:11 | |
*** links has quit IRC | 13:12 | |
*** wanghao has joined #openstack-cinder | 13:13 | |
*** kaisers has quit IRC | 13:15 | |
*** dave-mccowan has joined #openstack-cinder | 13:16 | |
*** lhx__ has quit IRC | 13:16 | |
*** lkuchlan has joined #openstack-cinder | 13:17 | |
*** armaan has quit IRC | 13:19 | |
*** armaan has joined #openstack-cinder | 13:20 | |
*** gcb has joined #openstack-cinder | 13:25 | |
*** kaisers has joined #openstack-cinder | 13:28 | |
*** erlon has joined #openstack-cinder | 13:30 | |
*** lbragstad has joined #openstack-cinder | 13:37 | |
*** mriedem1 has joined #openstack-cinder | 13:43 | |
*** crushil has quit IRC | 13:43 | |
*** lhx_ has joined #openstack-cinder | 13:44 | |
*** wanghao_ has joined #openstack-cinder | 13:44 | |
*** mriedem has quit IRC | 13:48 | |
*** wanghao has quit IRC | 13:48 | |
*** slunkad has quit IRC | 13:48 | |
*** slunkad has joined #openstack-cinder | 13:48 | |
*** mriedem1 is now known as mriedem | 13:49 | |
*** exarr has joined #openstack-cinder | 13:50 | |
exarr | Hi 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 config | 13:51 |
exarr | Except they aren't present. | 13:51 |
exarr | ... just saying. | 13:51 |
*** gnufied has quit IRC | 13:51 | |
*** psachin has joined #openstack-cinder | 13:53 | |
smcginnis | exarr: Which part? | 13:53 |
*** gouthamr has joined #openstack-cinder | 13:54 | |
*** chlong has joined #openstack-cinder | 13:54 | |
*** armax has joined #openstack-cinder | 13:56 | |
exarr | smcginnis: /etc/cinder/cinder.conf | 13:56 |
exarr | ""in the [database] section, configure database access:"" | 13:56 |
exarr | There is only the DEFAULT section in the respective config file. No commented out entries or anything. | 13:57 |
smcginnis | exarr: 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 |
mriedem | are 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/1723928 | 13:58 |
openstack | Launchpad 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 |
smcginnis | mriedem: Saw that - none that I am aware of. | 13:58 |
smcginnis | First I heard of this issue. | 13:58 |
smcginnis | I should state that more clearly - this bug was the first I had heard of multipath problems. | 13:59 |
*** crushil has joined #openstack-cinder | 14:01 | |
*** gnufied has joined #openstack-cinder | 14:02 | |
*** kaisers_ has joined #openstack-cinder | 14:04 | |
*** swright has joined #openstack-cinder | 14:04 | |
*** mchlumsky has joined #openstack-cinder | 14:05 | |
*** kaisers has quit IRC | 14:07 | |
*** arnewiebalck_ has quit IRC | 14:09 | |
*** masber has joined #openstack-cinder | 14:12 | |
*** dhinesh has joined #openstack-cinder | 14:13 | |
*** arnewiebalck_ has joined #openstack-cinder | 14:13 | |
*** david-lyle has joined #openstack-cinder | 14:14 | |
*** dhinesh has quit IRC | 14:17 | |
*** masber has quit IRC | 14:17 | |
*** kaisers_ has quit IRC | 14:21 | |
exarr | smcginnis: Sure. No biggie. Just mentioning it as it's different from the usual convention. | 14:25 |
*** geguileo_ is now known as geguileo | 14:28 | |
*** david-lyle has quit IRC | 14:29 | |
*** david-lyle has joined #openstack-cinder | 14:30 | |
evrardjp | jungleboyj: thanks for the recheck, but I expect we have more work to do for openstack-tox-py35 job since zuul v3? | 14:42 |
evrardjp | or am I wrong? | 14:42 |
*** lkuchlan has quit IRC | 14:44 | |
*** kaisers has joined #openstack-cinder | 14:44 | |
*** gcb has quit IRC | 14:54 | |
*** sdague has quit IRC | 15:07 | |
*** sdague has joined #openstack-cinder | 15:07 | |
jungleboyj | evrardjp: 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 IRC | 15:08 | |
evrardjp | :) | 15:10 |
*** dave-mccowan has quit IRC | 15:13 | |
*** armax has quit IRC | 15:14 | |
*** xyang1 has joined #openstack-cinder | 15:14 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Fix backup compression unit tests https://review.openstack.org/507509 | 15:16 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Run backup compression on native thread https://review.openstack.org/507510 | 15:16 |
*** e0ne has joined #openstack-cinder | 15:17 | |
*** catintheroof has joined #openstack-cinder | 15:30 | |
*** sbezverk has quit IRC | 15:31 | |
*** kaisers has quit IRC | 15:35 | |
*** catintheroof has quit IRC | 15:38 | |
*** catintheroof has joined #openstack-cinder | 15:38 | |
openstackgerrit | Eric Young proposed openstack/cinder master: ScaleIO Driver: Backup volume via snapshot https://review.openstack.org/511921 | 15:40 |
*** e0ne_ has joined #openstack-cinder | 15:42 | |
*** e0ne_ has quit IRC | 15:44 | |
*** e0ne has quit IRC | 15:45 | |
*** markvoelker has quit IRC | 15:47 | |
*** wanghao_ has quit IRC | 15:47 | |
*** wanghao has joined #openstack-cinder | 15:47 | |
*** wanghao has quit IRC | 15:48 | |
*** wanghao has joined #openstack-cinder | 15:48 | |
*** wanghao has quit IRC | 15:49 | |
*** wanghao has joined #openstack-cinder | 15:49 | |
openstackgerrit | Merged openstack/cinder master: Remove deprecated nova config options https://review.openstack.org/503596 | 15:49 |
*** wanghao has quit IRC | 15:50 | |
*** wanghao has joined #openstack-cinder | 15:50 | |
*** wanghao has quit IRC | 15:50 | |
*** wanghao has joined #openstack-cinder | 15:51 | |
*** e0ne has joined #openstack-cinder | 15:51 | |
*** wanghao has quit IRC | 15:51 | |
*** wanghao has joined #openstack-cinder | 15:52 | |
*** jccates has joined #openstack-cinder | 15:55 | |
*** wanghao has quit IRC | 15:56 | |
*** swright has quit IRC | 16:00 | |
*** jccates has quit IRC | 16:02 | |
*** dave-mccowan has joined #openstack-cinder | 16:03 | |
*** jccates has joined #openstack-cinder | 16:04 | |
*** dhinesh has joined #openstack-cinder | 16:14 | |
*** mvk has quit IRC | 16:14 | |
*** gkadam has joined #openstack-cinder | 16:14 | |
*** dustins has quit IRC | 16:15 | |
*** armax has joined #openstack-cinder | 16:16 | |
*** dhinesh has quit IRC | 16:18 | |
*** markvoelker has joined #openstack-cinder | 16:20 | |
*** kaisers has joined #openstack-cinder | 16:21 | |
*** e0ne has quit IRC | 16:23 | |
*** ntpttr_laptop has joined #openstack-cinder | 16:23 | |
*** e0ne has joined #openstack-cinder | 16:24 | |
*** e0ne has quit IRC | 16:24 | |
*** ntpttr_laptop has quit IRC | 16:24 | |
*** e0ne has joined #openstack-cinder | 16:25 | |
*** kaisers has quit IRC | 16:26 | |
*** e0ne has quit IRC | 16:26 | |
*** e0ne has joined #openstack-cinder | 16:27 | |
*** e0ne has quit IRC | 16:27 | |
*** e0ne has joined #openstack-cinder | 16:28 | |
*** ociuhandu has quit IRC | 16:28 | |
*** e0ne has quit IRC | 16:29 | |
*** e0ne has joined #openstack-cinder | 16:29 | |
*** chlong has quit IRC | 16:29 | |
*** e0ne has quit IRC | 16:29 | |
*** sbezverk has joined #openstack-cinder | 16:32 | |
*** tesseract has quit IRC | 16:35 | |
*** AlexeyAbashkin has quit IRC | 16:36 | |
*** lkuchlan has joined #openstack-cinder | 16:47 | |
*** links has joined #openstack-cinder | 16:51 | |
*** mvk has joined #openstack-cinder | 16:54 | |
openstackgerrit | Merged openstack/cinder master: VMAX driver - documentation updates around versions https://review.openstack.org/510888 | 16:55 |
*** lpetrut has quit IRC | 16:59 | |
hemna_ | mriedem, first I've heard of that | 17:04 |
hemna_ | I might be able to find time to look into that one | 17:05 |
*** chlong has joined #openstack-cinder | 17: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-cinder | 17:20 | |
patrickeast | hemna_: 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 not | 17: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 while | 17:22 |
hemna_ | https://github.com/openstack/os-brick/blob/master/os_brick/initiator/linuxscsi.py#L154-L167 | 17:23 |
patrickeast | yea, would be weird | 17:23 |
hemna_ | anyway, I bet it's just not installed or running on his env. | 17:23 |
patrickeast | if 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 releases | 17:23 |
openstackgerrit | MonicaJoshi proposed openstack/cinder master: Message change for Policy error message change when user is not-authorized https://review.openstack.org/511195 | 17:25 |
*** dustins has joined #openstack-cinder | 17:26 | |
*** cali_boxer has quit IRC | 17:26 | |
*** cali_boxer has joined #openstack-cinder | 17:28 | |
*** Taseer has joined #openstack-cinder | 17:31 | |
Taseer | Hi 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-cinder | 17:36 | |
sgu | hi | 17:37 |
sgu | wanted to know if infra issue is fixed? | 17:37 |
hemna_ | patrickeast, yah I would think so. we'll see how he replies to the bug | 17:38 |
hemna_ | looks like DuncanT might be in a hurricane in Gallway ? | 17:39 |
jungleboyj | sgu: Infra issue? | 17:40 |
*** links has quit IRC | 17:42 | |
*** dhinesh has joined #openstack-cinder | 17:42 | |
*** dave-mccowan has quit IRC | 17:42 | |
jungleboyj | Taseer: What do you mean? | 17:45 |
sgu | jungleboyj: yes during gerrit CI runs | 17:45 |
sgu | jungleboyj: last ran it on oct 11 and jenkins passed but zuul failed | 17:46 |
jungleboyj | sgu: So, they have switched to Zuulv3 and to me it looks like .... oh. | 17:46 |
sgu | jungleboyj: cannot figure out anything from errors what exactly caused it | 17:46 |
jungleboyj | sgu: Which patch? | 17:46 |
sgu | jungleboyj: Ib701edb733567831ee80a317d0e3b3945a312760 | 17:47 |
Taseer | jungleboyj: I mean how to do from CLI ? | 17:47 |
jungleboyj | sgu: 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 |
Taseer | enforcing on hypervisor level and backend level | 17:48 |
jungleboyj | Taseer: 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 IRC | 17:49 | |
jungleboyj | Whether the QoS is enforced by Nova or the Backend depends on what specs you set and what is supported by your backend. | 17:49 |
Taseer | jungleboyj: okay, thanks. | 17:50 |
jungleboyj | Taseer: 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-cinder | 17:51 | |
jungleboyj | Trying to think who has done more with QoS. | 17:51 |
sgu | jungleboyj: thank you | 17:51 |
jungleboyj | Taseer: 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 |
jungleboyj | sgu: Welcome. | 17:52 |
*** wanghao has joined #openstack-cinder | 17:52 | |
Taseer | jungleboyj: thank you for the help. I'll try and dig in deeper myself. | 17:52 |
jungleboyj | Taseer: Sure. Let me know what you find. :-) | 17:52 |
Taseer | jungleboyj: I definitely will. | 17:53 |
jungleboyj | sgu: 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 |
jungleboyj | sgu How is the CI coming along for that? | 17:54 |
*** wanghao has quit IRC | 17:57 | |
*** AlexeyAbashkin has joined #openstack-cinder | 17:59 | |
*** tonygunk has joined #openstack-cinder | 18:00 | |
*** dave-mccowan has joined #openstack-cinder | 18:00 | |
*** psachin has quit IRC | 18:00 | |
*** crushil has quit IRC | 18:00 | |
*** AlexeyAbashkin has quit IRC | 18:03 | |
*** ChubYann has joined #openstack-cinder | 18:04 | |
*** sgu has quit IRC | 18:05 | |
*** catintheroof has quit IRC | 18:07 | |
*** catintheroof has joined #openstack-cinder | 18:08 | |
*** catinthe_ has joined #openstack-cinder | 18:16 | |
*** markvoelker has quit IRC | 18:16 | |
*** markvoelker has joined #openstack-cinder | 18:17 | |
*** kaisers has joined #openstack-cinder | 18:17 | |
*** catintheroof has quit IRC | 18:18 | |
*** markvoelker has quit IRC | 18:21 | |
*** catinthe_ has quit IRC | 18:23 | |
*** shaner has quit IRC | 18:24 | |
*** shaner_ has joined #openstack-cinder | 18:24 | |
*** lpetrut has joined #openstack-cinder | 18:25 | |
*** jmlowe has joined #openstack-cinder | 18:32 | |
*** Aditya has joined #openstack-cinder | 18:38 | |
*** ociuhandu has joined #openstack-cinder | 18:40 | |
*** shaner_ has quit IRC | 18:43 | |
*** shaner has joined #openstack-cinder | 18:44 | |
*** Aditya has quit IRC | 18:46 | |
*** sgu has joined #openstack-cinder | 18:48 | |
sgu | jungleboyj: It is in final stages and will be done by end of this week. | 18:49 |
jungleboyj | sgu: Ok, good news. | 18:49 |
sgu | jungleboyj: also how to issue recheck on just zuul tests | 18:49 |
jungleboyj | sgu: Don't think you can do just zuul. Have to do the broad recheck. | 18:50 |
sgu | jungleboyj: ok thank you | 18:51 |
jungleboyj | Welcome. :-) | 18:51 |
*** lhx_ has quit IRC | 18:52 | |
*** mmethot has joined #openstack-cinder | 18:52 | |
*** jmlowe has quit IRC | 18:53 | |
sgu | jungleboyj: 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 |
jungleboyj | I think what you did is fine. | 18:56 |
jungleboyj | sgu ... You just submitted a reply with 'recheck' right? | 18:57 |
sgu | yes | 18:57 |
jungleboyj | Ok, then in a little while you should see a job for your patch number pop up here: http://zuulv3.openstack.org/ | 18:57 |
sgu | jungleboyj: ok | 18:58 |
*** lkuchlan has quit IRC | 19:03 | |
*** edmondsw has quit IRC | 19:04 | |
*** edmondsw has joined #openstack-cinder | 19:04 | |
*** lkuchlan has joined #openstack-cinder | 19:08 | |
*** edmondsw has quit IRC | 19:09 | |
*** lkuchlan has quit IRC | 19:14 | |
openstackgerrit | MonicaJoshi proposed openstack/cinder master: DS8000 ConsistencyGroup failed to update group with volumes https://review.openstack.org/511212 | 19:15 |
sgu | jungleboyj: it didn't start | 19:16 |
jgriffith | well I'm about ready to give up on these unit tests :( | 19:20 |
*** hemna_ has quit IRC | 19:20 | |
jgriffith | just wrap the save call in a try/except block and forget it | 19:21 |
jgriffith | and I don't know what magic nonsense was used for things like the v3 api unit tests but it's evil | 19:21 |
*** mordred has quit IRC | 19:31 | |
smcginnis | sgu: The recheck comment can't have any text before "recheck" | 19:32 |
*** datasundae has joined #openstack-cinder | 19:33 | |
jungleboyj | smcginnis: I wondered if that had happened. | 19:33 |
datasundae | OK | 19:34 |
*** hemna_ has joined #openstack-cinder | 19:34 | |
jungleboyj | datasundae: Welcome! | 19:34 |
ildikov | jgriffith: which way did you go with those tests? | 19:34 |
datasundae | /message jungleboyj | 19:34 |
ildikov | jgriffith: I mean with mocking | 19:34 |
jgriffith | ildikov: I have been trying to "do the right thing" | 19:34 |
jgriffith | and go through the existing tests and mock the service_update call where it's being indirectly called | 19:35 |
jgriffith | I had some success doing class-wide mock patches in setUp for things like the volume and scheduler tests | 19:35 |
jgriffith | but then I got to api/v3 | 19:35 |
jgriffith | and there's all sorts of inheritance things linking forward and backwards (and sideways) that makes that approach fall apart a bit | 19:36 |
smcginnis | jgriffith: Doesn't it just work to mock class-wide for the v2 tests that get inherited by the v3 ones? | 19:37 |
jgriffith | maybe I'll go through each test individually | 19:37 |
ildikov | jgriffith: what if you mock save itself for the API tests? | 19:37 |
jgriffith | smcginnis: I thought it would | 19:37 |
jgriffith | smcginnis: I added the mock to the setUp in the V2 class but it didn't work | 19:37 |
jgriffith | smcginnis: so I also added it to the V3 class as well... but then the contrib versions failed | 19:38 |
jgriffith | and some other things blew up | 19:38 |
jgriffith | and I got annoyed | 19:38 |
smcginnis | Ah, I bet the v3 ones would have to call super(v3test, self).setUp(). | 19:38 |
ildikov | jgriffith: is that uploaded to Gerrit or all local at this point? | 19:39 |
smcginnis | Annoyance is understood and empathised. ;) | 19:39 |
jgriffith | ildikov: the mocks? No, not in gerrit because frankly it's hideous anyway | 19:40 |
jgriffith | even if it all worked, it's terrible | 19:40 |
ildikov | jgriffith: just wanted to take a look how you do the mocking in those API tests | 19:40 |
*** crushil has joined #openstack-cinder | 19:40 | |
ildikov | Yeah, I get the disappointment, certainly sounds to be very far from rainbows and unicorns... | 19:42 |
*** mordred has joined #openstack-cinder | 19:42 | |
ildikov | jgriffith: is there an alternative? | 19:43 |
jgriffith | ildikov: there's lots of alternatives :) | 19:43 |
jgriffith | take a vacation, play bingo, nap.... | 19:43 |
jgriffith | all sorts of things | 19:43 |
*** sgu has quit IRC | 19:43 | |
ildikov | jgriffith: I knew I should've specified the question more, lol | 19:44 |
jgriffith | so we could allow the column to be nullable in the service object, but that's not *right* | 19:44 |
jgriffith | and honestly at some point we're going to need to address this | 19:44 |
jgriffith | I guess I'll just go through tests one by one | 19:44 |
jgriffith | or look at other options | 19:45 |
ildikov | The API ones you mean? | 19:45 |
jgriffith | just in general | 19:45 |
jgriffith | but i had kinda hoped that maybe folks smarter than me would take a look at this and see if they had ideas/suggestions | 19:45 |
ildikov | I would say find a way to get that patch working and refactor from there | 19:45 |
jgriffith | LOL... I don't even know what that means? | 19:46 |
ildikov | Which obviously leads to the fact that we will forget about refactoring | 19:46 |
ildikov | Which part did sound so unfamiliar? :) | 19:47 |
jgriffith | "get that patch workign and refactor from there" | 19:47 |
jgriffith | getting the patch working means getting the unit tests working as the functional code works | 19:47 |
jgriffith | unless you mean put a try/except block around the save call and hope for the best | 19:47 |
jgriffith | but that's kinda bad | 19:48 |
ildikov | I know and you said that it looks terrible even where it works | 19:48 |
jgriffith | well, I guess it won't look any more terrible than what we already have :) | 19:48 |
ildikov | Nah, I meant to polish how the tests are running and using the mocks | 19:48 |
jgriffith | what's one more mock.patch decorator :) | 19:48 |
ildikov | I'm fine with that :) | 19:49 |
jgriffith | also 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 |
ildikov | Used a lot of those in Nova :) | 19:49 |
jgriffith | so I guess that just means going through each test one by one until all 198 of them are addressed... sigh | 19:50 |
ildikov | jgriffith: why are the unit tests calling update? | 19:50 |
*** armax has quit IRC | 19:50 | |
jgriffith | ildikov: because we do updates all over the place with objects | 19:50 |
jgriffith | that's how it works | 19:50 |
ildikov | jgriffith: if you upload the current version I can play with it a bit this afternoon | 19:50 |
*** markvoelker has joined #openstack-cinder | 19:51 | |
jgriffith | it's uploaded, feel free :) | 19:51 |
*** markvoelker has quit IRC | 19:51 | |
*** markvoelker has joined #openstack-cinder | 19:51 | |
jgriffith | I even put a comment in the patch pointing out exactly where the problem is | 19:51 |
*** wes_dillingham has quit IRC | 19:51 | |
ildikov | jgriffith: ok, will work from what's up there now | 19:51 |
ildikov | jgriffith: I know, I think I reflected on that over the weekend :) | 19:52 |
*** wanghao has joined #openstack-cinder | 19:53 | |
jgriffith | I 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 though | 19:53 |
*** edmondsw has joined #openstack-cinder | 19:54 | |
jgriffith | if the answer is to just add a mock decorator in each test that needs it that's fine | 19:54 |
* jungleboyj wishes jgriffith Hadn't mentioned a nap. That sounds so good. | 19:54 | |
jgriffith | just feels like there should be another way to go about it | 19:54 |
jgriffith | jungleboyj: :) | 19:54 |
jungleboyj | And my bed is so close. | 19:54 |
jgriffith | jungleboyj: naps are *ALWAYS* good, but I never seem to actually take them | 19:54 |
jungleboyj | jgriffith: I love them when I take them but feel so guilty afterwards. | 19:55 |
jgriffith | jungleboyj: pishhh... reall power naps are done in your chair at your desk | 19:55 |
*** shaner has quit IRC | 19:55 | |
jgriffith | That'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 |
jungleboyj | http://gph.is/29qftCt | 19:55 |
jgriffith | jungleboyj: hehe... been there | 19:56 |
*** markvoelker_ has joined #openstack-cinder | 19:57 | |
*** wanghao has quit IRC | 19:57 | |
*** shaner has joined #openstack-cinder | 19:59 | |
ildikov | I wish I could do a power nap... | 20:00 |
*** markvoelker has quit IRC | 20:01 | |
*** dustins has quit IRC | 20:09 | |
*** diablo_rojo has quit IRC | 20:10 | |
openstackgerrit | Marc 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/512111 | 20:14 |
*** salv-orlando has joined #openstack-cinder | 20:16 | |
openstackgerrit | Marc proposed openstack/cinder master: Allowed to delete up to same day marked deleted records https://review.openstack.org/512111 | 20:17 |
jgriffith | ildikov: 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 populated | 20:21 |
jgriffith | but 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 do | 20:21 |
jgriffith | the 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 it | 20:22 |
*** armax has joined #openstack-cinder | 20:24 | |
smcginnis | Seems like it would be worth doing (at some point) to fake out those things for all unit tests. | 20:24 |
smcginnis | Since a unit test shouldn't really be interacting with a database. | 20:24 |
ildikov | jgriffith: do you mean the fake service is not always used? Or that update is mocked on a per case basis anyway? | 20:24 |
ildikov | smcginnis: don't get jgriffith started on that :) | 20:24 |
*** bjolo has joined #openstack-cinder | 20:25 | |
*** diablo_rojo has joined #openstack-cinder | 20:25 | |
jgriffith | ildikov: what I mean is there's no global fake service for fake-db being used in all places | 20:28 |
jgriffith | there 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 |
ildikov | jgriffith: ah ok, got it | 20:28 |
jgriffith | which is great!! Until you try and change something in the fundamental object code :) | 20:29 |
*** tpsilva has quit IRC | 20:43 | |
*** tonygunk has quit IRC | 20:44 | |
*** edmondsw has quit IRC | 20:56 | |
*** cali_boxer has quit IRC | 20:58 | |
*** cali_boxer has joined #openstack-cinder | 20:58 | |
*** kfarr has joined #openstack-cinder | 21:08 | |
openstackgerrit | John Griffith proposed openstack/cinder master: Fix migration 112 to use live_data_migration API https://review.openstack.org/510201 | 21:15 |
jgriffith | ildikov: so if you're curious ^^ | 21:16 |
jgriffith | smcginnis: you might be too | 21:16 |
jgriffith | so the interesting one in there is the api test_volume_metadata | 21:16 |
jgriffith | I 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 IRC | 21:17 | |
smcginnis | Wait, microversion tests implemented in v2 unit tests? | 21:18 |
jgriffith | smcginnis: that was my point :) | 21:19 |
smcginnis | Well that doesn't seem right. :) | 21:19 |
smcginnis | jgriffith: But you did those mocks in the v2 test setUp and the v3 inheriting ones failed? | 21:20 |
*** kfarr has quit IRC | 21:20 | |
jgriffith | unless it's mv 3.0 I guess? | 21:20 |
jgriffith | I did | 21:20 |
jgriffith | aww crap... hold on | 21:20 |
jgriffith | looking that patch doesn't have the v2 mock in it... let me make sure that doesn't work still | 21:20 |
jgriffith | I was doing the binary test thing add/remove | 21:21 |
jgriffith | maybe I screwed someting up | 21:21 |
ildikov | jgriffith: thank you, pretty much appreciated | 21:21 |
ildikov | jgriffith: always happy to share some pain :) | 21:21 |
jgriffith | smcginnis: 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't | 21:22 |
jgriffith | ildikov: well, I just figured I'm not good at articulating code sometimes so it's easier to just show it | 21:22 |
smcginnis | jgriffith: Yeah, that would have been my expectation too. | 21:22 |
jgriffith | smcginnis: it's also tricky because there are two classes in the v3 volume_metadata test | 21:23 |
jgriffith | and 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 |
jgriffith | but I fully admit I'm not the sharpest tool in the shed with the unit test magic | 21:23 |
smcginnis | Mock works in mysterious ways too. | 21:23 |
jgriffith | smcginnis: +1 | 21:24 |
jgriffith | it's a wonderful beast, until something goes awry :) | 21:24 |
jgriffith | I'm about to yank the stupd zone-manager tests too btw | 21:24 |
smcginnis | Something wrong with them too? | 21:24 |
jgriffith | or I guess spend the time to figure out why they're soooo sloowwwwwwwww | 21:24 |
smcginnis | Oh, right. | 21:25 |
smcginnis | I could never really tell if it was them, or if it was something right after they run. | 21:25 |
smcginnis | But always seems to hang around there. | 21:25 |
smcginnis | And awhile back when we were getting OOM errors and weird test run hangs, it always seemed to be there too. | 21:25 |
jgriffith | smcginnis: it's one of them, because I delete that dir and it's okie-dokie :) | 21:25 |
smcginnis | Hmm... | 21:26 |
jgriffith | smcginnis: and | 21:26 |
jgriffith | cinder.tests.unit.zonemanager.test_brcd_fc_zone_driver.TestBrcdFcZoneDriver.test_add_connection_for_invalid_fabric 129.277 | 21:26 |
smcginnis | Yikes | 21:26 |
jgriffith | 129.277 seconds | 21:26 |
jgriffith | cra-cra | 21:26 |
smcginnis | Well that could explain why it hangs so long there. | 21:26 |
smcginnis | Must be a sleep or something in there. Can't imagine it actually being busy for two minutes. | 21:26 |
jgriffith | smcginnis: so here's an example: cinder.tests.unit.api.v3.test_volume_metadata.VolumeMetaDataTestNoMicroversion.test_update_metadata_item_keys_value_none | 21:27 |
jgriffith | yeah, probably missing a patch sleep | 21:27 |
jgriffith | I haven't looked | 21:27 |
jgriffith | and the last thing I need is another distraction :) | 21:27 |
jgriffith | smcginnis: so taht test | 21:27 |
smcginnis | Right! | 21:27 |
jgriffith | that | 21:27 |
jgriffith | grep on that | 21:27 |
*** lpetrut has quit IRC | 21:27 | |
*** ganso has quit IRC | 21:28 | |
jgriffith | so what's weird too is that I have a mock.patch service_update in both the V2 setUp and both of the V3 setUp classes | 21:28 |
jgriffith | but it's still trying to pull from the db. unless... | 21:29 |
*** crushil has quit IRC | 21:29 | |
smcginnis | Dang you git blame!! | 21:29 |
jgriffith | haha | 21:29 |
jgriffith | I can just explicitly mock that test, but I'd really like to understand *what* is going on here | 21:30 |
smcginnis | jgriffith: 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 |
jgriffith | and I'm also concerned that I'm making more of a mess than fixing anything | 21:31 |
jgriffith | because the mocks in the various setUps fixes some tests but not others | 21:31 |
jgriffith | so I've just got crap scattered all over the place | 21:31 |
jgriffith | smcginnis: yeah... maybe I'm just doing something incorrrectly | 21:31 |
jgriffith | but it seems it should be pretty straight forward | 21:32 |
smcginnis | Should be. | 21:32 |
jgriffith | it figures it would be volume_metadata that's such a mess :) | 21:35 |
smcginnis | jgriffith: Haha, didn't think of that. How appropriate. :D | 21:35 |
jgriffith | smcginnis: I swear it's a curse that will haunt me forever!!! | 21:36 |
* smcginnis writes up storage-sig post on adding metadata | 21:36 | |
* jgriffith throws his hands in the air and steps away from the computer | 21:36 | |
* jungleboyj wants to cry while writing Jira stories. | 21:37 | |
*** chlong has quit IRC | 21:45 | |
jungleboyj | @!h | 21:48 |
jungleboyj | Wait, where is pewp bot!?! | 21:48 |
jungleboyj | hemna: ^^^ | 21:48 |
*** wanghao has joined #openstack-cinder | 21:54 | |
*** xyang1 has quit IRC | 21:58 | |
*** bjolo has quit IRC | 22:00 | |
*** wanghao has quit IRC | 22:03 | |
*** crushil has joined #openstack-cinder | 22:06 | |
*** diablo_rojo has quit IRC | 22:09 | |
*** salv-orlando has quit IRC | 22:14 | |
*** mchlumsky has quit IRC | 22:19 | |
*** edmondsw has joined #openstack-cinder | 22:25 | |
*** masber has joined #openstack-cinder | 22:26 | |
*** edmondsw has quit IRC | 22:29 | |
*** felipemonteiro_ has quit IRC | 22:37 | |
*** gouthamr has quit IRC | 22:37 | |
*** cali_boxer has quit IRC | 22:43 | |
*** wes_dillingham has joined #openstack-cinder | 22:43 | |
hemna_ | oh noes | 22:48 |
tommylikehu | jungleboyj, smcginnis : https://review.openstack.org/#/c/510722/ can you take a look? thanks! | 22:51 |
openstackgerrit | John Cates proposed openstack/cinder master: FlashSystem: Add CMMVC6045E CLI error for multi-host mapping https://review.openstack.org/511645 | 22:52 |
*** sdague has quit IRC | 22:56 | |
*** lbragstad has quit IRC | 22:57 | |
*** liverpooler has quit IRC | 22:57 | |
*** catintheroof has joined #openstack-cinder | 22:58 | |
*** gouthamr has joined #openstack-cinder | 22:58 | |
*** jccates has quit IRC | 23:02 | |
*** dhinesh has quit IRC | 23:05 | |
*** ganso has joined #openstack-cinder | 23:13 | |
*** salv-orlando has joined #openstack-cinder | 23:15 | |
*** salv-orlando has quit IRC | 23:21 | |
jgriffith | smcginnis: ok, got it down to 53; remaining are manual adds needed to response checks etc | 23:31 |
jgriffith | should be able to get this working tomorrow | 23:31 |
jgriffith | smcginnis: so the interesting thing about mocking those calls in the metadata tests is it needs to be done before calling super | 23:31 |
ildikov | jgriffith: sounds good, thank you | 23:37 |
jgriffith | ildikov: good times :) | 23:37 |
ildikov | jgriffith: 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 week | 23:37 |
ildikov | jgriffith: fingers crossed... :) | 23:38 |
jgriffith | ildikov: I haven't looked at that yet | 23:38 |
ildikov | jgriffith: I don't think you want to ;) | 23:38 |
ildikov | jgriffith: 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 already | 23:39 |
ildikov | the live_migrate patch also needs to be updated to have the attachment_update call covered in it elsewhere | 23:39 |
ildikov | jgriffith: 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!