| *** diogogmt has quit IRC | 00:00 | |
| *** smoriya_ has joined #openstack-cinder | 00:00 | |
| *** pratap has quit IRC | 00:01 | |
| *** alonma has joined #openstack-cinder | 00:02 | |
| *** angela-s is now known as angela-s|afk | 00:05 | |
| *** alonma has quit IRC | 00:06 | |
| *** alonma has joined #openstack-cinder | 00:10 | |
| *** arif-ali has quit IRC | 00:12 | |
| *** ntpttr has quit IRC | 00:14 | |
| *** alonma has quit IRC | 00:15 | |
| *** haomaiwang has joined #openstack-cinder | 00:18 | |
| *** arif-ali has joined #openstack-cinder | 00:20 | |
| *** alonma has joined #openstack-cinder | 00:20 | |
| *** dims has joined #openstack-cinder | 00:22 | |
| openstackgerrit | Merged openstack/cinder: Enable consisgroups in SolidFire driver https://review.openstack.org/259638 | 00:24 |
|---|---|---|
| *** haomaiwang has quit IRC | 00:24 | |
| *** alonma has quit IRC | 00:24 | |
| *** akerr has quit IRC | 00:24 | |
| *** akerr has joined #openstack-cinder | 00:24 | |
| *** salv-orlando has quit IRC | 00:25 | |
| *** salv-orlando has joined #openstack-cinder | 00:25 | |
| *** alonma has joined #openstack-cinder | 00:27 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: LeftHand: Implement v2 replication (managed) https://review.openstack.org/255002 | 00:29 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: LeftHand: Implement v2 replication (unmanaged) https://review.openstack.org/255544 | 00:29 |
| *** alonma has quit IRC | 00:31 | |
| *** mriedem has quit IRC | 00:31 | |
| *** openstackgerrit has quit IRC | 00:32 | |
| *** openstackgerrit has joined #openstack-cinder | 00:33 | |
| *** daneyon_ has joined #openstack-cinder | 00:35 | |
| openstackgerrit | Vasanthi Thirumalai proposed openstack/cinder: Removes the Violin 6000 FC and iSCSI drivers https://review.openstack.org/264904 | 00:35 |
| *** mriedem has joined #openstack-cinder | 00:36 | |
| *** alonma has joined #openstack-cinder | 00:37 | |
| *** daneyon has quit IRC | 00:37 | |
| *** lcurtis has quit IRC | 00:39 | |
| *** alonma has quit IRC | 00:42 | |
| *** alonma has joined #openstack-cinder | 00:44 | |
| *** crose has quit IRC | 00:46 | |
| *** pots3 has quit IRC | 00:47 | |
| *** daneyon_ has quit IRC | 00:47 | |
| *** alonma has quit IRC | 00:48 | |
| *** zhangjn has joined #openstack-cinder | 00:54 | |
| *** zhangjn has quit IRC | 00:54 | |
| *** zhangjn has joined #openstack-cinder | 00:55 | |
| *** garthb__ has quit IRC | 00:56 | |
| *** garthb__ has joined #openstack-cinder | 00:59 | |
| *** mudassirlatif has quit IRC | 01:00 | |
| *** akerr is now known as akerr_away | 01:01 | |
| *** cheneydc has joined #openstack-cinder | 01:02 | |
| *** zhangjn has quit IRC | 01:06 | |
| *** crose has joined #openstack-cinder | 01:08 | |
| *** hemna is now known as hemnafk | 01:12 | |
| *** haomaiwang has joined #openstack-cinder | 01:12 | |
| *** EinstCrazy has joined #openstack-cinder | 01:14 | |
| *** shelvankar has joined #openstack-cinder | 01:14 | |
| *** vinayp has quit IRC | 01:15 | |
| *** zhangjn has joined #openstack-cinder | 01:15 | |
| *** shelvankar has quit IRC | 01:15 | |
| *** vinayp has joined #openstack-cinder | 01:16 | |
| *** angela-s|afk has quit IRC | 01:16 | |
| *** haomaiwang has quit IRC | 01:16 | |
| *** mtanino has quit IRC | 01:18 | |
| *** Lee1092 has joined #openstack-cinder | 01:34 | |
| *** garthb__ has quit IRC | 01:34 | |
| *** apoorvad has joined #openstack-cinder | 01:35 | |
| *** changbl has joined #openstack-cinder | 01:36 | |
| *** fthiagogv has quit IRC | 01:39 | |
| *** gouthamr has quit IRC | 01:39 | |
| *** alonma has joined #openstack-cinder | 01:44 | |
| *** zhangjn has quit IRC | 01:48 | |
| *** alonma has quit IRC | 01:49 | |
| *** smoriya_ has quit IRC | 01:49 | |
| *** alonma has joined #openstack-cinder | 01:50 | |
| *** leeantho has quit IRC | 01:52 | |
| *** alonma has quit IRC | 01:54 | |
| *** apoorvad has quit IRC | 01:57 | |
| *** rajinir has quit IRC | 02:05 | |
| *** shelvankar has joined #openstack-cinder | 02:06 | |
| *** vinayp has quit IRC | 02:09 | |
| *** haomaiwang has joined #openstack-cinder | 02:10 | |
| *** vinayp has joined #openstack-cinder | 02:11 | |
| *** shelvankar has quit IRC | 02:12 | |
| *** zhangjn has joined #openstack-cinder | 02:23 | |
| *** zhonghua has quit IRC | 02:30 | |
| *** apoorvad has joined #openstack-cinder | 02:30 | |
| *** jgregor has joined #openstack-cinder | 02:32 | |
| *** yangyapeng has joined #openstack-cinder | 02:32 | |
| mriedem | how long has multiattach been in the cinder api? | 02:35 |
| *** itzdilip has joined #openstack-cinder | 02:36 | |
| *** dims has quit IRC | 02:36 | |
| *** houming has joined #openstack-cinder | 02:37 | |
| *** uberjay has quit IRC | 02:39 | |
| *** cknight has joined #openstack-cinder | 02:39 | |
| *** zhangjn_ has joined #openstack-cinder | 02:39 | |
| *** uberjay has joined #openstack-cinder | 02:40 | |
| *** zhangjn has quit IRC | 02:42 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Huawei: Refactor driver for the second time https://review.openstack.org/256920 | 02:44 |
| *** alonma has joined #openstack-cinder | 02:50 | |
| *** crose has quit IRC | 02:52 | |
| *** shelvankar has joined #openstack-cinder | 02:53 | |
| *** zhangjn has joined #openstack-cinder | 02:53 | |
| *** alonma has quit IRC | 02:55 | |
| *** zhangjn_ has quit IRC | 02:56 | |
| *** alonma has joined #openstack-cinder | 02:57 | |
| *** vinayp has quit IRC | 02:57 | |
| *** jgregor has quit IRC | 02:58 | |
| *** cinder_ has joined #openstack-cinder | 02:58 | |
| *** bardia has quit IRC | 03:00 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** alonma has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:01 | |
| *** zhonghua-lee has joined #openstack-cinder | 03:07 | |
| *** shelvankar has quit IRC | 03:14 | |
| *** coolsvap|away is now known as coolsvap | 03:15 | |
| *** shelvankar has joined #openstack-cinder | 03:16 | |
| *** zhangjn_ has joined #openstack-cinder | 03:17 | |
| *** zhangjn has quit IRC | 03:17 | |
| *** zhangjn_ has quit IRC | 03:17 | |
| *** zhangjn has joined #openstack-cinder | 03:18 | |
| *** apoorvad has quit IRC | 03:23 | |
| *** shelvankar has quit IRC | 03:29 | |
| *** shelvankar has joined #openstack-cinder | 03:38 | |
| *** cknight has quit IRC | 03:43 | |
| *** shelvankar has quit IRC | 03:44 | |
| *** rushiagr_away is now known as rushiagr | 03:44 | |
| *** vinayp has joined #openstack-cinder | 03:49 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Add finish_volume_migration to volume object https://review.openstack.org/257667 | 03:50 |
| *** zhangjn has quit IRC | 03:56 | |
| *** shelvankar has joined #openstack-cinder | 03:56 | |
| *** alonma has joined #openstack-cinder | 03:57 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 04:01 | |
| *** alonma has quit IRC | 04:01 | |
| *** alonma has joined #openstack-cinder | 04:04 | |
| *** edmondsw has quit IRC | 04:05 | |
| *** alonma has quit IRC | 04:08 | |
| *** salv-orl_ has joined #openstack-cinder | 04:09 | |
| *** alonma has joined #openstack-cinder | 04:11 | |
| *** salv-orlando has quit IRC | 04:12 | |
| *** diogogmt has joined #openstack-cinder | 04:14 | |
| *** alonma has quit IRC | 04:15 | |
| *** rlrossit has joined #openstack-cinder | 04:15 | |
| *** alonma has joined #openstack-cinder | 04:17 | |
| *** shelvankar has quit IRC | 04:19 | |
| *** mriedem has quit IRC | 04:20 | |
| *** alonma has quit IRC | 04:21 | |
| *** tobe has joined #openstack-cinder | 04:25 | |
| *** tobe has quit IRC | 04:26 | |
| *** links has joined #openstack-cinder | 04:27 | |
| *** yhayashi has joined #openstack-cinder | 04:27 | |
| *** salv-orl_ has quit IRC | 04:30 | |
| *** salv-orlando has joined #openstack-cinder | 04:30 | |
| *** diogogmt has quit IRC | 04:32 | |
| *** shyama has joined #openstack-cinder | 04:34 | |
| *** cknight has joined #openstack-cinder | 04:35 | |
| *** markvoelker has quit IRC | 04:38 | |
| jgriffith | hemnafk: ping | 04:46 |
| jgriffith | doh | 04:46 |
| jgriffith | mriedem: About a year | 04:49 |
| jgriffith | mriedem: https://review.openstack.org/#/c/85847/ | 04:50 |
| *** rushiagr is now known as rushiagr_away | 04:55 | |
| *** cknight has quit IRC | 04:55 | |
| *** haomaiwa_ has quit IRC | 05:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 05:01 | |
| openstackgerrit | ting wang proposed openstack/python-cinderclient: Use the oslo.utils.reflection to extract the class name https://review.openstack.org/265073 | 05:02 |
| jgriffith | vinayp: Ahh... I see what hemnafk is asking for here, makes sense. Refactor those checks you added up into the base class (InitiatorConnector) to ensure the lun number is checked/converted regardless of which connector type is being used. | 05:03 |
| jgriffith | vinayp: not sure it's applicable to anything but iSCSI and FC but regardless | 05:03 |
| *** shausy has joined #openstack-cinder | 05:03 | |
| jgriffith | vinayp: also to avoid confusion WRT the regex, you might want to consider just using something like "if '0x' in lun: ....." | 05:04 |
| jgriffith | vinayp: the bug, and comments in the code seemed clear enough but meh... fair enough | 05:04 |
| jgriffith | vinayp: also, you'll need to work on the commit message.... | 05:04 |
| jgriffith | vinayp: have a look at this page: https://wiki.openstack.org/wiki/GitCommitMessages | 05:05 |
| openstackgerrit | ting wang proposed openstack/python-cinderclient: Remove openstack-common.conf https://review.openstack.org/265074 | 05:05 |
| jgriffith | DOH and gerrit is no longer available for me | 05:06 |
| openstackgerrit | Yatin Kumbhare proposed openstack/cinder: Use assertTrue/False instead of assertEqual(T/F) https://review.openstack.org/264116 | 05:09 |
| *** salv-orlando has quit IRC | 05:12 | |
| *** alonma has joined #openstack-cinder | 05:12 | |
| *** salv-orlando has joined #openstack-cinder | 05:12 | |
| *** alonma has quit IRC | 05:17 | |
| *** harlowja_at_home has joined #openstack-cinder | 05:18 | |
| *** alonma has joined #openstack-cinder | 05:19 | |
| *** chhavi has joined #openstack-cinder | 05:24 | |
| *** alonma has quit IRC | 05:24 | |
| *** alonma has joined #openstack-cinder | 05:25 | |
| *** lpetrut has joined #openstack-cinder | 05:26 | |
| *** jamielennox is now known as jamielennox|away | 05:27 | |
| *** alonma has quit IRC | 05:29 | |
| *** shyama has quit IRC | 05:30 | |
| *** esker has joined #openstack-cinder | 05:31 | |
| *** alonma has joined #openstack-cinder | 05:33 | |
| *** rlrossit has quit IRC | 05:37 | |
| *** alonma has quit IRC | 05:38 | |
| *** jwcroppe has quit IRC | 05:40 | |
| *** mragupat has joined #openstack-cinder | 05:40 | |
| *** jwcroppe has joined #openstack-cinder | 05:41 | |
| *** alonma has joined #openstack-cinder | 05:41 | |
| *** mragupat has quit IRC | 05:42 | |
| *** mragupat has joined #openstack-cinder | 05:42 | |
| *** jwcroppe has quit IRC | 05:45 | |
| *** alonma has quit IRC | 05:45 | |
| *** alonma has joined #openstack-cinder | 05:47 | |
| *** dave-mccowan has quit IRC | 05:49 | |
| *** alonma has quit IRC | 05:51 | |
| *** zhangjn has joined #openstack-cinder | 05:52 | |
| *** harlowja_at_home has quit IRC | 05:53 | |
| *** alonma has joined #openstack-cinder | 05:53 | |
| *** alonma has quit IRC | 05:57 | |
| *** zhangjn_ has joined #openstack-cinder | 05:58 | |
| *** haomaiwa_ has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-cinder | 06:01 | |
| *** alonma has joined #openstack-cinder | 06:01 | |
| *** zhangjn_ has quit IRC | 06:02 | |
| *** zhangjn has quit IRC | 06:02 | |
| *** jwcroppe has joined #openstack-cinder | 06:02 | |
| *** zhangjn has joined #openstack-cinder | 06:03 | |
| *** lprice1 has quit IRC | 06:04 | |
| *** lprice has joined #openstack-cinder | 06:04 | |
| *** dslevin has quit IRC | 06:06 | |
| *** alonma has quit IRC | 06:06 | |
| *** dslevin has joined #openstack-cinder | 06:07 | |
| *** jwcroppe has quit IRC | 06:07 | |
| *** logan- has quit IRC | 06:08 | |
| *** alonma has joined #openstack-cinder | 06:08 | |
| *** logan- has joined #openstack-cinder | 06:08 | |
| *** mgagne has quit IRC | 06:09 | |
| *** alonma has quit IRC | 06:12 | |
| *** vgridnev has joined #openstack-cinder | 06:13 | |
| *** esker has quit IRC | 06:13 | |
| *** markvoelker has joined #openstack-cinder | 06:13 | |
| *** mgagne has joined #openstack-cinder | 06:16 | |
| *** mgagne is now known as Guest41804 | 06:16 | |
| *** cfriesen__ has joined #openstack-cinder | 06:16 | |
| *** lpetrut has quit IRC | 06:18 | |
| *** markvoelker has quit IRC | 06:18 | |
| *** vgridnev has quit IRC | 06:20 | |
| *** ChubYann has quit IRC | 06:21 | |
| *** shyama has joined #openstack-cinder | 06:23 | |
| *** shausy has quit IRC | 06:25 | |
| *** shausy has joined #openstack-cinder | 06:26 | |
| *** cfriesen__ has quit IRC | 06:32 | |
| *** hunters1094 has joined #openstack-cinder | 06:33 | |
| *** Lickitysplitted has joined #openstack-cinder | 06:36 | |
| *** EinstCrazy has quit IRC | 06:37 | |
| *** mragupat has quit IRC | 06:45 | |
| *** anshul has joined #openstack-cinder | 06:49 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/264414 | 06:50 |
| hunters1094 | are there anyone here? | 06:51 |
| hunters1094 | I have some problem with cinder encryption | 06:51 |
| *** coolsvap is now known as coolsvap|away | 06:52 | |
| *** deepakcs has joined #openstack-cinder | 06:55 | |
| *** zhonghua-lee has quit IRC | 06:59 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:01 | |
| *** alonma has joined #openstack-cinder | 07:08 | |
| *** alonma has quit IRC | 07:13 | |
| *** nkrinner has joined #openstack-cinder | 07:13 | |
| *** coolsvap|away is now known as coolsvap | 07:13 | |
| *** alonma has joined #openstack-cinder | 07:14 | |
| *** anshul has quit IRC | 07:17 | |
| *** alonma has quit IRC | 07:19 | |
| *** belmoreira has joined #openstack-cinder | 07:26 | |
| *** alonma has joined #openstack-cinder | 07:30 | |
| *** anshul has joined #openstack-cinder | 07:32 | |
| *** alonma has quit IRC | 07:34 | |
| *** cfriesen__ has joined #openstack-cinder | 07:41 | |
| *** logan- has quit IRC | 07:44 | |
| *** EinstCrazy has joined #openstack-cinder | 07:49 | |
| hunters1094 | hi all | 07:51 |
| hunters1094 | are there anyone here? | 07:51 |
| *** shyama has quit IRC | 07:52 | |
| *** shyama has joined #openstack-cinder | 07:58 | |
| *** salv-orlando has quit IRC | 08:00 | |
| *** salv-orlando has joined #openstack-cinder | 08:00 | |
| *** coolsvap is now known as coolsvap|away | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-cinder | 08:02 | |
| *** zhangjn_ has joined #openstack-cinder | 08:04 | |
| *** coolsvap|away is now known as coolsvap | 08:05 | |
| *** ankit_ag has joined #openstack-cinder | 08:05 | |
| *** zhangjn has quit IRC | 08:07 | |
| *** markvoelker has joined #openstack-cinder | 08:15 | |
| openstackgerrit | Merged openstack/cinder: Remove API races from delete methods https://review.openstack.org/205835 | 08:18 |
| *** markvoelker has quit IRC | 08:19 | |
| *** haomaiwang has quit IRC | 08:19 | |
| *** jistr has joined #openstack-cinder | 08:24 | |
| *** haomaiwang has joined #openstack-cinder | 08:24 | |
| *** markus_z has joined #openstack-cinder | 08:26 | |
| *** alonma has joined #openstack-cinder | 08:30 | |
| *** geguileo_mobile has quit IRC | 08:34 | |
| *** alonma has quit IRC | 08:35 | |
| *** salv-orlando has quit IRC | 08:38 | |
| *** salv-orlando has joined #openstack-cinder | 08:39 | |
| openstackgerrit | ting wang proposed openstack/cinder: Trivial: replace deprecated _impl_messaging https://review.openstack.org/265140 | 08:41 |
| *** zhangjn_ has quit IRC | 08:42 | |
| *** zhangjn has joined #openstack-cinder | 08:43 | |
| *** geguileo_phone has joined #openstack-cinder | 08:47 | |
| *** haomaiwang has quit IRC | 08:50 | |
| *** haomaiwa_ has joined #openstack-cinder | 08:54 | |
| *** haomaiwa_ has quit IRC | 09:01 | |
| *** zhangjn has quit IRC | 09:02 | |
| *** haomaiwa_ has joined #openstack-cinder | 09:05 | |
| *** geguileo_phone has quit IRC | 09:05 | |
| *** manous has joined #openstack-cinder | 09:06 | |
| *** geguileo_phone has joined #openstack-cinder | 09:10 | |
| *** nkrinner has quit IRC | 09:10 | |
| *** shausy has quit IRC | 09:12 | |
| *** nkrinner has joined #openstack-cinder | 09:12 | |
| *** shausy has joined #openstack-cinder | 09:13 | |
| *** cinder_ has quit IRC | 09:18 | |
| *** Neoburner has left #openstack-cinder | 09:23 | |
| *** shyama has quit IRC | 09:23 | |
| *** alonma has joined #openstack-cinder | 09:31 | |
| *** lpetrut has joined #openstack-cinder | 09:31 | |
| *** alonma has quit IRC | 09:35 | |
| *** ndipanov has joined #openstack-cinder | 09:37 | |
| *** jordanP has joined #openstack-cinder | 09:39 | |
| *** geguileo_phone has quit IRC | 09:41 | |
| *** alonma has joined #openstack-cinder | 09:43 | |
| *** alonma has quit IRC | 09:47 | |
| *** haomaiwa_ has quit IRC | 09:47 | |
| *** haomaiwang has joined #openstack-cinder | 09:50 | |
| *** geguileo_phone has joined #openstack-cinder | 09:52 | |
| *** [1]Thelo has joined #openstack-cinder | 09:54 | |
| *** shyama has joined #openstack-cinder | 09:54 | |
| *** alonma has joined #openstack-cinder | 09:55 | |
| *** Thelo has quit IRC | 09:56 | |
| *** [1]Thelo is now known as Thelo | 09:56 | |
| *** smoriya_ has joined #openstack-cinder | 09:56 | |
| *** alonma has quit IRC | 09:59 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** houming has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-cinder | 10:01 | |
| *** cinder_ has joined #openstack-cinder | 10:02 | |
| *** cheneydc has quit IRC | 10:02 | |
| *** alonma has joined #openstack-cinder | 10:03 | |
| *** EinstCrazy has quit IRC | 10:04 | |
| *** yhayashi has quit IRC | 10:05 | |
| *** ociuhandu has quit IRC | 10:06 | |
| *** alonma has quit IRC | 10:07 | |
| *** sgotliv has joined #openstack-cinder | 10:08 | |
| openstackgerrit | Yusuke Hayashi proposed openstack/cinder: Add Fujitsu ETERNUS DX Volume Driver (FC part) https://review.openstack.org/263941 | 10:08 |
| *** salv-orl_ has joined #openstack-cinder | 10:09 | |
| *** salv-orlando has quit IRC | 10:12 | |
| *** markvoelker has joined #openstack-cinder | 10:15 | |
| *** markvoelker has quit IRC | 10:20 | |
| *** lprice has quit IRC | 10:35 | |
| *** lprice has joined #openstack-cinder | 10:36 | |
| *** hunters1094 has quit IRC | 10:48 | |
| *** ccesario has quit IRC | 10:57 | |
| *** haomaiwang has quit IRC | 11:01 | |
| *** haomaiwang has joined #openstack-cinder | 11:01 | |
| *** alonma has joined #openstack-cinder | 11:03 | |
| *** smoriya_ has quit IRC | 11:06 | |
| *** alonma has quit IRC | 11:08 | |
| *** dims has joined #openstack-cinder | 11:09 | |
| *** alonma has joined #openstack-cinder | 11:09 | |
| *** ccesario has joined #openstack-cinder | 11:10 | |
| *** alonma has quit IRC | 11:13 | |
| *** zhangjn has joined #openstack-cinder | 11:16 | |
| *** shyama has quit IRC | 11:16 | |
| *** zhangjn has quit IRC | 11:16 | |
| *** yangyapeng has quit IRC | 11:16 | |
| *** zhangjn has joined #openstack-cinder | 11:17 | |
| *** zhangjn has quit IRC | 11:17 | |
| *** zhangjn has joined #openstack-cinder | 11:17 | |
| *** alonma has joined #openstack-cinder | 11:17 | |
| *** cinder_ has quit IRC | 11:19 | |
| *** alonma has quit IRC | 11:22 | |
| openstackgerrit | Szymon Borkowski proposed openstack/cinder: Update copy_volume_to_image to use volume object https://review.openstack.org/263765 | 11:22 |
| *** ociuhandu has joined #openstack-cinder | 11:23 | |
| *** alonma has joined #openstack-cinder | 11:24 | |
| chhavi | ildikov: I have a question on the https://review.openstack.org/#/c/153033/37/nova/db/sqlalchemy/api.py | 11:27 |
| ildikov | chhavi: hi, shoot | 11:28 |
| *** haomaiwang has quit IRC | 11:28 | |
| *** alonma has quit IRC | 11:28 | |
| chhavi | block_device_mapping_get_by_volume_id() Line 4030 | 11:28 |
| chhavi | Here currently we are picking first() but if the volume is shared then we might expect multiple BDM entries | 11:29 |
| chhavi | which is valid | 11:29 |
| ildikov | what you miss there is that the search includes the instance_uuid | 11:30 |
| ildikov | which identifies the BDM, there should not be more | 11:30 |
| *** alonma has joined #openstack-cinder | 11:30 | |
| ildikov | there's a problem though with BDM uniqueness, but that's a generic issue | 11:30 |
| chhavi | sorry i mean line 4041 | 11:30 |
| chhavi | block_device_mapping_get_by_volume_id | 11:30 |
| ildikov | yeah, in that case that's true | 11:31 |
| chhavi | where user wants BDMs by volume_id | 11:31 |
| ildikov | that's an existing function I cannot modify it | 11:31 |
| ildikov | the solution here is not call it if possible | 11:31 |
| openstackgerrit | Szymon Borkowski proposed openstack/cinder: Update terminate_connection to use volume object https://review.openstack.org/264797 | 11:31 |
| ildikov | the only function it uses it and where we dont have instance_uuid in place is create_volume_snapshot | 11:31 |
| ildikov | that's disabled for now with multiattach volumes | 11:32 |
| chhavi | in that case i think it will be good to add a new query to get_all | 11:32 |
| ildikov | new query? | 11:32 |
| chhavi | yes, block_device_mapping_get_all_by_volume_id | 11:33 |
| *** JoseMello has joined #openstack-cinder | 11:33 | |
| ildikov | where would you use it? | 11:34 |
| *** Roamer` has joined #openstack-cinder | 11:34 | |
| *** alonma has quit IRC | 11:34 | |
| chhavi | it can be used to identify if the volume is shared or not | 11:36 |
| chhavi | if multiple entries that means volume is already attached | 11:36 |
| ildikov | not necessarily, there are race issues in the code | 11:36 |
| ildikov | I use the volume info in every case for that | 11:36 |
| ildikov | that's the let's say ultimate source of truth | 11:37 |
| ildikov | as I said BDM uniqueness is not a solved problem today, there are patches up for review for it, but they'r enot merged yet | 11:37 |
| *** alonma has joined #openstack-cinder | 11:38 | |
| *** alonma has quit IRC | 11:42 | |
| *** alonma has joined #openstack-cinder | 11:46 | |
| *** alonma has quit IRC | 11:50 | |
| *** akerr_away is now known as akerr | 11:51 | |
| *** ociuhandu_ has joined #openstack-cinder | 11:54 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: VNX: Replication V2 support(managed) https://review.openstack.org/256246 | 11:54 |
| *** ociuhandu has quit IRC | 11:56 | |
| *** ociuhandu_ is now known as ociuhandu | 11:56 | |
| *** zhangjn_ has joined #openstack-cinder | 11:58 | |
| *** zhangjn_ has quit IRC | 11:58 | |
| chhavi | ildikov: BDM uniqueness, means ? | 11:58 |
| *** diablo_rojo has joined #openstack-cinder | 11:59 | |
| *** zhangjn has quit IRC | 11:59 | |
| chhavi | ildikov: For a particular instance there will always be a single BDM, | 11:59 |
| *** zhangjn has joined #openstack-cinder | 11:59 | |
| *** diablo_rojo has quit IRC | 12:00 | |
| *** markvoelker has joined #openstack-cinder | 12:01 | |
| *** zhangjn has quit IRC | 12:03 | |
| *** akerr has quit IRC | 12:05 | |
| *** markvoelker has quit IRC | 12:06 | |
| ildikov | chhavi: I haven't checked the flow or how the db record is generated, the patch I meant is here: https://review.openstack.org/#/c/242602/ | 12:08 |
| jordanP | #success My 3rd party CI correctly warned me that a patch would break my driver | 12:09 |
| openstackstatus | jordanP: Added success to Success page | 12:09 |
| ildikov | chhavi: either case the volume should be a good source for information | 12:10 |
| chhavi | that will involve a cinder api call from nova | 12:11 |
| openstackgerrit | Peter Wang proposed openstack/cinder: VNX: Replication V2 support(managed) https://review.openstack.org/256246 | 12:14 |
| ildikov | chhavi: I know, but the multiattach flag is stored only in the volume for instance, so that's required anyway | 12:14 |
| ildikov | chhavi: on the other hand a volume can be attached to a host too, not just to a VM instance | 12:15 |
| chhavi | ildikov: I will check the details of this patch, but high level what i see its majorly focused when we upgrade | 12:15 |
| ildikov | I know it came up regarding one of my patches | 12:15 |
| chhavi | ildikov: whether volume is attached to the host or to the instance, BDM for an instance will be unique. | 12:16 |
| *** raildo-afk is now known as raildo | 12:16 | |
| chhavi | For example: if a single volume is attached to multiple instances each having their own BDMs then this should return all the BDM entries | 12:16 |
| ildikov | chhavi: but does not tell the information correctly about a volume still in the sense of it is already attached or not | 12:16 |
| chhavi | since previously we do not support shared volume it was meant to return a single entry | 12:17 |
| chhavi | ildikov: If there is a BDM entry having the volume-id, it means that instance is attached to the BDM. | 12:18 |
| *** EinstCrazy has joined #openstack-cinder | 12:18 | |
| chhavi | During volume detach we update the BDM info | 12:18 |
| ildikov | I got it, but still I need the volume info anyway, therefore it's better to check that if the data is there anyway | 12:22 |
| chhavi | ildikov: remove_volume_connection: https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L4912 | 12:24 |
| chhavi | here we are using this | 12:24 |
| ildikov | see John's comment at line 4048: https://review.openstack.org/#/c/193133/30/nova/db/sqlalchemy/api.py | 12:24 |
| *** deepakcs has quit IRC | 12:25 | |
| ildikov | chhavi: I got it, but I still don't see the value at this stage of that extra function, also that call you referred will return only one BDM in any case | 12:26 |
| *** EinstCrazy has quit IRC | 12:28 | |
| ildikov | chhavi: there's also a this check and note here: https://github.com/openstack/nova/blob/master/nova/objects/block_device.py#L207, I wonder what case it was needed for | 12:28 |
| ildikov | s/was/is/ | 12:28 |
| ildikov | anyway, what you're saying could be an optimization if we could save one round trip with it, but we can't | 12:29 |
| ildikov | BDM is very fragile in Nova | 12:29 |
| *** coolsvap is now known as coolsvap|away | 12:34 | |
| chhavi | ildikov: I need to check more details on the same, because i did not see any such behavior for BDM till now | 12:37 |
| chhavi | the one u mentioned stands only when we upgrade, | 12:37 |
| chhavi | let me go through the details of the BDM change set shared for more clarity | 12:37 |
| chhavi | ildikov: thanks for sharing this info | 12:38 |
| ildikov | chhavi: I'm not experienced enough, I can only rely on the concerns of the cores | 12:38 |
| ildikov | I mean in Nova | 12:38 |
| ildikov | what you suggest can be done later, when we get there to optimize | 12:38 |
| chhavi | yeah i understand, me too learning updated the review comments to hear some more inputs from john on the same. | 12:39 |
| ildikov | the non-priority feature freeze for Nova is Jan 21, so until that I fix only what's critical and I'm still afraid even a mimum setup will not make it again :( | 12:39 |
| *** dims_ has joined #openstack-cinder | 12:40 | |
| *** dims has quit IRC | 12:41 | |
| *** markvoelker has joined #openstack-cinder | 12:45 | |
| *** alonma has joined #openstack-cinder | 12:47 | |
| *** EinstCrazy has joined #openstack-cinder | 12:49 | |
| *** rushiagr_away is now known as rushiagr | 12:49 | |
| *** alonma has quit IRC | 12:51 | |
| *** EinstCrazy has quit IRC | 12:52 | |
| *** alonma has joined #openstack-cinder | 12:53 | |
| *** EinstCrazy has joined #openstack-cinder | 12:54 | |
| *** alonma has quit IRC | 12:58 | |
| *** sgotliv has quit IRC | 12:58 | |
| *** alonma has joined #openstack-cinder | 13:00 | |
| *** alonma has quit IRC | 13:04 | |
| *** martyturner has joined #openstack-cinder | 13:07 | |
| *** alonma has joined #openstack-cinder | 13:09 | |
| *** rushiagr is now known as rushiagr_away | 13:09 | |
| *** esker has joined #openstack-cinder | 13:13 | |
| *** alonma has quit IRC | 13:13 | |
| *** dims_ has quit IRC | 13:16 | |
| *** alonma has joined #openstack-cinder | 13:17 | |
| *** esker has quit IRC | 13:20 | |
| *** alonma has quit IRC | 13:22 | |
| *** links has quit IRC | 13:25 | |
| *** alonma has joined #openstack-cinder | 13:25 | |
| *** dims has joined #openstack-cinder | 13:26 | |
| *** ankit_ag has quit IRC | 13:27 | |
| *** alonma has quit IRC | 13:30 | |
| *** alonma has joined #openstack-cinder | 13:34 | |
| *** edmondsw has joined #openstack-cinder | 13:35 | |
| *** EinstCrazy has quit IRC | 13:38 | |
| *** alonma has quit IRC | 13:38 | |
| *** gouthamr has joined #openstack-cinder | 13:40 | |
| *** vignesh has joined #openstack-cinder | 13:43 | |
| *** sileht has quit IRC | 13:46 | |
| *** cheneydc has joined #openstack-cinder | 13:47 | |
| *** sileht has joined #openstack-cinder | 13:48 | |
| *** akerr has joined #openstack-cinder | 13:50 | |
| *** annasort has joined #openstack-cinder | 13:50 | |
| *** e0ne has joined #openstack-cinder | 13:52 | |
| openstackgerrit | Philipp Marek proposed openstack/cinder: Infrastructure to use the DRBD transport for NOVA. https://review.openstack.org/259973 | 13:56 |
| *** dsunkum has joined #openstack-cinder | 14:03 | |
| *** lprice1 has joined #openstack-cinder | 14:06 | |
| *** lprice has quit IRC | 14:06 | |
| *** rlrossit has joined #openstack-cinder | 14:08 | |
| *** salv-orl_ has quit IRC | 14:08 | |
| *** salv-orlando has joined #openstack-cinder | 14:08 | |
| *** cheneydc has quit IRC | 14:09 | |
| *** xyang has joined #openstack-cinder | 14:10 | |
| *** dslevin has quit IRC | 14:10 | |
| *** dustins has joined #openstack-cinder | 14:10 | |
| *** dave-mccowan has joined #openstack-cinder | 14:13 | |
| *** Yogi11 has joined #openstack-cinder | 14:13 | |
| *** jordanP is now known as jordanP|away | 14:14 | |
| *** anshul has quit IRC | 14:14 | |
| *** merooney has joined #openstack-cinder | 14:14 | |
| *** shyama has joined #openstack-cinder | 14:15 | |
| *** esker has joined #openstack-cinder | 14:17 | |
| *** shausy has quit IRC | 14:17 | |
| openstackgerrit | Szymon Borkowski proposed openstack/cinder: Update accept_transfer to use volume object https://review.openstack.org/265266 | 14:21 |
| *** jgregor has joined #openstack-cinder | 14:21 | |
| *** esker has quit IRC | 14:22 | |
| *** esker has joined #openstack-cinder | 14:22 | |
| *** diablo_rojo has joined #openstack-cinder | 14:23 | |
| *** C1ShaunID has joined #openstack-cinder | 14:25 | |
| openstackgerrit | Merged openstack/cinder: Remove duplicate keys from dictionary https://review.openstack.org/246369 | 14:26 |
| smcginnis | jordanP|away: +1 :) | 14:31 |
| *** cknight has joined #openstack-cinder | 14:32 | |
| *** dsunkum has quit IRC | 14:33 | |
| *** alonma has joined #openstack-cinder | 14:34 | |
| *** alonma has quit IRC | 14:39 | |
| *** mriedem has joined #openstack-cinder | 14:42 | |
| jgriffith | mriedem: sent you a response but you weren't actually here | 14:43 |
| jgriffith | mriedem: https://review.openstack.org/#/c/85847/ | 14:43 |
| jgriffith | mriedem: multi-attach in Cinder about a year ago | 14:43 |
| mriedem | ok so it got into kilo | 14:44 |
| mriedem | was just wondering if like a mitaka nova would be talking to a juno cinder | 14:44 |
| *** alonma has joined #openstack-cinder | 14:44 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Report discard support for Dell SC connections https://review.openstack.org/265281 | 14:46 |
| mriedem | ildikov: ^ is there anything about the nova multiattach spec that would be impacted by that? | 14:46 |
| jgriffith | mriedem: ha! | 14:46 |
| jgriffith | mriedem: all I have to say is "ha" | 14:46 |
| mriedem | i know it sounds crazy... | 14:46 |
| mriedem | but isn't rax running with a really old cinder or something? | 14:46 |
| guitarzan | mriedem: yes | 14:47 |
| mriedem | how old? | 14:47 |
| guitarzan | I | 14:47 |
| ildikov | mriedem: you mean compatibility? | 14:47 |
| jgriffith | mriedem: I used to worry about that | 14:47 |
| guitarzan | if nova breaks, we'll just fix our stuff :) | 14:47 |
| jgriffith | and even try it... but our library dep chain has made me realize that trying to do that usually results in bigger problems | 14:47 |
| ildikov | mriedem: my opinion is that it's not that rabbit hole I would like to go down to | 14:47 |
| jgriffith | and once you throw in Glance and Keystone.... "forget about it" | 14:47 |
| *** esker has quit IRC | 14:48 | |
| ildikov | in the sense of aiming to support that kind of version differences | 14:48 |
| ildikov | that would kill any kind of cross-project development IMHO | 14:48 |
| guitarzan | ildikov: in the past the answer has just been simply "don't change the contract" | 14:48 |
| mriedem | sounds like it employs a lot of engineers at rackspace though :) | 14:48 |
| jgriffith | mriedem: but yeah, I think the way that code has been written it "may" create an issue in that scenario | 14:49 |
| jgriffith | mriedem: I don't quite know why the code couldn't be implemented to handle either way though | 14:49 |
| *** alonma has quit IRC | 14:49 | |
| jgriffith | mriedem: shouldn't be that hard I wouldn't think | 14:49 |
| smcginnis | Wasn't there a cross project statement to only support N-1? | 14:50 |
| mriedem | if there was, i don't know that the ops abide by it | 14:50 |
| *** jordanP|away is now known as jordanP | 14:50 | |
| mriedem | there are threads in the ops list right now about disjointed releases of components working together | 14:50 |
| mriedem | more for keystone though | 14:50 |
| *** pradipm has joined #openstack-cinder | 14:50 | |
| *** haomaiwa_ has joined #openstack-cinder | 14:51 | |
| jgriffith | mriedem: yeah, good luck with that | 14:51 |
| jgriffith | Glance chose to be a snowflake | 14:51 |
| guitarzan | that's because keystone seems to like changing their interface | 14:51 |
| jgriffith | so did keystone | 14:51 |
| ildikov | jgriffith: what are you referring to here: "but yeah, I think the way that code has been written it "may" create an issue in that scenario", I might miss some context here | 14:52 |
| jgriffith | anybody that says "yeah you can do that" either: | 14:52 |
| jgriffith | a. hasn't actually tried it | 14:52 |
| jgriffith | b. has folks patching code internally to make it work | 14:52 |
| jgriffith | hell, I can barely get glance to work with n == n any more :( | 14:52 |
| guitarzan | glance v1 forever | 14:52 |
| jgriffith | because they don't understand what an API interface is apparantly | 14:52 |
| jgriffith | guitarzan: +1 | 14:53 |
| jgriffith | mriedem: sounds like Nova's getting ready to punt on the multi-attach again then? | 14:54 |
| ildikov | user facing things are tough :( | 14:54 |
| *** esker has joined #openstack-cinder | 14:54 | |
| johnthetubaguy | its in review, just we are not really converging right now :( | 14:54 |
| jgriffith | mriedem: kinda need to know cuz if that's the case I'd prefer not to waste cycles in M writing anything for it in Cinder :) | 14:54 |
| jgriffith | johnthetubaguy: got ya | 14:54 |
| mriedem | jgriffith: no, i was just asking | 14:54 |
| mriedem | jgriffith: i was reviewing the nova spec last night | 14:55 |
| mriedem | since i never reviewed it the first time before it was approved | 14:55 |
| jgriffith | mriedem: ahhh, I see | 14:55 |
| johnthetubaguy | mriedem brought up some good new points the other day, but I think ildikov has mostly patched those, or was about to | 14:56 |
| ildikov | johnthetubaguy: yes, I did | 14:56 |
| mriedem | johnthetubaguy: i raised a concern about minimum required libvirt in the amended spec change last night too | 14:56 |
| mriedem | i'd like danpb to weigh in on that | 14:56 |
| jgriffith | johnthetubaguy: cool, so we need to figure out a better model for Cinder/Nova at some point I think. | 14:56 |
| ildikov | johnthetubaguy: the only thing I'm playing with now is the libvirt version check | 14:56 |
| johnthetubaguy | ildikov: mriedem: was there a boot from volume thing that needed sorting out, or were we happy about that now? | 14:56 |
| jgriffith | johnthetubaguy: we're a bit too "loose" on our side at times I think, and there are more complex relationships between the two these days that make these changes a pain | 14:57 |
| jgriffith | "our side" == "cinder" | 14:57 |
| ildikov | jgriffith: when the spec landed we agreed to have a working version even with more limitations and then refactor later, where possible to have smt more optimal than now | 14:57 |
| jgriffith | ildikov: hey... don't get me wrong, not negative comments about what you and everybody else have done | 14:58 |
| *** lprice1 has quit IRC | 14:58 | |
| ildikov | johnthetubaguy: it's disabled, when the volume allows multiattach and it's set boot_index=0 in the BDM dict | 14:58 |
| *** merooney has quit IRC | 14:58 | |
| *** esker has quit IRC | 14:58 | |
| jgriffith | ildikov: I was just thinking out loud that we should do something more organized in our API's between Cinder/Nova in the future to make these sorts of things go smoother, and address some of the compat issues that mriedem just pointed out | 14:58 |
| ildikov | jgriffith: sorry, I spent too much time with it in the past few weeks and on the other hand I also see it's not a straight forward activity in any sense | 14:59 |
| jgriffith | ildikov: I hear ya | 14:59 |
| ildikov | jgriffith: yeah, I think I or someone else addig this topic to the mid-cycle too to have a cross-project discussion about this | 14:59 |
| johnthetubaguy | jgriffith: we are having similar "coupling" struggles with neutron APIs in a similar way, things like brick and os-vif-lib are helping some, but I think we probably need to go a bit further, but the fact we are talking about it these days gives me hope :) | 14:59 |
| *** merooney has joined #openstack-cinder | 15:00 | |
| jgriffith | johnthetubaguy: YES | 15:00 |
| ildikov | johnthetubaguy: +1 :) | 15:00 |
| jgriffith | johnthetubaguy: the fact that it's even a topic is certainly great progress IMO | 15:00 |
| *** haomaiwa_ has quit IRC | 15:01 | |
| *** vgridnev has joined #openstack-cinder | 15:01 | |
| *** merooney has quit IRC | 15:01 | |
| johnthetubaguy | jgriffith: as with most things, what makes it hard right now is continuing to support live upgrades, etc, at the same time as evolving, but yeah, its on the radar now | 15:01 |
| jgriffith | johnthetubaguy: certainly wasn't a criticism, just a note that there's a real pain point that we're starting to hit more often and admit to, so might be interesting to get together in a room in Austin and brain-storm on some ideas | 15:01 |
| johnthetubaguy | jgriffith: +1 | 15:01 |
| *** esker has joined #openstack-cinder | 15:01 | |
| jgriffith | johnthetubaguy: don't get me started on live-upgrades :) | 15:01 |
| johnthetubaguy | jgriffith: heh | 15:02 |
| ildikov | mriedem: I fixed your comments in the spec amendment BTW | 15:02 |
| *** haomaiwang has joined #openstack-cinder | 15:02 | |
| johnthetubaguy | ildikov: you got the link for that spec change? | 15:02 |
| ildikov | mriedem: and as I said the code is missing only the libvirt version check | 15:02 |
| ildikov | johnthetubaguy: https://review.openstack.org/#/c/264965/ | 15:02 |
| johnthetubaguy | ildikov: sweet, thank you | 15:03 |
| mriedem | ildikov: ok, i haven't read your reply yet | 15:03 |
| mriedem | i have to get stable/kilo working today first | 15:03 |
| ildikov | mriedem: sure, no probs | 15:04 |
| scottda | while on the subject, johnthetubaguy should we have a hangout during the Cinder/Nova mid-cycle week a the end of the month? Perhaps just to bring up subjects like multi-attach, API changes, etc and keep the conversation going? or is that too vague of an agenda? | 15:05 |
| *** eharney has joined #openstack-cinder | 15:08 | |
| *** esker has quit IRC | 15:08 | |
| *** esker has joined #openstack-cinder | 15:09 | |
| smcginnis | scottda, johnthetubaguy: We should block a time that we can link up the two midcycles. | 15:10 |
| smcginnis | scottda, johnthetubaguy: Having time blocked off at the design summit was useful IMO. | 15:10 |
| ildikov | smcginnis: +1 | 15:10 |
| smcginnis | johnthetubaguy, ildikov: Is there an agenda set for the nova midcycle yet? | 15:11 |
| johnthetubaguy | smcginnis: thats a good idea, how do we want to connect? via IRC? | 15:11 |
| smcginnis | johnthetubaguy: Either IRC, or I know we will have a Google hangout. | 15:11 |
| ildikov | johnthetubaguy: or Hangouts maybe? | 15:11 |
| johnthetubaguy | smcginnis: not really, we have an etherpad: https://wiki.openstack.org/wiki/Sprints/NovaMitakaSprint | 15:12 |
| smcginnis | The hangout would be nice to have it a little more face-to-face. | 15:12 |
| johnthetubaguy | its just hang out for a room to another room can really suck, but we can try it | 15:12 |
| johnthetubaguy | can always augment that with IRC I guess | 15:12 |
| smcginnis | johnthetubaguy: Sure, we can give it a shot and fail back to IRC if needed. | 15:12 |
| scottda | Let's give hangout a try. | 15:12 |
| smcginnis | johnthetubaguy: We don't have a set agenda yet either: https://etherpad.openstack.org/p/mitaka-cinder-midcycle | 15:13 |
| johnthetubaguy | would tuesday morning NC, afternoon UK work | 15:13 |
| smcginnis | johnthetubaguy: Or Wednesday? | 15:13 |
| scottda | People can use irc at the same time to make sure their point gets across. | 15:13 |
| smcginnis | johnthetubaguy: Just thinking we start Tuesday morning, so that might be too soon to jump right in to cross-project discussion. | 15:13 |
| scottda | Wed is probably better | 15:13 |
| smcginnis | It might be better to have a day to "get our house in order" before we do it. | 15:14 |
| johnthetubaguy | smcginnis: good point, I meant wednesday, oops | 15:14 |
| smcginnis | johnthetubaguy: Throw in timezones and everything gets more difficult. ;) | 15:14 |
| smcginnis | johnthetubaguy: That sounds good. We'll reserve Wednesday morning for Nova discussions. | 15:14 |
| *** esker has quit IRC | 15:15 | |
| scottda | I'll make sure it's known at the next meeting for both teams | 15:15 |
| smcginnis | scottda: Thanks! | 15:15 |
| johnthetubaguy | smcginnis: cool, added that into our etherpad, lets try for tht | 15:17 |
| smcginnis | johnthetubaguy: Perfect! | 15:17 |
| *** vignesh has quit IRC | 15:17 | |
| Roamer` | hmm, I know it's kind of close to the deadline, but could somebody take a quick look at https://review.openstack.org/#/c/220155/ - reintroducing the StorPool driver? | 15:18 |
| smcginnis | Roamer`: What is the status of the CI for StorPool? | 15:19 |
| *** esker has joined #openstack-cinder | 15:20 | |
| Roamer` | smcginnis, it's mostly operational, running in silent mode | 15:20 |
| smcginnis | Roamer`: What is your plan to enable it? | 15:21 |
| Roamer` | smcginnis, I thought I'd ask somebody from Cinder to check it out and let -infra know that it should be reenabled today, but something else happened: I've been talking to the guys in -infra about a problem with another CI that killed my Zuul | 15:21 |
| Roamer` | smcginnis, there's a third-party CI that is sending comments in a loop to https://review.openstack.org/#/c/264056 and our CI is not handling it very well, it can't get a word in edgewise | 15:22 |
| Roamer` | smcginnis, right now yolanda in #openstack-infra is trying to fix that... after that, our CI should start picking up on new Gerrit events again | 15:22 |
| *** dsunkum has joined #openstack-cinder | 15:22 | |
| *** dansmith is now known as superdan | 15:23 | |
| smcginnis | Roamer`: OK, we'll need to see that up and reliable reporting before your driver can be accepted. | 15:23 |
| smcginnis | Roamer`: My recommendation is to make that your top priority. | 15:23 |
| Roamer` | smcginnis, fair enough | 15:23 |
| openstackgerrit | Kazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver https://review.openstack.org/255239 | 15:23 |
| Roamer` | smcginnis, thanks | 15:24 |
| smcginnis | Roamer`: No problem. Good luck! | 15:24 |
| smcginnis | rhe00: ping | 15:24 |
| *** esker has quit IRC | 15:25 | |
| *** diogogmt has joined #openstack-cinder | 15:25 | |
| *** lprice has joined #openstack-cinder | 15:26 | |
| *** esker has joined #openstack-cinder | 15:30 | |
| *** breitz has quit IRC | 15:30 | |
| *** nkrinner has quit IRC | 15:30 | |
| *** esker has quit IRC | 15:30 | |
| *** breitz has joined #openstack-cinder | 15:31 | |
| *** mtanino has joined #openstack-cinder | 15:31 | |
| *** shyama has quit IRC | 15:32 | |
| *** cfriesen__ has quit IRC | 15:35 | |
| *** crose has joined #openstack-cinder | 15:35 | |
| *** cfriesen__ has joined #openstack-cinder | 15:35 | |
| *** xyang has quit IRC | 15:36 | |
| *** mragupat has joined #openstack-cinder | 15:38 | |
| *** timcl has quit IRC | 15:39 | |
| *** geguileo_phone has quit IRC | 15:40 | |
| *** cfriesen__ has quit IRC | 15:40 | |
| *** geguileo_phone has joined #openstack-cinder | 15:41 | |
| *** timcl has joined #openstack-cinder | 15:42 | |
| *** e0ne has quit IRC | 15:42 | |
| *** xyang has joined #openstack-cinder | 15:45 | |
| *** xyang has quit IRC | 15:45 | |
| *** alonma has joined #openstack-cinder | 15:45 | |
| *** links has joined #openstack-cinder | 15:47 | |
| *** sgotliv has joined #openstack-cinder | 15:49 | |
| *** diogogmt has quit IRC | 15:50 | |
| *** alonma has quit IRC | 15:50 | |
| *** garthb__ has joined #openstack-cinder | 15:51 | |
| *** zhangjn has joined #openstack-cinder | 15:52 | |
| *** xyang has joined #openstack-cinder | 15:52 | |
| *** alonma has joined #openstack-cinder | 15:52 | |
| *** lcurtis has joined #openstack-cinder | 15:53 | |
| *** markus_z has quit IRC | 15:53 | |
| *** zhangjn has quit IRC | 15:54 | |
| openstackgerrit | Petrut Lucian proposed openstack/cinder: Refactor Windows drivers using os-win https://review.openstack.org/243565 | 15:55 |
| *** zhangjn has joined #openstack-cinder | 15:55 | |
| *** alonma has quit IRC | 15:56 | |
| *** jwcroppe has joined #openstack-cinder | 15:57 | |
| *** shyama has joined #openstack-cinder | 15:57 | |
| *** lcurtis has quit IRC | 15:58 | |
| *** sgotliv has quit IRC | 15:58 | |
| *** esker has joined #openstack-cinder | 16:00 | |
| *** sgotliv has joined #openstack-cinder | 16:00 | |
| *** esker has quit IRC | 16:00 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** C1ShaunID has left #openstack-cinder | 16:01 | |
| *** haomaiwang has joined #openstack-cinder | 16:01 | |
| *** rhe00 is now known as rhedlind | 16:02 | |
| rhedlind | smcginnis: I'm online now | 16:02 |
| *** diogogmt has joined #openstack-cinder | 16:03 | |
| smcginnis | rhedlind: Great, saw your other comments. Just wanted to make sure you were aware of what was going on. | 16:04 |
| *** alonma has joined #openstack-cinder | 16:04 | |
| *** zhangjn has quit IRC | 16:05 | |
| *** lpetrut has quit IRC | 16:06 | |
| mtanino | smcginnis: Hi, May I add trivial feature(without BP or spec which I asked you in previous) on the bottom of tracking? https://etherpad.openstack.org/p/mitaka-cinder-spec-review-tracking | 16:07 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Fix tox -e fast8 https://review.openstack.org/263792 | 16:07 |
| *** diogogmt has quit IRC | 16:08 | |
| smcginnis | mtanino: Yeah, that should be fine. | 16:08 |
| mtanino | smcginnis: Thanks :) | 16:08 |
| smcginnis | mtanino: No problem. | 16:08 |
| *** salv-orl_ has joined #openstack-cinder | 16:08 | |
| *** alonma has quit IRC | 16:08 | |
| *** zigo has quit IRC | 16:09 | |
| *** alonma has joined #openstack-cinder | 16:10 | |
| *** zigo has joined #openstack-cinder | 16:10 | |
| *** belmoreira has quit IRC | 16:10 | |
| *** salv-orlando has quit IRC | 16:11 | |
| *** kfarr has joined #openstack-cinder | 16:11 | |
| *** diogogmt has joined #openstack-cinder | 16:12 | |
| *** alonma has quit IRC | 16:14 | |
| Swanson | xyang, Thanks for kicking off the receck on the Dell replication v2 patch. Seems to have run through fine the second time. | 16:15 |
| *** hemnafk is now known as hemna | 16:16 | |
| *** jungleboyj has joined #openstack-cinder | 16:17 | |
| xyang | Swanson: np, I'll take a look | 16:17 |
| Swanson | xyang, Thanks for the reviews on that. | 16:18 |
| xyang | Swanson: you are welcome | 16:18 |
| *** alonma has joined #openstack-cinder | 16:18 | |
| *** vinayp has quit IRC | 16:19 | |
| *** mriedem is now known as mriedem_meeting | 16:21 | |
| *** alonma has quit IRC | 16:23 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Add missing glance_metadata field to Snapshot https://review.openstack.org/263317 | 16:24 |
| *** pradipm has quit IRC | 16:24 | |
| *** dims has quit IRC | 16:25 | |
| *** sheel has joined #openstack-cinder | 16:26 | |
| *** dims has joined #openstack-cinder | 16:26 | |
| *** links has quit IRC | 16:27 | |
| *** salv-orl_ has quit IRC | 16:29 | |
| *** ntpttr has joined #openstack-cinder | 16:30 | |
| *** salv-orlando has joined #openstack-cinder | 16:30 | |
| jgriffith | dulek: ping | 16:31 |
| dulek | jgriffith: Pong | 16:31 |
| jgriffith | dulek: hey... do you understand the purpose of all this glance_image_metadata ? | 16:32 |
| dulek | jgriffith: I believe I do. What's up? | 16:32 |
| jgriffith | dulek: well, I've never understood: | 16:33 |
| jgriffith | 1. Where the explosion of all these metadata entries came from :( | 16:33 |
| *** jistr has quit IRC | 16:33 | |
| jgriffith | 2. What purpose they serve | 16:33 |
| jgriffith | dulek: I posted a comment here: https://review.openstack.org/#/c/263317/3 | 16:33 |
| jgriffith | for example, grep cinder and glance and there's no direct usage of that info at all | 16:33 |
| jgriffith | I know HP public cloud wanted it for some billing stuff | 16:34 |
| jgriffith | but given they don't exist anymore does anybody else care? | 16:34 |
| dulek | jgriffith: Riiight… Let me try to help with that. | 16:34 |
| jgriffith | and FWIW I never quite followed the logic when they argued for it either | 16:34 |
| *** lcurtis has joined #openstack-cinder | 16:34 | |
| dulek | So metadata are used internally to do a lot of things. | 16:34 |
| dulek | For example metadata on flavor can express hardware requirements. | 16:35 |
| jgriffith | sure | 16:35 |
| dulek | You can also set a lot of metadata on images in Glance. | 16:35 |
| dulek | These are used for example when booting a VM directly from an image. | 16:35 |
| dulek | Let me find an example… | 16:35 |
| jgriffith | dulek: used by Nova? | 16:36 |
| jgriffith | dulek: here's the thing that's always bugged me.... | 16:36 |
| dulek | https://github.com/openstack/nova/blob/master/nova/scheduler/filters/image_props_filter.py#L106-L107 | 16:36 |
| dulek | That's an example. | 16:36 |
| jgriffith | dulek: we have: volume_metadata, volume_admin_metadata, volume_glance_metadata, snapshot_metadata, snapshot_glance_metadata, service_metadata.... | 16:37 |
| jgriffith | It's a bit out of control IMHO | 16:37 |
| hemna | can we get metadata_metadata ? | 16:37 |
| dulek | Okay, so just let me finish with glance ones. ;) | 16:37 |
| jgriffith | hemna: I think we have it :) | 16:37 |
| dulek | hemna: metametadata | 16:37 |
| hemna | :) | 16:37 |
| jgriffith | dulek: yeah, so that example helps | 16:37 |
| dulek | But there's more. | 16:37 |
| jgriffith | dulek: but here's what annoys me about it... | 16:37 |
| dulek | You can for example request certain amount of RAM allocated to graphics through image metadata. | 16:38 |
| jgriffith | dulek: this is the same case.... we already have glance_property (they don't call it metadata) so why aren't we just uploading that as generic info? | 16:38 |
| jgriffith | dulek: rather than creating all these references | 16:38 |
| jgriffith | dulek: or just embed it in the snapshot/volume metadata that we already had | 16:38 |
| jgriffith | volume_metadata = {glance_prop: {....}} | 16:39 |
| dulek | jgriffith: Yeah, that would probably be better. | 16:39 |
| jgriffith | or I guess s/glance_prop/image_prop/ | 16:39 |
| dulek | Because we're storing glance_metadata to pass them to nova when booting from volume. | 16:39 |
| dulek | And on snapshot to copy them to a volume when creating it from a snapshot. | 16:39 |
| jgriffith | dulek: ok, well thanks for helping me with that | 16:40 |
| dulek | And probably to copy them back to Glance when copying volume to image. | 16:40 |
| *** dslevin has joined #openstack-cinder | 16:40 | |
| jgriffith | dulek: I wanted to make sure there wasn't something I was missing somewhere | 16:40 |
| dulek | I haven't ever thought of how to do that in a proper way. | 16:40 |
| dulek | It was like "there" when I came in to the project. ;) | 16:40 |
| jgriffith | I guess I don't understand like I said why the sub key in the metadata wouldn't be fine? | 16:41 |
| jgriffith | so you just have snapshot_metadata and volume_metadata | 16:41 |
| johnthetubaguy | FWIW, we have an object for that stuff thats in glance now: https://github.com/openstack/nova/blob/master/nova/objects/image_meta.py | 16:41 |
| jgriffith | get rid of all the crazy other ones | 16:41 |
| jgriffith | johnthetubaguy: looking... | 16:41 |
| dulek | I think it's bound to 255 chars, so that would probably be a little small space. | 16:41 |
| dulek | I mean - metadata key. | 16:42 |
| jgriffith | dulek: true | 16:42 |
| *** dslevin has quit IRC | 16:42 | |
| jgriffith | dulek: but there has to be a better way than what we're doing now... looking at the nova code johnthetubaguy just mentioned | 16:42 |
| *** dslevin__ has joined #openstack-cinder | 16:42 | |
| jgriffith | johnthetubaguy: dulek ok, so this seems like it would be wayyy better | 16:43 |
| dulek | Let me take a look… | 16:43 |
| jgriffith | Use a full image_metadata (or image-property) table | 16:43 |
| jgriffith | and backref that to a volume or snapshot or whatever | 16:44 |
| jgriffith | that also means it's got a chance at being something that's standardized and enforced | 16:44 |
| dulek | Yeah, looks great… But how to keep the list in sync between Nova, Cinder and GLance? | 16:44 |
| jgriffith | as opposed to just a bunch of random stuff that people shove things in to | 16:44 |
| dulek | I mean - when Nova adds a property key that does something we would also need to do su. | 16:44 |
| dulek | s/su/so | 16:44 |
| jgriffith | dulek: so that's something else that troubles me :) | 16:45 |
| jgriffith | dulek: there's no real single source of truth right now | 16:45 |
| jgriffith | well... I guess Nova would be | 16:45 |
| *** dslevi___ has joined #openstack-cinder | 16:45 | |
| jgriffith | because.. well, that's what really need/uses this data | 16:45 |
| jgriffith | I hate to say this... but it makes me think that this should actually be a shared library | 16:46 |
| jgriffith | that way everybody is synced up off of one single version | 16:46 |
| dulek | There are some descriptions in Glance also that are used to display properties in Horizon metadata view. | 16:46 |
| jgriffith | or we just stay in lock step with what Nova has | 16:46 |
| jgriffith | dulek: well... so here's where my selfishness comes in | 16:46 |
| *** timcl has quit IRC | 16:46 | |
| jgriffith | dulek: I've been burned by Glance enough in the past changing their properties and breaking them that I'd prefer they didn't own them :) | 16:47 |
| dulek | If we have a DB column per property, then that lib would need to add migrations and stuff when things change. This gets a little complicated. | 16:47 |
| jgriffith | dulek: yeah, it's certainly not trivial | 16:47 |
| *** dslevin__ has quit IRC | 16:47 | |
| dulek | Nova should certainly own them. Just mentioning Glance is having a catalog with them also. | 16:47 |
| jgriffith | glance should just go away... and be absorbed by modules in Nova :) | 16:47 |
| * dulek thinks of Glance like a Swift API. ;) | 16:48 | |
| jgriffith | I | 16:48 |
| dulek | s/like a/like of | 16:48 |
| *** edtubill has joined #openstack-cinder | 16:48 | |
| jgriffith | Yeah, but then they decided to be more. And you run into issues because there's Ceph, NFS etc | 16:49 |
| jgriffith | hmmm | 16:49 |
| dulek | And speaking about the bluex patch you've mentioned in the beginning - it's just synchronizing objects with what's in DB. | 16:49 |
| dulek | And to be honest it's not that critical now - just kind of refactoring. | 16:50 |
| mtanino | seems interesting topic for me. | 16:51 |
| scottda | hemna: pong | 16:51 |
| hemna | hey | 16:51 |
| dulek | mtanino: Your IRC client highlights the word "refactoring"? :D | 16:52 |
| jgriffith | dulek: LOL | 16:52 |
| mtanino | dulek: catch the keyword glance metadata :) | 16:52 |
| scottda | hemna: You ping'ed yesterday PM. Perhaps the ping has expired.... | 16:53 |
| *** dsunkum has quit IRC | 16:53 | |
| *** jwcroppe has quit IRC | 16:54 | |
| hemna | oh, yah I don't remember now | 16:54 |
| hemna | oh! | 16:54 |
| hemna | heh | 16:54 |
| hemna | microversions | 16:54 |
| scottda | The solution to everything® | 16:54 |
| openstackgerrit | Alex Meade proposed openstack/cinder: WIP: Try to get service name in logs https://review.openstack.org/265351 | 16:55 |
| smcginnis | scottda: :) | 16:55 |
| mtanino | dulek: Seems you have good knowledge about glance metadata, could you give me feedback for this? Support image_volume_type in image metadata https://review.openstack.org/#/c/258649/ | 16:56 |
| dulek | jgriffith: So summing up - I think we can make the way we handle metadata better. Maybe we can follow up on the midcycle? I'll be lucky enough to be there. :) | 16:56 |
| hemna | scottda, https://review.openstack.org/#/c/224910/ | 16:56 |
| mtanino | oh, midcycle. | 16:56 |
| hemna | jenkins isn't happy | 16:57 |
| scottda | hemna: Yes, see the commit message | 16:57 |
| dulek | mtanino: Added to the review queue. I'm swamped, but I'll try to look before Monday morning US time. | 16:57 |
| scottda | Grenade fails and needs a patch | 16:57 |
| smcginnis | hemna: Need ceilometer updated first. | 16:57 |
| smcginnis | scottda: :P | 16:57 |
| hemna | doh | 16:57 |
| jgriffith | dulek: yeah, that would be cool | 16:57 |
| mtanino | dulek: thank you for your hard work :) | 16:57 |
| scottda | Patch is up for review and linked in commit message | 16:57 |
| jgriffith | dulek: we should poke at what Nova has and see if there's a way to tie in there that isn't brittle | 16:57 |
| hemna | scottda, ok, lets get that through :) | 16:58 |
| smcginnis | scottda: Jenkins hasn't reported on that yet, has it. | 16:58 |
| * smcginnis looking for link | 16:58 | |
| scottda | Yeah, I reckon we are pretty sure we want microversions, so the grenade patch can go. | 16:58 |
| hemna | smcginnis, not yet | 16:58 |
| *** dslevi___ has quit IRC | 16:59 | |
| scottda | smcginnis: Yes, I had a bug in the grenade patch. patrickeast pointed it out in review. New patch uploaded, so grenade patch still needs to pass | 16:59 |
| smcginnis | hemna: Thanks. It looks good now, so I think once Jenkins is OK we should probably move ahead. | 16:59 |
| * dulek never thought that the fact he started developing in OpenStack from Glance would make any difference | 16:59 | |
| scottda | Should be mere moments before we know..... | 16:59 |
| smcginnis | scottda: Awesome! | 16:59 |
| *** mragupat has quit IRC | 16:59 | |
| scottda | BTW, I'm looking for someone who is expert in api_paste | 16:59 |
| jgriffith | dulek: :) | 16:59 |
| scottda | Any ideas whom that might be? | 17:00 |
| scottda | thingee: ? | 17:00 |
| *** mragupat has joined #openstack-cinder | 17:00 | |
| *** haomaiwang has quit IRC | 17:01 | |
| hemna | scottda, that'd be my guess | 17:01 |
| *** haomaiwang has joined #openstack-cinder | 17:01 | |
| *** erlon has joined #openstack-cinder | 17:03 | |
| *** sombrafam has joined #openstack-cinder | 17:05 | |
| *** mriedem_meeting is now known as mriedem | 17:07 | |
| *** dslevin has joined #openstack-cinder | 17:07 | |
| *** lpetrut has joined #openstack-cinder | 17:07 | |
| smcginnis | scottda: Jenkins not too happy. | 17:09 |
| hemna | boo | 17:09 |
| jordanP | guys I am a not sure about https://review.openstack.org/#/c/261215/11 It wants to remove testing of cinder backup when using Cinder API v1 | 17:09 |
| jordanP | "backups API exists on cinder v2 API only" is that true ? | 17:09 |
| *** dslevin has quit IRC | 17:10 | |
| hemna | scottda, 2016-01-08 17:01:52.755 | cp: cannot stat '/opt/stack/new/etc/cinder/api_paste.ini': No such file or directory | 17:11 |
| jgriffith | jordanP: not true | 17:11 |
| hemna | http://logs.openstack.org/95/264995/2/check/gate-grenade-dsvm/206892b/logs/grenade.sh.txt.gz | 17:11 |
| jgriffith | jordanP: backups are an extension so they're in both | 17:11 |
| jordanP | jgriffith, thanks. Could you leave a quick comment on the patch ? | 17:11 |
| jgriffith | jordanP: will do | 17:12 |
| jordanP | as you are a core reviewer for Cinder, that should weight | 17:12 |
| jgriffith | jordanP: I'm just wading through trying to figure out what this patch is trying to do :) | 17:12 |
| jgriffith | Oh... hehe | 17:12 |
| jgriffith | they just delete teh backup tests in V1 | 17:12 |
| jgriffith | nope | 17:13 |
| *** leeantho has joined #openstack-cinder | 17:14 | |
| jgriffith | jordanP: comment added, lemme know if that makes sense | 17:15 |
| jgriffith | jordanP: the main point being is that this logic would indicate you remove ALL cinder-v1 tests | 17:15 |
| jgriffith | jordanP: which as you point out is no good at all | 17:15 |
| hemna | scottda, I think I found it. | 17:15 |
| hemna | scottda, I added it in the review | 17:16 |
| jordanP | jgriffith, agreed. Thanks a lot ! | 17:16 |
| *** esker has joined #openstack-cinder | 17:16 | |
| guitarzan | jgriffith: you just gave someone else an idea for a patch... | 17:17 |
| jgriffith | guitarzan: ha | 17:17 |
| scottda | hemna: thanks | 17:18 |
| hemna | scottda, does that look right ? | 17:18 |
| *** sgotliv has quit IRC | 17:18 | |
| *** alonma has joined #openstack-cinder | 17:19 | |
| *** leeantho has quit IRC | 17:20 | |
| *** shyama has quit IRC | 17:21 | |
| *** mriedem is now known as mriedem_lunch | 17:23 | |
| *** alonma has quit IRC | 17:23 | |
| *** esker has quit IRC | 17:25 | |
| scottda | hemna: yes, thanks | 17:25 |
| *** alonma has joined #openstack-cinder | 17:25 | |
| scottda | I'm a bit sloppy here. I should run Grenade locally to verify before submitting . | 17:25 |
| *** e0ne has joined #openstack-cinder | 17:25 | |
| hemna | ok coolio np | 17:25 |
| *** superdan has quit IRC | 17:27 | |
| *** dansmith has joined #openstack-cinder | 17:27 | |
| *** timcl has joined #openstack-cinder | 17:28 | |
| *** mudassirlatif has joined #openstack-cinder | 17:29 | |
| *** alonma has quit IRC | 17:30 | |
| *** dansmith is now known as superdan | 17:31 | |
| *** salv-orlando has quit IRC | 17:35 | |
| *** edtubill has quit IRC | 17:35 | |
| *** salv-orlando has joined #openstack-cinder | 17:36 | |
| sheel | jgriffith: Hi | 17:36 |
| sheel | jgriffith: are we considering "https://bugs.launchpad.net/cinder/+bug/1158328" for next release? | 17:37 |
| openstack | Launchpad bug 1158328 in Cinder "passwords in config files stored in plaintext" [Wishlist,Confirmed] | 17:37 |
| sheel | jgriffith:or we are planning to wontfix like nova did? | 17:37 |
| *** chhavi has quit IRC | 17:38 | |
| jgriffith | sheel: in the past I'd been of the opinion that security on the nodes was what it took, but there's been enough feedback from some enterprises that don't want to or can't fight issues that I think it's worth looking at a solution | 17:39 |
| *** dslevin__ has joined #openstack-cinder | 17:39 | |
| *** esker has joined #openstack-cinder | 17:39 | |
| jgriffith | sheel: the trick is to make sure it's compatable | 17:39 |
| jgriffith | sheel: so "old" config files still work | 17:39 |
| jgriffith | sheel: even just adding a decrypt utility to Cinder would solve this | 17:40 |
| sheel | jgriffith:hmmm, right. | 17:40 |
| sheel | jgriffith:this seems a bit interesting | 17:41 |
| jgriffith | sheel: it's tricky, I agree with most of the comments from sdague, fungi and others | 17:41 |
| jgriffith | sheel: but at the same time I get hammered by a handful of customers on this subject | 17:42 |
| jgriffith | sheel: it's easy for me to say "secure you're node, it's not a regulatory issue" as I'm not the one running their data center :) | 17:42 |
| *** dslevin__ has quit IRC | 17:42 | |
| sheel | jgriffith:yes, but I think almost all systems take care of such security things on application layer..so i just pinged to remind of this... | 17:43 |
| jgriffith | sheel: yeah, there's valid points on both side of the argument | 17:44 |
| *** esker has quit IRC | 17:44 | |
| jgriffith | sheel: the reality is however I've not come up with a solution that will satisfy everyone :( | 17:44 |
| *** gouthamr has quit IRC | 17:44 | |
| jgriffith | sheel: I'm certainly open to suggestions and innovation though :) | 17:44 |
| *** jordanP has quit IRC | 17:45 | |
| sheel | jgriffith:sure, I would look into it in next days...will discuss in some cinder meeting sooner or later.. | 17:45 |
| jgriffith | sheel: my enterprise is "my" enterprise behind firewalls, passwords etc so I don't worry about this much | 17:45 |
| jgriffith | sheel: not to mention I don't exactly have any secret/critical data :) | 17:45 |
| sheel | jgriffith: :) | 17:45 |
| jgriffith | sheel: awesome, keep me posted, it would be cool if you have a good solution :) | 17:46 |
| sheel | jgriffith: sure, please share in case any suggestions are already discussed at some place, some document other than on bug itself | 17:47 |
| *** dslevin has joined #openstack-cinder | 17:48 | |
| *** dslevin has quit IRC | 17:48 | |
| *** dslevin has joined #openstack-cinder | 17:48 | |
| jgriffith | sheel: I think you've got about as much info as I do :) | 17:53 |
| jgriffith | sheel: the customers I talk to don't have solutions/ideas, just that plain text is unacceptable :) | 17:53 |
| sheel | jgriffith: ok, thank you :) | 17:55 |
| jgriffith | sheel: for sure | 17:55 |
| *** timcl1 has joined #openstack-cinder | 17:56 | |
| *** timcl has quit IRC | 17:59 | |
| jgriffith | hemna: hey... can you explain more what the idea is here: https://review.openstack.org/#/c/204191/17 | 18:00 |
| jgriffith | hemna: so I believe the idea is to get path info *from cinder* somehow? | 18:00 |
| *** haomaiwang has quit IRC | 18:01 | |
| *** e0ne has quit IRC | 18:01 | |
| jgriffith | but I'm unclear how that would work, I'm also uncertain about calling initialize_connection on a device that's already attached? | 18:01 |
| *** haomaiwang has joined #openstack-cinder | 18:01 | |
| jgriffith | then throw in the use of factories and well... I dunno | 18:01 |
| *** dslevin has quit IRC | 18:02 | |
| *** dims_ has joined #openstack-cinder | 18:02 | |
| jgriffith | also curious, if brick becomes wide-spread and lives out on Nova and other places... how do you "know" this is a cinder-volume node or not | 18:02 |
| *** cknight1 has joined #openstack-cinder | 18:02 | |
| *** dslevin has joined #openstack-cinder | 18:02 | |
| *** mragupat_ has joined #openstack-cinder | 18:02 | |
| jgriffith | because based on the commit mesage it states it's to be used on a cinder-volume node | 18:03 |
| jgriffith | but meanwhile cinder-volume nodes don't have "paths" | 18:03 |
| jgriffith | so I ARE confused | 18:03 |
| *** kfarr has quit IRC | 18:04 | |
| *** dims has quit IRC | 18:05 | |
| *** mragupat has quit IRC | 18:06 | |
| *** cknight has quit IRC | 18:06 | |
| *** rhedlind has quit IRC | 18:06 | |
| *** rhe00 has joined #openstack-cinder | 18:06 | |
| *** dslevin has quit IRC | 18:07 | |
| *** alonma has joined #openstack-cinder | 18:09 | |
| *** alonma has quit IRC | 18:13 | |
| *** alonma has joined #openstack-cinder | 18:15 | |
| *** alonma has quit IRC | 18:19 | |
| openstackgerrit | Dan Nguyen proposed openstack/cinder: Fixes cinder quota mgmt for keystone v3 https://review.openstack.org/253759 | 18:21 |
| *** alonma has joined #openstack-cinder | 18:24 | |
| *** alonma has quit IRC | 18:28 | |
| hemna | hey | 18:30 |
| hemna | back | 18:30 |
| hemna | I'm going through the review and answering | 18:30 |
| hemna | but, cinder already has to be safe wrt initialize_connection | 18:30 |
| hemna | nova already calls initialize_connection multiple times for the volume that's already attached. Nova simply wants the return value from that, not any action being taken on cinder's side. | 18:31 |
| *** alonma has joined #openstack-cinder | 18:33 | |
| *** leeantho has joined #openstack-cinder | 18:34 | |
| *** gouthamr has joined #openstack-cinder | 18:35 | |
| *** mriedem_lunch is now known as mriedem | 18:35 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Add volume fail-back capabilities https://review.openstack.org/265399 | 18:36 |
| jgriffith | hemna: calls to initialize_connection are the least of my concerns with that patch | 18:36 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Add volume fail-back capabilities https://review.openstack.org/265399 | 18:37 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Adding volume checks to manage snapshot API https://review.openstack.org/260728 | 18:37 |
| hemna | sure | 18:37 |
| hemna | I'll finish my answers | 18:37 |
| hemna | sec | 18:37 |
| *** alonma has quit IRC | 18:38 | |
| *** gouthamr_ has joined #openstack-cinder | 18:38 | |
| *** alonma has joined #openstack-cinder | 18:39 | |
| hemna | ok I posted them | 18:41 |
| hemna | fwiw | 18:41 |
| hemna | leeantho, | 18:41 |
| *** gouthamr has quit IRC | 18:41 | |
| leeantho | hemna, hey | 18:42 |
| hemna | ^^ | 18:42 |
| hemna | leeantho, need to clarify some things on your os-brick tool patch | 18:42 |
| *** alonma has quit IRC | 18:43 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Add volume fail-back capabilities https://review.openstack.org/265399 | 18:45 |
| jgriffith | leeantho: sorry, didn't know your nic on IRC :) | 18:45 |
| jgriffith | leeantho: maybe you can clairfy some of the questions I posted? | 18:45 |
| jgriffith | It's still really unclear *who* and *where* this script is being run, and if/why it needs any real cinder interaction at all. but I fully admitted that I am probably just really dense here | 18:46 |
| *** rlrossit has quit IRC | 18:47 | |
| leeantho | jgriffith, I'll read through them and try to clarify some of the sections | 18:47 |
| *** alonma has joined #openstack-cinder | 18:48 | |
| *** rlrossit_ has joined #openstack-cinder | 18:50 | |
| *** alonma has quit IRC | 18:53 | |
| *** delattec has joined #openstack-cinder | 18:53 | |
| *** delatte has quit IRC | 18:56 | |
| *** sombrafam has quit IRC | 18:57 | |
| *** alonma has joined #openstack-cinder | 18:57 | |
| *** haomaiwang has quit IRC | 19:01 | |
| *** haomaiwang has joined #openstack-cinder | 19:01 | |
| *** alonma has quit IRC | 19:02 | |
| *** timcl1 has quit IRC | 19:02 | |
| *** timcl has joined #openstack-cinder | 19:10 | |
| openstackgerrit | Merged openstack/python-brick-cinderclient-ext: Use git.openstack.org https://review.openstack.org/263672 | 19:11 |
| leeantho | jgriffith, the tool would be used by an admin/developer/tester/CI who want to verify that attach and detach operations are not leaving volume paths behind. The problem originally came up with live migration testing a while ago but could be useful for other attach/detach scenarios. I've had the patch for the tool in os-brick for some time but that might not be the best place for it to exist even though it uses brick APIs. | 19:11 |
| *** cfriesen__ has joined #openstack-cinder | 19:13 | |
| jgriffith | leeantho: ok, but then I have to wonder... why all the hoops? I mean, why not just "ls /dev/disk/by-path" and "iscsiadm sessions" ? | 19:13 |
| openstackgerrit | Michael Krotscheck proposed openstack/cinder: Added Keystone and RequestID headers to CORS middleware https://review.openstack.org/265409 | 19:15 |
| *** gouthamr_ has quit IRC | 19:15 | |
| leeantho | It was mainly to help automate the process because sometimes paths were only intermittently left behind. Instead of having to do the same steps many times manually (if doing something like 100+ tests) you could automate calls to the tool to verify and check after each attach/detach or live migration attempt. That could then just keep track of pass/fails for the attempts (outside of that brick tool) | 19:16 |
| leeantho | brick also made it easier to check different connector types outside of iscsi with its apis. some connectors use different locations for volume paths | 19:19 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Add finish_volume_migration to volume object https://review.openstack.org/257667 | 19:21 |
| *** sombrafam has joined #openstack-cinder | 19:22 | |
| *** geguileo_mobile has joined #openstack-cinder | 19:23 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Add volume fail-back capabilities https://review.openstack.org/265399 | 19:23 |
| *** geguileo_phone has quit IRC | 19:25 | |
| openstackgerrit | Michael Krotscheck proposed openstack/cinder: Added Keystone and RequestID headers to CORS middleware https://review.openstack.org/265409 | 19:25 |
| smcginnis | scottda: Jenkins still not happy with the api-paste bit. :/ | 19:28 |
| *** lpetrut has quit IRC | 19:28 | |
| hemna | what! | 19:28 |
| smcginnis | hemna: Jenkins is cranky today. | 19:28 |
| *** sombrafam has quit IRC | 19:29 | |
| scottda | yeah, I'm sure it's me.. | 19:29 |
| *** lpetrut has joined #openstack-cinder | 19:29 | |
| hemna | cp: cannot stat '/opt/stack/new/cinder/etc/cinder/api_paste.ini': No such file or directory | 19:29 |
| hemna | wth | 19:29 |
| *** ociuhandu has quit IRC | 19:29 | |
| hemna | sorry scottda I thought that was the right path. | 19:29 |
| hemna | :(! | 19:29 |
| bswartz | smcginnis: when is jenkins not cranky? | 19:30 |
| smcginnis | bswartz: Jenkins needs to get more sleep and more coffee I think. | 19:30 |
| smcginnis | Always complaining! :) | 19:30 |
| hemna | scottda, do we need a sudo in front of the cp ? | 19:30 |
| *** chris_morrell has joined #openstack-cinder | 19:30 | |
| scottda | hemna: I thought about that, but the file isn't owned by root on my devstack. | 19:31 |
| *** rushiagr_away is now known as rushiagr | 19:31 | |
| scottda | But it could be correct to use sudo | 19:31 |
| hemna | wth | 19:31 |
| hemna | the line above it is copying stuff from the same place | 19:31 |
| hemna | volume.filters | 19:31 |
| smcginnis | +1 | 19:31 |
| scottda | but line above uses sudo, so you might be right | 19:31 |
| hemna | smh | 19:31 |
| hemna | I dunno | 19:31 |
| hemna | try it ? | 19:32 |
| scottda | yep | 19:32 |
| hemna | you'd think it would complain about perms | 19:32 |
| *** cfriesen__ has quit IRC | 19:32 | |
| hemna | wait! | 19:32 |
| hemna | I know what it is | 19:32 |
| hemna | :P | 19:32 |
| hemna | that's hard to see | 19:33 |
| hemna | scottda, check the review :P | 19:33 |
| smcginnis | hemna: No comment in there. | 19:34 |
| hemna | bah | 19:34 |
| hemna | crap | 19:35 |
| hemna | <rant> I really hate the new gerrit UI </rant> | 19:35 |
| hemna | my daily ration of hatred for the new ui. | 19:35 |
| smcginnis | hemna: That was a short one today. :) | 19:35 |
| hemna | hehe yah | 19:36 |
| scottda | sh$!*t !!! | 19:36 |
| hemna | I guess you eventually get used to having your fingers smashed with a hammer. | 19:36 |
| hemna | daily | 19:36 |
| scottda | I think it's best if I go back to bed for a little while, and try this again. | 19:37 |
| hemna | :) | 19:38 |
| *** xyang has quit IRC | 19:39 | |
| *** delatte has joined #openstack-cinder | 19:40 | |
| *** delattec has quit IRC | 19:40 | |
| jgriffith | scottda: just go shovel some snow for a while... clears the mind :) | 19:46 |
| *** itzdilip has quit IRC | 19:47 | |
| scottda | hehe..good idea. I already did, But it's piled up again... | 19:47 |
| *** sombrafam has joined #openstack-cinder | 19:47 | |
| *** merooney has joined #openstack-cinder | 19:53 | |
| *** [1]Thelo has joined #openstack-cinder | 19:55 | |
| *** delatte has quit IRC | 19:55 | |
| *** Thelo has quit IRC | 19:57 | |
| *** [1]Thelo is now known as Thelo | 19:57 | |
| *** chris_morrell has quit IRC | 19:58 | |
| *** alonma has joined #openstack-cinder | 19:58 | |
| *** cknight1 has quit IRC | 19:59 | |
| *** haomaiwang has quit IRC | 20:01 | |
| *** haomaiwang has joined #openstack-cinder | 20:01 | |
| *** sombrafam has quit IRC | 20:01 | |
| *** chris_morrell has joined #openstack-cinder | 20:02 | |
| *** alonma has quit IRC | 20:02 | |
| *** chris_morrell has quit IRC | 20:03 | |
| *** alonma has joined #openstack-cinder | 20:04 | |
| *** bardia has joined #openstack-cinder | 20:04 | |
| *** chris_morrell has joined #openstack-cinder | 20:05 | |
| *** diablo_rojo has quit IRC | 20:06 | |
| *** chris_morrell has quit IRC | 20:07 | |
| *** chris_morrell has joined #openstack-cinder | 20:07 | |
| *** sheel has quit IRC | 20:07 | |
| *** ChubYann has joined #openstack-cinder | 20:08 | |
| *** alonma has quit IRC | 20:08 | |
| *** chris_morrell has quit IRC | 20:10 | |
| *** Lee1092 has quit IRC | 20:11 | |
| *** cdelatte has joined #openstack-cinder | 20:13 | |
| openstackgerrit | Merged openstack/cinder: Replace deprecated library function os.popen() with subprocess https://review.openstack.org/262729 | 20:14 |
| *** gouthamr has joined #openstack-cinder | 20:14 | |
| *** lpetrut has quit IRC | 20:15 | |
| *** gouthamr_ has joined #openstack-cinder | 20:15 | |
| *** cbader has joined #openstack-cinder | 20:16 | |
| *** alonma has joined #openstack-cinder | 20:16 | |
| *** gouthamr has quit IRC | 20:18 | |
| *** alonma has quit IRC | 20:20 | |
| mtreinish | jgriffith: I just had a random idea/question have you tried to run openstack-health ontop of your subunit2sql db from sos-ci? | 20:22 |
| jgriffith | mtreinish: no, I haven't actually looked at openstack-health | 20:23 |
| jgriffith | reading up on it now | 20:23 |
| *** chris_morrell has joined #openstack-cinder | 20:23 | |
| mtreinish | jgriffith: http://status.openstack.org/openstack-health/#/ | 20:23 |
| *** alonma has joined #openstack-cinder | 20:23 | |
| jgriffith | mtreinish: oh happy!!! | 20:23 |
| jgriffith | Ok, yes... I want that!!! | 20:23 |
| jgriffith | mtreinish: in fact, I want it to be mandatory for ALL the CI's | 20:24 |
| smcginnis | Nice! | 20:24 |
| mtreinish | it might not work 100% for your db because it does expect certain values in the run metadata | 20:24 |
| jgriffith | mtreinish: well, I can certainly *adjust* things | 20:24 |
| mtreinish | like build_name: XXX and project: XXX | 20:24 |
| jgriffith | mtreinish: ok, cool... I'll def take a look at adding this in | 20:25 |
| mtreinish | cool | 20:25 |
| jgriffith | mtreinish: that reminds me, I've still never pushed the round of commits for the latest stuff I added in it for subunit etc | 20:25 |
| jgriffith | mtreinish: I'm curious... | 20:26 |
| openstackgerrit | Merged openstack/os-brick: Actually log the command used in _run_iscsiadm https://review.openstack.org/264310 | 20:26 |
| jgriffith | mtreinish: how is pass rate being calculated (far too lazy to look) :) | 20:26 |
| *** chris_morrell has quit IRC | 20:26 | |
| jgriffith | mtreinish: I ask because for example genuine patch induced failures | 20:26 |
| mtreinish | jgriffith: so this is only from gate (thats all we collect in the infra subunit2sql db) | 20:26 |
| jgriffith | I'm assuming these are part of verify not gate? Or am I wrong | 20:26 |
| mtreinish | and only tempest runs | 20:26 |
| jgriffith | ok | 20:26 |
| jgriffith | cool | 20:27 |
| mtreinish | it also misses failures before tempest starts, because we need a subunit stream to write into the db | 20:27 |
| mtreinish | but I'm working on that aspect with: https://review.openstack.org/#/q/status:open+project:openstack-dev/devstack+branch:master+topic:devstack-subunit | 20:27 |
| mtreinish | and then a corresponding thing for d-g when that's working | 20:27 |
| jgriffith | mtreinish: so one thing about that, is it would be cool if some day we had a way to convey what failed... | 20:28 |
| *** alonma has quit IRC | 20:28 | |
| jgriffith | ie if it was a random test failure, stack.sh or something introduced by the patch | 20:28 |
| mtreinish | and collecting more than tempest is just a matter of config: https://github.com/openstack-infra/system-config/blob/master/modules/openstack_project/files/logstash/jenkins-log-client.yaml#L14-L25 | 20:28 |
| jgriffith | I know that's a pretty pie in the sky request :) | 20:28 |
| jgriffith | mtreinish: ok, doesn't look like any of those should be difficult to NOT have a translation over to sos | 20:29 |
| jgriffith | mtreinish: or.. well... you know what I mean :) | 20:29 |
| mtreinish | heh, I have an idea on how to do that that leveraging elastic recheck and doing an on demand query to logstash | 20:29 |
| jgriffith | mtreinish: :) | 20:29 |
| mtreinish | all the data to do that is in the db, but it pushes my js scripts too far | 20:29 |
| mtreinish | s/scripts/lack of skill | 20:30 |
| jgriffith | mtreinish: :) | 20:30 |
| jgriffith | I was doing some stuff to just leverage out a keyword list | 20:30 |
| jgriffith | but it's pretty basic | 20:30 |
| jgriffith | I just comb my ansible output to see if I ever get the success response from stack.sh | 20:31 |
| jgriffith | the random failure thing... well, that's a whole different story:) | 20:31 |
| mtreinish | I have patches up to start adding a list of recent runs w/ links to the logs: https://review.openstack.org/264383 | 20:31 |
| jgriffith | mtreinish: I started playing with building a list of known "troulbesome" tests but I don't like that | 20:31 |
| jgriffith | mtreinish: hmmm | 20:32 |
| * jgriffith has never wanted to learn js... but | 20:32 | |
| mtreinish | jgriffith: heh, that's simple it's always test volume boot pattern at fault :) | 20:32 |
| *** timcl has quit IRC | 20:32 | |
| jgriffith | mtreinish: well... that's the funny thing!! | 20:32 |
| jgriffith | :) | 20:32 |
| jgriffith | mtreinish: it is in my case... every time | 20:32 |
| jgriffith | That's the only failure I have in my data for the last 3 months | 20:32 |
| *** salv-orlando has quit IRC | 20:32 | |
| mtreinish | haha, nice | 20:32 |
| jgriffith | mtreinish: what's worse is it keeps moving | 20:33 |
| jgriffith | mtreinish: the signature in the logs that is | 20:33 |
| *** salv-orlando has joined #openstack-cinder | 20:33 | |
| jgriffith | from networking, to db timeouts, and most recently just login failure | 20:33 |
| *** alonma has joined #openstack-cinder | 20:33 | |
| mtreinish | ah the generic failed to ssh error | 20:33 |
| jgriffith | anyway... some day I'm going to figure out how to make that sucker stable | 20:33 |
| jgriffith | yes! I was thinking of adding an ssh retry loop... didn't see if that was already in Tempest | 20:34 |
| mtreinish | sdague and I were talking about some improvements to the logging that are really needed there to make figuring out why it cant connect easier | 20:34 |
| mtreinish | but that ssh code is a ratsnest and really needs a simplification/refactor | 20:34 |
| jgriffith | reason being is that in my ci for example I kept hitting the pitfall of thinking "ACTIVE" was actually ready | 20:34 |
| mtreinish | jgriffith: it does a retry loop with an exponential backoff | 20:34 |
| jgriffith | damn | 20:34 |
| jgriffith | mtreinish: I kinda figured folks were smarter than that | 20:35 |
| jgriffith | mtreinish: but I still had "hope" :) | 20:35 |
| mtreinish | jgriffith: here's an example: http://logs.openstack.org/57/257557/7/check/gate-tempest-dsvm-neutron-full/b35e1bb/logs/tempest.txt.gz?level=WARNING#_2016-01-08_19_50_17_227 | 20:35 |
| jgriffith | mtreinish: oh wow... and that's a pretty long period of time | 20:37 |
| *** alonma has quit IRC | 20:38 | |
| mtreinish | jgriffith: https://github.com/openstack/tempest-lib/blob/master/tempest_lib/common/ssh.py#L53-L94 | 20:38 |
| *** chris_morrell has joined #openstack-cinder | 20:38 | |
| *** manous has quit IRC | 20:38 | |
| mtreinish | I guess its just a linear backoff, I thought it was exponential | 20:39 |
| *** chris_morrell has quit IRC | 20:41 | |
| openstackgerrit | Merged openstack/os-brick: Use assertTrue/False instead of assertEqual(T/F) https://review.openstack.org/264600 | 20:46 |
| *** alonma has joined #openstack-cinder | 20:47 | |
| *** chris_morrell has joined #openstack-cinder | 20:50 | |
| *** chris_morrell has quit IRC | 20:52 | |
| *** alonma has quit IRC | 20:52 | |
| *** alonma has joined #openstack-cinder | 20:55 | |
| *** chris_morrell has joined #openstack-cinder | 20:56 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Remove access_mode 'rw' setting in drivers https://review.openstack.org/265443 | 20:57 |
| *** chris_morrell has quit IRC | 20:58 | |
| *** garthb_ has joined #openstack-cinder | 20:58 | |
| *** garthb__ has quit IRC | 20:59 | |
| *** alonma has quit IRC | 20:59 | |
| *** chris_morrell has joined #openstack-cinder | 21:00 | |
| *** haomaiwang has quit IRC | 21:01 | |
| *** haomaiwang has joined #openstack-cinder | 21:01 | |
| *** chris_morrell has quit IRC | 21:02 | |
| *** alonma has joined #openstack-cinder | 21:05 | |
| *** eharney has quit IRC | 21:06 | |
| *** harlowja_at_home has joined #openstack-cinder | 21:07 | |
| *** cdelatte has quit IRC | 21:09 | |
| *** alonma has quit IRC | 21:09 | |
| patrickeast | hemna: dang, so i'm starting to have the same offline fc hba problems you guys did... had to restart a node last night :( | 21:11 |
| patrickeast | hemna: did you guys ever get a fix for it other than rebooting the host? | 21:12 |
| *** akerr has quit IRC | 21:12 | |
| *** merooney has quit IRC | 21:12 | |
| *** raildo is now known as raildo-afk | 21:17 | |
| *** edmondsw has quit IRC | 21:17 | |
| *** chris_morrell has joined #openstack-cinder | 21:19 | |
| *** chris_morrell has quit IRC | 21:21 | |
| kmartin | patrickeast, yeah, pretty sure we checked in the fc script... https://github.com/openstack/third-party-ci-tools/commit/f4e427216e8b7dd0166fbb4ba6dc533b7f11d24f | 21:24 |
| kmartin | patrickeast, it seems to have fixed it for us. | 21:25 |
| patrickeast | kmartin: oh, switching to use reattach fixed it? | 21:25 |
| patrickeast | kmartin: i thought that script just caught it earlier | 21:25 |
| patrickeast | lemmie give that a try and see if it works better | 21:26 |
| *** crose has quit IRC | 21:26 | |
| *** alonma has joined #openstack-cinder | 21:26 | |
| kmartin | with the combination of https://github.com/openstack/third-party-ci-tools/blob/master/provisioning_scripts/fibre_channel/invoke_fc_passthrough.sh | 21:26 |
| patrickeast | kmartin: gotcha, easy fix for me then if I just need to update to the newest passthrough script, the part i wasn't sure on was if the 'hopefully' part here had been confirmed https://github.com/openstack/third-party-ci-tools/blob/master/provisioning_scripts/fibre_channel/invoke_fc_passthrough.sh#L182 | 21:28 |
| *** lpetrut has joined #openstack-cinder | 21:29 | |
| kmartin | patrickeast, it works for us :) | 21:29 |
| *** rhagarty has joined #openstack-cinder | 21:29 | |
| patrickeast | sweet | 21:29 |
| kmartin | "hopefully" it will work for you. | 21:30 |
| *** alonma has quit IRC | 21:31 | |
| *** rhagarty_ has quit IRC | 21:31 | |
| *** rushiagr is now known as rushiagr_away | 21:32 | |
| *** rhagarty_ has joined #openstack-cinder | 21:37 | |
| *** alonma has joined #openstack-cinder | 21:38 | |
| *** ociuhandu has joined #openstack-cinder | 21:38 | |
| hemna | patrickeast, I had put a patch in that was supposed to fix that, or at least make that a LOT more stable. | 21:38 |
| hemna | it's been stable for us since | 21:38 |
| *** rhagarty has quit IRC | 21:39 | |
| *** cknight has joined #openstack-cinder | 21:39 | |
| *** eharney has joined #openstack-cinder | 21:39 | |
| *** diogogmt has quit IRC | 21:40 | |
| *** diogogmt has joined #openstack-cinder | 21:41 | |
| *** akerr has joined #openstack-cinder | 21:42 | |
| *** alonma has quit IRC | 21:43 | |
| *** akerr_ has joined #openstack-cinder | 21:44 | |
| *** garthb_ has quit IRC | 21:44 | |
| *** garthb_ has joined #openstack-cinder | 21:44 | |
| *** martyturner has quit IRC | 21:46 | |
| *** akerr has quit IRC | 21:47 | |
| *** alonma has joined #openstack-cinder | 21:48 | |
| *** merooney has joined #openstack-cinder | 21:51 | |
| *** alonma has quit IRC | 21:52 | |
| openstackgerrit | Merged openstack/cinder: Updated from global requirements https://review.openstack.org/264414 | 21:53 |
| *** akerr_ has quit IRC | 21:55 | |
| *** alonma has joined #openstack-cinder | 21:56 | |
| *** esker has joined #openstack-cinder | 21:59 | |
| *** esker has quit IRC | 21:59 | |
| *** haomaiwang has quit IRC | 22:01 | |
| *** alonma has quit IRC | 22:01 | |
| *** haomaiwang has joined #openstack-cinder | 22:01 | |
| *** esker has joined #openstack-cinder | 22:03 | |
| *** mriedem has quit IRC | 22:07 | |
| *** alonma has joined #openstack-cinder | 22:09 | |
| *** salv-orl_ has joined #openstack-cinder | 22:09 | |
| *** salv-orlando has quit IRC | 22:12 | |
| *** salv-orl_ has quit IRC | 22:12 | |
| *** salv-orlando has joined #openstack-cinder | 22:12 | |
| *** alonma has quit IRC | 22:13 | |
| *** esker has quit IRC | 22:17 | |
| *** fernnest has quit IRC | 22:17 | |
| *** jgregor has quit IRC | 22:18 | |
| *** rhagarty has joined #openstack-cinder | 22:18 | |
| *** alonma has joined #openstack-cinder | 22:18 | |
| *** jgregor has joined #openstack-cinder | 22:19 | |
| *** jgregor has quit IRC | 22:19 | |
| *** jungleboyj has quit IRC | 22:21 | |
| *** lprice has quit IRC | 22:21 | |
| *** rhagarty_ has quit IRC | 22:21 | |
| *** esker has joined #openstack-cinder | 22:22 | |
| *** alonma has quit IRC | 22:23 | |
| *** dustins has quit IRC | 22:24 | |
| *** chris_morrell has joined #openstack-cinder | 22:24 | |
| *** rhagarty has quit IRC | 22:24 | |
| *** ndipanov has quit IRC | 22:24 | |
| *** rhagarty has joined #openstack-cinder | 22:24 | |
| *** alonma has joined #openstack-cinder | 22:25 | |
| *** esker has quit IRC | 22:27 | |
| *** rhagarty_ has joined #openstack-cinder | 22:27 | |
| *** JoseMello has quit IRC | 22:28 | |
| *** rhagarty has quit IRC | 22:29 | |
| *** alonma has quit IRC | 22:29 | |
| *** alonma has joined #openstack-cinder | 22:31 | |
| *** alonma has quit IRC | 22:35 | |
| *** erlon has quit IRC | 22:36 | |
| *** alonma has joined #openstack-cinder | 22:40 | |
| *** lpetrut has quit IRC | 22:41 | |
| *** alonma has quit IRC | 22:44 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Remove access_mode 'rw' setting in drivers https://review.openstack.org/265443 | 22:44 |
| *** chris_morrell has quit IRC | 22:46 | |
| *** rhagarty has joined #openstack-cinder | 22:47 | |
| *** rhagarty_ has quit IRC | 22:49 | |
| *** rhagarty has quit IRC | 22:51 | |
| *** fernnest_ has quit IRC | 22:51 | |
| *** mragupat_ has quit IRC | 22:55 | |
| jidar | is there any recommendation to put '/usr/bin/rescan-scsi-bus.sh -r' in crontab or anything if you're doing FC based block storage? | 22:59 |
| *** merooney has quit IRC | 22:59 | |
| *** haomaiwang has quit IRC | 23:01 | |
| *** haomaiwang has joined #openstack-cinder | 23:01 | |
| *** rhagarty has joined #openstack-cinder | 23:05 | |
| Swanson | jidar, I don't believe so. | 23:06 |
| jidar | I figured as much, just making sure. Seems like I've had to do that at least once to get things working again | 23:07 |
| *** rhagarty_ has joined #openstack-cinder | 23:07 | |
| *** rhagarty has quit IRC | 23:10 | |
| *** rlrossit_ has quit IRC | 23:12 | |
| *** merooney has joined #openstack-cinder | 23:15 | |
| *** vgridnev has quit IRC | 23:22 | |
| openstackgerrit | Anthony Lee proposed openstack/os-brick: Adding a tool for querying volume paths https://review.openstack.org/204191 | 23:22 |
| *** jwcroppe has joined #openstack-cinder | 23:25 | |
| openstackgerrit | Anthony Lee proposed openstack/os-brick: Adding a tool for querying volume paths https://review.openstack.org/204191 | 23:26 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Add volume fail-back capabilities https://review.openstack.org/265399 | 23:27 |
| bardia | anyone know how i can add our driver to this page: https://www.openstack.org/marketplace/drivers/? | 23:34 |
| *** alonma has joined #openstack-cinder | 23:40 | |
| kmartin | bardia, working 3rd party CI and updating https://github.com/openstack/driverlog/blob/master/etc/default_data.json | 23:42 |
| bardia | thanks | 23:42 |
| *** alonma has quit IRC | 23:45 | |
| *** alonma has joined #openstack-cinder | 23:49 | |
| *** annasort has quit IRC | 23:53 | |
| *** alonma has quit IRC | 23:54 | |
| *** alonma has joined #openstack-cinder | 23:58 | |
| *** hemna is now known as hemnafk | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!