| *** yangyapeng has quit IRC | 00:01 | |
| *** garthb has quit IRC | 00:10 | |
| *** markvoelker has joined #openstack-cinder | 00:14 | |
| *** diablo_rojo has quit IRC | 00:20 | |
| *** lprice has joined #openstack-cinder | 00:20 | |
| *** suyash has quit IRC | 00:23 | |
| *** karthikp has joined #openstack-cinder | 00:23 | |
| *** mtanino has quit IRC | 00:28 | |
| *** leeantho_ has quit IRC | 00:34 | |
| *** Julien-zte has joined #openstack-cinder | 00:36 | |
| *** laughterwym has joined #openstack-cinder | 00:39 | |
| *** laughterwym has quit IRC | 00:41 | |
| *** laughterwym has joined #openstack-cinder | 00:41 | |
| *** laughterwym has quit IRC | 00:42 | |
| *** laughterwym has joined #openstack-cinder | 00:42 | |
| *** Julien-zte has quit IRC | 00:46 | |
| *** raunak has quit IRC | 00:49 | |
| *** Julien-zte has joined #openstack-cinder | 00:51 | |
| *** chris1 has joined #openstack-cinder | 00:58 | |
| *** chris1 has quit IRC | 00:59 | |
| *** laughterwym has quit IRC | 01:00 | |
| *** cheneydc has joined #openstack-cinder | 01:03 | |
| *** laughterwym has joined #openstack-cinder | 01:03 | |
| *** iceyao has joined #openstack-cinder | 01:06 | |
| *** yangyapeng has joined #openstack-cinder | 01:11 | |
| *** iceyao_ has joined #openstack-cinder | 01:12 | |
| *** yangyapeng has quit IRC | 01:12 | |
| *** sdake has quit IRC | 01:13 | |
| *** itzdilip has joined #openstack-cinder | 01:14 | |
| *** iceyao has quit IRC | 01:15 | |
| *** vkmc is now known as Guest9874 | 01:20 | |
| *** Guest9874 is now known as cmkv | 01:22 | |
| *** cmkv is now known as violenciarivas | 01:22 | |
| *** karthikp has quit IRC | 01:23 | |
| *** violenciarivas is now known as vkmc | 01:23 | |
| *** EinstCrazy has joined #openstack-cinder | 01:24 | |
| *** EinstCrazy has quit IRC | 01:28 | |
| *** yangyapeng has joined #openstack-cinder | 01:28 | |
| *** EinstCrazy has joined #openstack-cinder | 01:29 | |
| *** EinstCrazy has quit IRC | 01:29 | |
| *** EinstCrazy has joined #openstack-cinder | 01:30 | |
| *** EinstCra_ has joined #openstack-cinder | 01:32 | |
| *** EinstCrazy has quit IRC | 01:33 | |
| *** EinstCrazy has joined #openstack-cinder | 01:33 | |
| *** wxy has joined #openstack-cinder | 01:33 | |
| *** iceyao_ has quit IRC | 01:35 | |
| *** iceyao_ has joined #openstack-cinder | 01:37 | |
| *** EinstCra_ has quit IRC | 01:37 | |
| *** iceyao__ has joined #openstack-cinder | 01:38 | |
| *** yangxi has joined #openstack-cinder | 01:38 | |
| *** iceyao_ has quit IRC | 01:38 | |
| *** iceyao___ has joined #openstack-cinder | 01:38 | |
| *** EinstCrazy has quit IRC | 01:39 | |
| *** EinstCrazy has joined #openstack-cinder | 01:41 | |
| *** iceyao__ has quit IRC | 01:43 | |
| *** ozialien10 has joined #openstack-cinder | 01:46 | |
| *** ozialien10 has quit IRC | 01:46 | |
| *** ozialien10 has joined #openstack-cinder | 01:47 | |
| *** raunak has joined #openstack-cinder | 01:50 | |
| *** gouthamr has joined #openstack-cinder | 01:55 | |
| *** apoorvad has quit IRC | 01:56 | |
| *** Poornima has joined #openstack-cinder | 01:57 | |
| *** EinstCrazy has quit IRC | 01:59 | |
| *** EinstCrazy has joined #openstack-cinder | 02:00 | |
| *** yangxi has quit IRC | 02:01 | |
| *** Lee1092 has joined #openstack-cinder | 02:01 | |
| *** yangxi has joined #openstack-cinder | 02:04 | |
| *** EinstCra_ has joined #openstack-cinder | 02:04 | |
| *** itzdilip has quit IRC | 02:07 | |
| *** akapil has joined #openstack-cinder | 02:08 | |
| *** EinstCrazy has quit IRC | 02:08 | |
| *** cdelatte has quit IRC | 02:15 | |
| *** bardia has quit IRC | 02:16 | |
| *** EinstCra_ has quit IRC | 02:22 | |
| *** ebalduf has quit IRC | 02:22 | |
| *** zhongjun_ has joined #openstack-cinder | 02:22 | |
| *** hemna_ has joined #openstack-cinder | 02:23 | |
| *** yangxi has quit IRC | 02:27 | |
| *** raunak has quit IRC | 02:28 | |
| *** karthikp has joined #openstack-cinder | 02:28 | |
| *** itzdilip has joined #openstack-cinder | 02:29 | |
| *** itzdilip has quit IRC | 02:33 | |
| *** itzdilip has joined #openstack-cinder | 02:33 | |
| *** ebalduf has joined #openstack-cinder | 02:37 | |
| *** laughterwym has quit IRC | 02:38 | |
| *** laughterwym has joined #openstack-cinder | 02:38 | |
| *** laughterwym has quit IRC | 02:39 | |
| *** laughter_ has joined #openstack-cinder | 02:39 | |
| *** EinstCrazy has joined #openstack-cinder | 02:44 | |
| *** EinstCrazy has quit IRC | 02:45 | |
| *** mriedem has quit IRC | 02:47 | |
| *** markvoelker has quit IRC | 02:52 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: os-brick refactor get_connector_properties https://review.openstack.org/275943 | 02:52 |
|---|---|---|
| *** ozialien10 has quit IRC | 02:52 | |
| *** ozialien10 has joined #openstack-cinder | 02:52 | |
| *** arch-nemesis has quit IRC | 02:54 | |
| *** nnex has joined #openstack-cinder | 02:55 | |
| *** yangxi has joined #openstack-cinder | 02:57 | |
| *** EinstCrazy has joined #openstack-cinder | 02:57 | |
| *** yangxi has quit IRC | 02:58 | |
| *** yangxi has joined #openstack-cinder | 02:58 | |
| *** EinstCrazy has quit IRC | 02:58 | |
| *** EinstCrazy has joined #openstack-cinder | 03:00 | |
| *** EinstCrazy has quit IRC | 03:01 | |
| *** EinstCrazy has joined #openstack-cinder | 03:03 | |
| *** EinstCrazy has quit IRC | 03:07 | |
| *** EinstCrazy has joined #openstack-cinder | 03:09 | |
| *** dave-mccowan has quit IRC | 03:10 | |
| *** EinstCra_ has joined #openstack-cinder | 03:12 | |
| *** Poornima has quit IRC | 03:13 | |
| *** EinstCra_ has quit IRC | 03:15 | |
| *** EinstCrazy has quit IRC | 03:15 | |
| *** Poornima has joined #openstack-cinder | 03:16 | |
| *** EinstCrazy has joined #openstack-cinder | 03:18 | |
| *** EinstCra_ has joined #openstack-cinder | 03:23 | |
| *** EinstCrazy has quit IRC | 03:23 | |
| *** jamielennox is now known as jamielennox|away | 03:28 | |
| *** itzdilip has quit IRC | 03:35 | |
| *** hemna_ has quit IRC | 03:36 | |
| *** jamielennox|away is now known as jamielennox | 03:37 | |
| *** itzdilip has joined #openstack-cinder | 03:37 | |
| *** itzdilip has quit IRC | 03:42 | |
| *** itzdilip has joined #openstack-cinder | 03:46 | |
| *** yangxi has quit IRC | 03:47 | |
| *** markvoelker has joined #openstack-cinder | 03:52 | |
| *** markvoelker has quit IRC | 03:57 | |
| *** yangxi has joined #openstack-cinder | 03:58 | |
| *** e0ne has joined #openstack-cinder | 03:59 | |
| *** karthikp has quit IRC | 04:00 | |
| *** e0ne has quit IRC | 04:01 | |
| *** e0ne has joined #openstack-cinder | 04:02 | |
| *** EinstCra_ has quit IRC | 04:03 | |
| *** EinstCrazy has joined #openstack-cinder | 04:04 | |
| *** e0ne has quit IRC | 04:05 | |
| *** karthikp has joined #openstack-cinder | 04:05 | |
| *** EinstCrazy has quit IRC | 04:05 | |
| *** EinstCrazy has joined #openstack-cinder | 04:08 | |
| *** EinstCrazy has quit IRC | 04:09 | |
| *** liverpooler has quit IRC | 04:09 | |
| *** EinstCrazy has joined #openstack-cinder | 04:10 | |
| *** EinstCrazy has quit IRC | 04:10 | |
| *** Julien-zte has quit IRC | 04:10 | |
| *** EinstCrazy has joined #openstack-cinder | 04:12 | |
| *** EinstCrazy has quit IRC | 04:13 | |
| *** EinstCrazy has joined #openstack-cinder | 04:14 | |
| *** EinstCrazy has quit IRC | 04:14 | |
| *** EinstCrazy has joined #openstack-cinder | 04:15 | |
| *** EinstCrazy has quit IRC | 04:15 | |
| *** gouthamr has quit IRC | 04:15 | |
| *** EinstCrazy has joined #openstack-cinder | 04:16 | |
| *** EinstCrazy has quit IRC | 04:16 | |
| *** EinstCrazy has joined #openstack-cinder | 04:17 | |
| *** EinstCrazy has quit IRC | 04:19 | |
| *** EinstCrazy has joined #openstack-cinder | 04:21 | |
| *** EinstCra_ has joined #openstack-cinder | 04:22 | |
| *** EinstCrazy has quit IRC | 04:22 | |
| *** EinstCra_ has quit IRC | 04:22 | |
| *** EinstCrazy has joined #openstack-cinder | 04:24 | |
| *** EinstCrazy has quit IRC | 04:24 | |
| *** links has joined #openstack-cinder | 04:25 | |
| *** EinstCra_ has joined #openstack-cinder | 04:27 | |
| *** sgotliv has quit IRC | 04:28 | |
| *** EinstCra_ has quit IRC | 04:28 | |
| *** EinstCrazy has joined #openstack-cinder | 04:32 | |
| *** EinstCrazy has quit IRC | 04:33 | |
| *** EinstCra_ has joined #openstack-cinder | 04:37 | |
| *** EinstCra_ has quit IRC | 04:38 | |
| *** EinstCra_ has joined #openstack-cinder | 04:38 | |
| *** EinstCra_ has quit IRC | 04:38 | |
| *** nnex_ has joined #openstack-cinder | 04:40 | |
| *** avishay has joined #openstack-cinder | 04:42 | |
| *** nnex has quit IRC | 04:42 | |
| *** EinstCrazy has joined #openstack-cinder | 04:43 | |
| *** EinstCrazy has quit IRC | 04:44 | |
| *** amit213 has quit IRC | 04:44 | |
| *** EinstCrazy has joined #openstack-cinder | 04:45 | |
| *** EinstCrazy has quit IRC | 04:45 | |
| *** EinstCrazy has joined #openstack-cinder | 04:46 | |
| *** EinstCrazy has quit IRC | 04:46 | |
| *** akapil has quit IRC | 04:46 | |
| *** laughter_ has quit IRC | 04:46 | |
| *** EinstCrazy has joined #openstack-cinder | 04:46 | |
| *** Julien-zte has joined #openstack-cinder | 04:46 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:46 | |
| *** laughterwym has joined #openstack-cinder | 04:46 | |
| *** kaisers1 has quit IRC | 04:47 | |
| *** kaisers has quit IRC | 04:47 | |
| *** itzdilip has quit IRC | 04:47 | |
| *** EinstCra_ has joined #openstack-cinder | 04:49 | |
| *** raunak has joined #openstack-cinder | 04:49 | |
| *** EinstCrazy has quit IRC | 04:50 | |
| *** laughter_ has joined #openstack-cinder | 04:51 | |
| *** laughterwym has quit IRC | 04:51 | |
| *** kaisers has joined #openstack-cinder | 04:52 | |
| *** EinstCra_ has quit IRC | 04:53 | |
| *** laughter_ has quit IRC | 04:56 | |
| *** akapil has joined #openstack-cinder | 04:56 | |
| *** itzdilip has joined #openstack-cinder | 04:58 | |
| *** EinstCrazy has joined #openstack-cinder | 04:58 | |
| *** EinstCrazy has quit IRC | 04:58 | |
| *** EinstCrazy has joined #openstack-cinder | 04:58 | |
| *** EinstCrazy has quit IRC | 04:59 | |
| *** akapil has quit IRC | 05:01 | |
| *** martyturner has joined #openstack-cinder | 05:01 | |
| *** savihou has joined #openstack-cinder | 05:01 | |
| *** itzdilip has quit IRC | 05:02 | |
| *** harlowja_at_home has quit IRC | 05:02 | |
| *** laughterwym has joined #openstack-cinder | 05:02 | |
| *** laughterwym has quit IRC | 05:02 | |
| *** laughterwym has joined #openstack-cinder | 05:03 | |
| *** laughterwym has quit IRC | 05:04 | |
| *** laughterwym has joined #openstack-cinder | 05:04 | |
| *** laughterwym has quit IRC | 05:05 | |
| *** laughterwym has joined #openstack-cinder | 05:05 | |
| *** EinstCrazy has joined #openstack-cinder | 05:05 | |
| *** martyturner has quit IRC | 05:06 | |
| *** EinstCrazy has quit IRC | 05:06 | |
| *** EinstCrazy has joined #openstack-cinder | 05:07 | |
| *** EinstCrazy has quit IRC | 05:07 | |
| *** laughterwym has quit IRC | 05:07 | |
| *** EinstCrazy has joined #openstack-cinder | 05:07 | |
| *** laughterwym has joined #openstack-cinder | 05:07 | |
| *** laughterwym has quit IRC | 05:07 | |
| *** laughterwym has joined #openstack-cinder | 05:08 | |
| openstackgerrit | Merged openstack/cinder: Move and rename tempest_tests to cinder/tests/tempest https://review.openstack.org/311715 | 05:08 |
| *** itzdilip has joined #openstack-cinder | 05:10 | |
| *** ChubYann has quit IRC | 05:11 | |
| *** shausy has joined #openstack-cinder | 05:11 | |
| *** Julien-zte has quit IRC | 05:11 | |
| *** laughterwym has quit IRC | 05:12 | |
| *** itzdilip has quit IRC | 05:20 | |
| *** itzdilip has joined #openstack-cinder | 05:21 | |
| *** laughterwym has joined #openstack-cinder | 05:33 | |
| *** bardia has joined #openstack-cinder | 05:36 | |
| *** akapil has joined #openstack-cinder | 05:38 | |
| *** jamielennox is now known as jamielennox|away | 05:40 | |
| *** mnaser has quit IRC | 05:41 | |
| *** akapil has quit IRC | 05:43 | |
| *** yangxi has quit IRC | 05:43 | |
| *** raunak has quit IRC | 05:43 | |
| *** karthikp has quit IRC | 05:47 | |
| *** yangxi has joined #openstack-cinder | 05:47 | |
| *** liverpooler has joined #openstack-cinder | 05:55 | |
| *** mnaser has joined #openstack-cinder | 05:55 | |
| *** david-lyle has quit IRC | 06:05 | |
| *** savihou has quit IRC | 06:07 | |
| *** salv-orlando has joined #openstack-cinder | 06:07 | |
| *** savihou has joined #openstack-cinder | 06:07 | |
| *** ankit_ag has joined #openstack-cinder | 06:09 | |
| *** bardia has quit IRC | 06:13 | |
| *** openstackgerrit has quit IRC | 06:17 | |
| *** mjura has joined #openstack-cinder | 06:18 | |
| *** openstackgerrit has joined #openstack-cinder | 06:18 | |
| *** itzdilip has quit IRC | 06:19 | |
| *** itzdilip has joined #openstack-cinder | 06:19 | |
| *** mjura has quit IRC | 06:23 | |
| *** mjura has joined #openstack-cinder | 06:24 | |
| *** itzdilip has quit IRC | 06:26 | |
| *** salv-orlando has quit IRC | 06:36 | |
| *** itzdilip has joined #openstack-cinder | 06:36 | |
| *** itzdilip has quit IRC | 06:39 | |
| *** itzdilip has joined #openstack-cinder | 06:39 | |
| *** vgridnev has joined #openstack-cinder | 06:40 | |
| *** lpetrut has joined #openstack-cinder | 06:40 | |
| *** e0ne has joined #openstack-cinder | 06:41 | |
| openstackgerrit | jiamin proposed openstack/cinder: Failback causes cinder to enable backend which may be frozen before failover https://review.openstack.org/312334 | 06:44 |
| *** itzdilip has quit IRC | 06:47 | |
| DuncanT | Anybody able to take a look at https://review.openstack.org/#/c/309086 please? Fixes a 3par bug, not sure why others don't have the same issue | 06:49 |
| *** ebalduf has quit IRC | 06:50 | |
| *** savihou has quit IRC | 06:52 | |
| *** mjura has quit IRC | 06:52 | |
| *** markvoelker has joined #openstack-cinder | 06:54 | |
| *** laughterwym has quit IRC | 06:56 | |
| *** laughterwym has joined #openstack-cinder | 06:57 | |
| *** markvoelker has quit IRC | 07:00 | |
| *** tesseract has joined #openstack-cinder | 07:01 | |
| *** tesseract is now known as Guest23319 | 07:01 | |
| *** laughter_ has joined #openstack-cinder | 07:01 | |
| *** laughterwym has quit IRC | 07:01 | |
| openstackgerrit | Stefan Amann proposed openstack/cinder-specs: Add support for FICON attached ECKD volumes for Linux on System z https://review.openstack.org/302221 | 07:10 |
| lixiaoy1 | DuncanT: I added some comments in the https://review.openstack.org/#/c/309086 | 07:11 |
| DuncanT | Thanks | 07:11 |
| *** yangxi has quit IRC | 07:15 | |
| *** akapil has joined #openstack-cinder | 07:17 | |
| *** yangxi has joined #openstack-cinder | 07:19 | |
| *** akapil has quit IRC | 07:21 | |
| *** jamielennox|away is now known as jamielennox | 07:22 | |
| *** savihou has joined #openstack-cinder | 07:27 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder-specs: Inspection Mechanism For Capacity Limited Host https://review.openstack.org/308869 | 07:27 |
| *** akapil has joined #openstack-cinder | 07:34 | |
| openstackgerrit | LisaLi proposed openstack/cinder-specs: Retype encrypted volumes https://review.openstack.org/248593 | 07:34 |
| *** akapil has quit IRC | 07:35 | |
| *** salv-orlando has joined #openstack-cinder | 07:36 | |
| *** lpetrut has quit IRC | 07:40 | |
| *** salv-orlando has quit IRC | 07:41 | |
| *** itzdilip has joined #openstack-cinder | 07:50 | |
| *** lpetrut has joined #openstack-cinder | 07:54 | |
| *** EinstCra_ has joined #openstack-cinder | 07:57 | |
| *** lpetrut has quit IRC | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** itzdilip has quit IRC | 08:00 | |
| *** itzdilip has joined #openstack-cinder | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| *** EinstCrazy has quit IRC | 08:00 | |
| openstackgerrit | KongWei proposed openstack/cinder: Add ZTE Block Storage Driver https://review.openstack.org/312355 | 08:01 |
| *** arecknag has joined #openstack-cinder | 08:03 | |
| openstackgerrit | Wenzhi Yu proposed openstack/python-cinderclient: Replace keystoneclient.adapter with keystoneauth1.adapter https://review.openstack.org/312357 | 08:03 |
| *** geguileo has joined #openstack-cinder | 08:04 | |
| *** laughterwym has joined #openstack-cinder | 08:06 | |
| *** laughter_ has quit IRC | 08:10 | |
| *** laughter_ has joined #openstack-cinder | 08:10 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| *** laughter_ has quit IRC | 08:10 | |
| *** laughterwym has quit IRC | 08:10 | |
| *** itzdilip has quit IRC | 08:15 | |
| *** laughterwym has joined #openstack-cinder | 08:16 | |
| *** laughterwym has quit IRC | 08:16 | |
| *** laughterwym has joined #openstack-cinder | 08:16 | |
| *** mvk_ has joined #openstack-cinder | 08:23 | |
| openstackgerrit | Jean-Sebastien Bevilacqua proposed openstack/cinder: Better exception handling in volume_actions https://review.openstack.org/312057 | 08:23 |
| *** jistr has joined #openstack-cinder | 08:24 | |
| *** yuriy_n17 has joined #openstack-cinder | 08:24 | |
| *** mvk has quit IRC | 08:26 | |
| *** ozialien10 has quit IRC | 08:28 | |
| *** ozialien10 has joined #openstack-cinder | 08:29 | |
| *** bhbhavsa has quit IRC | 08:30 | |
| *** enriquetaso has quit IRC | 08:32 | |
| *** bhbhavsa has joined #openstack-cinder | 08:32 | |
| *** markus_z has joined #openstack-cinder | 08:33 | |
| *** yangxi has quit IRC | 08:34 | |
| *** salv-orlando has joined #openstack-cinder | 08:38 | |
| *** yangxi has joined #openstack-cinder | 08:38 | |
| *** itzdilip has joined #openstack-cinder | 08:39 | |
| *** salv-orlando has quit IRC | 08:42 | |
| *** avishay has quit IRC | 08:44 | |
| *** itzdilip has quit IRC | 08:44 | |
| *** itzdilip has joined #openstack-cinder | 08:46 | |
| openstackgerrit | Stefan Amann proposed openstack/cinder-specs: Add support for FICON attached ECKD volumes for Linux on System z https://review.openstack.org/302221 | 08:49 |
| *** markvoelker has joined #openstack-cinder | 08:50 | |
| *** sgotliv has joined #openstack-cinder | 08:50 | |
| *** itzdilip has quit IRC | 08:52 | |
| *** markvoelker has quit IRC | 08:55 | |
| *** avishay has joined #openstack-cinder | 08:56 | |
| *** jordanP has joined #openstack-cinder | 08:58 | |
| *** yangyape_ has joined #openstack-cinder | 09:02 | |
| *** iceyao___ has quit IRC | 09:04 | |
| *** yangyap__ has joined #openstack-cinder | 09:04 | |
| *** iceyao has joined #openstack-cinder | 09:04 | |
| *** yangyapeng has quit IRC | 09:06 | |
| *** yangyape_ has quit IRC | 09:07 | |
| *** iceyao_ has joined #openstack-cinder | 09:08 | |
| *** laughterwym has quit IRC | 09:10 | |
| *** laughterwym has joined #openstack-cinder | 09:11 | |
| *** iceyao has quit IRC | 09:12 | |
| *** laughterwym has quit IRC | 09:15 | |
| *** laughterwym has joined #openstack-cinder | 09:15 | |
| *** sgotliv has quit IRC | 09:19 | |
| *** sgotliv has joined #openstack-cinder | 09:20 | |
| *** laughterwym has quit IRC | 09:20 | |
| *** yangxi has quit IRC | 09:22 | |
| *** yangxi has joined #openstack-cinder | 09:24 | |
| *** laughterwym has joined #openstack-cinder | 09:32 | |
| *** akapil has joined #openstack-cinder | 09:32 | |
| *** sdague has joined #openstack-cinder | 09:33 | |
| openstackgerrit | wanghao proposed openstack/cinder: Catch GlanceMetadataNotFound in creating cgsnp https://review.openstack.org/302691 | 09:36 |
| *** laughterwym has quit IRC | 09:37 | |
| *** akapil has quit IRC | 09:37 | |
| *** lpetrut has joined #openstack-cinder | 09:38 | |
| *** salv-orlando has joined #openstack-cinder | 09:38 | |
| *** salv-orlando has quit IRC | 09:43 | |
| *** akapil has joined #openstack-cinder | 09:44 | |
| *** laughterwym has joined #openstack-cinder | 09:46 | |
| *** akapil has quit IRC | 09:48 | |
| *** laughterwym has quit IRC | 09:50 | |
| *** cheneydc has quit IRC | 10:01 | |
| *** laughterwym has joined #openstack-cinder | 10:02 | |
| openstackgerrit | Wenzhi Yu proposed openstack/python-cinderclient: Replace keystoneclient.adapter with keystoneauth1.adapter https://review.openstack.org/312357 | 10:04 |
| *** itzdilip has joined #openstack-cinder | 10:06 | |
| *** e0ne has quit IRC | 10:06 | |
| *** e0ne has joined #openstack-cinder | 10:06 | |
| *** laughterwym has quit IRC | 10:07 | |
| *** yangxi has quit IRC | 10:10 | |
| *** laughterwym has joined #openstack-cinder | 10:14 | |
| *** itzdilip has quit IRC | 10:15 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: VNX: Cinder driver refactor in Newton https://review.openstack.org/312427 | 10:16 |
| *** laughterwym has quit IRC | 10:18 | |
| *** ozialien10 has quit IRC | 10:18 | |
| *** ozialien10 has joined #openstack-cinder | 10:18 | |
| *** itzdilip has joined #openstack-cinder | 10:24 | |
| *** yangyap__ has quit IRC | 10:29 | |
| *** itzdilip has quit IRC | 10:29 | |
| *** itzdilip has joined #openstack-cinder | 10:30 | |
| *** EinstCra_ has quit IRC | 10:30 | |
| *** laughterwym has joined #openstack-cinder | 10:36 | |
| openstackgerrit | oliver-leahy-l proposed openstack/cinder: Add provider_location to cloned volume https://review.openstack.org/309086 | 10:39 |
| *** salv-orlando has joined #openstack-cinder | 10:40 | |
| *** laughterwym has quit IRC | 10:40 | |
| *** links has quit IRC | 10:45 | |
| *** salv-orlando has quit IRC | 10:45 | |
| *** mvk_ has quit IRC | 10:46 | |
| *** dave-mccowan has joined #openstack-cinder | 10:51 | |
| tbarron | dulek: About time! :-) You've been a very helpful reviewer in my experience so it's great to see the right thing happening. | 10:56 |
| *** david-lyle has joined #openstack-cinder | 10:58 | |
| *** iceyao_ has quit IRC | 11:01 | |
| sheel | tbarron: +1 | 11:12 |
| *** belmoreira has joined #openstack-cinder | 11:15 | |
| *** akapil has joined #openstack-cinder | 11:18 | |
| *** akapil has quit IRC | 11:22 | |
| *** laughterwym has joined #openstack-cinder | 11:27 | |
| *** laughterwym has quit IRC | 11:32 | |
| *** timcl has joined #openstack-cinder | 11:33 | |
| *** yangyapeng has joined #openstack-cinder | 11:37 | |
| *** salv-orlando has joined #openstack-cinder | 11:40 | |
| *** mjura has joined #openstack-cinder | 11:42 | |
| *** salv-orlando has quit IRC | 11:46 | |
| *** mvk_ has joined #openstack-cinder | 11:54 | |
| *** avishay has quit IRC | 11:59 | |
| *** links has joined #openstack-cinder | 11:59 | |
| *** plastikman has quit IRC | 12:01 | |
| *** plastikman has joined #openstack-cinder | 12:02 | |
| *** cdelatte has joined #openstack-cinder | 12:02 | |
| *** crose has joined #openstack-cinder | 12:03 | |
| *** markvoelker has joined #openstack-cinder | 12:04 | |
| *** e0ne has quit IRC | 12:05 | |
| *** itzdilip has quit IRC | 12:13 | |
| *** raildo-afk is now known as raildo | 12:13 | |
| *** martyturner has joined #openstack-cinder | 12:17 | |
| *** e0ne has joined #openstack-cinder | 12:22 | |
| *** dave-mcc_ has joined #openstack-cinder | 12:24 | |
| *** ozialien10 has quit IRC | 12:24 | |
| *** ozialien10 has joined #openstack-cinder | 12:25 | |
| *** shausy has quit IRC | 12:26 | |
| *** akapil has joined #openstack-cinder | 12:27 | |
| *** dave-mccowan has quit IRC | 12:27 | |
| *** akapil has quit IRC | 12:28 | |
| *** akapil has joined #openstack-cinder | 12:28 | |
| *** eharney has quit IRC | 12:29 | |
| arnewiebalck | I just noticed that in my test env “list —all” on Liberty and v2 comes back empty (while v1 works fine and lists all volumes). From what I see, this is due to a “all_tenants” to “all_projects” change … or am I missing sth? | 12:31 |
| *** akapil has quit IRC | 12:33 | |
| *** akapil has joined #openstack-cinder | 12:36 | |
| *** mjura has quit IRC | 12:39 | |
| *** iceyao has joined #openstack-cinder | 12:40 | |
| *** akapil has quit IRC | 12:41 | |
| *** salv-orlando has joined #openstack-cinder | 12:41 | |
| *** ankit_ag has quit IRC | 12:45 | |
| *** laughterwym has joined #openstack-cinder | 12:46 | |
| *** salv-orlando has quit IRC | 12:46 | |
| *** laughterwym has quit IRC | 12:46 | |
| *** links has quit IRC | 12:46 | |
| *** laughterwym has joined #openstack-cinder | 12:47 | |
| *** julim has joined #openstack-cinder | 12:47 | |
| *** laughterwym has quit IRC | 12:48 | |
| *** akerr has joined #openstack-cinder | 12:48 | |
| *** laughterwym has joined #openstack-cinder | 12:48 | |
| *** eharney has joined #openstack-cinder | 12:49 | |
| openstackgerrit | KongWei proposed openstack/cinder: Add ZTE Block Storage Driver https://review.openstack.org/312355 | 12:52 |
| *** jseiler__ has quit IRC | 12:52 | |
| *** jseiler__ has joined #openstack-cinder | 12:53 | |
| *** laughterwym has quit IRC | 12:53 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: tempest: Add Unicode volume name test https://review.openstack.org/310925 | 12:54 |
| *** akshai has joined #openstack-cinder | 12:55 | |
| *** rlrossit has joined #openstack-cinder | 12:55 | |
| *** andymaier has joined #openstack-cinder | 12:56 | |
| *** akapil has joined #openstack-cinder | 12:56 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tempest: Add volume manage/unmanage test https://review.openstack.org/310949 | 12:56 |
| *** ri0 has joined #openstack-cinder | 13:00 | |
| *** akapil has quit IRC | 13:01 | |
| *** merooney has joined #openstack-cinder | 13:03 | |
| *** nnex_ has quit IRC | 13:03 | |
| *** merooney has quit IRC | 13:09 | |
| *** salv-orlando has joined #openstack-cinder | 13:09 | |
| *** krotscheck_dcm is now known as krotscheck | 13:10 | |
| *** akapil has joined #openstack-cinder | 13:11 | |
| *** cheneydc has joined #openstack-cinder | 13:12 | |
| *** ri0 has left #openstack-cinder | 13:12 | |
| *** salv-orlando has quit IRC | 13:13 | |
| *** akapil_ has joined #openstack-cinder | 13:15 | |
| *** gouthamr has joined #openstack-cinder | 13:15 | |
| openstackgerrit | Gyorgy Szombathelyi proposed openstack/cinder: Improve readabilty of CA verifying code https://review.openstack.org/299361 | 13:15 |
| *** akapil has quit IRC | 13:16 | |
| *** porrua has joined #openstack-cinder | 13:16 | |
| *** cheneydc has quit IRC | 13:16 | |
| *** merooney has joined #openstack-cinder | 13:16 | |
| *** mriedem has joined #openstack-cinder | 13:16 | |
| *** akapil_ has quit IRC | 13:20 | |
| *** xyang1 has joined #openstack-cinder | 13:22 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: WIP: Use oslo.privsep for LIO https://review.openstack.org/312498 | 13:24 |
| *** dustins has joined #openstack-cinder | 13:24 | |
| *** diogogmt has quit IRC | 13:28 | |
| *** esker has quit IRC | 13:29 | |
| arnewiebalck | arnewiebalck: this is a bug in the openstack client (fixed in 1.8.0). The client was passing ‘all_projects’ in v2 which has been changed back to ‘all_tenants’. | 13:30 |
| *** diogogmt has joined #openstack-cinder | 13:31 | |
| *** The_Ball has quit IRC | 13:32 | |
| *** BigWillie has joined #openstack-cinder | 13:33 | |
| scottda | smcginnis: FYI Conference room is available for Dublin mid-cycle June 27-29 . (actually 15 miles from Dublin in Leixlip) | 13:35 |
| scottda | smcginnis: I'll likely not be sent to Dublin, so DuncanT would have to take charge of hosting. | 13:35 |
| smcginnis | scottda: Thanks! Hate to create more work, but did you also check hotel availability in Ft Collins for the R10 week? | 13:37 |
| scottda | tranlating R10... | 13:39 |
| scottda | smcginnis: July 25 -29? That was the 2nd choice, right? I did ask the hotel and they could do it, but week could get busy so I should be reserving rooms ASAP | 13:40 |
| openstackgerrit | jiamin proposed openstack/cinder: Failback will enable the host that has been frozen https://review.openstack.org/312334 | 13:41 |
| smcginnis | scottda: OK, thanks. Hopefully we can finalize this in today's meeting and get things set one way or the other. | 13:41 |
| smcginnis | scottda: Thanks for all your help with this. | 13:42 |
| scottda | smcginnis: NP | 13:42 |
| *** diablo_rojo has joined #openstack-cinder | 13:42 | |
| *** diogogmt has quit IRC | 13:43 | |
| *** Poornima has quit IRC | 13:45 | |
| *** zioproto has quit IRC | 13:47 | |
| *** sdake has joined #openstack-cinder | 13:48 | |
| *** sdake has quit IRC | 13:48 | |
| *** laughterwym has joined #openstack-cinder | 13:50 | |
| *** laughterwym has quit IRC | 13:50 | |
| *** akapil has joined #openstack-cinder | 13:50 | |
| *** laughterwym has joined #openstack-cinder | 13:50 | |
| *** laughterwym has quit IRC | 13:50 | |
| *** cknight has joined #openstack-cinder | 13:51 | |
| *** jgregor has joined #openstack-cinder | 13:51 | |
| *** laughterwym has joined #openstack-cinder | 13:51 | |
| *** akapil has quit IRC | 13:52 | |
| *** zioproto has joined #openstack-cinder | 13:52 | |
| *** merooney has quit IRC | 13:52 | |
| *** diablo_rojo has quit IRC | 13:53 | |
| smcginnis | Survey results again for those interested in the midcycle discussion: https://docs.google.com/document/d/1DT2Gj64efBDIvgG2j1W3jBw3u5Zs2btsWvQQOhn-j1g/edit?usp=sharing | 13:53 |
| *** akapil has joined #openstack-cinder | 13:53 | |
| *** savihou has quit IRC | 13:53 | |
| *** savihou has joined #openstack-cinder | 13:55 | |
| *** ametts has joined #openstack-cinder | 13:55 | |
| *** laughterwym has quit IRC | 13:56 | |
| openstackgerrit | oliver-leahy-l proposed openstack/cinder: Add provider_location to cloned volume https://review.openstack.org/309086 | 13:57 |
| *** dave-mcc_ has quit IRC | 13:57 | |
| *** diablo_rojo has joined #openstack-cinder | 13:57 | |
| *** laughterwym has joined #openstack-cinder | 13:58 | |
| *** merooney has joined #openstack-cinder | 13:59 | |
| *** jwcroppe has joined #openstack-cinder | 14:01 | |
| *** zioproto has quit IRC | 14:02 | |
| *** diogogmt has joined #openstack-cinder | 14:02 | |
| *** ivase has joined #openstack-cinder | 14:02 | |
| *** zioproto has joined #openstack-cinder | 14:05 | |
| *** jungleboyj has joined #openstack-cinder | 14:06 | |
| *** sdake has joined #openstack-cinder | 14:06 | |
| *** raunak has joined #openstack-cinder | 14:07 | |
| *** baumann has joined #openstack-cinder | 14:08 | |
| *** salv-orlando has joined #openstack-cinder | 14:10 | |
| *** salv-orlando has quit IRC | 14:15 | |
| *** jbernard has joined #openstack-cinder | 14:15 | |
| *** liverpooler has quit IRC | 14:15 | |
| *** merooney has quit IRC | 14:19 | |
| *** ebalduf has joined #openstack-cinder | 14:21 | |
| *** lamt has joined #openstack-cinder | 14:22 | |
| *** jbernard_ has left #openstack-cinder | 14:23 | |
| *** merooney has joined #openstack-cinder | 14:23 | |
| *** jwcroppe has quit IRC | 14:23 | |
| *** salv-orlando has joined #openstack-cinder | 14:24 | |
| *** The_Ball has joined #openstack-cinder | 14:26 | |
| *** akshai has quit IRC | 14:26 | |
| *** mtanino has joined #openstack-cinder | 14:28 | |
| *** salv-orlando has quit IRC | 14:29 | |
| *** akshai has joined #openstack-cinder | 14:30 | |
| *** BigWillie has quit IRC | 14:30 | |
| *** ozialien10 has quit IRC | 14:31 | |
| *** Yogi1 has joined #openstack-cinder | 14:31 | |
| *** jwcroppe has joined #openstack-cinder | 14:31 | |
| *** julim has quit IRC | 14:32 | |
| *** jwcroppe_ has joined #openstack-cinder | 14:32 | |
| *** sgotliv has quit IRC | 14:32 | |
| openstackgerrit | Alex Meade proposed openstack/python-cinderclient: Add v3 user messages https://review.openstack.org/299052 | 14:33 |
| openstackgerrit | Alex Meade proposed openstack/python-cinderclient: Add v3 user messages https://review.openstack.org/299052 | 14:34 |
| *** julim has joined #openstack-cinder | 14:34 | |
| *** jwcroppe has quit IRC | 14:36 | |
| *** ifat_afek has joined #openstack-cinder | 14:36 | |
| *** ifat_afek has left #openstack-cinder | 14:36 | |
| *** itzdilip has joined #openstack-cinder | 14:37 | |
| *** ankur-gupta-f1 has quit IRC | 14:38 | |
| openstackgerrit | KongWei proposed openstack/cinder: Add ZTE Block Storage Driver https://review.openstack.org/312355 | 14:40 |
| openstackgerrit | Alex Meade proposed openstack/python-cinderclient: Add v3 user messages https://review.openstack.org/299052 | 14:40 |
| openstackgerrit | Eric Harney proposed openstack/cinder: WIP: Use oslo.privsep for LIO https://review.openstack.org/312498 | 14:43 |
| *** crose has quit IRC | 14:43 | |
| *** jwcroppe_ has quit IRC | 14:44 | |
| *** jwcroppe has joined #openstack-cinder | 14:45 | |
| *** lprice has quit IRC | 14:46 | |
| *** e0ne has quit IRC | 14:50 | |
| *** akshai has quit IRC | 14:53 | |
| *** jungleboyj has quit IRC | 14:53 | |
| *** esker has joined #openstack-cinder | 14:54 | |
| *** salv-orlando has joined #openstack-cinder | 14:54 | |
| *** akshai has joined #openstack-cinder | 14:54 | |
| *** Julien-zte has joined #openstack-cinder | 14:55 | |
| *** garthb has joined #openstack-cinder | 14:56 | |
| *** ankur-gupta-f1 has joined #openstack-cinder | 14:56 | |
| *** karthikp has joined #openstack-cinder | 14:56 | |
| *** itzdilip has quit IRC | 14:56 | |
| *** itzdilip has joined #openstack-cinder | 14:56 | |
| *** esker has quit IRC | 14:58 | |
| *** haplo37 has joined #openstack-cinder | 14:58 | |
| *** esker has joined #openstack-cinder | 14:58 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tempest: Add volume manage/unmanage test https://review.openstack.org/310949 | 14:59 |
| *** salv-orlando has quit IRC | 14:59 | |
| *** akapil has quit IRC | 14:59 | |
| *** cknight has quit IRC | 14:59 | |
| *** merooney has quit IRC | 15:00 | |
| *** cknight has joined #openstack-cinder | 15:01 | |
| *** jdurgin1 has joined #openstack-cinder | 15:01 | |
| *** akapil has joined #openstack-cinder | 15:01 | |
| *** raunak has quit IRC | 15:03 | |
| *** merooney has joined #openstack-cinder | 15:04 | |
| cFouts | Would really appreciate a core review of https://review.openstack.org/#/c/309202/ Thanks! | 15:06 |
| *** jungleboyj has joined #openstack-cinder | 15:06 | |
| *** crose has joined #openstack-cinder | 15:06 | |
| *** lprice has joined #openstack-cinder | 15:07 | |
| *** raunak has joined #openstack-cinder | 15:07 | |
| *** vgridnev has quit IRC | 15:10 | |
| *** vgridnev has joined #openstack-cinder | 15:11 | |
| *** vgridnev has quit IRC | 15:11 | |
| *** arch-nemesis has joined #openstack-cinder | 15:11 | |
| jgriffith | cFouts: seems ok to me | 15:12 |
| jgriffith | cFouts: curioius... have you guys looked at the general implementation as well? | 15:12 |
| *** merooney has quit IRC | 15:12 | |
| *** nnex has joined #openstack-cinder | 15:12 | |
| jgriffith | cFouts: I've sort of abandoned my custom version to push people that direction | 15:12 |
| jgriffith | cFouts: might be something you want to consider as well | 15:13 |
| openstackgerrit | Alex Meade proposed openstack/cinder-specs: Promote backend after failover (cheesecake) https://review.openstack.org/312591 | 15:14 |
| cFouts | jgriffith: I don't know that we have. Do you have a WIP up? | 15:14 |
| jgriffith | cFouts: no WIP, it's merged :) | 15:14 |
| jgriffith | cFouts: patrickeast worked it up and has docs on it somewhere :) | 15:15 |
| *** raunak has quit IRC | 15:15 | |
| *** lprice1 has joined #openstack-cinder | 15:15 | |
| cFouts | hah. I'll look for it. A bit of that code needs to be touched anyway | 15:16 |
| jgriffith | cFouts: cool... other than some phrasing in your commit message patch looks fine to me, I gave it a +2 but if you think about it you might want to reword the commit mesg | 15:16 |
| *** lprice has quit IRC | 15:17 | |
| *** lpetrut has quit IRC | 15:18 | |
| *** jwcroppe has quit IRC | 15:19 | |
| smcginnis | cFouts: Here's the original patch: https://review.openstack.org/#/c/195795/ | 15:19 |
| *** jwcroppe has joined #openstack-cinder | 15:19 | |
| *** lpetrut has joined #openstack-cinder | 15:23 | |
| openstackgerrit | Rajini Ram proposed openstack/cinder: Dell Eqlx: Volume from snapshot now honors resize https://review.openstack.org/312160 | 15:24 |
| *** akapil has quit IRC | 15:25 | |
| *** xinli has joined #openstack-cinder | 15:26 | |
| jgriffith | eharney: ping | 15:26 |
| eharney | jgriffith: hey | 15:26 |
| jgriffith | eharney: howdy | 15:27 |
| *** xinli has quit IRC | 15:27 | |
| jgriffith | eharney: hey, I wanted to chat with you about that volume-attachments entry for initiator... | 15:27 |
| *** akapil has joined #openstack-cinder | 15:27 | |
| *** xinli has joined #openstack-cinder | 15:27 | |
| jgriffith | eharney: https://review.openstack.org/#/c/284867/2/cinder/db/sqlalchemy/migrate_repo/versions/065_add_connector_fields_to_volume_attachment.py | 15:27 |
| jgriffith | eharney: wondered if you had any ideas for a *generic* term to use there? | 15:27 |
| sheel | ameade: hi | 15:28 |
| jgriffith | eharney: I'm not familiar enough with things like Gluster, RBD etc to know what does or doesn't make sense | 15:28 |
| jgriffith | but the connector always has that field I believe so I just used it :) | 15:28 |
| *** akapil has quit IRC | 15:28 | |
| *** akapil has joined #openstack-cinder | 15:29 | |
| jgriffith | eharney: I guess my question is; if we want to call it something other than initiator that's fine... but what? | 15:30 |
| eharney | jgriffith: let me look at it for a minute, trying to get the idea in my head of what this is for exactly | 15:30 |
| jgriffith | eharney: intiator is pretty generic IMO | 15:30 |
| jgriffith | Ok, no rush | 15:30 |
| jgriffith | eharney: if you just want to think on it and ping me later some time that's cool | 15:30 |
| jgriffith | don't want to interrupt you if you're in the middle of something | 15:30 |
| eharney | jgriffith: it's just a unique id of someone attaching it, right? | 15:30 |
| jgriffith | eharney: no, actually | 15:31 |
| jgriffith | eharney: it's the actual iqn of the initiator | 15:31 |
| eharney | jgriffith: ah ok | 15:31 |
| jgriffith | eharney: that's how I was able to tell if it was multi-attached or not | 15:31 |
| *** akapil has quit IRC | 15:31 | |
| jgriffith | eharney: tables for volumes has initiator xyz >1 == multi-attach don't nuke it | 15:31 |
| jgriffith | eharney: that wasn't a very clear explanation :) What I'm saying is, I can grab all attahcments out of the attachment table for a volume... and know if it's attached to the same compute node more than once | 15:33 |
| jgriffith | eharney: by having the initiator column. And yes, to some peoples point I could just use host but I don't like that. There are possible contentious issues with multi-path I think | 15:33 |
| jgriffith | eharney: and just initiator is full-proof | 15:34 |
| jgriffith | I suppose it may not matter | 15:34 |
| *** belmoreira has quit IRC | 15:34 | |
| * eharney realizes he needs a lot more coffee | 15:35 | |
| jgriffith | hmmm... me too :) | 15:35 |
| eharney | jgriffith: nah, this makes sense... i'm just not sure how to translate it into what other drivers need (or maybe don't need?) | 15:35 |
| jgriffith | so I can certainly make this work just using host | 15:35 |
| jgriffith | eharney: yeah... me neither that's why I pinged you :) | 15:35 |
| jgriffith | Maybe I'll try a first pass just using host | 15:36 |
| jgriffith | kinda hacky IMO but hey... this is OpenStack | 15:36 |
| smcginnis | Quote of the day. :D | 15:36 |
| jgriffith | smcginnis: :) | 15:36 |
| *** jistr has quit IRC | 15:37 | |
| jgriffith | eharney: my real beef with all of this code was that we require it and already have it being passed in, but we don't persist it | 15:38 |
| jgriffith | eharney: we have it during initialize_connection, and THEN we also require it again on attach | 15:38 |
| jgriffith | eharney: seems like crap to me, duplicating everything over and over when we already had it | 15:38 |
| jgriffith | eharney: literally milliseconds ago | 15:38 |
| *** akapil has joined #openstack-cinder | 15:39 | |
| *** akapil has quit IRC | 15:39 | |
| openstackgerrit | Chuck Fouts proposed openstack/cinder: NetApp: Use Local Cache to Copy an Available Image https://review.openstack.org/309202 | 15:39 |
| jgriffith | eharney: I could do a generic thing similar to provider_location? | 15:39 |
| jgriffith | eharney: then folks that don't use those things or use other things can just have a different model | 15:39 |
| cFouts | jgriffith and smcginnis: Had to step away for standup. Thanks for the info. | 15:40 |
| cFouts | jgriffith: made the change to the commit message. thanks for the review | 15:40 |
| jgriffith | eharney: maybe something like: {attach_location: intiiator: foo, ip: xyz, multipath: true/false}} | 15:40 |
| *** sdake_ has joined #openstack-cinder | 15:41 | |
| *** leeantho has joined #openstack-cinder | 15:41 | |
| jgriffith | eharney: then for Gluster and others you guys can settle on a model that works for you? | 15:41 |
| jgriffith | cFouts: thanks! | 15:41 |
| jgriffith | eharney: ipv6 is a bit of a problem still though | 15:42 |
| jbernard | initiator can be anything, no? | 15:42 |
| jgriffith | jbernard: Yeah, I suppose so... it's just a string | 15:43 |
| jbernard | ok, that should work | 15:43 |
| jgriffith | jbernard: you can abuse that however you like :) | 15:43 |
| *** sdake has quit IRC | 15:43 | |
| eharney | i guess in the case of the NFS driver, it may be more of an id of the host that mounted the NFS export? | 15:43 |
| jbernard | ;) you read my mind :) | 15:43 |
| jgriffith | eharney: yeah, like a host-uuid? | 15:43 |
| *** iceyao has quit IRC | 15:43 | |
| jgriffith | eharney: or honestly if it's just not needed we can ignore it (if None ignore) | 15:43 |
| *** salv-orlando has joined #openstack-cinder | 15:44 | |
| jbernard | jgriffith: for rbd, there is no session to reference count, so i cant see any issue at the moment, but im still reading | 15:44 |
| jgriffith | eharney: well.. if None and NOT iSCSI | 15:44 |
| jgriffith | jbernard: cool | 15:44 |
| jgriffith | jbernard: ok, so maybe I'll move forward with that and add some comments and handling for folks that it doesn't mean anything to | 15:44 |
| eharney | the tricky part for *FS is that the question is a little different i think, since the mount/connection is for X volumes instead of 1 | 15:44 |
| jgriffith | eharney: yeah, I know I'm going to hit that shortly :) | 15:45 |
| jgriffith | eharney: but I think we do things differently there anyway | 15:45 |
| jgriffith | eharney: keep in mind this is really only for a handful of drivers that can't create multiple targets/connections | 15:45 |
| jgriffith | eharney: errr... multiple targets connected to same initiator | 15:45 |
| jgriffith | eharney: I think LIO is in that camp FWIW | 15:46 |
| ameade | hey sheel, thanks for the client review | 15:46 |
| *** sdake_ has quit IRC | 15:46 | |
| *** Guest23319 has quit IRC | 15:46 | |
| sheel | ameade: yw | 15:46 |
| ameade | i didn't rename message->message_id for that kwarg since the rest of the file doesnt | 15:46 |
| ameade | but i agree with that | 15:46 |
| sheel | ameade: its ok...very minor .. | 15:47 |
| sheel | ameade: I wanted to test that client part... | 15:47 |
| sheel | ameade: what change required for testing this from client? | 15:47 |
| ameade | all you need to do is pull the client and install it, then you can do cinder --os-volume-api-version 3.3 message-list | 15:48 |
| eharney | jgriffith: i think LIO isn't | 15:49 |
| eharney | but i could be missing something | 15:49 |
| sheel | ameade: oh quite easy... thanks for pointer | 15:49 |
| sheel | ameade: wil update review soon.. | 15:49 |
| jgriffith | eharney: even better then :) | 15:49 |
| ameade | sheel: awesome thanks | 15:49 |
| sheel | ameade: also i am going to submit patch for pagination support for messages | 15:49 |
| *** e0ne has joined #openstack-cinder | 15:49 | |
| jgriffith | ok... so jbernard can *abuse it* on his side, we'll just see what Gluster and NFS look like | 15:49 |
| ameade | kk cool, just lemme know and i'll review/test | 15:50 |
| sheel | ameade: sure, thanks... | 15:50 |
| ameade | scottda: hey for the cinderclient, what is the default behavior supposed to be for microversions? is it supposed to use the latest? | 15:51 |
| ameade | i have to explicitly say 3.3 atm for it to work with messages | 15:51 |
| scottda | ameade: Currently default is to use /v2 endpoint, so no microversions whatsovever | 15:51 |
| ameade | oh yeah that makes sense lol | 15:51 |
| scottda | I've some code to write that would negotiate the highest supported microversion between client and server, but I reckon that negotiation feature might be off by default.... | 15:52 |
| *** andymaier has quit IRC | 15:52 | |
| scottda | some folks don't want that extra round trip for each CLI call. | 15:52 |
| ameade | if i say CINDER_VERSION=3 should that use the latest? | 15:52 |
| scottda | That negotiation thing is what Manila does. | 15:53 |
| *** apoorvad has joined #openstack-cinder | 15:53 | |
| scottda | ameade: No, it will just use 3.0 | 15:53 |
| *** adrianofr has joined #openstack-cinder | 15:53 | |
| ameade | kk | 15:53 |
| *** plastikman has quit IRC | 15:53 | |
| smcginnis | ameade: Don't want anything to use LATEST by default except maybe tests. | 15:53 |
| ameade | yeah makes sense | 15:53 |
| smcginnis | ameade: Otherwise you might get a new behavior your not expecting. | 15:53 |
| scottda | ameade: You can set OS_VOLUME_API_VERSION=3.3 in the ENV | 15:53 |
| ameade | yeah we had that whole thing in manila | 15:54 |
| ameade | scottda: just tested and that works too | 15:54 |
| scottda | Latest = Bad! | 15:54 |
| *** asselin has joined #openstack-cinder | 15:56 | |
| *** sgotliv has joined #openstack-cinder | 15:56 | |
| *** jwcroppe has quit IRC | 15:57 | |
| *** jistr has joined #openstack-cinder | 15:58 | |
| *** asselin__ has quit IRC | 15:58 | |
| *** jwcroppe has joined #openstack-cinder | 15:59 | |
| *** salv-orlando has quit IRC | 15:59 | |
| *** sasukeh has joined #openstack-cinder | 16:00 | |
| *** salv-orlando has joined #openstack-cinder | 16:00 | |
| *** kfarr has joined #openstack-cinder | 16:01 | |
| *** lpetrut has quit IRC | 16:02 | |
| *** Julien-zte has quit IRC | 16:02 | |
| *** salv-orlando has quit IRC | 16:03 | |
| *** dustins has quit IRC | 16:03 | |
| *** xinli has quit IRC | 16:03 | |
| *** bardia has joined #openstack-cinder | 16:05 | |
| openstackgerrit | Tomaz Vieira proposed openstack/cinder: Improve hitachi hnas volume drivers log messages https://review.openstack.org/310785 | 16:08 |
| *** jwcroppe_ has joined #openstack-cinder | 16:11 | |
| *** jdurgin1 has quit IRC | 16:11 | |
| *** diablo_rojo has quit IRC | 16:13 | |
| *** jwcroppe has quit IRC | 16:14 | |
| *** diablo_rojo has joined #openstack-cinder | 16:14 | |
| *** markstur_ is now known as markstur | 16:19 | |
| *** rhagarty_ has quit IRC | 16:19 | |
| *** rhagarty has joined #openstack-cinder | 16:19 | |
| *** daneyon has quit IRC | 16:20 | |
| *** jordanP has quit IRC | 16:20 | |
| *** akapil has joined #openstack-cinder | 16:21 | |
| *** jwcroppe has joined #openstack-cinder | 16:22 | |
| *** asselin has quit IRC | 16:23 | |
| *** sdake has joined #openstack-cinder | 16:24 | |
| *** akapil has quit IRC | 16:25 | |
| *** jwcroppe_ has quit IRC | 16:25 | |
| *** xinli has joined #openstack-cinder | 16:27 | |
| *** laughterwym has quit IRC | 16:27 | |
| e0ne | leeantho: hi Anthony | 16:27 |
| leeantho | e0ne, hey | 16:28 |
| *** laughterwym has joined #openstack-cinder | 16:28 | |
| e0ne | leeantho: I've got few questions about privsep in os-brick if you've got a time | 16:29 |
| *** ozialien10 has joined #openstack-cinder | 16:29 | |
| xinli | eharney: Now I use 1) git pull 2) git checkout fix_branch 3) git rebase -i master, it should generate the build error in my local box, right? | 16:30 |
| xinli | eharney: I want to reproduce the same build error "invalid uuid" in my local environment | 16:31 |
| *** raunak has joined #openstack-cinder | 16:32 | |
| *** laughterwym has quit IRC | 16:33 | |
| *** itzdilip has quit IRC | 16:33 | |
| leeantho | e0ne, I have a short meeting now. I'll ping you when its over | 16:33 |
| e0ne | leeantho: great, thanks | 16:34 |
| ntpttr_ | scottda: if you're using horizon how do microversions work? You need to have that in your env on the horizon host or something? | 16:34 |
| scottda | Horizon has zero support for microversions. | 16:34 |
| e0ne | scottda: omg! | 16:35 |
| ntpttr_ | like for my patch that adds public/protected settings to upload to image, if the public and protected check boxes are added to horizon (which is in the works I think), they can't actually be used? | 16:35 |
| david-lyle | scottda: #truestory | 16:35 |
| *** merooney has joined #openstack-cinder | 16:36 | |
| *** raunak has quit IRC | 16:37 | |
| xinli | @smcginnis: I use git rebase -i master and it shows pick 080942a Make IBM drivers return snapshot object for cg ops | 16:38 |
| xinli | # Rebase 120f8bb..080942a onto 120f8bb | 16:38 |
| xinli | which command should I run? | 16:38 |
| scottda | david-lyle: Does that mean no microversion support for any service, i.e. manila, nova, ironic, anyone? | 16:38 |
| david-lyle | scottda: correct | 16:39 |
| david-lyle | the only version awareness comes via the clients | 16:39 |
| ankur-gupta-f1 | david-lyle: ntpttr_: well thats a bummer :( | 16:39 |
| scottda | david-lyle: Thanks. Any plans for this in Horizon? | 16:40 |
| david-lyle | no kidding | 16:40 |
| david-lyle | scottda: not as of yet, we're busy reinventing the wheel presently | 16:40 |
| david-lyle | but I'll probably take it one shortly | 16:41 |
| scottda | david-lyle: No problem. I very impressed you're paying enough attention to the channel to chime in :) | 16:41 |
| xinli | _alastor_: which command should I run after git rebase -i master, pick 080942a Make IBM drivers return snapshot object for cg ops # Rebase 120f8bb..080942a onto 120f8bb | 16:41 |
| david-lyle | I cheat and get pinged on "horizon" | 16:41 |
| ntpttr_ | david-lyle: clever! | 16:41 |
| david-lyle | mostly to head people off from wasting time on something in Horizon if I can fill in the blanks | 16:42 |
| *** tpsilva has joined #openstack-cinder | 16:42 | |
| *** akerr is now known as akerr_away | 16:43 | |
| *** akerr_away is now known as akerr | 16:43 | |
| *** raunak has joined #openstack-cinder | 16:44 | |
| *** e0ne has quit IRC | 16:46 | |
| *** itzdilip has joined #openstack-cinder | 16:47 | |
| *** e0ne has joined #openstack-cinder | 16:47 | |
| *** arecknag has quit IRC | 16:50 | |
| *** anna_zanna has joined #openstack-cinder | 16:51 | |
| *** markus_z has quit IRC | 16:51 | |
| *** _erick0zcr has joined #openstack-cinder | 16:53 | |
| *** jistr has quit IRC | 16:54 | |
| anna_zanna | Hi - I'm doing a time estimate for a cinder-driver update. Could someone tell me what sort of changes would be necessary to update a cinder driver that works with Juno so that it works with Mitaka? Have there been any major changes to the driver interface? | 16:54 |
| *** sdake has quit IRC | 16:55 | |
| *** plastikman has joined #openstack-cinder | 16:56 | |
| *** rdx_matt_ has quit IRC | 16:57 | |
| *** rdx_matt has joined #openstack-cinder | 16:57 | |
| *** itzdilip has quit IRC | 16:58 | |
| *** akshai has quit IRC | 17:00 | |
| smcginnis | diablo_rojo: Sorry, should have had that. | 17:00 |
| *** akapil has joined #openstack-cinder | 17:00 | |
| smcginnis | diablo_rojo: Couldn't find it easily and past time. | 17:01 |
| diablo_rojo | smcginnis: No worries | 17:01 |
| *** e0ne has quit IRC | 17:01 | |
| *** aimeeu has quit IRC | 17:01 | |
| diablo_rojo | smcginnis: Just thought it would be useful information to have :) | 17:01 |
| *** aimeeu has joined #openstack-cinder | 17:01 | |
| smcginnis | diablo_rojo: Yeah, it would have. | 17:01 |
| *** e0ne has joined #openstack-cinder | 17:02 | |
| diablo_rojo | smcginnis: Do you have it now? | 17:02 |
| *** akshai has joined #openstack-cinder | 17:02 | |
| *** rdx_matt has quit IRC | 17:02 | |
| smcginnis | Finally found it: https://etherpad.openstack.org/p/newton-cinder-midcycle | 17:02 |
| diablo_rojo | smcginnis: Awesome :) Thanks! | 17:02 |
| jungleboyj | smcginnis: Thanks. | 17:03 |
| *** darosale has joined #openstack-cinder | 17:03 | |
| jungleboyj | MSP to DEN looks to be about 450 to 500 right now. | 17:03 |
| *** merooney has quit IRC | 17:03 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Image virtual size doesn't fit to volume size https://review.openstack.org/311152 | 17:05 |
| smcginnis | jungleboyj: Wow, that's a little higher than normal I think. Or I'm still stuck in 10 year old pricing. | 17:05 |
| *** dustins has joined #openstack-cinder | 17:06 | |
| smcginnis | Not bad really. | 17:06 |
| *** raunak_ has joined #openstack-cinder | 17:06 | |
| *** lpetrut has joined #openstack-cinder | 17:07 | |
| jungleboyj | smcginnis: If I do Monday to Monday it is Cheaper. :-) | 17:07 |
| jungleboyj | 369 to 456 | 17:07 |
| smcginnis | For the sake of your companies budget, you better make it a long weekend. ;) | 17:07 |
| jungleboyj | :-) | 17:08 |
| *** suyash has joined #openstack-cinder | 17:08 | |
| openstackgerrit | Yosef Hoffman proposed openstack/cinder: Correct conditional/assignment in SolidFireDriver https://review.openstack.org/312641 | 17:08 |
| *** raunak has quit IRC | 17:09 | |
| *** raunak_ is now known as raunak | 17:09 | |
| eharney | xinli: yes, that should do it | 17:09 |
| openstackgerrit | Nate Potter proposed openstack/cinder-specs: Backend Usage Reporting https://review.openstack.org/301946 | 17:09 |
| *** bigdogstl has joined #openstack-cinder | 17:09 | |
| *** salv-orlando has joined #openstack-cinder | 17:12 | |
| *** ChubYann has joined #openstack-cinder | 17:13 | |
| *** bigdogstl has quit IRC | 17:15 | |
| openstackgerrit | Merged openstack/os-brick: os-brick refactor get_connector_properties https://review.openstack.org/275943 | 17:17 |
| *** merooney has joined #openstack-cinder | 17:18 | |
| *** diogogmt has quit IRC | 17:22 | |
| *** dave-mccowan has joined #openstack-cinder | 17:22 | |
| *** diablo_rojo has quit IRC | 17:23 | |
| *** diogogmt has joined #openstack-cinder | 17:24 | |
| *** wanghao has quit IRC | 17:26 | |
| *** porrua has quit IRC | 17:26 | |
| *** wanghao has joined #openstack-cinder | 17:27 | |
| patrickeast | hey all, just another ping that I'm going to try and get the image-volume cache enabled in devstack https://review.openstack.org/#/c/312305/ | 17:30 |
| scottda | smcginnis: Etherpad is updated with discount hotel room update https://etherpad.openstack.org/p/newton-cinder-midcycle | 17:30 |
| patrickeast | I mentioned it briefly in the last couple min of the meeting, but not sure how many saw | 17:31 |
| hemna | patrickeast, I was just looking at that | 17:31 |
| smcginnis | patrickeast, scottda: Thanks and thanks. :) | 17:31 |
| patrickeast | if anyone has issues with it, plz speak up on the review | 17:31 |
| scottda | smcginnis: And I've gotten rooms booked | 17:31 |
| hemna | I presume we have tested it with lvm and ceph ? | 17:31 |
| smcginnis | scottda: Oh really? They let you without a second mortgage? | 17:31 |
| hemna | (our reference drivers) | 17:31 |
| hemna | patrickeast, also does this work in a single backend setup? | 17:32 |
| hemna | I guess I can try it :) | 17:32 |
| smcginnis | scottda: What? $10 more a night. I'm outraged. :) | 17:32 |
| *** bhavik has joined #openstack-cinder | 17:32 | |
| eharney | patrickeast: so this is kind of a weird thing for ceph | 17:32 |
| eharney | patrickeast: because if you're using ceph-backed glance too, you don't really want to turn this on | 17:32 |
| scottda | smcginnis: Inflation is everywhere. You should see the house prices here... | 17:32 |
| smcginnis | scottda: Oh really? Eveyone's catching on it's a cool area? | 17:33 |
| ntpttr_ | scottda: feeling that same way in portland :/ | 17:33 |
| scottda | Yeah, you guys are welcome, but don't tell anyone else. | 17:33 |
| smcginnis | ntpttr_: Portland's been like that for a while it seems. | 17:33 |
| *** akshai has quit IRC | 17:34 | |
| ntpttr_ | smcginnis: seems to coincide a bit with the release of 'portlandia'... | 17:34 |
| smcginnis | ntpttr_: Hah, I suppose that did bring more attention. :) | 17:34 |
| hemna | put a bird on it. | 17:35 |
| smcginnis | :) | 17:35 |
| *** bigdogstl has joined #openstack-cinder | 17:35 | |
| ankur-gupta-f1 | smcginnis: portlandia is the worst. We do have some normal people here | 17:35 |
| ntpttr_ | ankur-gupta-f1: 'here'? thought you were a full blooded texan now ;) | 17:36 |
| ankur-gupta-f1 | ntpttr_: never | 17:36 |
| openstackgerrit | Michael Price proposed openstack/cinder: NetApp: E-Series driver using invalid host-types https://review.openstack.org/303598 | 17:37 |
| e0ne | hemna: hi. probably, you can help me | 17:37 |
| *** rlrossit has quit IRC | 17:38 | |
| *** nnex has quit IRC | 17:38 | |
| e0ne | hemna: I'm hitting with local attach feature and stucked with privsep-related code | 17:38 |
| hemna | sup | 17:39 |
| hemna | e0ne, have something I can test ? | 17:39 |
| e0ne | hemna: now | 17:39 |
| e0ne | s/now/no | 17:39 |
| hemna | heh ok | 17:40 |
| e0ne | as desided, we should be able to run something like 'sudo cinder local-attach <vol_id>' to attach volume to the local host | 17:40 |
| hemna | yah | 17:41 |
| e0ne | ^^ means that we have to use root/or sudo | 17:41 |
| e0ne | and nothing like rootwrap/privsep/etc | 17:41 |
| hemna | e0ne, https://review.openstack.org/#/c/312216/ | 17:41 |
| hemna | does that help? :) | 17:41 |
| *** igorn has joined #openstack-cinder | 17:42 | |
| e0ne | hm... | 17:42 |
| e0ne | I have to test | 17:42 |
| *** diablo_rojo has joined #openstack-cinder | 17:42 | |
| e0ne | because, I tried https://review.openstack.org/#/c/309201/ before and failed | 17:42 |
| hemna | wow | 17:43 |
| hemna | ok that's almost identical | 17:43 |
| hemna | hrmm | 17:43 |
| e0ne | patrickeast: ^^ | 17:43 |
| *** akshai has joined #openstack-cinder | 17:44 | |
| *** bigdogstl has quit IRC | 17:45 | |
| *** nnex has joined #openstack-cinder | 17:46 | |
| e0ne | hemna, patrickeast: I've kust tested both versions http://paste.openstack.org/show/496118/ | 17:46 |
| *** igornsa has joined #openstack-cinder | 17:46 | |
| *** igorn has quit IRC | 17:46 | |
| hemna | pastebin ? | 17:47 |
| e0ne | hemna: what do you mean? | 17:47 |
| hemna | the errors | 17:47 |
| e0ne | s/kust/just | 17:47 |
| openstackgerrit | Jay Mehta proposed openstack/cinder: Implement match-set type 3PAR FC VLUN creation https://review.openstack.org/309613 | 17:48 |
| e0ne | hemna: did you se my message above? with link to http://paste.openstack.org/show/496118/ | 17:48 |
| hemna | oh sorry | 17:48 |
| hemna | e0ne, you'll have to rebase now that my other refactor patch has landed fwiw | 17:48 |
| e0ne | no errors, but with patrickeast's patch I've got privsep log messages | 17:49 |
| hemna | are you specifically passing in execute ? | 17:49 |
| e0ne | I'm passing 'processutils.execute' | 17:51 |
| hemna | url ? | 17:51 |
| e0ne | https://review.openstack.org/#/c/263744/17/brick_cinderclient_ext/client.py - line #59 | 17:51 |
| *** merooney has quit IRC | 17:52 | |
| *** mriedem has quit IRC | 17:52 | |
| hemna | hrmm | 17:53 |
| hemna | ok let me fire up a new vm and play with it | 17:53 |
| hemna | I don't see anything obvious | 17:53 |
| *** mvk_ has quit IRC | 17:53 | |
| patrickeast | e0ne: oh funny, I didn't even see your change up for that | 17:54 |
| * patrickeast should have checked more thoroughly | 17:54 | |
| *** mriedem has joined #openstack-cinder | 17:54 | |
| e0ne | patrickeast: your patch doesn't work for me | 17:54 |
| patrickeast | hemna: e0ne: i'm happy to help push either one through, just let me know if you want me to drop mine | 17:54 |
| e0ne | patrickeast: but I could do mistake while testing | 17:55 |
| patrickeast | e0ne: is that paste the error? | 17:55 |
| patrickeast | e0ne: http://paste.openstack.org/show/496118/ ? | 17:56 |
| e0ne | patrickeast: http://paste.openstack.org/show/496118/ - no errors, but privsep logs are there | 17:56 |
| jbernard | patrickeast: wrt image caching, if the backend is capable of optimizing the copy, would caching enabled bypass that? | 17:56 |
| patrickeast | e0ne: oh, yea so maybe a difference in how things are defaulted | 17:56 |
| patrickeast | e0ne: unless you specify a custom execute in the code calling it, we still use privsep | 17:56 |
| e0ne | patrickeast: if I undestood everything right, https://github.com/openstack/os-brick/blob/master/os_brick/executor.py#L34 this line will change executor from putils.execute to privsep | 17:57 |
| patrickeast | jbernard: yep, it tries to do the driver.clone_image or whatever first | 17:57 |
| patrickeast | jbernard: and only if that fails it will try the cache, and then last up is downloading the image the old fashioned way | 17:57 |
| jbernard | patrickeast: ok, so we'd want to keep that diabled in the ceph gate job | 17:57 |
| patrickeast | jbernard: err wait, when i say yep | 17:57 |
| jbernard | patrickeast: ahh, i misunderstood | 17:58 |
| *** rdx_matt has joined #openstack-cinder | 17:58 | |
| patrickeast | jbernard: i mean it won't bypass it | 17:58 |
| jbernard | patrickeast: excellent! | 17:58 |
| patrickeast | jbernard: lemmie try again, it does the smart thing | 17:58 |
| e0ne | patrickeast: what options of execute do I have? putils.execute, priv_rootwrap.execute, what else? | 17:58 |
| e0ne | patrickeast: I don't want to use subproces.execute directly in the openstack ecosystem | 17:58 |
| patrickeast | e0ne: so i have a little tool that wraps subprocess.call() and does some logging to a specific place | 17:59 |
| patrickeast | e0ne: we are maybe solving two different use-cases in similar ways | 17:59 |
| patrickeast | e0ne: i want to use os-brick in a non-openstack environment | 17:59 |
| e0ne | patrickeast: got it! your case is valid | 17:59 |
| e0ne | patrickeast: but my one is more general | 17:59 |
| *** merooney has joined #openstack-cinder | 18:00 | |
| e0ne | patrickeast: because I'm try to allow putils.execute too | 18:00 |
| patrickeast | e0ne: yea that works for me | 18:00 |
| hemna | e0ne, so I just pulled both patches | 18:00 |
| hemna | and used my diediedie/brick_initiator.py script | 18:00 |
| hemna | and I see the same thing | 18:00 |
| hemna | w/ patrickeast's patch I get privsep starting up | 18:01 |
| hemna | even though I'm passing in execute=putils.execute | 18:01 |
| *** salv-orlando has quit IRC | 18:01 | |
| patrickeast | yea | 18:01 |
| patrickeast | there is code that looks for if execute == putils.execute (or whatever) | 18:01 |
| patrickeast | and swaps it for privsep | 18:01 |
| *** bhavik has quit IRC | 18:01 | |
| hemna | line 89 maybe | 18:02 |
| patrickeast | i didn't muck with that | 18:02 |
| patrickeast | but i think the changes e0ne did are better, which do let you still use it | 18:02 |
| e0ne | patrickeast: thanks | 18:02 |
| *** rdx_matt has quit IRC | 18:02 | |
| e0ne | patrickeast: but my changes breaks current cinder and nova | 18:02 |
| hemna | https://github.com/WaltHP/diediedie | 18:02 |
| e0ne | hemna: line 89 where? | 18:02 |
| hemna | that's what I use for testing out os-brick stuffs | 18:02 |
| hemna | https://github.com/WaltHP/diediedie/blob/master/diediedie/brick_initiator.py | 18:02 |
| patrickeast | https://github.com/openstack/os-brick/blob/master/os_brick/executor.py#L34 | 18:02 |
| hemna | patrickeast, oh yah I think that's it. | 18:03 |
| patrickeast | ^ that is where you are seeing the difference in behavior, if you pass in the putils.execute we override it | 18:03 |
| hemna | it forces priv_rootwrap | 18:03 |
| patrickeast | yah | 18:03 |
| hemna | if execute is putils | 18:03 |
| e0ne | yep | 18:03 |
| e0ne | so, we can/must fix nova and cinder to not pass putils.execute to the os-brick | 18:04 |
| e0ne | does it ^^ sounds reasonable? | 18:04 |
| hemna | the default is None | 18:04 |
| e0ne | none==privsep | 18:04 |
| hemna | cinder is calling the factory with a default of putils.execute | 18:05 |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/utils.py#L537 | 18:06 |
| *** akapil has quit IRC | 18:07 | |
| hemna | nova doesn't pass anything | 18:07 |
| hemna | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L1229 | 18:08 |
| e0ne | hemna, patrickeast: if we want to change https://github.com/openstack/os-brick/blob/master/os_brick/executor.py#L29 to not ignore putils.execute, we have to release os-brick now, fix nova and cinder, remove make changes like https://review.openstack.org/#/c/309201/1/os_brick/executor.py and release os-brick again | 18:09 |
| hemna | e0ne, well we can't change the API | 18:09 |
| hemna | because an os-brick update will affect < Newton | 18:09 |
| *** ChubYann has quit IRC | 18:09 | |
| hemna | Nova doesn't pass a value for execute | 18:10 |
| hemna | not even when calling the factory in the libvirt volume drivers | 18:10 |
| hemna | so it relies on brick to do the 'right thing' | 18:10 |
| hemna | which now is privsep | 18:10 |
| hemna | so we have to default to that | 18:10 |
| hemna | but if you explicitly call it with a value, then brick should honor that | 18:11 |
| hemna | and not override a value with privsep | 18:11 |
| e0ne | hemna: that is what I mean | 18:11 |
| e0ne | hemna: use privsep by default | 18:11 |
| e0ne | hemna: and allow to pass putils.execute | 18:11 |
| hemna | yah i think so | 18:12 |
| hemna | if you set execute=N | 18:12 |
| hemna | then N should be used. | 18:12 |
| e0ne | hemna: exactly | 18:12 |
| hemna | else default to privsep | 18:12 |
| patrickeast | hemna: e0ne: sry, stepped away for a min, yea i agree it needs to honor what we set execute to | 18:12 |
| e0ne | hemna: now, I don't understand yours "well we can't change the API" | 18:12 |
| hemna | meaning we can't change the default | 18:13 |
| *** akapil has joined #openstack-cinder | 18:13 | |
| patrickeast | heh, add a new parameter, execute_2 | 18:13 |
| hemna | lol | 18:13 |
| eharney | hemna, e0ne: FYI, i started hacking on Cinder privsep a bit (unrelated to most of this) -- it's working well so far in my testing. https://review.openstack.org/#/c/312498/ | 18:13 |
| hemna | so e0ne, we need your patch to pass jenkins :) | 18:13 |
| e0ne | hemna: yes. I have to fix few places in Cinder | 18:14 |
| e0ne | hemna: will try to do it tonight | 18:14 |
| hemna | e0ne, I think we should just change cinder to use the default, just like nova | 18:14 |
| hemna | eharney, ok | 18:14 |
| e0ne | hemna: agree | 18:14 |
| hemna | e0ne, otherwise cinder doesn't get the benefit of privsep as it is today. | 18:15 |
| e0ne | hemna: working on it right now | 18:15 |
| hemna | eharney, so migrating some cinder codes to use privsep | 18:15 |
| hemna | looks like | 18:15 |
| e0ne | hemna: while I'm fixing cinder, could you please +2 on https://review.openstack.org/#/c/265811/? :) | 18:15 |
| eharney | hemna: yes | 18:15 |
| hemna | eharney, sweet | 18:16 |
| hemna | eharney, so fwiw, I had a chat with gus about privsep | 18:16 |
| hemna | and he thinks we need the priveleged side to be a higher level than | 18:16 |
| hemna | instead of an execute | 18:16 |
| eharney | right, that's what i'm doing here, getting rid of executes | 18:16 |
| *** akapil has quit IRC | 18:17 | |
| hemna | more of a workflow has this privelege, not a particular call out to a single command. | 18:17 |
| eharney | my intent is to narrow down the LIO code to something that only really have privileges to manipulate /sys/ etc, and not everything else | 18:17 |
| eharney | which should be all it needs | 18:17 |
| hemna | where now we give privs to a single shell command | 18:17 |
| hemna | and anyone that uses that command has privs | 18:18 |
| hemna | which is a wide door | 18:18 |
| hemna | it's going to take some refactoring and rethinking how to do this differently | 18:18 |
| eharney | well, the LIO case is pretty well-defined, so i'm going to hack on it a bit and figure out how to get it done right | 18:19 |
| hemna | cool | 18:19 |
| eharney | because it was a python lib that i wrote a CLI for just to call via rootwrap | 18:19 |
| hemna | I'm sure gus would help review the patch and help | 18:19 |
| hemna | os-brick was the guinea pig for privsep | 18:19 |
| openstackgerrit | Merged openstack/cinder: Removed extra line of code https://review.openstack.org/310499 | 18:21 |
| *** ChubYann has joined #openstack-cinder | 18:21 | |
| *** sgotliv has quit IRC | 18:22 | |
| *** bardia has quit IRC | 18:22 | |
| cFouts | smcginnis: https://review.openstack.org/#/c/309202/ could use workflow if you don't mind. The e-series failures for netapp CI don't impact this patch :/ | 18:23 |
| *** angela-s has joined #openstack-cinder | 18:25 | |
| smcginnis | cFouts: OK, I'll take a look. | 18:25 |
| smcginnis | cFouts: So how does that fit in with the generic volume cache? | 18:25 |
| cFouts | I looked at the spec and ^^ conversation and I think we can refactor our copy offload feature to make use of generic image cache | 18:26 |
| cFouts | we'd still have a netapp config option to specify the binary | 18:26 |
| cFouts | smcginnis: we were looking for an excuse to clean up some of our nfs driver code anyway :) | 18:27 |
| smcginnis | cFouts: So... should we even do this one, or just wait and make sure you are working well with the generic image cache? | 18:27 |
| adrianofr | Guys, what is this backup policy for CIs you mentioned in the meeting? | 18:27 |
| smcginnis | cFouts: I just don't want to cause any confusion by adding a second way to do something. | 18:28 |
| smcginnis | adrianofr: For backup drivers to have CIs running to validate their drivers work. | 18:28 |
| cFouts | smcginnis: ah, this way already exists. this is a bug fix and a unit test refactor | 18:28 |
| *** akshai has quit IRC | 18:28 | |
| *** diogogmt has quit IRC | 18:28 | |
| smcginnis | cFouts: OK, cool. I'll review in a bit then. | 18:29 |
| cFouts | smcginnis: thanks! | 18:29 |
| smcginnis | np | 18:29 |
| adrianofr | smcginnis: So, this apply only for backup drivers | 18:29 |
| *** merooney has quit IRC | 18:29 | |
| *** martyturner has quit IRC | 18:29 | |
| *** nnex has quit IRC | 18:29 | |
| *** akapil has joined #openstack-cinder | 18:29 | |
| adrianofr | smcginnis: got it. Tks :) | 18:30 |
| *** akapil has quit IRC | 18:30 | |
| smcginnis | adrianofr: Correct. We already require CI for volume and fczm drivers, so this is just an extension of that policy to include backup drivers as well. | 18:30 |
| *** akshai has joined #openstack-cinder | 18:31 | |
| *** martyturner has joined #openstack-cinder | 18:31 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Pass default executor to os-brick https://review.openstack.org/312681 | 18:32 |
| *** bigdogstl has joined #openstack-cinder | 18:32 | |
| *** ivase has quit IRC | 18:32 | |
| e0ne | hemna, patrickeast: ^^ | 18:32 |
| *** merooney has joined #openstack-cinder | 18:32 | |
| adrianofr | smcginnis: Nice! And also to enforce them to listen to os-brick | 18:32 |
| hemna | e0ne, can't we just get away with removing the execute param completely ? | 18:33 |
| e0ne | I think we can | 18:34 |
| hemna | just have to look through all the uses of that function | 18:34 |
| hemna | and make sure noone is passing it in | 18:34 |
| hemna | if that's the case, then just remove it. | 18:35 |
| *** akapil has joined #openstack-cinder | 18:35 | |
| *** jgregor has quit IRC | 18:35 | |
| jgriffith | hemna: so on this: https://review.openstack.org/#/c/284867/2/cinder/volume/manager.py | 18:35 |
| jgriffith | hemna: using intiialize_connection for a refresh is wrong | 18:36 |
| hemna | jgriffith, correct | 18:36 |
| hemna | but that's exactly what nova is doing today. | 18:36 |
| hemna | in live migration | 18:36 |
| jgriffith | hemna: correct you agree with me that it's wrong? | 18:36 |
| hemna | yes | 18:36 |
| jgriffith | Yeah, they do it in a couple places | 18:36 |
| jgriffith | It's appauling | 18:36 |
| hemna | I talked with the nova guys on Friday about fixing that in Nova. | 18:36 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Pass default executor to os-brick https://review.openstack.org/312681 | 18:37 |
| jgriffith | hemna: so are you working on this then or am I? :) | 18:37 |
| hemna | all we need is a temp storage to keep the source n-cpu connection_properties around | 18:37 |
| jgriffith | it seems there was a lot of discussion on Friday when I wasn't there | 18:37 |
| hemna | I was going to try and hack something in nova | 18:37 |
| hemna | to eliminate it | 18:37 |
| hemna | yah there was | 18:37 |
| hemna | the nova guys pulled us into their meetnig | 18:37 |
| hemna | and we argued for a while :P | 18:37 |
| openstackgerrit | Clinton Knight proposed openstack/cinder: NetApp: Managing cDOT LUN by UUID fails https://review.openstack.org/312684 | 18:38 |
| *** timcl has quit IRC | 18:38 | |
| jgriffith | so I'm still confused. Some people made comments on my etherpad that "cinder folks" said the virtual volumes would work | 18:38 |
| hemna | everyone agreed that we should remove the initialize_connection call on disconnect | 18:38 |
| jgriffith | I would really like to just get this done once and for all | 18:38 |
| *** liverpooler has joined #openstack-cinder | 18:38 | |
| *** timcl has joined #openstack-cinder | 18:38 | |
| jgriffith | hemna: wait... what? | 18:38 |
| hemna | but nobody could figure out where to store the connection_properties | 18:38 |
| jgriffith | initialize on disconnect? | 18:38 |
| hemna | yes | 18:38 |
| jgriffith | you mean on the swap? | 18:38 |
| hemna | on disconnect | 18:38 |
| jgriffith | ummm | 18:39 |
| hemna | let me find it for you....just a second. | 18:39 |
| *** dustins has quit IRC | 18:39 | |
| *** bigdogstl has quit IRC | 18:39 | |
| hemna | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L6831-L6833 | 18:40 |
| hemna | that guy | 18:40 |
| hemna | that's what I want to eliminate | 18:40 |
| hemna | the problem is at that time the bdm has the connection_info from the destination n-cpu host. | 18:41 |
| hemna | which is wrong for the source n-cpu host. | 18:41 |
| hemna | or can be wrong, depending on the backend. | 18:41 |
| hemna | iqn, lun_id can be different | 18:41 |
| *** sdake has joined #openstack-cinder | 18:42 | |
| hemna | so, the solution is to save the bdm's initial connection_info at the start of live migration. | 18:42 |
| hemna | so it's around at disconnect time. | 18:42 |
| hemna | that will remove the initialize_connection call at disconnect. | 18:43 |
| hemna | https://drive.google.com/a/hemna.com/file/d/0B1Kp6K43HLHydU1wWFVIN29tc2s/view | 18:43 |
| hemna | shows the overall call chain | 18:43 |
| *** bigdogstl has joined #openstack-cinder | 18:43 | |
| hemna | initialize_connection is called twice during the live migration process | 18:43 |
| hemna | first to create the export on the destination n-cpu host, which then blows away the bdm's copy of the source n-cpu host. | 18:44 |
| hemna | which requires the initialize_connection call at post_live_migration time. | 18:44 |
| *** sdake has quit IRC | 18:44 | |
| *** bardia has joined #openstack-cinder | 18:45 | |
| hemna | the other major complaint about multi-attach from the Nova team is the volume state mgmt | 18:45 |
| hemna | I think I have an idea around that (re: virtual volume thingy discussion) | 18:45 |
| *** sdake has joined #openstack-cinder | 18:46 | |
| hemna | but I needed to do some more research to verify it | 18:46 |
| hemna | it might be easier just to have a call to talk about it. | 18:46 |
| *** vmtrooper has joined #openstack-cinder | 18:48 | |
| *** vgridnev has joined #openstack-cinder | 18:48 | |
| *** bigdogstl has quit IRC | 18:48 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Pass default executor to os-brick https://review.openstack.org/312681 | 18:49 |
| *** salv-orlando has joined #openstack-cinder | 18:51 | |
| *** vmtrooper has quit IRC | 18:51 | |
| *** martyturner has quit IRC | 18:52 | |
| *** rlrossit has joined #openstack-cinder | 18:54 | |
| *** zhonghua has joined #openstack-cinder | 18:55 | |
| *** jungleboyj has quit IRC | 18:55 | |
| *** salv-orlando has quit IRC | 18:56 | |
| *** salv-orlando has joined #openstack-cinder | 18:56 | |
| *** zhonghua-lee has quit IRC | 18:58 | |
| *** rdx_matt has joined #openstack-cinder | 18:58 | |
| *** vmtrooper has joined #openstack-cinder | 18:59 | |
| *** mvk_ has joined #openstack-cinder | 19:00 | |
| *** merooney has quit IRC | 19:01 | |
| *** jgregor has joined #openstack-cinder | 19:01 | |
| *** martyturner has joined #openstack-cinder | 19:02 | |
| *** rdx_matt has quit IRC | 19:03 | |
| *** jgregor has quit IRC | 19:03 | |
| *** vmtrooper has quit IRC | 19:03 | |
| openstackgerrit | Xinli Guan proposed openstack/cinder: Make IBM drivers return snapshot object for cg ops https://review.openstack.org/268283 | 19:04 |
| *** jgregor has joined #openstack-cinder | 19:06 | |
| *** vmtrooper has joined #openstack-cinder | 19:06 | |
| *** merooney has joined #openstack-cinder | 19:07 | |
| *** salv-orlando has quit IRC | 19:07 | |
| *** jgregor has left #openstack-cinder | 19:08 | |
| *** baumann1 has joined #openstack-cinder | 19:08 | |
| *** salv-orlando has joined #openstack-cinder | 19:08 | |
| *** baumann2 has joined #openstack-cinder | 19:09 | |
| *** baumann3 has joined #openstack-cinder | 19:10 | |
| *** vmtrooper has quit IRC | 19:10 | |
| *** baumann has quit IRC | 19:10 | |
| *** baumann1 has quit IRC | 19:13 | |
| *** salv-orlando has quit IRC | 19:13 | |
| *** rmcall has joined #openstack-cinder | 19:13 | |
| *** baumann2 has quit IRC | 19:14 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/os-brick: Add support for processutils.execute https://review.openstack.org/309201 | 19:15 |
| *** raunak has quit IRC | 19:15 | |
| e0ne | hemna, patrickeast: let's wait for CIs ^^ | 19:16 |
| patrickeast | e0ne: sounds good, thanks for getting that stuff sorted out | 19:16 |
| e0ne | patrickeast: np. I just want to make cinder a bit better :) | 19:17 |
| e0ne | patrickeast: and to sleep too :) | 19:17 |
| *** dustins has joined #openstack-cinder | 19:18 | |
| * e0ne bb in ~12 hours | 19:19 | |
| *** e0ne has quit IRC | 19:23 | |
| *** sdake has quit IRC | 19:25 | |
| *** sdake has joined #openstack-cinder | 19:26 | |
| *** bigdogstl has joined #openstack-cinder | 19:28 | |
| *** salv-orlando has joined #openstack-cinder | 19:30 | |
| *** apoorvad has quit IRC | 19:31 | |
| *** bigdogstl has quit IRC | 19:33 | |
| xinli | Does any one in community help this case? After I git review my PatchSet 2 for patch https://review.openstack.org/268283, the error message saying dsvm-tempest-test http://packages.cloudfounders.com/ci_logs/83/268283/2/check/dsvm-tempest-test/c4521bf : NOT_REGISTERED | 19:35 |
| *** porrua has joined #openstack-cinder | 19:35 | |
| eharney | smcginnis: have you looked in-depth at how to extend tempest clients for our in-tree tests? i think i need to make a new client in our tree... | 19:36 |
| patrickeast | xinli: you can just ignore that, its one of the 3rd party ci systems and they sometimes fail like that | 19:36 |
| xinli | patrickeast: but the build failed Build failed. For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures. For 3rd party ci contact info: https://wiki.openstack.org/wiki/ThirdPartySystems/ | 19:37 |
| *** timcl has quit IRC | 19:37 | |
| patrickeast | eharney: i did that for a in-house tempest plugin i made.... its kind of a huge pain | 19:37 |
| eharney | patrickeast: yeah, looks like it | 19:37 |
| *** anna_zanna has left #openstack-cinder | 19:38 | |
| patrickeast | xinli: yea but only failed for openvstorage, the NOT_REGISTERED error is like a internal server error with the CI system, nothing to do with your patch | 19:38 |
| xinli | eharney: this is the one for invaldi UUID patch, the test passed in my loacl environment | 19:38 |
| xinli | patrickeast: waht can I do now? just waiting for the queue? | 19:39 |
| patrickeast | xinli: you are waiting for the ci system named 'jenkins' to report its results | 19:39 |
| patrickeast | xinli: for example, https://review.openstack.org/#/c/312355/ where lots of ci systems have already weighed in | 19:40 |
| *** mkrcmari__ has joined #openstack-cinder | 19:40 | |
| patrickeast | xinli: the one that really matters, especially for unit test changes is 'jenkins', and then the ibm specific CI systems specific to that patch | 19:40 |
| eharney | patrickeast: so should we be aiming to have some kind of cinder client in tempest-lib? | 19:42 |
| patrickeast | eharney: there are some already in there for the volume operations that are supported, unfortunately it seems that using the actual python-cinderclient is frowned upon | 19:42 |
| *** timcl has joined #openstack-cinder | 19:43 | |
| patrickeast | eharney: so like for mine i made something and just based it on the existing ones http://paste.openstack.org/show/496141/ | 19:43 |
| patrickeast | eharney: taking some liberties to subclass from tempest internals | 19:43 |
| eharney | patrickeast: ohh, it's hiding under compute... ok | 19:43 |
| patrickeast | eharney: the tricky part is all the weird auth stuff | 19:43 |
| eharney | patrickeast: cool. i think i can just add the methods i need to this compute/volumes_client.py in tempest-lib and be fine | 19:43 |
| patrickeast | eharney: sweet, that makes it much easier | 19:44 |
| *** mvk_ has quit IRC | 19:44 | |
| eharney | patrickeast: yeah, i went through that when i first wrote this patch for tempest itself, it uses user and admin creds | 19:44 |
| patrickeast | eharney: yea took me a while of staring at logs to figure out that the different clients would have different project ids and everything | 19:45 |
| patrickeast | eharney: and that mixing and matching to get the api's you need with clients doesn't really work :( | 19:45 |
| *** gretchen_hall has joined #openstack-cinder | 19:46 | |
| *** e0ne has joined #openstack-cinder | 19:46 | |
| eharney | patrickeast: my assumption was that the hard part was already done since i have a working tempest test... i thought i just needed to copy it to the in-tree place, but, not quite :) | 19:46 |
| patrickeast | eharney: if only it was so easy | 19:46 |
| *** timcl has quit IRC | 19:46 | |
| *** openstackgerrit has quit IRC | 19:48 | |
| savihou | Hi Everyone, what is the polite way to nudge for a code review ;) ? | 19:48 |
| *** e0ne has quit IRC | 19:48 | |
| *** openstackgerrit has joined #openstack-cinder | 19:49 | |
| savihou | Just kidding | 19:49 |
| patrickeast | most of us accept bribes consisting of baked desserts and/or beer | 19:49 |
| savihou | was wondering if there's a list of pending for review list ... | 19:49 |
| jgriffith | savihou: yes there is | 19:50 |
| jgriffith | savihou: https://review.openstack.org/#/q/status:+open+project:+openstack/cinder | 19:50 |
| savihou | Bribe is my favorite expence :) | 19:50 |
| kmartin | savihou, it helps if you review some patches yourself and then ask someone in here with a link to the review | 19:50 |
| jgriffith | patrickeast: if you do both a desert and beer you're sure to get to the head of the list :) | 19:50 |
| kmartin | or what patrickeast said ^ | 19:51 |
| savihou | got it .. | 19:51 |
| savihou | Thanks! | 19:52 |
| savihou | Last question - promise , how can I verify that my status is open ? | 19:53 |
| savihou | https://review.openstack.org/#/c/263026/ | 19:54 |
| patrickeast | savihou: not sure if there is a status field or something on the review page, but on the list view you can filter by it, and if you look at review.openstack.org and click on "My"->"Changes" you should see it listed | 19:54 |
| patrickeast | savihou: that ones open | 19:55 |
| savihou | patrickeast: I can see it under my changes, but it is not filtered as "Open" | 19:55 |
| *** akshai has quit IRC | 19:55 | |
| patrickeast | savihou: iirc the "outgoing reviews" will only show open ones | 19:56 |
| *** akapil has quit IRC | 19:56 | |
| savihou | patrickeast: OK, ill check | 19:58 |
| *** martyturner has quit IRC | 19:59 | |
| *** daneyon has joined #openstack-cinder | 19:59 | |
| *** rdx_matt has joined #openstack-cinder | 19:59 | |
| savihou | patrickeast: it seems to be with no status , is that normal ? | 20:00 |
| patrickeast | savihou: yea its fine, as long as it isn't abandoned or something its ok | 20:00 |
| *** salv-orlando has quit IRC | 20:00 | |
| savihou | patrickeast: OK, so im leaving it as is - and starting making some sweets :) | 20:01 |
| *** martyturner has joined #openstack-cinder | 20:01 | |
| savihou | patrickeast: Thanks! | 20:01 |
| *** merooney has quit IRC | 20:02 | |
| *** e0ne has joined #openstack-cinder | 20:02 | |
| *** Yogi1 has quit IRC | 20:03 | |
| *** rdx_matt has quit IRC | 20:04 | |
| *** akerr has quit IRC | 20:04 | |
| *** sdague has quit IRC | 20:07 | |
| *** e0ne has quit IRC | 20:07 | |
| *** gouthamr has quit IRC | 20:09 | |
| *** harlowja has quit IRC | 20:09 | |
| *** gouthamr has joined #openstack-cinder | 20:09 | |
| *** akshai has joined #openstack-cinder | 20:10 | |
| *** e0ne has joined #openstack-cinder | 20:10 | |
| *** salv-orlando has joined #openstack-cinder | 20:11 | |
| *** akshai has quit IRC | 20:12 | |
| *** cdelatte has quit IRC | 20:12 | |
| e0ne | hemna, patrickeast, leeantho: thanks for the help for the privsep-related stuff | 20:14 |
| *** akshai has joined #openstack-cinder | 20:14 | |
| *** sdague has joined #openstack-cinder | 20:14 | |
| *** apoorvad has joined #openstack-cinder | 20:15 | |
| *** mvk_ has joined #openstack-cinder | 20:16 | |
| _erick0zcr | Hy folks, do you know why im getting [SSL: UNKNOWN_PROTOCOL] when listing volumes, I'm currently using keystone and Cinder with https endpoints. Keystone works well but I'm receiving an error code 400, message bad request sintax. | 20:16 |
| _erick0zcr | I made the certificates properly and set the ssl_* variables in cinder.conf, still nothing. The same happens with swift | 20:17 |
| *** ankur-gupta-f1 has left #openstack-cinder | 20:17 | |
| *** mvk has joined #openstack-cinder | 20:18 | |
| *** mkrcmari__ has quit IRC | 20:19 | |
| *** raunak has joined #openstack-cinder | 20:21 | |
| *** mvk_ has quit IRC | 20:21 | |
| *** mkrcmari__ has joined #openstack-cinder | 20:22 | |
| *** Lee1092 has quit IRC | 20:22 | |
| *** bigdogstl has joined #openstack-cinder | 20:22 | |
| *** daneyon_ has joined #openstack-cinder | 20:23 | |
| *** mvk has quit IRC | 20:25 | |
| *** daneyon has quit IRC | 20:26 | |
| *** bigdogstl has quit IRC | 20:27 | |
| smcginnis | _erick0zcr: I seem to remember a bug with SSL there. | 20:27 |
| *** baumann has joined #openstack-cinder | 20:30 | |
| *** cdelatte has joined #openstack-cinder | 20:31 | |
| *** baumann has quit IRC | 20:31 | |
| *** baumann has joined #openstack-cinder | 20:31 | |
| *** baumann1 has joined #openstack-cinder | 20:32 | |
| *** baumann3 has quit IRC | 20:32 | |
| *** dustins has quit IRC | 20:34 | |
| *** salv-orlando has quit IRC | 20:35 | |
| *** crose has quit IRC | 20:35 | |
| hemna | mriedem, ping | 20:35 |
| _erick0zcr | hmm, let me dig into, I think is the thing related to urllib3 or something | 20:36 |
| *** baumann has quit IRC | 20:36 | |
| hemna | jgriffith, so it seems that Nova calls initialize_connection at other odd times as well from what I can see | 20:36 |
| hemna | and I don't think it's associated with attaching a volume, but purely to ensure that the bdm's connection_info is 'up to date' | 20:37 |
| hemna | for example, https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2377 | 20:37 |
| mriedem | hemna: pong | 20:37 |
| *** e0ne has quit IRC | 20:37 | |
| hemna | mriedem, ^ | 20:37 |
| mriedem | hemna: that's correct | 20:38 |
| *** bigdogstl has joined #openstack-cinder | 20:38 | |
| hemna | mriedem, I'm trying to understand the reason why Nova calls refresh_connection_info() which turns into a call to cinder's initialize_connection | 20:38 |
| mriedem | to refresh the connection info? | 20:38 |
| mriedem | :) | 20:38 |
| hemna | well | 20:38 |
| hemna | calling initialize_connection in cinder means backends will try and export a volume | 20:38 |
| hemna | as well as collect the connection_info (target information) | 20:39 |
| hemna | which isn't good | 20:39 |
| mriedem | i guess you'd have to look at why it was added in each case | 20:39 |
| mriedem | https://github.com/openstack/nova/commit/329b594436aad814e43740ea26841342a8772aff | 20:39 |
| hemna | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2377 | 20:39 |
| hemna | I'm not sure I understand the reasoning for calling it for nova snapshoting cinder volumes | 20:39 |
| * hemna isn't smark | 20:39 | |
| hemna | smart either | 20:40 |
| hemna | heh | 20:40 |
| hemna | ew | 20:40 |
| mriedem | i agree that a get and create isn't good here | 20:41 |
| mriedem | but it's all nova has | 20:41 |
| hemna | hrmm | 20:41 |
| *** mvk has joined #openstack-cinder | 20:41 | |
| hemna | so, I'm trying to hack my way out of the call to initialize_connection during disconnect at live migration | 20:41 |
| hemna | just to see if I can put something up for folks to argue over | 20:41 |
| hemna | and I'm stuffing the original connection_info into the same bdm in a tmp key name | 20:42 |
| hemna | it sucks | 20:42 |
| *** gouthamr has quit IRC | 20:42 | |
| hemna | but I noticed other areas of nova are calling refresh_connection_info | 20:42 |
| hemna | this worries me on lots of levels | 20:42 |
| hemna | night sweats | 20:42 |
| hemna | panics | 20:42 |
| hemna | ,.... | 20:43 |
| hemna | :P | 20:43 |
| *** bigdogstl has quit IRC | 20:43 | |
| *** gouthamr has joined #openstack-cinder | 20:43 | |
| *** gouthamr has quit IRC | 20:43 | |
| mriedem | there are drugs for that sort of thing | 20:44 |
| *** mkrcmari__ has quit IRC | 20:44 | |
| *** mvk_ has joined #openstack-cinder | 20:44 | |
| *** salv-orlando has joined #openstack-cinder | 20:44 | |
| smcginnis | Having a separate query_connection_info would at least be semantically more accurate than calling initialize repeatedly. | 20:45 |
| hemna | smcginnis, yah I think so | 20:45 |
| hemna | and it seems nova needs it | 20:46 |
| mriedem | sure. also note that change was made specifically for glusterfs | 20:46 |
| hemna | but is calling initialize_connection because that's all Cinder has | 20:46 |
| mriedem | i'm not sure what is different on the backend in cinder between glusterfs and say lvm in this case | 20:46 |
| hemna | mriedem, yah I just started to look at the glusterfs driver | 20:46 |
| *** mkrcmari__ has joined #openstack-cinder | 20:47 | |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/glusterfs.py#L286 | 20:47 |
| hemna | that's probably the reason | 20:47 |
| *** mvk has quit IRC | 20:48 | |
| hemna | I'm guessing that once you do a snap | 20:48 |
| *** esker has quit IRC | 20:48 | |
| hemna | the active path changes | 20:48 |
| *** cknight has quit IRC | 20:48 | |
| *** akapil has joined #openstack-cinder | 20:49 | |
| *** daneyon_ has quit IRC | 20:49 | |
| *** mvk_ has quit IRC | 20:50 | |
| *** mvk has joined #openstack-cinder | 20:51 | |
| *** crose has joined #openstack-cinder | 20:53 | |
| *** mkrcmari__ has quit IRC | 20:53 | |
| *** mvk_ has joined #openstack-cinder | 20:53 | |
| *** mkrcmari__ has joined #openstack-cinder | 20:56 | |
| *** mvk has quit IRC | 20:56 | |
| *** julim has quit IRC | 20:59 | |
| *** mvk_ has quit IRC | 20:59 | |
| *** mvk_ has joined #openstack-cinder | 21:00 | |
| *** rdx_matt has joined #openstack-cinder | 21:00 | |
| *** julim has joined #openstack-cinder | 21:01 | |
| hemna | mriedem, something like this hack. https://gist.github.com/WaltHP/e54ed8e32a8b96bea44396643837bdc2 | 21:01 |
| hemna | it's all kinds of gross, but I don't know where else to save that source n-cpu node's connection_info | 21:02 |
| *** xinli has quit IRC | 21:03 | |
| mriedem | yeah, i've thought about the same before too | 21:03 |
| mriedem | we do things like that with flavors during migrate/resize | 21:03 |
| mriedem | we have old_flavor, new_flavor and current flavor | 21:03 |
| *** mkrcmari__ has quit IRC | 21:04 | |
| *** gretchen_hall has left #openstack-cinder | 21:04 | |
| *** rdx_matt has quit IRC | 21:06 | |
| *** baumann1 has left #openstack-cinder | 21:07 | |
| *** kfarr has quit IRC | 21:08 | |
| *** harlowja has joined #openstack-cinder | 21:10 | |
| *** haplo37 has quit IRC | 21:12 | |
| *** vmtrooper has joined #openstack-cinder | 21:14 | |
| *** rlrossit has quit IRC | 21:19 | |
| *** porrua has quit IRC | 21:22 | |
| *** crose has quit IRC | 21:23 | |
| *** sgotliv has joined #openstack-cinder | 21:23 | |
| *** daneyon has joined #openstack-cinder | 21:24 | |
| *** sgotliv has quit IRC | 21:27 | |
| *** sgotliv has joined #openstack-cinder | 21:31 | |
| *** eharney has quit IRC | 21:32 | |
| *** daneyon has quit IRC | 21:34 | |
| *** bigdogstl has joined #openstack-cinder | 21:34 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Support for paraVirtual image adapter type https://review.openstack.org/312761 | 21:38 |
| *** bigdogstl has quit IRC | 21:39 | |
| *** vgridnev has quit IRC | 21:40 | |
| *** chlong has quit IRC | 21:45 | |
| *** rmcall has quit IRC | 21:47 | |
| *** mriedem has quit IRC | 21:50 | |
| *** _erick0zcr has quit IRC | 21:50 | |
| *** sdague has quit IRC | 21:51 | |
| *** akapil has quit IRC | 21:53 | |
| diablo_rojo | Does anyone have the link to the CI triggers handy? | 21:53 |
| *** jwcroppe has quit IRC | 21:59 | |
| *** rdx_matt has joined #openstack-cinder | 22:03 | |
| angela-s | how do you kill off a volume(s) stuck in detaching state? Easily...? Development not going well today... | 22:05 |
| *** chlong has joined #openstack-cinder | 22:06 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/310827 | 22:06 |
| *** julim has quit IRC | 22:07 | |
| hemna | 'kill off' ? | 22:07 |
| hemna | angela-s, something poop out on nova side ? | 22:07 |
| angela-s | um, anger mgmt issues... | 22:07 |
| hemna | angela-s, you can always reset the state to available and then delete the vol. | 22:07 |
| *** rdx_matt has quit IRC | 22:07 | |
| hemna | still sadness there. | 22:07 |
| angela-s | no, my problem i think | 22:07 |
| angela-s | ok, i'll try that. | 22:08 |
| angela-s | just one of those days, i guess. nothing seems to want to work the way I want it to, argh | 22:09 |
| *** darosale has quit IRC | 22:09 | |
| *** raunak has quit IRC | 22:11 | |
| hemna | :( | 22:15 |
| *** darosale has joined #openstack-cinder | 22:22 | |
| *** rushiagr has quit IRC | 22:24 | |
| *** theanalyst has quit IRC | 22:26 | |
| *** tpsilva has quit IRC | 22:27 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Support for paraVirtual image adapter type https://review.openstack.org/312761 | 22:30 |
| *** earlephilhower has quit IRC | 22:34 | |
| *** lpetrut has quit IRC | 22:36 | |
| *** martyturner has quit IRC | 22:40 | |
| *** leeantho has quit IRC | 22:46 | |
| *** raunak has joined #openstack-cinder | 22:50 | |
| *** bwallis has joined #openstack-cinder | 22:50 | |
| *** ametts has quit IRC | 22:52 | |
| *** harlowja has quit IRC | 22:52 | |
| *** harlowja has joined #openstack-cinder | 22:53 | |
| *** darosale has quit IRC | 22:55 | |
| *** angela-s has quit IRC | 22:57 | |
| *** diablo_rojo has quit IRC | 22:58 | |
| *** rdx_matt has joined #openstack-cinder | 23:03 | |
| *** raunak has quit IRC | 23:05 | |
| *** rdx_matt has quit IRC | 23:08 | |
| *** arch-nemesis has quit IRC | 23:08 | |
| *** rchurch has quit IRC | 23:09 | |
| *** rchurch has joined #openstack-cinder | 23:10 | |
| *** akshai has quit IRC | 23:16 | |
| *** akshai has joined #openstack-cinder | 23:20 | |
| *** akshai has quit IRC | 23:21 | |
| *** arch-nemesis has joined #openstack-cinder | 23:24 | |
| *** sasukeh has quit IRC | 23:28 | |
| *** bapalm_ has quit IRC | 23:30 | |
| *** sdake has quit IRC | 23:37 | |
| *** salv-orlando has quit IRC | 23:39 | |
| *** raunak has joined #openstack-cinder | 23:42 | |
| *** bwallis has quit IRC | 23:49 | |
| *** amit213 has joined #openstack-cinder | 23:51 | |
| *** xyang1 has quit IRC | 23:55 | |
| *** vmtrooper has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!