| *** karthikp_ has joined #openstack-cinder | 00:02 | |
| *** yangyapeng has quit IRC | 00:02 | |
| *** salv-orlando has quit IRC | 00:03 | |
| *** karthikp_ has quit IRC | 00:07 | |
| *** enriquetaso has quit IRC | 00:08 | |
| *** markvoelker has joined #openstack-cinder | 00:12 | |
| *** markvoelker has quit IRC | 00:16 | |
| *** laughterwym has joined #openstack-cinder | 00:21 | |
| *** laughterwym has quit IRC | 00:21 | |
| *** laughterwym has joined #openstack-cinder | 00:21 | |
| *** karthikp_ has joined #openstack-cinder | 00:24 | |
| *** Suyi has quit IRC | 00:31 | |
| openstackgerrit | Weiwei Gu proposed openstack/cinder-specs: Support volume migration between different availability zones https://review.openstack.org/297550 | 00:40 |
|---|---|---|
| *** bardia has quit IRC | 00:43 | |
| *** Julien-zte has joined #openstack-cinder | 00:47 | |
| *** cheneydc has joined #openstack-cinder | 00:58 | |
| *** karthikp_ has quit IRC | 01:01 | |
| *** karthikp_ has joined #openstack-cinder | 01:02 | |
| *** karthikp_ has quit IRC | 01:04 | |
| *** karthikp_ has joined #openstack-cinder | 01:04 | |
| *** zhongjun_ has joined #openstack-cinder | 01:08 | |
| *** karthikp_ has quit IRC | 01:11 | |
| *** karthikp_ has joined #openstack-cinder | 01:11 | |
| *** markvoelker has joined #openstack-cinder | 01:13 | |
| *** JoseMello has joined #openstack-cinder | 01:15 | |
| *** markvoelker has quit IRC | 01:17 | |
| *** sheel has joined #openstack-cinder | 01:21 | |
| *** ducttape_ has joined #openstack-cinder | 01:21 | |
| *** karthikp_ has quit IRC | 01:25 | |
| *** karthikp_ has joined #openstack-cinder | 01:26 | |
| *** EinstCrazy has joined #openstack-cinder | 01:30 | |
| *** yangxi has joined #openstack-cinder | 01:31 | |
| openstackgerrit | KongWei proposed openstack/cinder: Add ZTE Block Storage Driver https://review.openstack.org/312355 | 01:31 |
| *** Apoorva has quit IRC | 01:31 | |
| *** karthik__ has joined #openstack-cinder | 01:31 | |
| *** karthikp_ has quit IRC | 01:32 | |
| *** yangxi has quit IRC | 01:33 | |
| *** yangyapeng has joined #openstack-cinder | 01:33 | |
| *** yangyapeng has quit IRC | 01:35 | |
| *** yangyapeng has joined #openstack-cinder | 01:35 | |
| *** Lee1092 has joined #openstack-cinder | 01:38 | |
| *** karthik__ has quit IRC | 01:38 | |
| *** karthikp_ has joined #openstack-cinder | 01:39 | |
| *** yangyape_ has joined #openstack-cinder | 01:40 | |
| *** sdake has joined #openstack-cinder | 01:40 | |
| *** yangxi has joined #openstack-cinder | 01:42 | |
| *** yangyape_ has quit IRC | 01:42 | |
| *** yangyape_ has joined #openstack-cinder | 01:42 | |
| *** ducttape_ has quit IRC | 01:42 | |
| *** karthikp_ has quit IRC | 01:43 | |
| *** yangyape_ has quit IRC | 01:43 | |
| *** yangyapeng has quit IRC | 01:43 | |
| *** yangyapeng has joined #openstack-cinder | 01:43 | |
| *** karthikp_ has joined #openstack-cinder | 01:44 | |
| *** sdake_ has joined #openstack-cinder | 01:52 | |
| *** Apoorva has joined #openstack-cinder | 01:53 | |
| *** sdake has quit IRC | 01:53 | |
| *** karthikp_ has quit IRC | 01:54 | |
| *** karthikp_ has joined #openstack-cinder | 01:58 | |
| *** yangxi has quit IRC | 01:59 | |
| *** chris_morrell has joined #openstack-cinder | 01:59 | |
| *** yangxi has joined #openstack-cinder | 02:02 | |
| *** Julien-zte has quit IRC | 02:04 | |
| *** julim has joined #openstack-cinder | 02:04 | |
| *** Apoorva has quit IRC | 02:04 | |
| *** bardia has joined #openstack-cinder | 02:04 | |
| *** chris_morrell has quit IRC | 02:04 | |
| *** karthikp_ has quit IRC | 02:08 | |
| *** karthikp_ has joined #openstack-cinder | 02:08 | |
| *** chris_morrell has joined #openstack-cinder | 02:09 | |
| *** sdake_ has quit IRC | 02:09 | |
| *** chris_morrell has quit IRC | 02:13 | |
| *** raunak has quit IRC | 02:15 | |
| *** merooney has joined #openstack-cinder | 02:18 | |
| *** karthikp_ has quit IRC | 02:23 | |
| *** karthikp_ has joined #openstack-cinder | 02:23 | |
| *** salv-orlando has joined #openstack-cinder | 02:27 | |
| *** iceyao has joined #openstack-cinder | 02:28 | |
| *** yangxi has quit IRC | 02:29 | |
| *** salv-orlando has quit IRC | 02:32 | |
| *** gouthamr_ has quit IRC | 02:33 | |
| jgriffith | smcginnis: I'll need to check it again since I changed things for unit tests | 02:38 |
| jgriffith | smcginnis: all the failing CI's isn't promissing, although I can't see logs on any of them so who knows :) | 02:38 |
| jgriffith | HGST passed at least | 02:38 |
| jgriffith | :) | 02:38 |
| jgriffith | glass 1/2 full | 02:38 |
| openstackgerrit | Weiwei Gu proposed openstack/python-cinderclient: add reset-state API for consistency groups https://review.openstack.org/320748 | 02:39 |
| *** karthikp_ has quit IRC | 02:56 | |
| openstackgerrit | Weiwei Gu proposed openstack/python-cinderclient: add reset-state API for consistency groups https://review.openstack.org/320748 | 02:56 |
| *** karthikp_ has joined #openstack-cinder | 02:57 | |
| *** raunak has joined #openstack-cinder | 02:59 | |
| *** karthikp_ has quit IRC | 02:59 | |
| *** yangxi has joined #openstack-cinder | 03:01 | |
| openstackgerrit | Weiwei Gu proposed openstack/python-cinderclient: add reset-state API for consistency groups https://review.openstack.org/320748 | 03:01 |
| *** chip_ has quit IRC | 03:03 | |
| *** chip_ has joined #openstack-cinder | 03:03 | |
| *** karthikp_ has joined #openstack-cinder | 03:11 | |
| *** karthik__ has joined #openstack-cinder | 03:15 | |
| *** chip_ has quit IRC | 03:15 | |
| *** sdake has joined #openstack-cinder | 03:15 | |
| *** martyturner has joined #openstack-cinder | 03:16 | |
| *** chip_ has joined #openstack-cinder | 03:16 | |
| *** karthikp_ has quit IRC | 03:17 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: [WIP] Switch service capabilities to ovo https://review.openstack.org/319040 | 03:19 |
| *** karthikp_ has joined #openstack-cinder | 03:19 | |
| *** karthik__ has quit IRC | 03:19 | |
| *** karthikp_ has quit IRC | 03:22 | |
| *** martyturner has quit IRC | 03:22 | |
| *** karthikp_ has joined #openstack-cinder | 03:24 | |
| *** bardia has quit IRC | 03:26 | |
| *** sdake has quit IRC | 03:27 | |
| *** karthikp_ has quit IRC | 03:32 | |
| *** karthikp_ has joined #openstack-cinder | 03:33 | |
| *** chip_ has quit IRC | 03:33 | |
| *** chip_ has joined #openstack-cinder | 03:34 | |
| *** sheel has quit IRC | 03:35 | |
| *** jdurgin1 has joined #openstack-cinder | 03:38 | |
| *** karthikp_ has quit IRC | 03:38 | |
| *** yangxi has quit IRC | 03:41 | |
| *** sdake has joined #openstack-cinder | 03:44 | |
| *** chip_ has quit IRC | 03:45 | |
| *** karthikp_ has joined #openstack-cinder | 03:45 | |
| *** sheel has joined #openstack-cinder | 03:46 | |
| *** chip_ has joined #openstack-cinder | 03:46 | |
| *** yangxi has joined #openstack-cinder | 03:50 | |
| openstackgerrit | Merged openstack/cinder: Revert "Add hacking check to prevent assert_called_once" https://review.openstack.org/320565 | 03:51 |
| *** yangxi has quit IRC | 03:54 | |
| *** sgotliv_ has joined #openstack-cinder | 03:55 | |
| *** salv-orlando has joined #openstack-cinder | 03:56 | |
| *** chip_ has quit IRC | 04:01 | |
| *** merooney has quit IRC | 04:01 | |
| *** links has joined #openstack-cinder | 04:01 | |
| *** salv-orlando has quit IRC | 04:02 | |
| *** chip_ has joined #openstack-cinder | 04:02 | |
| *** Poornima has joined #openstack-cinder | 04:18 | |
| *** chip_ has quit IRC | 04:27 | |
| *** chip_ has joined #openstack-cinder | 04:28 | |
| *** laughter_ has joined #openstack-cinder | 04:31 | |
| *** karthikp_ has quit IRC | 04:33 | |
| *** laughterwym has quit IRC | 04:34 | |
| *** karthikp_ has joined #openstack-cinder | 04:35 | |
| *** chip_ has quit IRC | 04:39 | |
| *** liverpooler has quit IRC | 04:39 | |
| *** chip_ has joined #openstack-cinder | 04:40 | |
| *** yangxi has joined #openstack-cinder | 04:42 | |
| *** karthikp_ has quit IRC | 04:42 | |
| *** karthikp_ has joined #openstack-cinder | 04:43 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/python-cinderclient: Support name option for volume restore https://review.openstack.org/275910 | 04:46 |
| *** karthikp_ has quit IRC | 04:49 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Updated volume, version, limit, metadata API-ref https://review.openstack.org/316587 | 04:49 |
| *** karthikp_ has joined #openstack-cinder | 04:50 | |
| *** jamielennox is now known as jamielennox|away | 04:50 | |
| *** pgadiya has joined #openstack-cinder | 04:52 | |
| *** jamielennox|away is now known as jamielennox | 05:04 | |
| *** lpetrut has joined #openstack-cinder | 05:06 | |
| *** sdake_ has joined #openstack-cinder | 05:11 | |
| *** GB21 has joined #openstack-cinder | 05:13 | |
| *** dave-mccowan has quit IRC | 05:13 | |
| *** sdake has quit IRC | 05:13 | |
| *** yangxi has quit IRC | 05:14 | |
| *** yangxi has joined #openstack-cinder | 05:19 | |
| *** Poornima has quit IRC | 05:19 | |
| *** karthikp_ has quit IRC | 05:22 | |
| *** Poornima has joined #openstack-cinder | 05:22 | |
| *** bardia has joined #openstack-cinder | 05:24 | |
| *** salv-orlando has joined #openstack-cinder | 05:26 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Support for checking JSON format https://review.openstack.org/320111 | 05:27 |
| *** zhongjun_ has quit IRC | 05:28 | |
| sheel | geguileo: hi | 05:28 |
| sheel | geguileo: one simple query, please refer https://review.openstack.org/#/c/313113/12/cinder/tests/unit/message/test_api.py | 05:30 |
| openstackgerrit | Peter Wang proposed openstack/cinder: VNX: New Cinder driver in Newton https://review.openstack.org/312427 | 05:31 |
| *** zhongjun_ has joined #openstack-cinder | 05:31 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: VNX: New Cinder driver in Newton https://review.openstack.org/312427 | 05:31 |
| *** laughter_ has quit IRC | 05:32 | |
| *** laughterwym has joined #openstack-cinder | 05:32 | |
| *** salv-orlando has quit IRC | 05:35 | |
| *** ChubYann has quit IRC | 05:35 | |
| *** yangxi_ has joined #openstack-cinder | 05:38 | |
| *** yangxi has quit IRC | 05:42 | |
| *** nkrinner_afk is now known as nkrinner | 05:42 | |
| *** savihou has joined #openstack-cinder | 05:42 | |
| *** liewegas has quit IRC | 05:45 | |
| *** liewegas has joined #openstack-cinder | 05:46 | |
| *** bardia has quit IRC | 05:48 | |
| *** salv-orlando has joined #openstack-cinder | 05:51 | |
| *** salv-orlando has quit IRC | 05:51 | |
| *** salv-orlando has joined #openstack-cinder | 05:51 | |
| *** jamielennox is now known as jamielennox|away | 05:54 | |
| *** liverpooler has joined #openstack-cinder | 05:58 | |
| *** laughterwym has quit IRC | 05:59 | |
| *** laughterwym has joined #openstack-cinder | 06:00 | |
| *** ig0r_ has joined #openstack-cinder | 06:01 | |
| *** laughter_ has joined #openstack-cinder | 06:02 | |
| *** laughterwym has quit IRC | 06:04 | |
| *** jwcroppe has joined #openstack-cinder | 06:05 | |
| *** avishay has joined #openstack-cinder | 06:07 | |
| *** jwcroppe_ has quit IRC | 06:07 | |
| *** jamielennox|away is now known as jamielennox | 06:10 | |
| *** noamsh has joined #openstack-cinder | 06:12 | |
| *** yangxi_ has quit IRC | 06:16 | |
| *** ig0r_ has quit IRC | 06:17 | |
| *** laughter_ has quit IRC | 06:17 | |
| *** laughterwym has joined #openstack-cinder | 06:18 | |
| *** jamielennox is now known as jamielennox|away | 06:21 | |
| *** rcernin has joined #openstack-cinder | 06:21 | |
| *** mjura has joined #openstack-cinder | 06:25 | |
| *** sgotliv_ has quit IRC | 06:26 | |
| *** yangxi has joined #openstack-cinder | 06:27 | |
| *** jamielennox|away is now known as jamielennox | 06:28 | |
| sheel | wanghao: hi | 06:28 |
| sheel | wanghao: regarding manage snapshot code - snapshot_manage.py | 06:29 |
| wanghao | sheel: hi | 06:29 |
| sheel | wanghao: I could see that we are using name and display name in parallel | 06:30 |
| sheel | # NOTE(wanghao) if name in request body, we are overriding the 'name' | 06:30 |
| sheel | snapshot_parameters['name'] = snapshot.get('name', | 06:30 |
| sheel | snapshot.get('display_name') | 06:30 |
| sheel | ) | 06:30 |
| sheel | wanghao: but i think this is not required for manage snapshot... may be only required in snapshot create for backward compatibility | 06:31 |
| *** laughterwym has quit IRC | 06:31 | |
| *** mjura has quit IRC | 06:31 | |
| sheel | wanghao: could you please confirm? | 06:31 |
| sheel | wanghao: i am refering libery code | 06:31 |
| *** laughterwym has joined #openstack-cinder | 06:32 | |
| wanghao | sheel: sure, checking it. | 06:32 |
| sheel | wanghao: :) | 06:32 |
| *** belmoreira has joined #openstack-cinder | 06:36 | |
| *** laughterwym has quit IRC | 06:37 | |
| wanghao | sheel: yeah, seems 'display_name' is useless here. | 06:37 |
| sheel | wanghao: thanks for confirmation, i wil update through patch | 06:37 |
| *** laughterwym has joined #openstack-cinder | 06:37 | |
| sheel | wanghao: there are multiple things in different code blocks where optimization is possible.. | 06:38 |
| wanghao | sheel: okay, thanks to working this. | 06:38 |
| sheel | wanghao: :) | 06:38 |
| *** raunak has quit IRC | 06:39 | |
| wanghao | sheel: indeed, it's a long time work. | 06:39 |
| sheel | wanghao: yes you are right.. | 06:40 |
| sheel | wanghao: I was looking into nova and cinder so found around 10-15 areas where things can be updated.. may be required in past but not required anymore.. | 06:40 |
| sheel | wanghao: thanks for your prompt response.. | 06:41 |
| wanghao | sheel: you're welcome, :) | 06:41 |
| sheel | :) | 06:41 |
| *** laughterwym has quit IRC | 06:42 | |
| *** laughterwym has joined #openstack-cinder | 06:43 | |
| *** mjura has joined #openstack-cinder | 06:43 | |
| *** tesseract has joined #openstack-cinder | 06:47 | |
| *** laughterwym has quit IRC | 06:52 | |
| *** e0ne has joined #openstack-cinder | 06:52 | |
| *** laughterwym has joined #openstack-cinder | 06:52 | |
| *** laughter_ has joined #openstack-cinder | 06:53 | |
| *** salv-orlando has quit IRC | 06:55 | |
| *** arecknag has joined #openstack-cinder | 06:55 | |
| *** laughterwym has quit IRC | 06:57 | |
| *** jsheeren has joined #openstack-cinder | 06:57 | |
| *** laughter_ has quit IRC | 06:58 | |
| openstackgerrit | Bhagyashri Shewale proposed openstack/cinder: Return BadRequest for invalid 4byte unicode character https://review.openstack.org/295097 | 07:01 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: 'name' is not used in snapshot_manage https://review.openstack.org/320805 | 07:01 |
| *** laughterwym has joined #openstack-cinder | 07:02 | |
| *** laughterwym has quit IRC | 07:04 | |
| *** savihou has quit IRC | 07:04 | |
| *** savihou has joined #openstack-cinder | 07:04 | |
| *** laughterwym has joined #openstack-cinder | 07:06 | |
| *** savihou has quit IRC | 07:07 | |
| openstackgerrit | wanghao proposed openstack/python-cinderclient: Improve deleting volumes with snapshots in CLI https://review.openstack.org/320809 | 07:19 |
| wanghao | sheel: hi, one commit in your patch regarding 'display_name' and 'name'. | 07:24 |
| wanghao | sheel: I think we shoud remove display_name here. | 07:24 |
| *** Poornima has quit IRC | 07:25 | |
| *** iceyao has quit IRC | 07:25 | |
| *** iceyao has joined #openstack-cinder | 07:25 | |
| *** laughterwym has quit IRC | 07:26 | |
| *** laughterwym has joined #openstack-cinder | 07:28 | |
| openstackgerrit | wanghao proposed openstack/python-cinderclient: Improve deleting volumes with snapshots in CLI https://review.openstack.org/320809 | 07:30 |
| *** adisky has joined #openstack-cinder | 07:30 | |
| geguileo | sheel: ping | 07:37 |
| sheel | wanghao: oops.. I mapped that with volume case | 07:37 |
| sheel | wanghao: will update, thanks | 07:37 |
| sheel | geguileo: hi | 07:37 |
| geguileo | sheel: Hi | 07:37 |
| *** laughter_ has joined #openstack-cinder | 07:37 | |
| geguileo | sheel: What I meant is that you are not making a request with marker argument | 07:38 |
| geguileo | sheel: That would query the second page | 07:38 |
| geguileo | sheel: Basically making the request with the next_link | 07:39 |
| sheel | geguileo: ohk... | 07:39 |
| geguileo | sheel: Or is there a test I missed? | 07:39 |
| sheel | geguileo: no you are right | 07:39 |
| sheel | geguileo: i wil update/add | 07:40 |
| geguileo | sheel: Thanks | 07:40 |
| sheel | geguileo: thank you.. | 07:40 |
| wanghao | sheel: Okay | 07:40 |
| *** laughterwym has quit IRC | 07:41 | |
| sheel | wanghao: :) | 07:41 |
| *** hoonetorg has quit IRC | 07:43 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: 'name' is not used in snapshot_manage https://review.openstack.org/320805 | 07:44 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: 'display-name' is not used in snapshot_manage https://review.openstack.org/320805 | 07:45 |
| *** chlong has quit IRC | 07:46 | |
| avishay | sheel: you're going to make me rebase again? :) ^^ | 07:49 |
| *** JoseMello has quit IRC | 07:49 | |
| sheel | avishay: you want me to hold it? | 07:49 |
| sheel | avishay: its ok to hold as this is not major change | 07:49 |
| avishay | sheel: no it's OK, thanks | 07:49 |
| *** jordanP has joined #openstack-cinder | 07:49 | |
| *** daneyon has quit IRC | 07:50 | |
| sheel | avishay: np | 07:50 |
| *** jwcroppe_ has joined #openstack-cinder | 07:55 | |
| *** sdake_ has quit IRC | 07:56 | |
| *** hoonetorg has joined #openstack-cinder | 07:56 | |
| *** jwcroppe has quit IRC | 07:57 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Manage/unmanage snapshot in ScaleIO driver https://review.openstack.org/316742 | 07:58 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| *** laughter_ has quit IRC | 08:04 | |
| *** avishay has quit IRC | 08:04 | |
| *** laughterwym has joined #openstack-cinder | 08:04 | |
| *** laughterwym has quit IRC | 08:04 | |
| *** laughterwym has joined #openstack-cinder | 08:05 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| *** salv-orlando has joined #openstack-cinder | 08:15 | |
| *** ig0r_ has joined #openstack-cinder | 08:20 | |
| *** salv-orlando has quit IRC | 08:21 | |
| *** salv-orlando has joined #openstack-cinder | 08:21 | |
| *** Poornima has joined #openstack-cinder | 08:22 | |
| *** avishay has joined #openstack-cinder | 08:23 | |
| *** Poornima has quit IRC | 08:24 | |
| *** arecknag has quit IRC | 08:27 | |
| *** _fortis has quit IRC | 08:32 | |
| *** arecknag has joined #openstack-cinder | 08:33 | |
| *** savihou has joined #openstack-cinder | 08:39 | |
| *** lpetrut has quit IRC | 08:41 | |
| *** laughterwym has quit IRC | 08:41 | |
| *** laughterwym has joined #openstack-cinder | 08:42 | |
| *** laughter_ has joined #openstack-cinder | 08:43 | |
| *** sdake has joined #openstack-cinder | 08:44 | |
| *** laughterwym has quit IRC | 08:46 | |
| *** laughter_ has quit IRC | 08:48 | |
| *** sdake has quit IRC | 08:49 | |
| *** sdake has joined #openstack-cinder | 08:51 | |
| *** lpetrut has joined #openstack-cinder | 08:52 | |
| *** daneyon has joined #openstack-cinder | 08:52 | |
| *** ociuhandu has quit IRC | 08:54 | |
| *** daneyon has quit IRC | 08:55 | |
| *** daneyon has joined #openstack-cinder | 08:56 | |
| *** yangyapeng has quit IRC | 08:56 | |
| *** lpetrut has quit IRC | 08:57 | |
| *** yangyapeng has joined #openstack-cinder | 08:57 | |
| *** sgotliv_ has joined #openstack-cinder | 08:58 | |
| *** yhayashi has quit IRC | 09:00 | |
| *** yangxi has quit IRC | 09:03 | |
| *** markvoelker has joined #openstack-cinder | 09:03 | |
| *** sdake has quit IRC | 09:05 | |
| *** yangxi has joined #openstack-cinder | 09:06 | |
| *** markvoelker has quit IRC | 09:08 | |
| *** avishay has quit IRC | 09:08 | |
| *** mvk has quit IRC | 09:22 | |
| *** gouthamr has joined #openstack-cinder | 09:35 | |
| *** laughterwym has joined #openstack-cinder | 09:49 | |
| *** laughterwym has quit IRC | 09:53 | |
| *** cheneydc has quit IRC | 09:54 | |
| *** mvk has joined #openstack-cinder | 09:55 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls https://review.openstack.org/252458 | 09:56 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec in create volume calls to ovo https://review.openstack.org/246600 | 09:56 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO https://review.openstack.org/253033 | 09:56 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype) https://review.openstack.org/257305 | 09:56 |
| *** arecknag has quit IRC | 09:59 | |
| *** sdague has joined #openstack-cinder | 10:03 | |
| *** markvoelker has joined #openstack-cinder | 10:04 | |
| *** GB21 has quit IRC | 10:05 | |
| *** al_indigo has quit IRC | 10:06 | |
| *** EinstCrazy has quit IRC | 10:09 | |
| *** markvoelker has quit IRC | 10:09 | |
| *** EinstCrazy has joined #openstack-cinder | 10:09 | |
| *** yangyapeng has quit IRC | 10:12 | |
| *** EinstCrazy has quit IRC | 10:14 | |
| *** ankit_ has joined #openstack-cinder | 10:15 | |
| *** lpetrut has joined #openstack-cinder | 10:21 | |
| *** yangxi has quit IRC | 10:22 | |
| *** avishay has joined #openstack-cinder | 10:23 | |
| *** ociuhandu has joined #openstack-cinder | 10:26 | |
| *** lprice1 has quit IRC | 10:26 | |
| *** lprice has joined #openstack-cinder | 10:27 | |
| *** jsheeren has quit IRC | 10:28 | |
| *** arecknag has joined #openstack-cinder | 10:32 | |
| *** gouthamr has quit IRC | 10:38 | |
| *** GB21 has joined #openstack-cinder | 10:59 | |
| *** markvoelker has joined #openstack-cinder | 11:05 | |
| *** avishay has quit IRC | 11:06 | |
| openstackgerrit | Merged openstack/cinder: Handle SSL termination proxies for version list https://review.openstack.org/305152 | 11:07 |
| *** markvoelker has quit IRC | 11:10 | |
| *** vnogin has joined #openstack-cinder | 11:14 | |
| *** openstackgerrit has quit IRC | 11:18 | |
| *** openstackgerrit has joined #openstack-cinder | 11:19 | |
| *** avishay has joined #openstack-cinder | 11:19 | |
| *** yangyapeng has joined #openstack-cinder | 11:20 | |
| *** yangxi has joined #openstack-cinder | 11:23 | |
| sheel | dulek: hi | 11:25 |
| dulek | sheel: hi, what's up? | 11:26 |
| sheel | dulek: all well | 11:26 |
| sheel | dulek: dulek: refering to your patches ^^, wont we need to bump volumetypelist versions to 1.7? | 11:26 |
| sheel | https://review.openstack.org/#/c/253033/22/cinder/objects/base.py | 11:26 |
| *** yangxi has quit IRC | 11:27 | |
| dulek | sheel: Whoops. Yes, we should. Thanks! | 11:27 |
| sheel | dulek: np | 11:28 |
| sheel | dulek: here as well : https://review.openstack.org/#/c/257305/23 | 11:29 |
| sheel | rebase required for same | 11:29 |
| *** belmoreira has quit IRC | 11:31 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO https://review.openstack.org/253033 | 11:33 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype) https://review.openstack.org/257305 | 11:33 |
| *** GB21 has quit IRC | 11:34 | |
| *** iceyao has quit IRC | 11:40 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Adding Scaling QoS for ScaleIO driver https://review.openstack.org/320923 | 11:48 |
| *** ankit_ has quit IRC | 11:49 | |
| *** ankit_ has joined #openstack-cinder | 11:50 | |
| *** GB21 has joined #openstack-cinder | 11:50 | |
| *** jamielennox is now known as jamielennox|away | 11:58 | |
| *** avishay has quit IRC | 12:04 | |
| *** jamielennox|away is now known as jamielennox | 12:07 | |
| *** markvoelker has joined #openstack-cinder | 12:10 | |
| *** ociuhandu has quit IRC | 12:11 | |
| *** ociuhandu has joined #openstack-cinder | 12:11 | |
| *** delatte has quit IRC | 12:12 | |
| *** gouthamr has joined #openstack-cinder | 12:13 | |
| *** jseiler_ has joined #openstack-cinder | 12:18 | |
| *** GB21 has quit IRC | 12:19 | |
| *** jseiler has quit IRC | 12:22 | |
| *** martyturner has joined #openstack-cinder | 12:25 | |
| *** itzdilip has joined #openstack-cinder | 12:30 | |
| *** lixiaoy1 has quit IRC | 12:31 | |
| *** lixiaoy1 has joined #openstack-cinder | 12:32 | |
| *** ccesario has quit IRC | 12:33 | |
| *** itzdilip has quit IRC | 12:35 | |
| *** tpsilva has joined #openstack-cinder | 12:36 | |
| *** pgadiya has quit IRC | 12:39 | |
| *** gcb has quit IRC | 12:42 | |
| *** markvoelker has quit IRC | 12:42 | |
| *** diablo_rojo has joined #openstack-cinder | 12:43 | |
| *** itzdilip has joined #openstack-cinder | 12:44 | |
| *** markvoelker has joined #openstack-cinder | 12:48 | |
| *** ccesario has joined #openstack-cinder | 12:49 | |
| *** laughterwym has joined #openstack-cinder | 12:58 | |
| *** ducttape_ has joined #openstack-cinder | 12:58 | |
| *** iceyao has joined #openstack-cinder | 12:58 | |
| *** cknight has joined #openstack-cinder | 12:59 | |
| *** edmondsw has joined #openstack-cinder | 12:59 | |
| *** laughter_ has joined #openstack-cinder | 12:59 | |
| *** eharney has joined #openstack-cinder | 13:00 | |
| *** belmoreira has joined #openstack-cinder | 13:01 | |
| *** laughterwym has quit IRC | 13:03 | |
| *** laughter_ has quit IRC | 13:03 | |
| *** laughterwym has joined #openstack-cinder | 13:04 | |
| *** porrua has joined #openstack-cinder | 13:04 | |
| *** ig0r__ has joined #openstack-cinder | 13:06 | |
| *** dave-mccowan has joined #openstack-cinder | 13:07 | |
| *** rlrossit has joined #openstack-cinder | 13:07 | |
| *** ig0r_ has quit IRC | 13:08 | |
| *** mjura has quit IRC | 13:08 | |
| *** ducttape_ has quit IRC | 13:09 | |
| *** itzdilip has quit IRC | 13:13 | |
| *** akerr has joined #openstack-cinder | 13:15 | |
| *** gman-tx has joined #openstack-cinder | 13:16 | |
| *** itzdilip has joined #openstack-cinder | 13:18 | |
| *** akshai has joined #openstack-cinder | 13:27 | |
| *** akshai_ has joined #openstack-cinder | 13:28 | |
| *** eharney has quit IRC | 13:29 | |
| *** timcl has joined #openstack-cinder | 13:30 | |
| *** xyang1 has joined #openstack-cinder | 13:31 | |
| *** akshai has quit IRC | 13:32 | |
| *** ivase has joined #openstack-cinder | 13:32 | |
| *** itzdilip has quit IRC | 13:33 | |
| openstackgerrit | Aleksey Ruban proposed openstack/cinder: Nexenta: Add NDB driver for NexentaEdge. https://review.openstack.org/320060 | 13:37 |
| *** dustins has joined #openstack-cinder | 13:37 | |
| *** lpetrut has quit IRC | 13:37 | |
| *** itzdilip has joined #openstack-cinder | 13:38 | |
| jgriffith | ildikov: smcginnis johnthetubaguy So this is interesting..... | 13:41 |
| smcginnis | jgriffith: "Interesting"? | 13:41 |
| ildikov | jgriffith: there are a few things, which one you mean? :) | 13:41 |
| jgriffith | ildikov: smcginnis johnthetubaguy it appears that although Nova will always do the flow of "reserve, initialize, attach" and that's what I'd expect to be enforced, there are some tempest tests that just fast fwd to the attach | 13:41 |
| jgriffith | https://github.com/openstack/tempest/blob/master/tempest/api/volume/test_volumes_actions.py#L62 | 13:41 |
| smcginnis | jgriffith: Oh great. Broken tempest tests now as well. | 13:42 |
| ildikov | jgriffith: boot from volume: when you attach an already existing Cinder volume at instance boot time skips reserve | 13:42 |
| jgriffith | ildikov: reserve is ok to skip, initialize_connection is the problem for me :( | 13:42 |
| ildikov | jgriffith: I mean Nova itself does not call reserve | 13:42 |
| smcginnis | ildikov: Oh right. Wasn't boot from volume and one other one identified in the last meeting as not following the reserve step? | 13:43 |
| ildikov | jgriffith: ah, ok got it | 13:43 |
| jgriffith | ildikov: that's fine | 13:43 |
| jgriffith | smcginnis: yeah, I think so and I can live with that for now | 13:43 |
| *** liverpooler has quit IRC | 13:43 | |
| smcginnis | jgriffith: OK, good. | 13:43 |
| ildikov | jgriffith: so this is why your code does not find the record in the db | 13:43 |
| jgriffith | smcginnis: ildikov my point here is that we apparantly have behavior in the wild that doesn't require initialize_connection :( | 13:43 |
| ildikov | jgriffith: as manually it worked fine | 13:43 |
| jgriffith | That makes me very very sad | 13:43 |
| ildikov | jgriffith: the host is Null in the DB though | 13:44 |
| jgriffith | ildikov: wait... one thing at a time ;) | 13:44 |
| jgriffith | ildikov: are you talking about the boot from volume case? | 13:44 |
| ildikov | jgriffith: sorry :) | 13:44 |
| ildikov | jgriffith: no, I applied your code in my Devstack env and checked what's happening when I attach a volume | 13:45 |
| jgriffith | ildikov: ohhh :) | 13:45 |
| ildikov | jgriffith: I couldn't get there yet though to check all the logs yet | 13:45 |
| jgriffith | ildikov: ok, let's talk about that | 13:45 |
| jgriffith | ildikov: you say the host was null in the DB? When, which entry? | 13:45 |
| jgriffith | ildikov: I'm assuming you mean for a regular "nova volume-attach" call | 13:46 |
| jgriffith | ildikov: did you check volume_attachment:attached_host ? | 13:46 |
| ildikov | jgriffith: yeah, I mean the regular nova volume-attach | 13:47 |
| *** ankit_ has quit IRC | 13:47 | |
| ildikov | jgriffith: and the attached_host is NULL as you say | 13:48 |
| *** links has quit IRC | 13:48 | |
| jgriffith | Ohhh, snap | 13:48 |
| jgriffith | I think I know what might be happening | 13:48 |
| ildikov | jgriffith: IIRC we know the host at initialize time and you store it as well in your code | 13:48 |
| ildikov | I think it's overwritten at attach | 13:49 |
| ildikov | but I couldn't get there to verify this thought | 13:49 |
| jgriffith | that should be set via the connector during initialize_connection. I wonder if during the attach call I'm updating and wiping it out accidentally | 13:49 |
| ildikov | ok, then we have the same thought on this :) | 13:49 |
| jgriffith | ildikov: yes, I think you must be right... I'll make a note to check that out | 13:49 |
| jgriffith | The more troubling thing for me here is the crappy fact that we've been allowing attach without reserve or initialize :( | 13:50 |
| jgriffith | that makes me very sad | 13:50 |
| jgriffith | I have to laugh after all of this.... folks always said "Nova makes bad assumptions". The reality is the Cinder flow is horribly broken and inconsistent. There's no way anybody could do anything but make bad assumptions :( | 13:51 |
| thrawn01 | jgriffith: detach is in a similar state | 13:51 |
| *** ametts has joined #openstack-cinder | 13:51 | |
| jgriffith | thrawn01: indeed | 13:51 |
| thrawn01 | jgriffith: it's possible for a user to use 'os-detach' as a poor mans reset state to available | 13:52 |
| thrawn01 | atleast it was that case in icehouse, i've not looked since. | 13:53 |
| jgriffith | thrawn01: icehouse... you mean back during the "ice age" , that was forever ago :) | 13:53 |
| thrawn01 | yeah, well, rackspace is just joining the present =/ | 13:53 |
| smcginnis | :) | 13:54 |
| jgriffith | thrawn01: hehe | 13:54 |
| *** xek has quit IRC | 13:54 | |
| *** jungleboyj has joined #openstack-cinder | 13:55 | |
| jgriffith | Well I must say I'm a bit annoyed | 13:55 |
| *** adrianofr has joined #openstack-cinder | 13:55 | |
| smcginnis | jgriffith: Well, on the bright side we are finally identifying some of these issues and starting to do something about it. :/ | 13:56 |
| jgriffith | I was trying to avoid just deprecating and rewriting API methods | 13:56 |
| jgriffith | Especially with all this micro-versioning madness. I'd vowed to never write an API change again | 13:56 |
| smcginnis | Cinder API /v4 FTW! | 13:56 |
| jgriffith | smcginnis: there you go with the glass 1/2 full again :) | 13:56 |
| *** salv-orl_ has joined #openstack-cinder | 13:56 | |
| jgriffith | smcginnis: LOL | 13:56 |
| jgriffith | smcginnis: I received some unpleasant feedback about OpenStack's microversioning implemntation yesterday afternoon from some users | 13:57 |
| jgriffith | scottda: which reminds me, we should make sure we make the experience less painful for Cinder consumers... you may already be doing that, but just a note | 13:59 |
| jgriffith | scottda: people seem to get upset when their version settings break and they can't figure out how to get them set correctly | 13:59 |
| *** salv-orlando has quit IRC | 13:59 | |
| *** porrua has quit IRC | 14:00 | |
| scottda | jgriffith: I'm interested in making the experience less painful as well. Is there something specific you can share, so we/I can look at fixing it? | 14:00 |
| *** porrua has joined #openstack-cinder | 14:00 | |
| scottda | I'm wondering how their version settings broke? | 14:00 |
| jgriffith | scottda: well other than the general "what is this crazy scheme for versioning all about"... there was a problem with I tihnk Packstack setting the wrong default version somewhere | 14:01 |
| *** martyturner has quit IRC | 14:01 | |
| jgriffith | scottda: apprantly it took a bit of unwinding for them | 14:01 |
| *** darosale has joined #openstack-cinder | 14:01 | |
| jgriffith | scottda: it's more of a dsitro/deployment issue, but apparantly it was difficult to unravel. | 14:02 |
| scottda | jgriffith: OK. Well, we are following the recommendation of the API wg, and the TC is behind microversioning as well. So I think it's something that's going to continue to crop up in OpenStack for all services. | 14:02 |
| jgriffith | scottda: there were some issues with their setting in nova.conf, and some interactions with keystone that hosed some things up... then apparantly they got into some dependency traps with some oslo-libs | 14:02 |
| jgriffith | scottda: yes, I mentioned that to them. Their response was less than enthusiastic | 14:03 |
| *** lprice1 has joined #openstack-cinder | 14:04 | |
| scottda | jgriffith: OK, well, Cinder mitaka and Nova still use /v2 endpoints as the default, so something specific in their distro would be needed to even switch over to /v3 and allow microversions. At least for Cinder. | 14:04 |
| *** ducttape_ has joined #openstack-cinder | 14:04 | |
| jgriffith | scottda: yeah... well not my area so I couldn't help them much. They talked about 2.1 or 2.5 on Nova and some other things... but who knows | 14:04 |
| *** lprice has quit IRC | 14:05 | |
| jgriffith | scottda: just thinking maybe we can learn from past experiences | 14:05 |
| *** sheel has quit IRC | 14:05 | |
| jgriffith | scottda: like gather feedback, bug info etc and have some insight of what people have or have not run into | 14:05 |
| *** martyturner has joined #openstack-cinder | 14:05 | |
| scottda | jgriffith: Yeah, there have been mistakes in how Nova did things that we need to learn from. I'm trying to stay on top of that. | 14:05 |
| jgriffith | scottda: cool | 14:06 |
| *** timcl1 has joined #openstack-cinder | 14:06 | |
| jgriffith | now that I've let my ADD run off there.... | 14:06 |
| scottda | :) | 14:06 |
| jgriffith | back to this attach/detach mess | 14:06 |
| *** timcl has quit IRC | 14:07 | |
| jgriffith | smcginnis: ildikov thrawn01 would it seem reasonable to put back in the 1/2 assed attachment record creation in the case that initialize was never called.... log it as a deprecated thing | 14:08 |
| *** lpetrut has joined #openstack-cinder | 14:08 | |
| jgriffith | and then for multi-attach require that the flow is enforced correctly? | 14:08 |
| *** ducttape_ has quit IRC | 14:09 | |
| jgriffith | I hate to add to the problem that's already been created :( | 14:09 |
| *** ducttape_ has joined #openstack-cinder | 14:09 | |
| jgriffith | I guess the other option whcih smcginnis and I talked about already, is just clean slate... new calls and deprecate the old ones | 14:09 |
| jgriffith | I suppose that honestly might be better at this point then continuing the spaghetti making | 14:10 |
| jgriffith | and I think smcginnis suggested it would be better before, but I ignored him :) | 14:10 |
| *** haplo37_ has joined #openstack-cinder | 14:10 | |
| ildikov | jgriffith: for multiattach we will need a Cinder microversion anyway most probably | 14:10 |
| ildikov | jgriffith: I think if we could finally do it right than we should | 14:11 |
| jgriffith | ildikov: I'll just give you a brand spanking new api call :) | 14:11 |
| jgriffith | alright, let me see how this works out | 14:11 |
| ildikov | jgriffith: I guess it means we will be busy with the fixes pretty much :) | 14:12 |
| ildikov | look, I think it's really the time to stop the madness | 14:12 |
| johnthetubaguy | jgriffith: I am curious what those API version settings are about? | 14:13 |
| *** lprice1 has quit IRC | 14:13 | |
| smcginnis | jgriffith: The advantage I see with the new calls is it's a clean slate and a very clear separation where we're not trying to do things in a different way while still trying to make old ones happy and ultimately ending up making compromises for both. | 14:13 |
| ildikov | smcginnis: +1 | 14:13 |
| ildikov | smcginnis: I think it's way better to do visible changes rather than having assumptions and workarounds hidden in the code | 14:14 |
| smcginnis | ildikov: Exactly! :) | 14:14 |
| *** gman-tx has quit IRC | 14:14 | |
| *** baumann has joined #openstack-cinder | 14:15 | |
| scottda | smcginnis: jgriffith +1 to new calls. If we continue to band-aid the old broken stuff it remains hard to maintain and understand. | 14:15 |
| jgriffith | righty ho then | 14:15 |
| johnthetubaguy | jgriffith: curious about the microversions issue | 14:16 |
| *** sheel has joined #openstack-cinder | 14:16 | |
| johnthetubaguy | jgriffith: what is the "default" version? | 14:16 |
| jgriffith | johnthetubaguy: that's what I didn't quite get, I'm honestly not familiar enough with the microversioning to understand completely | 14:17 |
| jgriffith | johnthetubaguy: but they had mentioned that something was set at 2.0 but needed to be 2.1? | 14:17 |
| johnthetubaguy | it honestly sounds like they are doing it wrongly, somewhere | 14:17 |
| jgriffith | does that make any sense? | 14:17 |
| johnthetubaguy | for microversions I always ask people to read this: https://dague.net/2015/06/05/the-nova-api-in-kilo-and-beyond-2/ | 14:17 |
| johnthetubaguy | its quite short, and explains the why | 14:18 |
| johnthetubaguy | and how to use it | 14:18 |
| scottda | johnthetubaguy: Yeah, that's where I always like to start people. | 14:18 |
| jgriffith | It also sounded like they had a mismatch between their conf settings an dkeystone | 14:18 |
| jgriffith | Oh... I forgot about that | 14:18 |
| jgriffith | johnthetubaguy: maybe I can contact them.... give them this link and get some other info | 14:18 |
| johnthetubaguy | jgriffith: that would be great | 14:19 |
| jgriffith | johnthetubaguy: I actually met the guy at a non-openstack/non-work event | 14:19 |
| johnthetubaguy | I am worried people are using it wrongly, our docs on using microversions are not great right now | 14:19 |
| jgriffith | just so happened he worked at a startup that was trying to deploy OpenStack for their internal dev teams | 14:19 |
| jgriffith | johnthetubaguy: yeah, I am sure that's the case | 14:20 |
| *** baumann has quit IRC | 14:20 | |
| *** diogogmt has quit IRC | 14:20 | |
| jgriffith | I keep saying "the guy" because I'm embarassingly terrible at remembering names :( | 14:20 |
| johnthetubaguy | jgriffith: I totally have that problem too | 14:21 |
| jgriffith | Throw in a few beers and forget it | 14:21 |
| smcginnis | Hah, same. | 14:21 |
| johnthetubaguy | so I used linked in to look people up now, its not fool proof, but it helps | 14:21 |
| johnthetubaguy | 14:21 | |
| jgriffith | johnthetubaguy: oh, not a bad idea | 14:22 |
| *** arecknag has quit IRC | 14:24 | |
| *** lprice has joined #openstack-cinder | 14:25 | |
| *** merooney has joined #openstack-cinder | 14:25 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Add VolumeAttachStatus Enum https://review.openstack.org/290885 | 14:26 |
| *** lprice has quit IRC | 14:30 | |
| *** adisky has quit IRC | 14:30 | |
| *** salv-orl_ has quit IRC | 14:33 | |
| *** jgregor has joined #openstack-cinder | 14:34 | |
| *** lprice has joined #openstack-cinder | 14:36 | |
| *** eharney has joined #openstack-cinder | 14:36 | |
| *** lprice1 has joined #openstack-cinder | 14:38 | |
| *** lprice has quit IRC | 14:40 | |
| *** merooney has quit IRC | 14:40 | |
| *** iceyao has quit IRC | 14:41 | |
| *** merooney has joined #openstack-cinder | 14:42 | |
| *** kfarr has joined #openstack-cinder | 14:45 | |
| *** pgadiya has joined #openstack-cinder | 14:46 | |
| *** baumann has joined #openstack-cinder | 14:48 | |
| *** EinstCrazy has joined #openstack-cinder | 14:50 | |
| *** xinli has joined #openstack-cinder | 14:53 | |
| xinli | geguileo: I have alerady fixed all the issues in your comments, could you please review the code? | 14:59 |
| geguileo | xinli: What patch? | 14:59 |
| xinli | geguileo: https://review.openstack.org/#/c/268283/ | 14:59 |
| *** martyturner has quit IRC | 15:00 | |
| *** Julien-zte has joined #openstack-cinder | 15:01 | |
| xinli | geguileo: In your last code review, you added 13 comments which really improves the code quarlity. | 15:01 |
| *** martyturner has joined #openstack-cinder | 15:01 | |
| geguileo | xinli: I don't understand why you set cgsnap status value here: https://review.openstack.org/#/c/268283/7..8/cinder/tests/unit/test_gpfs.py | 15:01 |
| Swanson | xyang1, If you're around can I get you to take a peek at https://review.openstack.org/#/c/310064/ again? Thanks! | 15:02 |
| geguileo | xinli: For example in test_create_cgsnapshot | 15:02 |
| *** EinstCrazy has quit IRC | 15:02 | |
| xinli | geguileo: actually I just use the cgsnap for test the model_update | 15:04 |
| *** martyturner1 has joined #openstack-cinder | 15:04 | |
| *** arch-nemesis has joined #openstack-cinder | 15:05 | |
| *** arch-nemesis has quit IRC | 15:05 | |
| *** arch-nemesis has joined #openstack-cinder | 15:05 | |
| geguileo | xinli: That's confusing | 15:05 |
| *** martyturner has quit IRC | 15:05 | |
| geguileo | xinli: Because you use it as an argument to the method you are testing | 15:06 |
| *** baumann has quit IRC | 15:06 | |
| geguileo | xinli: As well as for the expected part, but changing the value | 15:06 |
| *** pgadiya has quit IRC | 15:06 | |
| geguileo | xinli: If you want to test for available, you should just set the fields.ConsistencyGroupStatus.AVAILABLE in the reference dict, it's less confusing | 15:07 |
| *** baumann has joined #openstack-cinder | 15:07 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Remove mox usage from test_nexenta https://review.openstack.org/321055 | 15:07 |
| xinli | geguileo: I think this is what you mean: self.asertEqual(fields.ConsistencyGroupStatus.DELETED, model_update['status']) | 15:08 |
| geguileo | xinli: Better to have it closer to what you have now | 15:08 |
| *** sdake has joined #openstack-cinder | 15:09 | |
| xinli | grguileo: Please add new comments and I will change it, good point | 15:09 |
| geguileo | xinli: Equal({'status': ...DELETED}, model_update) | 15:09 |
| geguileo | xinli: That way you check that there are no additional keys, like you are doing now | 15:09 |
| geguileo | xinli: Ok, I'll add the comment | 15:09 |
| xinli | geguileo: Equal({'status': ...DELETED}, model_update) even better | 15:10 |
| *** lpetrut has quit IRC | 15:10 | |
| *** EinstCrazy has joined #openstack-cinder | 15:11 | |
| geguileo | xinli: Mmmm, isn't that what I wrote? r:-?? | 15:11 |
| *** GB21 has joined #openstack-cinder | 15:11 | |
| scottda | e0ne: Did you say that you got Devstack working with multi-backend using "CINDER_ENABLED_BACKENDS=lvm:lvmdriver-1,lvm:lvmdriver-2" ? | 15:11 |
| scottda | e0ne: Would you mind sharing your local.conf? | 15:12 |
| e0ne | scottda: hi. I'll try to find working conf | 15:12 |
| e0ne | scottda: I have not it right now | 15:12 |
| scottda | e0ne: Thanks. Whatever I try does not work, unless I change the default in lib/cinder and then it works fine. So it must be me, or something in my local.conf. | 15:12 |
| *** EinstCrazy has quit IRC | 15:13 | |
| scottda | e0ne: There is no hurry, but when you get a chance I think it would help me to have something that is known to work. | 15:13 |
| *** sdake_ has joined #openstack-cinder | 15:13 | |
| e0ne | scottda: it could be a bug in devstack too. I've tried it few months ago | 15:13 |
| *** sdake has quit IRC | 15:13 | |
| sheel | scottda: e0ne: I have tried migration after setting multiple backends | 15:13 |
| sheel | scottda: e0ne in case you dont find, I will share tomorrow | 15:14 |
| scottda | e0ne: Well, I'd be relieved if it was a bug, as I cannot get it to work. | 15:14 |
| *** belmoreira has quit IRC | 15:14 | |
| sheel | but i tried in rhel | 15:14 |
| sheel | ;) | 15:14 |
| scottda | sheel: My problem is that when I use that conf setting ^^^ in local.conf , it does not create the 2 backends. I must change the defaults to use that in devstack/lib/cinder and then it works. | 15:14 |
| sheel | scottda: ok, then may be specific to devstack only | 15:15 |
| scottda | sheel: This is also on Ubuntu | 15:15 |
| sheel | scottda: seems bug, wil also check on my ubuntu setup if works as in rhel or not... | 15:16 |
| sheel | scottda: will let you know tomorrow if this is your setup issue or devstack issue | 15:16 |
| scottda | sheel: Thanks. | 15:16 |
| sheel | scottda: np | 15:16 |
| *** EinstCrazy has joined #openstack-cinder | 15:18 | |
| *** itzdilip has quit IRC | 15:18 | |
| *** merooney has quit IRC | 15:18 | |
| *** diogogmt has joined #openstack-cinder | 15:19 | |
| *** EinstCrazy has quit IRC | 15:19 | |
| *** EinstCrazy has joined #openstack-cinder | 15:19 | |
| *** EinstCrazy has quit IRC | 15:22 | |
| *** timcl1 has quit IRC | 15:22 | |
| *** itzdilip has joined #openstack-cinder | 15:22 | |
| *** savihou has quit IRC | 15:24 | |
| *** lpetrut has joined #openstack-cinder | 15:25 | |
| *** EinstCrazy has joined #openstack-cinder | 15:25 | |
| *** nkrinner has quit IRC | 15:26 | |
| *** gman-tx has joined #openstack-cinder | 15:26 | |
| *** EinstCrazy has quit IRC | 15:26 | |
| *** itzdilip has quit IRC | 15:27 | |
| *** EinstCrazy has joined #openstack-cinder | 15:28 | |
| *** itzdilip has joined #openstack-cinder | 15:28 | |
| *** timcl has joined #openstack-cinder | 15:29 | |
| *** salv-orlando has joined #openstack-cinder | 15:33 | |
| *** sdake_ has quit IRC | 15:34 | |
| *** EinstCrazy has quit IRC | 15:35 | |
| *** salv-orlando has quit IRC | 15:38 | |
| *** itzdilip has quit IRC | 15:40 | |
| *** esker has joined #openstack-cinder | 15:50 | |
| *** leeantho has joined #openstack-cinder | 15:59 | |
| xyang1 | Swanson: sure | 16:00 |
| Swanson | xyang1, Thanks! | 16:01 |
| *** liverpooler has joined #openstack-cinder | 16:01 | |
| *** timcl has quit IRC | 16:03 | |
| *** avishay has joined #openstack-cinder | 16:03 | |
| *** rcernin has quit IRC | 16:04 | |
| avishay | xyang1 geguileo DuncanT smcginnis - sorry I can't join the meeting. I'd like to request some attention on my "list manageable" patches - I submitted them 3 months ago with the goal to have them in Newton with some decent driver support. Thank you very much. | 16:06 |
| smcginnis | avishay: OK, I'll bring it up. Thanks! | 16:06 |
| avishay | smcginnis: gracias | 16:07 |
| openstackgerrit | John Griffith proposed openstack/cinder: Fix attachment_get_by_* to return entire list https://review.openstack.org/321095 | 16:08 |
| *** links has joined #openstack-cinder | 16:11 | |
| *** avishay has quit IRC | 16:12 | |
| *** tesseract has quit IRC | 16:13 | |
| *** sgotliv_ has quit IRC | 16:21 | |
| *** timcl has joined #openstack-cinder | 16:22 | |
| *** esker has quit IRC | 16:23 | |
| *** esker has joined #openstack-cinder | 16:24 | |
| *** GB21 has quit IRC | 16:25 | |
| *** laughterwym has quit IRC | 16:26 | |
| *** laughterwym has joined #openstack-cinder | 16:27 | |
| xinli | geguileo: for the snapshot, it's difficult to compare the dict, since it has more items: {'id': '253b2878-ec60-4793-ad19-e65496ec7aab', 'status': 'available'}, id and status. | 16:27 |
| geguileo | xinli: But you do have the ID, right? | 16:28 |
| geguileo | xinli: So it just would have a bigger reference dictionary, right? | 16:28 |
| *** sdake has joined #openstack-cinder | 16:29 | |
| *** laughterwym has quit IRC | 16:31 | |
| *** lpetrut has quit IRC | 16:31 | |
| *** sayalilunkad has quit IRC | 16:32 | |
| *** merooney has joined #openstack-cinder | 16:35 | |
| *** bardia has joined #openstack-cinder | 16:38 | |
| *** raunak has joined #openstack-cinder | 16:40 | |
| *** merooney has quit IRC | 16:44 | |
| *** Julien-zte has quit IRC | 16:45 | |
| *** merooney has joined #openstack-cinder | 16:50 | |
| *** salv-orlando has joined #openstack-cinder | 16:50 | |
| *** Apoorva has joined #openstack-cinder | 16:51 | |
| openstackgerrit | Bardia Keyoumarsi proposed openstack/cinder: Coho Data: Reconnect to cluster on broken pipe https://review.openstack.org/318382 | 16:52 |
| xyang1 | DuncanT: will you be here after the meeting? | 16:57 |
| DuncanT | xyang1: For a while at least, yes | 16:57 |
| xyang1 | DuncanT: ok, I will ping you on a backup bug | 16:58 |
| DuncanT | xyang1: Sure | 16:58 |
| *** martyturner1 has quit IRC | 17:00 | |
| *** e0ne has quit IRC | 17:00 | |
| *** liverpooler has quit IRC | 17:01 | |
| *** diablo_rojo has quit IRC | 17:01 | |
| xinli | geguileo: I use the snapshot.id to compare the hole dict, the only item I don't modify is list comprehension. | 17:01 |
| xyang1 | DuncanT: hi, backup an in use volume and backup snapshot using the optimal path are both broken on local node as well | 17:01 |
| *** martyturner has joined #openstack-cinder | 17:01 | |
| geguileo | xinli: Ok, I'll review it again | 17:01 |
| xyang1 | DuncanT: I have a WIP patch: https://review.openstack.org/#/c/319434/ | 17:02 |
| geguileo | DuncanT: Ok, I'll look into adding the logging in another patch | 17:02 |
| DuncanT | xyang1: looking | 17:02 |
| xinli | geguileo: I try to write some small script, it fails and I am still learning the new skill :-) | 17:02 |
| xyang1 | DuncanT: my concern is this patch cannot be backported to mitaka because it changes a lot | 17:02 |
| xyang1 | DuncanT: the changes in LVM can be ignored, it is for my testing purpose | 17:03 |
| DuncanT | xyang1: The initial backport should probably be just a patch that disables anything that causes visible breakage | 17:03 |
| xyang1 | DuncanT: sure | 17:03 |
| xyang1 | DuncanT: thar means bring back code that finds volume manGer on local node | 17:04 |
| xyang1 | manager | 17:04 |
| DuncanT | xyang1: Or just disable the optimised path | 17:04 |
| *** jdurgin1 has quit IRC | 17:04 | |
| xyang1 | DuncanT: but that means something working in liberty is broken in mitaka | 17:04 |
| xyang1 | DuncanT: if we disable that | 17:05 |
| DuncanT | xyang1: Optimised backup of snap or in-use vol was working in liberty? | 17:05 |
| DuncanT | xyang1: I didn't realise we got that in | 17:05 |
| DuncanT | xyang1: I wonder if we can detect when we're on the same host and only use the optimised path in that case? | 17:06 |
| xyang1 | DuncanT: yes | 17:06 |
| DuncanT | xyang1: Not ideal but might be the least intrusive path initially | 17:06 |
| xyang1 | DuncanT: I will propose a patch and let you see | 17:07 |
| DuncanT | xyang1: Thanks. I know somebody at HP has a bug report about this, I'll find out who. | 17:07 |
| xyang1 | DuncanT: that will add back some code we removed for scaling backup | 17:07 |
| *** ducttape_ has quit IRC | 17:08 | |
| *** jordanP has quit IRC | 17:08 | |
| *** ducttape_ has joined #openstack-cinder | 17:08 | |
| openstackgerrit | Aleksey Ruban proposed openstack/cinder: Nexenta: Add NDB driver for NexentaEdge. https://review.openstack.org/320060 | 17:09 |
| DuncanT | xyang1: I don't think there's an elegant solution here for something back-portable. We need to figure out a way to CI this stuff or it is likely to get broken again | 17:09 |
| xyang1 | DuncanT: sure | 17:09 |
| *** tschrock52 has joined #openstack-cinder | 17:09 | |
| tschrock52 | Hi guys | 17:09 |
| tschrock52 | I asked a few questions in OpenStack's main channel and got no reply. Can you guys help me with a cinder problem in Kilo?? | 17:10 |
| *** ducttape_ has quit IRC | 17:10 | |
| DuncanT | tschrock52: Ask here, somebody might know something | 17:10 |
| tschrock52 | I'm trying to figure out how to make Cinder listen on multiple IPs. I.E. - my_ip would be two values. | 17:10 |
| *** ducttape_ has joined #openstack-cinder | 17:10 | |
| tschrock52 | DuncanT, thanks for the reply too man. Really appreciate it | 17:11 |
| DuncanT | tschrock52: I assume you mean for LVM/iscsi? | 17:11 |
| tschrock52 | Yes | 17:11 |
| tschrock52 | In cinder.conf | 17:11 |
| DuncanT | 0.0.0.0 will listen on all interfaces still I believe | 17:12 |
| DuncanT | If that is good enough | 17:12 |
| tschrock52 | I think that will solve that problem in particular | 17:12 |
| tschrock52 | Can you help with another one maybe? Same train of thought | 17:12 |
| DuncanT | Ask away... no promises I can help but you never know until you ask | 17:12 |
| tschrock52 | What happened was, I installed a new 10GB NIC on my storage controller. Then I set /etc/hosts on all my compute nodes to the new IP. So, assuming I wanted to take advantage of this new 10GB NIC, I'd need to change all iscsi targets in the nova database to my NEW IP address. I'm trying to avoid that like the plague. | 17:13 |
| tschrock52 | I.E. - Existing VM points to 10.10.0.109/etc/etc/etc/ and I want that to NOW point to the 10GB nic at 10.10.0.111 | 17:14 |
| tschrock52 | I'd have to do a massive amount of DB motifications to achieve this with my existing instances | 17:14 |
| tschrock52 | I'm wondering, outside of rebuilding all 70 of my VMs, is there a better way? | 17:14 |
| *** bardia has quit IRC | 17:14 | |
| DuncanT | tschrock52: Hmmm, not sure that you can avoid it. For an instance that is currently up with an attached volume, at the very least you'll need to do detach & attach, you can't change live AFAIK | 17:15 |
| tschrock52 | Can't do the detach and re-attach. It's the root volume on all instances | 17:15 |
| tschrock52 | Kilo won't let you detach the root filesystem volume from an instance, as I recall | 17:15 |
| DuncanT | tschrock52: Short of something nasty and manual I can't think of anything | 17:16 |
| DuncanT | If the old and new interfaces are on the same subnet you could swap their IPs? | 17:16 |
| tschrock52 | Very true | 17:16 |
| tschrock52 | It might be better to manage this at the hardware level, you are correct | 17:16 |
| tschrock52 | != might be. IS | 17:17 |
| openstack | tschrock52: Error: "=" is not a valid command. | 17:17 |
| DuncanT | It isn't a scenario we've every designed for I don't think, it's a rather specialised case | 17:17 |
| tschrock52 | Yeah, not a common problem, even out there on the web | 17:17 |
| *** lpetrut has joined #openstack-cinder | 17:17 | |
| DuncanT | You could probably script something, but it would be nasty and involved | 17:17 |
| DuncanT | You'd have to pause each VM while you swapped things around under it at a guess... should only need a fraction of a second pause though | 17:18 |
| tschrock52 | No sense in it. You're right, the hardware level is the way to handle this. Make the new NIC have the same IP as the old one. Then all should click into place and roll nice n smooth | 17:18 |
| tschrock52 | I can bring this stack offline | 17:19 |
| tschrock52 | So I will | 17:19 |
| DuncanT | That should work, yes, if it is possible for you | 17:19 |
| DuncanT | Good luck :-) | 17:19 |
| tschrock52 | Definitely is. Thank you so much DuncanT. Really man. You gave me exactly what I needed dude; another brain to bounce this off of. You rock. | 17:19 |
| DuncanT | tschrock52: You're welcome :-) | 17:19 |
| tschrock52 | *cheers* clinks glass | 17:20 |
| DuncanT | On that positive note, time for me to not be here. | 17:20 |
| tschrock52 | lolz | 17:20 |
| DuncanT | Sláinte | 17:21 |
| DuncanT | Or I guess I should say L'chaim these days | 17:22 |
| *** ociuhandu has quit IRC | 17:23 | |
| *** martyturner has quit IRC | 17:27 | |
| *** martyturner has joined #openstack-cinder | 17:28 | |
| openstackgerrit | Xinli Guan proposed openstack/cinder: Make IBM drivers return snapshot object for cg ops https://review.openstack.org/268283 | 17:32 |
| *** salv-orlando has quit IRC | 17:32 | |
| *** ChubYann has joined #openstack-cinder | 17:32 | |
| xinli | gegiileo: the new patchset posted, thank you very much for code review | 17:33 |
| *** sdake_ has joined #openstack-cinder | 17:33 | |
| *** sdake has quit IRC | 17:35 | |
| *** diablo_rojo has joined #openstack-cinder | 17:40 | |
| *** suyash has joined #openstack-cinder | 17:41 | |
| *** suyash has quit IRC | 17:42 | |
| *** suyash has joined #openstack-cinder | 17:42 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Move OVO methods from CinderObject to Persistent https://review.openstack.org/312930 | 17:44 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove remotable from OVOs https://review.openstack.org/312929 | 17:44 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Refactor create, save, and destroy OVO methods https://review.openstack.org/315541 | 17:44 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add auto relationship mappings to VOs https://review.openstack.org/307075 | 17:44 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Link VOs with their list counterparts https://review.openstack.org/307074 | 17:44 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Stop assuming persistent OVOs are also DictCompat https://review.openstack.org/321142 | 17:44 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add debug level logs on OVO backports https://review.openstack.org/321143 | 17:44 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Implement OVO class init mechanism https://review.openstack.org/321144 | 17:44 |
| *** tschrock52 has quit IRC | 17:45 | |
| geguileo | xinli: Reviewed :-) | 17:45 |
| geguileo | DuncanT: I have added the patch with the logging to the patch series | 17:45 |
| *** savihou has joined #openstack-cinder | 17:47 | |
| openstackgerrit | Kendall Nelson proposed openstack/cinder-specs: Dynamic Reconfiguration https://review.openstack.org/286234 | 17:48 |
| *** ociuhandu has joined #openstack-cinder | 17:50 | |
| *** bardia has joined #openstack-cinder | 17:51 | |
| *** ig0r__ has quit IRC | 17:51 | |
| *** links has quit IRC | 17:52 | |
| *** salv-orlando has joined #openstack-cinder | 17:53 | |
| *** e0ne has joined #openstack-cinder | 17:54 | |
| *** merooney has quit IRC | 17:56 | |
| *** crose has joined #openstack-cinder | 17:58 | |
| *** jgregor has quit IRC | 18:00 | |
| *** diablo_rojo has quit IRC | 18:00 | |
| *** jgregor has joined #openstack-cinder | 18:01 | |
| *** lpetrut has quit IRC | 18:03 | |
| *** diablo_rojo has joined #openstack-cinder | 18:06 | |
| *** enriquetaso has joined #openstack-cinder | 18:06 | |
| *** e0ne has quit IRC | 18:06 | |
| *** jungleboyj has quit IRC | 18:06 | |
| *** jungleboyj has joined #openstack-cinder | 18:09 | |
| *** e0ne has joined #openstack-cinder | 18:11 | |
| *** _ducttape_ has joined #openstack-cinder | 18:11 | |
| *** e0ne has quit IRC | 18:14 | |
| *** ducttape_ has quit IRC | 18:15 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Minimize cost of DB exists decorator checks https://review.openstack.org/319177 | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Improve DB volume check in LVM manage volume https://review.openstack.org/319323 | 18:18 |
| *** mvk has quit IRC | 18:20 | |
| openstackgerrit | Clinton Knight proposed openstack/cinder: NetApp: Add iterator handling to client library https://review.openstack.org/321164 | 18:21 |
| *** sdake_ is now known as sdake | 18:23 | |
| *** sdake_ has joined #openstack-cinder | 18:35 | |
| *** sdake has quit IRC | 18:37 | |
| *** sdake_ is now known as sdake | 18:48 | |
| *** lpetrut has joined #openstack-cinder | 18:52 | |
| *** sgotliv_ has joined #openstack-cinder | 18:53 | |
| *** rlrossit has quit IRC | 18:54 | |
| *** diablo_rojo has quit IRC | 18:56 | |
| *** liverpooler has joined #openstack-cinder | 18:57 | |
| *** jungleboyj has quit IRC | 18:59 | |
| *** raunak has quit IRC | 19:01 | |
| *** diablo_rojo has joined #openstack-cinder | 19:01 | |
| *** rlrossit has joined #openstack-cinder | 19:02 | |
| *** darosale has quit IRC | 19:03 | |
| *** jungleboyj has joined #openstack-cinder | 19:03 | |
| *** mvk has joined #openstack-cinder | 19:05 | |
| *** alyson_ has quit IRC | 19:07 | |
| *** raunak has joined #openstack-cinder | 19:13 | |
| *** liverpooler has quit IRC | 19:13 | |
| *** _ducttape_ has quit IRC | 19:17 | |
| *** daneyon has quit IRC | 19:26 | |
| *** merooney has joined #openstack-cinder | 19:27 | |
| *** ducttape_ has joined #openstack-cinder | 19:28 | |
| *** daneyon has joined #openstack-cinder | 19:33 | |
| *** sgotliv_ has quit IRC | 19:34 | |
| *** darosale has joined #openstack-cinder | 19:34 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Switched REST calls to async where available https://review.openstack.org/315226 | 19:37 |
| *** chenying has quit IRC | 19:39 | |
| *** chenying has joined #openstack-cinder | 19:40 | |
| *** diablo_rojo has quit IRC | 19:53 | |
| smcginnis | jgriffith: ping | 19:54 |
| *** porrua has quit IRC | 20:01 | |
| *** karthikp_ has joined #openstack-cinder | 20:01 | |
| *** merooney has quit IRC | 20:04 | |
| *** sheel has quit IRC | 20:05 | |
| *** karthikp_ has quit IRC | 20:05 | |
| *** karthikp_ has joined #openstack-cinder | 20:06 | |
| *** earlephilhower has quit IRC | 20:06 | |
| *** akerr has quit IRC | 20:07 | |
| *** diablo_rojo has joined #openstack-cinder | 20:11 | |
| *** darosale has quit IRC | 20:14 | |
| *** Apoorva has quit IRC | 20:15 | |
| *** lpetrut has quit IRC | 20:15 | |
| *** diablo_rojo has quit IRC | 20:24 | |
| *** zul has quit IRC | 20:29 | |
| *** karthikp_ has quit IRC | 20:30 | |
| patrickeast | any chance someone can +A this? https://review.openstack.org/#/c/313679/ all the +2's in the world and my stupid ci finally gave it the green-light | 20:31 |
| *** jwcroppe has joined #openstack-cinder | 20:40 | |
| *** gouthamr has quit IRC | 20:40 | |
| *** cknight has quit IRC | 20:43 | |
| *** jwcroppe_ has quit IRC | 20:43 | |
| geguileo | patrickeast: Done | 20:45 |
| patrickeast | geguileo: thanks! | 20:46 |
| smcginnis | patrickeast: Finally got it all back together? :) | 20:46 |
| *** sdake_ has joined #openstack-cinder | 20:46 | |
| patrickeast | smcginnis: mostly | 20:46 |
| patrickeast | smcginnis: some stuff didn't survive very well... so i ended up redeploying, and took the opportunity to try out mitaka | 20:47 |
| patrickeast | smcginnis: which went... about as well as could be expected | 20:47 |
| patrickeast | neutron linux bridge provider networks are sweet though | 20:47 |
| patrickeast | my networking has never been so simple | 20:47 |
| *** sdake has quit IRC | 20:48 | |
| scottda | ameade: Did those tempest tests that put plumbing in for microversions ever merge? | 20:48 |
| Swanson | patrickeast, do tell. How did you set it up? | 20:49 |
| patrickeast | Swanson: so in the past we've deployed all-in-one nodes so that vm's can do iscsi traffic right through their 10G nics to the array... and for doing the FC passthrough stuff | 20:50 |
| ameade | scottda: did gerrit change again? | 20:50 |
| patrickeast | Swanson: this new one switched to using something like this http://docs.openstack.org/mitaka/networking-guide/scenario-provider-lb.html | 20:51 |
| scottda | ameade: Not sure about gerrit, but I'm having a hard time finding the link for your tests. | 20:51 |
| patrickeast | but with flat networks for everything | 20:51 |
| ameade | scottda: https://review.openstack.org/#/c/300639/ | 20:51 |
| scottda | ameade: thx | 20:51 |
| ameade | no wait | 20:51 |
| scottda | ameade: Too late! | 20:51 |
| patrickeast | Swanson: so all the network traffic for VM's whether its iscsi or management just goes right through the linux bridge and onto the l2 physical network | 20:51 |
| ameade | scottda: yeah that's it i guess, unless that had one before it | 20:52 |
| patrickeast | Swanson: cuts like 8 million jumps between all kinds of openvswitch thingys | 20:52 |
| Swanson | patrickeast, oh. | 20:52 |
| *** earlephilhower has joined #openstack-cinder | 20:52 | |
| Swanson | patrickeast, I think those 8 million jumps were screwing me. | 20:52 |
| scottda | ameade: That looks like the one I was looking for. Thanks. | 20:52 |
| ameade | scottda: oh yeah, he had a separate patch but then squashed it back | 20:52 |
| ameade | cool | 20:52 |
| patrickeast | Swanson: yea it was a constant source of trouble for my nodes :( | 20:52 |
| *** Lee1092 has quit IRC | 20:53 | |
| *** raunak has quit IRC | 20:53 | |
| patrickeast | Swanson: i also ditched the fc passthrough scripts we were using and just use nova's built in stuff ala https://wiki.openstack.org/wiki/Pci_passthrough | 20:53 |
| patrickeast | Swanson: so i just make a vm with like the 'devstack-fc' flavor and i get one with two FC HBA's ready to roll | 20:53 |
| Swanson | patrickeast, I was meaning to ask you about how you did FC. This seems way cool. | 20:54 |
| patrickeast | Swanson: its a nice improvement over what we did before with https://github.com/openstack/third-party-ci-tools/blob/master/provisioning_scripts/fibre_channel/invoke_fc_passthrough.sh | 20:55 |
| *** xinli has quit IRC | 20:55 | |
| patrickeast | Swanson: so far it fails less, i think because if it can't do the passthrough nova fails to build the instance and it re-schedules it to another node | 20:55 |
| patrickeast | Swanson: whereas the other one would be failing after jenkins started the job | 20:55 |
| *** julim has quit IRC | 20:56 | |
| Swanson | patrickeast, nice! | 20:58 |
| *** gouthamr has joined #openstack-cinder | 21:00 | |
| *** karthikp_ has joined #openstack-cinder | 21:00 | |
| *** Apoorva has joined #openstack-cinder | 21:00 | |
| smcginnis | patrickeast: Very cool. | 21:00 |
| *** lcurtis has joined #openstack-cinder | 21:00 | |
| smcginnis | Nova-net is officially deprecated now, so I know I need to switch over to neutron sooner or later. | 21:00 |
| *** crose has quit IRC | 21:01 | |
| smcginnis | I did get a successful run using neutron with lb. Just have had enough other trouble with my CI that I didn't want to introduce yet another change. | 21:01 |
| *** haplo37_ has quit IRC | 21:01 | |
| *** karthikp_ has quit IRC | 21:03 | |
| patrickeast | ah yea, my understanding is that the lb provider networks are basically nova-net | 21:03 |
| *** karthikp_ has joined #openstack-cinder | 21:03 | |
| smcginnis | patrickeast: Hah! Well there you go. :) | 21:03 |
| patrickeast | i figured it was a good time to switch since everything was already broken | 21:03 |
| smcginnis | Right, at some point it can't get any worse, right? :) | 21:04 |
| patrickeast | haha exactly | 21:04 |
| smcginnis | My problem has been mines be mostly kinda working but not always. So trying to stabilize rather than redo. | 21:04 |
| patrickeast | yea thats hard | 21:04 |
| *** raunak has joined #openstack-cinder | 21:05 | |
| smcginnis | I did get very close to just scrapping the whole thing and trying to start over, but not quite there yet. | 21:05 |
| patrickeast | luckily i had enough hardware that i split it up and left some on the old-kinda-mostly working setup and did green-field setup on the rest | 21:05 |
| smcginnis | Nice! | 21:05 |
| *** dustins has quit IRC | 21:05 | |
| *** raildo is now known as raildo-afk | 21:09 | |
| *** darosale has joined #openstack-cinder | 21:11 | |
| *** karthikp_ has quit IRC | 21:13 | |
| *** karthikp_ has joined #openstack-cinder | 21:14 | |
| *** eharney has quit IRC | 21:18 | |
| *** karthikp_ has quit IRC | 21:24 | |
| *** karthikp_ has joined #openstack-cinder | 21:28 | |
| *** ametts has quit IRC | 21:28 | |
| *** rlrossit has quit IRC | 21:29 | |
| *** arch-nemesis has quit IRC | 21:37 | |
| *** tasker has joined #openstack-cinder | 21:37 | |
| tasker | Good day Cinder; With regards to this blueprint (https://blueprints.launchpad.net/cinder/+spec/support-import-export-snapshots), I have need to test the export of snapshots, and according to wanghao's comment, the client support was/is/should be add in Mitaka. Mitaka supports python-cinderclient 1.5.0 and 1.6.0, but I cannot find any function in the module that references snapshot import / export. I've also looked throu | 21:37 |
| *** ducttape_ has quit IRC | 21:37 | |
| *** jgregor has quit IRC | 21:38 | |
| patrickeast | tasker: its very possible that support wasn't added to the client yet | 21:39 |
| tasker | aww, sad face. | 21:40 |
| patrickeast | tasker: err wait, https://review.openstack.org/#/c/249090/ | 21:40 |
| patrickeast | the question then i guess is what version of the client is it in | 21:40 |
| *** edmondsw has quit IRC | 21:41 | |
| patrickeast | tasker: i see snapshot-manage in 1.6.0 | 21:42 |
| tasker | ok, so what does it mean to "manage" and "unmanage" a snapshot? I tried those functions of the cinderclient and it turned my snapshots into errors. I didn't get a sense of what those functions are doing. | 21:42 |
| *** jungleboyj has quit IRC | 21:42 | |
| patrickeast | manage means import some snapshot into cinder, unmanage means export it from cinders management | 21:43 |
| tasker | that is an odd choice of verbiage, but seems simple now. | 21:43 |
| patrickeast | i think the naming leaned towards manage vs import/export since it doesn't actually move the data or anything, just turns on/off cinder managing it | 21:44 |
| tasker | oh. | 21:44 |
| *** Julien-zte has joined #openstack-cinder | 21:44 | |
| tasker | wait, then am I going after functionality that doesn't exist? | 21:44 |
| patrickeast | haha | 21:44 |
| patrickeast | well | 21:45 |
| patrickeast | what do you want to do? | 21:45 |
| tasker | I was hoping for something like glance image upload / download. | 21:45 |
| *** sdake_ has quit IRC | 21:45 | |
| tasker | get the bits from Cinder to store on external media. | 21:45 |
| patrickeast | ah yea | 21:45 |
| patrickeast | so there are a few ways to get the data | 21:46 |
| patrickeast | none of which are a single command though XD | 21:46 |
| tasker | but you didn't say "no". . ) | 21:46 |
| tasker | what should I be looking for? | 21:47 |
| patrickeast | yea so you could use cinder backups or upload to image, and then snag it from whatever service those go off to | 21:47 |
| patrickeast | or you can, in theory, attach the cinder volume to a machine and just like dd the data | 21:47 |
| patrickeast | could even rig up a nova instance to attach the volume and expose some web-api for your management to get data in chunks from it | 21:47 |
| patrickeast | i dunno of any less hacky ways to get the data directly from cinder, aside from just going right to the backend device | 21:48 |
| tasker | all sound good. I like the idea of sending it back to glance. | 21:48 |
| patrickeast | yea, it might be slow, but it would work | 21:49 |
| tasker | ok. yea, I go directly to the back-end for now. was hoping to do it all via OS. | 21:49 |
| openstackgerrit | Merged openstack/python-cinderclient: Remove deprecated tempest_lib and use tempest.lib https://review.openstack.org/318618 | 21:49 |
| *** daneyon has quit IRC | 21:49 | |
| *** daneyon has joined #openstack-cinder | 21:50 | |
| tasker | great. Thanks a bunch Patrick. I'll look into some of those ideas. | 21:50 |
| patrickeast | np | 21:50 |
| *** sdake has joined #openstack-cinder | 21:51 | |
| *** baumann has left #openstack-cinder | 21:52 | |
| *** arch-nemesis has joined #openstack-cinder | 21:52 | |
| *** karthikp_ has quit IRC | 21:55 | |
| *** tasker has left #openstack-cinder | 21:55 | |
| *** bardia has quit IRC | 21:56 | |
| *** bardia has joined #openstack-cinder | 21:56 | |
| *** enriquetaso has quit IRC | 21:58 | |
| Swanson | Anyone around want to take a look at a failover_host failback patch? https://review.openstack.org/#/c/310064/ | 21:58 |
| *** bill_az_ has joined #openstack-cinder | 22:02 | |
| *** adrianofr has quit IRC | 22:02 | |
| openstackgerrit | Merged openstack/cinder: Assisted snapshot: use Keystone Session with Nova Client https://review.openstack.org/310204 | 22:07 |
| *** jwcroppe has quit IRC | 22:08 | |
| *** jwcroppe has joined #openstack-cinder | 22:08 | |
| openstackgerrit | Merged openstack/cinder: Specify a user-agent in Pure volume drivers https://review.openstack.org/313679 | 22:10 |
| *** darosale has quit IRC | 22:10 | |
| *** karthikp_ has joined #openstack-cinder | 22:12 | |
| *** jwcroppe has quit IRC | 22:13 | |
| *** leeantho has quit IRC | 22:14 | |
| karthikp | Hi all.. I wanted to create a non-model(non db) based oslo versioned object....What base class do I from inherit from. Currently my New ovo object inherits from base.CinderObject,base.CinderPersistentObject, base.CinderComparableObject | 22:21 |
| *** sdake has quit IRC | 22:25 | |
| *** earlephilhower has quit IRC | 22:26 | |
| *** diogogmt has quit IRC | 22:27 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: EMC VMAX - multi-attach failure to VMAX3 when SLO is omitted https://review.openstack.org/308952 | 22:29 |
| patrickeast | karthikp: i'm not 100% sure, but I think if you want a non-db object just drop the persistent base class | 22:29 |
| *** martyturner has quit IRC | 22:35 | |
| *** martyturner has joined #openstack-cinder | 22:36 | |
| *** akshai_ has quit IRC | 22:42 | |
| *** savihou has quit IRC | 22:43 | |
| karthikp | @patrickeast | 22:43 |
| karthikp | patrickeast: Thanks ..let me give it a shot | 22:43 |
| *** ducttape_ has joined #openstack-cinder | 22:46 | |
| *** karthikp_ has quit IRC | 22:47 | |
| *** chip_ has quit IRC | 22:47 | |
| *** tpsilva has quit IRC | 22:47 | |
| *** chip_ has joined #openstack-cinder | 22:49 | |
| *** kfarr has quit IRC | 22:49 | |
| *** salv-orl_ has joined #openstack-cinder | 22:50 | |
| *** merooney has joined #openstack-cinder | 22:50 | |
| *** salv-orlando has quit IRC | 22:53 | |
| *** karthikp_ has joined #openstack-cinder | 22:54 | |
| *** karthikp_ has quit IRC | 22:55 | |
| *** chip_ has quit IRC | 22:55 | |
| *** chip_ has joined #openstack-cinder | 22:57 | |
| *** merooney has quit IRC | 23:01 | |
| *** martyturner has quit IRC | 23:04 | |
| *** chlong has joined #openstack-cinder | 23:05 | |
| *** harlowja has quit IRC | 23:12 | |
| *** jwcroppe has joined #openstack-cinder | 23:14 | |
| *** lcurtis has quit IRC | 23:14 | |
| *** arch-nemesis has quit IRC | 23:22 | |
| *** ducttape_ has quit IRC | 23:25 | |
| *** Julien-zte has quit IRC | 23:26 | |
| *** salv-orl_ has quit IRC | 23:27 | |
| *** sdake has joined #openstack-cinder | 23:28 | |
| *** ducttape_ has joined #openstack-cinder | 23:28 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!