| *** Julien-zte has quit IRC | 00:01 | |
| *** Julien-zte has joined #openstack-cinder | 00:02 | |
| *** edmondsw has joined #openstack-cinder | 00:10 | |
| *** edmondsw has quit IRC | 00:14 | |
| *** salv-orlando has joined #openstack-cinder | 00:15 | |
| *** markvoelker has joined #openstack-cinder | 00:16 | |
| *** salv-orlando has quit IRC | 00:19 | |
| *** hongbin has quit IRC | 00:27 | |
| *** arahal_ has quit IRC | 00:33 | |
| *** Julien-zte has quit IRC | 00:33 | |
| *** Julien-zte has joined #openstack-cinder | 00:37 | |
| *** Julien-zte has quit IRC | 00:38 | |
| *** Julien-zte has joined #openstack-cinder | 00:41 | |
| *** Julien-zte has quit IRC | 00:43 | |
| *** Julien-zte has joined #openstack-cinder | 00:45 | |
| *** crushil has joined #openstack-cinder | 00:46 | |
| *** markvoelker has quit IRC | 00:49 | |
| *** kbaegis has joined #openstack-cinder | 00:50 | |
| *** crushil has quit IRC | 00:55 | |
| *** diablo_rojo has joined #openstack-cinder | 01:09 | |
| *** mriedem has joined #openstack-cinder | 01:10 | |
| *** kbaegis has quit IRC | 01:11 | |
| *** markstur has joined #openstack-cinder | 01:18 | |
| *** markstur has quit IRC | 01:22 | |
| *** diablo_rojo has quit IRC | 01:27 | |
| *** mriedem has quit IRC | 01:31 | |
| *** stevemar has left #openstack-cinder | 01:35 | |
| *** markstur has joined #openstack-cinder | 01:49 | |
| *** markstur has quit IRC | 01:53 | |
| *** kaisers_ has joined #openstack-cinder | 01:55 | |
| *** edmondsw has joined #openstack-cinder | 01:58 | |
| *** kaisers has quit IRC | 01:58 | |
| *** edmondsw has quit IRC | 02:02 | |
| *** esker has joined #openstack-cinder | 02:14 | |
| *** salv-orlando has joined #openstack-cinder | 02:16 | |
| *** crushil has joined #openstack-cinder | 02:22 | |
| *** salv-orlando has quit IRC | 02:22 | |
| *** esker has quit IRC | 02:27 | |
| *** esker has joined #openstack-cinder | 02:28 | |
| *** esker has quit IRC | 02:32 | |
| *** esker has joined #openstack-cinder | 02:38 | |
| *** edmondsw has joined #openstack-cinder | 02:38 | |
| *** markvoelker has joined #openstack-cinder | 02:46 | |
| *** edmondsw has quit IRC | 02:50 | |
| *** dikonoor has joined #openstack-cinder | 02:52 | |
| *** crushil has quit IRC | 02:54 | |
| *** edmondsw has joined #openstack-cinder | 02:58 | |
| *** gouthamr has quit IRC | 03:07 | |
| *** links has joined #openstack-cinder | 03:09 | |
| *** psachin has joined #openstack-cinder | 03:16 | |
| *** markvoelker has quit IRC | 03:20 | |
| *** links has quit IRC | 03:32 | |
| *** edmondsw has quit IRC | 03:34 | |
| *** kaisers_ has quit IRC | 03:35 | |
| *** psachin has quit IRC | 03:43 | |
| *** gkadam has joined #openstack-cinder | 03:45 | |
| *** markstur has joined #openstack-cinder | 03:48 | |
| *** links has joined #openstack-cinder | 03:49 | |
| *** markstur has quit IRC | 03:52 | |
| *** jwcroppe_ has quit IRC | 03:57 | |
| *** jwcroppe has joined #openstack-cinder | 03:58 | |
| *** psachin has joined #openstack-cinder | 04:00 | |
| *** Poornima has joined #openstack-cinder | 04:02 | |
| *** Dinesh_Bhor has quit IRC | 04:02 | |
| *** bhagyashris has quit IRC | 04:02 | |
| *** john5223_ has joined #openstack-cinder | 04:03 | |
| *** jwcroppe has quit IRC | 04:03 | |
| *** kaisers has joined #openstack-cinder | 04:05 | |
| *** arnewiebalck_ has joined #openstack-cinder | 04:05 | |
| *** esker has quit IRC | 04:06 | |
| *** esker has joined #openstack-cinder | 04:10 | |
| *** markvoelker has joined #openstack-cinder | 04:16 | |
| *** salv-orlando has joined #openstack-cinder | 04:19 | |
| *** zhangguoqing has joined #openstack-cinder | 04:19 | |
| *** zhangguoqing has quit IRC | 04:21 | |
| *** salv-orlando has quit IRC | 04:23 | |
| *** zhangguoqing has joined #openstack-cinder | 04:24 | |
| *** zhangguoqing has quit IRC | 04:25 | |
| *** bhagyashris has joined #openstack-cinder | 04:27 | |
| *** adisky_ has joined #openstack-cinder | 04:30 | |
| *** zhangguoqing has joined #openstack-cinder | 04:30 | |
| *** esker has quit IRC | 04:32 | |
| *** dineshbhor has joined #openstack-cinder | 04:33 | |
| *** dineshbhor has quit IRC | 04:37 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 04:37 | |
| *** zhangguo_ has joined #openstack-cinder | 04:39 | |
| *** salv-orlando has joined #openstack-cinder | 04:39 | |
| *** zhangguoqing has quit IRC | 04:40 | |
| *** edmondsw has joined #openstack-cinder | 04:45 | |
| *** kaisers has quit IRC | 04:47 | |
| *** kaisers has joined #openstack-cinder | 04:47 | |
| *** edmondsw has quit IRC | 04:50 | |
| *** markvoelker has quit IRC | 04:51 | |
| *** Poornima has quit IRC | 04:53 | |
| *** kaisers has quit IRC | 04:55 | |
| *** kaisers has joined #openstack-cinder | 04:56 | |
| *** arnewiebalck_ has quit IRC | 04:58 | |
| *** arnewiebalck_ has joined #openstack-cinder | 04:59 | |
| *** kaisers has quit IRC | 05:00 | |
| *** arnewiebalck_ has quit IRC | 05:11 | |
| *** kaisers has joined #openstack-cinder | 05:26 | |
| *** e0ne has joined #openstack-cinder | 05:27 | |
| *** markstur has joined #openstack-cinder | 05:37 | |
| *** markstur has quit IRC | 05:42 | |
| *** zhangguo_ has quit IRC | 05:45 | |
| *** zhangguoqing has joined #openstack-cinder | 05:45 | |
| *** markvoelker has joined #openstack-cinder | 05:48 | |
| *** Poornima_K has joined #openstack-cinder | 05:52 | |
| *** pgadiya has joined #openstack-cinder | 05:55 | |
| *** pgadiya has quit IRC | 05:55 | |
| *** rcernin has joined #openstack-cinder | 06:14 | |
| *** salv-orlando has quit IRC | 06:19 | |
| *** savvas has joined #openstack-cinder | 06:20 | |
| *** markvoelker has quit IRC | 06:21 | |
| *** jsuchome has joined #openstack-cinder | 06:23 | |
| *** ChubYann has quit IRC | 06:25 | |
| *** aselius has quit IRC | 06:26 | |
| *** savvas has quit IRC | 06:30 | |
| *** sdatko has quit IRC | 06:33 | |
| *** edmondsw has joined #openstack-cinder | 06:34 | |
| *** edmondsw has quit IRC | 06:38 | |
| *** e0ne has quit IRC | 06:41 | |
| *** e0ne has joined #openstack-cinder | 06:49 | |
| *** danpawlik has quit IRC | 06:58 | |
| *** chhavi has joined #openstack-cinder | 07:00 | |
| *** e0ne has quit IRC | 07:01 | |
| *** tesseract has joined #openstack-cinder | 07:08 | |
| *** dalgaaf has quit IRC | 07:09 | |
| *** dalgaaf has joined #openstack-cinder | 07:09 | |
| *** salv-orlando has joined #openstack-cinder | 07:11 | |
| *** salv-orl_ has joined #openstack-cinder | 07:27 | |
| *** salv-orlando has quit IRC | 07:30 | |
| *** pcaruana has joined #openstack-cinder | 07:33 | |
| *** pcaruana has quit IRC | 07:39 | |
| *** pcaruana has joined #openstack-cinder | 07:39 | |
| *** savvas has joined #openstack-cinder | 07:42 | |
| *** savvas has quit IRC | 07:46 | |
| *** Julien-zte has quit IRC | 07:54 | |
| *** Julien-zte has joined #openstack-cinder | 07:56 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| *** Julien-zte has quit IRC | 08:02 | |
| *** Julien-zte has joined #openstack-cinder | 08:03 | |
| *** bjornar_ has joined #openstack-cinder | 08:03 | |
| *** Julien-zte has quit IRC | 08:06 | |
| *** Julien-zte has joined #openstack-cinder | 08:07 | |
| *** Julien-zte has quit IRC | 08:15 | |
| *** Julien-zte has joined #openstack-cinder | 08:16 | |
| *** openstackgerrit has quit IRC | 08:18 | |
| *** markvoelker has joined #openstack-cinder | 08:18 | |
| *** Julien-zte has quit IRC | 08:20 | |
| *** Julien-zte has joined #openstack-cinder | 08:21 | |
| *** edmondsw has joined #openstack-cinder | 08:22 | |
| *** openstackgerrit has joined #openstack-cinder | 08:26 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Add is_replicated() to Volume and VolumeType OVOs https://review.openstack.org/400806 | 08:26 |
|---|---|---|
| openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Make failover DB changes consistent https://review.openstack.org/400805 | 08:26 |
| *** edmondsw has quit IRC | 08:26 | |
| *** Julien-zte has quit IRC | 08:27 | |
| *** e0ne has joined #openstack-cinder | 08:27 | |
| *** Julien-zte has joined #openstack-cinder | 08:27 | |
| *** bhagyashris has quit IRC | 08:37 | |
| *** Julien-zte has quit IRC | 08:39 | |
| *** Julien-zte has joined #openstack-cinder | 08:42 | |
| *** markvoelker has quit IRC | 08:52 | |
| *** Julien-zte has quit IRC | 09:03 | |
| *** Julien-zte has joined #openstack-cinder | 09:05 | |
| *** bhagyashris has joined #openstack-cinder | 09:06 | |
| *** Julien-zte has quit IRC | 09:08 | |
| *** Julien-zte has joined #openstack-cinder | 09:10 | |
| *** Julien-zte has quit IRC | 09:12 | |
| *** Julien-zte has joined #openstack-cinder | 09:13 | |
| *** Guest59427 is now known as asettle | 09:15 | |
| openstackgerrit | jiaohaolin proposed openstack/cinder master: Inspur Cinder FC/iSCSI driver https://review.openstack.org/439488 | 09:22 |
| *** frakt_ has quit IRC | 09:27 | |
| *** jmccrory has quit IRC | 09:27 | |
| *** eglute has quit IRC | 09:27 | |
| *** links has quit IRC | 09:27 | |
| *** pjm6 has quit IRC | 09:27 | |
| *** nug- has quit IRC | 09:27 | |
| *** shaner has quit IRC | 09:27 | |
| *** jidar has quit IRC | 09:27 | |
| *** Trident has quit IRC | 09:27 | |
| *** aarefiev_afk has quit IRC | 09:27 | |
| *** mugsie has quit IRC | 09:27 | |
| *** karthikp_ has quit IRC | 09:27 | |
| *** Julien-zte has quit IRC | 09:28 | |
| *** john5223_ has quit IRC | 09:29 | |
| *** Julien-zte has joined #openstack-cinder | 09:31 | |
| *** dalgaaf has quit IRC | 09:31 | |
| *** Julien-zte has quit IRC | 09:31 | |
| *** dalgaaf has joined #openstack-cinder | 09:31 | |
| *** Julien-zte has joined #openstack-cinder | 09:32 | |
| *** Julien-zte has quit IRC | 09:38 | |
| *** Julien-zte has joined #openstack-cinder | 09:39 | |
| *** Poornima_K has quit IRC | 09:42 | |
| *** mvk has quit IRC | 09:44 | |
| *** Julien-zte has quit IRC | 09:45 | |
| *** Julien-zte has joined #openstack-cinder | 09:46 | |
| *** eglute has joined #openstack-cinder | 09:49 | |
| *** jmccrory has joined #openstack-cinder | 09:49 | |
| *** frakt has joined #openstack-cinder | 09:49 | |
| *** markvoelker has joined #openstack-cinder | 09:49 | |
| *** slunkad has quit IRC | 09:50 | |
| openstackgerrit | jiaohaolin proposed openstack/cinder master: Inspur Cinder FC/iSCSI driver-tests-CI https://review.openstack.org/469030 | 09:51 |
| *** karthikp_ has joined #openstack-cinder | 09:52 | |
| *** Poornima_K has joined #openstack-cinder | 09:52 | |
| *** Julien-zte has quit IRC | 09:53 | |
| *** Julien-zte has joined #openstack-cinder | 09:54 | |
| *** Julien-zte has quit IRC | 09:57 | |
| *** Julien-zte has joined #openstack-cinder | 09:58 | |
| *** Julien-zte has quit IRC | 10:01 | |
| *** Julien-zte has joined #openstack-cinder | 10:01 | |
| *** lpetrut has joined #openstack-cinder | 10:06 | |
| *** Poornima_K has quit IRC | 10:08 | |
| *** mvk has joined #openstack-cinder | 10:08 | |
| *** mvk has quit IRC | 10:09 | |
| *** edmondsw has joined #openstack-cinder | 10:10 | |
| *** bjornar_ has quit IRC | 10:12 | |
| *** danpawlik has joined #openstack-cinder | 10:13 | |
| *** links has joined #openstack-cinder | 10:13 | |
| *** pjm6 has joined #openstack-cinder | 10:13 | |
| *** nug has joined #openstack-cinder | 10:13 | |
| *** shaner has joined #openstack-cinder | 10:13 | |
| *** jidar has joined #openstack-cinder | 10:13 | |
| *** Trident has joined #openstack-cinder | 10:13 | |
| *** aarefiev_afk has joined #openstack-cinder | 10:13 | |
| *** mugsie has joined #openstack-cinder | 10:13 | |
| *** mvk has joined #openstack-cinder | 10:13 | |
| *** Julien-zte has quit IRC | 10:13 | |
| *** nug is now known as Guest81873 | 10:14 | |
| *** edmondsw has quit IRC | 10:14 | |
| *** mvk has quit IRC | 10:14 | |
| *** mvk has joined #openstack-cinder | 10:14 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Update OVO devref https://review.openstack.org/469040 | 10:15 |
| *** Julien-zte has joined #openstack-cinder | 10:15 | |
| *** dikonoor has quit IRC | 10:15 | |
| *** mtreinish has quit IRC | 10:15 | |
| *** dims has quit IRC | 10:15 | |
| *** mtreinish has joined #openstack-cinder | 10:16 | |
| *** dikonoor has joined #openstack-cinder | 10:16 | |
| *** Poornima_K has joined #openstack-cinder | 10:18 | |
| *** dims has joined #openstack-cinder | 10:18 | |
| *** danpawlik has quit IRC | 10:19 | |
| *** danpawlik has joined #openstack-cinder | 10:20 | |
| *** markvoelker has quit IRC | 10:22 | |
| *** Julien-zte has quit IRC | 10:23 | |
| *** bjornar_ has joined #openstack-cinder | 10:24 | |
| *** Julien-zte has joined #openstack-cinder | 10:26 | |
| *** Julien-zte has quit IRC | 10:27 | |
| *** Julien-zte has joined #openstack-cinder | 10:29 | |
| *** stakeda has quit IRC | 10:34 | |
| *** Poornima_K has quit IRC | 10:41 | |
| *** Julien-zte has quit IRC | 10:53 | |
| *** Julien-zte has joined #openstack-cinder | 10:56 | |
| *** Julien-zte has quit IRC | 11:02 | |
| *** Julien-zte has joined #openstack-cinder | 11:02 | |
| *** dims has quit IRC | 11:05 | |
| *** kbaegis has joined #openstack-cinder | 11:07 | |
| *** dims has joined #openstack-cinder | 11:11 | |
| *** Poornima_K has joined #openstack-cinder | 11:12 | |
| *** gkadam is now known as gkadam-afk | 11:17 | |
| *** Poornima_K has quit IRC | 11:19 | |
| *** markvoelker has joined #openstack-cinder | 11:20 | |
| *** nicolasbock has joined #openstack-cinder | 11:30 | |
| *** Poornima_K has joined #openstack-cinder | 11:36 | |
| *** psachin has quit IRC | 11:40 | |
| *** abishop has joined #openstack-cinder | 11:46 | |
| *** Julien-zte has quit IRC | 11:48 | |
| *** Poornima_K has quit IRC | 11:49 | |
| *** Julien-zte has joined #openstack-cinder | 11:52 | |
| *** zhangguoqing has quit IRC | 11:52 | |
| *** markvoelker has quit IRC | 11:52 | |
| *** markvoelker has joined #openstack-cinder | 11:52 | |
| openstackgerrit | Merged openstack/cinder master: Update OVO devref https://review.openstack.org/469040 | 11:53 |
| *** Poornima_K has joined #openstack-cinder | 11:58 | |
| *** chlong has joined #openstack-cinder | 11:58 | |
| *** psachin has joined #openstack-cinder | 12:00 | |
| *** chlong has quit IRC | 12:01 | |
| openstackgerrit | Chhavi Agarwal proposed openstack/cinder master: Storwize Support iSCSI host when FC volume is there https://review.openstack.org/461850 | 12:01 |
| *** links has quit IRC | 12:10 | |
| *** frontrunner has joined #openstack-cinder | 12:10 | |
| *** sileht has left #openstack-cinder | 12:14 | |
| *** Julien-zte has quit IRC | 12:15 | |
| *** Julien-zte has joined #openstack-cinder | 12:17 | |
| *** edmondsw has joined #openstack-cinder | 12:19 | |
| *** MVenesio has joined #openstack-cinder | 12:22 | |
| openstackgerrit | Kushal Wathore proposed openstack/cinder master: HPE Lefthand: Fixed cloning operation https://review.openstack.org/462624 | 12:23 |
| *** eharney has joined #openstack-cinder | 12:36 | |
| *** dave-mcc_ has joined #openstack-cinder | 12:36 | |
| *** Julien-zte has quit IRC | 12:42 | |
| *** Julien-zte has joined #openstack-cinder | 12:43 | |
| *** eharney has quit IRC | 12:45 | |
| *** chhavi_ has joined #openstack-cinder | 12:49 | |
| *** chhavi has quit IRC | 12:51 | |
| *** gkadam-afk is now known as gkadam | 12:58 | |
| *** Julien-zte has quit IRC | 12:59 | |
| *** eharney has joined #openstack-cinder | 13:00 | |
| *** Julien-zte has joined #openstack-cinder | 13:01 | |
| *** Julien-zte has quit IRC | 13:04 | |
| *** Julien-z_ has joined #openstack-cinder | 13:05 | |
| *** chlong has joined #openstack-cinder | 13:08 | |
| *** Julien-z_ has quit IRC | 13:12 | |
| *** xyang1 has joined #openstack-cinder | 13:14 | |
| *** lucasxu has joined #openstack-cinder | 13:14 | |
| *** mtanino_ has quit IRC | 13:15 | |
| *** eharney has quit IRC | 13:15 | |
| *** Julien-zte has joined #openstack-cinder | 13:17 | |
| *** Julien-zte has quit IRC | 13:18 | |
| *** Julien-zte has joined #openstack-cinder | 13:19 | |
| *** Julien-zte has quit IRC | 13:22 | |
| *** Julien-zte has joined #openstack-cinder | 13:23 | |
| *** gouthamr has joined #openstack-cinder | 13:26 | |
| *** salv-orlando has joined #openstack-cinder | 13:27 | |
| *** Julien-zte has quit IRC | 13:27 | |
| *** Julien-zte has joined #openstack-cinder | 13:28 | |
| *** salv-orl_ has quit IRC | 13:30 | |
| *** winston-d_ has joined #openstack-cinder | 13:30 | |
| *** Julien-zte has quit IRC | 13:32 | |
| *** Julien-zte has joined #openstack-cinder | 13:33 | |
| *** mriedem has joined #openstack-cinder | 13:37 | |
| *** Julien-zte has quit IRC | 13:37 | |
| *** Julien-zte has joined #openstack-cinder | 13:38 | |
| *** eharney has joined #openstack-cinder | 13:41 | |
| *** jwcroppe has joined #openstack-cinder | 13:43 | |
| *** Julien-zte has quit IRC | 13:44 | |
| *** Julien-zte has joined #openstack-cinder | 13:46 | |
| *** Poornima_K has quit IRC | 13:47 | |
| *** Julien-zte has quit IRC | 13:47 | |
| *** Julien-zte has joined #openstack-cinder | 13:49 | |
| *** gaurangt_ has joined #openstack-cinder | 13:51 | |
| *** Julien-zte has quit IRC | 13:52 | |
| *** Julien-zte has joined #openstack-cinder | 13:53 | |
| *** gaurangt has quit IRC | 13:53 | |
| *** gaurangt has joined #openstack-cinder | 13:55 | |
| *** gaurangt_ has quit IRC | 13:56 | |
| *** Julien-zte has quit IRC | 13:57 | |
| *** Julien-zte has joined #openstack-cinder | 13:57 | |
| *** arnewiebalck_ has joined #openstack-cinder | 14:00 | |
| *** salv-orlando has quit IRC | 14:02 | |
| *** aselius has joined #openstack-cinder | 14:02 | |
| *** crushil has joined #openstack-cinder | 14:04 | |
| *** eharney has quit IRC | 14:07 | |
| *** kfarr has joined #openstack-cinder | 14:07 | |
| *** esker has joined #openstack-cinder | 14:10 | |
| *** Julien-zte has quit IRC | 14:10 | |
| *** Julien-zte has joined #openstack-cinder | 14:11 | |
| *** arnewiebalck_ has quit IRC | 14:16 | |
| *** Thelo has joined #openstack-cinder | 14:17 | |
| *** gouthamr has quit IRC | 14:20 | |
| *** gouthamr has joined #openstack-cinder | 14:23 | |
| *** arahal_ has joined #openstack-cinder | 14:23 | |
| *** Guest81873 is now known as cFouts | 14:23 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Periodic task to clean expired reservation https://review.openstack.org/467867 | 14:26 |
| *** mtanino has joined #openstack-cinder | 14:27 | |
| *** gkadam has quit IRC | 14:35 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - deletion of bootable volume may take a long time https://review.openstack.org/469139 | 14:36 |
| *** tomh_ has joined #openstack-cinder | 14:42 | |
| *** vivsoni_ has joined #openstack-cinder | 14:43 | |
| *** vivsoni has quit IRC | 14:43 | |
| *** tomh__ has quit IRC | 14:43 | |
| *** dikonoor has quit IRC | 14:45 | |
| *** diablo_rojo has joined #openstack-cinder | 14:46 | |
| *** hongbin has joined #openstack-cinder | 14:48 | |
| *** adisky_ has quit IRC | 14:48 | |
| hemna | floop | 14:54 |
| *** eharney has joined #openstack-cinder | 14:56 | |
| jgriffith | snoop | 14:56 |
| hemna | at least it's not Monday starting the work week | 14:57 |
| jgriffith | hemna somehow that doesn't seem to be making it any better :( | 14:57 |
| hemna | probably not. | 14:57 |
| *** tobasco_ is now known as tobasco | 14:59 | |
| *** Thelo has left #openstack-cinder | 15:01 | |
| ildikov | if it's snoop it should be Snoopy :) | 15:01 |
| jgriffith | ildikov Snoopy it is! | 15:01 |
| ildikov | jgriffith: cool! Err, cute! :) | 15:02 |
| *** markstur has joined #openstack-cinder | 15:03 | |
| jgriffith | https://s-media-cache-ak0.pinimg.com/564x/69/c5/73/69c573a4b243d2943d826b1a95939e9d.jpg | 15:03 |
| *** ericyoung has joined #openstack-cinder | 15:04 | |
| openstackgerrit | Thomas Bechtold proposed openstack/python-cinderclient master: Do not require network for test_noauth_plugin() https://review.openstack.org/469153 | 15:04 |
| *** jonesn has joined #openstack-cinder | 15:07 | |
| *** rcernin has quit IRC | 15:09 | |
| ildikov | jgriffith: that's how I'm gonna look like when I start figuring out how to use Cinder microversions with Nova :) | 15:11 |
| jgriffith | ildikov ha! If you're lucky!! | 15:11 |
| *** frontrunner has quit IRC | 15:11 | |
| *** dulek has quit IRC | 15:17 | |
| *** bjornar_ has quit IRC | 15:18 | |
| ildikov | jgriffith: I'm not sure I would like to know what you have in mind for me if I'm not... :) | 15:18 |
| *** Swanson has quit IRC | 15:19 | |
| *** frontrunner has joined #openstack-cinder | 15:23 | |
| *** catintheroof has joined #openstack-cinder | 15:25 | |
| *** catintheroof has quit IRC | 15:25 | |
| *** dulek has joined #openstack-cinder | 15:25 | |
| *** catintheroof has joined #openstack-cinder | 15:26 | |
| *** Swanson has joined #openstack-cinder | 15:30 | |
| *** vbala has joined #openstack-cinder | 15:33 | |
| *** gaurangt_ has joined #openstack-cinder | 15:34 | |
| *** gaurangt has quit IRC | 15:35 | |
| *** lucasxu has quit IRC | 15:36 | |
| *** lucasxu has joined #openstack-cinder | 15:37 | |
| *** arnewiebalck_ has joined #openstack-cinder | 15:38 | |
| *** Poornima has joined #openstack-cinder | 15:41 | |
| *** e0ne has quit IRC | 15:42 | |
| *** kaisers has quit IRC | 15:46 | |
| *** kaisers has joined #openstack-cinder | 15:46 | |
| *** vbala has quit IRC | 15:49 | |
| *** vbala has joined #openstack-cinder | 15:50 | |
| *** Julien-zte has quit IRC | 15:52 | |
| *** Poornima has quit IRC | 15:53 | |
| *** Poornima has joined #openstack-cinder | 15:54 | |
| *** vbala has quit IRC | 15:54 | |
| *** arnewiebalck_ has quit IRC | 15:54 | |
| *** crose has joined #openstack-cinder | 15:54 | |
| *** lucasxu has quit IRC | 15:54 | |
| *** vbala has joined #openstack-cinder | 15:54 | |
| *** Julien-zte has joined #openstack-cinder | 15:54 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Return metadata in volume summary https://review.openstack.org/459143 | 15:56 |
| *** vbala has quit IRC | 15:58 | |
| *** tesseract has quit IRC | 16:03 | |
| *** ericyoung has quit IRC | 16:06 | |
| *** Julien-zte has quit IRC | 16:07 | |
| *** Apoorva has joined #openstack-cinder | 16:07 | |
| *** catintheroof has quit IRC | 16:08 | |
| *** arunkant has joined #openstack-cinder | 16:09 | |
| *** catintheroof has joined #openstack-cinder | 16:09 | |
| *** arunkant_ has joined #openstack-cinder | 16:09 | |
| *** arunkant_ has quit IRC | 16:09 | |
| *** kaisers has quit IRC | 16:09 | |
| *** kbaegis has quit IRC | 16:10 | |
| *** Julien-zte has joined #openstack-cinder | 16:10 | |
| *** Julien-zte has quit IRC | 16:10 | |
| *** mnestratov|2 has joined #openstack-cinder | 16:11 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Return metadata in volume summary https://review.openstack.org/459143 | 16:13 |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Update version show response example https://review.openstack.org/448469 | 16:17 |
| *** gaurangt has joined #openstack-cinder | 16:18 | |
| *** dikonoor has joined #openstack-cinder | 16:18 | |
| ildikov | jgriffith: I think I found the perfect one for multi-attach: https://vignette3.wikia.nocookie.net/peanuts/images/f/f4/Snoopy_with_Woodstock_in_bunny_ears.png/revision/latest?cb=20130727162323 :) | 16:19 |
| jgriffith | LOL!!! | 16:19 |
| hemna | ildikov, hehe nice | 16:20 |
| *** gaurangt- has joined #openstack-cinder | 16:20 | |
| *** Poornima has quit IRC | 16:20 | |
| ildikov | a good logo candidate when we get there :) | 16:21 |
| *** gaurangt_ has quit IRC | 16:22 | |
| *** kaisers has joined #openstack-cinder | 16:22 | |
| *** gaurangt has quit IRC | 16:22 | |
| smcginnis | :D | 16:23 |
| *** kbaegis has joined #openstack-cinder | 16:29 | |
| *** salv-orlando has joined #openstack-cinder | 16:29 | |
| *** kaisers has quit IRC | 16:30 | |
| *** kaisers has joined #openstack-cinder | 16:31 | |
| *** david-lyle has joined #openstack-cinder | 16:32 | |
| *** mvk has quit IRC | 16:33 | |
| *** itlinux has joined #openstack-cinder | 16:34 | |
| *** uberjay has quit IRC | 16:39 | |
| *** kbaegis1 has joined #openstack-cinder | 16:40 | |
| *** uberjay has joined #openstack-cinder | 16:40 | |
| *** kbaegis has quit IRC | 16:40 | |
| *** lpetrut has quit IRC | 16:42 | |
| *** lucasxu has joined #openstack-cinder | 16:43 | |
| *** pcaruana has quit IRC | 16:43 | |
| jungleboyj | ildikov: Nice! That would be better than this: http://gph.is/2e4hPg0 | 16:48 |
| *** felipemonteiro has joined #openstack-cinder | 16:49 | |
| smcginnis | This one kind of sums up the multiattach work so far: http://gph.is/2efWSOy | 16:50 |
| * jungleboyj is laughing | 16:51 | |
| ildikov | smcginnis: we have a ton of things merged in Nova so do not de-motivate people around plz :) | 16:51 |
| *** itlinux has quit IRC | 16:58 | |
| smcginnis | :) | 16:58 |
| *** Apoorva_ has joined #openstack-cinder | 16:59 | |
| *** Apoorva has quit IRC | 17:03 | |
| *** eharney has quit IRC | 17:09 | |
| *** mvk has joined #openstack-cinder | 17:12 | |
| *** harlowja has joined #openstack-cinder | 17:14 | |
| *** lpetrut has joined #openstack-cinder | 17:17 | |
| Swanson | I knew that if I ignored it the CI would start to pass again. | 17:19 |
| openstackgerrit | Merged openstack/cinder master: Add a local bindep.txt override https://review.openstack.org/468169 | 17:20 |
| *** ChubYann has joined #openstack-cinder | 17:22 | |
| *** eharney has joined #openstack-cinder | 17:22 | |
| smcginnis | Swanson: That works maybe 50% of the time. | 17:23 |
| openstackgerrit | Merged openstack/cinder master: Add the missing "vol_type_id" param for qos association api-ref https://review.openstack.org/468431 | 17:24 |
| *** nicolasbock has quit IRC | 17:25 | |
| *** dhinesh has joined #openstack-cinder | 17:28 | |
| *** catintheroof has quit IRC | 17:29 | |
| *** lpetrut has quit IRC | 17:29 | |
| *** gaurangt has joined #openstack-cinder | 17:37 | |
| *** gaurangt_ has joined #openstack-cinder | 17:38 | |
| *** gaurangt- has quit IRC | 17:41 | |
| *** gaurangt has quit IRC | 17:42 | |
| openstackgerrit | Merged openstack/cinder master: change consistencygroup_id to group_id https://review.openstack.org/468469 | 17:42 |
| *** jsuchome has quit IRC | 17:48 | |
| dhinesh | local tox py27 unit test passes but after i push the code for review it fails. Should i be updating the code base at my local, if so how would u do that? | 18:00 |
| hemna | dhinesh, sounds like you need to rebase ? | 18:01 |
| dhinesh | hemna: i rebased it 2 patches before, is it required to rebase often? | 18:02 |
| smcginnis | dhinesh: Do you have a link to the failure? | 18:02 |
| smcginnis | dhinesh: Maybe it's just a random gate failure. | 18:03 |
| *** frontrunner has quit IRC | 18:03 | |
| dhinesh | http://logs.openstack.org/41/433341/18/check/gate-cinder-python27-ubuntu-xenial/55649ed/console.html.gz | 18:03 |
| smcginnis | dhinesh: No idea how that could pass locally. The failure says exactly what the issue is. | 18:04 |
| dhinesh | smcginnis: yeah not sure how it passed to | 18:05 |
| dhinesh | smcginnis: so would rebasing make a difference? | 18:09 |
| smcginnis | dhinesh: No. | 18:09 |
| *** frontrunner has joined #openstack-cinder | 18:10 | |
| *** david-lyle has quit IRC | 18:13 | |
| *** mnestratov|2 has quit IRC | 18:13 | |
| *** Apoorva_ has quit IRC | 18:15 | |
| *** Apoorva has joined #openstack-cinder | 18:15 | |
| *** dalgaaf has quit IRC | 18:23 | |
| *** frontrunner has quit IRC | 18:23 | |
| *** dalgaaf has joined #openstack-cinder | 18:24 | |
| *** david-lyle has joined #openstack-cinder | 18:26 | |
| *** frontrunner has joined #openstack-cinder | 18:26 | |
| *** crushil has quit IRC | 18:31 | |
| *** crushil has joined #openstack-cinder | 18:32 | |
| *** chhavi_ has quit IRC | 18:35 | |
| openstackgerrit | Gaurang Tapase proposed openstack/cinder master: GPFS: Fix forceful delete for consistency group https://review.openstack.org/468760 | 18:43 |
| *** nicolasbock has joined #openstack-cinder | 18:53 | |
| smcginnis | Seeing a case in the gate where volume_clear='none' but still seeing dd'ing out the volume. Anyone else see this? | 18:53 |
| *** felipemonteiro has quit IRC | 19:00 | |
| _alastor_ | hey, I need to get read/write permissions for a device returned by the os_brick iscsi connector, is there a utility in os_brick for that? | 19:02 |
| openstackgerrit | Merged openstack/cinder master: Fix typos in README.md https://review.openstack.org/468621 | 19:06 |
| smcginnis | _alastor_: Not from os-brick AFAIK. Wouldn't you just take the returned device path and open it? | 19:07 |
| _alastor_ | smcginnis: Yes, but I get a PermissionDenied error. I'm writing a backup driver and need to get access to that device so I can write the backup data to it | 19:08 |
| *** eharney has quit IRC | 19:08 | |
| smcginnis | _alastor_: You would need to go through rootwrap. | 19:09 |
| _alastor_ | smcginnis: I've never had to use rootwrap before, do you have an example for device permissions? | 19:11 |
| smcginnis | _alastor_: Not off hand, but you can grep through for other instances. Pretty much any instance where you need to "sudo x", you need to use rootwrap. | 19:12 |
| smcginnis | And/or someday privsep, but we've only convereted os-brick for that. | 19:12 |
| _alastor_ | smcginnis: looking at the rootwrap os_brick module: https://github.com/openstack/os-brick/blob/master/os_brick/privileged/rootwrap.py#L13 | 19:16 |
| _alastor_ | smcginnis: That seems a bit worrying... | 19:16 |
| *** itlinux has joined #openstack-cinder | 19:17 | |
| smcginnis | Hah! Yes, thus the desire to move to privsep. Then it's just a smaller security backdoor instead of a massive one. | 19:17 |
| *** eharney has joined #openstack-cinder | 19:22 | |
| _alastor_ | smcginnis: I'm not coming up with a good way to get a file object from a subprocess command, which seems to be all that rootwrap provides | 19:22 |
| jgriffith | smcginnis so in order to make the window smaller we blow out the wall temporarily while building a new window frame? | 19:22 |
| jgriffith | :) | 19:22 |
| smcginnis | _alastor_: Right, you will need to be able to call out to commands, but my guess is you want to just access it in your code. | 19:22 |
| smcginnis | jgriffith: Just put up a tarp. | 19:23 |
| jgriffith | :) | 19:23 |
| _alastor_ | smcginnis: Maybe some chmod commands via rootwrap, then access it normally? | 19:24 |
| smcginnis | Ironic as I have a couple workers below my right now blowing a whole in my basement foundation to put in a window. | 19:24 |
| stvnoyes | jgriffith: working on live migration with your new attach patch. hitting a problem... | 19:24 |
| smcginnis | _alastor_: Hmm, worth a shot. If you don't mind chmod'ing your device access. | 19:24 |
| *** e0ne has joined #openstack-cinder | 19:24 | |
| jgriffith | stvnoyes what's up? | 19:24 |
| stvnoyes | in migration, we now try to create a new attachment on the destination host. | 19:24 |
| smcginnis | _alastor_: Maybe just put things back where you found them when you're done? :) | 19:24 |
| jgriffith | stvnoyes and it fails because the volume already has an attachment? | 19:25 |
| stvnoyes | but the cinder api throws an exception as the volume is in-use, so the reserve fails | 19:25 |
| jgriffith | stvnoyes yeah, we need to add something on Cinder for the special case | 19:25 |
| stvnoyes | ah ok, i wasn't sure if I was missing something | 19:26 |
| jgriffith | It used to be we didn't check, but that was problematic due to some concerns around the whole check_attach debate | 19:26 |
| jgriffith | stvnoyes nah, sadly you're not missing anything; it's an artificial block put in Cinder's API | 19:26 |
| _alastor_ | smcginnis: Seems reasonable. I'll be the only one accessing them, but just to be clean I can change permissions back when the writing is done | 19:26 |
| stvnoyes | a check something like if the volume is already attached to the same instance, it would be ok? | 19:27 |
| *** salv-orl_ has joined #openstack-cinder | 19:27 | |
| jgriffith | stvnoyes so the question is how we want to deal with it, the best solution is likely to add something to attachment_create that indicates we're doing a speical transitional type thing and allow it | 19:27 |
| jgriffith | stvnoyes so the same instance trick would work... and I believe that's similar to a path that was gone down in the past | 19:27 |
| *** e0ne has quit IRC | 19:27 | |
| stvnoyes | yeah, might be easier than adding something to the api. would adding that mean a microversion change? | 19:28 |
| jgriffith | stvnoyes depends on who you ask :) | 19:28 |
| stvnoyes | :-) | 19:28 |
| jgriffith | if you ask me the answer is "no" | 19:28 |
| stvnoyes | ok, i'll put in a hack for now to work around it. | 19:29 |
| jgriffith | The "good"? thing is that this API has never been consumed by Nova as of yet | 19:29 |
| jgriffith | stvnoyes yeah, I'll take a look at addressing it on the Cinder side and we can sync back up on what we want to do | 19:29 |
| stvnoyes | yes, that is good. so no version change for this then | 19:29 |
| stvnoyes | great thanks. | 19:29 |
| *** salv-orlando has quit IRC | 19:30 | |
| jgriffith | sure, thank you! | 19:30 |
| ildikov | jgriffith: stvnoyes: so I think in the Nova patch we said that if it's the same instance but different host that should be allowed with attachment_create | 19:30 |
| *** eharney has quit IRC | 19:31 | |
| ildikov | jgriffith: stvnoyes: so I think that was what we kind of agreed to try so I would go that direction and well, will keep my eyes and ears closed when it comes to microversions here :) | 19:31 |
| jgriffith | stvnoyes ildikov BTW, IIRC that's all Nova side checks currently no? | 19:32 |
| *** dikonoor has quit IRC | 19:32 | |
| jgriffith | Cinder IIRC doesn't actually care, it's actually just the check_attach in Nova that re-introduces that | 19:32 |
| ildikov | jgriffith: check_attach is removed | 19:33 |
| jgriffith | stvnoyes so back to that bug you filed against cinder a while back that you can do this very thing | 19:33 |
| ildikov | jgriffith: so in Cinder that can be reserve | 19:33 |
| jgriffith | ildikov hmm... ok, let me go look then. I could've sworn I left Cinder so you could create as many attachments as you like regardless of volume status | 19:34 |
| jgriffith | Bahh | 19:34 |
| jgriffith | yeah :( | 19:34 |
| ildikov | ? | 19:34 |
| jgriffith | cinder.volume.api:_attachment_reserve is where it catches that | 19:35 |
| jgriffith | I would love to just remove those expected states :( | 19:37 |
| jgriffith | only disallow "error" | 19:37 |
| openstackgerrit | ravi edpuganti proposed openstack/cinder master: Added CG capability to volume group in CoprHD https://review.openstack.org/459610 | 19:37 |
| jgriffith | and turn on multi-attach now :) | 19:37 |
| *** itlinux has quit IRC | 19:38 | |
| ildikov | you need to fix detach first :) | 19:39 |
| ildikov | I completely forgot that check | 19:39 |
| stvnoyes | jgriffith: i did update the bug with the note about live migration being an exception. i haven't been back to it for a while. I'm not sure if it's a real bug, or at least a cinder bug. | 19:39 |
| *** mnestratov|2 has joined #openstack-cinder | 19:40 | |
| stvnoyes | meaning it might be a nova bug... | 19:40 |
| ildikov | stvnoyes: how did you create those multiple attachments to the same volume? | 19:41 |
| *** crose has quit IRC | 19:41 | |
| ildikov | I got myself confused with the different bugs and scenarios now... | 19:42 |
| jgriffith | ildikov if you create without connector | 19:44 |
| *** eharney has joined #openstack-cinder | 19:44 | |
| jgriffith | although we added reserved so that shouldn't behave that way now either | 19:45 |
| ildikov | jgriffith: oh ok, you don't check reserved state as who cares | 19:46 |
| *** nicolasbock has quit IRC | 19:47 | |
| ildikov | jgriffith: yeah, it doesn't really have a use case to reserve the same volume multiple times | 19:47 |
| ildikov | jgriffith: so better not even allowing it | 19:47 |
| jgriffith | ildikov so it doesn't allow it any more, it used to. Now it's doing the usual check that the volume must be in available or downloading | 19:48 |
| stvnoyes | ildikov: for migrate? in pre_live_migrate, i called attachment_create. As a hack, I added 'in-use' to the list of expected status's so it won't throw | 19:49 |
| jgriffith | I am wondering if something like a force option would make sense here? Something more generic for the various special cases | 19:50 |
| stvnoyes | no connector was provided | 19:50 |
| jgriffith | stvnoyes that might be a way to keep it relatively safe | 19:51 |
| ildikov | stvnoyes: I meant the other bug that you had for Cinder when you created multiple attachments for the same instance on the same host | 19:51 |
| jgriffith | but I also don't want to make it more confusing or add too many moving parts if that makes sense | 19:51 |
| ildikov | stvnoyes: and yeah I was wondering how you made live migrate work as well :) | 19:51 |
| stvnoyes | oh that, I just was running attach a lot of times. and because the attach code was early, the attaches were mostly failing | 19:51 |
| jgriffith | yeah, they never actually used to flip the state of the volume :( | 19:52 |
| stvnoyes | me too :-) | 19:52 |
| ildikov | :) | 19:52 |
| ildikov | so when we were evaluating these things in the Nova spec we were thinking that saying the same instance with a different host should be a good indicator | 19:53 |
| jgriffith | stvnoyes ildikov so there's one other idea... | 19:53 |
| stvnoyes | that seems like a reasonable way to go | 19:53 |
| ildikov | although I would not like to be someone let's say a year from today trying to figure out why that's a special case purely from the code | 19:53 |
| * ildikov just thinking out loud now | 19:54 | |
| *** nicolasbock has joined #openstack-cinder | 19:54 | |
| jgriffith | what if we allowed to create all the time, but like you said ( stvnoyes ) if there's a connector we enforce the check | 19:54 |
| jgriffith | meh... maybe that doesn't work either | 19:55 |
| stvnoyes | i think the allow-if-same-instance check should be enough for cinder. nova should probably add another level of check to prevent this happening in the non migrate case | 19:57 |
| ildikov | so let's say we have a volume which is available | 19:57 |
| ildikov | you say it can be reserved with an empty attachment | 19:57 |
| ildikov | if it's reserved then you cannot reserve it again | 19:57 |
| ildikov | so you can only finalize the attach or delete the empty attachment and put the volume back to available | 19:58 |
| ildikov | if you finalized the attachment then it gets to in-use | 19:58 |
| ildikov | and in this case you could reserve it again | 19:58 |
| ildikov | I wonder what state the volume moves from here, I would say reserved | 19:59 |
| *** Apoorva_ has joined #openstack-cinder | 19:59 | |
| ildikov | and then create the connection on the destination host and remove the attachment on the source | 20:00 |
| ildikov | the flow can work, but we still need to check the instance | 20:01 |
| ildikov | like having the volume attached to one and get reserved by another one | 20:01 |
| ildikov | it can be a valid multi-attach case, but in any other case that should not happen | 20:01 |
| jgriffith | Yeah, that's sort of what I was thinking earlier | 20:02 |
| jgriffith | but I'm not completely clear on why the instance ID matters in that scenario either? | 20:02 |
| jgriffith | also, maybe we should just change how live-migrate works :) | 20:02 |
| ildikov | which scenario? | 20:02 |
| *** Apoorva has quit IRC | 20:02 | |
| jgriffith | ildikov the idea of being able to create another attachment but not connect it (yet) | 20:03 |
| ildikov | so if we stick with the flow I described then another instance can reserve the volume | 20:04 |
| ildikov | no one else can do another reserve | 20:04 |
| *** dave-mcc_ has quit IRC | 20:04 | |
| *** rarora has quit IRC | 20:04 | |
| ildikov | and the instance getting live migrated will not have an attachment_id to finalize | 20:04 |
| ildikov | so race condition | 20:04 |
| ildikov | and if we want to sort that out on the Nova side that would require a new check_attach and I would like to avoid that | 20:06 |
| * jgriffith is kinda confused now | 20:06 | |
| jgriffith | so what are you proposing? Just allow it if it's the same instance? Or something different? | 20:07 |
| ildikov | jgriffith: I might lost a bit on what's the preferred way forward now | 20:07 |
| *** crushil has quit IRC | 20:07 | |
| ildikov | jgriffith: I was just evaluating your idea that you mentioned above | 20:07 |
| jgriffith | ahh | 20:07 |
| jgriffith | ok, that makes sense then | 20:07 |
| ildikov | jgriffith: and I got to the conclusion that we cannot skip checking the instance id | 20:07 |
| openstackgerrit | ravi edpuganti proposed openstack/cinder master: Added CG capability to volume group in CoprHD https://review.openstack.org/459610 | 20:12 |
| *** vivsoni_ has quit IRC | 20:13 | |
| *** tomh_ has quit IRC | 20:13 | |
| ildikov | jgriffith: so we go with checking if it's the same instance and allow reserving if yes like how stvnoyes said above? | 20:13 |
| jgriffith | Yeah, that's what I'm messing with right now | 20:13 |
| *** felipemonteiro has joined #openstack-cinder | 20:14 | |
| ildikov | ok | 20:14 |
| jgriffith | stvnoyes ildikov FYI https://bugs.launchpad.net/cinder/+bug/1694530 | 20:18 |
| openstack | Launchpad bug 1694530 in Cinder "Cinders new attach API doesn't allow novas live-migrate cheat" [Undecided,New] | 20:18 |
| jgriffith | feel free to add to it if I missed something important there | 20:18 |
| smcginnis | "novas live-migrate cheat" :D | 20:19 |
| jgriffith | well, it kinda is :) | 20:20 |
| smcginnis | jgriffith: Context switch: simple thing that could speed up gate tests: https://review.openstack.org/#/c/469227/ | 20:20 |
| jgriffith | smcginnis sure, but I turned that off in the gate config like 3 years ago :) | 20:21 |
| *** eharney has quit IRC | 20:21 | |
| ildikov | jgriffith: there's a live migrate section here that describes: https://github.com/openstack/nova-specs/blob/master/specs/pike/approved/cinder-new-attach-apis.rst | 20:21 |
| jgriffith | smcginnis the other option is use thin-prov which doesn't do that anyway :) | 20:21 |
| smcginnis | jgriffith: It's turned off in the gate in the [DEFAULT] section. Which actually has zero impact. | 20:22 |
| ildikov | jgriffith: well, the new one is a bit better by being more visible | 20:22 |
| jgriffith | sigh.. because of the change we made that moved that stuff out of default? | 20:22 |
| smcginnis | jgriffith: Maybe part of that. Or when multibackend was added. | 20:23 |
| ildikov | the current version is a real cheat by not telling anything about disconnecting the volume on the source and connecting it on the destination host and playing with initialize_connection in a nasty way, etc. | 20:23 |
| smcginnis | Didn't track that down. | 20:23 |
| jgriffith | nahh... looks like the usual churn and it just got lost | 20:23 |
| smcginnis | OK, unimportant - back to multiattach. :) | 20:23 |
| jgriffith | I see what you mean now, it's no longer even evaluated :( | 20:23 |
| smcginnis | Before ildikov gets mad at me! | 20:24 |
| jgriffith | smcginnis yeah... +1 regardless | 20:24 |
| *** kfarr has quit IRC | 20:24 | |
| jgriffith | ildikov yeah, that's what I mean by "cheat" :) | 20:24 |
| ildikov | smcginnis: no more coffee for you if you keep on with these context switches :) | 20:24 |
| *** Guest63352 has joined #openstack-cinder | 20:24 | |
| * smcginnis will keep quiet for now on... | 20:24 | |
| ildikov | smcginnis: and I found these Korean ones pretty good so far I have to admit :) | 20:25 |
| ildikov | smcginnis: lol | 20:25 |
| smcginnis | ildikov: I really did like that one. Oops, now I'll be quiet. | 20:25 |
| ildikov | jgriffith: yeah, no live migrate without cheating I guess | 20:25 |
| ildikov | smcginnis: I have a dedicated thread for coffee in my brain, so that's fine :) | 20:26 |
| jgriffith | ildikov well... for now at least, and that's up to Nova folks to do what works best for them so that's cool | 20:26 |
| ildikov | jgriffith: I don't think it can be cleaned up fully considering the use case, but doing the reserve with attachment_create should be still a better way to do it | 20:27 |
| dhinesh | smcginnis: i am using san_login variable to define the username which conflicts with oslo.cfg, is there a way to overwrite it? | 20:28 |
| smcginnis | dhinesh: No, that's the problem is you are trying to overwrite it. | 20:29 |
| *** crushil has joined #openstack-cinder | 20:29 | |
| smcginnis | dhinesh: It's not a conflict with oslo.cfg. It's a conflict with the existing config option. | 20:29 |
| dhinesh | smcginnis: which would be the existing config option? | 20:30 |
| smcginnis | san_login | 20:31 |
| *** wN has quit IRC | 20:31 | |
| dhinesh | smcginnis: in which case i can define it something like <vendor_name>_san_login and should be good? | 20:33 |
| *** Guest63352 is now known as ryanender | 20:33 | |
| smcginnis | dhinesh: No, I think I had already commented on there that it would be better if you didn't do that. That way you can keep the config settings consistent. | 20:34 |
| *** gkadam has joined #openstack-cinder | 20:35 | |
| *** lucasxu has quit IRC | 20:36 | |
| *** lucasxu has joined #openstack-cinder | 20:36 | |
| dhinesh | smcginnis: yeah true, am trying to find the alternative to make it work, i see other vendors use san_user, maybe thats an option, how do i make sure i don't have overrding problems? | 20:36 |
| *** winston-d_ has quit IRC | 20:36 | |
| *** eharney has joined #openstack-cinder | 20:37 | |
| smcginnis | dhinesh: Right, just use that existing set of config options: san_user, san_ip, san_etc. | 20:37 |
| smcginnis | dhinesh: Then there is nothing to override. | 20:37 |
| *** Apoorva_ has quit IRC | 20:39 | |
| *** Apoorva has joined #openstack-cinder | 20:39 | |
| *** kaisers has quit IRC | 20:39 | |
| *** kaisers has joined #openstack-cinder | 20:39 | |
| *** jose-phillips has quit IRC | 20:45 | |
| dhinesh | smcginnis: ok thanks | 20:46 |
| *** jose-phillips has joined #openstack-cinder | 20:46 | |
| *** david-lyle has quit IRC | 20:53 | |
| *** david-lyle has joined #openstack-cinder | 20:54 | |
| *** jonesn has quit IRC | 20:55 | |
| *** ryanender has quit IRC | 20:59 | |
| *** lucasxu has quit IRC | 21:03 | |
| *** Apoorva_ has joined #openstack-cinder | 21:06 | |
| stvnoyes | ildikov: i have an idea to make all the new nova stuff easier to debug. | 21:09 |
| *** Apoorva has quit IRC | 21:10 | |
| stvnoyes | we call the new apis when the attachment_id is not None, it would be helpfule if the old cinder v2 apis (reserve, terminate, etc) would throw an exception if they get called with a non-null attach_id. what do you think? | 21:11 |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add an instance-uuid check on attachment_reserve https://review.openstack.org/469252 | 21:11 |
| *** david-lyle has quit IRC | 21:12 | |
| stvnoyes | since we will probably miss places in nova, especially on error paths where we might revert to the old api when we shouldn't | 21:12 |
| stvnoyes | jgriffith ^ | 21:12 |
| ildikov | stvnoyes: but then we would need to add the attachment_id to the old flow too | 21:12 |
| *** MVenesio has quit IRC | 21:12 | |
| jgriffith | stvnoyes first thought is I think that would be great, just would have to verify there aren't cases we're using it | 21:13 |
| *** KeithMnemonic has joined #openstack-cinder | 21:13 | |
| *** eharney has quit IRC | 21:13 | |
| stvnoyes | i'm running into this now debugging migrate. | 21:13 |
| stvnoyes | ildikov: shouldn't have to add attach_id to the old flow, just a check to see that it's None. | 21:14 |
| jgriffith | ildikov I'm assuming we could just make sure it's NOT in the body no? | 21:15 |
| jgriffith | somethign like this: https://hastebin.com/wutodopatu.py | 21:16 |
| jgriffith | ildikov shouldn't you be asleep by now? | 21:17 |
| stvnoyes | ah i now see what you meant. yes, as John says, not present (vs None) | 21:18 |
| ildikov | hmm, I got lost now a bit :) | 21:19 |
| jgriffith | none of the old calls the next layer down (volume.api) consume that, so we could put that on the old calls in volume-actions api code and raise back to Nova if they're there | 21:20 |
| jgriffith | would tell us rather quickly and easily that we *missed* something | 21:20 |
| jgriffith | ildikov so I think the idea is in contrib/volume_actions.py which is the entrypoint for all the old flow, just make sure nobody is trying to pass an attachment_id into the body of the request there | 21:22 |
| jgriffith | ildikov if they are, it likely indicates something went screwy in our logic on the Nova call somewhere | 21:22 |
| jgriffith | stvnoyes is that what you're thinking? | 21:22 |
| ildikov | but we need to add the attachment_id to the old flow in order to check whether it's null or not | 21:22 |
| jgriffith | ildikov nah, just the simple check I put in that hastebin I think? | 21:23 |
| ildikov | as we're kind of re-writing everything right now | 21:23 |
| jgriffith | Of course it only works when the new attach code is supposed to work :) | 21:23 |
| stvnoyes | are you saying do this in nova? I was thinking cinder. but it might be ok to do it in nova's cinder.py module | 21:23 |
| jgriffith | stvnoyes nope, in Cinder... just in cinder/api/contrib/volume_actions.py | 21:23 |
| jgriffith | sorry... does that diff not make sense? | 21:24 |
| ildikov | the diff make sense, but there will never be an attachment_id in the initialize_connection call | 21:24 |
| jgriffith | ildikov so I guess I thought that's what stvnoyes was saying he wanted to make sure of, but now maybe I'm just confused :) | 21:25 |
| *** lucasxu has joined #openstack-cinder | 21:26 | |
| *** felipemonteiro has quit IRC | 21:26 | |
| jgriffith | because, yeah, what I thought makes ZERO sense, given cinder.py can't send that... so ignore me | 21:26 |
| ildikov | so that's how that call looks in Nova: initialize_connection(self, context, volume_id, connector) | 21:26 |
| jgriffith | ildikov yeah, i'm with ya :) | 21:26 |
| jgriffith | so I thought I understood, but didn't... stvnoyes maybe you can explain a bit more what you mean? | 21:26 |
| ildikov | so the base idea would work if we ever used attachment_id anywhere else than detach | 21:26 |
| ildikov | but we didn't | 21:26 |
| stvnoyes | i see what you mean, the most these calls have in them are connectors. which have no attach_id params. | 21:27 |
| *** felipemonteiro has joined #openstack-cinder | 21:27 | |
| openstackgerrit | TommyLike proposed openstack/python-cinderclient master: Support generalized resource filter in client https://review.openstack.org/465002 | 21:28 |
| stvnoyes | so there would have to be some lookup somewhere (if that's possible). seems like too much work/time. never mind. | 21:28 |
| *** abishop has quit IRC | 21:28 | |
| openstackgerrit | John Griffith proposed openstack/cinder master: Add an instance-uuid check on attachment_reserve https://review.openstack.org/469252 | 21:30 |
| *** gkadam has quit IRC | 21:30 | |
| *** lucasxu has quit IRC | 21:31 | |
| *** crushil has quit IRC | 21:32 | |
| *** crushil has joined #openstack-cinder | 21:33 | |
| openstackgerrit | xing-yang proposed openstack/cinder master: Tiramisu: replication group support https://review.openstack.org/352228 | 21:38 |
| *** frontrunner has quit IRC | 21:39 | |
| ildikov | stvnoyes: what's in migrate that gave you the idea? | 21:39 |
| stvnoyes | i found that i was calling remove_volume_connection after a failed migrate. for some reason the bdm was missing its attachment id. i'm running that down now. | 21:42 |
| stvnoyes | that calls terminate_connection | 21:42 |
| ildikov | hmm, interesting | 21:44 |
| ildikov | I mean if the happy path works I wonder what went wrong with the attachment_id in that case | 21:46 |
| ildikov | like how error handling in Nova can make debugging difficult | 21:47 |
| ildikov | in the sense of having the BDM destroyed and you don't know about it as the API response didn't show any errors and so forth | 21:48 |
| stvnoyes | i haven't got to the happy case yet. the migrate is failing because of the exception during cinder create attach (which i haven't patched yet until I understand this) | 21:49 |
| *** felipemonteiro has quit IRC | 21:50 | |
| *** salv-orl_ has quit IRC | 21:50 | |
| *** salv-orlando has joined #openstack-cinder | 21:50 | |
| *** jwcroppe has quit IRC | 21:54 | |
| *** salv-orlando has quit IRC | 21:54 | |
| *** jwcroppe has joined #openstack-cinder | 21:55 | |
| *** gaurangt_ has quit IRC | 21:57 | |
| *** gaurangt has joined #openstack-cinder | 21:59 | |
| *** edmondsw has quit IRC | 22:08 | |
| *** mnestratov|2 has quit IRC | 22:08 | |
| *** edmondsw has joined #openstack-cinder | 22:09 | |
| ildikov | stvnoyes: oh ok, I thought you "patched" that part already | 22:11 |
| ildikov | stvnoyes: I saw your bug report on the rollback too, nice catch | 22:12 |
| *** gouthamr has quit IRC | 22:13 | |
| *** gouthamr has joined #openstack-cinder | 22:14 | |
| *** edmondsw has quit IRC | 22:14 | |
| *** crushil has quit IRC | 22:16 | |
| *** xyang1 has quit IRC | 22:20 | |
| openstackgerrit | Merged openstack/cinder master: Periodic task to clean expired reservation https://review.openstack.org/467867 | 22:44 |
| *** jwcroppe has quit IRC | 22:47 | |
| *** Apoorva_ has quit IRC | 22:50 | |
| *** Apoorva has joined #openstack-cinder | 22:51 | |
| *** frontrunner has joined #openstack-cinder | 22:51 | |
| *** savvas has joined #openstack-cinder | 22:58 | |
| *** gouthamr has quit IRC | 22:59 | |
| *** gouthamr has joined #openstack-cinder | 22:59 | |
| *** gouthamr has quit IRC | 23:02 | |
| *** gouthamr has joined #openstack-cinder | 23:03 | |
| *** kbaegis1 has quit IRC | 23:10 | |
| *** itlinux has joined #openstack-cinder | 23:11 | |
| *** savvas has quit IRC | 23:13 | |
| *** savvas has joined #openstack-cinder | 23:13 | |
| *** kbaegis has joined #openstack-cinder | 23:19 | |
| *** kbaegis has quit IRC | 23:19 | |
| *** kbaegis has joined #openstack-cinder | 23:19 | |
| *** kbaegis has quit IRC | 23:19 | |
| *** kbaegis has joined #openstack-cinder | 23:19 | |
| *** kbaegis has quit IRC | 23:19 | |
| *** kbaegis has joined #openstack-cinder | 23:20 | |
| *** kbaegis has quit IRC | 23:20 | |
| *** kbaegis has joined #openstack-cinder | 23:20 | |
| *** kbaegis has quit IRC | 23:20 | |
| *** mtanino has quit IRC | 23:26 | |
| *** frontrunner has quit IRC | 23:38 | |
| *** gouthamr has quit IRC | 23:50 | |
| *** gouthamr_ has joined #openstack-cinder | 23:50 | |
| *** stakeda has joined #openstack-cinder | 23:55 | |
| *** gouthamr_ has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
