| *** s-shiono has joined #openstack-cinder | 00:00 | |
| *** ntpttr_ has quit IRC | 00:04 | |
| *** viks has quit IRC | 00:06 | |
| *** esker has quit IRC | 00:08 | |
| *** ntpttr_ has joined #openstack-cinder | 00:09 | |
| *** yangyapeng has joined #openstack-cinder | 00:19 | |
| *** yangyapeng has quit IRC | 00:20 | |
| *** mayela has quit IRC | 00:22 | |
| *** lixiaoy1 has joined #openstack-cinder | 00:23 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder master: Make cinder-manage online migrations more verbose https://review.openstack.org/438754 | 00:23 |
|---|---|---|
| *** dave-mccowan has quit IRC | 00:29 | |
| *** Julien-zte has joined #openstack-cinder | 00:32 | |
| *** tobasco has quit IRC | 00:44 | |
| *** Shunli has joined #openstack-cinder | 00:44 | |
| *** tobasco has joined #openstack-cinder | 00:44 | |
| *** dave-mccowan has joined #openstack-cinder | 00:48 | |
| *** wanghao has joined #openstack-cinder | 00:49 | |
| *** dave-mcc_ has joined #openstack-cinder | 00:55 | |
| *** jichen has joined #openstack-cinder | 00:56 | |
| *** dave-mccowan has quit IRC | 00:57 | |
| *** winston-2 has quit IRC | 00:59 | |
| *** winston-1 has quit IRC | 01:05 | |
| *** dave-mccowan has joined #openstack-cinder | 01:09 | |
| *** wxy has quit IRC | 01:10 | |
| *** wxy has joined #openstack-cinder | 01:10 | |
| *** dave-mcc_ has quit IRC | 01:11 | |
| *** xinli has joined #openstack-cinder | 01:15 | |
| *** winston-d has joined #openstack-cinder | 01:16 | |
| *** huanxuan has joined #openstack-cinder | 01:19 | |
| *** Julien-zte has quit IRC | 01:25 | |
| *** zhurong has joined #openstack-cinder | 01:26 | |
| *** Julien-zte has joined #openstack-cinder | 01:27 | |
| *** kaisers has joined #openstack-cinder | 01:31 | |
| *** zhangguoqing has joined #openstack-cinder | 01:32 | |
| openstackgerrit | Yuki Kasahara proposed openstack/cinder master: Sheepdog: fix conflict of temporary snapshot name https://review.openstack.org/449386 | 01:33 |
| *** kaisers1 has quit IRC | 01:33 | |
| *** jamielennox is now known as jamielennox|away | 01:39 | |
| *** wanghao has quit IRC | 01:41 | |
| *** wanghao has joined #openstack-cinder | 01:42 | |
| *** zul has quit IRC | 01:43 | |
| *** zul has joined #openstack-cinder | 01:45 | |
| *** Apoorva_ has quit IRC | 01:47 | |
| *** jamielennox|away is now known as jamielennox | 01:48 | |
| *** gcb has joined #openstack-cinder | 01:48 | |
| openstackgerrit | jiamin proposed openstack/cinder master: DS8K driver: change the design of CG https://review.openstack.org/419321 | 01:50 |
| openstackgerrit | christopher.uhler proposed openstack/os-brick master: os-brick connector for Veritas HyperScale https://review.openstack.org/442754 | 01:58 |
| *** zhangguoqing has quit IRC | 02:01 | |
| *** zhangguoqing has joined #openstack-cinder | 02:04 | |
| *** kukacz has quit IRC | 02:20 | |
| *** wanghao_ has joined #openstack-cinder | 02:21 | |
| *** wanghao has quit IRC | 02:25 | |
| *** yangyapeng has joined #openstack-cinder | 02:25 | |
| openstackgerrit | Kushal Wathore proposed openstack/cinder master: HPE 3PAR: ISCSI/FC driver – code refactoring https://review.openstack.org/446475 | 02:28 |
| *** namnh has joined #openstack-cinder | 02:33 | |
| openstackgerrit | Kushal Wathore proposed openstack/cinder master: HPE 3PAR: Handle manage and unmanage hosts present https://review.openstack.org/409124 | 02:33 |
| *** gkadam has joined #openstack-cinder | 02:35 | |
| *** tbarron has quit IRC | 02:38 | |
| *** knikolla has quit IRC | 02:38 | |
| *** tbarron has joined #openstack-cinder | 02:48 | |
| *** knikolla has joined #openstack-cinder | 02:50 | |
| *** zhangguoqing has quit IRC | 02:51 | |
| *** zhangguoqing has joined #openstack-cinder | 02:51 | |
| *** zhangguoqing has quit IRC | 02:56 | |
| *** kaisers_ has joined #openstack-cinder | 02:57 | |
| *** gcb has quit IRC | 02:58 | |
| *** gcb has joined #openstack-cinder | 02:59 | |
| *** kaisers_ has quit IRC | 03:02 | |
| *** dave-mccowan has quit IRC | 03:08 | |
| *** prashkre has joined #openstack-cinder | 03:26 | |
| openstackgerrit | wzyuliyang proposed openstack/cinder master: fix the count error of chunked based volume backup. the current result number of object should consider the sha256file too ,so we need to increase the object_count https://review.openstack.org/449419 | 03:27 |
| *** zhangguoqing has joined #openstack-cinder | 03:28 | |
| *** zhangguoqing has quit IRC | 03:30 | |
| *** zhangguoqing has joined #openstack-cinder | 03:31 | |
| *** edmondsw has quit IRC | 03:32 | |
| *** zhangguoqing has quit IRC | 03:32 | |
| *** Julien-zte has quit IRC | 03:33 | |
| *** zhangguoqing has joined #openstack-cinder | 03:45 | |
| *** zhangguoqing has quit IRC | 03:49 | |
| *** edmondsw has joined #openstack-cinder | 03:59 | |
| *** edmondsw has quit IRC | 04:04 | |
| *** zhurong has quit IRC | 04:07 | |
| *** zhangguoqing has joined #openstack-cinder | 04:11 | |
| *** xinli has quit IRC | 04:11 | |
| *** prashkre has quit IRC | 04:19 | |
| *** links has joined #openstack-cinder | 04:20 | |
| *** zhangguoqing has quit IRC | 04:26 | |
| *** prashkre has joined #openstack-cinder | 04:34 | |
| *** yangyapeng has quit IRC | 04:40 | |
| *** kaisers_ has joined #openstack-cinder | 04:42 | |
| *** kaisers_ has quit IRC | 04:42 | |
| *** kaisers_ has joined #openstack-cinder | 04:42 | |
| *** jdurgin has joined #openstack-cinder | 04:44 | |
| *** namnh has quit IRC | 04:47 | |
| *** zhangguoqing has joined #openstack-cinder | 04:48 | |
| *** pdeore has joined #openstack-cinder | 04:49 | |
| *** Julien-zte has joined #openstack-cinder | 04:53 | |
| *** shausy has joined #openstack-cinder | 04:55 | |
| *** kaisers_ has quit IRC | 04:59 | |
| openstackgerrit | junboli proposed openstack/python-cinderclient master: Remove log translations https://review.openstack.org/449429 | 05:02 |
| *** psachin has joined #openstack-cinder | 05:07 | |
| *** pdeore has quit IRC | 05:09 | |
| *** jdurgin has quit IRC | 05:10 | |
| openstackgerrit | jiamin proposed openstack/cinder master: DS8K driver: change the design of CG https://review.openstack.org/419321 | 05:14 |
| *** zhurong has joined #openstack-cinder | 05:16 | |
| *** pgadiya has joined #openstack-cinder | 05:18 | |
| *** prashkre has quit IRC | 05:22 | |
| *** prashkre has joined #openstack-cinder | 05:22 | |
| *** lpetrut has joined #openstack-cinder | 05:29 | |
| *** pdeore has joined #openstack-cinder | 05:31 | |
| *** adisky_ has joined #openstack-cinder | 05:32 | |
| *** jiaopengju has joined #openstack-cinder | 05:37 | |
| *** Jack_Iv has joined #openstack-cinder | 05:52 | |
| *** hidekazu has joined #openstack-cinder | 05:58 | |
| hidekazu | hello, Could anyone teach me where i can know cinder notification list, create_volume, delete_volume etc ? | 06:01 |
| *** Julien-zte has quit IRC | 06:05 | |
| *** Jack_Iv has quit IRC | 06:08 | |
| tommylikehu | hidekazu, what do you mean by notification list? | 06:22 |
| hidekazu | tommylikehu: i want to consume cinder notification. | 06:23 |
| tommylikehu | user messages? | 06:24 |
| hidekazu | tommylikehu: watcher project has list : https://docs.openstack.org/developer/watcher/dev/notifications.html | 06:24 |
| *** imansari has joined #openstack-cinder | 06:25 | |
| *** gcb has quit IRC | 06:28 | |
| hidekazu | notify_about_volume_usage method in cinder.volume.utils module may emit volume usage notification. | 06:30 |
| hidekazu | investigation will take long time. | 06:30 |
| tommylikehu | hidekazu: I found this: https://blueprints.launchpad.net/cinder/+spec/cinder-notifications | 06:31 |
| hidekazu | tommylikehu: thanks, but this is only code, bacause old days cinder pj does not have specs. | 06:34 |
| *** lpetrut has quit IRC | 06:34 | |
| *** lpetrut has joined #openstack-cinder | 06:35 | |
| tommylikehu | hidekazu: yes,this one is little older. hope some one else could help us :) | 06:36 |
| hidekazu | tommylikehu: i will analyze only i need. ;) | 06:38 |
| hidekazu | tommylikehu: thanks helping me :) | 06:39 |
| *** gouthamr has joined #openstack-cinder | 06:44 | |
| *** lpetrut has quit IRC | 06:44 | |
| *** gcb has joined #openstack-cinder | 06:45 | |
| *** mvk has quit IRC | 06:47 | |
| *** Jack_Iv has joined #openstack-cinder | 06:49 | |
| *** ChubYann has quit IRC | 06:54 | |
| *** lpetrut has joined #openstack-cinder | 06:57 | |
| *** kaisers_ has joined #openstack-cinder | 06:59 | |
| *** edmondsw has joined #openstack-cinder | 06:59 | |
| openstackgerrit | Merged openstack/cinder master: Update db from drivers in default groups methods https://review.openstack.org/447681 | 06:59 |
| *** Julien-zte has joined #openstack-cinder | 07:00 | |
| *** edmondsw has quit IRC | 07:04 | |
| *** kaisers_ has quit IRC | 07:04 | |
| *** lpetrut has quit IRC | 07:05 | |
| openstackgerrit | junboli proposed openstack/python-cinderclient master: Remove log translations https://review.openstack.org/449429 | 07:06 |
| *** zhangguoqing has quit IRC | 07:10 | |
| *** gouthamr has quit IRC | 07:12 | |
| *** nkrinner_afk is now known as nkrinner | 07:12 | |
| *** tommylikehu_ has joined #openstack-cinder | 07:15 | |
| openstackgerrit | Jeremy Zhang proposed openstack/os-brick master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/449449 | 07:15 |
| *** psachin has quit IRC | 07:18 | |
| *** psachin has joined #openstack-cinder | 07:18 | |
| *** tommylikehu_ has quit IRC | 07:20 | |
| *** lpetrut has joined #openstack-cinder | 07:21 | |
| *** zhangguoqing has joined #openstack-cinder | 07:30 | |
| *** tesseract has joined #openstack-cinder | 07:31 | |
| *** Jack_Iv has quit IRC | 07:31 | |
| *** lpetrut has quit IRC | 07:35 | |
| *** Jack_Iv has joined #openstack-cinder | 07:36 | |
| *** Jack_Iv has quit IRC | 07:40 | |
| *** lixiaoy1 has quit IRC | 07:50 | |
| *** Jack_Iv has joined #openstack-cinder | 07:54 | |
| *** yangyapeng has joined #openstack-cinder | 07:56 | |
| *** prashkre has quit IRC | 07:57 | |
| *** prashkre has joined #openstack-cinder | 07:58 | |
| *** belmoreira has joined #openstack-cinder | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:01 | |
| *** Julien-zte has quit IRC | 08:04 | |
| *** kongwei has joined #openstack-cinder | 08:06 | |
| *** mriedem has quit IRC | 08:16 | |
| *** kongwei has quit IRC | 08:16 | |
| *** Julien-zte has joined #openstack-cinder | 08:18 | |
| *** mlakat has joined #openstack-cinder | 08:23 | |
| *** prashkre has quit IRC | 08:23 | |
| *** prashkre has joined #openstack-cinder | 08:24 | |
| *** aarefiev_pto is now known as aarefiev | 08:42 | |
| *** Jack_Iv has quit IRC | 08:58 | |
| *** tommylikehu_ has joined #openstack-cinder | 09:02 | |
| *** tommylikehu_ has quit IRC | 09:02 | |
| *** tommylikehu_ has joined #openstack-cinder | 09:06 | |
| *** s-shiono has quit IRC | 09:09 | |
| *** tommylikehu_ has quit IRC | 09:11 | |
| *** tommylikehu_ has joined #openstack-cinder | 09:11 | |
| *** tommylikehu_ has quit IRC | 09:11 | |
| *** Jack_Iv has joined #openstack-cinder | 09:16 | |
| *** tommylikehu_ has joined #openstack-cinder | 09:18 | |
| *** tommylikehu_ has quit IRC | 09:18 | |
| *** tommylikehu_ has joined #openstack-cinder | 09:19 | |
| *** tommylikehu_ has quit IRC | 09:20 | |
| *** wanghao_ has quit IRC | 09:20 | |
| *** pbandark has joined #openstack-cinder | 09:29 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: [Devref] Refactor attach/detach v2 devref https://review.openstack.org/449526 | 09:33 |
| *** Julien-zte has quit IRC | 09:37 | |
| *** prashkre has quit IRC | 09:40 | |
| *** prashkre has joined #openstack-cinder | 09:42 | |
| *** prashkre_ has joined #openstack-cinder | 09:43 | |
| *** prashkre has quit IRC | 09:43 | |
| *** lpetrut has joined #openstack-cinder | 09:45 | |
| *** Shunli has quit IRC | 09:46 | |
| *** prashkre_ has quit IRC | 09:46 | |
| *** lpetrut has quit IRC | 09:46 | |
| *** prashkre has joined #openstack-cinder | 09:47 | |
| *** lpetrut has joined #openstack-cinder | 09:49 | |
| *** prashkre has quit IRC | 09:49 | |
| *** prashkre has joined #openstack-cinder | 09:50 | |
| *** prashkre has quit IRC | 09:50 | |
| *** gkadam has quit IRC | 09:51 | |
| *** prashkre has joined #openstack-cinder | 09:52 | |
| *** e0ne has joined #openstack-cinder | 09:53 | |
| *** prashkre_ has joined #openstack-cinder | 09:56 | |
| *** mvk has joined #openstack-cinder | 09:57 | |
| *** prashkre has quit IRC | 09:58 | |
| *** kaisers_ has joined #openstack-cinder | 10:00 | |
| *** kaisers_ has quit IRC | 10:04 | |
| *** pcaruana has joined #openstack-cinder | 10:05 | |
| *** ociuhandu has joined #openstack-cinder | 10:08 | |
| *** jichen has quit IRC | 10:09 | |
| *** zhangguoqing has quit IRC | 10:10 | |
| *** zhangguoqing has joined #openstack-cinder | 10:10 | |
| *** shausy has quit IRC | 10:18 | |
| *** prashkre__ has joined #openstack-cinder | 10:23 | |
| *** prashkre_ has quit IRC | 10:24 | |
| *** prashkre_ has joined #openstack-cinder | 10:24 | |
| *** prashkre__ has quit IRC | 10:28 | |
| *** jiaopengju has quit IRC | 10:29 | |
| *** hidekazu has quit IRC | 10:30 | |
| openstackgerrit | Silvan Kaiser proposed openstack/cinder master: Reverts quobyte_volume_url option back to type StrOpt https://review.openstack.org/449553 | 10:30 |
| *** pbandark has quit IRC | 10:31 | |
| *** edmondsw has joined #openstack-cinder | 10:35 | |
| *** nicolasbock has joined #openstack-cinder | 10:38 | |
| *** zhurong has quit IRC | 10:40 | |
| *** edmondsw has quit IRC | 10:40 | |
| *** pbandark has joined #openstack-cinder | 10:44 | |
| *** gcb has quit IRC | 10:56 | |
| *** zhangguoqing has quit IRC | 10:56 | |
| *** prashkre__ has joined #openstack-cinder | 11:05 | |
| *** Dinesh_Bhor has quit IRC | 11:07 | |
| *** prashkre_ has quit IRC | 11:08 | |
| *** e0ne has quit IRC | 11:10 | |
| *** e0ne has joined #openstack-cinder | 11:10 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: [Devref] Refactor attach/detach v2 devref https://review.openstack.org/449526 | 11:12 |
| *** prashkre__ has quit IRC | 11:14 | |
| *** prashkre has joined #openstack-cinder | 11:15 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 11:20 | |
| *** tommylikehu_ has joined #openstack-cinder | 11:21 | |
| *** sdague has joined #openstack-cinder | 11:21 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/python-cinderclient master: Fix noauth support https://review.openstack.org/427143 | 11:25 |
| *** tommylikehu_ has quit IRC | 11:27 | |
| *** edmondsw has joined #openstack-cinder | 11:32 | |
| *** pgadiya has quit IRC | 11:38 | |
| *** abishop has joined #openstack-cinder | 11:46 | |
| *** abishop has quit IRC | 11:48 | |
| *** abishop has joined #openstack-cinder | 11:48 | |
| *** abishop has quit IRC | 11:49 | |
| *** abishop has joined #openstack-cinder | 11:50 | |
| *** abishop has quit IRC | 11:51 | |
| *** abishop has joined #openstack-cinder | 11:51 | |
| *** abishop has quit IRC | 11:52 | |
| *** prashkre has quit IRC | 11:52 | |
| *** prashkre has joined #openstack-cinder | 11:52 | |
| *** abishop has joined #openstack-cinder | 11:52 | |
| *** pdeore has quit IRC | 11:53 | |
| *** abishop has quit IRC | 12:00 | |
| *** kaisers_ has joined #openstack-cinder | 12:00 | |
| *** abishop has joined #openstack-cinder | 12:02 | |
| *** links has quit IRC | 12:02 | |
| *** kaisers_ has quit IRC | 12:05 | |
| *** links has joined #openstack-cinder | 12:05 | |
| *** liverpooler has quit IRC | 12:26 | |
| *** liverpooler has joined #openstack-cinder | 12:26 | |
| *** yangyapeng has quit IRC | 12:31 | |
| *** Jack_Iv has quit IRC | 12:37 | |
| *** Julien-zte has joined #openstack-cinder | 12:41 | |
| *** jdurgin has joined #openstack-cinder | 12:42 | |
| tommylikehu | hey smcginnis | 12:46 |
| *** prashkre has quit IRC | 12:48 | |
| *** lamt has joined #openstack-cinder | 12:52 | |
| *** lamt has quit IRC | 12:52 | |
| smcginnis | tommylikehu: hi | 12:54 |
| tommylikehu | smcginnis, Thanks two quetions about cinder's new attachment API | 12:54 |
| tommylikehu | when creating new attachment the volume id and instance id are both required, is that correct? | 12:55 |
| *** david-lyle has quit IRC | 12:55 | |
| *** Jack_Iv has joined #openstack-cinder | 12:55 | |
| smcginnis | tommylikehu: If I remember correct, yes. | 12:56 |
| *** lamt has joined #openstack-cinder | 12:56 | |
| *** xyang1 has joined #openstack-cinder | 12:56 | |
| tommylikehu | but In our cinder client, the instance-id is an optional argument. why is that | 12:56 |
| smcginnis | Oh, I think I was wrong. It can also be used to "reserve" a volume. But I think jgriffith can give the full detail better. | 12:59 |
| tommylikehu | @smcginnis, I am afraid we have a inconsistent between server and client | 13:00 |
| tommylikehu | here: https://github.com/openstack/cinder/blob/master/cinder/api/v3/attachments.py#L151 | 13:02 |
| jgriffith | tommylikehu as smcginnis pointed out you can use it as a reserve only (only take the volumeID) or a reserve and intialize which takes the connector info | 13:03 |
| *** dustins has joined #openstack-cinder | 13:04 | |
| jgriffith | tommylikehu I'm unclear on the inconsistency... what did I screw up? :) | 13:04 |
| jgriffith | https://github.com/openstack/python-cinderclient/blob/master/cinderclient/v3/attachments.py | 13:05 |
| jgriffith | Oh... the instance_uuid being required? | 13:05 |
| jgriffith | I forgot about that... there was a reason I added that back in..... | 13:06 |
| *** mriedem has joined #openstack-cinder | 13:06 | |
| tommylikehu | jgriffith: so it's required or not :) | 13:06 |
| jgriffith | tommylikehu yes | 13:06 |
| jgriffith | :) | 13:06 |
| jgriffith | tommylikehu so I don't want it to be required, it should NOT be required | 13:07 |
| jgriffith | tommylikehu I'm not sure if I had to add that for something or if it's a bad refactor | 13:07 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove deprecated 'host' option for cinder-volume https://review.openstack.org/449616 | 13:07 |
| jgriffith | tommylikehu so there are cases where Nova does a reserve and doesn't have an instance_uuid yet | 13:08 |
| smcginnis | jgriffith: Didn't Nova John push for that for some reason? | 13:08 |
| *** gouthamr has joined #openstack-cinder | 13:08 | |
| jgriffith | smcginnis that's what I'm trying to remember, I think he did; but I can't remember the details | 13:08 |
| ildikov | jgriffith: I think we went a bit back and forth on that one, but I cannot recall a reason now to require it | 13:08 |
| jgriffith | I may ahve to go back and look at the review | 13:08 |
| tommylikehu | jgriffith: We don't have logic to update the instance_id later either | 13:09 |
| jgriffith | tommylikehu I'm heading out the door right now, but will get it sorted and update you (unless you guys all figure it out before me) :) | 13:09 |
| jgriffith | tommylikehu yeah, I can fix that :) | 13:09 |
| tommylikehu | jgriffith: thanks | 13:09 |
| ildikov | jgriffith: wasn't it to be able to figure out what the volume is reserved for? | 13:09 |
| ildikov | anyway, I will try to look up what we discussed about that | 13:10 |
| tommylikehu | ildikov I am always here :) | 13:10 |
| tommylikehu | anytime if I could help | 13:10 |
| * ildikov thought we resolved the bigger inconsistencies with those patches :/ | 13:10 | |
| ildikov | tommylikehu: cool, tnx :) | 13:11 |
| * smcginnis though so too | 13:11 | |
| smcginnis | *thought | 13:11 |
| tommylikehu | @smcginnis: ok next question :) | 13:11 |
| smcginnis | ildikov: Brought back 2kg of coffee beans from Italy. | 13:11 |
| smcginnis | tommylikehu: OK :) | 13:12 |
| *** tommylikehu_ has joined #openstack-cinder | 13:12 | |
| tommylikehu | Should we expose the mode option in the connector to the end user by our cinder client ? | 13:12 |
| smcginnis | tommylikehu: It's really internal cloud info, so generally not. | 13:13 |
| jgriffith | tommylikehu I just make it part of the connector | 13:14 |
| jgriffith | the idea being to NOT have all the switches and options, just hide it in the data structure | 13:14 |
| ildikov | smcginnis: awesome! (I mean the coffee :) | 13:15 |
| ildikov | smcginnis: I had my checked-in bag on the edge of being overweighted by the time I got to Barcelona... | 13:15 |
| ildikov | smcginnis: not to mention I got sick by the time I could've gone to buy coffee | 13:15 |
| tommylikehu | smcginnis, jgriffith, thanks, I just wanna ensure that's in what we designed to. | 13:15 |
| ildikov | smcginnis: so it's good we didn't get into a business on that :) | 13:16 |
| ildikov | jgriffith: +1 | 13:16 |
| smcginnis | ildikov: Sorry to hear you got sick, though I'm not too suprised with how much you were travelling there! | 13:16 |
| jgriffith | tommylikehu https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1966 | 13:17 |
| *** tommylikehu_ has quit IRC | 13:17 | |
| ildikov | we moved everything into the connector as we had a few items separately, but it just got things inconsistent and messy... | 13:17 |
| jgriffith | ildikov and now a +1 for you | 13:17 |
| ildikov | jgriffith: ;) | 13:17 |
| jgriffith | Ok, now I really need to go | 13:17 |
| smcginnis | :) | 13:17 |
| ildikov | smcginnis: I think I got food poisoning in Barcelona and maybe also got sick from the craziness of Mobile World Congress :) | 13:18 |
| smcginnis | ildikov: 10's of thousands of people all converging in one place. What could go wrong? :) | 13:18 |
| ildikov | smcginnis: it was a short one, ruined mostly my time in Barcelona | 13:19 |
| smcginnis | ildikov: Really too bad. Barcelona can be fun. | 13:19 |
| ildikov | smcginnis: with that many Telco people around? Well... :) | 13:19 |
| smcginnis | :) | 13:19 |
| ildikov | smcginnis: yeah, I was really annoyed being locked into a small hotel room :( | 13:20 |
| *** yangyapeng has joined #openstack-cinder | 13:21 | |
| tommylikehu | ildikov: ok, jgriffith is not here, as a response from his link, but I don't know how could we specify the instance_id in our CLI, | 13:22 |
| tommylikehu | when update a attachment | 13:23 |
| *** Jack_Iv has quit IRC | 13:24 | |
| ildikov | tommylikehu: now you can say what you want :) | 13:25 |
| *** psachin has quit IRC | 13:25 | |
| ildikov | tommylikehu: do you want to specify it to update? | 13:25 |
| tommylikehu | ildikov: ok my initial question is whether the instance_id is require when create an attachment:) | 13:26 |
| ildikov | tommylikehu: is the question because the client itslef does not fail if you don't specify it? | 13:26 |
| tommylikehu | yes, becasue CLI told me it's an optional parameter but it failed if I don't specify | 13:27 |
| ildikov | oh, I see now, default=None | 13:30 |
| tommylikehu | ildikov: so it's a bug or my misunderstanding? | 13:30 |
| *** gcb has joined #openstack-cinder | 13:31 | |
| ildikov | I need to look up the reviews and discussions about it as we made that required on purpose | 13:31 |
| ildikov | the default value note was left in the client by mistake | 13:31 |
| smcginnis | tommylikehu: I think jgriffith said there's an issue there because the service side ended up changing from the original idea. | 13:32 |
| ildikov | but I don't remember the exact reason at this point... | 13:32 |
| ildikov | smcginnis: if you look into the delete the code is still inconsistent there, but we left it like that until we figure what we want as how it looks like today does not do any harm | 13:33 |
| ildikov | smcginnis: not good, but we had to start somewhere | 13:33 |
| smcginnis | ildikov: Yeah, things were in flux for awhile, so I'm not surprised. | 13:33 |
| *** xinli has joined #openstack-cinder | 13:35 | |
| tommylikehu | ildikov: one more question if the instane_id is not required, we need add CLI argument.(Sorry I really don't know the inconsistent means we need or not :) ) | 13:35 |
| tommylikehu | in the update command | 13:36 |
| *** Dinesh_Bhor has quit IRC | 13:39 | |
| ildikov | tommylikehu: we didn't have an explicit request from Nova, but decided to save that info anyhow as Nova should have it and that's still the majority of the use cases | 13:40 |
| ildikov | tommylikehu: we thought to make it optional later if that makes more sense, but try to keep some info in the "reserved" attachment for now to see what's going on | 13:40 |
| ildikov | tommylikehu: so the latest is it is required | 13:41 |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: SMBFS: enable Nova assisted volume snapshots https://review.openstack.org/428789 | 13:41 |
| ildikov | tommylikehu: also having a note in the CLI help about a default value should not be the driving force IMHO | 13:41 |
| *** tommylikehu_ has joined #openstack-cinder | 13:42 | |
| xinli | e@ne: could you please help to review the patch https://review.openstack.org/#/c/448622 | 13:42 |
| ildikov | tommylikehu: need to get that out of there to avoid confusion though | 13:43 |
| *** huanxuan has quit IRC | 13:43 | |
| tommylikehu | ildikov: I can do that, to be more explicit should we change the cmd to attachment-create xxxx xxxxxx or keep same as we currently have | 13:45 |
| *** tommylikehu_ has quit IRC | 13:46 | |
| ildikov | tommylikehu: I'm not sure about the conventions in Cinder | 13:46 |
| ildikov | smcginnis: ^^? | 13:47 |
| tommylikehu | smcginnis: it's your call | 13:47 |
| *** dansmith is now known as superdan | 13:49 | |
| *** mtanino has joined #openstack-cinder | 13:49 | |
| *** cknight has joined #openstack-cinder | 13:51 | |
| *** links has quit IRC | 13:53 | |
| *** asselin has joined #openstack-cinder | 13:53 | |
| *** kaisers_ has joined #openstack-cinder | 14:01 | |
| *** pooja_jadhav has quit IRC | 14:04 | |
| *** eharney has joined #openstack-cinder | 14:04 | |
| *** kaisers_ has quit IRC | 14:05 | |
| smcginnis | tommylikehu: Sorry, lot's going on here today. What was the question? | 14:07 |
| *** tommylikehu_ has joined #openstack-cinder | 14:07 | |
| *** dave-mccowan has joined #openstack-cinder | 14:07 | |
| tommylikehu | the instance_id is required eventually, so I think our CLI's behaviour is not correct | 14:08 |
| tommylikehu | should we change the cmd to attachment-create xxxx xxxxxx or keep same as we currently have | 14:08 |
| tommylikehu | @smcginnis | 14:08 |
| *** lamt has quit IRC | 14:09 | |
| smcginnis | tommylikehu: Depends, what are the xxxx's? :) | 14:09 |
| *** scottda has joined #openstack-cinder | 14:09 | |
| ildikov | smcginnis: I think he means not using --instance-id xxxxx just add the volume and instance uuid after the command as attachment-creat xxxxx yyyyyyy | 14:11 |
| tommylikehu | thanks ildikov | 14:12 |
| ildikov | tommylikehu: please correct me if I understood the question wrong | 14:12 |
| ildikov | tommylikehu: :) | 14:12 |
| tommylikehu | that's correct | 14:12 |
| ildikov | cool! | 14:13 |
| ildikov | I know Nova uses the latter | 14:13 |
| ildikov | for volume-attach | 14:13 |
| smcginnis | ildikov: Positional arguments? | 14:13 |
| ildikov | smcginnis: I guess that's the name I could not remember :) | 14:14 |
| tommylikehu | smcginnis: yes it's a decision use a positonal argument or optional argument | 14:15 |
| smcginnis | tommylikehu: Yeah, I think that makes sense. A little better user experience I think. | 14:16 |
| ildikov | smcginnis: +1 | 14:17 |
| tommylikehu | smcginnis: ok, let me fix/optimise it | 14:18 |
| *** nkrinner is now known as nkrinner_afk | 14:21 | |
| *** zhangguoqing has joined #openstack-cinder | 14:23 | |
| *** zhangguoqing has quit IRC | 14:23 | |
| *** zhangguoqing has joined #openstack-cinder | 14:23 | |
| *** chlong has joined #openstack-cinder | 14:25 | |
| *** david-lyle has joined #openstack-cinder | 14:26 | |
| *** tommylikehu_ has quit IRC | 14:37 | |
| *** erhudy has quit IRC | 14:50 | |
| *** yangyapeng has quit IRC | 14:51 | |
| *** yangyapeng has joined #openstack-cinder | 14:52 | |
| satya1304 | smcginnis: thanks for the comments on this https://review.openstack.org/#/c/444494/ . I have addressed them | 14:55 |
| smcginnis | satya1304: Thanks! | 14:57 |
| *** zhangguoqing has quit IRC | 14:59 | |
| *** lucasxu has joined #openstack-cinder | 14:59 | |
| *** eharney has quit IRC | 15:00 | |
| *** eharney has joined #openstack-cinder | 15:00 | |
| hemna | at least it's Friday | 15:01 |
| smcginnis | yerp | 15:03 |
| jungleboyj | Yay Friday! One day closer to vacation for me. | 15:04 |
| hemna | good :) | 15:04 |
| *** david-lyle has quit IRC | 15:15 | |
| *** gkadam has joined #openstack-cinder | 15:24 | |
| *** kaisers_ has joined #openstack-cinder | 15:26 | |
| openstackgerrit | Merged openstack/cinder master: Refactor volumes summary https://review.openstack.org/425643 | 15:27 |
| *** kaisers_ has quit IRC | 15:31 | |
| openstackgerrit | Lucian Petrut proposed openstack/cinder master: SMBFS: enable Nova assisted volume snapshots https://review.openstack.org/428789 | 15:33 |
| *** prashkre has joined #openstack-cinder | 15:36 | |
| *** jdurgin has quit IRC | 15:37 | |
| jgriffith | jungleboyj and that means a vacation for "us" as well! | 16:03 |
| *** pcaruana has quit IRC | 16:03 | |
| jungleboyj | jgriffith: :-p | 16:03 |
| *** blancos has joined #openstack-cinder | 16:05 | |
| blancos | Hi, I had some general Cinder policy questions: Is Cinder planning on moving to default policies in code (similar to Nova and Keystone)? Are the policies included in the Cinder policy.json intended to be an exhaustive list (meaning, are those all the policies that exist in Cinder)? | 16:07 |
| *** e0ne has quit IRC | 16:11 | |
| openstackgerrit | Merged openstack/cinder master: Fix 500 error when 'mountpoint' is not provided to os-attach https://review.openstack.org/449223 | 16:12 |
| openstackgerrit | Merged openstack/cinder master: api-ref: Fix volume_id parameter for the path https://review.openstack.org/449226 | 16:12 |
| xinli | xyang1: could you please help to review this patch ? http://paste.openstack.org/show/604061/? | 16:13 |
| openstackgerrit | Michael Dovgal proposed openstack/cinder master: WIP: Add cinder backup service initialize check Checked only for swift driver (not thoroughly). https://review.openstack.org/446518 | 16:14 |
| xyang1 | xinli: sure | 16:14 |
| xinli | xyang1: thx | 16:14 |
| *** chenhb has quit IRC | 16:19 | |
| *** chenhb has joined #openstack-cinder | 16:19 | |
| *** pbandark has quit IRC | 16:20 | |
| *** belmoreira has quit IRC | 16:21 | |
| *** gouthamr has quit IRC | 16:23 | |
| *** lamt has joined #openstack-cinder | 16:23 | |
| *** kaisers_ has joined #openstack-cinder | 16:24 | |
| openstackgerrit | Merged openstack/cinder master: HPE LeftHand: extend volume if larger than snapshot https://review.openstack.org/296371 | 16:26 |
| *** dustins has quit IRC | 16:28 | |
| *** kaisers_ has quit IRC | 16:29 | |
| *** lpetrut has quit IRC | 16:30 | |
| *** gouthamr has joined #openstack-cinder | 16:32 | |
| *** david-lyle has joined #openstack-cinder | 16:33 | |
| *** Jack_Iv has joined #openstack-cinder | 16:38 | |
| *** Apoorva has joined #openstack-cinder | 16:42 | |
| *** Apoorva has quit IRC | 16:42 | |
| jungleboyj | blancos: I believe that policy.json is supposed to be exhaustive. | 16:42 |
| *** Apoorva has joined #openstack-cinder | 16:42 | |
| jungleboyj | blancos: As for the question about moving default policies in code, I feel like we talked about that ... | 16:43 |
| jungleboyj | blancos: Hmm, We talked about adding policy actions that were more granular at the PTG. That was what I was thinking of. | 16:44 |
| jungleboyj | blancos: So, I am not sure on that question. | 16:44 |
| *** mlakat has quit IRC | 16:47 | |
| *** aarefiev is now known as aarefiev_afk | 16:49 | |
| *** kaisers_ has joined #openstack-cinder | 16:52 | |
| *** kaisers_ has quit IRC | 16:52 | |
| *** kaisers_ has joined #openstack-cinder | 16:52 | |
| *** psachin has joined #openstack-cinder | 16:55 | |
| *** e0ne has joined #openstack-cinder | 16:56 | |
| smcginnis | blancos: I had started a spec for the "policies in code" thing, but need to get back to it. | 16:59 |
| *** gouthamr has quit IRC | 17:01 | |
| *** zhangguoqing has joined #openstack-cinder | 17:02 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder master: Make cinder-manage online migrations more verbose https://review.openstack.org/438754 | 17:04 |
| *** gouthamr has joined #openstack-cinder | 17:05 | |
| *** zhangguoqing has quit IRC | 17:06 | |
| *** kaisers_ has quit IRC | 17:07 | |
| *** blancos has quit IRC | 17:08 | |
| *** Jack_Iv has quit IRC | 17:14 | |
| *** catintheroof has joined #openstack-cinder | 17:14 | |
| *** catintheroof has quit IRC | 17:15 | |
| *** Jack_Iv has joined #openstack-cinder | 17:15 | |
| *** catintheroof has joined #openstack-cinder | 17:15 | |
| *** lamt has quit IRC | 17:17 | |
| *** chris_morrell has joined #openstack-cinder | 17:24 | |
| *** lucasxu has quit IRC | 17:36 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Fix boolean opts defined as strings https://review.openstack.org/449756 | 17:38 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Clean up index links to removed content https://review.openstack.org/449759 | 17:40 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder master: Add __init__ for cinder.config module https://review.openstack.org/449760 | 17:42 |
| *** mtanino has quit IRC | 17:43 | |
| *** tesseract has quit IRC | 17:50 | |
| *** dustins has joined #openstack-cinder | 17:55 | |
| *** Jack_Iv_ has joined #openstack-cinder | 18:02 | |
| *** Jack_Iv has quit IRC | 18:04 | |
| *** kaisers_ has joined #openstack-cinder | 18:07 | |
| *** thiagolib has joined #openstack-cinder | 18:09 | |
| *** kaisers_ has quit IRC | 18:12 | |
| *** mtanino has joined #openstack-cinder | 18:13 | |
| *** gkadam has quit IRC | 18:15 | |
| *** arunkant has joined #openstack-cinder | 18:18 | |
| *** abhi89 has joined #openstack-cinder | 18:18 | |
| *** e0ne has quit IRC | 18:19 | |
| abhi89 | xyang1: Hi | 18:19 |
| *** viks has joined #openstack-cinder | 18:21 | |
| *** mvk has quit IRC | 18:23 | |
| viks | hi all need some help on procedures... | 18:23 |
| viks | PLease refer to https://review.openstack.org/#/c/436670/ | 18:23 |
| viks | As commented the spec is not required because we have linked it with cinder driver blueprint https://blueprints.launchpad.net/cinder/+spec/veritas-hyperscale-cinder-driver | 18:24 |
| viks | Should we abandon the spec then? | 18:24 |
| *** chlong has quit IRC | 18:24 | |
| viks | just trying to close the loop here | 18:25 |
| abhi89 | xyang1: please review https://review.openstack.org/#/c/448901/1.. The change set is already approved & merged in openstack master. Need to merge in ocata.. | 18:25 |
| eharney | viks: yes | 18:25 |
| viks | thanks a ton eharney.. | 18:25 |
| *** e0ne has joined #openstack-cinder | 18:25 | |
| abhi89 | eOne: Hi | 18:26 |
| abhi89 | eOne: please review https://review.openstack.org/#/c/448901/1 | 18:26 |
| *** ociuhandu has quit IRC | 18:29 | |
| openstackgerrit | Eric Harney proposed openstack/python-cinderclient master: Tests: Add info to assert_called failure message https://review.openstack.org/449775 | 18:31 |
| openstackgerrit | Eric Harney proposed openstack/python-cinderclient master: Handle dashes in encryption-type-create arguments https://review.openstack.org/449776 | 18:31 |
| *** kaisers_ has joined #openstack-cinder | 18:33 | |
| *** kaisers_ has quit IRC | 18:34 | |
| *** kaisers_ has joined #openstack-cinder | 18:35 | |
| *** e0ne has quit IRC | 18:35 | |
| xinli | @smcginnis: could you please help to review this: http://paste.openstack.org/show/604061/ | 18:36 |
| *** e0ne has joined #openstack-cinder | 18:37 | |
| *** imansari has quit IRC | 18:38 | |
| *** kaisers_ has quit IRC | 18:39 | |
| *** lamt has joined #openstack-cinder | 18:40 | |
| *** lpetrut has joined #openstack-cinder | 18:41 | |
| *** abhi89 has quit IRC | 18:44 | |
| *** lpetrut has quit IRC | 18:44 | |
| *** lucasxu has joined #openstack-cinder | 18:50 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Catch Castellan errors in create volume https://review.openstack.org/449780 | 18:51 |
| *** Jack_Iv_ has quit IRC | 18:55 | |
| *** Jack_Iv has joined #openstack-cinder | 18:56 | |
| *** Jack_Iv has quit IRC | 18:57 | |
| *** Jack_Iv has joined #openstack-cinder | 18:57 | |
| *** psachin has quit IRC | 18:58 | |
| *** e0ne has quit IRC | 18:59 | |
| *** adisky_ has quit IRC | 18:59 | |
| *** crushil has joined #openstack-cinder | 18:59 | |
| *** mvk has joined #openstack-cinder | 19:00 | |
| *** Jack_Iv has quit IRC | 19:02 | |
| *** lpetrut has joined #openstack-cinder | 19:03 | |
| *** jordanP has joined #openstack-cinder | 19:05 | |
| *** crushil has quit IRC | 19:11 | |
| *** erhudy has joined #openstack-cinder | 19:20 | |
| openstackgerrit | Emily Hugenbruch proposed openstack/cinder master: Separate out routine for getting qemu_img_info https://review.openstack.org/448887 | 19:23 |
| *** abhi89 has joined #openstack-cinder | 19:24 | |
| *** yangyapeng has quit IRC | 19:24 | |
| *** yangyapeng has joined #openstack-cinder | 19:24 | |
| *** Jack_Iv has joined #openstack-cinder | 19:27 | |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Don't use tempest special_fields https://review.openstack.org/449787 | 19:30 |
| *** crushil has joined #openstack-cinder | 19:30 | |
| *** abhi89 has quit IRC | 19:31 | |
| *** crushil has quit IRC | 19:38 | |
| *** gcb has quit IRC | 19:49 | |
| *** kaisers_ has joined #openstack-cinder | 19:50 | |
| *** xinli has quit IRC | 19:53 | |
| openstackgerrit | Merged openstack/cinder master: Mark Blockbridge volume driver as unsupported https://review.openstack.org/449121 | 19:53 |
| *** ebalduf has quit IRC | 19:53 | |
| *** ebalduf has joined #openstack-cinder | 19:54 | |
| *** liverpooler has quit IRC | 19:55 | |
| openstackgerrit | Merged openstack/cinder master: Fix version numbers on media-types https://review.openstack.org/448312 | 19:55 |
| *** blancos has joined #openstack-cinder | 19:56 | |
| blancos | jungleboyj smcginnis Thanks for the information re: policies from before. | 19:57 |
| smcginnis | np | 19:57 |
| blancos | jungleboyj: I had one more question. If there are policies that aren't included in the policy.json for whatever reason, should that be filed as a bug? | 19:57 |
| *** kaisers_ has quit IRC | 19:57 | |
| jungleboyj | blancos: I think that would be ok to do. It seems like we shouldn't be missing policies. | 19:59 |
| *** kaisers_ has joined #openstack-cinder | 19:59 | |
| openstackgerrit | Merged openstack/cinder master: Fixed inconsistent naming conventions https://review.openstack.org/448415 | 19:59 |
| openstackgerrit | Merged openstack/cinder master: Fix: SnapshotStatus missing in Cinder https://review.openstack.org/445725 | 19:59 |
| *** Jack_Iv has quit IRC | 20:05 | |
| blancos | jungleboyj Okay, thank you :) | 20:07 |
| *** blancos has quit IRC | 20:07 | |
| jungleboyj | Welcome. | 20:11 |
| *** eharney has quit IRC | 20:12 | |
| *** kaisers_ has quit IRC | 20:14 | |
| *** thiagolib has quit IRC | 20:21 | |
| openstackgerrit | Chuck Fouts proposed openstack/cinder master: NetApp E-Series CI name update https://review.openstack.org/449807 | 20:22 |
| *** gouthamr has quit IRC | 20:23 | |
| mriedem | scottda: abandon this probably now https://review.openstack.org/#/c/305419/ | 20:33 |
| karthikp | smcginnis: Hi | 20:34 |
| karthikp | smcginnis: I have made changes as per your remarks...please could you provide some feedback in your free time : https://review.openstack.org/#/c/371169/ Thanks! | 20:35 |
| scottda | mriedem: yeah, will do | 20:35 |
| *** kaisers_ has joined #openstack-cinder | 20:45 | |
| *** kaisers_ has quit IRC | 20:49 | |
| *** abishop has quit IRC | 20:50 | |
| *** kaisers_ has joined #openstack-cinder | 20:54 | |
| *** dave-mccowan has quit IRC | 21:01 | |
| *** e0ne has joined #openstack-cinder | 21:04 | |
| *** links has joined #openstack-cinder | 21:05 | |
| hemna | fwiw, I think this guy is good to go. tested. https://review.openstack.org/#/c/427143/ | 21:11 |
| *** ChubYann has joined #openstack-cinder | 21:13 | |
| *** lpetrut has quit IRC | 21:19 | |
| *** mpotdar has quit IRC | 21:21 | |
| *** mmotiani has quit IRC | 21:21 | |
| *** karthikp- has quit IRC | 21:21 | |
| *** crushil has joined #openstack-cinder | 21:23 | |
| *** pbandark has joined #openstack-cinder | 21:24 | |
| *** pbandark has quit IRC | 21:24 | |
| *** e0ne has quit IRC | 21:32 | |
| *** dustins has quit IRC | 21:33 | |
| *** knikolla has left #openstack-cinder | 22:03 | |
| *** prashkre has quit IRC | 22:08 | |
| *** lucasxu has quit IRC | 22:27 | |
| *** xyang1 has quit IRC | 22:38 | |
| *** mtanino has quit IRC | 22:57 | |
| *** mmotiani has joined #openstack-cinder | 22:59 | |
| *** erhudy has quit IRC | 23:00 | |
| *** mpotdar has joined #openstack-cinder | 23:03 | |
| *** mmotiani has quit IRC | 23:03 | |
| *** mpotdar has quit IRC | 23:03 | |
| *** karthikp- has joined #openstack-cinder | 23:06 | |
| *** mmotiani has joined #openstack-cinder | 23:07 | |
| *** mpotdar has joined #openstack-cinder | 23:09 | |
| *** chris_morrell has quit IRC | 23:10 | |
| *** cknight has quit IRC | 23:11 | |
| *** markvoelker has quit IRC | 23:24 | |
| *** links has quit IRC | 23:24 | |
| *** kaisers_ has quit IRC | 23:29 | |
| *** jordanP has quit IRC | 23:39 | |
| *** lamt has quit IRC | 23:41 | |
| *** sdague has quit IRC | 23:43 | |
| *** lucasxu has joined #openstack-cinder | 23:45 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!