| *** mylu_ has quit IRC | 00:00 | |
| *** Yogi1 has joined #openstack-cinder | 00:00 | |
| *** mylu has joined #openstack-cinder | 00:01 | |
| *** bnemec has joined #openstack-cinder | 00:03 | |
| *** rushil has quit IRC | 00:03 | |
| *** ebalduf has quit IRC | 00:10 | |
| *** bnemec has quit IRC | 00:10 | |
| *** andreykurilin__ has quit IRC | 00:15 | |
| *** lixiaoy1 has joined #openstack-cinder | 00:24 | |
| *** mylu has quit IRC | 00:24 | |
| *** bnemec has joined #openstack-cinder | 00:25 | |
| *** mylu has joined #openstack-cinder | 00:25 | |
| jgriffith | NightKhaos: that would be cool to look at | 00:27 |
|---|---|---|
| jgriffith | NightKhaos: problem I ran into before though was the scsi refresh | 00:27 |
| jgriffith | NightKhaos: not sure if what you came up with gets around that or not, or what it does to the file-system | 00:28 |
| *** dannywil_ has quit IRC | 00:28 | |
| *** mylu has quit IRC | 00:29 | |
| *** salv-orl_ has quit IRC | 00:29 | |
| *** dims has joined #openstack-cinder | 00:30 | |
| *** bnemec has quit IRC | 00:30 | |
| *** leeantho has quit IRC | 00:31 | |
| *** tiny-hands has joined #openstack-cinder | 00:31 | |
| *** bnemec has joined #openstack-cinder | 00:44 | |
| *** mriedem has joined #openstack-cinder | 00:45 | |
| NightKhaos | jgriffith: Okay, I'll write up a quick outline of how I did it. Bare in mind this is specific to ceph rbds. | 00:45 |
| *** adurbin_ has quit IRC | 00:47 | |
| *** smoriya has joined #openstack-cinder | 00:49 | |
| *** tsekiyam_ has joined #openstack-cinder | 00:49 | |
| jgriffith | NightKhaos: yeah, may not work the same with other protocols, but would be cool to look at | 00:50 |
| jgriffith | NightKhaos: FWIW, the Trove folks tried to get something like this off the ground last year | 00:51 |
| *** xyang1 has quit IRC | 00:51 | |
| NightKhaos | jgriffith: I can understand why they're want this. Means they can dynamically create databases without worrying about pre-allocation. Personally I'd solve that problem by providing thinly provisioned volumes, but that's just me. :) | 00:51 |
| jgriffith | NightKhaos: :) | 00:52 |
| jgriffith | NightKhaos: I like the way you think :) | 00:52 |
| *** tsekiyama has quit IRC | 00:53 | |
| *** mtanino has quit IRC | 00:53 | |
| *** tsekiyam_ has quit IRC | 00:53 | |
| NightKhaos | jgriffith: http://pastebin.com/dR5BzGxr | 00:58 |
| jgriffith | NightKhaos: hmm... that should work actually | 00:58 |
| jgriffith | NightKhaos: if it's just a function of virt-io driver | 00:58 |
| NightKhaos | jgriffith: Yeah, it worked just fine in the times I've tried it. It also looks like if you select a size smaller than the disk (e.g. forget the unit), the opereation will timeout. | 00:59 |
| jgriffith | NightKhaos: might be a good thing do check with Dberange on... | 00:59 |
| NightKhaos | jgriffith: Where can we find them? | 01:00 |
| jgriffith | I just looked in dev he's not there... maybe Nova... | 01:00 |
| jgriffith | NightKhaos: might be worth a post to the ML? | 01:01 |
| NightKhaos | jgriffith: Maybe, maybe. Never done that before. I'm kinda new at this. I just new enough about libvirt and found you could do a volume resize by reseting state without issues with Ceph RDBs... put 2 and 2, and violia. | 01:02 |
| jgriffith | NightKhaos: I'm still a bit nervous about what one does on the file-system.... but I guess if they're smart enough to type "cinder extend" it's kinda their problem | 01:02 |
| jgriffith | :) | 01:02 |
| jgriffith | NightKhaos: :) | 01:02 |
| jgriffith | NightKhaos: well, welcome first off.... second off you're doing things right IMO | 01:03 |
| jgriffith | NightKhaos: so you can post to openstack-dev mailing list, with the subject [nova][cinder] Extending attached disks | 01:03 |
| jgriffith | NightKhaos: and throw your idea out asking for feedback | 01:03 |
| jgriffith | NightKhaos: or we can try and grab some nova folks tomorrow.... either way, just so you know this would be next release before anything really happened with it | 01:04 |
| jgriffith | NightKhaos: everybody is going into feature freeze right now | 01:04 |
| NightKhaos | jgriffith: I've got a reproducable method I can give to ops for the moment, so they'll be happy regardless. | 01:05 |
| jgriffith | NightKhaos: sure, that's always good | 01:05 |
| jgriffith | NightKhaos: but it would be interesting to propose it back upstream to the community IMO | 01:05 |
| NightKhaos | jgriffith: Which is why I'm here! | 01:05 |
| NightKhaos | jgriffith: Should i post this to the general list? Or Future Dev/ | 01:08 |
| jgriffith | openstack-dev | 01:08 |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Nested Quota Driver: Get Project Hierarchy https://review.openstack.org/206171 | 01:10 |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Cinder Nested Quota Driver https://review.openstack.org/205369 | 01:10 |
| *** diogogmt has joined #openstack-cinder | 01:11 | |
| *** haigang has joined #openstack-cinder | 01:16 | |
| *** bill_az_ has quit IRC | 01:25 | |
| *** dims has quit IRC | 01:25 | |
| *** mylu has joined #openstack-cinder | 01:26 | |
| *** skylerberg has quit IRC | 01:28 | |
| *** HoloIRCUser3 has quit IRC | 01:28 | |
| *** mylu has quit IRC | 01:30 | |
| *** Apoorva has quit IRC | 01:30 | |
| NightKhaos | jgriffith: Can I get a proof read? I've updated that paste with the email I intend to send to the list | 01:31 |
| * jgriffith is reading... | 01:32 | |
| jgriffith | NightKhaos: nice! | 01:32 |
| *** vilobhmm1 has quit IRC | 01:33 | |
| jgriffith | NightKhaos: I think that's great and I'd be one of the first to respond to it :) | 01:33 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 01:33 |
| *** bill_az has joined #openstack-cinder | 01:34 | |
| NightKhaos | jgriffith: Sent. | 01:35 |
| *** bill_az has quit IRC | 01:35 | |
| NightKhaos | jgriffith: Hmm, it's saying I have to be subscribed but I AM subscribed... hmm. Give me a moment. | 01:35 |
| jgriffith | NightKhaos: hmm... I'm happy to send it by proxy, but better if you just get it sorted and keep it under your name etc | 01:36 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds the migration progress support for migration https://review.openstack.org/204953 | 01:38 |
| *** Apoorva has joined #openstack-cinder | 01:39 | |
| NightKhaos | jgriffith: Okay, fixed it. Apparently the first time the confirmation email didn't come through. | 01:39 |
| *** Lee1092 has joined #openstack-cinder | 01:39 | |
| NightKhaos | jgriffith: Whoops. Looks like it didn't send in plaintext. Should I resend? | 01:43 |
| jgriffith | NightKhaos: nah... it happens | 01:45 |
| NightKhaos | jgriffith: It's pretty hard to read the HTML version through | 01:45 |
| jgriffith | NightKhaos: you can reply with an apology if you like before somebody complains but shouldn't be a big deal | 01:45 |
| jgriffith | Ohh... lemme look | 01:45 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: WIP: Adds migration abortion https://review.openstack.org/207754 | 01:45 |
| jgriffith | NightKhaos: looks fine for me :) But I'm using gmail so it just works out ok :) | 01:47 |
| NightKhaos | http://lists.openstack.org/pipermail/openstack-dev/2015-August/072495.html It's here where it looks pretty bad: | 01:47 |
| jgriffith | NightKhaos: just reply acking that you made a mistake and won't do it again :) | 01:47 |
| jgriffith | NightKhaos: ahh... I forgot about that | 01:48 |
| jgriffith | NightKhaos: sure, just resend in plain text | 01:48 |
| NightKhaos | jgriffith: http://lists.openstack.org/pipermail/openstack-dev/2015-August/072496.html MUCH better. | 01:49 |
| jgriffith | Very nice! | 01:49 |
| *** boris-42 has quit IRC | 01:50 | |
| NightKhaos | jgriffith: Now to wait for a reply... | 01:51 |
| * NightKhaos goes back to documenting Neutron networks. | 01:51 | |
| *** haomaiwang has joined #openstack-cinder | 01:53 | |
| jgriffith | NightKhaos: thanks for posting, hope to talk again and see something get submitted | 01:54 |
| *** Yogi1 has quit IRC | 01:58 | |
| NightKhaos | jgriffith: Any chance you can knock up an iSCSI test rig and give it a try? | 02:02 |
| jgriffith | NightKhaos: for sure, but probably not tonight | 02:02 |
| jgriffith | Maybe tomorrow | 02:02 |
| *** _cjones_ has quit IRC | 02:02 | |
| NightKhaos | jgriffith: obviously not now! Just curious if you have the hardware lieing around. | 02:02 |
| *** davechen has joined #openstack-cinder | 02:04 | |
| *** p0rtal_ has quit IRC | 02:06 | |
| *** haomaiwang has quit IRC | 02:09 | |
| *** haomaiwang has joined #openstack-cinder | 02:10 | |
| *** hw_chenying has joined #openstack-cinder | 02:10 | |
| *** netapp has quit IRC | 02:11 | |
| jgriffith | NightKhaos: I do, and honestly normally I would try it right now | 02:12 |
| *** netapp has joined #openstack-cinder | 02:12 | |
| jgriffith | NightKhaos: but tonight I'm tied up with other things | 02:12 |
| *** baojg has joined #openstack-cinder | 02:16 | |
| *** hw_chenying has quit IRC | 02:17 | |
| *** dims has joined #openstack-cinder | 02:21 | |
| *** hw_chenying has joined #openstack-cinder | 02:21 | |
| *** ebalduf_ has quit IRC | 02:26 | |
| *** mylu has joined #openstack-cinder | 02:27 | |
| *** ebalduf has joined #openstack-cinder | 02:27 | |
| *** tiny-hands has quit IRC | 02:29 | |
| *** dims has quit IRC | 02:29 | |
| *** dims has joined #openstack-cinder | 02:30 | |
| *** mylu has quit IRC | 02:31 | |
| *** dims has quit IRC | 02:34 | |
| *** tiny-hands has joined #openstack-cinder | 02:35 | |
| *** bkopilov has quit IRC | 02:43 | |
| *** mriedem has quit IRC | 02:44 | |
| *** salv-orlando has joined #openstack-cinder | 02:44 | |
| *** markvoelker has quit IRC | 02:46 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: WIP: Adds migration abortion https://review.openstack.org/207754 | 02:47 |
| *** salv-orlando has quit IRC | 02:51 | |
| *** hakimo has joined #openstack-cinder | 02:52 | |
| *** hakimo_ has quit IRC | 02:54 | |
| *** salv-orlando has joined #openstack-cinder | 02:55 | |
| *** Apoorva has quit IRC | 02:56 | |
| *** diogogmt has quit IRC | 03:01 | |
| *** salv-orlando has quit IRC | 03:02 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 03:07 |
| *** haomaiwang has quit IRC | 03:09 | |
| *** haomaiwang has joined #openstack-cinder | 03:10 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Update the devref for volume migration https://review.openstack.org/214941 | 03:18 |
| openstackgerrit | Thang Pham proposed openstack/cinder: WIP: Make RPC and objects backwards compatible https://review.openstack.org/214943 | 03:21 |
| *** zhongjun has joined #openstack-cinder | 03:23 | |
| *** rushil has joined #openstack-cinder | 03:36 | |
| *** tiny-hands has quit IRC | 03:42 | |
| *** rushil_ has joined #openstack-cinder | 03:44 | |
| *** rushil_ has quit IRC | 03:46 | |
| *** markvoelker has joined #openstack-cinder | 03:46 | |
| *** baojg has quit IRC | 03:47 | |
| *** skylerberg has joined #openstack-cinder | 03:48 | |
| *** rushil has quit IRC | 03:48 | |
| *** lpetrut has joined #openstack-cinder | 03:51 | |
| *** markvoelker has quit IRC | 03:51 | |
| *** baojg has joined #openstack-cinder | 03:54 | |
| *** mylu has joined #openstack-cinder | 03:56 | |
| *** skylerberg has quit IRC | 04:07 | |
| *** zhongjun has quit IRC | 04:08 | |
| *** haomaiwang has quit IRC | 04:09 | |
| *** haomaiwang has joined #openstack-cinder | 04:10 | |
| *** skylerberg has joined #openstack-cinder | 04:10 | |
| *** hw_chenying has quit IRC | 04:24 | |
| *** bkopilov has joined #openstack-cinder | 04:30 | |
| *** lpetrut has quit IRC | 04:31 | |
| *** salv-orlando has joined #openstack-cinder | 04:32 | |
| *** salv-orlando has quit IRC | 04:35 | |
| *** baojg has quit IRC | 04:37 | |
| *** baojg has joined #openstack-cinder | 04:38 | |
| *** sgotliv has joined #openstack-cinder | 04:43 | |
| *** salv-orlando has joined #openstack-cinder | 04:47 | |
| *** skylerberg has quit IRC | 04:48 | |
| *** sgotliv has quit IRC | 04:51 | |
| *** salv-orlando has quit IRC | 04:51 | |
| *** zul has quit IRC | 05:04 | |
| *** mylu has quit IRC | 05:07 | |
| *** mylu has joined #openstack-cinder | 05:08 | |
| *** afazekas has joined #openstack-cinder | 05:08 | |
| *** haomaiwang has quit IRC | 05:09 | |
| *** danieru has quit IRC | 05:10 | |
| *** haomaiwang has joined #openstack-cinder | 05:10 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 05:12 |
| *** mylu has quit IRC | 05:12 | |
| *** mylu has joined #openstack-cinder | 05:12 | |
| *** haigang has quit IRC | 05:14 | |
| *** afazekas has quit IRC | 05:15 | |
| *** haomaiwang has quit IRC | 05:19 | |
| openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Adding delete-wait-loop for CloudByte Volumes https://review.openstack.org/201059 | 05:27 |
| *** shausy has joined #openstack-cinder | 05:44 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 05:44 |
| *** mylu has quit IRC | 05:45 | |
| *** mylu has joined #openstack-cinder | 05:45 | |
| *** danieru has joined #openstack-cinder | 05:52 | |
| *** nkrinner has joined #openstack-cinder | 05:52 | |
| openstackgerrit | wanghao proposed openstack/cinder: Forbid creating volume from image which size is 0 https://review.openstack.org/207696 | 05:54 |
| *** haomaiwa_ has joined #openstack-cinder | 06:00 | |
| *** hw_chenying has joined #openstack-cinder | 06:04 | |
| *** BharatK has joined #openstack-cinder | 06:05 | |
| *** salv-orlando has joined #openstack-cinder | 06:05 | |
| *** haomaiwa_ has quit IRC | 06:09 | |
| *** haomaiwang has joined #openstack-cinder | 06:13 | |
| *** salv-orlando has quit IRC | 06:16 | |
| *** haomaiwang has quit IRC | 06:17 | |
| *** vlaza has joined #openstack-cinder | 06:21 | |
| *** afazekas has joined #openstack-cinder | 06:21 | |
| *** baojg has quit IRC | 06:22 | |
| *** baojg has joined #openstack-cinder | 06:24 | |
| *** ankit_ag has joined #openstack-cinder | 06:24 | |
| *** haomaiwang has joined #openstack-cinder | 06:29 | |
| *** zhonghua-lee has joined #openstack-cinder | 06:30 | |
| *** salv-orlando has joined #openstack-cinder | 06:32 | |
| *** haigang has joined #openstack-cinder | 06:37 | |
| *** skylerberg has joined #openstack-cinder | 06:37 | |
| *** afazekas has quit IRC | 06:37 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/214468 | 06:41 |
| *** vlaza has quit IRC | 06:44 | |
| *** vlaza has joined #openstack-cinder | 06:46 | |
| *** haomaiwang has quit IRC | 06:47 | |
| *** haomaiwang has joined #openstack-cinder | 06:52 | |
| *** anshul has joined #openstack-cinder | 06:53 | |
| *** ebalduf has quit IRC | 06:56 | |
| *** ebalduf has joined #openstack-cinder | 06:57 | |
| *** mylu has quit IRC | 07:00 | |
| *** mylu has joined #openstack-cinder | 07:00 | |
| *** ronis_ has joined #openstack-cinder | 07:01 | |
| *** ronis__ has joined #openstack-cinder | 07:03 | |
| *** mylu has quit IRC | 07:05 | |
| *** ishant has joined #openstack-cinder | 07:06 | |
| *** ronis_ has quit IRC | 07:06 | |
| *** hw_chenying has quit IRC | 07:07 | |
| *** hw_chenying has joined #openstack-cinder | 07:07 | |
| *** haomaiwang has quit IRC | 07:09 | |
| *** haomaiwang has joined #openstack-cinder | 07:10 | |
| *** mylu has joined #openstack-cinder | 07:11 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/213891 | 07:15 |
| *** nkrinner has quit IRC | 07:17 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/210918 | 07:20 |
| *** nkrinner has joined #openstack-cinder | 07:21 | |
| *** gardenshed has joined #openstack-cinder | 07:28 | |
| *** zhonghua-lee has quit IRC | 07:32 | |
| *** salv-orlando has quit IRC | 07:40 | |
| *** skylerberg has quit IRC | 07:41 | |
| *** alexschm has joined #openstack-cinder | 07:42 | |
| *** skylerberg has joined #openstack-cinder | 07:45 | |
| *** agarciam has joined #openstack-cinder | 07:47 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 07:47 |
| *** markvoelker has joined #openstack-cinder | 07:48 | |
| *** IanGovett has joined #openstack-cinder | 07:51 | |
| *** markvoelker has quit IRC | 07:53 | |
| *** markus_z has joined #openstack-cinder | 07:53 | |
| *** shyama has quit IRC | 07:54 | |
| *** skylerberg has quit IRC | 07:54 | |
| *** shyama has joined #openstack-cinder | 07:55 | |
| *** alexschm has quit IRC | 07:55 | |
| *** salv-orlando has joined #openstack-cinder | 07:55 | |
| *** sgotliv has joined #openstack-cinder | 08:08 | |
| *** haomaiwang has quit IRC | 08:09 | |
| *** alexschm has joined #openstack-cinder | 08:10 | |
| *** haomaiwa_ has joined #openstack-cinder | 08:10 | |
| *** mylu has quit IRC | 08:14 | |
| *** afazekas has joined #openstack-cinder | 08:18 | |
| *** jistr has joined #openstack-cinder | 08:28 | |
| *** jordanP has joined #openstack-cinder | 08:28 | |
| *** sayali has quit IRC | 08:35 | |
| *** sayali has joined #openstack-cinder | 08:37 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add instructions on how to deploy API under Apache https://review.openstack.org/207020 | 08:39 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add Cinder API WSGI app to run under Apache/Nginx https://review.openstack.org/200551 | 08:39 |
| *** aix has joined #openstack-cinder | 08:41 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add instructions on how to deploy API under Apache https://review.openstack.org/207020 | 08:44 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add Cinder API WSGI app to run under Apache/Nginx https://review.openstack.org/200551 | 08:44 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add instructions on how to deploy API under Apache https://review.openstack.org/207020 | 08:45 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add Cinder API WSGI app to run under Apache/Nginx https://review.openstack.org/200551 | 08:45 |
| openstackgerrit | Peter Wang proposed openstack/cinder: Support efficient non-disruptive volume backup in VNX https://review.openstack.org/214626 | 08:47 |
| *** aagrawal has joined #openstack-cinder | 08:52 | |
| *** yuriy_n17 has joined #openstack-cinder | 08:54 | |
| *** ankit_ag has quit IRC | 08:55 | |
| *** alexschm has quit IRC | 08:59 | |
| *** markus_z has quit IRC | 08:59 | |
| *** lixiaoy1 has quit IRC | 09:02 | |
| *** ociuhandu has quit IRC | 09:07 | |
| *** haomaiwa_ has quit IRC | 09:09 | |
| *** haomaiwang has joined #openstack-cinder | 09:10 | |
| *** alexschm has joined #openstack-cinder | 09:11 | |
| *** markus_z has joined #openstack-cinder | 09:18 | |
| *** bluex-pl has joined #openstack-cinder | 09:19 | |
| *** jistr has quit IRC | 09:19 | |
| *** bluex-pl has quit IRC | 09:20 | |
| *** bluex-pl has joined #openstack-cinder | 09:20 | |
| *** jistr has joined #openstack-cinder | 09:20 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: Update update_migrated_volume according to Cinder https://review.openstack.org/215036 | 09:28 |
| *** zhonghua-lee has joined #openstack-cinder | 09:38 | |
| openstackgerrit | zhaohua proposed openstack/cinder: Huawei: fix multi REST-URLs bug https://review.openstack.org/214500 | 09:38 |
| *** ebalduf has quit IRC | 09:49 | |
| *** markvoelker has joined #openstack-cinder | 09:49 | |
| *** wilson has joined #openstack-cinder | 09:49 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 09:50 |
| *** aix has quit IRC | 09:51 | |
| *** gardenshed has quit IRC | 09:51 | |
| geguileo | tbarron: ping | 09:52 |
| *** markvoelker has quit IRC | 09:54 | |
| *** davechen has left #openstack-cinder | 09:54 | |
| *** dims has joined #openstack-cinder | 09:56 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: POC: Tooz locks demo https://review.openstack.org/185646 | 10:04 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 10:04 |
| *** tiny-hands has joined #openstack-cinder | 10:06 | |
| *** ociuhandu has joined #openstack-cinder | 10:06 | |
| *** tiny-hands has quit IRC | 10:08 | |
| *** gardenshed has joined #openstack-cinder | 10:08 | |
| *** yrabl has quit IRC | 10:09 | |
| *** haomaiwang has quit IRC | 10:09 | |
| *** yrabl has joined #openstack-cinder | 10:09 | |
| *** BharatK has left #openstack-cinder | 10:09 | |
| *** haomaiwang has joined #openstack-cinder | 10:10 | |
| *** yrabl has quit IRC | 10:13 | |
| *** yrabl has joined #openstack-cinder | 10:13 | |
| *** markus_z has quit IRC | 10:15 | |
| *** baojg has quit IRC | 10:33 | |
| *** baojg has joined #openstack-cinder | 10:34 | |
| *** hw_chenying has quit IRC | 10:34 | |
| *** baojg has quit IRC | 10:38 | |
| *** lpetrut has joined #openstack-cinder | 10:38 | |
| *** haomaiwang has quit IRC | 10:48 | |
| *** agarciam1 has joined #openstack-cinder | 10:48 | |
| *** gardenshed has quit IRC | 10:48 | |
| *** agarciam has quit IRC | 10:48 | |
| *** agarciam has joined #openstack-cinder | 10:51 | |
| *** agarciam1 has quit IRC | 10:52 | |
| *** zhonghua-lee has quit IRC | 11:01 | |
| *** agarciam1 has joined #openstack-cinder | 11:07 | |
| *** agarciam has quit IRC | 11:08 | |
| *** agarciam has joined #openstack-cinder | 11:10 | |
| *** agarciam1 has quit IRC | 11:11 | |
| *** dims has quit IRC | 11:15 | |
| *** dims has joined #openstack-cinder | 11:16 | |
| *** aix has joined #openstack-cinder | 11:16 | |
| *** takedakn has joined #openstack-cinder | 11:18 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/213891 | 11:20 |
| *** dims has quit IRC | 11:20 | |
| *** ganso_ has joined #openstack-cinder | 11:27 | |
| *** markus_z has joined #openstack-cinder | 11:30 | |
| *** dims has joined #openstack-cinder | 11:33 | |
| *** markvoelker has joined #openstack-cinder | 11:35 | |
| *** kairo has joined #openstack-cinder | 11:36 | |
| *** takedakn has quit IRC | 11:39 | |
| *** markvoelker has quit IRC | 11:40 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Python 3 incompatible expression fix https://review.openstack.org/215099 | 11:40 |
| *** alexschm has quit IRC | 11:42 | |
| *** bkopilov has quit IRC | 11:42 | |
| *** alexschm has joined #openstack-cinder | 11:43 | |
| *** salv-orlando has quit IRC | 11:45 | |
| *** afazekas has quit IRC | 11:46 | |
| *** dansmith has quit IRC | 11:47 | |
| *** dansmith has joined #openstack-cinder | 11:48 | |
| *** dansmith is now known as Guest95328 | 11:48 | |
| *** gardenshed has joined #openstack-cinder | 11:59 | |
| *** bluex-pl has quit IRC | 12:02 | |
| *** lpetrut has quit IRC | 12:02 | |
| *** bluex-pl has joined #openstack-cinder | 12:03 | |
| *** gardenshed has quit IRC | 12:03 | |
| *** bswartz has quit IRC | 12:08 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 12:11 |
| *** salv-orlando has joined #openstack-cinder | 12:11 | |
| *** lpetrut has joined #openstack-cinder | 12:17 | |
| *** haigang has quit IRC | 12:18 | |
| *** erlon has joined #openstack-cinder | 12:19 | |
| *** agarciam1 has joined #openstack-cinder | 12:19 | |
| *** takedakn has joined #openstack-cinder | 12:20 | |
| *** agarciam has quit IRC | 12:20 | |
| *** agarciam has joined #openstack-cinder | 12:22 | |
| *** edmondsw has joined #openstack-cinder | 12:23 | |
| *** agarciam1 has quit IRC | 12:23 | |
| *** ociuhandu_ has joined #openstack-cinder | 12:32 | |
| *** ociuhandu has quit IRC | 12:33 | |
| *** ociuhandu_ is now known as ociuhandu | 12:33 | |
| *** lpetrut has quit IRC | 12:39 | |
| *** markvoelker has joined #openstack-cinder | 12:39 | |
| *** mriedem has joined #openstack-cinder | 12:42 | |
| *** chlong has joined #openstack-cinder | 12:46 | |
| *** mriedem has quit IRC | 12:49 | |
| *** annegentle has joined #openstack-cinder | 12:50 | |
| *** martyturner has joined #openstack-cinder | 12:50 | |
| *** mriedem has joined #openstack-cinder | 12:51 | |
| *** cbader has quit IRC | 12:51 | |
| *** smoriya has quit IRC | 12:51 | |
| *** jistr is now known as jistr|mtg | 12:53 | |
| *** aix has quit IRC | 12:54 | |
| *** shausy has quit IRC | 12:56 | |
| *** shausy has joined #openstack-cinder | 12:56 | |
| *** bswartz has joined #openstack-cinder | 12:57 | |
| *** shausy has quit IRC | 12:58 | |
| *** kjnelson has joined #openstack-cinder | 12:58 | |
| *** porrua has joined #openstack-cinder | 12:58 | |
| *** lpetrut has joined #openstack-cinder | 12:59 | |
| *** aix has joined #openstack-cinder | 13:00 | |
| *** chlong has quit IRC | 13:00 | |
| *** bill_az has joined #openstack-cinder | 13:00 | |
| *** gouthamr has joined #openstack-cinder | 13:02 | |
| *** chlong has joined #openstack-cinder | 13:04 | |
| *** tiny-hands has joined #openstack-cinder | 13:05 | |
| *** Yogi1 has joined #openstack-cinder | 13:07 | |
| *** dustins has joined #openstack-cinder | 13:07 | |
| *** akerr has joined #openstack-cinder | 13:07 | |
| *** gardenshed has joined #openstack-cinder | 13:09 | |
| *** raildo-afk is now known as raildo | 13:10 | |
| *** kjnelson is now known as diablo_rojo | 13:12 | |
| *** bkopilov has joined #openstack-cinder | 13:15 | |
| *** krtaylor has quit IRC | 13:18 | |
| *** ishant has quit IRC | 13:21 | |
| *** aagrawal has quit IRC | 13:22 | |
| *** xyang1 has joined #openstack-cinder | 13:25 | |
| *** dustins has quit IRC | 13:26 | |
| *** dustins has joined #openstack-cinder | 13:26 | |
| *** jgregor has joined #openstack-cinder | 13:30 | |
| *** jamielennox is now known as jamielennox|away | 13:30 | |
| *** zul has joined #openstack-cinder | 13:30 | |
| *** krtaylor has joined #openstack-cinder | 13:31 | |
| *** dims_ has joined #openstack-cinder | 13:35 | |
| *** eharney has joined #openstack-cinder | 13:37 | |
| bswartz | thingee: ping | 13:37 |
| bswartz | thingee: nm, I found what I was looking for | 13:38 |
| *** diogogmt has joined #openstack-cinder | 13:39 | |
| * bswartz notes that google is faster than pinging people on IRC | 13:39 | |
| *** dims has quit IRC | 13:39 | |
| jordanP | bswartz, especially if they are asleep ! | 13:41 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Use cinder internal tenant to create the destination volume https://review.openstack.org/210237 | 13:43 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Python 3 incompatible expression fix https://review.openstack.org/215099 | 13:44 |
| *** dims_ has quit IRC | 13:44 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: Update update_migrated_volume according to Cinder https://review.openstack.org/215036 | 13:45 |
| *** dims has joined #openstack-cinder | 13:46 | |
| *** julim has joined #openstack-cinder | 13:46 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds the migration progress support for migration https://review.openstack.org/204953 | 13:46 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: WIP: Adds migration abortion https://review.openstack.org/207754 | 13:51 |
| *** thangp has joined #openstack-cinder | 13:51 | |
| *** xyang1 has quit IRC | 13:52 | |
| *** Lee1092 has quit IRC | 13:53 | |
| *** haomaiwang has joined #openstack-cinder | 13:53 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 13:56 |
| *** lcurtis has joined #openstack-cinder | 13:58 | |
| *** xyang1 has joined #openstack-cinder | 13:59 | |
| *** skraynev has quit IRC | 13:59 | |
| *** skraynev has joined #openstack-cinder | 14:00 | |
| *** Guest95328 is now known as dansmith | 14:00 | |
| *** diogogmt has quit IRC | 14:04 | |
| *** aix has quit IRC | 14:04 | |
| *** afazekas has joined #openstack-cinder | 14:07 | |
| *** jistr|mtg is now known as jistr | 14:08 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Use cinder internal tenant to create the destination volume https://review.openstack.org/210237 | 14:09 |
| *** aix has joined #openstack-cinder | 14:09 | |
| *** haomaiwang has quit IRC | 14:09 | |
| *** bill_az has quit IRC | 14:10 | |
| *** haomaiwang has joined #openstack-cinder | 14:10 | |
| *** ebalduf has joined #openstack-cinder | 14:11 | |
| *** Lee1092 has joined #openstack-cinder | 14:16 | |
| *** diogogmt has joined #openstack-cinder | 14:17 | |
| *** p0rtal has joined #openstack-cinder | 14:17 | |
| *** zul has quit IRC | 14:17 | |
| *** mtanino has joined #openstack-cinder | 14:17 | |
| *** vlaza has left #openstack-cinder | 14:26 | |
| *** sayali has quit IRC | 14:27 | |
| tbarron | geguileo: pong | 14:30 |
| geguileo | tbarron: Hi | 14:30 |
| tbarron | hi, and congrats! | 14:31 |
| *** albertom-afk is now known as albertom | 14:31 | |
| geguileo | tbarron: Thanks! | 14:31 |
| tbarron | i've been learning a lot from your reviews. But what's up? | 14:31 |
| geguileo | tbarron: I'm looking at some code you introduced: https://review.openstack.org/#/c/202570/1 | 14:31 |
| geguileo | tbarron: Currently in: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/netapp/dataontap/nfs_base.py#L681 | 14:31 |
| openstackgerrit | Jordan Pittier proposed openstack/cinder: Rework Scality SOFS driver to use RemoteFS class https://review.openstack.org/200610 | 14:31 |
| geguileo | tbarron: And I see a potential problem | 14:32 |
| geguileo | tbarron: Wanted to run it by you to confirm | 14:32 |
| tbarron | sure, please do | 14:32 |
| geguileo | tbarron: First the problem: If we have a list of locations and the first location is not an NFS location we will not look at any other location | 14:32 |
| *** yrabl has quit IRC | 14:33 | |
| geguileo | tbarron: Solution: Replace urls.append(url) and break with continue | 14:33 |
| geguileo | tbarron: Unless I'm missing something | 14:33 |
| tbarron | geguileo: I think you are right. | 14:33 |
| tbarron | is there another problem you see as well? | 14:34 |
| *** diogogmt has quit IRC | 14:34 | |
| geguileo | tbarron: Not really, that's the only one I saw | 14:34 |
| tbarron | damn, rcallawa isn't on this channel right now. He wrote that routine. | 14:35 |
| geguileo | tbarron: But I was just looking at that specific patch | 14:35 |
| tbarron | But I wrote the unit tests and did the commit, so it's just as much my fault or more !! | 14:35 |
| tbarron | geguileo: I'll make a bug unless you already have. | 14:36 |
| geguileo | tbarron: You didn't add negative tests | 14:36 |
| geguileo | tbarron: No, I wanted to talk it with you first | 14:36 |
| *** lpetrut has quit IRC | 14:37 | |
| tbarron | fix will be TDD: write the tests that make it fail, then get it to pass with the continue. | 14:37 |
| geguileo | tbarron: You can go ahead and file the bug | 14:37 |
| tbarron | geguileo: I will. Thanks much! | 14:37 |
| geguileo | tbarron: Thank you | 14:37 |
| *** lpetrut has joined #openstack-cinder | 14:38 | |
| *** cbader has joined #openstack-cinder | 14:39 | |
| *** RobertMetcalf has joined #openstack-cinder | 14:41 | |
| *** RobertMetcalf has quit IRC | 14:41 | |
| *** tsekiyama has joined #openstack-cinder | 14:41 | |
| *** rmetcalf has joined #openstack-cinder | 14:41 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Use cinder internal tenant to create the destination volume https://review.openstack.org/210237 | 14:42 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 14:42 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds the migration progress support for migration https://review.openstack.org/204953 | 14:44 |
| *** diablo_rojo has quit IRC | 14:45 | |
| *** dustins has quit IRC | 14:45 | |
| *** dustins has joined #openstack-cinder | 14:46 | |
| *** takedakn1 has joined #openstack-cinder | 14:46 | |
| *** takedakn has quit IRC | 14:50 | |
| *** raildo is now known as raildo-afk | 14:51 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 14:51 |
| *** agarciam1 has joined #openstack-cinder | 14:53 | |
| *** agarciam has quit IRC | 14:54 | |
| tbarron | geguileo: https://bugs.launchpad.net/grenade/+bug/1487074 | 14:54 |
| openstack | Launchpad bug 1487074 in grenade "NetApp NFS driver can miss valid glance image locations" [Undecided,New] - Assigned to Tom Barron (tpb) | 14:54 |
| geguileo | tbarron: Nice! | 14:55 |
| *** raildo-afk has quit IRC | 14:55 | |
| *** agarciam has joined #openstack-cinder | 14:56 | |
| *** agarciam1 has quit IRC | 14:57 | |
| *** annegentle has quit IRC | 14:58 | |
| *** annegentle has joined #openstack-cinder | 14:58 | |
| *** raildo has joined #openstack-cinder | 14:58 | |
| *** kjnelson has joined #openstack-cinder | 15:00 | |
| *** zul has joined #openstack-cinder | 15:02 | |
| *** alexschm has quit IRC | 15:03 | |
| *** vivek_nandavanam has joined #openstack-cinder | 15:03 | |
| *** p0rtal has quit IRC | 15:05 | |
| *** takedakn has joined #openstack-cinder | 15:07 | |
| *** alejandrito has joined #openstack-cinder | 15:08 | |
| *** hemnafk is now known as hemna | 15:08 | |
| *** takedakn1 has quit IRC | 15:09 | |
| *** haomaiwang has quit IRC | 15:09 | |
| *** haomaiwang has joined #openstack-cinder | 15:10 | |
| *** vincent_hou has joined #openstack-cinder | 15:12 | |
| vincent_hou | geguileo:hi | 15:12 |
| geguileo | vincent_hou: Hi | 15:12 |
| vincent_hou | How are you, dude? I am about to ask you a question. | 15:13 |
| geguileo | vincent_hou: I hope it's not too difficult :-) | 15:13 |
| *** ronis__ has quit IRC | 15:13 | |
| vincent_hou | will 'deleting' be the ending status for migration_status? | 15:13 |
| geguileo | vincent_hou: No | 15:14 |
| geguileo | vincent_hou: I modified the method that sets it to delete and it always set migration to None | 15:14 |
| *** nkrinner has quit IRC | 15:14 | |
| *** hodos has joined #openstack-cinder | 15:14 | |
| geguileo | vincent_hou: Let me look it in the patch | 15:15 |
| *** rmetcalf has quit IRC | 15:15 | |
| vincent_hou | OK. cinder/volume/api.py Line 358 | 15:15 |
| vincent_hou | if volume['migration_status'] not in (None, 'deleting'): | 15:16 |
| *** diogogmt has joined #openstack-cinder | 15:16 | |
| geguileo | vincent_hou: No, that's to make sure you can delete a volume that errored during migration | 15:16 |
| *** anshul has quit IRC | 15:17 | |
| *** gardenshed has quit IRC | 15:17 | |
| geguileo | vincent_hou: https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/api.py#L1210 | 15:17 |
| *** bluex-pl has quit IRC | 15:17 | |
| geguileo | vincent_hou: When we destroy a volume we set migration_status to None, that's the final value | 15:18 |
| vincent_hou | Then I need you to help me on https://review.openstack.org/#/c/186312/. In my patch, I will put either success or error to migration_status | 15:18 |
| vincent_hou | in the end. | 15:18 |
| geguileo | vincent_hou: I'm not sure that's a good idea | 15:19 |
| vincent_hou | This is according to the BP for migration. | 15:19 |
| geguileo | vincent_hou: Then you'll need to change a lot of code in Cinder | 15:19 |
| geguileo | vincent_hou: Because everywhere we expect that to be None if we are not migrating | 15:19 |
| *** diogogmt has quit IRC | 15:19 | |
| *** eharney has quit IRC | 15:19 | |
| vincent_hou | migration_status is used to saved the state for the last migration | 15:19 |
| vincent_hou | This is what I follow from vancouver summit. | 15:20 |
| vincent_hou | You can check my patch and my BP spec. | 15:20 |
| geguileo | vincent_hou: I'll have to check it indeed | 15:20 |
| thingee | vincent_hou: https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L358 | 15:21 |
| geguileo | vincent_hou: I guess that's part of the priority review list, right? | 15:21 |
| thingee | vincent_hou: you won't be able to delete the volume | 15:21 |
| *** dannywilson has joined #openstack-cinder | 15:21 | |
| *** BharatK has joined #openstack-cinder | 15:21 | |
| *** jistr is now known as jistr|mtg | 15:22 | |
| geguileo | vincent_hou: ^ That's one the cases I was referring to | 15:22 |
| *** diogogmt has joined #openstack-cinder | 15:22 | |
| geguileo | vincent_hou: And there are more places | 15:22 |
| geguileo | vincent_hou: Like you won't be able to migrate it again | 15:23 |
| geguileo | vincent_hou: Or probably retype it... | 15:23 |
| geguileo | vincent_hou: Everywhere we assume that if migration succeeded None is set in there | 15:23 |
| vincent_hou | geguileo: Please take a close look at my patch and the BP in it. We change the rule in the BP | 15:23 |
| vincent_hou | Admin should know if his previous migration is ok or not. | 15:24 |
| geguileo | vincent_hou: I only said that if you change the rule there are a lot of places that need to be changed | 15:24 |
| vincent_hou | That is OK. Not that many. | 15:24 |
| geguileo | vincent_hou: Can't he do that now? | 15:24 |
| vincent_hou | migration_status is always none in the end of migration. | 15:25 |
| vincent_hou | That is the current situation. | 15:25 |
| vincent_hou | admin can know it fails or not by looking at the log. | 15:25 |
| vincent_hou | thingee: Thanks, that is what I mean. | 15:26 |
| openstackgerrit | John Griffith proposed openstack/cinder: Move then newly added genconfig to its own testenv https://review.openstack.org/215192 | 15:31 |
| jgriffith | smcginnis: geguileo what do you guys think about just putting the full genconfig test back in the gate and making it non-voting? | 15:33 |
| *** zul has quit IRC | 15:33 | |
| smcginnis | jgriffith: Just reading your commit message now. | 15:33 |
| smcginnis | jgriffith: Sure, that makes sense. | 15:33 |
| smcginnis | jgriffith: I was a little unsure of that being part of pep8 testing. | 15:33 |
| jgriffith | smcginnis: There's been resistance to that in the past, I'm indifferent | 15:33 |
| jgriffith | smcginnis: well, I think part of the reasoning there was it "used to be there" and it was convenient to make it just get picked up by the existing gate jobs :) | 15:34 |
| *** andreykurilin__ has joined #openstack-cinder | 15:34 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 15:34 |
| *** haomaiwang has quit IRC | 15:35 | |
| *** r-daneel has joined #openstack-cinder | 15:36 | |
| *** takedakn has quit IRC | 15:36 | |
| jgriffith | BAHHH | 15:37 |
| jgriffith | I need to add the sample to ignores :( | 15:37 |
| openstackgerrit | John Griffith proposed openstack/cinder: Move then newly added genconfig to its own testenv https://review.openstack.org/215192 | 15:38 |
| jgriffith | at least for now | 15:38 |
| jgriffith | anyway... I'm not that set on it, just thought I'd throw some ideas out there | 15:39 |
| dannywilson | jgriffith: good morning, my apologies on your replication patch yesterday, I seem to have borked it while submitting my patch based on yours | 15:39 |
| jgriffith | dannywilson: LOL | 15:40 |
| jgriffith | dannywilson: it happens | 15:40 |
| dannywilson | jgriffith: yeah, and I probably shouldn't submit code while in a meeting either, oops | 15:40 |
| geguileo | jgriffith: Sounds like a good idea | 15:40 |
| mtanino | geguileo: hi, | 15:41 |
| dannywilson | jgriffith: do you want me to just re-submit patch set 8 or is there a better way to fix it? | 15:41 |
| geguileo | mtanino: Hi | 15:41 |
| mtanino | geguileo: I have a question about your comment for https://review.openstack.org/#/c/201243/ | 15:41 |
| jgriffith | dannywilson: I can do it, no problem. Have some updates to add for xyang1 in there anyway :) | 15:41 |
| geguileo | mtanino: Ok | 15:42 |
| jgriffith | dannywilson: gimme a half hour and I'll get it fixed up | 15:42 |
| xyang1 | jgriffith: The replication patch? | 15:42 |
| dannywilson | jgriffith: okay, no hurry, thanks | 15:42 |
| jgriffith | xyang1: yes | 15:42 |
| xyang1 | jgriffith: Good:) | 15:42 |
| mtanino | geguileo: your recommendation is init_vendor_properties() just returns a dictionary like a initialize_connection()? | 15:43 |
| *** vincent_hou has quit IRC | 15:44 | |
| mtanino | geguileo: without using set_vendor_properties(). | 15:44 |
| geguileo | mtanino: Yes, returns a dictionary with the properties it wants to add | 15:44 |
| openstackgerrit | Kurt Martin proposed openstack/cinder: Port 3PAR drivers to use ABCMeta driver model https://review.openstack.org/215200 | 15:44 |
| geguileo | mtanino: Withought the 'properties' key | 15:44 |
| *** Yogi1 has quit IRC | 15:45 | |
| geguileo | mtanino: Just what it wants to add | 15:45 |
| kvidvans | thingee, jgriffith: How do I get https://review.openstack.org/#/c/210608/11 added to https://etherpad.openstack.org/p/cinder-liberty-3-reviews? It passes all the requirements set out in:http://lists.openstack.org/pipermail/openstack-dev/2015-August/071505.html .Thanks | 15:45 |
| geguileo | mtanino: And in init_capabilites we either add the vendor_ part or just confirm that all keys have it | 15:46 |
| *** mriedem is now known as mriedem_away | 15:46 | |
| geguileo | mtanino: That way we are sure we are not overwriting any key from standard_capabilities | 15:46 |
| thingee | kvidvans: you type it on the etherpad. anyone can edit it | 15:46 |
| kvidvans | thanks..will do...Just wanted to confirm | 15:47 |
| geguileo | mtanino: And it's easier to see that we are setting capabilites['properties'] to the contents of those 2 returned dictionaries | 15:47 |
| geguileo | mtanino: The way it is now it's a lot harder to see that those 2 add their contents to capabilites['properties'] | 15:48 |
| mtanino | geguileo: Is it ok for creating a dictionary using existing set_vendor_properties or set_properties? | 15:48 |
| mtanino | geguileo: or using it's is harder to read what in the dictionary? | 15:48 |
| geguileo | mtanino: I don't follow what you mean | 15:49 |
| *** belmoreira has joined #openstack-cinder | 15:49 | |
| mtanino | geguileo: current code set properties using set method such as set_vendor_properties or set_properties. | 15:50 |
| *** crose has joined #openstack-cinder | 15:51 | |
| geguileo | mtanino: Current code is setting it in an instance attribute | 15:51 |
| *** salv-orlando has quit IRC | 15:52 | |
| mtanino | geguileo: yes. | 15:52 |
| geguileo | mtanino: It shouldn't | 15:52 |
| geguileo | mtanino: That's why I'm saying to return a dictionary in _init_standard_capabilities | 15:53 |
| mtanino | geguileo: and after that set the returned dictionary to self.capabilities['properties'] at init_capabilities()? | 15:55 |
| geguileo | mtanino: If you want you can pass a dictionary to _set_properties (and rename it to _set_property since we no longer are setting 'properties' key there) and do the same thing you are doing with set_capabilities but with that variable instead | 15:55 |
| geguileo | mtanino: Yes | 15:55 |
| mtanino | geguileo: ok, let me consider again. | 15:56 |
| geguileo | mtanino: Sure | 15:56 |
| *** AlexeyElagin has joined #openstack-cinder | 15:57 | |
| *** belmoreira has quit IRC | 15:58 | |
| *** eharney has joined #openstack-cinder | 15:59 | |
| *** salv-orlando has joined #openstack-cinder | 15:59 | |
| *** JoseMello has joined #openstack-cinder | 16:01 | |
| *** esker has quit IRC | 16:07 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Continue conversion to volume object https://review.openstack.org/201404 | 16:09 |
| *** leeantho has joined #openstack-cinder | 16:09 | |
| *** jistr|mtg is now known as jistr | 16:10 | |
| *** crose has quit IRC | 16:10 | |
| openstackgerrit | Diem Tran proposed openstack/cinder: Local cache feature of Oracle ZFSSA drivers https://review.openstack.org/212155 | 16:10 |
| *** porrua has quit IRC | 16:10 | |
| *** _cjones_ has joined #openstack-cinder | 16:13 | |
| *** Apoorva has joined #openstack-cinder | 16:14 | |
| hemna | patrickeast, ping | 16:18 |
| *** rajinir has quit IRC | 16:18 | |
| *** krtaylor has quit IRC | 16:18 | |
| patrickeast | hemna: hey | 16:19 |
| hemna | so, friendly names SUCK | 16:19 |
| hemna | suck suck suck | 16:19 |
| hemna | looks like there is a problem with flushing mpath devices when friendly names is enabled | 16:19 |
| hemna | multipath -f <wwn/dm-X> doesn't do anything | 16:19 |
| patrickeast | crap | 16:19 |
| hemna | turn off friendly names and multipath -f <wwn/dm-X> works | 16:19 |
| hemna | looks like a bug in the daemon or the userspace multipath tool | 16:20 |
| patrickeast | yea that seems pretty weird | 16:20 |
| hemna | so I'm going to send the dm folks an email, as they don't seem to have a web site for tracking bugs. | 16:20 |
| hemna | dm@redhat.com looks like a contact ? | 16:21 |
| hemna | err | 16:21 |
| hemna | dm-devel@redhat.com | 16:21 |
| hemna | http://christophe.varoqui.free.fr/ | 16:21 |
| patrickeast | sounds like a good idea | 16:21 |
| patrickeast | although even if it is a bug, would we be able to still use the naming shortcut? | 16:22 |
| hemna | so it looks like the /dev/disk/by-id/dm-uuid-mpath-<WWN> does work | 16:23 |
| hemna | I'm working on updating my patch to look for that | 16:23 |
| hemna | it's there with or without friendly names enabled | 16:23 |
| hemna | but I'm also adding a fallback lookup for /dev/mapper/<WWN> | 16:23 |
| patrickeast | gotcha | 16:24 |
| hemna | and then another fallback to run multipath -ll <original /dev/disk/by-path/holy crap long device path> | 16:24 |
| hemna | to try and make this stuff much more robust | 16:24 |
| *** zul has joined #openstack-cinder | 16:24 | |
| hemna | the problem is friendly names on detach | 16:24 |
| hemna | multipath -f <device> does nothing | 16:24 |
| hemna | :(!! | 16:24 |
| hemna | so a bunch of crap is left around in /dev/disk/by-* | 16:25 |
| *** lpetrut has quit IRC | 16:26 | |
| *** crose has joined #openstack-cinder | 16:26 | |
| *** mriedem_away is now known as mriedem | 16:27 | |
| *** mylu has joined #openstack-cinder | 16:27 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Cinder replication V2 https://review.openstack.org/200269 | 16:28 |
| jgriffith | dannywilson: patrickeast xyang1 ok, fixed up whatever happened with that bad merge I think | 16:29 |
| jgriffith | dannywilson: patrickeast xyang1 responses are in patch 8, updated most of them in the new set | 16:29 |
| jgriffith | xyang1: bahhh.... backup controller!!! | 16:30 |
| dannywilson | jgriffith: thanks | 16:30 |
| *** salv-orlando has quit IRC | 16:30 | |
| xyang1 | jgriffith: thanks, I'll take a look of your new patch | 16:30 |
| openstackgerrit | John Griffith proposed openstack/cinder: Cinder replication V2 https://review.openstack.org/200269 | 16:32 |
| *** julim has quit IRC | 16:34 | |
| *** yuriy_n17 has quit IRC | 16:34 | |
| *** jwcroppe_ has quit IRC | 16:35 | |
| dannywilson | jgriffith: any progress on parsing of replication_devices values in base class? | 16:37 |
| jgriffith | dannywilson: not sure which part you mean :) | 16:38 |
| dannywilson | jgriffith: parsing out the replication_devices='backend':['biz', ....] info | 16:38 |
| jgriffith | dannywilson: in the list_secondaries call? | 16:39 |
| dannywilson | jgriffith: or even at set up time | 16:39 |
| aorourke_ | xyang1, I noticed that if you add a volume to a consistency group, then add that same volume to a new cg it immediately overwrites the volume to belong to the new cg. Is this something that could cause issues? Seems like it could be really easy to break consistency. Wouldn't it be best to force the user to delete the volume from the CG before it can be added to a new one? | 16:39 |
| dannywilson | jgriffith: is there an easy way to parse those out? I might be missing a simple explanation | 16:40 |
| jgriffith | dannywilson: sorry... I'm like deer in the headlights right now | 16:40 |
| jgriffith | dannywilson: not exactly sure which part you are looking at; so the config file entries? No, but I'll work that out next | 16:40 |
| dannywilson | jgriffith: no problem, yes the config file entries | 16:41 |
| jgriffith | dannywilson: there's also the part about how to specify the optional secondary, I kinda need to play with that | 16:41 |
| jgriffith | dannywilson: ahh... ok, yeah; so the doc I added it, but I'm going to go back to allowing multiples I think. | 16:41 |
| jgriffith | dannywilson: that's next on my list after the two things I'm working right now :) | 16:41 |
| *** bill_az has joined #openstack-cinder | 16:42 | |
| dannywilson | jgriffith: sounds good, thanks | 16:42 |
| xyang1 | aorourke_: It seems that a check is missing to prevent that. Let me think if there were any reason why this is allowed. By the way, a CG itself is not guaranteed to be consistent, only the Cgsnapshot is | 16:43 |
| *** jistr has quit IRC | 16:46 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add ability to update provider_id during init https://review.openstack.org/215231 | 16:49 |
| *** agarciam has quit IRC | 16:49 | |
| *** lpetrut has joined #openstack-cinder | 16:51 | |
| *** lpetrut has quit IRC | 16:52 | |
| openstackgerrit | Jordan Pittier proposed openstack/cinder: Rework Scality SOFS driver to use RemoteFS class https://review.openstack.org/200610 | 16:55 |
| *** Yogi1 has joined #openstack-cinder | 16:55 | |
| aorourke_ | xyang1, makes sense. I just meant it would be very easy for someone else to come along and assign a volume to a new CG and not even realize they removed it from a different one. | 16:59 |
| xyang1 | aorourke_: Ya, I think a check should be added | 17:00 |
| *** bill_az has quit IRC | 17:02 | |
| *** afazekas has quit IRC | 17:02 | |
| *** vilobhmm1 has joined #openstack-cinder | 17:03 | |
| DuncanT | xyang1: Definitely sounds like a check that should be there, otherwise it 'feels' like a volume is a member of 2 cgs | 17:03 |
| *** tsekiyama has quit IRC | 17:04 | |
| xyang1 | DuncanT: I think you are right, it will be removed silently from group1 and added to group2:( | 17:05 |
| *** skylerberg has joined #openstack-cinder | 17:06 | |
| *** jordanP has quit IRC | 17:06 | |
| DuncanT | xyang1: Just a bug, easy to fix | 17:06 |
| xyang1 | DuncanT: Ya, should be a easy fix, I'll fix it soon | 17:06 |
| xyang1 | aorourke_: You want to open a bug and send me the link | 17:07 |
| *** markus_z has quit IRC | 17:08 | |
| *** edtubill has joined #openstack-cinder | 17:11 | |
| aorourke_ | xyang1, yes I can | 17:14 |
| *** crose has quit IRC | 17:17 | |
| *** ronis__ has joined #openstack-cinder | 17:19 | |
| *** julim has joined #openstack-cinder | 17:20 | |
| *** aix has quit IRC | 17:22 | |
| *** mylu has quit IRC | 17:22 | |
| *** mylu has joined #openstack-cinder | 17:23 | |
| aorourke_ | xyang1, https://bugs.launchpad.net/cinder/+bug/1487151 | 17:29 |
| openstack | Launchpad bug 1487151 in Cinder "There should be a check in place when adding a volume that is already in a CG to a new CG " [Undecided,New] | 17:29 |
| xyang1 | aorourke_: Thanks | 17:29 |
| *** kjnelson has quit IRC | 17:29 | |
| *** sgotliv has quit IRC | 17:36 | |
| *** edtubill has left #openstack-cinder | 17:36 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Prevent volume already in CG to be added to another https://review.openstack.org/215262 | 17:39 |
| xyang1 | aorourke_: DuncanT , fix is submitted | 17:39 |
| *** gardenshed has joined #openstack-cinder | 17:40 | |
| *** salv-orlando has joined #openstack-cinder | 17:42 | |
| aorourke_ | xyang1, testing it now | 17:42 |
| xyang1 | aorourke_: Thanks | 17:42 |
| *** Lee1092 has quit IRC | 17:43 | |
| *** vilobhmm1 has quit IRC | 17:44 | |
| *** vilobhmm1 has joined #openstack-cinder | 17:45 | |
| *** salv-orlando has quit IRC | 17:46 | |
| *** krtaylor has joined #openstack-cinder | 17:47 | |
| *** lpetrut has joined #openstack-cinder | 17:48 | |
| *** breitz has quit IRC | 17:52 | |
| *** breitz has joined #openstack-cinder | 17:52 | |
| *** vilobhmm11 has joined #openstack-cinder | 17:54 | |
| *** vilobhmm1 has quit IRC | 17:54 | |
| *** vilobhmm1 has joined #openstack-cinder | 17:55 | |
| *** dustins has quit IRC | 17:56 | |
| *** tsekiyama has joined #openstack-cinder | 17:56 | |
| *** kjnelson has joined #openstack-cinder | 17:57 | |
| *** vilobhmm11 has quit IRC | 17:58 | |
| *** gardenshed has quit IRC | 18:01 | |
| *** kjnelson has quit IRC | 18:04 | |
| *** kjnelson has joined #openstack-cinder | 18:05 | |
| *** lpetrut has quit IRC | 18:06 | |
| *** bill_az has joined #openstack-cinder | 18:09 | |
| *** bapalm has quit IRC | 18:11 | |
| *** bapalm has joined #openstack-cinder | 18:14 | |
| *** vivek_nandavanam has quit IRC | 18:18 | |
| *** porrua has joined #openstack-cinder | 18:23 | |
| *** tpsilva has joined #openstack-cinder | 18:23 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: FC Stop calling multipath command line https://review.openstack.org/213389 | 18:23 |
| hemna | patrickeast, xyang1 ^^ | 18:25 |
| xyang1 | hemna: Hi | 18:25 |
| hemna | patrickeast, xyang1 so I think that one might be the winner (to mostly avoid calling multipath -l) for FC attaches/detaches | 18:27 |
| hemna | and it seems to work with friendly names enabled and disabled | 18:27 |
| hemna | friendly names needs to DIE DIE DIE | 18:27 |
| patrickeast | lol | 18:28 |
| xyang1 | hemna: Ok, I'll take a look:) | 18:28 |
| smcginnis | hemna: How do you really feel? | 18:30 |
| hemna | ok, maybe not that harsh.... | 18:31 |
| hemna | how about DIE DIE instead ? | 18:31 |
| smcginnis | :) | 18:31 |
| *** jwcroppe has joined #openstack-cinder | 18:35 | |
| *** dustins has joined #openstack-cinder | 18:37 | |
| *** dannywilson has quit IRC | 18:44 | |
| *** dannywilson has joined #openstack-cinder | 18:46 | |
| *** porrua has quit IRC | 18:51 | |
| thrawn01 | I don't really expect anyone to know, but why is 'imageRef' on the volume create call, camel case and all the other parameters are underscore? | 18:54 |
| *** shyama has quit IRC | 18:55 | |
| *** porrua has joined #openstack-cinder | 18:55 | |
| *** afazekas has joined #openstack-cinder | 18:55 | |
| jbernard | hemna: have you ever seen a case where oslo failed to correctly serialize a connector (os-brick) object and threw a cicrulare reference value error? | 18:56 |
| hemna | ?! | 18:56 |
| hemna | no | 18:56 |
| hemna | why is oslo serializing a connector object? | 18:56 |
| jbernard | hemna: so, im calling rpcapi.terminate_connection() | 18:57 |
| hemna | ok, so that connector is just a dict | 18:57 |
| jbernard | and im passing the connector object as the third paramater | 18:57 |
| *** ociuhandu has quit IRC | 18:58 | |
| jbernard | when i pass anything else (None, properties, whatever) it succeeds, but when i pass the correct connector, i get: http://sprunge.us/XLbC | 18:58 |
| hemna | hrmm | 18:58 |
| jbernard | admitedly, i havent dug into this yet, just curious if you had run into it before or had a hunch | 18:59 |
| hemna | no I've not seen that | 19:00 |
| jbernard | alright, thanks for looking; ill see if I can get it sorted out | 19:00 |
| *** zhonghua-lee has joined #openstack-cinder | 19:00 | |
| *** hw_chenying has joined #openstack-cinder | 19:01 | |
| *** kjnelson has quit IRC | 19:02 | |
| *** afazekas has quit IRC | 19:03 | |
| openstackgerrit | Sonia Ghanekar proposed openstack/cinder: Add manage/unmanage volume support for Nimble https://review.openstack.org/214392 | 19:04 |
| *** dannywilson has quit IRC | 19:04 | |
| *** dannywilson has joined #openstack-cinder | 19:05 | |
| *** zhonghua-lee has quit IRC | 19:06 | |
| *** kjnelson has joined #openstack-cinder | 19:09 | |
| *** dannywilson has quit IRC | 19:10 | |
| *** dannywilson has joined #openstack-cinder | 19:10 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Update provider_id column on SolidFire init https://review.openstack.org/215292 | 19:13 |
| *** bill_az has quit IRC | 19:13 | |
| jgriffith | hmmm... not sure how to make that work: http://logs.openstack.org/31/215231/1/check/gate-cinder-pep8/1a29393/console.html#_2015-08-20_17_55_40_221 | 19:14 |
| openstackgerrit | Diem Tran proposed openstack/cinder: Local cache feature of Oracle ZFSSA drivers https://review.openstack.org/212155 | 19:14 |
| smcginnis | jgriffith: hmmm | 19:15 |
| *** hw_chenying has quit IRC | 19:15 | |
| jgriffith | smcginnis: :) | 19:15 |
| smcginnis | indexted? :P | 19:16 |
| jgriffith | trying it now, but it blew up the first thing I tried (just replacing dict(xxx) with a comprehnsion | 19:16 |
| jgriffith | smcginnis: hmm? What do you mean? | 19:16 |
| jgriffith | return dict((d[key], dict(d, index=i)) for (i, d) in enumerate(seq)) | 19:17 |
| smcginnis | jgriffith: Spelling error in the commit. -2!! | 19:17 |
| jgriffith | is what it doesn't like | 19:17 |
| * smcginnis is kidding around | 19:17 | |
| jgriffith | DANG it... noooooooo | 19:17 |
| smcginnis | s/commit/comment/ | 19:17 |
| jgriffith | shoot | 19:18 |
| smcginnis | Seriously though, I'll try a couple things and see if I can figure it out. | 19:18 |
| jgriffith | its' not spelled wrong at least :) | 19:18 |
| jgriffith | just the wrong word :) | 19:18 |
| smcginnis | I think it should be indexed, not indexted. But seriously, I just pointed it out to be a PITA. Not really a concern. :) | 19:19 |
| jgriffith | Ohh... I thought you meant I type commit instead of comment DURP | 19:20 |
| smcginnis | Oh, nope. I even correct my own typos. | 19:20 |
| smcginnis | :) | 19:20 |
| smcginnis | I should really just shut up, I know. | 19:20 |
| jgriffith | smcginnis: I'll fix it if I can figure out how the heck to convert that to a comprehension | 19:20 |
| smcginnis | jgriffith: Definitely the more important matter. Trying a couple variations right now to see if I can make it happy. | 19:21 |
| *** zul has quit IRC | 19:23 | |
| *** dims_ has joined #openstack-cinder | 19:26 | |
| smcginnis | jgriffith: return {i : dict(d, index=i) for (i, d) in enumerate(lst)} | 19:27 |
| smcginnis | ? | 19:27 |
| *** nzrf has quit IRC | 19:27 | |
| *** rex_lee has quit IRC | 19:27 | |
| *** jamielennox|away has quit IRC | 19:27 | |
| *** dims has quit IRC | 19:28 | |
| *** DuncanT has quit IRC | 19:28 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Small cleanups in BaseVD/VolumeDriver https://review.openstack.org/215301 | 19:28 |
| *** r-daneel has quit IRC | 19:28 | |
| *** serverascode has quit IRC | 19:28 | |
| *** bill_az has joined #openstack-cinder | 19:29 | |
| *** DuncanT has joined #openstack-cinder | 19:31 | |
| *** serverascode has joined #openstack-cinder | 19:32 | |
| *** nzrf has joined #openstack-cinder | 19:33 | |
| *** rex_lee has joined #openstack-cinder | 19:34 | |
| jgriffith | smcginnis: indeed, that works.... let's see what pep8 thinks of it | 19:38 |
| *** crose has joined #openstack-cinder | 19:39 | |
| jgriffith | smcginnis: makes pep8 happy | 19:41 |
| smcginnis | Happy dance. | 19:41 |
| *** salv-orlando has joined #openstack-cinder | 19:41 | |
| jgriffith | smcginnis: but it doesn't set the key correct it appears... hmmm, let's see | 19:42 |
| smcginnis | jgriffith: Dang. | 19:43 |
| *** breitz has quit IRC | 19:43 | |
| jgriffith | smcginnis: ahh... I see... | 19:43 |
| *** breitz has joined #openstack-cinder | 19:43 | |
| jgriffith | hmm... so I just need to figure out how to suck out that specific key | 19:43 |
| jgriffith | smcginnis: what you have is cool, but it just uses the enumerate methods index | 19:44 |
| jgriffith | (as the index key) | 19:44 |
| *** r-daneel has joined #openstack-cinder | 19:44 | |
| jgriffith | think I got it.... | 19:45 |
| smcginnis | jgriffith: Hopefully not too far off. | 19:46 |
| *** esker has joined #openstack-cinder | 19:46 | |
| *** esker has quit IRC | 19:47 | |
| *** alejandrito has quit IRC | 19:47 | |
| *** jamielennox|away has joined #openstack-cinder | 19:47 | |
| *** esker has joined #openstack-cinder | 19:47 | |
| *** jamielennox|away is now known as jamielennox | 19:47 | |
| *** cppforlife_ has quit IRC | 19:49 | |
| jgriffith | smcginnis: Eureka!!! {d[key]: dict(d, index=d[key]) for (i, d) in enumerate(seq)} | 19:49 |
| jgriffith | smcginnis: thanks for the help! | 19:49 |
| smcginnis | jgriffith: No problem. Glad you figured it out. | 19:49 |
| *** alejandrito has joined #openstack-cinder | 19:50 | |
| *** kairo has left #openstack-cinder | 19:50 | |
| jgriffith | s/you/we/ | 19:50 |
| *** cppforlife_ has joined #openstack-cinder | 19:50 | |
| smcginnis | :) | 19:51 |
| *** leeantho has quit IRC | 19:52 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add ability to update provider_id during init https://review.openstack.org/215231 | 19:53 |
| *** resker has joined #openstack-cinder | 19:53 | |
| jgriffith | smcginnis: It's python!!! Nobody worries about things like memory consumption :) | 19:54 |
| *** andreykurilin__ has quit IRC | 19:55 | |
| *** resker has quit IRC | 19:56 | |
| smcginnis | jgriffith: True. ;) | 19:56 |
| *** resker has joined #openstack-cinder | 19:57 | |
| *** esker has quit IRC | 19:57 | |
| thingee | tsekiyama: how do I enable the store? https://review.openstack.org/#/c/201754/ | 19:58 |
| thingee | tsekiyama: I want to test this | 19:59 |
| tsekiyama | thingee: Just by adding "stores = cinder [,file, ...]" into [glance_store] of glance-api.conf | 20:00 |
| *** ronis__ has quit IRC | 20:01 | |
| thingee | tsekiyama: what? | 20:03 |
| tsekiyama | thingee: Do you mean you want to know how to activate cinder store in Glance? | 20:03 |
| *** annegentle has quit IRC | 20:04 | |
| *** resker has quit IRC | 20:04 | |
| thingee | yes | 20:04 |
| *** annegentle has joined #openstack-cinder | 20:04 | |
| tsekiyama | thingee: glance-api.conf has a "store" entry which is a comma-separated list of enabled stores in [glance_store] section, so you should add "cinder" to it. | 20:05 |
| *** angela-s has joined #openstack-cinder | 20:05 | |
| thingee | that's it? | 20:06 |
| *** IanGovett has quit IRC | 20:06 | |
| tsekiyama | thingee: yeah, as far as default settings is used. | 20:06 |
| thingee | I pull this patch in for cinder, add that to the glance-api.conf file and that's it. | 20:06 |
| tsekiyama | thingee: right. Ah, if you want to use internal tenant for storing images, the Glance patch and additional settings is needed | 20:08 |
| * thingee is confused | 20:09 | |
| thingee | tsekiyama: for this patch https://review.openstack.org/#/c/201754/ ... what are all the steps I need to test it? | 20:09 |
| *** alejandrito has quit IRC | 20:09 | |
| thingee | tsekiyama: I can't approve this change until I'm able to verify things | 20:09 |
| tsekiyama | For cinder.conf, glance_api_version=2, allowed_direct_url_schemes=cinder, and image_upload_use_cinder_backend=True. | 20:10 |
| tsekiyama | For glance-api.conf, "store=cinder", show_multiple_locations=True. | 20:11 |
| tsekiyama | thingee: that's all. | 20:11 |
| thingee | tsekiyama: thanks | 20:12 |
| thingee | tsekiyama: how does glance document how to enable certian glance stores? | 20:13 |
| *** alejandrito has joined #openstack-cinder | 20:14 | |
| tsekiyama | thingee: please see http://docs.openstack.org/developer/glance/configuring.html "Configuring Glance Storage Backends" | 20:15 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 20:17 |
| *** ndipanov has joined #openstack-cinder | 20:20 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Update provider_id column on SolidFire init https://review.openstack.org/215292 | 20:22 |
| *** crose has quit IRC | 20:25 | |
| *** BharatK has quit IRC | 20:27 | |
| *** e0ne has joined #openstack-cinder | 20:31 | |
| *** martyturner has quit IRC | 20:32 | |
| patrickeast | eharney: hey, any chance you could take another look at https://review.openstack.org/#/c/211804/ ? | 20:34 |
| eharney | patrickeast: sure, thanks for filling me in there | 20:34 |
| *** ndipanov has quit IRC | 20:35 | |
| patrickeast | eharney: np | 20:37 |
| openstackgerrit | Patrick East proposed openstack/cinder: Implement manage/unmanage snapshot in Pure drivers https://review.openstack.org/213855 | 20:38 |
| *** zul has joined #openstack-cinder | 20:39 | |
| *** boris-42 has joined #openstack-cinder | 20:41 | |
| *** divab has joined #openstack-cinder | 20:43 | |
| *** thangp has quit IRC | 20:43 | |
| *** divab is now known as abhi | 20:48 | |
| openstackgerrit | Sonia Ghanekar proposed openstack/cinder: Add manage/unmanage volume support for Nimble https://review.openstack.org/214392 | 20:48 |
| *** bswartz has quit IRC | 20:50 | |
| *** bill_az has quit IRC | 20:50 | |
| *** hodos has quit IRC | 20:51 | |
| *** tpsilva has quit IRC | 20:57 | |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Cinder Nested Quota Driver https://review.openstack.org/205369 | 20:57 |
| *** e0ne has quit IRC | 20:58 | |
| *** dannywil_ has joined #openstack-cinder | 21:00 | |
| *** ndipanov has joined #openstack-cinder | 21:01 | |
| openstackgerrit | Nikesh proposed openstack/cinder: Avoid returning volume metadata in DotHill driver https://review.openstack.org/215330 | 21:01 |
| *** ThermalSpan has joined #openstack-cinder | 21:02 | |
| ThermalSpan | Hello, would this be an appropriate channel for general cinder questions? If not where should I go? | 21:03 |
| *** dannywilson has quit IRC | 21:03 | |
| *** ThermalSpan has left #openstack-cinder | 21:04 | |
| *** p0rtal has joined #openstack-cinder | 21:09 | |
| *** p0rtal has quit IRC | 21:09 | |
| *** p0rtal has joined #openstack-cinder | 21:10 | |
| *** g_guest_t has joined #openstack-cinder | 21:12 | |
| *** crose has joined #openstack-cinder | 21:13 | |
| g_guest_t | hi everyone. I've got a driver for NEC M100 for Cinder Icehouse. Will it work fine for the Juno release or there must be some considerable changes done? | 21:14 |
| *** zul has quit IRC | 21:18 | |
| *** erlon has quit IRC | 21:18 | |
| *** annegentle has quit IRC | 21:20 | |
| *** annegentle has joined #openstack-cinder | 21:20 | |
| *** tiny-hands has left #openstack-cinder | 21:21 | |
| *** eharney has quit IRC | 21:21 | |
| *** krtaylor has quit IRC | 21:26 | |
| *** gouthamr has quit IRC | 21:27 | |
| openstackgerrit | Sonia Ghanekar proposed openstack/cinder: Add manage/unmanage volume support for Nimble https://review.openstack.org/214392 | 21:30 |
| *** crose has quit IRC | 21:32 | |
| *** dustins has quit IRC | 21:32 | |
| *** vilobhmm1 has quit IRC | 21:34 | |
| *** ndipanov has quit IRC | 21:35 | |
| *** vilobhmm1 has joined #openstack-cinder | 21:36 | |
| *** vilobhmm11 has joined #openstack-cinder | 21:37 | |
| *** dannywil_ has quit IRC | 21:40 | |
| *** vilobhmm1 has quit IRC | 21:40 | |
| *** nikeshm has joined #openstack-cinder | 21:40 | |
| *** annegentle has quit IRC | 21:42 | |
| *** edmondsw has quit IRC | 21:44 | |
| *** pv has joined #openstack-cinder | 21:44 | |
| *** jgregor has left #openstack-cinder | 21:44 | |
| pv | hey i had a question | 21:45 |
| pv | im testing my driver's attach/detach functionality and its an FC driver | 21:45 |
| pv | my initialize_connection code works up to the point where it says it completed successfully from the volume manager | 21:45 |
| pv | but my n-cpu log is giving me an error saying the attach failed because "no fc hosts could be found" | 21:45 |
| pv | my machine is definitely hba enabled, and when i do a systool -c fc_host -v the wwn matches the one that the connector passes into initialize_connection | 21:46 |
| pv | n-cpu log http://pastebin.com/wJHeaTsn | 21:47 |
| pv | 2015-08-20 14:42:30.639 INFO cinder.volume.manager [req-0a07de9b-0110-49fa-9106-ed7dbae54e8c None] [volume-6415de2f-a2af-4ddb-a60a-b3a26c315bc9] Initialize volume connection completed successfully. | 21:47 |
| pv | from my c-vol log | 21:47 |
| *** alejandrito has quit IRC | 21:49 | |
| *** bapalm has quit IRC | 21:49 | |
| *** bapalm has joined #openstack-cinder | 21:52 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add SolidFire svip to config options https://review.openstack.org/215354 | 21:52 |
| *** alejandrito has joined #openstack-cinder | 21:53 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Update provider_id column on SolidFire init https://review.openstack.org/215292 | 21:54 |
| *** dannywilson has joined #openstack-cinder | 21:54 | |
| *** akerr has quit IRC | 21:55 | |
| jgriffith | xyang1: awesome! Thanks for testing and feedback on the replication patch | 21:55 |
| jgriffith | xyang1: I'm working on the list targets and config stuff now | 21:56 |
| xyang1 | jgriffith: Welcome! Great | 21:56 |
| xyang1 | jgriffith: Ya, I couldn't figure out how to test that:) | 21:56 |
| xyang1 | jgriffith: I mean list replication targets | 21:57 |
| *** bapalm has quit IRC | 21:57 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Adds framework for get_capabilities() feature https://review.openstack.org/201243 | 21:57 |
| jgriffith | xyang1: yeah... there's no good way to unit test it as it's all up to the driver really | 21:57 |
| xyang1 | jgriffith: I am trying to call it from cinderclient | 21:58 |
| xyang1 | jgriffith: But couldn't send the request over | 21:58 |
| xyang1 | jgriffith: One question on the volume that has been failed over | 22:00 |
| *** porrua has quit IRC | 22:00 | |
| *** Yogi1 has quit IRC | 22:00 | |
| *** diogogmt_ has joined #openstack-cinder | 22:00 | |
| *** diogogmt has quit IRC | 22:00 | |
| *** diogogmt_ is now known as diogogmt | 22:00 | |
| jgriffith | xyang1: sure? | 22:00 |
| xyang1 | jgriffith: So after failover is complete and volume is in available status, we can still use it as usual? | 22:01 |
| jgriffith | xyang1: yes | 22:01 |
| xyang1 | jgriffith: Ok | 22:01 |
| jgriffith | xyang1: so the idea was to just update the model info as needed so it's somewhat seemless | 22:01 |
| xyang1 | jgriffith: sure | 22:02 |
| *** kjnelson has quit IRC | 22:02 | |
| xyang1 | jgriffith: So if the host is changed, driver needs to return that info thru model update? | 22:03 |
| kmartin | pv, is your FC SAN zoned correctly or are you using the FC Zone Manager? | 22:03 |
| *** bapalm has joined #openstack-cinder | 22:03 | |
| jgriffith | xyang1: Yes, so that would be something in the replication_updates data | 22:03 |
| pv | also in https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connector.py#L853 the requirements for FC in connection_properties say target_iqn, target_portal and target_lun | 22:03 |
| jgriffith | xyang1: informs the manager that it needs to update something | 22:03 |
| kmartin | pv, either way it sounds like a zoning problem | 22:03 |
| pv | thats a typo right? | 22:04 |
| xyang1 | jgriffith: Ok, thanks | 22:04 |
| jgriffith | xyang1: sure | 22:04 |
| pv | its zoned correctly, im not using zone manager currently | 22:04 |
| kmartin | pv, is this on a single node devstack? | 22:05 |
| pv | yes | 22:05 |
| *** p0rtal has quit IRC | 22:05 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Efficient volume copy for generic volume migration https://review.openstack.org/183701 | 22:06 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Allow specified backend capabilities to be retrieved https://review.openstack.org/212861 | 22:06 |
| *** bapalm has quit IRC | 22:08 | |
| kmartin | pv, can you paste your systool output and the connection info...it sure seems like a zoning issue | 22:11 |
| *** rlucio has joined #openstack-cinder | 22:11 | |
| hemna | pv, https://review.openstack.org/#/c/209762/ | 22:12 |
| *** bapalm has joined #openstack-cinder | 22:14 | |
| hemna | pv, sounds like a zoning problem, and or a connectivity problem. | 22:15 |
| *** pv has quit IRC | 22:16 | |
| angela-s | LOL, oh man, was going to try and help with the supposed zoning problem, but we lost pv | 22:18 |
| *** krtaylor has joined #openstack-cinder | 22:18 | |
| hemna | angela-s, I don't think he was using the FCZM fwiw | 22:18 |
| hemna | angela-s, hi by the way :) | 22:18 |
| angela-s | hemna: yeah, but i can still help with zoning issue... if it's a brocade switch | 22:19 |
| angela-s | oh yeah, hi! | 22:19 |
| *** bapalm has quit IRC | 22:19 | |
| *** annegentle has joined #openstack-cinder | 22:20 | |
| angela-s | hemna: yeah, i've been MIA for last week or so, vacation | 22:20 |
| hemna | what's a vacation ? | 22:20 |
| hemna | is that the thing where you like, don't go to work ? | 22:20 |
| angela-s | apparently. i actually was having so much fun at Disneyland, i forgot about work for once | 22:21 |
| hemna | whoa | 22:21 |
| hemna | I forgot what that's like | 22:21 |
| angela-s | i know, didn't even realize it for about 2 days. | 22:21 |
| angela-s | it was a revelation | 22:22 |
| hemna | huh, and the building was there when you got back to work?! | 22:22 |
| hemna | :) | 22:22 |
| angela-s | yeah, but there was lots of stuff on the fans, if you know what i mean | 22:23 |
| nikeshm | smcginnis: uploaded fix https://review.openstack.org/#/c/215330/ for CI failure for dothill driver for volume conversion | 22:23 |
| *** chlong has quit IRC | 22:25 | |
| *** zul has joined #openstack-cinder | 22:26 | |
| *** lcurtis has quit IRC | 22:28 | |
| *** bapalm has joined #openstack-cinder | 22:29 | |
| *** annegentle has quit IRC | 22:33 | |
| *** annegentle has joined #openstack-cinder | 22:33 | |
| *** bapalm has quit IRC | 22:34 | |
| *** mriedem is now known as mriedem_away | 22:35 | |
| *** bapalm has joined #openstack-cinder | 22:35 | |
| *** annegentle has quit IRC | 22:38 | |
| *** xyang1 has quit IRC | 22:38 | |
| *** JoseMello has quit IRC | 22:39 | |
| *** bapalm has quit IRC | 22:43 | |
| *** bapalm has joined #openstack-cinder | 22:43 | |
| *** bapalm has quit IRC | 22:48 | |
| *** dannywilson has quit IRC | 22:48 | |
| *** gouthamr has joined #openstack-cinder | 22:49 | |
| *** bapalm has joined #openstack-cinder | 22:51 | |
| *** p0rtal has joined #openstack-cinder | 22:54 | |
| *** dannywilson has joined #openstack-cinder | 22:57 | |
| *** tsbot_ has quit IRC | 23:01 | |
| *** leeantho has joined #openstack-cinder | 23:03 | |
| *** angela-s has quit IRC | 23:04 | |
| *** alejandrito has quit IRC | 23:06 | |
| *** tiny-hands has joined #openstack-cinder | 23:07 | |
| *** gouthamr has quit IRC | 23:10 | |
| *** gouthamr has joined #openstack-cinder | 23:10 | |
| *** diogogmt has quit IRC | 23:12 | |
| *** diogogmt has joined #openstack-cinder | 23:14 | |
| *** hemna is now known as hemnafk | 23:16 | |
| *** gouthamr has quit IRC | 23:18 | |
| *** dannywilson has quit IRC | 23:19 | |
| *** r-daneel has quit IRC | 23:27 | |
| *** dannywilson has joined #openstack-cinder | 23:30 | |
| *** p0rtal_ has joined #openstack-cinder | 23:34 | |
| jgriffith | dannywilson: ok, here's my plan | 23:37 |
| *** p0rtal has quit IRC | 23:37 | |
| dannywilson | jgriffith: all ears, or eyes I guess | 23:38 |
| jgriffith | dannywilson: instead of trying to be clever and concisely combing managed and unmanaged config options under the same variable.. | 23:38 |
| jgriffith | dannywilson: I have two config options: | 23:38 |
| *** p0rtal_ has quit IRC | 23:38 | |
| jgriffith | 1. managed_replication_devices = <backend-a>,<backend-b>..... | 23:38 |
| jgriffith | 2. unmanaged_replication = <san_ip=1.1.1.1:san_login=admin:san_password=admin....>,<xxxxx> | 23:39 |
| *** gouthamr has joined #openstack-cinder | 23:39 | |
| jgriffith | dannywilson: so there's plenty of flexibility for everyone | 23:40 |
| jgriffith | dannywilson: and you can provide multiple devices | 23:40 |
| dannywilson | jgriffith: sounds good to me | 23:40 |
| jgriffith | dannywilson: I then do a check when parsing config, and throw an error for invalid config if you configured both | 23:40 |
| jgriffith | dannywilson: we can tackle dealing with both in M :) | 23:40 |
| dannywilson | jgriffith: agreed | 23:41 |
| jgriffith | dannywilson: I'll have a patch in a bit with those updates, as well as updates in the docs | 23:41 |
| dannywilson | jgriffith: so base driver will make those lists available to derived drivers? | 23:41 |
| dannywilson | jgriffith: excellent | 23:41 |
| jgriffith | dannywilson: yes, I'm putting these in base-driver as we type :) | 23:42 |
| jgriffith | dannywilson: the help is a bit wordy, but I'd rather that than the opposite I guess :) | 23:42 |
| dannywilson | jgriffith: indeed | 23:42 |
| jgriffith | dannywilson: I've been playing with different ways to delimit this.. maybe you have a preference? | 23:43 |
| dannywilson | jgriffith: thanks very much, will be updating our implementation to match tomorrow morning probably | 23:43 |
| dannywilson | jgriffith: not really, as long as it works ;) | 23:43 |
| jgriffith | dannywilson: {san_ip=1.1.1.1:san_login=foomaster:san_password=4shnizzle},{san_ip=2.2.2.2:san_login=foomaster:san_password=4shnizzle} | 23:43 |
| jgriffith | or | 23:44 |
| jgriffith | {san_ip=1.1.1.1 san_login=foomaster san_password=4shnizzle},{san_ip=2.2.2.2 san_login=foomaster san_password=4shnizzle} | 23:44 |
| jgriffith | ^^ spaces instead of ':' | 23:44 |
| jgriffith | Or s/=/:/ with spaces as delimeters | 23:45 |
| dannywilson | jgriffith: spaces are a little easier on my eyes in IRC anyway | 23:45 |
| jgriffith | Or any of the above but drop the {} enclosures | 23:45 |
| jgriffith | spaces it is then | 23:45 |
| jgriffith | :) | 23:45 |
| jgriffith | I'm not sure if I want to include the {}, they're just extra "things" to parse | 23:46 |
| dannywilson | jgriffith: okay, and actually for our driver we don't need login/password but we use token based auth | 23:46 |
| jgriffith | though it might make it more readable though | 23:46 |
| dannywilson | jgriffith: yes, helps to separate them | 23:46 |
| jgriffith | dannywilson: yeah, so the good thing is those keys=values can be whatever you want | 23:46 |
| jgriffith | completely up to you | 23:46 |
| *** salv-orlando has quit IRC | 23:46 | |
| dannywilson | okay, cool | 23:47 |
| jgriffith | alrigthy...lemme stop screwing around on this somewhat meaningless part and move on then :) | 23:47 |
| jgriffith | thanks for the input | 23:47 |
| dannywilson | np, thank you too | 23:47 |
| *** gouthamr has quit IRC | 23:51 | |
| *** dannywilson has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!