| *** vokt has quit IRC | 00:01 | |
| *** dannywilson has quit IRC | 00:01 | |
| *** gouthamr_ has joined #openstack-cinder | 00:02 | |
| jamielennox | patrickeast: so it appears that volume-type create exists for the cinder v2 api, but not the v1 api in OSC | 00:02 |
|---|---|---|
| *** gouthamr has quit IRC | 00:03 | |
| jamielennox | patrickeast: is there a chance those people are pinning devstack to work with cinder v1 | 00:03 |
| *** mutoulbj has quit IRC | 00:03 | |
| patrickeast | jamielennox: definitely possible | 00:04 |
| patrickeast | maybe, i’m missing something, but isn’t the gate using cinder v1 too? http://logs.openstack.org/83/186683/14/check/check-tempest-dsvm-full/5c460ca/logs/etc/cinder/cinder.conf.txt.gz | 00:04 |
| jamielennox | patrickeast: i'm not sure what i'm looking at there | 00:05 |
| patrickeast | jamielennox: oh uh, ‘enable_v1_api = true’ | 00:05 |
| patrickeast | jamielennox: iirc it needs to be ‘enable_v2_api = true’ | 00:05 |
| patrickeast | unless it defaults to do that | 00:06 |
| jamielennox | by my guess enable_v1_api will expose the v1 api in cinder, but in OSC i guess it's defaulting to use the v2 api | 00:06 |
| patrickeast | gotcha | 00:06 |
| jamielennox | generally you control that with OS_VOLUME_API_VERSION | 00:06 |
| jamielennox | patrickeast: both enable_v1_api and enable_v2_api default to True | 00:08 |
| *** skylerberg has quit IRC | 00:09 | |
| *** jungleboyj has quit IRC | 00:11 | |
| ameade | so OSC needs the command for the v2 api to be added? and in the meantime we use vers 1? | 00:12 |
| *** mtanino has quit IRC | 00:13 | |
| jamielennox | ameade: no, the opposite, it appears to be working for v2 which is what devstack gate will be using | 00:14 |
| jamielennox | the command is not implemented for v1 on the client side | 00:14 |
| *** tsekiyama has quit IRC | 00:15 | |
| *** Fdaisuke has quit IRC | 00:17 | |
| jamielennox | oh - this is weird, apparently the default is to use the v1 api https://github.com/openstack/python-openstackclient/blob/master/openstackclient/volume/client.py#L22 | 00:18 |
| *** primechuck has quit IRC | 00:20 | |
| jamielennox | how did this ever work.... | 00:21 |
| ameade | jamielennox: I think you have it reversed, it looks like v2 doesnt have create. | 00:21 |
| ameade | also, i think some of us are having issues because openrc defaults to setting these to v2 | 00:21 |
| ameade | so if you have ever stacked before it will use v2 | 00:21 |
| ameade | but if it's the first stack it's v1 | 00:22 |
| ameade | or something like that | 00:22 |
| jamielennox | ameade: sorry - you are right | 00:22 |
| jamielennox | https://github.com/openstack/python-openstackclient/blob/master/setup.cfg#L363 | 00:22 |
| ameade | yeah looks like setting 'CINDER_VERSION=1' fixed it for me | 00:23 |
| jamielennox | ameade: ok, so there are a couple of fixes to do here and either i can or someone who knows how to test it can | 00:24 |
| *** salv-orlando has quit IRC | 00:24 | |
| jamielennox | (because i don't know cinder) | 00:24 |
| jamielennox | adding --os-volume-api-version 1 to the type-create command should force it to use v1 for that command regardless of what is set in the environment | 00:25 |
| jamielennox | is there a reason that a similar volume type create command doesn't exist in OSC volume v2? | 00:25 |
| ameade | i have no idea | 00:25 |
| *** salv-orlando has joined #openstack-cinder | 00:26 | |
| jamielennox | thingee: does volume type create exist in cinder v2? | 00:26 |
| ameade | i'd volunteer to do that but I dont have permission to make changes as I want | 00:26 |
| *** Yogi1 has joined #openstack-cinder | 00:27 | |
| ameade | jamielennox: actually, i suppose we dont have to rush to get that part done | 00:27 |
| jamielennox | it appears to exist: http://developer.openstack.org/api-ref-blockstorage-v2.html#createVolumeType - i wonder why it wasn't implemented | 00:27 |
| *** garthb has quit IRC | 00:29 | |
| jamielennox | ameade: no worries, i can make those changes | 00:29 |
| ameade | jamielennox: thanks a lot for your help | 00:29 |
| *** _cjones_ has quit IRC | 00:29 | |
| jamielennox | ameade: do you know if there is a devstack bug filed for this? | 00:30 |
| ameade | jamielennox: I dont think there is yet | 00:30 |
| *** Apoorva has quit IRC | 00:30 | |
| *** gouthamr_ has quit IRC | 00:30 | |
| *** annashen has joined #openstack-cinder | 00:32 | |
| *** vilobhmm2 has quit IRC | 00:32 | |
| *** annashen has quit IRC | 00:33 | |
| *** salv-orlando has quit IRC | 00:34 | |
| *** heyun has joined #openstack-cinder | 00:34 | |
| *** lpabon has quit IRC | 00:37 | |
| *** topshare has joined #openstack-cinder | 00:37 | |
| *** gouthamr has joined #openstack-cinder | 00:40 | |
| *** Yogi1 has quit IRC | 00:40 | |
| jamielennox | ameade: are you able to run devstack with https://review.openstack.org/202340 and see if that fixes the issue for you | 00:44 |
| jamielennox | (and unset CINDER_VERSION) | 00:44 |
| *** divab has joined #openstack-cinder | 00:45 | |
| *** divab has left #openstack-cinder | 00:48 | |
| *** primechuck has joined #openstack-cinder | 00:50 | |
| *** laughterwym has joined #openstack-cinder | 00:53 | |
| *** leeantho has quit IRC | 00:53 | |
| ameade | jamielennox: yeah i can | 00:57 |
| ameade | it's running | 01:00 |
| ameade | jamielennox: doing a bunch of multitasking atm | 01:00 |
| *** Longgeek has joined #openstack-cinder | 01:00 | |
| *** mutoulbj has joined #openstack-cinder | 01:03 | |
| jamielennox | ameade: that's ok -thanks | 01:06 |
| ameade | jamielennox: looks like it got past that command | 01:07 |
| *** mutoulbj has quit IRC | 01:09 | |
| ameade | jamielennox: I think we are in good shape, I'll run with your patch until it merges | 01:09 |
| ameade | thanks | 01:09 |
| *** primechuck has quit IRC | 01:10 | |
| *** casusbelli has joined #openstack-cinder | 01:12 | |
| jgriffith | jamielennox: thanks a ton! | 01:13 |
| jgriffith | I just got home and checked to see how things went :) | 01:13 |
| jgriffith | Now I'm going to go eat dinner and have a drink :) | 01:14 |
| *** casusbel_ has quit IRC | 01:14 | |
| *** openstack has joined #openstack-cinder | 01:24 | |
| *** annegentle has quit IRC | 01:26 | |
| *** breitz has quit IRC | 01:29 | |
| *** breitz has joined #openstack-cinder | 01:30 | |
| *** rushil has joined #openstack-cinder | 01:35 | |
| *** patrickeast has quit IRC | 01:35 | |
| *** annegentle has joined #openstack-cinder | 01:37 | |
| *** jdandrea has quit IRC | 01:38 | |
| *** lpabon has joined #openstack-cinder | 01:39 | |
| *** leeantho has joined #openstack-cinder | 01:42 | |
| *** annegentle has quit IRC | 01:42 | |
| *** kaisers1 has quit IRC | 01:44 | |
| *** primechuck has joined #openstack-cinder | 01:44 | |
| *** IanGovett has joined #openstack-cinder | 01:54 | |
| *** dims_ has quit IRC | 01:58 | |
| *** zul has joined #openstack-cinder | 02:00 | |
| *** IanGovett has quit IRC | 02:01 | |
| *** annegentle has joined #openstack-cinder | 02:10 | |
| *** heyun1 has joined #openstack-cinder | 02:11 | |
| *** annegentle has quit IRC | 02:13 | |
| *** heyun has quit IRC | 02:15 | |
| *** lpabon has quit IRC | 02:16 | |
| *** dims__ has joined #openstack-cinder | 02:18 | |
| *** markvoelker has quit IRC | 02:22 | |
| *** vilobhmm has joined #openstack-cinder | 02:26 | |
| *** vilobhmm1 has joined #openstack-cinder | 02:28 | |
| *** jdurgin has quit IRC | 02:28 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Fix 3PAR driver handling of existing VLUNs https://review.openstack.org/202354 | 02:29 |
| *** Longgeek has joined #openstack-cinder | 02:30 | |
| *** dims__ has quit IRC | 02:30 | |
| *** vilobhmm has quit IRC | 02:30 | |
| *** gouthamr has quit IRC | 02:31 | |
| *** gouthamr has joined #openstack-cinder | 02:31 | |
| *** bkopilov has quit IRC | 02:31 | |
| *** Longgeek has quit IRC | 02:35 | |
| *** rushil has quit IRC | 02:37 | |
| *** gouthamr_ has joined #openstack-cinder | 02:41 | |
| *** diogogmt has joined #openstack-cinder | 02:43 | |
| *** gouthamr has quit IRC | 02:43 | |
| *** julim has joined #openstack-cinder | 02:52 | |
| *** hakimo_ has joined #openstack-cinder | 02:53 | |
| *** hakimo has quit IRC | 02:54 | |
| *** gouthamr_ has quit IRC | 03:06 | |
| *** lpabon has joined #openstack-cinder | 03:07 | |
| openstackgerrit | Zhenguo Niu proposed openstack/python-cinderclient: Remove H302 https://review.openstack.org/202364 | 03:09 |
| *** casusbelli has quit IRC | 03:11 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Dell SC: Add support for driver retype https://review.openstack.org/200765 | 03:11 |
| *** casusbelli has joined #openstack-cinder | 03:11 | |
| *** primechuck has quit IRC | 03:17 | |
| *** casusbelli has quit IRC | 03:18 | |
| *** casusbelli has joined #openstack-cinder | 03:19 | |
| *** davechen1 has joined #openstack-cinder | 03:23 | |
| *** annashen has joined #openstack-cinder | 03:23 | |
| *** lpabon has quit IRC | 03:25 | |
| *** annashen has quit IRC | 03:28 | |
| *** xyang1 has quit IRC | 03:30 | |
| *** mylu has joined #openstack-cinder | 03:32 | |
| *** markvoelker has joined #openstack-cinder | 03:33 | |
| *** dannywilson has joined #openstack-cinder | 03:34 | |
| *** markvoelker has quit IRC | 03:34 | |
| *** markvoelker has joined #openstack-cinder | 03:34 | |
| *** markvoelker_ has joined #openstack-cinder | 03:37 | |
| *** markvoelker has quit IRC | 03:39 | |
| openstackgerrit | wanghao proposed openstack/cinder: Support volume backup of qcow2 format volume https://review.openstack.org/182276 | 03:40 |
| *** leeantho has quit IRC | 03:42 | |
| *** sweston has quit IRC | 03:44 | |
| *** briancurtin has quit IRC | 03:45 | |
| *** scottda has quit IRC | 03:45 | |
| *** wanghao has quit IRC | 03:45 | |
| *** serverascode has quit IRC | 03:45 | |
| *** zhiyan has quit IRC | 03:45 | |
| *** ctracey has quit IRC | 03:45 | |
| *** rhefner has quit IRC | 03:45 | |
| *** Lee1092 has quit IRC | 03:46 | |
| *** dalgaaf has quit IRC | 03:46 | |
| *** rex_lee has quit IRC | 03:46 | |
| *** h00327910_ has quit IRC | 03:46 | |
| *** erlon has quit IRC | 03:46 | |
| *** boris-42 has quit IRC | 03:46 | |
| *** thingee has quit IRC | 03:47 | |
| *** coolsvap|away is now known as coolsvap | 03:47 | |
| *** wanghao has joined #openstack-cinder | 03:48 | |
| *** scottda has joined #openstack-cinder | 03:48 | |
| *** kaisers1 has joined #openstack-cinder | 03:49 | |
| *** dannywilson has quit IRC | 03:52 | |
| *** dannywilson has joined #openstack-cinder | 03:52 | |
| *** zhiyan has joined #openstack-cinder | 03:54 | |
| *** briancurtin has joined #openstack-cinder | 03:56 | |
| *** thingee has joined #openstack-cinder | 03:58 | |
| *** serverascode has joined #openstack-cinder | 03:58 | |
| *** bkopilov has joined #openstack-cinder | 03:59 | |
| *** rhefner has joined #openstack-cinder | 04:01 | |
| *** ctracey has joined #openstack-cinder | 04:02 | |
| *** rex_lee has joined #openstack-cinder | 04:02 | |
| *** boris-42 has joined #openstack-cinder | 04:07 | |
| *** erlon has joined #openstack-cinder | 04:08 | |
| *** annashen has joined #openstack-cinder | 04:08 | |
| *** dalgaaf has joined #openstack-cinder | 04:10 | |
| *** h00327910_ has joined #openstack-cinder | 04:11 | |
| *** hemna has joined #openstack-cinder | 04:11 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add updated_at into response of querying cinder resource detail https://review.openstack.org/202029 | 04:15 |
| *** belmoreira has joined #openstack-cinder | 04:15 | |
| *** sweston has joined #openstack-cinder | 04:16 | |
| *** jwcroppe has joined #openstack-cinder | 04:16 | |
| *** Longgeek has joined #openstack-cinder | 04:17 | |
| *** Lee1092 has joined #openstack-cinder | 04:19 | |
| *** Longgeek has quit IRC | 04:25 | |
| *** Longgeek has joined #openstack-cinder | 04:25 | |
| *** belmoreira has quit IRC | 04:28 | |
| *** annashen has quit IRC | 04:28 | |
| *** hemna has quit IRC | 04:30 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:33 | |
| *** mylu has quit IRC | 04:40 | |
| *** shyama has joined #openstack-cinder | 04:41 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 04:47 |
| openstackgerrit | Anish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm. https://review.openstack.org/193451 | 04:48 |
| *** markvoelker has joined #openstack-cinder | 04:49 | |
| *** markvoelker_ has quit IRC | 04:53 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 05:09 |
| *** deepakcs has joined #openstack-cinder | 05:14 | |
| *** daneyon has joined #openstack-cinder | 05:24 | |
| *** laughterwym_ has joined #openstack-cinder | 05:32 | |
| *** laughterwym has quit IRC | 05:35 | |
| *** angela-s|afk has quit IRC | 05:37 | |
| *** ankit has joined #openstack-cinder | 05:49 | |
| *** e0ne has joined #openstack-cinder | 06:00 | |
| *** e0ne has quit IRC | 06:02 | |
| *** vincent_hou has joined #openstack-cinder | 06:04 | |
| *** nkrinner_afk is now known as nkrinner | 06:07 | |
| *** harlowja_at_home has quit IRC | 06:10 | |
| *** annashen has joined #openstack-cinder | 06:12 | |
| openstackgerrit | Anish Bhatt proposed openstack/os-brick: [WIP] os-brick : Add support for --interface option in iscsiadm. https://review.openstack.org/193451 | 06:15 |
| *** vlaza has joined #openstack-cinder | 06:20 | |
| *** belmoreira has joined #openstack-cinder | 06:25 | |
| *** dannywilson has quit IRC | 06:26 | |
| *** vlaza has quit IRC | 06:27 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add multiple pools support for Huawei driver https://review.openstack.org/188251 | 06:27 |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add CHAP support for Huawei driver https://review.openstack.org/188360 | 06:30 |
| *** vilobhmm1 has quit IRC | 06:31 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add hypermetro support for huawei volume driver https://review.openstack.org/202023 | 06:32 |
| *** annashen has quit IRC | 06:37 | |
| *** abhishekk has joined #openstack-cinder | 06:39 | |
| *** agarciam has joined #openstack-cinder | 06:43 | |
| *** vlaza has joined #openstack-cinder | 06:45 | |
| *** abhishekk has quit IRC | 06:49 | |
| *** jdurgin has joined #openstack-cinder | 06:50 | |
| *** jdurgin has quit IRC | 06:50 | |
| *** Longgeek has quit IRC | 06:51 | |
| *** Longgeek has joined #openstack-cinder | 06:52 | |
| *** markus_z has joined #openstack-cinder | 07:05 | |
| *** bkopilov has quit IRC | 07:08 | |
| *** anshprat has quit IRC | 07:11 | |
| *** BharatK has joined #openstack-cinder | 07:13 | |
| *** anshprat has joined #openstack-cinder | 07:13 | |
| *** abehl has joined #openstack-cinder | 07:19 | |
| *** bkopilov has joined #openstack-cinder | 07:21 | |
| *** winston-d has quit IRC | 07:24 | |
| *** DTadrzak has quit IRC | 07:28 | |
| *** winston-d has joined #openstack-cinder | 07:32 | |
| *** lan has quit IRC | 07:33 | |
| *** lpetrut has joined #openstack-cinder | 07:34 | |
| *** Maike has joined #openstack-cinder | 07:38 | |
| *** jordanP has joined #openstack-cinder | 07:40 | |
| *** rhefner has quit IRC | 07:41 | |
| *** DTadrzak has joined #openstack-cinder | 07:43 | |
| *** laughterwym__ has joined #openstack-cinder | 07:48 | |
| *** laughterwym_ has quit IRC | 07:51 | |
| *** davechen1 has left #openstack-cinder | 07:55 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 07:55 |
| *** davechen has joined #openstack-cinder | 07:57 | |
| *** markvoelker has quit IRC | 08:04 | |
| *** pck has quit IRC | 08:06 | |
| *** pck has joined #openstack-cinder | 08:06 | |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder-specs: Proposal for backup service that uses ProtecTIER as the backend. https://review.openstack.org/153865 | 08:10 |
| *** markvoelker has joined #openstack-cinder | 08:11 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add updated_at into response of querying cinder resource detail https://review.openstack.org/202029 | 08:13 |
| *** markvoelker has quit IRC | 08:16 | |
| *** jistr has joined #openstack-cinder | 08:16 | |
| *** markvoelker has joined #openstack-cinder | 08:17 | |
| *** topshare has quit IRC | 08:17 | |
| *** topshare has joined #openstack-cinder | 08:19 | |
| *** yrabl has quit IRC | 08:19 | |
| *** yrabl has joined #openstack-cinder | 08:19 | |
| *** shyama has quit IRC | 08:20 | |
| *** topshare has quit IRC | 08:23 | |
| *** markvoelker has quit IRC | 08:25 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Add the notification and the migration progress indication https://review.openstack.org/189614 | 08:26 |
| *** vincent_hou has quit IRC | 08:26 | |
| *** topshare has joined #openstack-cinder | 08:27 | |
| *** markvoelker has joined #openstack-cinder | 08:29 | |
| *** boris-42 has quit IRC | 08:32 | |
| *** markvoelker has quit IRC | 08:33 | |
| *** laughterwym__ has quit IRC | 08:34 | |
| *** aarefiev has joined #openstack-cinder | 08:35 | |
| openstackgerrit | Andrey Pavlov proposed openstack/cinder: Fix doc string definitions https://review.openstack.org/193124 | 08:39 |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Add missing backup/restore methods to Sheepdog driver https://review.openstack.org/162927 | 08:40 |
| *** markvoelker has joined #openstack-cinder | 08:44 | |
| openstackgerrit | Ronen Mesonzhnik proposed openstack/cinder-specs: Proposal for backup service that uses ProtecTIER as the backend. https://review.openstack.org/153865 | 08:45 |
| *** salv-orlando has joined #openstack-cinder | 08:46 | |
| *** markvoelker has quit IRC | 08:48 | |
| *** aix has joined #openstack-cinder | 08:48 | |
| openstackgerrit | WenPing Zhang proposed openstack/cinder: blueprint cinder-driver-for-yeesan https://review.openstack.org/202440 | 08:48 |
| *** belmoreira has quit IRC | 08:51 | |
| *** bluex-pl has joined #openstack-cinder | 08:53 | |
| *** bluex-pl has quit IRC | 08:54 | |
| *** bluex-pl has joined #openstack-cinder | 08:54 | |
| *** markvoelker has joined #openstack-cinder | 08:58 | |
| *** markvoelker has quit IRC | 09:03 | |
| *** belmoreira has joined #openstack-cinder | 09:05 | |
| *** bkopilov has quit IRC | 09:14 | |
| *** ndipanov has joined #openstack-cinder | 09:14 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add updated_at into response of querying cinder resource detail https://review.openstack.org/202029 | 09:21 |
| *** bkopilov has joined #openstack-cinder | 09:28 | |
| openstackgerrit | Jordan Pittier proposed openstack/cinder: Rework the Scality SOFS driver to use the RemoteFS class https://review.openstack.org/200610 | 09:28 |
| openstackgerrit | wanghao proposed openstack/cinder: Add updated_at into response of querying cinder resource detail https://review.openstack.org/202029 | 09:31 |
| *** shyama has joined #openstack-cinder | 09:35 | |
| *** vlaza has quit IRC | 09:38 | |
| *** annashen has joined #openstack-cinder | 09:38 | |
| *** salv-orlando has quit IRC | 09:40 | |
| *** annashen has quit IRC | 09:42 | |
| *** kjelly has quit IRC | 09:48 | |
| *** kjelly has joined #openstack-cinder | 09:48 | |
| *** kjelly has joined #openstack-cinder | 09:49 | |
| *** lixiaoy11 has quit IRC | 09:52 | |
| *** davechen has left #openstack-cinder | 09:54 | |
| *** ociuhandu has joined #openstack-cinder | 09:55 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 09:55 |
| *** vlaza has joined #openstack-cinder | 09:58 | |
| *** aix has quit IRC | 09:58 | |
| *** e0ne has joined #openstack-cinder | 10:02 | |
| *** zhenguo has quit IRC | 10:03 | |
| *** afazekas has joined #openstack-cinder | 10:10 | |
| openstackgerrit | Lei Li proposed openstack/cinder: Adjust to right exception for chunkeddriver https://review.openstack.org/202481 | 10:11 |
| *** ociuhandu has quit IRC | 10:12 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Validate name and description string https://review.openstack.org/182994 | 10:22 |
| *** dims_ has joined #openstack-cinder | 10:24 | |
| *** e0ne is now known as e0ne_ | 10:25 | |
| *** aix has joined #openstack-cinder | 10:28 | |
| *** e0ne_ is now known as e0ne | 10:28 | |
| *** laughterwym has joined #openstack-cinder | 10:29 | |
| *** heyun1 has quit IRC | 10:33 | |
| *** afazekas has quit IRC | 10:35 | |
| openstackgerrit | WenPing Zhang proposed openstack/cinder: blueprint cinder-driver-for-yeesan https://review.openstack.org/202440 | 10:38 |
| *** dims_ has quit IRC | 10:39 | |
| *** salv-orlando has joined #openstack-cinder | 10:40 | |
| *** shyama has quit IRC | 10:46 | |
| *** shyama_ has joined #openstack-cinder | 10:46 | |
| *** shyama_ is now known as shyama | 10:46 | |
| *** afazekas has joined #openstack-cinder | 10:48 | |
| *** IanGovett has joined #openstack-cinder | 10:55 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Add consistency group tests to test_volume_rpcapi. https://review.openstack.org/202496 | 10:57 |
| *** coolsvap has quit IRC | 10:58 | |
| *** coolsvap has joined #openstack-cinder | 10:58 | |
| tbarron | BharatK: Thanks for understanding on this ... | 11:03 |
| *** topshare has quit IRC | 11:03 | |
| BharatK | tbarron, Np thanks your valuable opinion. | 11:03 |
| *** e0ne is now known as e0ne_ | 11:13 | |
| *** belmoreira has quit IRC | 11:15 | |
| *** IanGovett has quit IRC | 11:17 | |
| *** timcl has joined #openstack-cinder | 11:27 | |
| *** e0ne_ is now known as e0ne | 11:30 | |
| openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: Posix backup driver https://review.openstack.org/163647 | 11:33 |
| BharatK | tbarron, Modified the patch as per your suggestion, please review it. ^^ | 11:33 |
| *** kjelly_ has joined #openstack-cinder | 11:39 | |
| *** belmoreira has joined #openstack-cinder | 11:40 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 11:40 |
| tbarron | BharatK: looking | 11:43 |
| BharatK | tbarron, thanks. | 11:43 |
| *** ganso_ has joined #openstack-cinder | 11:48 | |
| *** topshare has joined #openstack-cinder | 11:52 | |
| *** topshare has quit IRC | 11:52 | |
| *** Longgeek has quit IRC | 11:54 | |
| *** bluex-pl has quit IRC | 11:55 | |
| *** IanGovett has joined #openstack-cinder | 11:58 | |
| *** bluex-pl has joined #openstack-cinder | 11:59 | |
| *** topshare has joined #openstack-cinder | 12:02 | |
| *** bkopilov has quit IRC | 12:04 | |
| *** Longgeek has joined #openstack-cinder | 12:05 | |
| *** bluex-pl has quit IRC | 12:06 | |
| *** deepakcs has quit IRC | 12:13 | |
| openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: Posix backup driver https://review.openstack.org/163647 | 12:14 |
| *** dims_ has joined #openstack-cinder | 12:14 | |
| BharatK | eharney, tbarron Modified the "Posix backup driver" patch as per your comments please review it. ^^ | 12:14 |
| *** bluex-pl has joined #openstack-cinder | 12:17 | |
| openstackgerrit | Jordan Pittier proposed openstack/os-brick: Remotefs: add ScalityFS support https://review.openstack.org/196631 | 12:18 |
| openstackgerrit | Jordan Pittier proposed openstack/os-brick: Remotefs: add ScalityFS support https://review.openstack.org/196631 | 12:18 |
| *** nkrinner is now known as nkrinner_afk | 12:18 | |
| *** BharatK has quit IRC | 12:27 | |
| *** bswartz has quit IRC | 12:29 | |
| *** markvoelker has joined #openstack-cinder | 12:29 | |
| *** dims_ has quit IRC | 12:30 | |
| *** salv-orlando has quit IRC | 12:31 | |
| *** dims__ has joined #openstack-cinder | 12:32 | |
| *** markvoelker has quit IRC | 12:34 | |
| *** topshare has quit IRC | 12:38 | |
| *** timcl1 has joined #openstack-cinder | 12:38 | |
| *** dims__ has quit IRC | 12:39 | |
| *** IanGovett has quit IRC | 12:39 | |
| *** IanGovett1 has joined #openstack-cinder | 12:39 | |
| *** lpetrut has quit IRC | 12:41 | |
| *** timcl has quit IRC | 12:41 | |
| *** markvoelker has joined #openstack-cinder | 12:43 | |
| *** merooney has joined #openstack-cinder | 12:43 | |
| *** IanGovett has joined #openstack-cinder | 12:44 | |
| *** krtaylor has quit IRC | 12:44 | |
| *** topshare has joined #openstack-cinder | 12:44 | |
| *** IanGovett1 has quit IRC | 12:45 | |
| *** topshare has quit IRC | 12:45 | |
| openstackgerrit | wanghao proposed openstack/cinder: Support volume backup of qcow2 format volume https://review.openstack.org/182276 | 12:45 |
| *** topshare has joined #openstack-cinder | 12:45 | |
| *** topshare has quit IRC | 12:45 | |
| *** topshare has joined #openstack-cinder | 12:46 | |
| *** topshare has quit IRC | 12:46 | |
| *** topshare has joined #openstack-cinder | 12:47 | |
| *** markvoelker has quit IRC | 12:48 | |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Adds manage/unmanage methods for HNAS drivers https://review.openstack.org/197999 | 12:49 |
| *** IanGovett1 has joined #openstack-cinder | 12:49 | |
| openstackgerrit | Gaurang Tapase proposed openstack/cinder: Adding NFS support to the GPFS Driver https://review.openstack.org/197884 | 12:50 |
| *** IanGovett has quit IRC | 12:50 | |
| *** krtaylor has joined #openstack-cinder | 12:51 | |
| *** salv-orlando has joined #openstack-cinder | 12:53 | |
| *** lpetrut has joined #openstack-cinder | 12:54 | |
| *** IanGovett has joined #openstack-cinder | 12:54 | |
| *** diogogmt has quit IRC | 12:55 | |
| *** diogogmt has joined #openstack-cinder | 12:56 | |
| *** IanGovett1 has quit IRC | 12:56 | |
| *** markvoelker has joined #openstack-cinder | 12:57 | |
| *** bswartz has joined #openstack-cinder | 12:59 | |
| *** bluex-pl has quit IRC | 13:00 | |
| *** zhenguo has joined #openstack-cinder | 13:01 | |
| *** markvoelker has quit IRC | 13:02 | |
| *** mylu has joined #openstack-cinder | 13:04 | |
| *** markvoelker has joined #openstack-cinder | 13:05 | |
| *** lpabon has joined #openstack-cinder | 13:05 | |
| *** markvoelker_ has joined #openstack-cinder | 13:06 | |
| *** timcl1 has quit IRC | 13:09 | |
| *** markvoelker has quit IRC | 13:10 | |
| *** gouthamr has joined #openstack-cinder | 13:12 | |
| *** coolsvap is now known as coolsvap|away | 13:13 | |
| *** bluex-pl has joined #openstack-cinder | 13:13 | |
| *** nug is now known as cFouts | 13:14 | |
| *** bnemec has joined #openstack-cinder | 13:14 | |
| *** dims_ has joined #openstack-cinder | 13:16 | |
| *** abhinavsgsits has joined #openstack-cinder | 13:17 | |
| *** mylu has quit IRC | 13:18 | |
| *** mylu has joined #openstack-cinder | 13:18 | |
| *** bluex-pl has quit IRC | 13:20 | |
| *** bluex-pl has joined #openstack-cinder | 13:20 | |
| *** akerr has joined #openstack-cinder | 13:21 | |
| *** Yogi1 has joined #openstack-cinder | 13:22 | |
| *** timcl has joined #openstack-cinder | 13:22 | |
| *** crose has joined #openstack-cinder | 13:25 | |
| *** dustins has joined #openstack-cinder | 13:26 | |
| *** topshare has quit IRC | 13:27 | |
| *** edmondsw has joined #openstack-cinder | 13:28 | |
| *** primechuck has joined #openstack-cinder | 13:28 | |
| *** ankit has quit IRC | 13:30 | |
| *** agarciam has quit IRC | 13:32 | |
| *** agarciam has joined #openstack-cinder | 13:32 | |
| *** mylu has quit IRC | 13:32 | |
| *** rhefner has joined #openstack-cinder | 13:32 | |
| *** topshare has joined #openstack-cinder | 13:33 | |
| *** jaypipes has joined #openstack-cinder | 13:35 | |
| *** thangp has joined #openstack-cinder | 13:36 | |
| *** bkopilov has joined #openstack-cinder | 13:37 | |
| *** timcl has quit IRC | 13:38 | |
| *** timcl has joined #openstack-cinder | 13:39 | |
| *** ronis has joined #openstack-cinder | 13:41 | |
| *** e0ne is now known as e0ne_ | 13:44 | |
| *** mylu has joined #openstack-cinder | 13:46 | |
| *** jdandrea has joined #openstack-cinder | 13:46 | |
| *** e0ne_ is now known as e0ne | 13:49 | |
| *** abhinavsgsits is now known as aagrawal | 13:51 | |
| *** mriedem_away is now known as mriedem | 13:56 | |
| *** liuxg has joined #openstack-cinder | 13:57 | |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add iSCSI multipath support for Huawei driver https://review.openstack.org/188365 | 13:58 |
| *** topshare has quit IRC | 13:59 | |
| *** topshare has joined #openstack-cinder | 14:00 | |
| *** thangp has quit IRC | 14:02 | |
| *** topshare has quit IRC | 14:04 | |
| *** thangp has joined #openstack-cinder | 14:07 | |
| *** Yogi1 has quit IRC | 14:11 | |
| *** BharatK has joined #openstack-cinder | 14:12 | |
| *** Yogi1 has joined #openstack-cinder | 14:13 | |
| *** mutoulbj has joined #openstack-cinder | 14:13 | |
| *** aagrawal has quit IRC | 14:17 | |
| *** xyang has joined #openstack-cinder | 14:19 | |
| *** fifieldt_ has quit IRC | 14:19 | |
| *** BharatK has quit IRC | 14:21 | |
| *** Tross1 has quit IRC | 14:21 | |
| *** markvoelker_ has quit IRC | 14:24 | |
| *** mriedem has left #openstack-cinder | 14:26 | |
| *** mriedem has joined #openstack-cinder | 14:27 | |
| *** jordanP has quit IRC | 14:29 | |
| *** annegentle has joined #openstack-cinder | 14:30 | |
| *** vlaza has quit IRC | 14:34 | |
| *** ronis has quit IRC | 14:36 | |
| *** e0ne is now known as e0ne_ | 14:36 | |
| *** jungleboyj has joined #openstack-cinder | 14:37 | |
| openstackgerrit | Anna Sortland proposed openstack/cinder: Remove "volume:services" rule from policy.json https://review.openstack.org/202599 | 14:38 |
| *** BharatK has joined #openstack-cinder | 14:39 | |
| *** lpetrut has quit IRC | 14:41 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: POC for RBD thin provisioning accounting https://review.openstack.org/202602 | 14:42 |
| openstackgerrit | Liu Xinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 14:43 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: POC for RBD thin provisioning accounting https://review.openstack.org/202602 | 14:45 |
| *** garthb has joined #openstack-cinder | 14:53 | |
| *** markvoelker has joined #openstack-cinder | 14:53 | |
| *** Yogi11 has joined #openstack-cinder | 14:55 | |
| *** Yogi1 has quit IRC | 14:56 | |
| *** lcurtis has joined #openstack-cinder | 14:58 | |
| *** Longgeek has quit IRC | 14:58 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder-specs: Add Cinder Volume A/A HA spec https://review.openstack.org/202615 | 14:58 |
| *** dustins has quit IRC | 15:00 | |
| *** Longgeek has joined #openstack-cinder | 15:00 | |
| *** ociuhandu has joined #openstack-cinder | 15:00 | |
| *** daneyon_ has joined #openstack-cinder | 15:02 | |
| *** dustins has joined #openstack-cinder | 15:02 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Continue conversion to volume object https://review.openstack.org/201404 | 15:04 |
| *** mriedem is now known as mriedem_away | 15:04 | |
| *** daneyon has quit IRC | 15:04 | |
| *** Tross has joined #openstack-cinder | 15:05 | |
| *** e0ne_ is now known as e0ne | 15:05 | |
| dulek | DuncanT: Hi, you may be interested in this: https://review.openstack.org/202615 | 15:05 |
| dulek | dulek: I've felt that we're doing very slow progress, so created a spec to start discussion on paper. | 15:06 |
| *** coolsvap|away is now known as coolsvap | 15:06 | |
| *** Longgeek has quit IRC | 15:07 | |
| *** yuriy_n17 has joined #openstack-cinder | 15:07 | |
| DuncanT | dulek: Having one place to discuss it is probably a good plan. I expect that there'll be sub-specs coming out of this, since there is not just one issue preventing A/A | 15:10 |
| DuncanT | dulek: I'll take a read and comment | 15:10 |
| DuncanT | dulek: Thanks | 15:11 |
| dulek | DuncanT: I'll welcome any comments, thank you! | 15:11 |
| *** abehl has quit IRC | 15:12 | |
| *** belmoreira has quit IRC | 15:13 | |
| *** mutoulbj has quit IRC | 15:16 | |
| *** dannywilson has joined #openstack-cinder | 15:16 | |
| *** hemna has joined #openstack-cinder | 15:18 | |
| *** afazekas has quit IRC | 15:20 | |
| *** hemna has quit IRC | 15:20 | |
| *** liuxg has quit IRC | 15:22 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Fix Consistency Group issues https://review.openstack.org/201685 | 15:22 |
| *** BharatK has quit IRC | 15:25 | |
| *** e0ne has quit IRC | 15:26 | |
| *** e0ne has joined #openstack-cinder | 15:26 | |
| *** e0ne has quit IRC | 15:27 | |
| *** e0ne has joined #openstack-cinder | 15:27 | |
| *** e0ne has quit IRC | 15:28 | |
| *** Yogi11 has quit IRC | 15:29 | |
| *** tsekiyama has joined #openstack-cinder | 15:30 | |
| *** Yogi1 has joined #openstack-cinder | 15:30 | |
| *** _________ has joined #openstack-cinder | 15:30 | |
| *** Tross has quit IRC | 15:31 | |
| *** _________ has quit IRC | 15:31 | |
| *** dustins has quit IRC | 15:31 | |
| *** thangp has quit IRC | 15:31 | |
| *** e0ne has joined #openstack-cinder | 15:32 | |
| e0ne | eharney: hi. thanks for really quick feedback for yuriy_n17's patch (https://review.openstack.org/202602, RBD thin provisioning accounting). i've added some comments there | 15:34 |
| *** Tross has joined #openstack-cinder | 15:34 | |
| *** thangp has joined #openstack-cinder | 15:34 | |
| *** ociuhandu has quit IRC | 15:35 | |
| *** Tross has quit IRC | 15:37 | |
| *** bluex-pl has quit IRC | 15:38 | |
| *** ig0r_ has quit IRC | 15:40 | |
| *** afazekas has joined #openstack-cinder | 15:44 | |
| *** harlowja_at_home has joined #openstack-cinder | 15:45 | |
| *** kjelly_ has quit IRC | 15:46 | |
| *** lpetrut has joined #openstack-cinder | 15:49 | |
| *** markus_z has quit IRC | 15:49 | |
| *** annegentle has quit IRC | 15:52 | |
| *** laughterwym has quit IRC | 15:52 | |
| *** annegentle has joined #openstack-cinder | 15:52 | |
| *** laughterwym has joined #openstack-cinder | 15:53 | |
| *** leeantho has joined #openstack-cinder | 15:54 | |
| *** ronis has joined #openstack-cinder | 16:02 | |
| *** Tross has joined #openstack-cinder | 16:04 | |
| *** laughterwym has quit IRC | 16:04 | |
| *** coolsvap is now known as coolsvap|away | 16:05 | |
| *** lpetrut has quit IRC | 16:05 | |
| openstackgerrit | j_king proposed openstack/cinder: Add get_trust_context method to BackupManager https://review.openstack.org/202669 | 16:06 |
| *** mutoulbj has joined #openstack-cinder | 16:07 | |
| jgriffith | scottda: thingee FYI, here's a first draft at least: http://j-griffith.blogspot.com/2015/07/volume-attach-code-flow-in-cinder.html | 16:08 |
| *** Yogi1 has quit IRC | 16:09 | |
| *** Tross has quit IRC | 16:10 | |
| *** merooney has quit IRC | 16:12 | |
| j_king | that's a WIP commit and I'd be grateful for any reviews to point me in the right direction. | 16:12 |
| *** Yogi1 has joined #openstack-cinder | 16:12 | |
| *** Yogi11 has joined #openstack-cinder | 16:13 | |
| *** mriedem_away is now known as mriedem | 16:13 | |
| *** Tross has joined #openstack-cinder | 16:13 | |
| *** vokt has joined #openstack-cinder | 16:14 | |
| *** Tross has quit IRC | 16:14 | |
| *** harlowja_at_home has quit IRC | 16:16 | |
| scottda | jgriffith: cool. I don't know if you noticed, but I posted a diagram for detach: http://imgur.com/Bmv1AJz | 16:16 |
| *** Yogi1 has quit IRC | 16:17 | |
| scottda | It's linked in the NovaCinderAPI etherpad under use cases. | 16:18 |
| openstackgerrit | Anna Sortland proposed openstack/cinder: Add delete_snapshot and update_snapshot rules https://review.openstack.org/202680 | 16:19 |
| *** edmondsw has quit IRC | 16:26 | |
| jgriffith | scottda: nice | 16:27 |
| e0ne | jgriffith: great! thank you for writing it! | 16:27 |
| *** lpetrut has joined #openstack-cinder | 16:28 | |
| *** dims_ has quit IRC | 16:30 | |
| *** _cjones_ has joined #openstack-cinder | 16:30 | |
| openstackgerrit | Eric Brown proposed openstack/cinder: Add bandit for security static analysis testing https://review.openstack.org/179568 | 16:31 |
| *** dims_ has joined #openstack-cinder | 16:32 | |
| *** dims_ has quit IRC | 16:32 | |
| *** vilobhmm has joined #openstack-cinder | 16:36 | |
| *** mutoulbj has quit IRC | 16:37 | |
| *** lpetrut has quit IRC | 16:38 | |
| *** mylu has quit IRC | 16:39 | |
| *** mylu has joined #openstack-cinder | 16:41 | |
| *** Apoorva has joined #openstack-cinder | 16:41 | |
| *** hemnafk is now known as hemna | 16:41 | |
| *** sghanekar_ has joined #openstack-cinder | 16:41 | |
| *** mylu has quit IRC | 16:42 | |
| *** Yogi11 has quit IRC | 16:42 | |
| *** rwsu has quit IRC | 16:44 | |
| *** annashen has joined #openstack-cinder | 16:45 | |
| scottda | Adrea Rosa wrote the diagram, he gets the credit | 16:45 |
| scottda | s/Adrea/Andrea | 16:45 |
| hemna | jgriffith, just reading through your blog post | 16:46 |
| hemna | jgriffith, so I believe that most drivers aren't really doing what you describe in the blog post with respect to create_export and initialize_connection. | 16:46 |
| hemna | our drivers included. | 16:47 |
| hemna | mostly because create_export doesn't have any of the initiator side information. it only passes the volume. | 16:47 |
| hemna | from what I'm seeing in most initialize_connection methods for most drivers, like ours, we have to have the connector (initiator information), in order to create an exportable target at all. | 16:48 |
| jgriffith | hemna: sorry, I don't know what your'e talking about exactly | 16:48 |
| hemna | so in that case, create_export does nothing. | 16:48 |
| hemna | jgriffith, your blog post. http://j-griffith.blogspot.com/2015/07/volume-attach-code-flow-in-cinder.html | 16:48 |
| jgriffith | hemna: which I suspect is why your driver is having problems with live-migration (not sure though) I commented on that in the ether pad BTW | 16:48 |
| hemna | you detail driver.create_export and driver.initialize_connection and what they are supposed to do. | 16:48 |
| hemna | but | 16:48 |
| jgriffith | hemna: right, and because you do it differently you're saying that it's wrong? | 16:49 |
| jgriffith | hemna: That's based on the "reference" driver as noted in the blog | 16:49 |
| hemna | the reason why drivers do the exporting in initialize_connection, is because it's the only call into the driver that has the required information to do an export. | 16:49 |
| *** mylu has joined #openstack-cinder | 16:49 | |
| hemna | I'm just pointing out the information. | 16:49 |
| hemna | and our drivers aren't the only ones that do this | 16:49 |
| hemna | I'm actually documenting other drivers that do this same thing on the etherpad. I started working on that yesterday | 16:50 |
| jgriffith | hemna: I'm just pointing out that part of the reason there's a mismatch (as noted in the ether pad) is because a number of folks "did their own thing" rather than follow the reference. Granted the reference is hard to follow in some cases | 16:50 |
| jgriffith | hemna: Cool... | 16:50 |
| hemna | yah, that's fine. | 16:50 |
| hemna | but for those of us that "did their own thing", we had no choice | 16:50 |
| hemna | because create_export doesn't have the connector | 16:51 |
| jgriffith | hemna: I am of the opinion however that if the reference does it one way, and you do it another... the reference is the one that's "correct" | 16:51 |
| hemna | nope | 16:51 |
| jgriffith | hehe | 16:51 |
| jgriffith | fair enough | 16:51 |
| *** mylu has quit IRC | 16:51 | |
| hemna | the connector isn't passed in create_export | 16:51 |
| hemna | so many of us can't even create a target w/o having the information of who we are creating the target for. | 16:51 |
| hemna | re: connector | 16:52 |
| hemna | so, I think if we just pass the connector into create_export | 16:52 |
| jgriffith | hemna: so why don't you change that? | 16:52 |
| hemna | then that fixes that, and then initialize_connection can be exactly as you describe in the blog, nothing more than collecting existing information about the target, and passing it back. | 16:52 |
| *** mutoulbj has joined #openstack-cinder | 16:52 | |
| jgriffith | hemna: if you required that why wouldn't you just add it rather than go off and do something completely different? | 16:52 |
| hemna | that would fix some of the confusion with how/why Nova calls initialize_connection many times during live migration | 16:52 |
| jgriffith | hemna: well, initialize_connection is the only call Nova has | 16:53 |
| hemna | well, because I didn't understand it well, and it's not entirely clear in the code/documentation. | 16:53 |
| hemna | nor was it clear how/when nova calls init_connection and for what reasons. | 16:53 |
| jgriffith | hemna: what code documentation :) | 16:53 |
| hemna | jgriffith, yah I know. | 16:53 |
| hemna | heh | 16:53 |
| jgriffith | hemna: which is why I wrote that this morning FWIW | 16:54 |
| hemna | I'm not trying to come off as slamming cinder in any way, but more about pointing out existing issues that can help solidify live migration for a lot of folks. | 16:54 |
| hemna | as it's clear we aren't the only ones with this issue. :( | 16:54 |
| jgriffith | hemna: and also I noted that I suspect this is the live-migration issue with some drivers in the ether pad last week.. but it was completely ignored | 16:54 |
| hemna | yah I think the blog is great | 16:54 |
| hemna | it really helps clear up the purpose of the methods and what they are supposed to provide. | 16:54 |
| hemna | sorry, I must have missed that | 16:55 |
| jgriffith | I had hoped it would help... glad if it does even a little | 16:55 |
| hemna | anyway, so I'm going to finish going through the drivers and point them out in the etherpad | 16:55 |
| hemna | then create a spec that simple adds connector to create_export | 16:55 |
| hemna | yah absolutely | 16:55 |
| Swanson | Can we rename initialize_connection to something sane if it isn't actually initializing the connection? | 16:56 |
| hemna | I'll post a patch that adds the connector to create_export and update every driver to at least accept it. | 16:56 |
| hemna | Swanson, good luck with that :P | 16:56 |
| hemna | Swanson, you'll need to shim that, and/or change every Nova call to it. | 16:57 |
| Swanson | hmmm | 16:58 |
| *** DericHorn-HP has joined #openstack-cinder | 16:59 | |
| openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Adding delete wait loop for CloudByte ISCSI volumes. https://review.openstack.org/201059 | 16:59 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 17:00 |
| *** aix has quit IRC | 17:00 | |
| *** cloudrancher has joined #openstack-cinder | 17:00 | |
| *** jistr has quit IRC | 17:00 | |
| *** cloudrancher has left #openstack-cinder | 17:00 | |
| openstackgerrit | Anna Sortland proposed openstack/cinder: Add delete/update_volume_metadata policy rules https://review.openstack.org/202701 | 17:01 |
| *** dustins has joined #openstack-cinder | 17:02 | |
| *** mylu has joined #openstack-cinder | 17:02 | |
| *** bswartz has quit IRC | 17:02 | |
| *** abehl has joined #openstack-cinder | 17:02 | |
| *** merooney has joined #openstack-cinder | 17:03 | |
| *** dustins has quit IRC | 17:03 | |
| *** merooney has quit IRC | 17:03 | |
| *** dustins has joined #openstack-cinder | 17:03 | |
| *** Tross has joined #openstack-cinder | 17:04 | |
| *** merooney has joined #openstack-cinder | 17:04 | |
| *** e0ne has quit IRC | 17:09 | |
| *** Yogi1 has joined #openstack-cinder | 17:10 | |
| *** yuriy_n17 has quit IRC | 17:11 | |
| *** krtaylor has quit IRC | 17:13 | |
| *** Maike has quit IRC | 17:17 | |
| *** bswartz has joined #openstack-cinder | 17:18 | |
| *** IanGovett1 has joined #openstack-cinder | 17:18 | |
| *** IanGovett has quit IRC | 17:19 | |
| *** xyang has quit IRC | 17:20 | |
| mylu | Hi guys, is there a way to use python-cinderclient to create a cinder client that just uses the token I pass in instead of authenticating it? I pass in `auth`, `session`. session is keystone session auth contains the auth_url and token I got from keystone | 17:21 |
| mylu | I'm using cinderclient.client.Client to create a client right now, but I'm open any suggestions. Thanks! | 17:22 |
| scottda | Swanson: I think the better answer is to create Cinder APIs as needed for things like get_connector_info_dict, so that Nova can call that instead of initialize_connection... | 17:25 |
| scottda | And cleanup the Nova code so that initialize_connection only means initialize_connection. | 17:25 |
| *** mtanino has joined #openstack-cinder | 17:25 | |
| DuncanT | scottda: Changing the meaning of initialize_connection is hard... creating a new API that does some subset of the work of initialize_connection, then killing initialize_connection is far easier | 17:27 |
| *** jordanP has joined #openstack-cinder | 17:27 | |
| j_king | hi DuncanT | 17:28 |
| *** vilobhmm has quit IRC | 17:28 | |
| *** vilobhmm has joined #openstack-cinder | 17:28 | |
| hemna | scottda, so, I think if I can get patch in that updates the create_export call to add the connector, then drivers can eventually move to use that | 17:28 |
| *** sghanekar_ has quit IRC | 17:28 | |
| hemna | and initialize_connection should work as nova intends | 17:28 |
| hemna | re: look at the lvm and solidfire drivers | 17:29 |
| hemna | initialize_connection in those is nothing more than information gathering | 17:29 |
| hemna | vs. almost every other driver | 17:29 |
| hemna | that do the target creation/exporting inside initialize_connection today. | 17:29 |
| hemna | scottda, if I do that, then we don't need to change nova calling initialize_connection | 17:30 |
| hemna | it'll just work. | 17:30 |
| hemna | and as a side note to this, we need to do a better job in Cinder of documenting what each of the driver functions are supposed to do. | 17:30 |
| *** dustins has quit IRC | 17:30 | |
| hemna | as many aren't following it, and core reviewers are also not catching it (re: create_export vs. initialize_connection), myself included. | 17:31 |
| openstackgerrit | Eric Brown proposed openstack/cinder: Add bandit for security static analysis testing https://review.openstack.org/179568 | 17:31 |
| rhe00_ | Does anyone here know if there is a way to tell tempest to fail the whole suite immediately if a sub test fails? That would speed up response time and simplify debugging, not having to wait for tempest to run through the whole test suite. | 17:31 |
| hemna | rhe00_, ^C ? | 17:31 |
| rhe00_ | yes, when running manually. I would like to have it for the CI as well. :) | 17:32 |
| jgriffith | rhe00_: there's a "stop on first failure" option IIRC, but I'm not sure how to use it or exactly how it works. | 17:32 |
| jgriffith | rhe00_: maybe check out the docs on ostestr | 17:32 |
| rhe00_ | jgriffith: Ok, I will take a look there. Thanks! | 17:32 |
| *** dims_ has joined #openstack-cinder | 17:32 | |
| *** dustins has joined #openstack-cinder | 17:33 | |
| jgriffith | rhe00_: https://wiki.openstack.org/wiki/Testr#How_can_I_exit_a_test_run_after_the_first_failure.3F | 17:33 |
| *** annegentle has quit IRC | 17:33 | |
| jgriffith | rhe00_: that's what I was thinking of... should do what you want | 17:33 |
| *** casusbelli has left #openstack-cinder | 17:36 | |
| *** dims_ has quit IRC | 17:39 | |
| scottda | DuncanT: I'm not saying to change the meaning of initialize_connection, just to make sure Nova only uses it as intended... | 17:39 |
| scottda | not, for example, to return a dict of connector info | 17:40 |
| *** annashen has quit IRC | 17:40 | |
| scottda | hemna: I'm hopeful that documenting the driver functions will be a part of this API effort...in fact, I'll add that to the goals now. At least for interactions with Nova. | 17:41 |
| *** annashen has joined #openstack-cinder | 17:41 | |
| *** rushil has joined #openstack-cinder | 17:41 | |
| hemna | scottda, ok that'd be awesome. | 17:43 |
| *** zhenguo has quit IRC | 17:43 | |
| hemna | scottda, I'm goig to try and get a cinder-spec put together today | 17:44 |
| scottda | swell | 17:44 |
| *** dustins has quit IRC | 17:46 | |
| *** rushil has quit IRC | 17:46 | |
| *** krtaylor has joined #openstack-cinder | 17:47 | |
| *** rushil has joined #openstack-cinder | 17:47 | |
| *** annashen has quit IRC | 17:47 | |
| *** rushil has quit IRC | 17:49 | |
| *** edmondsw has joined #openstack-cinder | 17:49 | |
| *** merooney has quit IRC | 17:49 | |
| *** annashen has joined #openstack-cinder | 17:49 | |
| *** edmondsw has quit IRC | 17:51 | |
| *** edmondsw has joined #openstack-cinder | 17:51 | |
| *** dims_ has joined #openstack-cinder | 17:52 | |
| *** dims_ has quit IRC | 17:53 | |
| *** sghanekar_ has joined #openstack-cinder | 17:55 | |
| *** dims_ has joined #openstack-cinder | 17:57 | |
| *** merooney has joined #openstack-cinder | 17:59 | |
| *** divab has joined #openstack-cinder | 17:59 | |
| *** DericHorn-HP has quit IRC | 18:04 | |
| *** bswartz has quit IRC | 18:05 | |
| *** angela-s has joined #openstack-cinder | 18:06 | |
| *** patrickeast has joined #openstack-cinder | 18:07 | |
| *** merooney has quit IRC | 18:08 | |
| *** abehl has quit IRC | 18:09 | |
| *** xek has quit IRC | 18:10 | |
| *** DTadrzak has quit IRC | 18:10 | |
| *** DTadrzak has joined #openstack-cinder | 18:12 | |
| *** xek has joined #openstack-cinder | 18:12 | |
| *** merooney has joined #openstack-cinder | 18:20 | |
| *** rushil has joined #openstack-cinder | 18:21 | |
| Swanson | hemna: What I've added to the dell driver is a check to see if I've already mapped the volume to the given server. If so I just return info. Is that peachy for liberty? | 18:22 |
| *** mtanino has quit IRC | 18:22 | |
| Swanson | hemna: initialize_connection is the context for that statement. | 18:22 |
| hemna | Swanson, yah, I'm not looking very deep into what I see | 18:23 |
| hemna | but I'm just pointing out potential pain spots for individual drivers | 18:23 |
| hemna | because almost EVERY driver is doing nothing in create_export, and doing real work against the array in initialize_connection | 18:23 |
| hemna | which is 180 degrees contrary to jgriffith's blog post. | 18:23 |
| hemna | there are a few that are doing things as documented in the post. | 18:24 |
| hemna | I suppose I should be noting those as well in the etherpad, but I'm lazy | 18:24 |
| hemna | I'm almost done going through them | 18:24 |
| Swanson | hemna: Oh, yeah. My driver doesn't do that NOW. The patch I'm working on includes that as a fix. So it will map things and give info as appropriate. Assuming create_export is called before initialize_connection it should behave as nova expects. | 18:24 |
| hemna | Swanson, create_export is indeed called prior to initialize_connection. but it doesn't contain the connector. | 18:25 |
| hemna | so, I'm going to push up a cinder-spec and patch that changes create_export to add the connector | 18:25 |
| hemna | then drivers can have the ability to do the actual exporting in create_export, and do the right thing in initialize_connection. | 18:26 |
| *** porrua has joined #openstack-cinder | 18:26 | |
| hemna | once everyone does that, then live migration will at least have a chance of working for everyone and not leave orphaned volumes behind on the source compute host. | 18:26 |
| angela-s | thingee: ping | 18:26 |
| *** porrua has left #openstack-cinder | 18:26 | |
| Swanson | hemna: I think my patch achieves the same thing. If your patch is in soon I'll move to splitting things between create_export and initialize_connection. | 18:27 |
| hemna | Swanson, so my patch will simply enable the passing of the connector to create_export. | 18:27 |
| hemna | and add that param to every existing definition of create_export in drivers. I can't fix every driver's internals of create_export. | 18:28 |
| hemna | but this will at least enable them to do the right thing. | 18:28 |
| hemna | maybe it's worth filing bugs against all these drivers? | 18:28 |
| Swanson | hemna: yeah, that's why I didn't connect the volume up in create_export. Not enough information to do anything. | 18:28 |
| hemna | not sure what to do about that. | 18:28 |
| *** dustins has joined #openstack-cinder | 18:29 | |
| Swanson | At some point they'll be hit with live migration failing. Once they look at that it should be an easy fix. Ignoring the 3,000 mock test changes. | 18:30 |
| hemna | heh :) | 18:30 |
| hemna | yup. | 18:30 |
| *** erlon has quit IRC | 18:30 | |
| *** Yogi1 has quit IRC | 18:34 | |
| *** mutoulbj has quit IRC | 18:35 | |
| *** merooney has quit IRC | 18:38 | |
| *** jwang has joined #openstack-cinder | 18:38 | |
| hemna | ok phew, I think I'm done | 18:45 |
| hemna | so 28 drivers have issues in initialize_connection | 18:45 |
| patrickeast | :o | 18:45 |
| hemna | out of...what 40 ? | 18:45 |
| patrickeast | is pure on the list? | 18:45 |
| hemna | yup | 18:46 |
| hemna | https://etherpad.openstack.org/p/CinderNovaAPI | 18:46 |
| hemna | patrickeast, line 279 | 18:46 |
| hemna | so these are potential issues | 18:46 |
| *** Yogi1 has joined #openstack-cinder | 18:46 | |
| hemna | as I can't look at the internals of every call to see or not | 18:46 |
| patrickeast | ohh interesting | 18:46 |
| patrickeast | yea | 18:46 |
| hemna | but almost all drivers do their work of creating the lun exports inside of initialize_connection | 18:46 |
| hemna | and not inside of create_export | 18:46 |
| *** Yogi11 has joined #openstack-cinder | 18:47 | |
| hemna | some drivers seem to do checks for existing exports and are probably o | 18:47 |
| hemna | ok | 18:47 |
| hemna | so I'd say live migration only works for a very small handful of Cinder backends. | 18:47 |
| smcginnis | hemna: Thanks for doing all the research on that. | 18:47 |
| *** mtanino has joined #openstack-cinder | 18:47 | |
| hemna | I'm not saying these drivers are broken, but potentially broken. | 18:48 |
| patrickeast | so the issue is that initialize_connection gets called multiple times for the same volume+host combo? | 18:48 |
| hemna | but what's clear is that almost everyone is confused about the purpose of initialize_connection. | 18:48 |
| patrickeast | err initiatior* | 18:48 |
| hemna | patrickeast, yes | 18:48 |
| hemna | multiple times | 18:48 |
| patrickeast | i know i don’t really understand what the export methods are for | 18:48 |
| *** mutoulbj has joined #openstack-cinder | 18:48 | |
| patrickeast | ok cool | 18:49 |
| hemna | nova simply wants the target information | 18:49 |
| smcginnis | I had asked way back what the difference was and no one at the time could give a clear answer. | 18:49 |
| *** bswartz has joined #openstack-cinder | 18:49 | |
| hemna | and isn't calling it to do an attachment/export | 18:49 |
| *** merooney has joined #openstack-cinder | 18:49 | |
| hemna | there is yet another problem in nova in post_live_migration that is probably broken for some cinder backends as well | 18:49 |
| hemna | nova is currently making an assumption that the connection information for a volume on compute host1 is exactly the same for compute host2 | 18:50 |
| hemna | including target_portal, iqn, lun id (and WWN for FC) | 18:50 |
| hemna | which is certainly not the case for all cinder backends | 18:50 |
| *** Yogi1 has quit IRC | 18:50 | |
| patrickeast | haha yea thats a scary assumption | 18:51 |
| hemna | what happens is that once nove brings up the VM on the destination compute node, it saves the connection information for the volume attachment into the BDM | 18:51 |
| hemna | then uses that same information to disconnect the volume from the source host. | 18:51 |
| *** dims_ has quit IRC | 18:51 | |
| hemna | if any aspect of the volume connection is different, then nova could remove the wrong volume from the source host. | 18:51 |
| hemna | or not remove anything, leaving orphaned volumes. | 18:52 |
| patrickeast | wow | 18:52 |
| hemna | which will create corruption in the future. | 18:52 |
| patrickeast | thats seriously bad | 18:52 |
| *** dims_ has joined #openstack-cinder | 18:52 | |
| hemna | yah :( | 18:52 |
| hemna | I'm having my coworker write up that bug and post a patch fix for it asap. | 18:52 |
| hemna | it's a fairly simple fix. | 18:52 |
| hemna | but.....requires calling initialize_connection again :) | 18:52 |
| hemna | so..... | 18:53 |
| patrickeast | i have tasks to test out migration stuff with our backend in my backlog somewhere…. guess i need to move them up a bit | 18:53 |
| hemna | initialize_connection needs to get fixed for cinder drivers that want live migration to work. | 18:53 |
| hemna | patrickeast, https://etherpad.openstack.org/p/live-migration-results | 18:53 |
| hemna | we have that etherpad going on | 18:53 |
| *** mutoulbj has quit IRC | 18:53 | |
| hemna | from the our testing with our 3 drivers + lvm | 18:53 |
| patrickeast | somewhat related, i need to enabled multiattach for my driver | 18:53 |
| patrickeast | is there a way to test that easily? | 18:53 |
| hemna | we are also putting together some scripts to help test verification | 18:53 |
| hemna | patrickeast, not really. | 18:54 |
| hemna | patrickeast, there are 5+ patches in nova still outstanding | 18:54 |
| patrickeast | i think it should pretty much just work if i enable the flag | 18:54 |
| hemna | one of them just got out of WIP | 18:54 |
| patrickeast | but i want to make sure | 18:54 |
| hemna | so I don't know what the state of Nova is really. | 18:54 |
| patrickeast | hmm ok maybe ill hold off a while then | 18:54 |
| patrickeast | or try and cobble together a nova branch with all the patches | 18:54 |
| hemna | there will probably be a whole new set of issues come up with multi-attach and live migration. | 18:55 |
| hemna | but we won't know until its enabled in nova. | 18:55 |
| patrickeast | makes sense | 18:55 |
| patrickeast | is it looking like those features will make it into L for nova? | 18:55 |
| hemna | at this point, I'm just trying to get live migration to a point between nova and cinder in general working. | 18:55 |
| hemna | and leave it up to driver writers to fix their own issues. | 18:56 |
| hemna | but at least raising awareness is step 1 | 18:56 |
| patrickeast | haha, thats really as much as you can do | 18:56 |
| *** shyama has quit IRC | 18:56 | |
| *** dims_ has quit IRC | 18:56 | |
| patrickeast | alright well i’m off to lunch, i’ll put up a patch to fix pure’s initialize_connection later today | 18:57 |
| *** annashen has quit IRC | 19:00 | |
| *** liuxg has joined #openstack-cinder | 19:01 | |
| *** liuxg has quit IRC | 19:06 | |
| hemna | ok I'll finish up the cinder-spec after lunch...bbiab | 19:08 |
| *** belmoreira has joined #openstack-cinder | 19:18 | |
| *** ganso_ has quit IRC | 19:18 | |
| *** lpetrut has joined #openstack-cinder | 19:20 | |
| *** annegentle has joined #openstack-cinder | 19:21 | |
| *** divab has quit IRC | 19:21 | |
| *** divab has joined #openstack-cinder | 19:21 | |
| openstackgerrit | Gaurang Tapase proposed openstack/cinder: Adding NFS support to the GPFS Driver https://review.openstack.org/197884 | 19:22 |
| *** divab has quit IRC | 19:27 | |
| *** lpabon has quit IRC | 19:29 | |
| *** Apoorva has quit IRC | 19:32 | |
| *** annashen has joined #openstack-cinder | 19:33 | |
| *** ronis has quit IRC | 19:43 | |
| *** mylu has quit IRC | 19:43 | |
| *** harlowja has joined #openstack-cinder | 19:49 | |
| *** harlowja_ has quit IRC | 19:49 | |
| *** mylu has joined #openstack-cinder | 19:50 | |
| *** harlowja has quit IRC | 19:52 | |
| *** harlowja has joined #openstack-cinder | 19:53 | |
| *** Yogi11 has quit IRC | 19:57 | |
| *** Apoorva has joined #openstack-cinder | 20:03 | |
| *** Zanatoz has quit IRC | 20:06 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder-specs: Add Connector to create_export https://review.openstack.org/202765 | 20:07 |
| *** xyang1 has joined #openstack-cinder | 20:09 | |
| *** belmoreira has quit IRC | 20:11 | |
| *** salv-orlando has quit IRC | 20:11 | |
| *** DericHorn-HP has joined #openstack-cinder | 20:11 | |
| tsekiyama | Hi, is it intended for non-admin users to be allowed to get a result by "cinder show <volume-uuid-in-others-tenant>" ? | 20:12 |
| tsekiyama | They cannot list or delete volumes in not authorized tenant, but if they know the UUID, they can get the info. I wonder this is correct behavior | 20:15 |
| *** merooney has quit IRC | 20:15 | |
| patrickeast | tsekiyama: haha i noticed that too when playing with the image cache, there isn’t any check to restrict that from what I saw | 20:16 |
| patrickeast | at least once you are past the first api calls | 20:17 |
| Swanson | hemna: So what is remove_export for? | 20:17 |
| patrickeast | i didn’t look much higher up the stack | 20:17 |
| hemna | heh | 20:17 |
| hemna | Swanson, I was JUST looking at that, and I think the connector needs to be there as well. | 20:17 |
| *** jordanP has quit IRC | 20:17 | |
| Swanson | hemna: My first question is what is expected to happen in terminate_connection vs remove_export? | 20:18 |
| patrickeast | hemna: Swanson: is there documentation somewhere that explains what each of the methods are actually supposed to be doing? | 20:18 |
| hemna | Swanson, good question | 20:18 |
| hemna | I'm not sure it's really documented, which is part of the problem. | 20:18 |
| Swanson | patrickeast: this is the closest I know of ... http://j-griffith.blogspot.com/ | 20:18 |
| mriedem | anyone here work on the scality volume driver? | 20:19 |
| *** divab has joined #openstack-cinder | 20:20 | |
| hemna | so remove_export is called at the bottom end of detach_volume | 20:21 |
| hemna | in the volume manager | 20:21 |
| hemna | which is after terminate_connection | 20:22 |
| hemna | so the problem with waiting until remove_export to actual remove the export on the array, is that some volumes may show back up on the compute host. | 20:22 |
| hemna | between the time libvirt volume's disconnect_volume, then cinder's terminate_connection, then cinder's detach_volume. | 20:23 |
| Swanson | One wonders what it is for. | 20:23 |
| hemna | if a rescan happens between disconnect_volume and remove_export. | 20:23 |
| hemna | we'll get orphaned volumes on the host that had the volume. | 20:24 |
| Swanson | My code actually says " We do nothing here to match the nothing we do in create export." | 20:24 |
| hemna | lol | 20:24 |
| hemna | yah | 20:24 |
| patrickeast | haha | 20:24 |
| patrickeast | same | 20:24 |
| hemna | I haven't looked into how many driver do anything inside of remove_export | 20:24 |
| hemna | terminate_connection seems like the correct spot to do "remove the exporting of a volume" from the backend. | 20:25 |
| hemna | but then again I could be wrong. | 20:25 |
| hemna | it just seems that remove_export is way too late. | 20:25 |
| patrickeast | i think thats what most of them do anyways | 20:25 |
| hemna | have to look into that one. | 20:26 |
| hemna | but maybe we can get rid of it | 20:26 |
| hemna | dunno | 20:26 |
| hemna | but I'm not into boiling the ocean. | 20:26 |
| patrickeast | you dont want to refactor alllll the drivers? | 20:27 |
| *** e0ne has joined #openstack-cinder | 20:27 | |
| hemna | :P | 20:28 |
| hemna | I'm working on the patch to just add the connector to create_export. | 20:28 |
| Swanson | I just went through a ton and most do nothing. | 20:28 |
| *** annegentle has quit IRC | 20:28 | |
| hemna | basically touching every driver. | 20:28 |
| hemna | Swanson, that's what I had thought. | 20:29 |
| hemna | cool | 20:29 |
| hemna | Swanson, the solidfire driver does do work in remove_export | 20:30 |
| Swanson | lvm does something. Just not sure what. Calls the target driver remove export. I don't think this is used by backends that are separate from the server. | 20:30 |
| *** jasondotstar has joined #openstack-cinder | 20:30 | |
| hemna | it calls the target_driver.remove_export, which for iscsi does 'stuff' | 20:30 |
| *** jungleboyj has quit IRC | 20:30 | |
| hemna | heh | 20:30 |
| Swanson | Yep. Same for LVM. | 20:30 |
| hemna | jgriffith is probably the only person that understands it, since he wrote the target object stuff. | 20:31 |
| Swanson | Yep. | 20:31 |
| hemna | the ISCSITarget.remove_export looks like it does remove the export. | 20:31 |
| hemna | and ISCSITarget.terminate_connection does nothing | 20:32 |
| hemna | there is a race condition there I think w/ iscsi rescans in the call chain | 20:32 |
| hemna | going to have to research that one | 20:33 |
| hemna | we might be leaving orphaned volumes behind on the host. | 20:33 |
| *** akerr has quit IRC | 20:33 | |
| tsekiyama | ISCSITarget.terminate_connection does something (delete initiator wwn from the iSCSI target) if target_helper = "lioadm", IIRC. | 20:33 |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/volume/targets/iscsi.py#L306 | 20:34 |
| hemna | ISCSITarget.terminate_connection is only a pass | 20:34 |
| hemna | does nothing | 20:35 |
| jgriffith | hemna: it's only used by LIO | 20:35 |
| tsekiyama | that is overriden here https://github.com/openstack/cinder/blob/master/cinder/volume/targets/lio.py#L192 | 20:35 |
| jgriffith | hemna: it's special for LIO targets | 20:35 |
| hemna | jgriffith, what is the target object that solidfire uses? | 20:35 |
| jgriffith | hemna: we use our own | 20:36 |
| jgriffith | hemna: inherited from the base iscsi class | 20:36 |
| hemna | haven't actually looked in the solidfire terminate_connection code to see if it's calling a target or not ? | 20:36 |
| hemna | ok | 20:36 |
| hemna | I'm just trying to wrap my head around all of this | 20:36 |
| jgriffith | hemna: nah... SF doesn't work like that | 20:36 |
| jgriffith | hemna: we rely on things to be done from the initiator side | 20:36 |
| *** krtaylor has quit IRC | 20:36 | |
| hemna | ok, so I'll ignore remove_export for now | 20:37 |
| *** dims_ has joined #openstack-cinder | 20:37 | |
| jgriffith | hemna: the detach/remove export stuff is another ball of wax | 20:37 |
| *** piyanai has joined #openstack-cinder | 20:37 | |
| hemna | yah, I have to put my blinders on for now. | 20:37 |
| *** cdelatte has quit IRC | 20:37 | |
| *** delattec has quit IRC | 20:37 | |
| jgriffith | hemna: if you *need* it, just take a look at how like LIO uses it | 20:37 |
| jgriffith | hemna: they're the only ones I know of that need it | 20:37 |
| jgriffith | hemna: there may be drivers over-riding it internally as well.... | 20:38 |
| hemna | I'm just going through the drivers now and adding the connector to create_export | 20:38 |
| jgriffith | hemna: OH!! Cool | 20:38 |
| hemna | I have the spec up | 20:38 |
| hemna | and the BP | 20:38 |
| jgriffith | hemna: I actually started that, but wanted to jump out the window | 20:38 |
| hemna | I'm working on the code now | 20:38 |
| jgriffith | hemna: awesome | 20:38 |
| hemna | once I have a WIP up, I'm going to have my coworker, update our drivers to do the actual exporting in create_export, and change how our initialize_connection works. | 20:39 |
| *** thangp has quit IRC | 20:39 | |
| *** xyang1 has quit IRC | 20:39 | |
| jgriffith | hemna: this is one of those things that makes me say "all driver methods should have a kwargs" :) | 20:39 |
| hemna | this might help other see how to change theirs | 20:39 |
| hemna | :) | 20:39 |
| hemna | yah, that's the hammer approach | 20:39 |
| *** xyang1 has joined #openstack-cinder | 20:39 | |
| hemna | would be future proof though | 20:39 |
| jgriffith | hemna: FWIW, I think there's another way you can make that work | 20:40 |
| hemna | so, let me post up this WIP and see what folks think. | 20:40 |
| hemna | outside of this, I think we are in desperate need of a tutorial/HOWTO/documentation for driver writers | 20:41 |
| hemna | your blog post was great | 20:41 |
| guitarzan | kwargs all the things sounds like fun | 20:42 |
| guitarzan | sounds bad, but fun! | 20:42 |
| hemna | lets just kwargs everything. Imagine all the fun we'll have. | 20:42 |
| hemna | :P | 20:42 |
| *** salv-orlando has joined #openstack-cinder | 20:45 | |
| *** xyang has joined #openstack-cinder | 20:46 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Efficient volume copy for generic volume migration https://review.openstack.org/183701 | 20:51 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Adds framework for get_capabilities() feature https://review.openstack.org/201243 | 20:51 |
| *** rushil has quit IRC | 20:54 | |
| *** piyanai has quit IRC | 20:55 | |
| *** piyanai has joined #openstack-cinder | 20:59 | |
| *** ociuhandu has joined #openstack-cinder | 21:02 | |
| *** piyanai has quit IRC | 21:04 | |
| *** cdelatte has joined #openstack-cinder | 21:05 | |
| hemna | arg | 21:09 |
| hemna | what was the magic sauce again for running tox -epy27 cinder.tests.unit.my.individual.test ? | 21:09 |
| hemna | *sigh* | 21:09 |
| jgriffith | -- -n | 21:09 |
| hemna | it was something magic needed to make ostestr happy | 21:10 |
| jgriffith | hemna: ^^ Look up | 21:10 |
| hemna | jgriffith, thanks | 21:10 |
| *** Lee1092 has quit IRC | 21:15 | |
| *** e0ne has quit IRC | 21:16 | |
| *** mylu has quit IRC | 21:18 | |
| openstackgerrit | Tom Jose Kalapura proposed openstack/python-cinderclient: Add response message when volume delete https://review.openstack.org/202790 | 21:19 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/python-cinderclient: Implement reset-state for attach_status and migration_status https://review.openstack.org/164309 | 21:26 |
| openstackgerrit | Matt Riedemann proposed openstack/cinder: scality: add export and name keys to os-initialize_connection info https://review.openstack.org/202795 | 21:29 |
| *** kjnelson has quit IRC | 21:32 | |
| *** piyanai has joined #openstack-cinder | 21:33 | |
| *** krtaylor has joined #openstack-cinder | 21:36 | |
| *** mriedem has quit IRC | 21:38 | |
| *** boris-42 has joined #openstack-cinder | 21:42 | |
| *** edmondsw has quit IRC | 21:45 | |
| *** bnemec has quit IRC | 21:46 | |
| *** dustins has quit IRC | 21:47 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Don’t log warnings in Pure initialize_connection https://review.openstack.org/202805 | 21:48 |
| *** mutoulbj has joined #openstack-cinder | 21:51 | |
| *** sghanekar_ has quit IRC | 21:53 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add connector object to create_export https://review.openstack.org/202808 | 21:56 |
| *** mutoulbj has quit IRC | 21:56 | |
| *** annashen has quit IRC | 21:57 | |
| *** sghanekar_ has joined #openstack-cinder | 21:59 | |
| *** vilobhmm has quit IRC | 21:59 | |
| *** vilobhmm has joined #openstack-cinder | 22:00 | |
| *** DericHorn-HP has quit IRC | 22:02 | |
| Swanson | Look at the file count before you hit the all changes link. | 22:04 |
| *** ociuhandu has quit IRC | 22:06 | |
| *** gouthamr has quit IRC | 22:07 | |
| *** porrua has joined #openstack-cinder | 22:09 | |
| *** piyanai has quit IRC | 22:12 | |
| *** primechuck has quit IRC | 22:13 | |
| *** primechuck has joined #openstack-cinder | 22:15 | |
| *** jasondotstar has quit IRC | 22:15 | |
| *** primechuck has quit IRC | 22:16 | |
| *** mriedem has joined #openstack-cinder | 22:32 | |
| *** xyang1 has quit IRC | 22:33 | |
| *** porrua has quit IRC | 22:33 | |
| *** lpetrut has quit IRC | 22:37 | |
| *** IanGovett1 has quit IRC | 22:38 | |
| *** lpetrut has joined #openstack-cinder | 22:45 | |
| *** lcurtis has quit IRC | 22:54 | |
| *** vilobhmm has quit IRC | 22:57 | |
| *** gouthamr has joined #openstack-cinder | 22:57 | |
| *** vilobhmm has joined #openstack-cinder | 22:57 | |
| *** gouthamr has quit IRC | 22:59 | |
| *** gouthamr has joined #openstack-cinder | 23:00 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: FC discover existing devices for removal https://review.openstack.org/202826 | 23:13 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/201336 | 23:14 |
| *** daneyon_ has left #openstack-cinder | 23:15 | |
| *** salv-orlando has quit IRC | 23:16 | |
| *** akerr has joined #openstack-cinder | 23:16 | |
| *** crose has quit IRC | 23:17 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/200898 | 23:20 |
| anish | hemna: testcases | 23:20 |
| hemna | ok cool | 23:20 |
| *** zigo has quit IRC | 23:21 | |
| *** zigo has joined #openstack-cinder | 23:22 | |
| *** markstur is now known as markstur_away | 23:24 | |
| *** jasondotstar has joined #openstack-cinder | 23:25 | |
| *** hemna is now known as hemnafk | 23:25 | |
| *** markstur_away has left #openstack-cinder | 23:27 | |
| *** sghanekar_ has quit IRC | 23:30 | |
| *** openstackstatus has joined #openstack-cinder | 23:33 | |
| *** ChanServ sets mode: +v openstackstatus | 23:33 | |
| *** lpetrut has quit IRC | 23:34 | |
| *** erlon has joined #openstack-cinder | 23:39 | |
| *** jaypipes has quit IRC | 23:50 | |
| *** barra204 has joined #openstack-cinder | 23:54 | |
| *** mriedem1 has joined #openstack-cinder | 23:55 | |
| *** mriedem has quit IRC | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!