| *** ducttape_ has joined #openstack-cinder | 00:00 | |
| *** david-lyle has joined #openstack-cinder | 00:00 | |
| *** lixiaoy1 has joined #openstack-cinder | 00:03 | |
| *** david-lyle_ has joined #openstack-cinder | 00:03 | |
| *** bardia_ has joined #openstack-cinder | 00:04 | |
| *** bardia has quit IRC | 00:04 | |
| *** david-lyle has quit IRC | 00:05 | |
| *** yangyapeng has quit IRC | 00:05 | |
| *** david-lyle_ has quit IRC | 00:09 | |
| *** wilson1 has joined #openstack-cinder | 00:11 | |
| *** wilson_liu has quit IRC | 00:15 | |
| *** bigdogstl has joined #openstack-cinder | 00:15 | |
| *** lixiaoy1 has quit IRC | 00:16 | |
| *** sdake has joined #openstack-cinder | 00:16 | |
| *** ducttape_ has quit IRC | 00:18 | |
| *** bigdogstl has quit IRC | 00:21 | |
| *** bigdogstl has joined #openstack-cinder | 00:21 | |
| *** Suyi has quit IRC | 00:21 | |
| openstackgerrit | Rodrigo Freire proposed openstack/os-brick: Multipath: fix race when detaching devices https://review.openstack.org/331375 | 00:26 |
|---|---|---|
| *** ducttape_ has joined #openstack-cinder | 00:29 | |
| *** alonma has joined #openstack-cinder | 00:34 | |
| *** Julien-zte has joined #openstack-cinder | 00:34 | |
| *** bigdogstl has quit IRC | 00:37 | |
| *** alonma has quit IRC | 00:38 | |
| *** bigdogstl has joined #openstack-cinder | 00:38 | |
| *** laughterwym has joined #openstack-cinder | 00:40 | |
| *** laughterwym has joined #openstack-cinder | 00:40 | |
| *** bigdogstl has quit IRC | 00:41 | |
| *** Suyi has joined #openstack-cinder | 00:42 | |
| *** bigdogstl has joined #openstack-cinder | 00:43 | |
| *** bigdogst_ has joined #openstack-cinder | 00:47 | |
| *** bigdogstl has quit IRC | 00:48 | |
| *** gouthamr has joined #openstack-cinder | 00:52 | |
| *** bigdogst_ has quit IRC | 00:53 | |
| *** cheneydc has joined #openstack-cinder | 00:56 | |
| *** cheneydc has quit IRC | 00:57 | |
| *** Apoorva has quit IRC | 00:59 | |
| *** bigdogstl has joined #openstack-cinder | 01:01 | |
| *** bigdogstl has quit IRC | 01:06 | |
| *** sdake_ has joined #openstack-cinder | 01:06 | |
| *** sdake has quit IRC | 01:10 | |
| *** jamielennox is now known as jamielennox|away | 01:11 | |
| *** alonma has joined #openstack-cinder | 01:15 | |
| *** salv-orl_ has joined #openstack-cinder | 01:15 | |
| *** jamielennox|away is now known as jamielennox | 01:16 | |
| *** salv-orlando has quit IRC | 01:18 | |
| *** sdake_ has quit IRC | 01:19 | |
| *** alonma has quit IRC | 01:20 | |
| *** yangyapeng has joined #openstack-cinder | 01:21 | |
| *** sdake has joined #openstack-cinder | 01:22 | |
| *** salv-orl_ has quit IRC | 01:26 | |
| *** ducttape_ has quit IRC | 01:28 | |
| *** ducttape_ has joined #openstack-cinder | 01:28 | |
| *** Lee1092 has joined #openstack-cinder | 01:29 | |
| *** yangyape_ has joined #openstack-cinder | 01:32 | |
| *** yangyapeng has quit IRC | 01:35 | |
| *** alonma has joined #openstack-cinder | 01:36 | |
| *** raunak has quit IRC | 01:37 | |
| *** coolsvap has quit IRC | 01:39 | |
| *** alonma has quit IRC | 01:40 | |
| *** andymaier has joined #openstack-cinder | 01:41 | |
| *** markvoelker has joined #openstack-cinder | 01:44 | |
| *** markvoelker_ has joined #openstack-cinder | 01:46 | |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Add filters support to get_pools https://review.openstack.org/321492 | 01:47 |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Revmove extra 'u' from cli output https://review.openstack.org/347804 | 01:48 |
| *** yangyape_ has quit IRC | 01:49 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:50 | |
| *** lixiaoy1 has quit IRC | 01:50 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:50 | |
| *** markvoelker has quit IRC | 01:50 | |
| *** EinstCra_ has joined #openstack-cinder | 02:01 | |
| *** mtanin___ has quit IRC | 02:07 | |
| *** bardia_ has quit IRC | 02:07 | |
| *** zhangguoqing has joined #openstack-cinder | 02:15 | |
| *** EinstCra_ has quit IRC | 02:17 | |
| *** alonma has joined #openstack-cinder | 02:18 | |
| openstackgerrit | Merged openstack/os-brick: RBD: Fix typo in rados timeout assignment https://review.openstack.org/358806 | 02:20 |
| *** tommylikehu has joined #openstack-cinder | 02:21 | |
| *** alonma has quit IRC | 02:22 | |
| *** EinstCra_ has joined #openstack-cinder | 02:22 | |
| *** raunak has joined #openstack-cinder | 02:28 | |
| *** yangyapeng has joined #openstack-cinder | 02:36 | |
| *** yangyape_ has joined #openstack-cinder | 02:37 | |
| *** EinstCra_ has quit IRC | 02:39 | |
| *** yangyapeng has quit IRC | 02:40 | |
| *** yangyape_ has quit IRC | 02:47 | |
| *** jamielennox is now known as jamielennox|away | 02:49 | |
| *** yangyape_ has joined #openstack-cinder | 02:50 | |
| *** yangyap__ has joined #openstack-cinder | 02:56 | |
| *** gouthamr_ has joined #openstack-cinder | 02:57 | |
| *** gouthamr has quit IRC | 02:58 | |
| *** yangyape_ has quit IRC | 02:58 | |
| *** alonma has joined #openstack-cinder | 02:59 | |
| *** yangyapeng has joined #openstack-cinder | 03:01 | |
| *** gouthamr_ is now known as gouthamr | 03:01 | |
| *** andymaier has quit IRC | 03:03 | |
| *** alonma has quit IRC | 03:04 | |
| *** yangyap__ has quit IRC | 03:05 | |
| *** jamielennox|away is now known as jamielennox | 03:06 | |
| *** lixiaoy1 has quit IRC | 03:06 | |
| *** lixiaoy1 has joined #openstack-cinder | 03:06 | |
| *** yangyapeng has quit IRC | 03:17 | |
| *** ebalduf has joined #openstack-cinder | 03:20 | |
| *** yangyapeng has joined #openstack-cinder | 03:22 | |
| *** EinstCrazy has joined #openstack-cinder | 03:24 | |
| *** Suyi has quit IRC | 03:25 | |
| *** salv-orlando has joined #openstack-cinder | 03:30 | |
| *** bigdogstl has joined #openstack-cinder | 03:31 | |
| *** yangyape_ has joined #openstack-cinder | 03:34 | |
| *** yangyapeng has quit IRC | 03:36 | |
| *** EinstCrazy has quit IRC | 03:38 | |
| *** salv-orlando has quit IRC | 03:38 | |
| *** asselin_ has joined #openstack-cinder | 03:38 | |
| *** ducttape_ has quit IRC | 03:39 | |
| *** bigdogstl has quit IRC | 03:41 | |
| *** zul has quit IRC | 03:41 | |
| *** asselin has quit IRC | 03:42 | |
| *** gouthamr has quit IRC | 03:44 | |
| *** bigdogstl has joined #openstack-cinder | 03:44 | |
| *** zul has joined #openstack-cinder | 03:46 | |
| *** abhitechie has joined #openstack-cinder | 03:47 | |
| *** yangyape_ has quit IRC | 03:50 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 04:01 |
| *** Jaison has joined #openstack-cinder | 04:15 | |
| *** markvoelker has joined #openstack-cinder | 04:21 | |
| *** markvoelker_ has quit IRC | 04:22 | |
| *** Suyi has joined #openstack-cinder | 04:23 | |
| *** Jaison has quit IRC | 04:24 | |
| *** jraju has joined #openstack-cinder | 04:26 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 04:28 |
| *** markvoelker has quit IRC | 04:28 | |
| *** EinstCrazy has joined #openstack-cinder | 04:30 | |
| *** laughter_ has joined #openstack-cinder | 04:36 | |
| *** lixiaoy1 has quit IRC | 04:36 | |
| *** yangyape_ has joined #openstack-cinder | 04:37 | |
| *** salv-orlando has joined #openstack-cinder | 04:37 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/python-cinderclient: Showing the metadata readonly value as a separate field https://review.openstack.org/332480 | 04:38 |
| *** laughterwym has quit IRC | 04:39 | |
| *** ducttape_ has joined #openstack-cinder | 04:40 | |
| *** ducttape_ has quit IRC | 04:44 | |
| *** yangyape_ has quit IRC | 04:45 | |
| *** EinstCrazy has quit IRC | 04:47 | |
| *** lixiaoy1 has joined #openstack-cinder | 04:48 | |
| *** salv-orlando has quit IRC | 04:49 | |
| *** jamielennox is now known as jamielennox|away | 04:49 | |
| openstackgerrit | venkata krishna proposed openstack/cinder: Add replication failback in Kaminario K2 drivers https://review.openstack.org/356475 | 04:50 |
| *** yangyape_ has joined #openstack-cinder | 04:50 | |
| *** enriquetaso has quit IRC | 04:51 | |
| *** bigdogstl has quit IRC | 04:53 | |
| *** bigdogstl has joined #openstack-cinder | 04:53 | |
| *** yangyape_ has quit IRC | 04:55 | |
| *** bigdogstl has quit IRC | 04:56 | |
| *** bigdogstl has joined #openstack-cinder | 04:56 | |
| *** salv-orlando has joined #openstack-cinder | 04:58 | |
| *** bigdogstl has quit IRC | 05:01 | |
| *** yangyape_ has joined #openstack-cinder | 05:04 | |
| *** lkuchlan has joined #openstack-cinder | 05:05 | |
| *** laughter_ has quit IRC | 05:06 | |
| *** laughterwym has joined #openstack-cinder | 05:07 | |
| *** yangyape_ has quit IRC | 05:09 | |
| *** ducttape_ has joined #openstack-cinder | 05:10 | |
| *** abhinavtechie has joined #openstack-cinder | 05:12 | |
| *** Suyi has quit IRC | 05:12 | |
| *** bigdogstl has joined #openstack-cinder | 05:13 | |
| *** abhitechie has quit IRC | 05:14 | |
| *** aagrawal has joined #openstack-cinder | 05:14 | |
| *** ducttape_ has quit IRC | 05:15 | |
| *** abhitechie has joined #openstack-cinder | 05:16 | |
| *** bigdogstl has quit IRC | 05:17 | |
| *** abhinavtechie has quit IRC | 05:18 | |
| *** lkuchlan has quit IRC | 05:19 | |
| *** aagrawal has quit IRC | 05:20 | |
| *** yangyape_ has joined #openstack-cinder | 05:20 | |
| *** lkuchlan has joined #openstack-cinder | 05:21 | |
| *** sdake_ has joined #openstack-cinder | 05:21 | |
| *** jamielennox|away is now known as jamielennox | 05:23 | |
| *** sdake has quit IRC | 05:24 | |
| *** alonma has joined #openstack-cinder | 05:25 | |
| *** yangyape_ has quit IRC | 05:28 | |
| *** bkopilov has joined #openstack-cinder | 05:29 | |
| *** markvoelker has joined #openstack-cinder | 05:29 | |
| *** laughterwym has quit IRC | 05:29 | |
| *** alonma has quit IRC | 05:30 | |
| *** laughterwym has joined #openstack-cinder | 05:30 | |
| *** laughter_ has joined #openstack-cinder | 05:31 | |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Fix useless api_version of Manager class https://review.openstack.org/309283 | 05:32 |
| *** laughter_ has quit IRC | 05:32 | |
| openstackgerrit | gengchc2 proposed openstack/cinder: Using oslo_config's min attribute https://review.openstack.org/358960 | 05:32 |
| *** yangyape_ has joined #openstack-cinder | 05:33 | |
| *** laughterwym has quit IRC | 05:34 | |
| *** raunak has quit IRC | 05:35 | |
| *** raunak has joined #openstack-cinder | 05:36 | |
| *** sandanar has joined #openstack-cinder | 05:37 | |
| *** markvoelker has quit IRC | 05:39 | |
| *** sdake_ has quit IRC | 05:41 | |
| *** raunak has quit IRC | 05:42 | |
| *** nkrinner_afk is now known as nkrinner | 05:48 | |
| *** pbandark has joined #openstack-cinder | 05:49 | |
| *** yangyape_ has quit IRC | 05:50 | |
| *** laughterwym has joined #openstack-cinder | 05:52 | |
| *** laughterwym has quit IRC | 05:52 | |
| *** laughterwym has joined #openstack-cinder | 05:53 | |
| *** mdenny has quit IRC | 05:55 | |
| *** mdenny has joined #openstack-cinder | 05:55 | |
| *** laughterwym has quit IRC | 05:57 | |
| *** yangyape_ has joined #openstack-cinder | 05:59 | |
| *** yangyap__ has joined #openstack-cinder | 06:00 | |
| *** yangyap__ has quit IRC | 06:01 | |
| *** alonma has joined #openstack-cinder | 06:01 | |
| *** yangyap__ has joined #openstack-cinder | 06:02 | |
| *** iceyao has joined #openstack-cinder | 06:02 | |
| *** iceyao has quit IRC | 06:02 | |
| *** arecknag has joined #openstack-cinder | 06:03 | |
| *** yangyapeng has joined #openstack-cinder | 06:03 | |
| *** asselin__ has joined #openstack-cinder | 06:04 | |
| *** yangyape_ has quit IRC | 06:05 | |
| *** gcb_ has joined #openstack-cinder | 06:05 | |
| *** alonma has quit IRC | 06:06 | |
| *** yangyapeng has quit IRC | 06:07 | |
| *** yangyap__ has quit IRC | 06:07 | |
| *** asselin_ has quit IRC | 06:07 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/357623 | 06:08 |
| *** jamielennox is now known as jamielennox|away | 06:11 | |
| *** abhitechie has quit IRC | 06:11 | |
| *** abhinavtechie has joined #openstack-cinder | 06:11 | |
| *** ducttape_ has joined #openstack-cinder | 06:11 | |
| *** pcaruana has joined #openstack-cinder | 06:14 | |
| *** ducttape_ has quit IRC | 06:17 | |
| *** Poornima_K has joined #openstack-cinder | 06:20 | |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Add filters support to get_pools https://review.openstack.org/321492 | 06:22 |
| *** EinstCrazy has joined #openstack-cinder | 06:32 | |
| *** laughterwym has joined #openstack-cinder | 06:34 | |
| *** lpetrut has joined #openstack-cinder | 06:35 | |
| *** laughterwym has quit IRC | 06:36 | |
| *** markvoelker has joined #openstack-cinder | 06:36 | |
| *** yangyape_ has joined #openstack-cinder | 06:38 | |
| *** EinstCrazy has quit IRC | 06:38 | |
| *** laughterwym has joined #openstack-cinder | 06:41 | |
| *** laughterwym has joined #openstack-cinder | 06:42 | |
| *** yangyapeng has joined #openstack-cinder | 06:42 | |
| *** yangyape_ has quit IRC | 06:42 | |
| *** markvoelker has quit IRC | 06:42 | |
| *** Poornima_K has quit IRC | 06:43 | |
| *** EinstCra_ has joined #openstack-cinder | 06:43 | |
| *** yangyape_ has joined #openstack-cinder | 06:44 | |
| *** GB21 has joined #openstack-cinder | 06:47 | |
| *** Poornima_K has joined #openstack-cinder | 06:48 | |
| *** yangyapeng has quit IRC | 06:48 | |
| *** lixiaoy1 has quit IRC | 06:55 | |
| *** lixiaoy1 has joined #openstack-cinder | 06:57 | |
| *** mjura has joined #openstack-cinder | 06:59 | |
| *** EinstCra_ has quit IRC | 07:01 | |
| *** yangyape_ has quit IRC | 07:01 | |
| *** abhitechie has joined #openstack-cinder | 07:02 | |
| *** abhinavtechie has quit IRC | 07:05 | |
| *** yangyapeng has joined #openstack-cinder | 07:05 | |
| *** EinstCrazy has joined #openstack-cinder | 07:05 | |
| *** fmccrthy has quit IRC | 07:06 | |
| *** fmccrthy has joined #openstack-cinder | 07:08 | |
| *** ducttape_ has joined #openstack-cinder | 07:12 | |
| *** salv-orl_ has joined #openstack-cinder | 07:16 | |
| *** ducttape_ has quit IRC | 07:16 | |
| *** tesseract- has joined #openstack-cinder | 07:18 | |
| *** andymaier has joined #openstack-cinder | 07:18 | |
| *** salv-orlando has quit IRC | 07:19 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Add volume name in vCenter to conn info https://review.openstack.org/344069 | 07:20 |
| *** Poornima_K has quit IRC | 07:21 | |
| *** salv-orl_ has quit IRC | 07:21 | |
| openstackgerrit | Cao ShuFeng proposed openstack/python-cinderclient: Remove extra 'u' in "cinder show" https://review.openstack.org/358995 | 07:21 |
| *** qeelee has joined #openstack-cinder | 07:21 | |
| *** EinstCrazy has quit IRC | 07:24 | |
| *** EinstCrazy has joined #openstack-cinder | 07:25 | |
| *** salv-orlando has joined #openstack-cinder | 07:25 | |
| *** lpetrut has quit IRC | 07:26 | |
| *** wxy__ has quit IRC | 07:28 | |
| *** wxy__ has joined #openstack-cinder | 07:29 | |
| *** Poornima_K has joined #openstack-cinder | 07:29 | |
| *** EinstCra_ has joined #openstack-cinder | 07:36 | |
| *** markvoelker has joined #openstack-cinder | 07:38 | |
| *** EinstCrazy has quit IRC | 07:39 | |
| *** lpetrut has joined #openstack-cinder | 07:40 | |
| *** markvoelker has quit IRC | 07:43 | |
| *** e0ne has joined #openstack-cinder | 07:45 | |
| *** GB21 has quit IRC | 07:45 | |
| *** zouyapeng has joined #openstack-cinder | 07:48 | |
| *** abhitechie has quit IRC | 07:53 | |
| *** abhitechie has joined #openstack-cinder | 07:53 | |
| *** ebalduf has quit IRC | 07:54 | |
| *** rcernin has quit IRC | 07:56 | |
| *** zzzeek has quit IRC | 08:00 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Pin RPC server's serializer to min obj version https://review.openstack.org/350534 | 08:00 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Cleanup upgrade-related FIXME's https://review.openstack.org/350533 | 08:00 |
| *** zzzeek has joined #openstack-cinder | 08:01 | |
| *** lpetrut has quit IRC | 08:01 | |
| *** openstackgerrit has quit IRC | 08:03 | |
| *** openstackgerrit has joined #openstack-cinder | 08:05 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| *** ducttape_ has joined #openstack-cinder | 08:13 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Sending ScaleIO volume id in attach and detach volume https://review.openstack.org/355231 | 08:16 |
| *** ducttape_ has quit IRC | 08:17 | |
| *** pgadiya has joined #openstack-cinder | 08:22 | |
| *** coolsvap has joined #openstack-cinder | 08:25 | |
| *** alonma has joined #openstack-cinder | 08:26 | |
| *** qeelee has quit IRC | 08:30 | |
| *** alonma has quit IRC | 08:31 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Changes stats report for ScaleIO driver https://review.openstack.org/356120 | 08:36 |
| *** markvoelker has joined #openstack-cinder | 08:39 | |
| *** lpetrut has joined #openstack-cinder | 08:41 | |
| *** laughterwym has quit IRC | 08:43 | |
| *** salv-orl_ has joined #openstack-cinder | 08:44 | |
| *** markvoelker has quit IRC | 08:44 | |
| *** salv-orlando has quit IRC | 08:45 | |
| *** laughterwym has joined #openstack-cinder | 08:45 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: ScaleIO over subscription support https://review.openstack.org/356547 | 08:47 |
| *** salv-orl_ has quit IRC | 08:49 | |
| *** salv-orlando has joined #openstack-cinder | 08:50 | |
| *** lpetrut has quit IRC | 08:50 | |
| *** laughterwym has quit IRC | 08:51 | |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: Unify ploop format names in Openstack projects https://review.openstack.org/357679 | 08:54 |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: vzstorage: fix create/delete snapshots https://review.openstack.org/276465 | 08:54 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Merge two fake drivers into the one FakeLoggingVolumeDriver https://review.openstack.org/348428 | 08:57 |
| openstackgerrit | xianming.mao proposed openstack/cinder: Use assertEqual() instead of assertDictEqual() https://review.openstack.org/359053 | 08:59 |
| lixiaoy1 | dulek: may I have a question about your patch https://review.openstack.org/#/c/350534/ ? | 09:08 |
| dulek | lixiaoy1: Sure. | 09:08 |
| lixiaoy1 | dulek: if a old object is returned, but in new services, what happens if new attribute is used? | 09:09 |
| *** kaisers_ has joined #openstack-cinder | 09:10 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX Driver - Live Migration for VMAX3 https://review.openstack.org/330778 | 09:10 |
| dulek | lixiaoy1: Newer services should be able to handle missing data. | 09:10 |
| dulek | lixiaoy1: Because they can always get older version of an object from older services. | 09:10 |
| lixiaoy1 | dulek: yes. | 09:11 |
| dulek | lixiaoy1: Similar thing happened when we were adding cascade to volume delete RPC calls - when running in compatibility mode we've just raised an exception on API level. | 09:11 |
| dulek | lixiaoy1: To make sure a missing parameter that was accepted won't be ignored further when calls will get backported. | 09:12 |
| lixiaoy1 | dulek: but seems the problem can't be prohibited effectively. Dev may add a field, and doesn't update rpcapi interface. But as we transfer such object in rpcapi, this may lead problems | 09:14 |
| dulek | lixiaoy1: Worst thing that can happen is that some kind of user's request will get ignored. Consider that: | 09:15 |
| lixiaoy1 | dulek: would like to confirm: If I have an object with version 1.0, and then add a field in version 1.1. When I got its instance wiht version 1.0, I can't access the new field, yes? | 09:15 |
| dulek | lixiaoy1: It's not enteirely true. | 09:16 |
| dulek | lixiaoy1: We have a new service trying to send 1.1 object to other service (it don't know which service version will get it). | 09:17 |
| dulek | lixiaoy1: So the object gets backported to 1.0 and a field is removed. | 09:17 |
| dulek | lixiaoy1: Now it happens that object is received by newer service that can understand 1.1. | 09:17 |
| dulek | lixiaoy1: Serializer will actually unserialize it to 1.1 with that new field unset. | 09:18 |
| dulek | lixiaoy1: That's it. | 09:18 |
| dulek | lixiaoy1: https://github.com/openstack/cinder/blob/625cab15b05339fbdb1d71250ba62ab30c1294b5/cinder/objects/base.py#L476-L481 | 09:19 |
| *** aarefiev_ is now known as aarefiev | 09:19 | |
| dulek | lixiaoy1: Good thing is that we should be able to find any problems with multinode grenade job. It's in experimental queue now and is testing older c-vol with newer c-api and c-sch. | 09:20 |
| lixiaoy1 | dulek: but here self.version_cap is min obj version | 09:21 |
| *** salv-orlando has quit IRC | 09:21 | |
| *** salv-orlando has joined #openstack-cinder | 09:21 | |
| *** lpetrut has joined #openstack-cinder | 09:21 | |
| dulek | lixiaoy1: Ah, you're right - this is serialization, deserialization is just left as o.vo base version. | 09:22 |
| lixiaoy1 | dulek: oh, this is just for serialize_entity. we should have a deserialization function to change 1.0 to 1.1? | 09:24 |
| dulek | lixiaoy1: We were considering that at the mid cycle in Raleigh - to for example refetch the object from the DB. | 09:24 |
| dulek | lixiaoy1: We've decided to leave that to programmer to be handled explicitly. | 09:25 |
| lixiaoy1 | lixiaoy1: got it. I remember this point. | 09:27 |
| lixiaoy1 | dulek: with this patch, I think problems will be exploded if there are. | 09:28 |
| *** nkrinner has quit IRC | 09:28 | |
| dulek | lixiaoy1: Hm… Before this patch we've had backporting of parameters sent to RPC calls/casts. | 09:29 |
| dulek | lixiaoy1: With it we're adding backporting to objects returned from RPC calls/casts. | 09:29 |
| lixiaoy1 | dulek: yes | 09:29 |
| dulek | lixiaoy1: I'm not sure why this is fundamentally different. Can you elaborate? | 09:30 |
| lixiaoy1 | dulek: not much. think again, this patch just backport the objects returned, and seems that we have no such case from mitaka | 09:32 |
| *** pgadiya_ has joined #openstack-cinder | 09:33 | |
| lixiaoy1 | dulek: thank you for helping me clear | 09:33 |
| dulek | lixiaoy1: Actually we've had such cases. | 09:33 |
| lixiaoy1 | dulek: which case? | 09:33 |
| dulek | lixiaoy1: We're returning volume or snapshot object in backup_device dictionary. | 09:33 |
| *** pgadiya has quit IRC | 09:34 | |
| dulek | lixiaoy1: That's why I'm introducing the patch - I've tested that it may happen that older c-bak will receive a Volume object from newer c-vol. If that happens - it will not understand the object. | 09:34 |
| lixiaoy1 | dulek: yes | 09:34 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Reduce the runtime of drivers/ibm/test_storwize_svc https://review.openstack.org/359070 | 09:35 |
| lixiaoy1 | dulek: thank you. I am going to leave office. Have a nice day | 09:35 |
| dulek | lixiaoy1: Bye, have a nice evening! :) | 09:36 |
| *** EinstCra_ is now known as EinstCrazy | 09:36 | |
| *** lixiaoy1 has quit IRC | 09:36 | |
| *** lixiaoy1 has joined #openstack-cinder | 09:36 | |
| openstackgerrit | Nguyen Hung Phuong proposed openstack/cinder: Update code to use Newton, Mitaka as the code name https://review.openstack.org/359071 | 09:37 |
| *** wxy__ has quit IRC | 09:38 | |
| *** gcb_ has quit IRC | 09:38 | |
| *** wxy__ has joined #openstack-cinder | 09:39 | |
| *** gcb_ has joined #openstack-cinder | 09:39 | |
| *** markvoelker has joined #openstack-cinder | 09:40 | |
| *** nkrinner has joined #openstack-cinder | 09:42 | |
| *** markvoelker has quit IRC | 09:44 | |
| *** laughterwym has joined #openstack-cinder | 09:57 | |
| *** EinstCrazy has quit IRC | 09:57 | |
| *** EinstCrazy has joined #openstack-cinder | 09:57 | |
| *** EinstCra_ has joined #openstack-cinder | 09:58 | |
| *** EinstCra_ is now known as EinstCrazy_ | 09:59 | |
| *** EinstCrazy has quit IRC | 10:02 | |
| *** qeelee has joined #openstack-cinder | 10:05 | |
| *** lpetrut1 has joined #openstack-cinder | 10:10 | |
| *** ducttape_ has joined #openstack-cinder | 10:10 | |
| *** lpetrut has quit IRC | 10:11 | |
| *** lpetrut1 is now known as lpetrut | 10:11 | |
| *** alonma has joined #openstack-cinder | 10:14 | |
| *** ducttape_ has quit IRC | 10:16 | |
| *** Julien-zte has quit IRC | 10:18 | |
| *** alonma has quit IRC | 10:19 | |
| openstackgerrit | Cedric Zhuang proposed openstack/cinder: Performance issue for VNX attache LUN https://review.openstack.org/358720 | 10:20 |
| *** lprice has quit IRC | 10:22 | |
| *** lprice has joined #openstack-cinder | 10:24 | |
| *** yangyapeng has quit IRC | 10:28 | |
| *** EinstCrazy_ has quit IRC | 10:28 | |
| *** ntpttr has quit IRC | 10:30 | |
| openstackgerrit | Szymon Borkowski proposed openstack/cinder: Convert backup_device to OVO https://review.openstack.org/329332 | 10:31 |
| *** ntpttr has joined #openstack-cinder | 10:35 | |
| *** qeelee has quit IRC | 10:43 | |
| *** coolsvap is now known as coolsvap_ | 10:53 | |
| *** alonma has joined #openstack-cinder | 10:56 | |
| *** alonma has quit IRC | 11:00 | |
| *** sheel has joined #openstack-cinder | 11:02 | |
| *** alyson_ has joined #openstack-cinder | 11:06 | |
| *** markvoelker has joined #openstack-cinder | 11:09 | |
| *** ducttape_ has joined #openstack-cinder | 11:13 | |
| *** mjura_ has joined #openstack-cinder | 11:14 | |
| *** ducttape_ has quit IRC | 11:17 | |
| *** bkopilov has quit IRC | 11:19 | |
| *** markvoelker has quit IRC | 11:20 | |
| *** JoseMello has joined #openstack-cinder | 11:21 | |
| *** thiagolib has quit IRC | 11:23 | |
| *** thiagolib has joined #openstack-cinder | 11:24 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: ScaleIO over subscription support https://review.openstack.org/356547 | 11:30 |
| swamireddy | DuncanT: Hi - Can you please review https://review.openstack.org/#/c/335067/ ( Updated the patch as per your review comments).. | 11:31 |
| *** yuriy_n17 has joined #openstack-cinder | 11:33 | |
| openstackgerrit | xiexs proposed openstack/cinder: Convert InvalidVolumeMetadataSize to webob.exc.* https://review.openstack.org/356213 | 11:34 |
| *** xiexs has joined #openstack-cinder | 11:37 | |
| openstackgerrit | JiWei proposed openstack/cinder: Implement nfs manage/unmanage of a volume https://review.openstack.org/349146 | 11:38 |
| *** netapp has quit IRC | 11:41 | |
| *** rcernin has joined #openstack-cinder | 11:43 | |
| yuriy_n17 | geguileo: Hi. Very sorry for disturbing. Please review https://review.openstack.org/#/c/356624/ whenever you have free time. | 11:44 |
| *** netapp has joined #openstack-cinder | 11:48 | |
| *** abhinavtechie has joined #openstack-cinder | 11:50 | |
| geguileo | yuriy_n17: Will do | 11:50 |
| *** abhitechie has quit IRC | 11:51 | |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick: Fix cmd execution stderr, stdout unicode errors https://review.openstack.org/341085 | 11:52 |
| *** abhinavtechie has quit IRC | 11:52 | |
| yuriy_n17 | geguileo: Thanks. | 11:53 |
| *** abhinavtechie has joined #openstack-cinder | 11:53 | |
| *** lkuchlan has quit IRC | 11:53 | |
| openstackgerrit | xiexs proposed openstack/cinder: [Trivial] Refactor the case test_storwize_vdisk_copy_ops https://review.openstack.org/356194 | 12:00 |
| *** yangyapeng has joined #openstack-cinder | 12:01 | |
| openstackgerrit | xiexs proposed openstack/cinder: [Trivial] Refactor the using of dict.get() in the test assertion https://review.openstack.org/355323 | 12:03 |
| *** pgadiya_ is now known as pgadiya | 12:03 | |
| *** abhinavtechie has quit IRC | 12:04 | |
| *** Poornima_K has quit IRC | 12:04 | |
| *** xyang1 has joined #openstack-cinder | 12:05 | |
| openstackgerrit | xiexs proposed openstack/cinder: Convert InvalidVolumeMetadataSize to webob.exc.* https://review.openstack.org/356213 | 12:08 |
| *** tpsilva has joined #openstack-cinder | 12:08 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Changes stats report for ScaleIO driver https://review.openstack.org/356120 | 12:09 |
| *** ducttape_ has joined #openstack-cinder | 12:10 | |
| *** kaisers_ has quit IRC | 12:11 | |
| *** andymaier has quit IRC | 12:13 | |
| *** Lee1092 has quit IRC | 12:14 | |
| dmsimard | lixiaoy1: I added a comment in your review https://review.openstack.org/#/c/358670/ | 12:16 |
| dmsimard | lixiaoy1: it looks like barbican encrypted volumes are really broken now :( | 12:16 |
| *** Poornima_K has joined #openstack-cinder | 12:18 | |
| *** pauloewerton has joined #openstack-cinder | 12:18 | |
| openstackgerrit | Tuan proposed openstack/cinder: Remove self.__dict__ for formatting strings https://review.openstack.org/358491 | 12:21 |
| *** ducttape_ has quit IRC | 12:21 | |
| *** wxy__ has quit IRC | 12:22 | |
| *** wxy__ has joined #openstack-cinder | 12:23 | |
| *** rcernin has quit IRC | 12:23 | |
| *** mjura_ has quit IRC | 12:23 | |
| *** andymaier has joined #openstack-cinder | 12:25 | |
| *** gouthamr has joined #openstack-cinder | 12:26 | |
| *** rcernin has joined #openstack-cinder | 12:28 | |
| *** Poornima_K has quit IRC | 12:34 | |
| *** coolsvap_ is now known as coolsvap | 12:36 | |
| *** edmondsw has joined #openstack-cinder | 12:40 | |
| e0ne | DuncanT: hi. are you around? | 12:41 |
| e0ne | DuncanT: I tried to implement noauth support for cinderclient | 12:42 |
| *** takedakn has joined #openstack-cinder | 12:42 | |
| e0ne | DuncanT: it's very depended on keystone | 12:43 |
| e0ne | DuncanT: according to help, it can work w/o keystone by provider cinder url and other options, but at the first glance, this feature is broken | 12:43 |
| e0ne | DuncanT: I'll try to take a look on it once again later this week | 12:44 |
| *** Poornima_K has joined #openstack-cinder | 12:48 | |
| DuncanT | e0ne: It was very broken last I looked for sure | 12:50 |
| *** lkuchlan has joined #openstack-cinder | 12:51 | |
| DuncanT | e0ne: If I deleted enough code then I could make it work, but it was a bit here, a bit there, so nothing easy to clean up | 12:51 |
| *** lkuchlan has quit IRC | 12:52 | |
| DuncanT | e0ne: It wouldn't surprise me if it has gotten worse since then. Many things are now written to assume keystone | 12:52 |
| *** lkuchlan has joined #openstack-cinder | 12:52 | |
| *** lkuchlan is now known as lkuchlan_wfh | 12:52 | |
| dulek | Anyone knows where we're keeping v3 api-ref? | 12:54 |
| dulek | I can only see v1 and v2 folders in cinder/api-ref… | 12:55 |
| *** chlong has quit IRC | 12:56 | |
| *** xyang has joined #openstack-cinder | 12:57 | |
| *** dustins has joined #openstack-cinder | 13:00 | |
| *** coolsvap is now known as coolsvap_ | 13:00 | |
| *** rlrossit has joined #openstack-cinder | 13:00 | |
| scottda | dulek: There was a patch up to add v3 api-ref...let me look... | 13:02 |
| *** rcernin has quit IRC | 13:03 | |
| swamireddy | DuncanT: Hi - Can you please review https://review.openstack.org/#/c/335067/ ( Updated the patch as per your review comments).. | 13:03 |
| DuncanT | swamireddy: Probably tomorrow before I get time, sorry | 13:04 |
| swamireddy | DuncanT: Thank you.. NOP. | 13:06 |
| swamireddy | dulek: Thanks for the review (https://review.openstack.org/#/c/335067/ ) with +2. | 13:07 |
| dulek | scottda: Oh, cool. No need to block 335067 because of that however. | 13:07 |
| scottda | dulek: Maybe I'm confusing patches, i'm not finding it. We might not have it yet, but I thought there was a review out there... | 13:07 |
| dulek | scottda: Thanks anyway! :) | 13:07 |
| scottda | dulek: Filed a bug to track the need for v3 api-ref: https://bugs.launchpad.net/cinder/+bug/1616072 | 13:11 |
| openstack | Launchpad bug 1616072 in Cinder "Cinder lacks v3 api-ref" [Undecided,New] | 13:11 |
| *** porrua has joined #openstack-cinder | 13:11 | |
| *** andymaier has quit IRC | 13:12 | |
| *** chlong has joined #openstack-cinder | 13:13 | |
| *** qeelee has joined #openstack-cinder | 13:13 | |
| *** qeelee has quit IRC | 13:15 | |
| *** raunak has joined #openstack-cinder | 13:18 | |
| *** akerr has joined #openstack-cinder | 13:21 | |
| *** julim has joined #openstack-cinder | 13:21 | |
| *** ducttape_ has joined #openstack-cinder | 13:21 | |
| *** andymaier has joined #openstack-cinder | 13:22 | |
| *** arecknag has quit IRC | 13:22 | |
| nikeshm | hi if any one gets time, please give one more +2 on https://review.openstack.org/#/c/356475, its already have one +2, thanks | 13:22 |
| *** pgadiya has quit IRC | 13:23 | |
| *** andymaier has quit IRC | 13:23 | |
| *** bigdogstl has joined #openstack-cinder | 13:24 | |
| *** andymaier_ has joined #openstack-cinder | 13:24 | |
| *** david-lyle has joined #openstack-cinder | 13:24 | |
| *** timcl has joined #openstack-cinder | 13:24 | |
| *** takedakn has quit IRC | 13:24 | |
| *** takedakn has joined #openstack-cinder | 13:25 | |
| patrickeast | hemna: so finally got some good test runs, looks like https://review.openstack.org/#/c/231135/ is indeed needed to make FC moar happy | 13:26 |
| patrickeast | hemna: i still had some intermittent mpath devices left behind : / but I'm suspicious of my multipath.conf being wrong | 13:27 |
| *** takedakn has quit IRC | 13:30 | |
| smcginnis | patrickeast: Oh good, I was going to ask if you ever figured that out. | 13:30 |
| *** Julien-zte has joined #openstack-cinder | 13:30 | |
| smcginnis | patrickeast: Didn't think you would be online so early. ;) | 13:30 |
| patrickeast | smcginnis: haha yea, in NY this week | 13:32 |
| *** bkopilov has joined #openstack-cinder | 13:32 | |
| patrickeast | Usually wouldn't be awake, or at least working yet | 13:32 |
| *** jungleboyj has joined #openstack-cinder | 13:33 | |
| smcginnis | patrickeast: Oh, that's right. | 13:33 |
| smcginnis | patrickeast: Are you attending the ops midcycle too? Or heading back after East is over? | 13:33 |
| patrickeast | smcginnis: heading back Thursday morning | 13:35 |
| *** nkrinner is now known as nkrinner_afk | 13:35 | |
| smcginnis | patrickeast: Ah, too bad. I get in late Wednesday. Couldn't do OpenStack East but wanted to go to the Ops event. | 13:36 |
| *** sdake has joined #openstack-cinder | 13:36 | |
| *** sheel has quit IRC | 13:36 | |
| patrickeast | smcginnis: ah gotcha, how late Wednesday? :D | 13:37 |
| *** sdake_ has joined #openstack-cinder | 13:37 | |
| smcginnis | patrickeast: 11:20 PM (what was I thinking!) | 13:37 |
| *** ducttape_ has quit IRC | 13:38 | |
| dulek | patrickeast: OpenStack East seems like an event with name crafted for you. :) | 13:38 |
| patrickeast | smcginnis: ah perfect, can just head right over to whatever bar we're at | 13:39 |
| patrickeast | dulek: haha exactly! | 13:39 |
| *** raunak has quit IRC | 13:39 | |
| erlon | smcginnis: Hi Sean, can you give a look on this 2 patches when you have a chance? It's being around for a while with no core reviews: | 13:40 |
| erlon | smcginnis: https://review.openstack.org/#/c/333992/ | 13:40 |
| erlon | smcginnis: https://review.openstack.org/#/c/310785/ | 13:40 |
| *** jraju has quit IRC | 13:40 | |
| *** raunak has joined #openstack-cinder | 13:40 | |
| *** sdake has quit IRC | 13:41 | |
| *** raunak has quit IRC | 13:42 | |
| openstackgerrit | Aleksey Ruban proposed openstack/cinder: Nexenta: NexentaStor4 NFS driver cache capacity and use sessions https://review.openstack.org/356142 | 13:43 |
| *** bigdogstl has quit IRC | 13:43 | |
| smcginnis | patrickeast: Hah, maybe. ;) | 13:43 |
| smcginnis | erlon: Got it in my queue now. | 13:43 |
| *** sdague has joined #openstack-cinder | 13:43 | |
| erlon | smcginnis: thanks! | 13:44 |
| *** ducttape_ has joined #openstack-cinder | 13:45 | |
| *** sandanar has quit IRC | 13:45 | |
| *** diablo_rojo_ has joined #openstack-cinder | 13:48 | |
| *** Poornima_K has quit IRC | 13:48 | |
| *** rlrossit_ has joined #openstack-cinder | 13:51 | |
| *** eharney has joined #openstack-cinder | 13:53 | |
| *** rlrossit has quit IRC | 13:55 | |
| *** jgregor has joined #openstack-cinder | 13:55 | |
| *** bigdogstl has joined #openstack-cinder | 13:57 | |
| *** kaisers_ has joined #openstack-cinder | 13:59 | |
| *** bigdogstl has quit IRC | 14:02 | |
| *** pcaruana has quit IRC | 14:02 | |
| *** david-lyle has quit IRC | 14:04 | |
| *** kaisers_ has quit IRC | 14:04 | |
| *** tbarron|gone is now known as tbarron | 14:05 | |
| *** coolsvap_ is now known as coolsvap | 14:05 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Fix racy volume unit test https://review.openstack.org/358807 | 14:09 |
| *** ebalduf has joined #openstack-cinder | 14:12 | |
| *** takedakn has joined #openstack-cinder | 14:12 | |
| *** alyson_ has quit IRC | 14:13 | |
| *** alonma has joined #openstack-cinder | 14:14 | |
| *** cknight has joined #openstack-cinder | 14:15 | |
| *** pcaruana has joined #openstack-cinder | 14:17 | |
| erlon | jungleboyj: ping | 14:18 |
| jungleboyj | erlon: Pong | 14:19 |
| *** alonma has quit IRC | 14:19 | |
| erlon | jungleboyj: Hey Jay | 14:19 |
| erlon | jungleboyj: I see you have updated the NFS patch | 14:19 |
| *** rajinir has joined #openstack-cinder | 14:19 | |
| jungleboyj | erlon: Yes. | 14:19 |
| erlon | jungleboyj: have you found a way to attach a disk with snapshots? | 14:20 |
| jungleboyj | I haven't gotten that far yet. Just got it rebased and fixed failing test cases. | 14:21 |
| jungleboyj | Going to start on the other issues next week hopefully. | 14:21 |
| erlon | jungleboyj: for what I have found, the problem (at least the bigger part of it) is that nova is not considering the snapshot files in the attachment | 14:21 |
| erlon | jungleboyj: so, let me try to update you on what I have discovered so far | 14:22 |
| jungleboyj | Ok. | 14:22 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 14:23 |
| erlon | jungleboyj: there are several permission problems, when libvirt creates snapshot from a file owned by ubuntu:ubuntu, it gives some files to qemu-kvm:kvm, while others goes to root | 14:24 |
| jungleboyj | erlon: Interesting. | 14:24 |
| erlon | jungleboyj: those problems does not happens if you use nfs_secure = defaut | 14:24 |
| erlon | jungleboyj: so, Im using this to continue investigating the other problem | 14:25 |
| jungleboyj | Interesting. I have seen other issues around using the default option for security. I need to investigate those. Wonder if we have something more systemic. | 14:26 |
| *** pbandark has quit IRC | 14:27 | |
| erlon | jungleboyj: the other problem is that, once you create a snapshot chain for a volume, Nova only considers the base file, so, for example, you attach a file A, create online snapshots A.S1, A.S2, A.S3. Great, this works, Nova writes now to A.S3. But, if you detach the volume, and re-attach it, Nova will attach only A, disconsidering A.S1, A.S2 amd A.S3 | 14:28 |
| *** sdague has quit IRC | 14:28 | |
| *** david-lyle has joined #openstack-cinder | 14:28 | |
| erlon | jungleboyj: what do you mean by systemic? | 14:28 |
| erlon | jungleboyj: in this last problem, it also happens if you create the snapshot offline and tries to attach it. | 14:29 |
| erlon | jungleboyj: so, I started to try to attach a volume with a backing chain in libvirt, disconsidering Nova | 14:30 |
| *** sage___ has quit IRC | 14:30 | |
| eharney | something is going very wrong if Nova is not loading the correct file... | 14:31 |
| jungleboyj | erlon: I am wondering if there are other problems coming from how security/permissions are being handled. | 14:31 |
| erlon | jungleboyj: annnd, I'm still here. It can be a libvirt or a QEMU problem, which is what I'm at the moment | 14:31 |
| eharney | because Cinder should be telling it to load the file at the tip of the chain | 14:31 |
| *** timcl has quit IRC | 14:31 | |
| *** sage has joined #openstack-cinder | 14:32 | |
| erlon | eharney: yes, Cinder should send the backing chain to Nova, and nova should attach the top file of the chain | 14:32 |
| eharney | Cinder doesn't send the whole chain, libvirt inspects that when it attaches the files | 14:32 |
| erlon | eharney: exactly, but this information is not even being sent to Nova | 14:32 |
| eharney | erlon: that information doesn't need to be sent to Nova, only the last filename does | 14:32 |
| erlon | eharney: it doesn't, should it? | 14:32 |
| eharney | erlon: why should it? | 14:33 |
| erlon | eharney: hmm, yeah, that should be enought, but as I didnt know how libvirt does the attach I tried to send all chain | 14:33 |
| eharney | erlon: libvirt will follow the qcow2 backing file pointers in the files, and you can see the result in libvirt dumpxml on the domain after it's attached | 14:34 |
| erlon | eharney: I mean libvirt is not inspecting the file chain | 14:34 |
| erlon | eharney: hmmm | 14:34 |
| eharney | erlon: this should all mostly work in some form regardless, but what version of libvirt? | 14:35 |
| erlon | eharney: well, I tryed to do that with libvirt but couldn't | 14:35 |
| *** sdake_ has quit IRC | 14:35 | |
| erlon | eharney: jungleboyj: https://www.redhat.com/archives/libvirt-users/2016-August/msg00048.html | 14:35 |
| erlon | eharney: 1.3.1 | 14:36 |
| *** Lee1092 has joined #openstack-cinder | 14:36 | |
| eharney | erlon: does qemu-img info show the backing file pointer? | 14:36 |
| *** vern has quit IRC | 14:36 | |
| erlon | eharney: yes, have doubled checked that | 14:36 |
| erlon | eharney: all with the correct permissions | 14:36 |
| eharney | erlon: i haven't tried doing this w/ virsh attach-device | 14:36 |
| erlon | eharney: the guys that is replying the thread said he coud attach but, didn't give enough details and I couldn't reproduce | 14:38 |
| eharney | erlon: but the <disk> block in your dumpxml and <backingStore> look correct | 14:38 |
| *** takedakn has quit IRC | 14:38 | |
| eharney | erlon: certainly possible that it's permissions issues somewhere | 14:39 |
| erlon | eharney: mhm, I just got the <disk> snip with the snapshot and tried to attach | 14:39 |
| erlon | eharney: mhm, yep, libvirt might be setting permissions and then asking qemu to read the files | 14:39 |
| erlon | eharney: because looking at libvirt log, I can see the error returned from qemu | 14:40 |
| *** sdake has joined #openstack-cinder | 14:41 | |
| erlon | eharney: I'll try to add the disk direct to qemu, to bypass libvirt | 14:41 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls https://review.openstack.org/252458 | 14:46 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO https://review.openstack.org/253033 | 14:46 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype) https://review.openstack.org/257305 | 14:46 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 14:47 |
| *** lprice1 has joined #openstack-cinder | 14:48 | |
| *** lprice has quit IRC | 14:49 | |
| *** bigdogstl has joined #openstack-cinder | 14:52 | |
| *** Yogi1 has joined #openstack-cinder | 14:53 | |
| *** xinli has joined #openstack-cinder | 14:54 | |
| *** david-lyle has quit IRC | 14:54 | |
| *** timcl has joined #openstack-cinder | 14:55 | |
| *** bwallis has quit IRC | 14:55 | |
| *** raunak has joined #openstack-cinder | 14:55 | |
| *** alonma has joined #openstack-cinder | 14:56 | |
| jungleboyj | erlon: Thank you for working on that. Keep me updated on your progress. If you are working through that I can work on the fact that cloning is currently listed as supported but isn't implemented. | 14:57 |
| *** bigdogstl has quit IRC | 14:57 | |
| *** enriquetaso has joined #openstack-cinder | 14:58 | |
| erlon | jungleboyj: ok I will, hmmm that would be good thanks | 15:00 |
| *** alonma has quit IRC | 15:00 | |
| jungleboyj | erlon: Yeah, that one kind-of caught us by surprise when we found it. | 15:00 |
| scottda | geguileo: I think that snapshot-create is now cluster aware with your latest HA patches. Is that correct? | 15:02 |
| geguileo | scottda: Yes | 15:03 |
| geguileo | scottda: All operations should be cluster aware now | 15:04 |
| *** vern has joined #openstack-cinder | 15:04 | |
| scottda | geguileo: Cool. That is what I'm seeing. Your blog on manual testing still talks about snapshot-create Not being cluster aware, but I'd remembered you'd changed that. Testing confirms that it works. | 15:04 |
| geguileo | scottda: I believe I updated the post... | 15:05 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Reduce the runtime of drivers/ibm/test_storwize_svc https://review.openstack.org/359070 | 15:05 |
| scottda | geguileo: I don't see an update in http://gorka.eguileor.com/manual-validation-of-cinder-aa-patches/ after a browser refresh . Section 2.2 | 15:06 |
| geguileo | scottda: Oh, I must have forgotten to change it there, let me see | 15:06 |
| geguileo | scottda: I updated it in "Current cluster aware operations" | 15:07 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Reduce the runtime of drivers/ibm/test_storwize_svc https://review.openstack.org/359070 | 15:07 |
| geguileo | scottda: Oooops, yes, I should update that one as well XD | 15:07 |
| scottda | geguileo: NP. It's great stuff. And all working as expected. | 15:07 |
| geguileo | scottda: Great to hear it, I'm working on the API stuff now | 15:08 |
| *** salv-orlando has quit IRC | 15:08 | |
| *** salv-orlando has joined #openstack-cinder | 15:08 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Reduce the runtime of drivers/ibm/test_storwize_svc https://review.openstack.org/359070 | 15:10 |
| *** leeantho has joined #openstack-cinder | 15:11 | |
| scottda | geguileo: Is it correct that after a c-vol has been disabled, that the heartbeat is still updated? | 15:12 |
| geguileo | scottda: I believe that is true for master as well | 15:12 |
| geguileo | scottda: They are different things | 15:12 |
| scottda | geguileo: OK, just checking. I need to look at that code.... | 15:13 |
| karthikp_ | eharney: Hi. I wanted to check with you about https://review.openstack.org/#/c/322977/.... Since there was other changes like volumeAttach status ..i thought it was good to have this | 15:13 |
| *** mjura has quit IRC | 15:15 | |
| *** Julien-zte has quit IRC | 15:15 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Add dell_server_os configuration option https://review.openstack.org/358878 | 15:16 |
| *** alonma has joined #openstack-cinder | 15:16 | |
| geguileo | scottda: Disabled services will still receive requests to attach/detach, delete, etc. So it's important that they appear as up ;-) | 15:17 |
| openstackgerrit | Nam Nguyen Hoai proposed openstack/cinder: Clean import in code https://review.openstack.org/358480 | 15:17 |
| scottda | geguileo: Hmmm...Well, this is after I've killed c-vol. It shows as "down" in service-list. | 15:19 |
| geguileo | scottda: If you kill c-vol it should appear as down | 15:19 |
| geguileo | scottda: If you only have 1 node | 15:19 |
| *** andymaier_ has quit IRC | 15:19 | |
| geguileo | scottda: Is it reporting as down when you only stop 1 of the 2 nodes? | 15:19 |
| geguileo | s/nodes/services | 15:20 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add new supported driver checks https://review.openstack.org/355608 | 15:20 |
| *** bwallis has joined #openstack-cinder | 15:21 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Mask out passwords when tracing https://review.openstack.org/354370 | 15:22 |
| *** david-lyle has joined #openstack-cinder | 15:23 | |
| *** xinli has quit IRC | 15:23 | |
| scottda | geguileo: cinder-volume for 1st c-vol is 'down' in 'cinder service-list' after I kill it. And it is 'up' for c-vol2. As expected. It is the 'last heartbeat' field from 'cluster-list' that confused me. But that is the heartbeat for the cluster, not the hosts, so it is correct. | 15:23 |
| *** xinli has joined #openstack-cinder | 15:23 | |
| geguileo | scottda: Oh, ok, yes, that's correct | 15:24 |
| * geguileo I should have mentioned that in the post | 15:25 | |
| *** cdelatte has joined #openstack-cinder | 15:27 | |
| *** david-lyle has quit IRC | 15:33 | |
| *** alonma has quit IRC | 15:36 | |
| *** bwallis has quit IRC | 15:37 | |
| *** sdague has joined #openstack-cinder | 15:38 | |
| *** tesseract- has quit IRC | 15:46 | |
| *** bigdogstl has joined #openstack-cinder | 15:46 | |
| *** alyson_ has joined #openstack-cinder | 15:47 | |
| stevemar | can anyone make heads or tails of this? https://bugs.launchpad.net/python-cinderclient/+bug/1616070 ? | 15:48 |
| openstack | Launchpad bug 1616070 in python-openstackclient "all volume v1 commands broken" [Undecided,New] - Assigned to Rui Chen (kiwik-chenrui) | 15:48 |
| *** cdelatte has quit IRC | 15:48 | |
| *** kaisers_ has joined #openstack-cinder | 15:48 | |
| *** mmedvede has quit IRC | 15:53 | |
| *** kaisers_ has quit IRC | 15:53 | |
| *** bigdogstl has quit IRC | 15:53 | |
| *** sdague has quit IRC | 15:54 | |
| smcginnis | stevemar: Hmm, logger is not getting initialized somehow. | 15:55 |
| *** pcaruana has quit IRC | 15:56 | |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO: fix the returned value of CG actions https://review.openstack.org/346831 | 15:57 |
| smcginnis | stevemar: Passing in self.logger instead of self._logger: http://git.openstack.org/cgit/openstack/python-cinderclient/tree/cinderclient/client.py#n120 | 15:57 |
| smcginnis | Wow, since April. | 16:00 |
| smcginnis | That's a good sign that not many people are still using v1. :) | 16:00 |
| scottda | I don't see the issue in cinderclient. | 16:00 |
| smcginnis | scottda: Passing in v1? | 16:01 |
| smcginnis | I was able to repro (and verify fix) here. | 16:01 |
| scottda | smcginnis: Oh! I had '--debug' and it worked. | 16:01 |
| scottda | smcginnis: But without '--debug' it doesn't | 16:01 |
| smcginnis | scottda: Yeah, works fine with debug. | 16:01 |
| *** david-lyle has joined #openstack-cinder | 16:01 | |
| scottda | got it. Sorry | 16:01 |
| stevemar | scottda: ah | 16:01 |
| smcginnis | scottda: Which is actually weird. How is that set with debug but not otherwise? | 16:01 |
| stevemar | sounds like cinderclient has a legit bug :O | 16:02 |
| smcginnis | stevemar: Yeah, looks legit. | 16:02 |
| scottda | smcginnis: Do you have time to put up a patch, or do you want me to? | 16:02 |
| *** lkuchlan_wfh has quit IRC | 16:02 | |
| smcginnis | scottda: I added that to the bug report. He has it assigned to himself, so I was going to see if they would take care of it. | 16:03 |
| scottda | smcginnis: Forget it, I see it is assigned | 16:03 |
| smcginnis | scottda: ;) | 16:03 |
| smcginnis | scottda: If no activity soon we can just grab it I suppose. | 16:03 |
| smcginnis | But if it's been there since april maybe not a huge rush. | 16:03 |
| scottda | Yeah, like you guys said, the most interesting thing is that noone's found it sooner, meaning not much use of /v1 | 16:04 |
| smcginnis | Must be something in keystoneauth1 that sets logger in Adapter on debug but not otherwise. But should still be self._logger. | 16:04 |
| scottda | DuncanT, aspiers, geguileo, scottda, dulek, ddeja, patrickeast Want to talk about HA? | 16:04 |
| aspiers | o/ | 16:05 |
| geguileo | scottda: Sure, though I don't have news | 16:05 |
| aspiers | I have a question about active/passive cinder-volume, but not sure if it's in scope for this meeting | 16:05 |
| scottda | #startmeeting cinder_aaha | 16:05 |
| openstack | Meeting started Tue Aug 23 16:05:24 2016 UTC and is due to finish in 60 minutes. The chair is scottda. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:05 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:05 |
| openstack | The meeting name has been set to 'cinder_aaha' | 16:05 |
| geguileo | scottda: I always welcome the opportunity to beg for reviews ;-) | 16:05 |
| scottda | geguileo: Yeah, that's a good use of this meeting. Everyone Review!!! and Test !!! | 16:05 |
| scottda | Anyone interested in HA/AA should run through geguileo 's manual tests: http://gorka.eguileor.com/manual-validation-of-cinder-aa-patches/ | 16:06 |
| scottda | They make some of the inner workings very clear. | 16:07 |
| geguileo | There are 2 patches that can be merged in this cycle: https://review.openstack.org/#/c/344224/ https://review.openstack.org/#/c/353068 | 16:07 |
| *** bwallis has joined #openstack-cinder | 16:07 | |
| scottda | There may not be much else to talk about...so maybe aspiers , you should ask about active/passive? | 16:07 |
| aspiers | ok :) | 16:07 |
| aspiers | so an alternative approach to a/p has been proposed | 16:07 |
| aspiers | I guess using a virtual hostname is not unusual | 16:08 |
| aspiers | but this conflicts with cinder-scheduler | 16:08 |
| aspiers | so the suggestion is a kind of hack: | 16:08 |
| aspiers | if a machine needs to run both cinder-scheduler and cinder-volume, use separate cinder.conf files for each | 16:08 |
| aspiers | which only differ with the hostname setting | 16:09 |
| aspiers | does that make sense? | 16:09 |
| geguileo | aspiers: You don't need different cinder.conf files | 16:09 |
| *** pcaruana has joined #openstack-cinder | 16:09 | |
| aspiers | oh? | 16:09 |
| geguileo | aspiers: You just need the cinder-volume to add an additional config file | 16:09 |
| geguileo | aspiers: That just changes the host value | 16:09 |
| aspiers | oh ok, even cleaner | 16:09 |
| aspiers | but the same principle | 16:09 |
| geguileo | aspiers: That's how I do it in my HA A/A tests | 16:10 |
| aspiers | I didn't know it was possible to overlay config files | 16:10 |
| geguileo | aspiers: Yes, as many as you want | 16:10 |
| aspiers | cool, thanks! is that documented somewhere? | 16:10 |
| geguileo | aspiers: They overwrite the previous values | 16:10 |
| aspiers | I could work with beekhof to get it into the ha-guide | 16:10 |
| aspiers | so you specify -c multiple times? or reference the underlying cinder.conf from within the one on top? | 16:11 |
| scottda | aspiers: Just like a normal config file, and just add to the startup line: | 16:14 |
| scottda | user@localhost:$ usr/bin/cinder-volume --config-file /etc/cinder/cinder.conf --config-file /etc/cinder/host1.conf & echo $! >/opt/stack/status/stack/c-vol.pid; fg || echo "c-vol failed to start" | tee "/opt/stack/status/stack/c-vol.failure" | 16:14 |
| scottda | https://www.irccloud.com/pastebin/UVZti2rm/ | 16:15 |
| aspiers | scottda: perfect, thanks a lot! | 16:15 |
| scottda | aspiers: Just that "cluster = mycluster" will be overriden (or added if it didn't exist) | 16:15 |
| aspiers | yup, makes sense | 16:16 |
| scottda | ok, keep talking amongst yourselfs. But meeting seems over... | 16:16 |
| scottda | #endmeeting | 16:16 |
| openstack | Meeting ended Tue Aug 23 16:16:39 2016 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:16 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_aaha/2016/cinder_aaha.2016-08-23-16.05.html | 16:16 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_aaha/2016/cinder_aaha.2016-08-23-16.05.txt | 16:16 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_aaha/2016/cinder_aaha.2016-08-23-16.05.log.html | 16:16 |
| *** rlrossit_ has quit IRC | 16:21 | |
| *** bigdogstl has joined #openstack-cinder | 16:23 | |
| *** rlrossit has joined #openstack-cinder | 16:25 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: WIP: Dell SC: Add exclude_domain_ip option https://review.openstack.org/359328 | 16:27 |
| *** Yogi1 has quit IRC | 16:27 | |
| *** bigdogstl has quit IRC | 16:27 | |
| *** zhangguoqing has quit IRC | 16:29 | |
| *** eharney has quit IRC | 16:35 | |
| *** rmcall has joined #openstack-cinder | 16:35 | |
| *** zul has quit IRC | 16:36 | |
| *** Suyi has joined #openstack-cinder | 16:36 | |
| *** jraju has joined #openstack-cinder | 16:38 | |
| *** fernnest has joined #openstack-cinder | 16:38 | |
| *** jraju has quit IRC | 16:39 | |
| *** laughterwym has quit IRC | 16:43 | |
| nikeshm | smcginnis: hi | 16:47 |
| *** lkuchlan_wfh has joined #openstack-cinder | 16:47 | |
| *** asselin has joined #openstack-cinder | 16:47 | |
| nikeshm | hemna: hi | 16:48 |
| *** zul has joined #openstack-cinder | 16:48 | |
| nikeshm | hi gentle reminder, if u get time, please review https://review.openstack.org/#/c/356475 | 16:51 |
| *** e0ne has quit IRC | 16:52 | |
| nikeshm | already have +2 from xyang, it is replication failback support for kaminario drivers | 16:52 |
| nikeshm | thanks | 16:52 |
| openstackgerrit | Raunak Kumar proposed openstack/cinder: Add support for force backup for Nimble Storage https://review.openstack.org/349086 | 16:55 |
| *** mtanino has joined #openstack-cinder | 16:57 | |
| karthikp_ | smcginnis: Hi | 17:01 |
| karthikp_ | smcginnis: I wanted to know what you feel about this change: https://review.openstack.org/#/c/322977/ | 17:03 |
| *** Apoorva has joined #openstack-cinder | 17:04 | |
| smcginnis | karthikp_: You should probably talk to eharney about it. | 17:04 |
| *** Apoorva has quit IRC | 17:04 | |
| smcginnis | karthikp_: But I believe we've switched some other stuff over to enums, so not sure. | 17:05 |
| *** cdelatte has joined #openstack-cinder | 17:05 | |
| *** Apoorva has joined #openstack-cinder | 17:05 | |
| smcginnis | karthikp_: What we have does work, but enums help make things clear and limited to correct, expected values. So not sure. Guess I don't have as strong of an opinion as Eric does. | 17:05 |
| *** Guest25180 is now known as med_ | 17:06 | |
| *** med_ has joined #openstack-cinder | 17:06 | |
| *** med_ is now known as medberry | 17:06 | |
| *** medberry is now known as med_ | 17:06 | |
| karthikp_ | smcginnis: Sure. Yes we have changed many things to enums! So I am really not sure why not this. I will check with him | 17:06 |
| openstackgerrit | Raunak Kumar proposed openstack/cinder: Add support for force backup for Nimble Storage https://review.openstack.org/349086 | 17:07 |
| smcginnis | karthikp_: Cool. I think it would be good to discuss it with him. Looks fine to me. | 17:07 |
| *** asselin has quit IRC | 17:09 | |
| *** lkuchlan_wfh has quit IRC | 17:09 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Live Volume Autofailover https://review.openstack.org/350782 | 17:10 |
| *** mtanin___ has joined #openstack-cinder | 17:10 | |
| *** mtanino has quit IRC | 17:12 | |
| *** asselin has joined #openstack-cinder | 17:15 | |
| *** bigdogstl has joined #openstack-cinder | 17:15 | |
| karthikp_ | dulek: geguileo: Can I work on the review to create a mapping of release to object & rpc version pins?? mentioned by Gorka in https://review.openstack.org/#/c/350533/1/cinder/objects/service.py@170 | 17:16 |
| geguileo | karthikp_: I have no objections and I'm actually glad if you do :-) | 17:17 |
| *** rmk has quit IRC | 17:18 | |
| karthikp_ | geguileo: | 17:19 |
| karthikp_ | thanks | 17:19 |
| *** cdelatte has quit IRC | 17:19 | |
| *** bigdogstl has quit IRC | 17:20 | |
| *** alonma has joined #openstack-cinder | 17:21 | |
| dmsimard | lixiaoy1: hi, are you around ? | 17:24 |
| dmsimard | lixiaoy1: do you think you could address the comment in https://review.openstack.org/#/c/358670/ ? Other than that I think it looks okay. | 17:24 |
| *** alonma has quit IRC | 17:26 | |
| *** zul has quit IRC | 17:26 | |
| *** xyang has quit IRC | 17:27 | |
| *** xyang has joined #openstack-cinder | 17:28 | |
| *** cdelatte has joined #openstack-cinder | 17:30 | |
| *** cdelatte has quit IRC | 17:31 | |
| *** lpetrut has quit IRC | 17:33 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Use constraints for api-ref environment https://review.openstack.org/359362 | 17:34 |
| *** gcb_ has quit IRC | 17:36 | |
| *** bwallis has quit IRC | 17:36 | |
| *** gcb_ has joined #openstack-cinder | 17:36 | |
| *** kaisers_ has joined #openstack-cinder | 17:37 | |
| *** e0ne has joined #openstack-cinder | 17:39 | |
| *** cdelatte has joined #openstack-cinder | 17:40 | |
| *** eharney has joined #openstack-cinder | 17:40 | |
| *** kaisers_ has quit IRC | 17:42 | |
| *** cdelatte has quit IRC | 17:42 | |
| *** rmcall has quit IRC | 17:44 | |
| *** lkuchlan_wfh has joined #openstack-cinder | 17:45 | |
| *** rmcall has joined #openstack-cinder | 17:49 | |
| *** bigdogstl has joined #openstack-cinder | 17:51 | |
| *** zhugaoxiao has joined #openstack-cinder | 17:53 | |
| *** e0ne has quit IRC | 17:55 | |
| *** zul has joined #openstack-cinder | 17:55 | |
| *** Administrator__ has quit IRC | 17:55 | |
| *** eharney has quit IRC | 17:56 | |
| *** lkuchlan_wfh has quit IRC | 17:56 | |
| *** bigdogstl has quit IRC | 17:58 | |
| *** e0ne has joined #openstack-cinder | 18:00 | |
| *** jamielennox|away is now known as jamielennox | 18:01 | |
| *** cdelatte has joined #openstack-cinder | 18:07 | |
| *** eharney has joined #openstack-cinder | 18:08 | |
| *** bardia has joined #openstack-cinder | 18:10 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Disabling online volume copy in HPE 3PAR drivers https://review.openstack.org/348060 | 18:10 |
| *** alonma has joined #openstack-cinder | 18:23 | |
| *** raunak has quit IRC | 18:23 | |
| *** alonma has quit IRC | 18:30 | |
| *** lpetrut has joined #openstack-cinder | 18:33 | |
| *** cdelatte has quit IRC | 18:37 | |
| *** xyang has quit IRC | 18:38 | |
| *** zul has quit IRC | 18:46 | |
| openstackgerrit | Sofia Enriquez proposed openstack/cinder: RBD: Error when manage an already-managed volume https://review.openstack.org/354289 | 18:50 |
| *** xyang has joined #openstack-cinder | 18:51 | |
| *** timcl has quit IRC | 18:53 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: LeftHand add entry point tracing https://review.openstack.org/339128 | 18:53 |
| *** raunak has joined #openstack-cinder | 18:55 | |
| *** raunak has quit IRC | 18:56 | |
| *** markvoelker has joined #openstack-cinder | 18:58 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Fixes the length of resource field in database for quota tables https://review.openstack.org/355214 | 18:58 |
| *** markvoelker has quit IRC | 19:01 | |
| *** zul has joined #openstack-cinder | 19:01 | |
| *** xyang has quit IRC | 19:02 | |
| *** xyang has joined #openstack-cinder | 19:03 | |
| dmsimard | Anyone familiar with encrypted volumes ? Seeing issues after recent merges. | 19:06 |
| dmsimard | Particularly curious around the docs (and the tempest test) claiming to use aes-512 while that doesn't seem to really exist | 19:07 |
| dmsimard | http://docs.openstack.org/mitaka/config-reference/block-storage/volume-encryption.html | 19:07 |
| dmsimard | https://github.com/openstack/tempest/blob/master/tempest/scenario/test_encrypted_cinder_volumes.py#L56 | 19:07 |
| *** alee has joined #openstack-cinder | 19:08 | |
| dmsimard | FWIW it's mentioned in the review that introduced the issue https://review.openstack.org/#/c/251503 | 19:10 |
| dmsimard | "Another fix will be needed to address the use case of aes-xts with a key size of 512 -- key managers may not be able to create 512 bit AES keys." | 19:11 |
| *** timcl has joined #openstack-cinder | 19:11 | |
| *** cdelatte has joined #openstack-cinder | 19:13 | |
| eharney | dmsimard: what platform is the test running on? | 19:20 |
| dmsimard | eharney: CentOS + RDO trunk installed/configured through puppet-openstack-integration | 19:20 |
| *** pcaruana has quit IRC | 19:20 | |
| *** andymaier_ has joined #openstack-cinder | 19:21 | |
| dmsimard | I just added a comment in https://bugs.launchpad.net/cinder/+bug/1514546 | 19:21 |
| openstack | Launchpad bug 1514546 in Cinder "Cinder volume encryption uses default parameters for keys" [Undecided,Fix released] - Assigned to Lisa Li (lisali) | 19:21 |
| *** cdelatte has quit IRC | 19:22 | |
| *** kaisers_ has joined #openstack-cinder | 19:26 | |
| * eharney waiting for gerrit to work | 19:26 | |
| *** salv-orlando has quit IRC | 19:28 | |
| *** salv-orlando has joined #openstack-cinder | 19:28 | |
| eharney | dmsimard: kind of a mess there, but i wonder if we can work around it for now by setting key size to 256 in tempest | 19:29 |
| dmsimard | eharney: I was thinking about that as well, at least temporarily | 19:29 |
| eharney | dmsimard: it looks like 512 "should" work, but there are a few issues in the way | 19:30 |
| dmsimard | eharney: even so, the key length issue is actually hidden behind https://review.openstack.org/#/c/358670/ | 19:30 |
| *** kfarr has joined #openstack-cinder | 19:30 | |
| eharney | dmsimard: right | 19:30 |
| kfarr | dmsimard ping | 19:30 |
| dmsimard | kfarr: ohai o/ | 19:30 |
| dmsimard | kfarr: can we chat cinder+barbican? :D | 19:30 |
| *** kaisers_ has quit IRC | 19:30 | |
| eharney | just the person we need :) | 19:30 |
| kfarr | dmsimard, yes! | 19:30 |
| kfarr | sorry for all the trouble D: | 19:31 |
| dmsimard | kfarr: it's okay, I think it highlights a gap in CI | 19:31 |
| alee | kfarr, we've been missing you! | 19:31 |
| dmsimard | RDO tends to catch a lot of those gaps :P | 19:31 |
| kfarr | dmsimard, so it looks like lisali has the fix up for the incorrect parameter, (which I saw you said you integrated already) | 19:32 |
| kfarr | dmsimard though to address the issue about aes 512... | 19:32 |
| alee | kfarr, so whats up with cinder asking for aes 512? | 19:32 |
| *** xinli has quit IRC | 19:32 | |
| kfarr | you're correct that aes 512 keys don't really exist | 19:32 |
| dmsimard | kfarr: yes, I think lisa will need to fix those tests first -- I bumped into the key length issue when manually packaging lisa's fix | 19:32 |
| kfarr | though like you pointed out, the xts changes things | 19:32 |
| *** salv-orlando has quit IRC | 19:32 | |
| alee | kfarr, in this case, barbican simple_crypto plugin rejects the request | 19:34 |
| kfarr | if we could change the tests to use xts-256, that would be great | 19:34 |
| alee | kfarr, because oit fails the supports()) method | 19:34 |
| dmsimard | kfarr: yeah I think changing the key length in the tempest test is the /easy/ fix | 19:34 |
| kfarr | alee, hmm I thought simple_crypto wasn't picky about bit lengths | 19:34 |
| alee | kfarr, you mean aes-256? | 19:34 |
| kfarr | aes-xts-256 | 19:35 |
| alee | kfarr, up to 256 :) | 19:35 |
| dmsimard | kfarr: but that fixes just tempest, the docs might warrant a note about this | 19:35 |
| kfarr | dmsimard, true | 19:35 |
| *** rlrossit_ has joined #openstack-cinder | 19:35 | |
| kfarr | I can attempt to explain the xts 512 bits vs 256 bits, though reaperhulk does a much better job | 19:37 |
| smcginnis | hemna: Can you take a look at this if/when you have a chance? https://review.openstack.org/#/c/231135/ | 19:37 |
| kfarr | alee, to support aes-xts-512, we'd essentially need to create two 256 bit keys and concatenate them | 19:38 |
| *** rlrossit has quit IRC | 19:38 | |
| dmsimard | kfarr, alee: https://review.openstack.org/359391 | 19:38 |
| dmsimard | eharney: ^ | 19:38 |
| eharney | dmsimard: great | 19:39 |
| kfarr | thanks dmsimard | 19:39 |
| dmsimard | kfarr: now we need those two reviews merged and we're in business :) | 19:40 |
| alee | kfarr, the simple plugin checks to see if we have an algorithm in the supported list and key size in that list too | 19:41 |
| alee | kfarr, see https://github.com/openstack/barbican/blob/master/barbican/plugin/crypto/crypto.py#L47 | 19:41 |
| alee | kfarr, the checking isn't sophisticated -- we dont match algorithm to key length | 19:42 |
| alee | kfarr, but it is the simple plugin | 19:43 |
| alee | kfarr, if we need to support 512 bit keys for some reason , then we should update barbican simple key plugin too | 19:43 |
| kfarr | alee, most key managers probably won't support 512 bit keys though, KMIP doesn't | 19:43 |
| alee | kfarr, right - I was curious about the original choice of size in the docs/test .. | 19:44 |
| *** raunak has joined #openstack-cinder | 19:45 | |
| kfarr | those docs were written when we were still using ConfKeyManager, and it didn't really care about key size | 19:45 |
| *** andymaier_ has quit IRC | 19:46 | |
| alee | kfarr, nice round number :) | 19:47 |
| alee | kfarr, I think we need a bug to change the docs .. | 19:47 |
| kfarr | alee, yes, that would probably be good | 19:47 |
| *** Apoorva has quit IRC | 19:48 | |
| *** andymaier_ has joined #openstack-cinder | 19:49 | |
| kfarr | alee, brb | 19:50 |
| *** andymaier_ has quit IRC | 19:50 | |
| *** andymaier_ has joined #openstack-cinder | 19:51 | |
| *** Apoorva has joined #openstack-cinder | 19:52 | |
| *** kfarr has quit IRC | 19:55 | |
| *** markvoelker has joined #openstack-cinder | 19:57 | |
| *** kfarr has joined #openstack-cinder | 19:57 | |
| dmsimard | kfarr, alee: thanks for your help | 19:58 |
| dmsimard | I wish we had the capacity (both human and hardware) to do third party CI on everything, we'd catch this sort of thing earlier in the process :) | 19:59 |
| alee | dmsimard, fun fun :) | 19:59 |
| *** lpetrut has quit IRC | 20:03 | |
| *** porrua has quit IRC | 20:04 | |
| *** dmsimard has left #openstack-cinder | 20:09 | |
| *** akerr has quit IRC | 20:09 | |
| *** e0ne has quit IRC | 20:12 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Fix policy check for group types https://review.openstack.org/359403 | 20:13 |
| *** edmondsw has quit IRC | 20:13 | |
| *** haplo37__ has joined #openstack-cinder | 20:14 | |
| *** alkhodos- has quit IRC | 20:14 | |
| *** xinli has joined #openstack-cinder | 20:18 | |
| *** e0ne has joined #openstack-cinder | 20:25 | |
| *** rhefner has quit IRC | 20:31 | |
| *** aswadr_ has quit IRC | 20:31 | |
| *** coolsvap has quit IRC | 20:31 | |
| *** rhefner has joined #openstack-cinder | 20:32 | |
| *** rhefner has quit IRC | 20:32 | |
| *** rhefner has joined #openstack-cinder | 20:33 | |
| *** e0ne has quit IRC | 20:37 | |
| *** lprice1 has quit IRC | 20:39 | |
| xyang | hemna: ping | 20:42 |
| *** rhefner has quit IRC | 20:43 | |
| scottda | Here's a spec update with a couple of +2's...if anyone else want to have a look... | 20:46 |
| scottda | https://review.openstack.org/#/c/327283/ | 20:46 |
| *** rhefner has joined #openstack-cinder | 20:48 | |
| smcginnis | scottda: Wish it was easy to diff between the two files in gerrit. | 20:49 |
| smcginnis | scottda: What's the main difference? | 20:49 |
| scottda | I agree...maybe you can diff em in the repo... | 20:49 |
| smcginnis | scottda: Yeah, I can pull it down. | 20:49 |
| smcginnis | scottda: Or just review the new file and make sure it makes sense. ;) | 20:50 |
| scottda | smcginnis: I reviewed it a while back, but I cannot recall the issues. I just was going through the H/A bp and trying to update status. | 20:50 |
| *** cknight has quit IRC | 20:50 | |
| *** alonma has joined #openstack-cinder | 20:51 | |
| *** markvoelker has quit IRC | 20:52 | |
| *** alonma has quit IRC | 20:56 | |
| *** timcl has quit IRC | 20:56 | |
| openstackgerrit | Merged openstack/cinder-specs: Update Job Distribution for A/A Specs https://review.openstack.org/327283 | 20:57 |
| *** gouthamr has quit IRC | 20:58 | |
| *** chris_morrell has quit IRC | 21:09 | |
| *** jgregor has quit IRC | 21:11 | |
| xyang | smcginnis: ScaleIO CI is broken now without this fix: https://review.openstack.org/#/c/355231/ | 21:12 |
| *** julim has quit IRC | 21:12 | |
| xyang | smcginnis: henna found out this issue | 21:13 |
| xyang | sorry, hemna | 21:14 |
| xyang | hemna: it is not me. it was auto corrected:) | 21:14 |
| smcginnis | :) | 21:15 |
| xyang | :) | 21:15 |
| *** kaisers_ has joined #openstack-cinder | 21:15 | |
| *** david-lyle has quit IRC | 21:15 | |
| smcginnis | xyang: Done | 21:16 |
| xyang | thanks! | 21:16 |
| *** david-lyle has joined #openstack-cinder | 21:18 | |
| *** enriquetaso has quit IRC | 21:19 | |
| *** kaisers_ has quit IRC | 21:20 | |
| *** pauloewerton has quit IRC | 21:22 | |
| *** ducttape_ has quit IRC | 21:23 | |
| *** bardia_ has joined #openstack-cinder | 21:24 | |
| *** bardia has quit IRC | 21:24 | |
| *** eharney has quit IRC | 21:27 | |
| openstackgerrit | Merged openstack/python-cinderclient: Add group types and group specs https://review.openstack.org/320157 | 21:30 |
| *** rlrossit_ has quit IRC | 21:30 | |
| *** salv-orlando has joined #openstack-cinder | 21:32 | |
| *** gouthamr has joined #openstack-cinder | 21:34 | |
| *** Lee1092 has quit IRC | 21:44 | |
| *** sdake has quit IRC | 21:45 | |
| *** sdake has joined #openstack-cinder | 21:45 | |
| *** sdake has quit IRC | 21:45 | |
| *** sdake has joined #openstack-cinder | 21:46 | |
| *** ducttape_ has joined #openstack-cinder | 21:46 | |
| openstackgerrit | Raunak Kumar proposed openstack/cinder: Add Fibre Channel support for Nimble Storage https://review.openstack.org/359448 | 21:56 |
| *** andymaier_ has quit IRC | 22:15 | |
| *** rmcall has quit IRC | 22:15 | |
| *** alee has quit IRC | 22:16 | |
| openstackgerrit | Kaitlin Farr proposed openstack/cinder: Fix incorrect parameter in create_key https://review.openstack.org/358670 | 22:18 |
| *** rmk has joined #openstack-cinder | 22:21 | |
| *** lpetrut has joined #openstack-cinder | 22:24 | |
| *** sdague has joined #openstack-cinder | 22:26 | |
| *** lpetrut has quit IRC | 22:28 | |
| ntpttr___ | hey everyone, I'm trying to back up to swift using devstack and running into this error, anyone know what's going wrong? http://paste.openstack.org/show/562764/ | 22:29 |
| *** sdague has quit IRC | 22:31 | |
| DuncanT | ntpttr___: Is it a clean devstack install or particularly modified? Devstack usually gets rootwrap filters right- it would show up on the gate pretty quick if it didn't I'd have expected | 22:32 |
| ntpttr___ | DuncanT: it's customized in that it's multinode - I have swift running on the controller and c-bak on another node | 22:33 |
| *** xyang1 has quit IRC | 22:36 | |
| *** xinli has quit IRC | 22:36 | |
| ntpttr___ | here are my local.confs, it's possible something is wrong in there http://paste.openstack.org/show/562765/ http://paste.openstack.org/show/562766/ | 22:36 |
| *** sdake has quit IRC | 22:44 | |
| *** sdake has joined #openstack-cinder | 22:44 | |
| *** jungleboyj has quit IRC | 22:53 | |
| *** alonma has joined #openstack-cinder | 22:56 | |
| karthikp_ | ntpttr___: did you try harcoding the service_host in local.conf | 22:57 |
| karthikp_ | ntpttr___: maybe it cant get the IP address there | 22:57 |
| *** leeantho has quit IRC | 22:58 | |
| ntpttr___ | karthikp_: the service_host is getting populated correctly | 23:00 |
| ntpttr___ | those are just ansible variables to get it dynamically, I made sure they were getting the right ones | 23:00 |
| *** alonma has quit IRC | 23:01 | |
| *** alee has joined #openstack-cinder | 23:02 | |
| *** chlong has quit IRC | 23:03 | |
| *** alyson_ has quit IRC | 23:03 | |
| *** xyang has quit IRC | 23:14 | |
| *** kfarr has quit IRC | 23:19 | |
| *** erhudy has quit IRC | 23:22 | |
| *** Julien-zte has joined #openstack-cinder | 23:22 | |
| *** bigdogstl has joined #openstack-cinder | 23:25 | |
| *** bigdogstl has quit IRC | 23:29 | |
| *** salv-orlando has quit IRC | 23:36 | |
| *** bigdogstl has joined #openstack-cinder | 23:45 | |
| *** Julien-zte has quit IRC | 23:47 | |
| *** bigdogstl has quit IRC | 23:50 | |
| *** ducttape_ has quit IRC | 23:57 | |
| *** ducttape_ has joined #openstack-cinder | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!