| *** gouthamr_ has quit IRC | 00:00 | |
| *** dims has quit IRC | 00:04 | |
| *** ndipanov has quit IRC | 00:04 | |
| *** mylu has quit IRC | 00:06 | |
| *** dims has joined #openstack-cinder | 00:11 | |
| *** alonma has joined #openstack-cinder | 00:11 | |
| *** mylu has joined #openstack-cinder | 00:13 | |
| *** mylu has quit IRC | 00:15 | |
| *** alonma has quit IRC | 00:15 | |
| *** porrua has quit IRC | 00:17 | |
| *** tpatil has joined #openstack-cinder | 00:17 | |
| *** alonma has joined #openstack-cinder | 00:17 | |
| *** jungleboyj has quit IRC | 00:19 | |
| *** mylu has joined #openstack-cinder | 00:22 | |
| *** alonma has quit IRC | 00:22 | |
| *** alonma has joined #openstack-cinder | 00:25 | |
| openstackgerrit | Michael Still proposed openstack/cinder: Cleanup unused conf variables https://review.openstack.org/280453 | 00:26 |
|---|---|---|
| *** laughterwym has joined #openstack-cinder | 00:26 | |
| *** laughterwym has quit IRC | 00:27 | |
| *** laughterwym has joined #openstack-cinder | 00:27 | |
| *** laughterwym has quit IRC | 00:28 | |
| *** alonma has quit IRC | 00:30 | |
| *** EinstCra_ has quit IRC | 00:31 | |
| *** laughterwym has joined #openstack-cinder | 00:31 | |
| *** markvoelker has joined #openstack-cinder | 00:35 | |
| *** rajinir has quit IRC | 00:35 | |
| *** gouthamr has joined #openstack-cinder | 00:35 | |
| *** salv-orl_ has quit IRC | 00:38 | |
| *** gouthamr_ has joined #openstack-cinder | 00:38 | |
| *** markvoelker has quit IRC | 00:39 | |
| *** gouthamr has quit IRC | 00:41 | |
| *** jgregor has joined #openstack-cinder | 00:41 | |
| *** nibalizer is now known as nibz_irssi | 00:49 | |
| *** Julien-zte has joined #openstack-cinder | 00:50 | |
| *** nibz_irssi has quit IRC | 00:53 | |
| *** smoriya_ has joined #openstack-cinder | 00:56 | |
| *** arch-nemesis has quit IRC | 00:58 | |
| *** cheneydc has joined #openstack-cinder | 01:02 | |
| *** dims has quit IRC | 01:04 | |
| openstackgerrit | huyang proposed openstack/cinder: Delete unuseful code in Huawei driver https://review.openstack.org/280460 | 01:04 |
| *** dongwenjuan has joined #openstack-cinder | 01:04 | |
| *** martyturner has joined #openstack-cinder | 01:05 | |
| *** mtanino has quit IRC | 01:06 | |
| *** dims has joined #openstack-cinder | 01:08 | |
| *** davechen has joined #openstack-cinder | 01:08 | |
| *** dongwenjuan_ has joined #openstack-cinder | 01:12 | |
| *** dongwenjuan has quit IRC | 01:13 | |
| *** dongwenjuan has joined #openstack-cinder | 01:17 | |
| *** mylu has quit IRC | 01:17 | |
| *** dongwenjuan_ has quit IRC | 01:18 | |
| *** arch-nemesis has joined #openstack-cinder | 01:21 | |
| *** david-lyle has joined #openstack-cinder | 01:23 | |
| *** Lee1092 has joined #openstack-cinder | 01:23 | |
| *** alonma has joined #openstack-cinder | 01:25 | |
| *** alonma has quit IRC | 01:30 | |
| *** laughterwym has quit IRC | 01:30 | |
| *** dongwenjuan has quit IRC | 01:31 | |
| *** laughterwym has joined #openstack-cinder | 01:33 | |
| *** alonma has joined #openstack-cinder | 01:33 | |
| *** alonma has quit IRC | 01:38 | |
| *** alonma has joined #openstack-cinder | 01:40 | |
| *** haomaiwa_ has joined #openstack-cinder | 01:42 | |
| *** alonma has quit IRC | 01:44 | |
| *** arch-nemesis has quit IRC | 01:45 | |
| *** alonma has joined #openstack-cinder | 01:46 | |
| *** mylu has joined #openstack-cinder | 01:48 | |
| *** alonma has quit IRC | 01:50 | |
| *** alonma has joined #openstack-cinder | 01:55 | |
| *** edmondsw_ has quit IRC | 01:56 | |
| *** huangtianhua has quit IRC | 01:58 | |
| *** alonma has quit IRC | 01:59 | |
| *** haomaiwa_ has quit IRC | 02:01 | |
| *** haomaiwang has joined #openstack-cinder | 02:01 | |
| *** gcb has joined #openstack-cinder | 02:03 | |
| *** alonma has joined #openstack-cinder | 02:03 | |
| *** baojg has joined #openstack-cinder | 02:05 | |
| *** laughterwym has quit IRC | 02:06 | |
| *** laughterwym has joined #openstack-cinder | 02:07 | |
| *** laughterwym has joined #openstack-cinder | 02:07 | |
| *** alonma has quit IRC | 02:08 | |
| *** daneyon_ has quit IRC | 02:09 | |
| *** alonma has joined #openstack-cinder | 02:10 | |
| openstackgerrit | Angus Lees proposed openstack/os-brick: Trivial rootwrap -> privsep replacement https://review.openstack.org/277224 | 02:11 |
| *** alonma has quit IRC | 02:14 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 https://review.openstack.org/275797 | 02:15 |
| *** dwj has joined #openstack-cinder | 02:16 | |
| *** krtaylor has quit IRC | 02:20 | |
| *** Julien-zte has quit IRC | 02:22 | |
| *** Julien-z_ has joined #openstack-cinder | 02:22 | |
| *** alonma has joined #openstack-cinder | 02:26 | |
| *** houming has joined #openstack-cinder | 02:26 | |
| *** dwj has quit IRC | 02:27 | |
| *** alonma has quit IRC | 02:30 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add LUN wwn check in Huawei driver https://review.openstack.org/271592 | 02:30 |
| *** krtaylor has joined #openstack-cinder | 02:31 | |
| *** alonma has joined #openstack-cinder | 02:34 | |
| *** markvoelker has joined #openstack-cinder | 02:36 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add LUN wwn check in Huawei driver https://review.openstack.org/271592 | 02:37 |
| *** alonma has quit IRC | 02:39 | |
| *** markvoelker has quit IRC | 02:40 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Don't show chap password in log https://review.openstack.org/271595 | 02:40 |
| *** alonma has joined #openstack-cinder | 02:41 | |
| *** martyturner has quit IRC | 02:43 | |
| *** mylu has quit IRC | 02:44 | |
| *** alonma has quit IRC | 02:46 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Don't fail when port group does not exist https://review.openstack.org/271599 | 02:46 |
| *** alonma has joined #openstack-cinder | 02:48 | |
| *** mylu has joined #openstack-cinder | 02:48 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Ensure the port is online https://review.openstack.org/271600 | 02:49 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Check the QoS status before we use https://review.openstack.org/271601 | 02:51 |
| *** mylu has quit IRC | 02:52 | |
| *** alonma has quit IRC | 02:52 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add description when rename LUN in Huawei driver https://review.openstack.org/271603 | 02:53 |
| *** laughterwym has quit IRC | 02:53 | |
| *** laughterwym has joined #openstack-cinder | 02:54 | |
| *** laughter_ has joined #openstack-cinder | 02:55 | |
| *** mylu has joined #openstack-cinder | 02:55 | |
| *** laughterwym has quit IRC | 02:58 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:01 | |
| *** dongwenjuan has joined #openstack-cinder | 03:09 | |
| *** dongwenjuan has quit IRC | 03:14 | |
| *** salv-orlando has joined #openstack-cinder | 03:16 | |
| *** neophy has joined #openstack-cinder | 03:17 | |
| *** alonma has joined #openstack-cinder | 03:21 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Match the ip more accurately in Huawei driver https://review.openstack.org/280481 | 03:24 |
| *** alonma has quit IRC | 03:26 | |
| *** salv-orlando has quit IRC | 03:28 | |
| *** dims has quit IRC | 03:37 | |
| *** cfriesen has joined #openstack-cinder | 03:46 | |
| *** IlyaG has joined #openstack-cinder | 03:47 | |
| *** IlyaG has quit IRC | 03:51 | |
| *** davechen is now known as davechen_afk | 03:52 | |
| *** jwcroppe has joined #openstack-cinder | 03:58 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-cinder | 04:01 | |
| *** gouthamr_ has quit IRC | 04:01 | |
| *** david-lyle has quit IRC | 04:03 | |
| *** mylu has quit IRC | 04:16 | |
| *** mylu has joined #openstack-cinder | 04:17 | |
| *** mylu has quit IRC | 04:19 | |
| *** alonma has joined #openstack-cinder | 04:21 | |
| *** alonma has quit IRC | 04:22 | |
| *** alonma has joined #openstack-cinder | 04:22 | |
| *** frakt_ has quit IRC | 04:24 | |
| *** alonma has quit IRC | 04:26 | |
| *** frakt has joined #openstack-cinder | 04:26 | |
| *** mylu has joined #openstack-cinder | 04:28 | |
| *** mylu has quit IRC | 04:29 | |
| *** mylu has joined #openstack-cinder | 04:31 | |
| *** markvoelker has joined #openstack-cinder | 04:36 | |
| *** bkumar has joined #openstack-cinder | 04:38 | |
| *** markvoelker has quit IRC | 04:41 | |
| *** jgregor has quit IRC | 04:43 | |
| *** dave-mccowan has quit IRC | 04:47 | |
| *** jwcroppe has quit IRC | 04:51 | |
| *** IlyaG has joined #openstack-cinder | 04:51 | |
| *** jwcroppe has joined #openstack-cinder | 04:51 | |
| *** laughterwym has joined #openstack-cinder | 04:52 | |
| *** david-lyle has joined #openstack-cinder | 04:55 | |
| *** IlyaG has quit IRC | 04:56 | |
| *** laughter_ has quit IRC | 04:56 | |
| *** jwcroppe has quit IRC | 04:56 | |
| *** shyama has joined #openstack-cinder | 04:58 | |
| *** edtubill has joined #openstack-cinder | 04:59 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-cinder | 05:01 | |
| *** Poornima has joined #openstack-cinder | 05:01 | |
| *** edtubill has quit IRC | 05:08 | |
| *** subscope has joined #openstack-cinder | 05:09 | |
| *** IlyaG_ has joined #openstack-cinder | 05:11 | |
| openstackgerrit | Kaitlin Farr proposed openstack/cinder: Replace key manager with Castellan https://review.openstack.org/280492 | 05:15 |
| *** IlyaG_ has quit IRC | 05:15 | |
| *** davechen_afk is now known as davechen | 05:18 | |
| *** tpatil has quit IRC | 05:24 | |
| *** sgotliv has joined #openstack-cinder | 05:24 | |
| *** mylu has quit IRC | 05:24 | |
| *** mylu has joined #openstack-cinder | 05:25 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver (First patch) https://review.openstack.org/280497 | 05:35 |
| *** frakt has quit IRC | 05:39 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver (Second patch) https://review.openstack.org/280499 | 05:40 |
| *** frakt has joined #openstack-cinder | 05:41 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver (Third patch) https://review.openstack.org/280500 | 05:42 |
| *** mylu has quit IRC | 05:42 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver (Fourth patch) https://review.openstack.org/280501 | 05:43 |
| *** laughterwym has quit IRC | 05:44 | |
| *** ankit_ag has joined #openstack-cinder | 05:44 | |
| *** laughterwym has joined #openstack-cinder | 05:45 | |
| *** aagrawal has quit IRC | 05:47 | |
| *** zhonghua-lee has quit IRC | 05:49 | |
| *** jiangliucheng has quit IRC | 05:49 | |
| *** jiangliucheng has joined #openstack-cinder | 05:49 | |
| *** zhonghua-lee has joined #openstack-cinder | 05:50 | |
| *** mylu has joined #openstack-cinder | 05:51 | |
| *** mylu has quit IRC | 05:54 | |
| *** coolsvap|away is now known as coolsvap | 05:56 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 06:01 | |
| *** laughterwym has quit IRC | 06:03 | |
| *** laughterwym has joined #openstack-cinder | 06:03 | |
| *** laughter_ has joined #openstack-cinder | 06:07 | |
| *** laughterwym has quit IRC | 06:08 | |
| *** laughter_ has quit IRC | 06:09 | |
| *** alonma has joined #openstack-cinder | 06:09 | |
| *** laughterwym has joined #openstack-cinder | 06:09 | |
| *** laughterwym has quit IRC | 06:10 | |
| *** laughterwym has joined #openstack-cinder | 06:11 | |
| *** jiangliucheng has quit IRC | 06:12 | |
| *** jiangliucheng has joined #openstack-cinder | 06:12 | |
| *** alonma has quit IRC | 06:13 | |
| *** huanan_M has joined #openstack-cinder | 06:14 | |
| *** neophy has quit IRC | 06:15 | |
| *** alonma has joined #openstack-cinder | 06:16 | |
| *** huanan_L has quit IRC | 06:16 | |
| *** chhavi has joined #openstack-cinder | 06:17 | |
| *** alonma has quit IRC | 06:20 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Scaling backup service https://review.openstack.org/262395 | 06:21 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: manage_existing_get_size for VMDK driver https://review.openstack.org/276063 | 06:22 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: manage_existing for VMDK driver https://review.openstack.org/280513 | 06:22 |
| *** alonma has joined #openstack-cinder | 06:22 | |
| *** vgridnev_ has joined #openstack-cinder | 06:22 | |
| *** alonma has quit IRC | 06:27 | |
| *** alonma has joined #openstack-cinder | 06:28 | |
| *** ChubYann has quit IRC | 06:30 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: manage_existing_get_size for VMDK driver https://review.openstack.org/276063 | 06:32 |
| *** alonma has quit IRC | 06:33 | |
| *** subscope has quit IRC | 06:34 | |
| *** alonma has joined #openstack-cinder | 06:36 | |
| *** markvoelker has joined #openstack-cinder | 06:37 | |
| *** jiangliucheng has quit IRC | 06:37 | |
| *** david-lyle has quit IRC | 06:39 | |
| *** david-lyle has joined #openstack-cinder | 06:40 | |
| *** alonma has quit IRC | 06:41 | |
| *** alonma has joined #openstack-cinder | 06:42 | |
| *** markvoelker has quit IRC | 06:42 | |
| *** huanan_M has quit IRC | 06:42 | |
| *** salv-orlando has joined #openstack-cinder | 06:43 | |
| *** huanan_M has joined #openstack-cinder | 06:43 | |
| *** zzzeek has quit IRC | 06:44 | |
| *** salv-orlando has quit IRC | 06:46 | |
| *** alonma has quit IRC | 06:47 | |
| *** alonma has joined #openstack-cinder | 06:49 | |
| *** lpetrut has joined #openstack-cinder | 06:50 | |
| *** subscope has joined #openstack-cinder | 06:52 | |
| *** alonma has quit IRC | 06:54 | |
| *** subscope has quit IRC | 06:55 | |
| *** alonma has joined #openstack-cinder | 06:55 | |
| *** coolsvap is now known as coolsvap|away | 06:58 | |
| *** shyama has quit IRC | 06:58 | |
| *** shyama has joined #openstack-cinder | 06:59 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: manage_existing for VMDK driver https://review.openstack.org/280513 | 07:00 |
| *** alonma has quit IRC | 07:00 | |
| *** haomaiwa_ has quit IRC | 07:01 | |
| *** cfriesen has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:01 | |
| *** alonma has joined #openstack-cinder | 07:01 | |
| *** e0ne has joined #openstack-cinder | 07:02 | |
| *** alonma has quit IRC | 07:06 | |
| *** alonma has joined #openstack-cinder | 07:08 | |
| *** alonma has quit IRC | 07:12 | |
| *** shyama has quit IRC | 07:13 | |
| *** shyama has joined #openstack-cinder | 07:13 | |
| openstackgerrit | Alon Marx proposed openstack/cinder: Remove a vol in error state from a CG https://review.openstack.org/270460 | 07:13 |
| *** alonma has joined #openstack-cinder | 07:14 | |
| *** alonma has quit IRC | 07:18 | |
| *** alonma has joined #openstack-cinder | 07:20 | |
| *** alonma has quit IRC | 07:25 | |
| *** alonma has joined #openstack-cinder | 07:26 | |
| *** alonma has quit IRC | 07:31 | |
| *** rcernin has joined #openstack-cinder | 07:38 | |
| *** jbernard has quit IRC | 07:43 | |
| *** jbernard has joined #openstack-cinder | 07:43 | |
| *** ekarlso has quit IRC | 07:44 | |
| *** Anticimex has quit IRC | 07:45 | |
| *** yuriy_n17 has quit IRC | 07:45 | |
| *** aarefiev has quit IRC | 07:45 | |
| *** Julien-z_ has quit IRC | 07:46 | |
| *** kragniz has quit IRC | 07:46 | |
| *** aarefiev has joined #openstack-cinder | 07:46 | |
| *** kragniz has joined #openstack-cinder | 07:46 | |
| *** Anticimex has joined #openstack-cinder | 07:46 | |
| *** yuriy_n17 has joined #openstack-cinder | 07:48 | |
| *** alonma has joined #openstack-cinder | 07:49 | |
| *** arecknag has joined #openstack-cinder | 07:53 | |
| *** alonma has quit IRC | 07:54 | |
| *** alonma has joined #openstack-cinder | 07:56 | |
| *** ekarlso has joined #openstack-cinder | 07:57 | |
| *** sgotliv has quit IRC | 07:57 | |
| *** boichev has joined #openstack-cinder | 07:58 | |
| *** alonma has quit IRC | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 08:01 | |
| *** alonma has joined #openstack-cinder | 08:02 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: EMC ScaleIO - fix bug in extend volume https://review.openstack.org/274391 | 08:03 |
| *** alonma has quit IRC | 08:06 | |
| *** coolsvap|away is now known as coolsvap | 08:09 | |
| *** lpetrut has quit IRC | 08:10 | |
| *** ociuhandu has joined #openstack-cinder | 08:11 | |
| *** e0ne has quit IRC | 08:13 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Scaling backup service https://review.openstack.org/262395 | 08:15 |
| *** alonma has joined #openstack-cinder | 08:19 | |
| *** lpetrut has joined #openstack-cinder | 08:19 | |
| *** subscope has joined #openstack-cinder | 08:21 | |
| *** ociuhandu has quit IRC | 08:21 | |
| *** alonma has quit IRC | 08:23 | |
| *** alonma has joined #openstack-cinder | 08:25 | |
| *** coolsvap is now known as coolsvap|away | 08:26 | |
| *** lpetrut has quit IRC | 08:26 | |
| *** jwcroppe has joined #openstack-cinder | 08:29 | |
| *** salv-orlando has joined #openstack-cinder | 08:30 | |
| *** alonma has quit IRC | 08:30 | |
| *** liverpooler has joined #openstack-cinder | 08:30 | |
| *** cheneydc_ has joined #openstack-cinder | 08:30 | |
| *** cheneydc_ has quit IRC | 08:30 | |
| *** alonma has joined #openstack-cinder | 08:31 | |
| *** salv-orlando has quit IRC | 08:32 | |
| *** laughterwym has quit IRC | 08:33 | |
| *** liverpooler has quit IRC | 08:33 | |
| *** vgridnev_ has quit IRC | 08:33 | |
| *** liverpooler has joined #openstack-cinder | 08:34 | |
| *** jwcroppe has quit IRC | 08:34 | |
| *** vgridnev has joined #openstack-cinder | 08:34 | |
| *** liverpoo1er has joined #openstack-cinder | 08:34 | |
| *** alonma has quit IRC | 08:36 | |
| *** laughterwym has joined #openstack-cinder | 08:36 | |
| *** alonma has joined #openstack-cinder | 08:37 | |
| *** vgridnev has quit IRC | 08:38 | |
| *** vgridnev_ has joined #openstack-cinder | 08:39 | |
| *** laughterwym has quit IRC | 08:41 | |
| *** alonma has quit IRC | 08:42 | |
| *** vgridnev_ has quit IRC | 08:42 | |
| *** vgridnev has joined #openstack-cinder | 08:43 | |
| *** alonma has joined #openstack-cinder | 08:44 | |
| *** jordanP has joined #openstack-cinder | 08:44 | |
| *** vgridnev has quit IRC | 08:44 | |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Fix omission of request_ids returned to user https://review.openstack.org/280551 | 08:48 |
| *** alonma has quit IRC | 08:48 | |
| *** baojg has quit IRC | 08:49 | |
| *** subscope has quit IRC | 08:50 | |
| *** baojg has joined #openstack-cinder | 08:53 | |
| *** adrianofr has quit IRC | 08:53 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: doc: Fix wrong description about adding RESTful API https://review.openstack.org/273342 | 08:54 |
| *** adrianofr has joined #openstack-cinder | 08:55 | |
| *** alonma has joined #openstack-cinder | 08:56 | |
| *** haomaiwa_ has quit IRC | 09:01 | |
| *** alonma has quit IRC | 09:01 | |
| *** alonma has joined #openstack-cinder | 09:01 | |
| *** haomaiwang has joined #openstack-cinder | 09:01 | |
| *** subscope has joined #openstack-cinder | 09:04 | |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add request_ids attribute to resource objects https://review.openstack.org/257199 | 09:05 |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Provide consistency for Wrapper classes https://review.openstack.org/280558 | 09:05 |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add request_ids attribute to resource objects https://review.openstack.org/257195 | 09:05 |
| openstackgerrit | Ankit Agrawal proposed openstack/python-cinderclient: Add request_ids attribute to resource objects https://review.openstack.org/257187 | 09:05 |
| openstackgerrit | javeme proposed openstack/cinder-specs: Implement the oslo_db enginefacade proposal https://review.openstack.org/270601 | 09:06 |
| *** salv-orlando has joined #openstack-cinder | 09:08 | |
| *** e0ne has joined #openstack-cinder | 09:16 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/cinder: Let method CONF.set_override with enfore_type=true by default https://review.openstack.org/253328 | 09:19 |
| *** wewe0901 has joined #openstack-cinder | 09:20 | |
| *** klkumar has joined #openstack-cinder | 09:25 | |
| *** mvk has quit IRC | 09:27 | |
| *** al_indig_ has joined #openstack-cinder | 09:27 | |
| *** al_indigo has quit IRC | 09:29 | |
| *** ndipanov has joined #openstack-cinder | 09:33 | |
| *** Thelo has joined #openstack-cinder | 09:35 | |
| Thelo | #openstack-infra | 09:35 |
| *** tpsilva has joined #openstack-cinder | 09:37 | |
| *** ndipanov has quit IRC | 09:38 | |
| *** ndipanov has joined #openstack-cinder | 09:38 | |
| *** subscope has quit IRC | 09:40 | |
| *** lpetrut has joined #openstack-cinder | 09:42 | |
| *** ociuhandu has joined #openstack-cinder | 09:43 | |
| *** sborkows has joined #openstack-cinder | 09:45 | |
| *** sborkows has quit IRC | 09:45 | |
| *** subscope has joined #openstack-cinder | 09:47 | |
| *** davechen has left #openstack-cinder | 09:49 | |
| *** ociuhandu has quit IRC | 09:50 | |
| *** sgotliv has joined #openstack-cinder | 09:52 | |
| *** lpetrut has quit IRC | 09:53 | |
| *** ociuhandu has joined #openstack-cinder | 09:53 | |
| *** mvk has joined #openstack-cinder | 09:54 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port netapp dataontap driver to Python 3 https://review.openstack.org/280331 | 09:56 |
| *** chhavi has quit IRC | 09:56 | |
| haypo | geguileo: hi. you wrote "I'd rather have a patch that fixed more tests, but this is fine." in https://review.openstack.org/#/c/280154/ but this change is followed by https://review.openstack.org/#/c/280331/ which enables all all volume drivers (unit) tests on Python 3 | 09:58 |
| haypo | geguileo: you may see that the second patch on netapp is not trivial, so i chose to have two different changes | 09:58 |
| haypo | (usually, i also prefer to only modify one driver per change, because of cinder complex CIs :-)) | 09:58 |
| geguileo | haypo: I think that patch is easy to follow as well | 09:59 |
| geguileo | haypo: But as I said I'm ok with the patch | 09:59 |
| haypo | geguileo: "a patch that fixed more tests" did you expect that i enable all volume drivers with the first change? | 09:59 |
| haypo | it is what i did in a previous version of the patch, but it was a mistatke :-p | 10:00 |
| geguileo | haypo: Yep, that's what I was expecting | 10:00 |
| geguileo | haypo: I remember the "mistake" ;-) | 10:00 |
| *** haomaiwang has quit IRC | 10:01 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - SnapVX and other snapshot improvements https://review.openstack.org/279881 | 10:01 |
| *** haomaiwang has joined #openstack-cinder | 10:01 | |
| *** laughterwym has joined #openstack-cinder | 10:02 | |
| *** salv-orl_ has joined #openstack-cinder | 10:02 | |
| haypo | geguileo: well, as i wrote, my second patch enables "all" (volume drivers) unit tests | 10:03 |
| geguileo | haypo: I saw it :-) | 10:03 |
| haypo | geguileo: if all my pending py3 patches are merged, all cinder unit tests will on py3 | 10:03 |
| geguileo | haypo: Awesome!! | 10:03 |
| geguileo | haypo: I'll review that netapp patch in a little bit | 10:03 |
| haypo | geguileo: yestderday i finished to fix the remaining annoying tests | 10:03 |
| haypo | https://review.openstack.org/#/q/topic:bp/cinder-python3 | 10:04 |
| *** salv-orlando has quit IRC | 10:05 | |
| haypo | geguileo: you may approve this one, since the NSX CI verified the change. https://review.openstack.org/#/c/280241/ | 10:05 |
| *** cheneydc has quit IRC | 10:05 | |
| geguileo | haypo: I use this one to see which ones I haven't reviewed yet: https://review.openstack.org/#/q/topic:bp/cinder-python3+-reviewer:self+status:open | 10:05 |
| haypo | geguileo: well, or let smcginnis do it. as you want | 10:05 |
| haypo | geguileo: a, i never used "reviewer: self" filter | 10:06 |
| haypo | ah* | 10:06 |
| *** laughterwym has quit IRC | 10:06 | |
| *** sborkows has joined #openstack-cinder | 10:08 | |
| *** sborkows_ has joined #openstack-cinder | 10:08 | |
| *** andymaier has joined #openstack-cinder | 10:09 | |
| geguileo | haypo: I don't see the VNX CI passing in that patch :-P | 10:10 |
| openstackgerrit | LisaLi proposed openstack/os-brick: Add sheepdog support https://review.openstack.org/280588 | 10:11 |
| haypo | geguileo: oh wait, i saw "VMware NSX CI" | 10:11 |
| geguileo | haypo: Not the same thing ;-) | 10:11 |
| haypo | geguileo: NSX != VNX... sorry | 10:11 |
| haypo | haha | 10:11 |
| geguileo | np | 10:11 |
| *** haomaiwang has quit IRC | 10:11 | |
| haypo | too many *SX acronyms | 10:12 |
| openstackgerrit | LisaLi proposed openstack/os-brick: Add sheepdog support https://review.openstack.org/280588 | 10:15 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder-specs: Deleting volume metadata keys with a single request https://review.openstack.org/279202 | 10:21 |
| *** lpetrut has joined #openstack-cinder | 10:22 | |
| *** andymaier has quit IRC | 10:23 | |
| *** alyson_ has joined #openstack-cinder | 10:27 | |
| *** dosaboy has quit IRC | 10:38 | |
| *** dosaboy has joined #openstack-cinder | 10:41 | |
| *** subscope has quit IRC | 10:47 | |
| *** erlon has joined #openstack-cinder | 10:48 | |
| openstackgerrit | Merged openstack/cinder: Move integrated tests to 'functional' directory https://review.openstack.org/267801 | 10:51 |
| *** dims has joined #openstack-cinder | 10:51 | |
| Anticimex | we're using multi-backend (different Ceph pools), and i'm just playing with creating an image. | 10:51 |
| Anticimex | i found that if i do specify Type, the correct backend is picked. but if i do not specify Type, the result seems to be that the image is created in a random backend | 10:52 |
| Anticimex | is there anyway to set a default backend when using multi-backend cinder? | 10:52 |
| Anticimex | (liberty) | 10:53 |
| *** baojg has quit IRC | 10:55 | |
| Anticimex | i can haz read docs: default_volume_type | 10:58 |
| *** yhayashi has quit IRC | 10:58 | |
| andrei__ | do you have different pools for glance and cinder? | 11:03 |
| *** dave-mccowan has joined #openstack-cinder | 11:04 | |
| dulek | Anticimex: Isn't setting default_volume_type to one of your types working? | 11:06 |
| *** houming has quit IRC | 11:06 | |
| Anticimex | i'm testing now. i hope it will work. i asked here before reading through ref cinder.conf. :s | 11:07 |
| *** dims has quit IRC | 11:12 | |
| *** bkumar has quit IRC | 11:13 | |
| *** bkumar has joined #openstack-cinder | 11:14 | |
| *** ildikov_ has quit IRC | 11:15 | |
| *** bkumar4 has joined #openstack-cinder | 11:17 | |
| *** bkumar4 has quit IRC | 11:17 | |
| *** subscope has joined #openstack-cinder | 11:18 | |
| *** bkumar has quit IRC | 11:20 | |
| *** jwcroppe has joined #openstack-cinder | 11:23 | |
| *** dims has joined #openstack-cinder | 11:24 | |
| *** sdague has joined #openstack-cinder | 11:27 | |
| *** ociuhandu has quit IRC | 11:29 | |
| Anticimex | so default_volume_type does set a type for me, when I do not supply one, but then os-vol-host-attr = None on newly created images, rather than what i usually have | 11:38 |
| *** dims has quit IRC | 11:38 | |
| *** haomaiwang has joined #openstack-cinder | 11:38 | |
| *** sdague has quit IRC | 11:39 | |
| Anticimex | may be unrelated. investigating error on create | 11:41 |
| openstackgerrit | liuke proposed openstack/cinder: Huawei: Create snapshot have a log error https://review.openstack.org/280622 | 11:43 |
| *** sdague has joined #openstack-cinder | 11:44 | |
| *** ildikov_ has joined #openstack-cinder | 11:44 | |
| *** ccesario has quit IRC | 11:44 | |
| Anticimex | ok works, yay | 11:50 |
| *** smoriya_ has quit IRC | 11:51 | |
| *** laughterwym has joined #openstack-cinder | 11:53 | |
| *** ildikov_ has quit IRC | 11:59 | |
| *** haomaiwang has quit IRC | 12:01 | |
| *** haomaiwang has joined #openstack-cinder | 12:01 | |
| *** jwcroppe has quit IRC | 12:02 | |
| *** jwcroppe has joined #openstack-cinder | 12:02 | |
| *** ociuhandu has joined #openstack-cinder | 12:04 | |
| *** jwcroppe has quit IRC | 12:06 | |
| *** raildo-afk is now known as raildo | 12:11 | |
| openstackgerrit | liuke proposed openstack/cinder: Huawei: Create hypermetro the remote pool not exist lead to judgment failure https://review.openstack.org/280641 | 12:12 |
| *** dongwenjuan has joined #openstack-cinder | 12:12 | |
| *** mvk has quit IRC | 12:13 | |
| *** ericksonsantos has joined #openstack-cinder | 12:13 | |
| *** gcb has quit IRC | 12:13 | |
| *** gcb has joined #openstack-cinder | 12:14 | |
| *** mvk has joined #openstack-cinder | 12:15 | |
| *** raildo is now known as raildo-afk | 12:16 | |
| *** raildo-afk is now known as raildo | 12:20 | |
| *** raildo is now known as raildo-afk | 12:25 | |
| *** sheel has joined #openstack-cinder | 12:27 | |
| *** raildo-afk is now known as raildo | 12:31 | |
| *** ccesario has joined #openstack-cinder | 12:31 | |
| *** ccesario has quit IRC | 12:36 | |
| *** chhavi has joined #openstack-cinder | 12:38 | |
| *** archers has joined #openstack-cinder | 12:41 | |
| *** archers has quit IRC | 12:43 | |
| *** zhonghua-lee has quit IRC | 12:46 | |
| *** zhonghua-lee has joined #openstack-cinder | 12:46 | |
| *** Poornima has quit IRC | 12:51 | |
| *** annasort has quit IRC | 12:52 | |
| *** ildikov has joined #openstack-cinder | 12:52 | |
| *** boris-42 has joined #openstack-cinder | 12:56 | |
| *** ankit_ag has quit IRC | 12:57 | |
| *** jgregor has joined #openstack-cinder | 12:59 | |
| *** timcl has joined #openstack-cinder | 12:59 | |
| *** davechen has joined #openstack-cinder | 13:00 | |
| *** haomaiwang has quit IRC | 13:01 | |
| *** cfriesen has joined #openstack-cinder | 13:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 13:01 | |
| *** davechen has left #openstack-cinder | 13:01 | |
| *** jgregor has left #openstack-cinder | 13:02 | |
| *** mvk has quit IRC | 13:02 | |
| *** edmondsw has joined #openstack-cinder | 13:03 | |
| *** jgregor has joined #openstack-cinder | 13:03 | |
| *** julim has joined #openstack-cinder | 13:05 | |
| *** mvk has joined #openstack-cinder | 13:05 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls https://review.openstack.org/252458 | 13:07 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec in create volume calls to ovo https://review.openstack.org/246600 | 13:07 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO https://review.openstack.org/253033 | 13:07 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype) https://review.openstack.org/257305 | 13:07 |
| *** cfriesen has quit IRC | 13:10 | |
| *** obutenko has quit IRC | 13:13 | |
| *** wN has quit IRC | 13:16 | |
| *** ildikov has quit IRC | 13:18 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Roll back reservations quota in RPC if necessary https://review.openstack.org/263473 | 13:22 |
| *** [1]Thelo has joined #openstack-cinder | 13:22 | |
| *** takedakn has joined #openstack-cinder | 13:24 | |
| *** Thelo has quit IRC | 13:24 | |
| *** [1]Thelo is now known as Thelo | 13:24 | |
| *** sgotliv_ has joined #openstack-cinder | 13:24 | |
| *** sgotliv has quit IRC | 13:25 | |
| *** jgregor has quit IRC | 13:30 | |
| *** salv-orl_ has quit IRC | 13:32 | |
| *** ildikov has joined #openstack-cinder | 13:34 | |
| *** salv-orlando has joined #openstack-cinder | 13:36 | |
| *** markvoelker has joined #openstack-cinder | 13:39 | |
| *** diablo_rojo has joined #openstack-cinder | 13:41 | |
| *** EinstCrazy has joined #openstack-cinder | 13:42 | |
| *** dims has joined #openstack-cinder | 13:45 | |
| *** akerr has joined #openstack-cinder | 13:48 | |
| *** jgregor has joined #openstack-cinder | 13:49 | |
| *** rlrossit has joined #openstack-cinder | 13:50 | |
| *** davechen has joined #openstack-cinder | 13:50 | |
| *** davechen has left #openstack-cinder | 13:50 | |
| *** dims has quit IRC | 13:51 | |
| *** subscope has quit IRC | 13:53 | |
| *** haomaiwa_ has quit IRC | 14:01 | |
| *** haomaiwang has joined #openstack-cinder | 14:01 | |
| *** jwcroppe has joined #openstack-cinder | 14:01 | |
| *** cknight has joined #openstack-cinder | 14:02 | |
| *** raildo is now known as raildo-afk | 14:03 | |
| *** avishay has joined #openstack-cinder | 14:03 | |
| *** Julien-zte has joined #openstack-cinder | 14:04 | |
| *** changbl has quit IRC | 14:04 | |
| *** arch-nemesis has joined #openstack-cinder | 14:06 | |
| *** salv-orlando has quit IRC | 14:08 | |
| *** ccesario has joined #openstack-cinder | 14:08 | |
| *** mvk has quit IRC | 14:09 | |
| *** xyang1 has joined #openstack-cinder | 14:09 | |
| *** takedakn has quit IRC | 14:10 | |
| *** raildo-afk is now known as raildo | 14:11 | |
| *** Guest76507 is now known as med_ | 14:12 | |
| *** med_ has quit IRC | 14:12 | |
| *** med_ has joined #openstack-cinder | 14:12 | |
| *** arch-nemesis has quit IRC | 14:13 | |
| *** gouthamr has joined #openstack-cinder | 14:17 | |
| *** avishay has quit IRC | 14:18 | |
| *** porrua has joined #openstack-cinder | 14:20 | |
| *** mvk has joined #openstack-cinder | 14:21 | |
| *** subscope has joined #openstack-cinder | 14:22 | |
| *** EinstCra_ has joined #openstack-cinder | 14:25 | |
| diablo_rojo | Someone want to explain this new DocImpact mechanism to me again? | 14:26 |
| *** eharney has quit IRC | 14:26 | |
| dulek | diablo_rojo: On merge of a patch with DocImpact a new bug is created in Cinder. You should comment on it with enough information on how to expand docs, set status to triaged and move it to documentation project. | 14:27 |
| *** arch-nemesis has joined #openstack-cinder | 14:27 | |
| dulek | geguileo: Hi. Which part of the code raises your concerns on SIGTERM/SIGKILL support? Did I missed something? | 14:28 |
| *** manous has joined #openstack-cinder | 14:28 | |
| diablo_rojo | dulek: Okay that's what I thought. Do we also want to enforce that people explain why there is a DocImpact tag in the commit message? | 14:28 |
| dulek | geguileo: I wonder if I haven't missed the same thing in case of my SIGHUP support patches. | 14:28 |
| geguileo | dulek: We are not passing graceful parameter to super stop method | 14:28 |
| *** EinstCrazy has quit IRC | 14:28 | |
| dulek | diablo_rojo: I'm not sure, I think this requirement was criticized in one of the openstack-dev post. | 14:29 |
| dulek | geguileo: Thanks, I'm looking through. | 14:29 |
| diablo_rojo | dulek: Recently? Or do you by chance have the title of the email so I can go look up the ML thread? | 14:30 |
| geguileo | dulek: But the draining works fine, which is the most important part :-) | 14:30 |
| dulek | diablo_rojo: Certainly there was DocImpact in the subject. :D I think it was about the infra job that was supposed to check commit messages for that. If I recall correctly Sean Dague was the person criticizing it. | 14:31 |
| dulek | geguileo: I'm terrified when something that was not supposed to work actually works. | 14:31 |
| diablo_rojo | dulek: Interesting. I will go hunting for that. Thanks :) | 14:31 |
| dulek | geguileo: This is much worse that the other way around. ;) | 14:31 |
| *** klkumar has quit IRC | 14:32 | |
| *** mriedem_away has quit IRC | 14:32 | |
| geguileo | dulek: It's not much worse, because I believe it's worse if we kill all threads when stopping the service XD | 14:33 |
| dulek | geguileo: Okay, got it. oslo.service's stop has graceful=False, but oslo.messaging's stop will wait for greenthreads started by RPC server anyway. | 14:34 |
| geguileo | dulek: That's what I imagined would be happening XD | 14:35 |
| geguileo | dulek: Good to have confirmation :-) | 14:35 |
| dulek | geguileo: But I believe periodic tasks will be killed in that case. Which is bad… | 14:35 |
| geguileo | dulek: In which case? | 14:35 |
| *** zzzeek has joined #openstack-cinder | 14:35 | |
| dulek | In case of not passing graceful=True to oslo.service. | 14:35 |
| *** cfriesen has joined #openstack-cinder | 14:36 | |
| geguileo | dulek: We close periodic tasks manually before that call | 14:36 |
| dulek | I believe periodic tasks are tied with oslo.service, not oslo.messaging. | 14:36 |
| geguileo | s/close/stop | 14:36 |
| *** ildikov has quit IRC | 14:36 | |
| dulek | geguileo: Oh, right… :D | 14:36 |
| geguileo | But we have a bug, because we don't wait for them to complete (I will submit a patch to fix that) | 14:36 |
| dulek | geguileo: When do you plan to do that? I would prefer to get graceful shutdown right in Mitaka. This is an element of upgrades. | 14:38 |
| geguileo | dulek: I'll try to do it later today | 14:38 |
| openstackgerrit | Duncan Thomas proposed openstack/cinder: DO NOT MERGE https://review.openstack.org/280731 | 14:38 |
| geguileo | dulek: I'm trying to figure out if we have another problem in that code... | 14:38 |
| dulek | geguileo: Okay, thank you! Feel free to add me to the review, I'll double check. | 14:39 |
| openstackgerrit | Merged openstack/cinder: Disable Rally backup-related scenarios https://review.openstack.org/280359 | 14:39 |
| geguileo | dulek: Because we either have code we don't use or it's a bug | 14:39 |
| geguileo | XD | 14:39 |
| *** merooney has joined #openstack-cinder | 14:40 | |
| dulek | DuncanT: Do-not-merge patches won't get you an ATC status. :D | 14:42 |
| geguileo | dulek: Do you know if we should be removing the Service entry from the DB when we kill a serivce? | 14:42 |
| dulek | geguileo: Ah, I see your problem… service.py:266-274? | 14:43 |
| DuncanT | dulek: Sometimes my code is such that people give me things if I promise never to merge it... | 14:43 |
| geguileo | DuncanT: Yep | 14:43 |
| geguileo | DuncanT: Sorry, not for you XD | 14:43 |
| diablo_rojo | DuncanT: The content of the patch made me laugh out loud. | 14:43 |
| geguileo | dulek: Yep | 14:44 |
| dulek | I've never seen Cinder removing a service DB entry when stopping the service. | 14:44 |
| DuncanT | diablo_rojo: Have you not seen 'The rIng'? | 14:44 |
| diablo_rojo | DuncanT: Actually no, but I am aware of the principle. | 14:45 |
| dulek | geguileo: Buuut… Nova also have this piece of code, and they don't do it either. | 14:45 |
| geguileo | dulek: Oh, well, that code is never called, so that's what I would expect | 14:45 |
| geguileo | dulek: That's what I mean, we either have to remove it or make it work | 14:45 |
| geguileo | DuncanT: About the node draining | 14:46 |
| diablo_rojo | DuncanT: I promise to not +1 even though this is quite possibly the best patch I have ever seen. | 14:46 |
| dulek | geguileo: This would be an operational change. I would say we shouldn't modify current behavior. | 14:46 |
| geguileo | DuncanT: Since we can drain the nodes manually, do we still need the operation in the API? | 14:47 |
| geguileo | dulek: That was also my initial assesment | 14:47 |
| geguileo | dulek: I'll remove it and fix the other thing | 14:47 |
| dulek | geguileo: :) | 14:47 |
| *** ildikov has joined #openstack-cinder | 14:48 | |
| DuncanT | geguileo: I guess not... still not liking nodes not being nodes (and services not being services... it's now a service-cluster, right?) | 14:48 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Remove unused columns from Service ORM model https://review.openstack.org/280739 | 14:49 |
| *** mdenny has quit IRC | 14:49 | |
| dulek | geguileo: Speaking of patches related to cinder.service - ^. | 14:49 |
| geguileo | DuncanT: nodes are nodes and hosts are host groups (one host may have multiple nodes) | 14:49 |
| geguileo | dulek: That's the one we discussed yesterday, right? | 14:50 |
| dulek | geguileo: Right - I'm removing what's unused in current model. | 14:50 |
| geguileo | dulek: You need to remove those fields from the DB, so you need a migration | 14:51 |
| DuncanT | Unless you're talking abotu the scheduler where node==host even if you're running several | 14:51 |
| dulek | geguileo: These were added in Liberty. If I would remove it from the model I would make Liberty services unable to run on Mitaka's DB which would break ability to live upgrade. | 14:52 |
| geguileo | dulek: But column drops were done at the end of the upgrade, right? | 14:53 |
| geguileo | dulek: So it wouldn't be a problem | 14:53 |
| *** nkrinner has joined #openstack-cinder | 14:54 | |
| dulek | geguileo: Ah, right, I can create a first post-upgrade DB migration here… | 14:54 |
| *** aarefiev has quit IRC | 14:54 | |
| *** nkrinner has quit IRC | 14:54 | |
| geguileo | dulek: That's what I meant (even if I didn't say it clearly the first time) | 14:54 |
| sheel | Dear All Cinder members: | 14:54 |
| sheel | need some eyes for https://review.openstack.org/#/c/273938/ | 14:55 |
| *** mriedem has joined #openstack-cinder | 14:55 | |
| *** aarefiev has joined #openstack-cinder | 14:55 | |
| *** aarefiev has quit IRC | 14:55 | |
| *** aarefiev has joined #openstack-cinder | 14:55 | |
| dulek | geguileo: Sure, just wondering if this is worth the work (post-upgrade migrations infrastructure isn't started) that late in the release. We can just merge the normal DB migration in early N. | 14:56 |
| geguileo | dulek: But we'll probably forget to do it... | 14:57 |
| geguileo | dulek: I'd rather we did both things in the same release | 14:57 |
| geguileo | dulek: We keep finding things that should have been removed | 14:57 |
| geguileo | dulek: And if it's not even in the model it will be easier to forget | 14:57 |
| *** mtanino has joined #openstack-cinder | 14:57 | |
| *** coolsvap|away has quit IRC | 14:57 | |
| dulek | geguileo: I'm keeping a list of do-in-early-N stuff, but sure, what if I get hit by a bus? ;) | 14:58 |
| geguileo | dulek: Then we won't have to worry about it anymore!!! };-) | 14:58 |
| *** edtubill has joined #openstack-cinder | 14:58 | |
| * dulek starts to wonder on the idea... | 14:59 | |
| *** subscope has quit IRC | 14:59 | |
| *** dongwenjuan has quit IRC | 14:59 | |
| *** haypo has left #openstack-cinder | 14:59 | |
| dulek | geguileo: But with complexity added we will probably merge removal in N. This makes it the same thing - column gets removed in N. | 14:59 |
| *** nkrinner has joined #openstack-cinder | 14:59 | |
| *** baumann has joined #openstack-cinder | 14:59 | |
| *** nkrinner has quit IRC | 15:00 | |
| geguileo | dulek: At least add a TODO to the code :-P | 15:00 |
| dulek | geguileo: I'll even add FIXME. ;) | 15:00 |
| *** nkrinner has joined #openstack-cinder | 15:00 | |
| *** haomaiwang has quit IRC | 15:01 | |
| *** coolsvap|away has joined #openstack-cinder | 15:01 | |
| *** Julien-z_ has joined #openstack-cinder | 15:01 | |
| *** haomaiwang has joined #openstack-cinder | 15:01 | |
| *** EinstCra_ has quit IRC | 15:04 | |
| *** EinstCrazy has joined #openstack-cinder | 15:04 | |
| *** Julien-zte has quit IRC | 15:05 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Remove unused columns from Service ORM model https://review.openstack.org/280739 | 15:05 |
| *** subscope has joined #openstack-cinder | 15:06 | |
| *** klkumar has joined #openstack-cinder | 15:08 | |
| *** lprice1 has joined #openstack-cinder | 15:08 | |
| manous | hello | 15:09 |
| *** lprice has quit IRC | 15:10 | |
| manous | i use cinder and gluster; i update my gluster and cinder can't acces to it | 15:10 |
| manous | how can i check it ? | 15:10 |
| *** EinstCrazy has quit IRC | 15:12 | |
| *** knomura has joined #openstack-cinder | 15:12 | |
| *** EinstCrazy has joined #openstack-cinder | 15:14 | |
| *** mkoderer__ has quit IRC | 15:16 | |
| *** mkoderer__ has joined #openstack-cinder | 15:18 | |
| *** arch-nemesis has quit IRC | 15:21 | |
| *** dustins has joined #openstack-cinder | 15:24 | |
| *** tongli has joined #openstack-cinder | 15:25 | |
| *** EinstCrazy has quit IRC | 15:25 | |
| smcginnis | manous: Maybe double check your cinder.conf settings and make sure everything is still accurate? | 15:26 |
| manous | every thing it fine | 15:27 |
| manous | i dint change anything | 15:27 |
| DuncanT | manous: anything informative in cinder-volume.log? That's the first place to start looking | 15:28 |
| manous | i only make yum update and it's update gluster to 3.7.8 | 15:28 |
| manous | and since i can't create volume | 15:28 |
| smcginnis | manous: Yeah, the log DuncanT referred to should have some clues as to what is failing. | 15:29 |
| *** dustins_ has joined #openstack-cinder | 15:29 | |
| *** dustins has quit IRC | 15:29 | |
| manous | my cinder log if ok i think http://pastie.org/10724208 | 15:30 |
| DuncanT | manous: You need to follow the log in detail as you try each operation (create, attach, etc) | 15:30 |
| smcginnis | manous: So it's mounted. Is the issue that you can't actually write to it? | 15:31 |
| manous | i turn debug in cinder log to view detail | 15:31 |
| *** xyang has joined #openstack-cinder | 15:31 | |
| manous | in shell i can write to it | 15:32 |
| *** eharney has joined #openstack-cinder | 15:32 | |
| manous | but when i make "cinder create 2" | 15:32 |
| manous | i have nothing in log | 15:32 |
| smcginnis | manous: OK, so just provisioning new storage. Yeah, debug logging should have something. | 15:33 |
| manous | new storage ??? i have some disk on old | 15:34 |
| manous | and i want to retrieve it | 15:34 |
| *** mdenny has joined #openstack-cinder | 15:35 | |
| openstackgerrit | Jose Porrua proposed openstack/python-cinderclient: Avoid logging sensitive info in http requests. https://review.openstack.org/275755 | 15:36 |
| manous | i have a failed mode in debug | 15:36 |
| manous | however df show me it mount | 15:37 |
| manous | http://pastie.org/10724214 | 15:37 |
| *** jungleboyj has joined #openstack-cinder | 15:37 | |
| *** EinstCrazy has joined #openstack-cinder | 15:39 | |
| *** mdenny has quit IRC | 15:39 | |
| *** mdenny has joined #openstack-cinder | 15:39 | |
| *** salv-orl_ has joined #openstack-cinder | 15:40 | |
| *** salv-orl_ has quit IRC | 15:40 | |
| *** salv-orlando has joined #openstack-cinder | 15:41 | |
| *** nkrinner has quit IRC | 15:45 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: WIP - 3PAR: Update replication to v2.1 https://review.openstack.org/279307 | 15:47 |
| *** laughterwym has quit IRC | 15:51 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell: Failed vol create could leave dead volumes https://review.openstack.org/280343 | 15:53 |
| *** dave-mccowan has quit IRC | 15:53 | |
| *** arch-nemesis has joined #openstack-cinder | 15:54 | |
| *** xyang has quit IRC | 15:56 | |
| *** changbl has joined #openstack-cinder | 15:56 | |
| *** dustins_ is now known as dustins | 15:57 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** haomaiwang has joined #openstack-cinder | 16:01 | |
| DuncanT | manous: 32 just means mount failed | 16:01 |
| DuncanT | manous: No details. Check dmesg on that host maybe? | 16:02 |
| *** salv-orl_ has joined #openstack-cinder | 16:02 | |
| manous | mountpoint is ok | 16:03 |
| manous | on cinder i can write on it and replica is correct | 16:04 |
| manous | but when i make cinder create nothing happend | 16:04 |
| *** mdenny has quit IRC | 16:05 | |
| *** salv-orlando has quit IRC | 16:05 | |
| *** mdenny has joined #openstack-cinder | 16:05 | |
| *** huanan_M has quit IRC | 16:07 | |
| DuncanT | cinder is calling mount and it is failing. That is clear from your log. | 16:07 |
| *** huanan_M has joined #openstack-cinder | 16:07 | |
| *** Julien-z_ has quit IRC | 16:11 | |
| *** dave-mccowan has joined #openstack-cinder | 16:12 | |
| *** JoseMello has joined #openstack-cinder | 16:13 | |
| *** EinstCrazy has quit IRC | 16:15 | |
| *** EinstCrazy has joined #openstack-cinder | 16:16 | |
| *** dustins has quit IRC | 16:18 | |
| *** EinstCrazy has quit IRC | 16:20 | |
| *** mvk has quit IRC | 16:21 | |
| *** bkumar has joined #openstack-cinder | 16:22 | |
| *** daneyon has joined #openstack-cinder | 16:27 | |
| *** merooney has quit IRC | 16:29 | |
| *** jdurgin1 has joined #openstack-cinder | 16:30 | |
| *** dustins has joined #openstack-cinder | 16:31 | |
| *** sdague has quit IRC | 16:31 | |
| *** boris-42 has quit IRC | 16:34 | |
| *** sdague has joined #openstack-cinder | 16:34 | |
| *** timcl has quit IRC | 16:37 | |
| lprice1 | smcginnis: We have a consistency group patch that is about ready to move upstream for one of our NetApp drivers. When is the feature freeze deadline for Cinder? Just trying to figure out if it's feasilble to get it in. | 16:37 |
| *** rcernin has quit IRC | 16:37 | |
| smcginnis | lprice1: Feature freeze is at the end of the month: | 16:38 |
| smcginnis | lprice1: http://releases.openstack.org/mitaka/schedule.html | 16:38 |
| smcginnis | lprice1: If you get it out there soon, assuming things are good it shouldn't be a problem. | 16:38 |
| *** gouthamr has quit IRC | 16:39 | |
| lprice1 | smcginnis: That freeze requires that the patch be merged by that time, is that correct? | 16:39 |
| smcginnis | lprice1: Correct. | 16:39 |
| *** merooney has joined #openstack-cinder | 16:39 | |
| *** timcl has joined #openstack-cinder | 16:39 | |
| lprice1 | smcginnis: Okay, thanks | 16:39 |
| smcginnis | lprice1: np, good luck. | 16:40 |
| *** gouthamr has joined #openstack-cinder | 16:40 | |
| *** merooney has quit IRC | 16:45 | |
| *** manous has quit IRC | 16:46 | |
| *** klkumar has quit IRC | 16:51 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver (First patch) https://review.openstack.org/280497 | 16:52 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Add 'conf' param for TextGuruMeditation autorun setup https://review.openstack.org/280815 | 16:53 |
| *** eharney has quit IRC | 16:53 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Fix iSCSI Multipath https://review.openstack.org/280817 | 16:53 |
| *** subscope has quit IRC | 16:56 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver (Second patch) https://review.openstack.org/280499 | 16:57 |
| *** haomaiwang has quit IRC | 17:01 | |
| *** haomaiwang has joined #openstack-cinder | 17:01 | |
| *** asselin has quit IRC | 17:03 | |
| *** e0ne has quit IRC | 17:04 | |
| *** apoorvad has joined #openstack-cinder | 17:07 | |
| *** apoorvad has quit IRC | 17:08 | |
| *** eharney has joined #openstack-cinder | 17:09 | |
| *** cfriesen has quit IRC | 17:09 | |
| *** rcernin has joined #openstack-cinder | 17:09 | |
| *** apoorvad has joined #openstack-cinder | 17:10 | |
| *** asselin has joined #openstack-cinder | 17:10 | |
| *** arecknag has quit IRC | 17:10 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver (Second patch) https://review.openstack.org/280499 | 17:11 |
| *** IlyaG has joined #openstack-cinder | 17:13 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver (patch - 2/4) https://review.openstack.org/280499 | 17:13 |
| *** mylu has joined #openstack-cinder | 17:13 | |
| *** fthiagogv_ has quit IRC | 17:14 | |
| *** jungleboyj has quit IRC | 17:15 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver (patch - 3/4) https://review.openstack.org/280500 | 17:15 |
| *** leeantho has joined #openstack-cinder | 17:17 | |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver (patch - 4/4) https://review.openstack.org/280501 | 17:17 |
| *** IlyaG has quit IRC | 17:17 | |
| *** martyturner has joined #openstack-cinder | 17:18 | |
| *** mylu has quit IRC | 17:18 | |
| hemna_ | 4 patches | 17:19 |
| *** garthb has joined #openstack-cinder | 17:22 | |
| scottda | mc_nair: ping | 17:22 |
| mc_nair | scottda: hey - how goes it? | 17:23 |
| smcginnis | hemna_: http://makeagif.com/_VPxjd | 17:23 |
| scottda | mc_nair: ok, I've some unit test failures for nested quota fix on stable/liberty... | 17:23 |
| hemna_ | :P | 17:23 |
| scottda | mc_nair: I'll PM you. Perhaps they are something you've seen | 17:23 |
| mc_nair | scottda: ok, sounds good | 17:24 |
| *** lcurtis has joined #openstack-cinder | 17:27 | |
| *** jdurgin1 has quit IRC | 17:27 | |
| *** mylu has joined #openstack-cinder | 17:31 | |
| *** shyama has quit IRC | 17:32 | |
| *** dims has joined #openstack-cinder | 17:34 | |
| *** ociuhandu has quit IRC | 17:35 | |
| *** lpetrut has quit IRC | 17:36 | |
| *** jungleboyj has joined #openstack-cinder | 17:36 | |
| *** bkumar has quit IRC | 17:38 | |
| openstackgerrit | Michael Price proposed openstack/cinder: NetApp: E-Series remove snapshot limitations https://review.openstack.org/278515 | 17:45 |
| *** delattec has joined #openstack-cinder | 17:45 | |
| *** leeantho has quit IRC | 17:47 | |
| *** leeantho has joined #openstack-cinder | 17:47 | |
| *** e0ne has joined #openstack-cinder | 17:47 | |
| *** cdelatte has quit IRC | 17:48 | |
| *** mriedem is now known as mriedem_lunch | 17:49 | |
| *** leeantho has quit IRC | 17:52 | |
| *** abhishekk has joined #openstack-cinder | 17:52 | |
| knomura | smcginnis: Hi, I submit separated patches. (https://review.openstack.org/280497 https://review.openstack.org/280499 https://review.openstack.org/280500 https://review.openstack.org/280501) | 17:55 |
| abhishekk | DuncanT: hi, could you please review patch https://review.openstack.org/280558 and its dependent patches when you get time? | 17:58 |
| *** leeantho has joined #openstack-cinder | 17:58 | |
| abhishekk | DuncanT: thank you.. | 17:58 |
| *** kfarr has joined #openstack-cinder | 17:59 | |
| *** merooney has joined #openstack-cinder | 17:59 | |
| *** shyama has joined #openstack-cinder | 18:01 | |
| *** haomaiwang has quit IRC | 18:01 | |
| *** delattec has quit IRC | 18:01 | |
| *** haomaiwang has joined #openstack-cinder | 18:01 | |
| *** delattec has joined #openstack-cinder | 18:03 | |
| *** vgridnev has joined #openstack-cinder | 18:03 | |
| *** mriedem_lunch is now known as mriedem | 18:06 | |
| *** kfarr has quit IRC | 18:07 | |
| *** abhishekk has quit IRC | 18:07 | |
| *** subscope has joined #openstack-cinder | 18:07 | |
| *** kfarr has joined #openstack-cinder | 18:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/280857 | 18:12 |
| *** mudassirlatif has joined #openstack-cinder | 18:14 | |
| *** fernnest has quit IRC | 18:19 | |
| *** Lee1092 has quit IRC | 18:22 | |
| openstackgerrit | Patrick East proposed openstack/cinder: WIP: Update Pure replication to cheesecake https://review.openstack.org/276981 | 18:23 |
| *** chhavi has quit IRC | 18:23 | |
| *** mylu has quit IRC | 18:24 | |
| *** sgotliv_ has quit IRC | 18:25 | |
| *** david-lyle has quit IRC | 18:27 | |
| *** ntpttr has joined #openstack-cinder | 18:28 | |
| *** shyama has quit IRC | 18:30 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Wait for periodic tasks to stop on exit https://review.openstack.org/280873 | 18:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove unused kill method from Service https://review.openstack.org/280874 | 18:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Propoerly call oslo's Service stop and wait https://review.openstack.org/280875 | 18:31 |
| geguileo | dulek: ^^ Those are the patches we discussed | 18:31 |
| dulek | geguileo: Thanks! | 18:32 |
| *** manous has joined #openstack-cinder | 18:33 | |
| geguileo | dulek: np, thank you for the discussion earlier :-) | 18:33 |
| *** e0ne has quit IRC | 18:33 | |
| *** ebalduf has joined #openstack-cinder | 18:34 | |
| dulek | geguileo: BTW - you may write a patch removing kill in Nova if you want to have this glorious notion of having a commit there. ;) | 18:34 |
| *** mriedem has quit IRC | 18:35 | |
| geguileo | dulek: I think I already have a commit in Nova XD | 18:35 |
| *** david-lyle has joined #openstack-cinder | 18:35 | |
| dulek | :) | 18:36 |
| *** jordanP has quit IRC | 18:39 | |
| *** knomura has quit IRC | 18:42 | |
| *** mriedem has joined #openstack-cinder | 18:42 | |
| *** lpetrut has joined #openstack-cinder | 18:44 | |
| mriedem | are there any netapp peeps here? | 18:47 |
| dulek | geguileo: Isn't 280875 (graceful=True) also solving the periodic tasks problem? | 18:47 |
| jgriffith | mriedem: kinda :) | 18:47 |
| * jgriffith isn't used to that yet | 18:47 | |
| geguileo | dulek: I don't think it is | 18:47 |
| akerr | mriedem: depends on the question | 18:48 |
| mriedem | any major known issues with netapp and multipath? | 18:48 |
| geguileo | dulek: I'll try to check it | 18:48 |
| * dulek is diving into oslo.service again. | 18:48 | |
| *** fernnest has joined #openstack-cinder | 18:48 | |
| akerr | mriedem: only with Ubuntu 14.04, but thats more a problem with Ubuntu than the NetApp driver | 18:48 |
| mriedem | ok, too broad a question anyway, sorry | 18:52 |
| *** ChubYann has joined #openstack-cinder | 18:54 | |
| *** garthb_ has joined #openstack-cinder | 18:55 | |
| *** garthb has quit IRC | 18:57 | |
| *** bardia has joined #openstack-cinder | 18:57 | |
| *** porrua has quit IRC | 19:00 | |
| *** haomaiwang has quit IRC | 19:01 | |
| *** porrua has joined #openstack-cinder | 19:01 | |
| *** porrua has quit IRC | 19:01 | |
| *** haomaiwang has joined #openstack-cinder | 19:01 | |
| *** porrua has joined #openstack-cinder | 19:02 | |
| smcginnis | jgriffith: Hey, missed you comment yesterday about the recheck trigger. | 19:02 |
| smcginnis | jgriffith: Is that documented somewhere to use the "recheck-CINAME"? | 19:03 |
| *** jungleboyj has quit IRC | 19:03 | |
| *** martyturner has quit IRC | 19:06 | |
| dulek | geguileo: To me it looks like this won't wait for periodic tasks - only for the looping call used to report from the service. | 19:06 |
| geguileo | dulek: I didn't expect to wait for the periodic tasks, because we had to stop them separately | 19:10 |
| geguileo | s/expect/expect it | 19:10 |
| geguileo | So it makes sense if we stop them on their own that we have to wait for them as well | 19:10 |
| dulek | geguileo: Give me one more minute… | 19:11 |
| geguileo | dulek: As long as you need | 19:11 |
| geguileo | :-) | 19:11 |
| *** mylu has joined #openstack-cinder | 19:11 | |
| *** mtanino__ has quit IRC | 19:12 | |
| dulek | geguileo: Okay, it will wait for them to finish. oslo_service.periodic_tasks is crazy complicated and I couldn't find info on if it is run in a new greenthread. | 19:13 |
| *** timcl1 has joined #openstack-cinder | 19:13 | |
| dulek | geguileo: Now I see that it's the cinder.service.Service that have it as looping call. | 19:13 |
| *** timcl1 has quit IRC | 19:13 | |
| dulek | s/have it/runs it | 19:13 |
| dulek | geguileo: Which is nice as with your patch we're waiting for all the looping calls to finish. | 19:14 |
| *** vgridnev has quit IRC | 19:14 | |
| *** jungleboyj has joined #openstack-cinder | 19:14 | |
| *** timcl1 has joined #openstack-cinder | 19:15 | |
| *** eharney has quit IRC | 19:17 | |
| *** timcl has quit IRC | 19:17 | |
| geguileo | dulek: Yes, that's the idea | 19:18 |
| *** angela-s has joined #openstack-cinder | 19:19 | |
| *** mylu has quit IRC | 19:20 | |
| *** mylu has joined #openstack-cinder | 19:21 | |
| openstackgerrit | Merged openstack/cinder: Port API contribs to Python 3 https://review.openstack.org/277861 | 19:22 |
| openstackgerrit | Merged openstack/cinder: Port API v1 and v2 to Python 3 https://review.openstack.org/277835 | 19:24 |
| *** martyturner has joined #openstack-cinder | 19:28 | |
| *** merooney has quit IRC | 19:30 | |
| *** merooney has joined #openstack-cinder | 19:32 | |
| *** dulek has left #openstack-cinder | 19:33 | |
| *** dulek has joined #openstack-cinder | 19:34 | |
| dulek | geguileo: +1 three times. Thanks for fixing that! | 19:34 |
| *** eharney has joined #openstack-cinder | 19:37 | |
| *** mylu has quit IRC | 19:40 | |
| *** mylu has joined #openstack-cinder | 19:43 | |
| hemna_ | patrickeast, https://review.openstack.org/#/c/277606 I just tested this guy against my array | 19:44 |
| hemna_ | and it looked good | 19:44 |
| hemna_ | not sure if you wanted to look at it before +A'ing it. | 19:44 |
| hemna_ | I'm good with it. | 19:44 |
| patrickeast | hemna_: sweet, i was going to try it out this afternoon but i'm pretty ok with it if it works on yours... afaik it isn't anything array specific | 19:45 |
| *** IlyaG has joined #openstack-cinder | 19:46 | |
| hemna_ | yah it's not specific to an array really. It's udev | 19:46 |
| *** mylu has quit IRC | 19:53 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Permit volume type operations for policy authorized users https://review.openstack.org/274204 | 19:54 |
| *** merooney has quit IRC | 20:00 | |
| *** martyturner has quit IRC | 20:00 | |
| *** haomaiwang has quit IRC | 20:01 | |
| *** martyturner has joined #openstack-cinder | 20:01 | |
| *** 21WAA3UQL has joined #openstack-cinder | 20:01 | |
| smcginnis | hemna_: How's this one going? https://review.openstack.org/#/c/280817/ | 20:02 |
| smcginnis | hemna_: looks like just one was missed. | 20:02 |
| *** e0ne has joined #openstack-cinder | 20:02 | |
| *** jwang_ has joined #openstack-cinder | 20:04 | |
| *** e0ne has quit IRC | 20:05 | |
| *** timcl has joined #openstack-cinder | 20:05 | |
| *** martyturner has quit IRC | 20:06 | |
| *** ntpttr has quit IRC | 20:06 | |
| *** merooney has joined #openstack-cinder | 20:06 | |
| *** jwang has quit IRC | 20:08 | |
| *** timcl1 has quit IRC | 20:08 | |
| *** martyturner has joined #openstack-cinder | 20:08 | |
| *** jungleboyj has quit IRC | 20:11 | |
| jgriffith | smcginnis: nah, it's not... it was a suggestion that I made a while back and I was told that "this wasn't the wild wild west" anymore | 20:14 |
| jgriffith | smcginnis: :) | 20:14 |
| jgriffith | not even sure what that meant... but anyway. I thought it seemed like a descent proposal to just ask people to use that convention | 20:14 |
| jgriffith | smcginnis: then you don't have to go track down a spread-sheet somewhere | 20:15 |
| patrickeast | jgriffith: that sounds way to reasonable, no way it will work :p | 20:15 |
| jgriffith | patrickeast: no kidding eh! | 20:16 |
| smcginnis | jgriffith: I would definitely prefer to standardize. | 20:16 |
| jgriffith | patrickeast: why do something reasonable and easy when you can go convoluted and control *all the things* | 20:16 |
| patrickeast | jgriffith: exactly! | 20:16 |
| jgriffith | smcginnis: it seems to me like it would make life simpoler | 20:16 |
| smcginnis | jgriffith: The only thing I have seen was I think on the main 3rd party page that said "recheck name" | 20:16 |
| smcginnis | jgriffith: But I don't like that that would also recheck Jenkins. | 20:17 |
| patrickeast | smcginnis: last i saw they removed that | 20:17 |
| jgriffith | smcginnis: yeah... that's the problem | 20:17 |
| smcginnis | patrickeast: Just noticed that. Good! | 20:17 |
| patrickeast | scottda: it just says they should respond to 'recheck' which triggers everything | 20:17 |
| jgriffith | smcginnis: the raw "recheck" will trigger Gate every time | 20:17 |
| patrickeast | jgriffith: smcginnis: lets just make it a cinder standard... update the wiki and bug folks at the meeting | 20:17 |
| jgriffith | patrickeast: That's my vote | 20:17 |
| smcginnis | So I added a meeting agenda item. Just so I can "request" that CI maintainers suppose that if they can. | 20:17 |
| jgriffith | patrickeast: smcginnis but I'll let you guys incur the wrath this time | 20:18 |
| jgriffith | :) | 20:18 |
| *** garthb_ has quit IRC | 20:18 | |
| *** garthb__ has joined #openstack-cinder | 20:18 | |
| smcginnis | patrickeast: +1 Yes, I will get it on our wiki so it's "official". | 20:18 |
| smcginnis | jgriffith: I'll have my Khhaaaaaaan moment. | 20:18 |
| *** david-lyle has quit IRC | 20:18 | |
| jgriffith | smcginnis: LOL!!!!! | 20:19 |
| *** ebalduf has quit IRC | 20:19 | |
| smcginnis | I have a feeling I confuse international folks with some references. ;) | 20:19 |
| jgriffith | haha | 20:20 |
| * jgriffith frantically google khhhaaaaaan in the background :) | 20:20 | |
| jgriffith | just kidding | 20:20 |
| patrickeast | lol | 20:20 |
| smcginnis | lol | 20:20 |
| patrickeast | could start a recommended watch list for cinder, sort of a ramp up program to understand the references | 20:21 |
| thingee | so I'm on a train speeding to london. taxi cab hit my head with his trunk | 20:21 |
| smcginnis | patrickeast: I like it! | 20:21 |
| smcginnis | thingee: You mean boot? :) | 20:21 |
| thingee | ops meet up was good | 20:21 |
| smcginnis | thingee: Nice! Wish I could have gone. | 20:22 |
| smcginnis | thingee: Seriously though - you're not talking really hit your head, are you? | 20:22 |
| thingee | he really did. it was accident, but I do have a nice bump on my head | 20:22 |
| smcginnis | Ouch | 20:23 |
| Swanson | "you're not talking really hit your head, are you?"... It is a language like english but different... | 20:23 |
| smcginnis | Swanson: :P | 20:24 |
| *** sgotliv_ has joined #openstack-cinder | 20:25 | |
| *** ntpttr has joined #openstack-cinder | 20:25 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - Recreating SG when it has been deleted https://review.openstack.org/279900 | 20:26 |
| *** jwang__ has joined #openstack-cinder | 20:28 | |
| *** lpetrut has quit IRC | 20:29 | |
| *** manous has quit IRC | 20:30 | |
| *** jwang has joined #openstack-cinder | 20:30 | |
| *** jwang_ has quit IRC | 20:32 | |
| hemna_ | smcginnis, I just started fixing the unit tests before lunch | 20:32 |
| hemna_ | I'll get 'er done in a bit | 20:32 |
| smcginnis | hemna_: Cool, thanks! | 20:32 |
| *** asselin_ has quit IRC | 20:33 | |
| *** jwang__ has quit IRC | 20:33 | |
| smcginnis | hemna_: Thinking just these two yet: https://review.openstack.org/280817 and https://review.openstack.org/280002 | 20:33 |
| *** rlrossit has quit IRC | 20:33 | |
| hemna_ | ok | 20:33 |
| *** mylu has joined #openstack-cinder | 20:33 | |
| *** jungleboyj has joined #openstack-cinder | 20:33 | |
| hemna_ | sounds good | 20:34 |
| Swanson | Got one that mostly shows up in our CI. Simple fix. https://review.openstack.org/#/c/280343/ | 20:36 |
| *** timcl has left #openstack-cinder | 20:37 | |
| *** ebalduf has joined #openstack-cinder | 20:38 | |
| *** timcl has joined #openstack-cinder | 20:38 | |
| *** jwang_ has joined #openstack-cinder | 20:38 | |
| angela-s | hey guys, need some help. In zoning lookup service, we use ssh_utils, which uses paramiko. As of Sunday morn, started getting error from rsa.verify. Anyone else see this? | 20:40 |
| smcginnis | angela-s: There was a thread on the ML about some libraries switching to pycryptodome that was incompatible with some calls. | 20:40 |
| angela-s | getting this exception NotImplementedError("Use module Crypto.Signature.pkcs1_15 instead" | 20:40 |
| * smcginnis looks up link... | 20:41 | |
| angela-s | ok, will look | 20:41 |
| *** mylu has quit IRC | 20:41 | |
| smcginnis | angela-s: Maybe related? http://lists.openstack.org/pipermail/openstack-dev/2016-February/086509.html | 20:41 |
| smcginnis | https://github.com/paramiko/paramiko/issues/637 | 20:41 |
| *** subscope has quit IRC | 20:42 | |
| *** jwang has quit IRC | 20:42 | |
| *** mudassirlatif has quit IRC | 20:43 | |
| angela-s | smcginnis: could be the same thing, argh, so zoning is broken until this is resolved with the pkg installation deps | 20:45 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Fix iSCSI Multipath https://review.openstack.org/280817 | 20:46 |
| smcginnis | angela-s: :( | 20:46 |
| smcginnis | angela-s: I didn't follow all of it. Do you know if there is a change in the works? | 20:46 |
| angela-s | smcginnis: from the thread, it doesn't look like there is a plan yet | 20:47 |
| smcginnis | dims: ping | 20:47 |
| *** jungleboyj has quit IRC | 20:50 | |
| *** garthb__ has quit IRC | 20:51 | |
| dims | smcginnis : pong | 20:51 |
| smcginnis | dims: Hey! Sorry to bother you. Wondering about the pycryptodome thread. | 20:52 |
| smcginnis | dims: We have a driver failing with a crypto error since Sunday. Wondering if it's related/ | 20:52 |
| dims | smcginnis : probably is...check if pycryptodome got installed by accident there | 20:52 |
| smcginnis | dims: Do you know if NotImplementedError("Use module Crypto.Signature.pkcs1_15 instead") is one of those dome/non-dome things? | 20:53 |
| smcginnis | angela-s: Do you capture pip-freeze in your CI output? | 20:53 |
| dims | smcginnis : i don't see a reference in our repo - http://codesearch.openstack.org/?q=pkcs1_15&i=nope&files=&repos= | 20:53 |
| smcginnis | dims: I think the exception text is coming from one of the libraries. But it's not everyone using paramiko, so a little strange. | 20:54 |
| smcginnis | dims: But sounds like that's not one of the errors you had seen? | 20:54 |
| smcginnis | dims: Don't want to pull you in to this. | 20:55 |
| smcginnis | dims: Just thought I would see if that was something obvious to you. | 20:55 |
| dims | smcginnis : y i had not seen that as i just look at the CI results we have in the community. however see here - https://github.com/Legrandin/pycryptodome/tree/master/lib/Crypto/Signature and https://github.com/dlitz/pycrypto/tree/master/lib/Crypto/Signature - the set of things available are different | 20:55 |
| smcginnis | dims: Also curious, that ML thread didn't have a conclusion on what we need to do. Do you know if there is a plan going forward? | 20:56 |
| dims | so it's quite possible angela-s is hitting pycryptodome | 20:56 |
| smcginnis | dims: OK, thanks! | 20:56 |
| dims | smcginnis : glance, nova have merged the "tolerate" either one patch from me | 20:56 |
| angela-s | smcginnis: unfortunately no | 20:56 |
| smcginnis | dims: I saw you posted and abandoned one to cinder. I need to look, but were there a lot of issues with it? | 20:57 |
| *** IlyaG_ has joined #openstack-cinder | 20:57 | |
| *** jwang__ has joined #openstack-cinder | 20:58 | |
| dims | smcginnis : all the voting jobs were green when i threw in pycryptodome - https://review.openstack.org/#/c/279978/ | 20:58 |
| dims | smcginnis : so cinder was good in my book | 20:58 |
| smcginnis | dims: OK, thanks! | 20:59 |
| *** IlyaG has quit IRC | 21:00 | |
| *** 21WAA3UQL has quit IRC | 21:01 | |
| *** jwang_ has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-cinder | 21:01 | |
| *** raildo is now known as raildo-afk | 21:01 | |
| *** ndipanov has quit IRC | 21:03 | |
| *** apoorvad has quit IRC | 21:04 | |
| sheel | dulek: hey | 21:04 |
| sheel | dulek: you there | 21:04 |
| sheel | dulek: openstack cp meeting.. reminder :) | 21:04 |
| dulek | sheel: Thanks, I'm there. :) | 21:05 |
| sheel | dulek:great | 21:05 |
| *** sdague has quit IRC | 21:07 | |
| *** asselin_ has joined #openstack-cinder | 21:12 | |
| *** apoorvad has joined #openstack-cinder | 21:12 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - SSl connection is not picking up values https://review.openstack.org/280933 | 21:17 |
| *** garthb has joined #openstack-cinder | 21:24 | |
| *** rlrossit has joined #openstack-cinder | 21:25 | |
| *** martyturner has quit IRC | 21:27 | |
| openstackgerrit | Merged openstack/os-brick: Lun id's > 255 should be converted to hex https://review.openstack.org/277606 | 21:29 |
| patrickeast | hemna_: ruh roh, i'm seeing a pretty consistent leftover iscsi session after disconnecting volumes using the latest os-brick (with and without the mp fix you have up) | 21:30 |
| hemna_ | :( | 21:30 |
| hemna_ | I don't think we changed the disconnect code much at all though | 21:30 |
| patrickeast | yea thats what i was thinking | 21:31 |
| patrickeast | i also could have sworn i checked that just a few days ago on another brick patch and things were ok | 21:31 |
| hemna_ | so after all targets are removed, you still see a live iscsi session? | 21:31 |
| hemna_ | iscsiadm -m session | 21:31 |
| patrickeast | yea the multipath device is gone, and 1/4 of the sessions is still there | 21:32 |
| * patrickeast is digging up the logs | 21:33 | |
| hemna_ | yah the patch diablo_rojo put in didn't touch disconnect | 21:33 |
| patrickeast | yea, should be something else | 21:34 |
| patrickeast | could even be my environment or somethin | 21:34 |
| hemna_ | so I'm confused | 21:34 |
| hemna_ | maybe it's a new bug ? | 21:34 |
| patrickeast | could be | 21:34 |
| diablo_rojo | Correct. I didn't really touch that at all. | 21:35 |
| *** martyturner has joined #openstack-cinder | 21:35 | |
| diablo_rojo | patrickeast: SO it's getting stuck in detaching state? or..? | 21:36 |
| patrickeast | diablo_rojo: nah as far as cinder/nova is concerned everything is great | 21:37 |
| patrickeast | diablo_rojo: hemna_ : http://paste.openstack.org/show/487180/ | 21:37 |
| *** dims has quit IRC | 21:37 | |
| patrickeast | just glancing at it there are only 3 logouts, i think should be 4 | 21:37 |
| hemna_ | k I'm not sure what I'm lookin at | 21:38 |
| *** JoseMello has quit IRC | 21:38 | |
| diablo_rojo | patrickeast: hemna_ Yeah kinda greek to me lol | 21:38 |
| patrickeast | oh | 21:39 |
| patrickeast | thats the log of the os-brick/cinder disconnect_volume | 21:39 |
| hemna_ | c-vol ? | 21:39 |
| patrickeast | yep | 21:39 |
| *** lpetrut has joined #openstack-cinder | 21:39 | |
| patrickeast | starts with the lock for connect/disconnect | 21:40 |
| patrickeast | you can see it check the multipath stuff | 21:40 |
| patrickeast | and figure out what paths to clean up | 21:40 |
| hemna_ | ok so anything os-brick in c-vol is copy volume <--> Image operations | 21:40 |
| patrickeast | yep, same thing in the n-cpu | 21:40 |
| patrickeast | the log output comes from os-brick | 21:40 |
| hemna_ | so I see 3 calls to disconnect from the iscsi portal | 21:42 |
| *** cknight has quit IRC | 21:43 | |
| *** akerr has quit IRC | 21:44 | |
| Swanson | Neato. I wonder if this is the bug my test guy reported 1/2 a year ago and which I've been meaning to look at ever since. | 21:45 |
| hemna_ | https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connector.py#L1225 | 21:45 |
| hemna_ | I think that's where it's being called | 21:45 |
| *** Suyash has joined #openstack-cinder | 21:45 | |
| hemna_ | this iSCSI code is painful | 21:46 |
| hemna_ | I guess I'm going to have to finally clean it up | 21:46 |
| patrickeast | yep thats the loop | 21:46 |
| hemna_ | we have calls in there to do multipath -ll still | 21:46 |
| hemna_ | :( | 21:46 |
| patrickeast | guess it means that the ips_iqns are missing one :/ | 21:46 |
| hemna_ | it could be | 21:46 |
| hemna_ | should dump that out and see what it says | 21:47 |
| patrickeast | yup yup, shotgunning in some printfs | 21:47 |
| hemna_ | https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connector.py#L1076-L1086 | 21:48 |
| hemna_ | that's where that list is built | 21:48 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/280857 | 21:50 |
| *** dims has joined #openstack-cinder | 21:50 | |
| dulek | sheel, jgregor: Gotta go now, I'll catch you later to discuss next steps with 4-byte unicode. | 21:53 |
| sheel | dulek: its great | 21:53 |
| sheel | dulek: thank you for your support in migration queries.. | 21:53 |
| jgregor | dulek: Sounds good! Thanks for the support. | 21:54 |
| *** mriedem has quit IRC | 21:55 | |
| *** IlyaG_ has quit IRC | 21:55 | |
| angela-s | smcginnis: my CI host had pycrypto 2.6.1 installed, so I removed it, and looks like it's back in business now. | 21:55 |
| smcginnis | angela-s: Huh, weird. Awesome though! Glad it was something as simple as that. | 21:56 |
| *** mudassirlatif has joined #openstack-cinder | 21:58 | |
| *** porrua has quit IRC | 21:58 | |
| *** tongli has quit IRC | 21:59 | |
| *** haomaiwang has quit IRC | 22:01 | |
| *** haomaiwang has joined #openstack-cinder | 22:01 | |
| *** ebalduf has quit IRC | 22:03 | |
| *** salv-orlando has joined #openstack-cinder | 22:03 | |
| openstackgerrit | Merged openstack/os-brick: Add missing release notes https://review.openstack.org/280002 | 22:05 |
| hemna_ | patrickeast, would be nice to see which iSCSI portal a lun came from | 22:05 |
| hemna_ | hrmm | 22:05 |
| *** salv-orl_ has quit IRC | 22:05 | |
| hemna_ | patrickeast, are you sending "target_portals" in the connection_properties? | 22:06 |
| *** tpsilva has quit IRC | 22:06 | |
| hemna_ | if not os-brick issues an iscsiadm sendtargets | 22:07 |
| *** mylu has joined #openstack-cinder | 22:07 | |
| *** jdurgin has quit IRC | 22:07 | |
| *** jdurgin has joined #openstack-cinder | 22:08 | |
| *** timcl has quit IRC | 22:11 | |
| patrickeast | hemna_: sry for the delay, got side-tracked with foosball.... anyway, i added in some extra logging like so: http://paste.openstack.org/show/487186/ which gives a log like: http://paste.openstack.org/show/487190/ | 22:11 |
| hemna_ | hehe | 22:11 |
| hemna_ | foosball.....bug.....foosball....bug....yah foosball. | 22:12 |
| patrickeast | hemna_: it looks like it sees all of them, and is filtering one out in https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connector.py#L1083-L1086 | 22:12 |
| smcginnis | patrickeast: I'll remember that as we hit the library deadline and I'm scambling to get os-brick out. :P | 22:12 |
| smcginnis | jk | 22:12 |
| patrickeast | lol | 22:12 |
| hemna_ | :) | 22:13 |
| hemna_ | patrickeast, so it's either 1 of 2 things | 22:13 |
| hemna_ | it's not in all_ips_iqns | 22:13 |
| hemna_ | or it's not in 'entries' | 22:13 |
| *** mylu has quit IRC | 22:14 | |
| patrickeast | hemna_: yep, looks like all 4 are in the 'all_ips_iqns' | 22:15 |
| patrickeast | hemna_: not sure why there are two in entries | 22:15 |
| hemna_ | only 2 items in entries | 22:15 |
| hemna_ | and 10.0.1.10 is one of them | 22:15 |
| *** IlyaG has joined #openstack-cinder | 22:15 | |
| hemna_ | which isn't getting logged out | 22:16 |
| hemna_ | for some reason that one is in 'entries' | 22:16 |
| *** IlyaG has quit IRC | 22:16 | |
| patrickeast | wait a minute, am i reading this right and both of the 'entries' are the same one? | 22:16 |
| hemna_ | heh | 22:17 |
| hemna_ | I was JUST looking at that | 22:17 |
| hemna_ | so many characters | 22:17 |
| hemna_ | it's hard to visually compare | 22:17 |
| hemna_ | oh | 22:17 |
| hemna_ | entries is populated from a list of existing paths in /dev/disk/by-path | 22:18 |
| hemna_ | that are 'iscsi' paths | 22:18 |
| hemna_ | so there could be multiples of the same I suppose | 22:18 |
| patrickeast | ohh | 22:18 |
| patrickeast | weird | 22:18 |
| patrickeast | there are two still in there | 22:18 |
| patrickeast | lun 8 and 9 (neither of which is the volume being used) | 22:19 |
| patrickeast | maybe from an older devstack test or something | 22:19 |
| hemna_ | yup | 22:19 |
| hemna_ | there you go | 22:19 |
| hemna_ | that's why it's not getting logged out | 22:19 |
| hemna_ | because there are 'volumes' from it there. | 22:19 |
| hemna_ | stale, or whatnot | 22:19 |
| patrickeast | yea their broken | 22:20 |
| patrickeast | but the links are still there in the system | 22:20 |
| hemna_ | maybe a previous stack run ? | 22:20 |
| hemna_ | that didn't detach ? | 22:20 |
| patrickeast | must be | 22:20 |
| *** baumann has quit IRC | 22:22 | |
| diablo_rojo | So not a bug then? | 22:24 |
| patrickeast | diablo_rojo: nah nothing new | 22:25 |
| diablo_rojo | patrickeast: Cool :) | 22:25 |
| *** IlyaG has joined #openstack-cinder | 22:25 | |
| *** diablo_rojo has quit IRC | 22:25 | |
| patrickeast | we might want to do something like log a warning if when disconnecting from a multipath device we leave some parts behind... seems like it would be rare that some admin would actually be mixing multipath with non-multipath on the same array | 22:26 |
| hemna_ | we'll get lots of warnings then | 22:26 |
| *** dave-mccowan has quit IRC | 22:26 | |
| patrickeast | hemna_: would we? | 22:26 |
| patrickeast | if we leave them alll behind its fine | 22:26 |
| hemna_ | because I'd expect that to be mostly the case on a crowded n-cpu host | 22:26 |
| hemna_ | with lots of volumes attached to lots of instances | 22:26 |
| patrickeast | but if we are disconnecting it should be all the multipath paths | 22:26 |
| patrickeast | not just part of em | 22:26 |
| *** jgregor has quit IRC | 22:27 | |
| hemna_ | https://review.openstack.org/#/c/280817/ | 22:29 |
| hemna_ | ok that passed jenkins | 22:29 |
| *** martyturner has quit IRC | 22:29 | |
| *** martyturner has joined #openstack-cinder | 22:31 | |
| *** mylu has joined #openstack-cinder | 22:32 | |
| *** garthb has quit IRC | 22:33 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - not cleaning up HW Resource WWPN initiators https://review.openstack.org/280967 | 22:34 |
| jgriffith | mc_nair: ping | 22:40 |
| mc_nair | jgriffith: yo yo | 22:41 |
| jgriffith | mc_nair: hey ya | 22:41 |
| jgriffith | On that replication review... thanks for the input | 22:41 |
| jgriffith | mc_nair: wanted to chat about at least one of the comments | 22:41 |
| jgriffith | mc_nair: https://review.openstack.org/#/c/275797/12/cinder/exception.py | 22:41 |
| jgriffith | mc_nair: so I'm not completely sure I follow what you're asking... are you saying since the exception is "ReplicationTarget" and the mesg is backend_id? | 22:42 |
| jgriffith | mc_nair: that was intentional... and the idea being not so much that theyr'e interchangeable | 22:42 |
| jgriffith | mc_nair: but that there's a a "replication taget" which is a thing.. and the backend_id is driver info on how to know what/where that thing is and how to access it | 22:43 |
| jgriffith | mc_nair: not sure if that makes sense, or if that is confusing to people? | 22:43 |
| mc_nair | jgriffith: sure - question is pretty basic, can see why it was tough to follow though. Let me try to rephrase is | 22:43 |
| smcginnis | InvalidReplicationTarget({'backend_id': be_id, 'reason': reason}) ? | 22:44 |
| *** lpetrut has quit IRC | 22:45 | |
| smcginnis | ^^ I don't think that's the intent. | 22:45 |
| jgriffith | smcginnis: oh.. hurumph | 22:47 |
| mc_nair | smcginnis: ok, yea I think you got to the crux of the question. I was trying to understand the intent of that, and since we weren't creating a InvalidReplicationTarget I was having trouble following what we were going for | 22:47 |
| *** j_king has quit IRC | 22:47 | |
| jgriffith | smcginnis: what was I smokin | 22:47 |
| *** dims has quit IRC | 22:47 | |
| jgriffith | mc_nair: smcginnis yeah... I think I was half thinking of providing a var for the backend-id that was being used as well as the reason/mesg :) | 22:48 |
| jgriffith | mc_nair: smcginnis I'll just dump that "(backend_id)" string out of there | 22:48 |
| jgriffith | leave it to log message to provide the details there | 22:48 |
| jgriffith | mc_nair: smcginnis make sense? | 22:48 |
| smcginnis | jgriffith: Works for me. Could be less confusing. | 22:49 |
| jgriffith | smcginnis: mc_nair cool.. ez enough | 22:49 |
| jgriffith | now... | 22:49 |
| jgriffith | aorourke: you're next :) | 22:49 |
| mc_nair | jgriffith: sounds good to me | 22:49 |
| aorourke | jgriffith, :) | 22:49 |
| jgriffith | aorourke: I don't know that there's much value in list_targets any more... do you think there is? | 22:50 |
| *** Nakato has quit IRC | 22:50 | |
| jgriffith | aorourke: or at least a reason for it in the first pass? | 22:50 |
| *** Nakato has joined #openstack-cinder | 22:50 | |
| jgriffith | aorourke: or do you want an error checker to determine what the driver configured? | 22:50 |
| aorourke | jgriffith, still on the fence. I think it would be nice to have a command to list possible targets when you have multiple | 22:50 |
| *** kfarr has quit IRC | 22:50 | |
| jgriffith | aorourke: ok... I still say "check the config file" but you do have a valid point | 22:50 |
| *** kvidvans has quit IRC | 22:50 | |
| aorourke | jgriffith, that way you do not have to remember/check cinder.conf to see your targets | 22:50 |
| jgriffith | aorourke: and it's not that hard to do... I'm just being super lazy :) | 22:51 |
| aorourke | jgriffith, i can see it being a nice command to have, but not the end of the world | 22:51 |
| jgriffith | aorourke: also, the migration tests... yes, it's 64. Note that 63 is remove iscsi targets table... the rep stuff is in fact 64 | 22:51 |
| jgriffith | aorourke: yeah... fair enough. I'll put it back together | 22:52 |
| aorourke | jgriffith, makes sense | 22:52 |
| *** changbl has quit IRC | 22:52 | |
| aorourke | jgriffith, what will the API calls for these commands look like? | 22:52 |
| aorourke | jgriffith, it is no longer on an individual volume | 22:52 |
| jgriffith | aorourke: no cheating... you'll just have to wait | 22:52 |
| jgriffith | aorourke: so I'm moving that stuff to services... so it's by host | 22:53 |
| jgriffith | aorourke: you say "cinder list-replicaton-targets <c-vol-host>" | 22:53 |
| aorourke | jgriffith, thought that would be the direction. Definitely belongs there now it is by host | 22:53 |
| jgriffith | vol stuff all goes away IMO | 22:53 |
| aorourke | jgriffith, are you also working on the python-cinderclient patch? | 22:53 |
| jgriffith | aorourke: just enough to run my tests | 22:54 |
| jgriffith | :) | 22:54 |
| jgriffith | aorourke: it's going to be hard enough to finish this and get unit tests while all the rest of you are writing your drivers :) | 22:54 |
| jgriffith | aorourke: and I keep hitting silly merge conflicts because I'm slow :( | 22:54 |
| aorourke | jgriffith, i was going to see if you needed any help on the cinderclient side but not sure of the API calls quite yet | 22:55 |
| jgriffith | aorourke: yeah, let's synch up tomorrow morning maybe | 22:55 |
| smcginnis | aorourke, jgriffith: Would be good to get the client side set. | 22:55 |
| smcginnis | Deadline for that is sooner. | 22:56 |
| jgriffith | I think I'll have enough of it that we can do that. And I can share what I have in the client if it's not something I can just push up as is | 22:56 |
| aorourke | jgriffith, sounds good | 22:56 |
| jgriffith | smcginnis: what is the client deadline by the way? | 22:56 |
| *** gouthamr has quit IRC | 22:56 | |
| jgriffith | aorourke: if you have bandwidth and want to help with things we can definitely work together on a few things | 22:56 |
| aorourke | jgriffith, and to clarify, a c-vol-host, not a backend name? So it is a foo@bar#pool type thing? | 22:57 |
| jgriffith | aorourke: yes | 22:57 |
| aorourke | jgriffith, I have some time to work on it as well | 22:57 |
| jgriffith | aorourke: I want to be able to use "cinder service-list" to get info | 22:58 |
| jgriffith | aorourke: rather than have the mess that was created with manage allowing people to just do *whatever* they want | 22:58 |
| aorourke | jgriffith, that would probably be best. don't even allow the driver to list targets | 22:58 |
| aorourke | just all replicated hosts | 22:58 |
| aorourke | from cinder service-list | 22:58 |
| jgriffith | aorourke: that's why I removed the list_targets thing | 22:58 |
| jgriffith | aorourke: and yes, I've already added "recpliation_status" and "active_backend_id" to 'cinder service-list' | 22:59 |
| aorourke | jgriffith, i see now. so it would not need to be in the driver/manager, just an api call for it | 22:59 |
| jgriffith | aorourke: so what I have written right now.. yes, you're correct. BUT if we bring back list-targets then that needs to go through down to the driver... but that's still host based | 23:00 |
| jgriffith | aorourke: not sure if we're going in a circle here :) | 23:00 |
| jgriffith | aorourke: I'll code it up, I think you'll be happy with it when you see it | 23:00 |
| aorourke | jgriffith, hah we might be. | 23:00 |
| aorourke | i'll take a look when ready | 23:00 |
| *** haomaiwang has quit IRC | 23:01 | |
| jgriffith | aorourke: cool, lemme roll it into this next update | 23:01 |
| *** haomaiwang has joined #openstack-cinder | 23:01 | |
| aorourke | jgriffith, awesome | 23:01 |
| *** rlrossit has quit IRC | 23:02 | |
| aorourke | smcginnis, as for the client stuff, do we just want to yank out all the original v2 stuff? Or is there some special thing we should do to deprecate them? | 23:03 |
| *** sgotliv_ has quit IRC | 23:05 | |
| *** edtubill has quit IRC | 23:05 | |
| *** jwcroppe has quit IRC | 23:06 | |
| *** ntpttr has quit IRC | 23:09 | |
| *** apoorvad has quit IRC | 23:09 | |
| smcginnis | aorourke: Nah, that stuff can just get yanked. | 23:09 |
| smcginnis | aorourke: It was in an official client release, but we never officially released the cinder side support for it. | 23:10 |
| *** daneyon has quit IRC | 23:10 | |
| smcginnis | aorourke: So I think we are better of just yanking it. If we left it as deprecated it could just cause confusion. | 23:10 |
| aorourke | smcginnis, cool. With a revert of the original patch set or just delete/move all that good stuff in the new patch? | 23:10 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Permit volume type operations for policy authorized users https://review.openstack.org/274204 | 23:11 |
| *** dustins has quit IRC | 23:11 | |
| smcginnis | aorourke: Hmm. I have a feeling it may be better to refactor what is there. | 23:11 |
| smcginnis | aorourke: But if it ends up easier to just revert and start over, that would work too. | 23:12 |
| *** apoorvad has joined #openstack-cinder | 23:12 | |
| aorourke | smcginnis, refactor would probably be best | 23:12 |
| jgriffith | smcginnis: aorourke last time I looked you're going to have to just remove it. Revert won't work because of history... it will bomb out | 23:13 |
| openstackgerrit | Merged openstack/python-cinderclient: Remove openstack-common.conf https://review.openstack.org/264644 | 23:13 |
| jgriffith | smcginnis: aorourke you'll have more work around fixing merge conflicts than just refactor/delete | 23:13 |
| smcginnis | jgriffith: Refactor it is then. :) | 23:15 |
| *** sgotliv has joined #openstack-cinder | 23:17 | |
| *** rcernin has quit IRC | 23:18 | |
| *** IlyaG has quit IRC | 23:18 | |
| Swanson | hemna_, thanks! | 23:19 |
| hemna_ | Swanson, np :) | 23:19 |
| *** [1]Thelo has joined #openstack-cinder | 23:22 | |
| Swanson | jgriffith, What is the plan for get_replication_updates? | 23:22 |
| jgriffith | Swanson: not doing it | 23:22 |
| jgriffith | :) | 23:23 |
| Swanson | jgriffith, Works for me. I don't think it has enough info for me to do anything if it was being called. | 23:23 |
| *** mylu has quit IRC | 23:24 | |
| *** Thelo has quit IRC | 23:24 | |
| *** [1]Thelo is now known as Thelo | 23:24 | |
| *** garthb has joined #openstack-cinder | 23:27 | |
| *** xyang1_ has joined #openstack-cinder | 23:29 | |
| *** wewe0901_ has joined #openstack-cinder | 23:30 | |
| *** rhefner_ has joined #openstack-cinder | 23:30 | |
| *** DuncanT_ has joined #openstack-cinder | 23:30 | |
| *** dave-mccowan has joined #openstack-cinder | 23:31 | |
| *** martyturner1 has joined #openstack-cinder | 23:31 | |
| *** mylu has joined #openstack-cinder | 23:31 | |
| *** tpeoples_ has joined #openstack-cinder | 23:32 | |
| *** xiaohui_ has joined #openstack-cinder | 23:32 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 https://review.openstack.org/275797 | 23:34 |
| *** sweston_ has joined #openstack-cinder | 23:35 | |
| *** EmilienM_ has joined #openstack-cinder | 23:35 | |
| *** cburgess has joined #openstack-cinder | 23:35 | |
| *** ByPasS_ has joined #openstack-cinder | 23:35 | |
| *** martyturner1 has quit IRC | 23:36 | |
| *** martyturner has quit IRC | 23:37 | |
| *** arch-nemesis has quit IRC | 23:37 | |
| *** xyang1 has quit IRC | 23:37 | |
| *** ccesario has quit IRC | 23:37 | |
| *** wewe0901 has quit IRC | 23:37 | |
| *** ekarlso has quit IRC | 23:37 | |
| *** ByPasS has quit IRC | 23:37 | |
| *** xiaohui has quit IRC | 23:37 | |
| *** tpeoples has quit IRC | 23:37 | |
| *** EmilienM has quit IRC | 23:37 | |
| *** dobson has quit IRC | 23:37 | |
| *** bswartz has quit IRC | 23:37 | |
| *** mdavidson has quit IRC | 23:37 | |
| *** cburgess_ has quit IRC | 23:37 | |
| *** DuncanT has quit IRC | 23:37 | |
| *** rhefner has quit IRC | 23:37 | |
| *** sweston has quit IRC | 23:37 | |
| *** ameade has quit IRC | 23:37 | |
| *** johnthetubaguy has quit IRC | 23:37 | |
| *** theanalyst has quit IRC | 23:37 | |
| *** xyang1_ is now known as xyang1 | 23:37 | |
| *** EmilienM_ is now known as EmilienM | 23:37 | |
| *** wewe0901_ is now known as wewe0901 | 23:37 | |
| *** EmilienM is now known as Guest28836 | 23:37 | |
| *** ByPasS_ is now known as ByPasS | 23:37 | |
| *** dobson has joined #openstack-cinder | 23:37 | |
| *** rhefner_ is now known as rhefner | 23:38 | |
| *** sweston_ is now known as sweston | 23:39 | |
| *** tpeoples_ is now known as tpeoples | 23:40 | |
| *** Guest28836 has quit IRC | 23:41 | |
| *** Guest28836 has joined #openstack-cinder | 23:41 | |
| *** Guest28836 is now known as EmilienM | 23:42 | |
| *** ameade has joined #openstack-cinder | 23:43 | |
| *** mdavidson has joined #openstack-cinder | 23:43 | |
| *** DuncanT_ is now known as DuncanT | 23:43 | |
| *** ccesario has joined #openstack-cinder | 23:43 | |
| *** dims_ has joined #openstack-cinder | 23:44 | |
| *** arch-nemesis has joined #openstack-cinder | 23:44 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Replication v2.1 https://review.openstack.org/275797 | 23:45 |
| jgriffith | bahh... oops | 23:45 |
| jgriffith | Oh well.. no harm | 23:45 |
| *** xyang1 has quit IRC | 23:46 | |
| *** mylu has quit IRC | 23:47 | |
| *** ekarlso has joined #openstack-cinder | 23:47 | |
| *** johnthetubaguy has joined #openstack-cinder | 23:47 | |
| *** ekarlso has quit IRC | 23:48 | |
| *** ekarlso has joined #openstack-cinder | 23:48 | |
| *** theanalyst has joined #openstack-cinder | 23:49 | |
| *** mylu has joined #openstack-cinder | 23:50 | |
| *** IlyaG has joined #openstack-cinder | 23:51 | |
| *** merooney has quit IRC | 23:52 | |
| *** angela-s has quit IRC | 23:52 | |
| *** yhayashi has joined #openstack-cinder | 23:54 | |
| *** mylu has quit IRC | 23:56 | |
| *** yangyapeng has joined #openstack-cinder | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!