| *** smoriya has joined #openstack-cinder | 00:05 | |
| *** dims_ has joined #openstack-cinder | 00:06 | |
| *** dims has quit IRC | 00:07 | |
| *** 64MADOR5X has quit IRC | 00:09 | |
| openstackgerrit | Cory Stone proposed openstack/cinder: Dont eager load volume type specs on volume list https://review.openstack.org/216908 | 00:10 |
|---|---|---|
| *** haomaiwang has joined #openstack-cinder | 00:10 | |
| *** simondodsley has joined #openstack-cinder | 00:11 | |
| guitarzan | jgriffith: hey | 00:12 |
| *** dims has joined #openstack-cinder | 00:12 | |
| guitarzan | jgriffith: I just did some weird extra specs patch... | 00:12 |
| *** IanGovett has quit IRC | 00:13 | |
| guitarzan | oh yours is different :) | 00:13 |
| *** asd112z has joined #openstack-cinder | 00:14 | |
| *** tsekiyam_ has joined #openstack-cinder | 00:15 | |
| *** dims_ has quit IRC | 00:15 | |
| *** garthb has quit IRC | 00:17 | |
| *** mtanino has quit IRC | 00:18 | |
| *** dflorea has quit IRC | 00:18 | |
| *** tsekiyama has quit IRC | 00:19 | |
| *** tsekiyam_ has quit IRC | 00:20 | |
| *** diogogmt has joined #openstack-cinder | 00:23 | |
| *** takedakn has left #openstack-cinder | 00:23 | |
| *** salv-orlando has quit IRC | 00:30 | |
| *** openstack has joined #openstack-cinder | 00:34 | |
| -cameron.freenode.net- [freenode-info] channel trolls and no channel staff around to help? please check with freenode support: http://freenode.net/faq.shtml#gettinghelp | 00:34 | |
| *** _cjones_ has quit IRC | 00:34 | |
| *** dflorea has joined #openstack-cinder | 00:36 | |
| guitarzan | extra specs...what a hassle | 00:37 |
| openstackgerrit | wanghao proposed openstack/cinder: Add pagination to backups https://review.openstack.org/204493 | 00:38 |
| *** diogogmt has quit IRC | 00:39 | |
| *** diogogmt has joined #openstack-cinder | 00:40 | |
| *** dflorea has quit IRC | 00:41 | |
| *** changbl has quit IRC | 00:42 | |
| *** Zhang has joined #openstack-cinder | 00:42 | |
| *** jwcroppe has quit IRC | 00:47 | |
| *** jwcroppe has joined #openstack-cinder | 00:47 | |
| *** pckizer has quit IRC | 00:49 | |
| *** pckizer has joined #openstack-cinder | 00:49 | |
| *** jwcroppe has quit IRC | 00:52 | |
| *** dflorea has joined #openstack-cinder | 00:56 | |
| *** asselin__ has joined #openstack-cinder | 00:56 | |
| *** asselin has quit IRC | 00:58 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:04 | |
| *** lixiaoy1 has quit IRC | 01:05 | |
| *** haomaiwang has quit IRC | 01:09 | |
| *** haomaiwang has joined #openstack-cinder | 01:10 | |
| *** IlyaG has joined #openstack-cinder | 01:12 | |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 01:14 |
| *** skylerberg has quit IRC | 01:16 | |
| *** zhenguo has joined #openstack-cinder | 01:17 | |
| *** jwcroppe has joined #openstack-cinder | 01:19 | |
| *** theanalyst has quit IRC | 01:19 | |
| openstackgerrit | xing-yang proposed openstack/cinder: ScaleIO driver should use os-brick connector https://review.openstack.org/214420 | 01:20 |
| *** lixiaoy1 has joined #openstack-cinder | 01:21 | |
| *** theanalyst has joined #openstack-cinder | 01:22 | |
| *** humble__ has joined #openstack-cinder | 01:23 | |
| *** asselin has joined #openstack-cinder | 01:24 | |
| *** dflorea has quit IRC | 01:24 | |
| *** bnemec has quit IRC | 01:26 | |
| *** asselin__ has quit IRC | 01:27 | |
| *** lixiaoy11 has joined #openstack-cinder | 01:28 | |
| *** lixiaoy1 has quit IRC | 01:30 | |
| openstackgerrit | xing-yang proposed openstack/cinder: ScaleIO driver should use os-brick connector https://review.openstack.org/214420 | 01:32 |
| *** zhongjun has joined #openstack-cinder | 01:33 | |
| *** Apoorva has quit IRC | 01:33 | |
| *** Lee1092 has joined #openstack-cinder | 01:33 | |
| *** edmondsw has joined #openstack-cinder | 01:39 | |
| *** edmondsw has quit IRC | 01:39 | |
| *** jwcroppe_ has joined #openstack-cinder | 01:40 | |
| *** jwcroppe has quit IRC | 01:42 | |
| openstackgerrit | xing-yang proposed openstack/cinder: ScaleIO driver should use os-brick connector https://review.openstack.org/214420 | 01:45 |
| *** davechen has joined #openstack-cinder | 01:51 | |
| *** gouthamr has joined #openstack-cinder | 01:54 | |
| *** julim has quit IRC | 01:57 | |
| Zhang | zhenguo Hi | 02:04 |
| *** haomaiwang has quit IRC | 02:09 | |
| *** haomaiwang has joined #openstack-cinder | 02:10 | |
| *** skylerberg has joined #openstack-cinder | 02:14 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add pagination to backups https://review.openstack.org/204493 | 02:18 |
| *** rushil has joined #openstack-cinder | 02:19 | |
| *** boris-42 has quit IRC | 02:20 | |
| *** IlyaG has quit IRC | 02:22 | |
| *** rushil has quit IRC | 02:23 | |
| *** rushil has joined #openstack-cinder | 02:24 | |
| Zhang | I have some problem when Integrated 18000 storage. | 02:24 |
| *** zhongjun has quit IRC | 02:26 | |
| *** chenying has joined #openstack-cinder | 02:32 | |
| *** annegent_ has quit IRC | 02:32 | |
| zhenguo | Zhang: you mean huawei driver? | 02:38 |
| Zhang | Yes | 02:39 |
| *** gouthamr_ has joined #openstack-cinder | 02:41 | |
| *** mtanino has joined #openstack-cinder | 02:41 | |
| *** annegentle has joined #openstack-cinder | 02:42 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: WIP: Update the devref for volume migration https://review.openstack.org/214941 | 02:44 |
| *** gouthamr has quit IRC | 02:44 | |
| zhenguo | liuxg: around? | 02:46 |
| *** dims has quit IRC | 02:46 | |
| *** gouthamr_ has quit IRC | 02:51 | |
| *** hakimo has joined #openstack-cinder | 02:52 | |
| *** hakimo_ has quit IRC | 02:54 | |
| *** haomaiwang has quit IRC | 03:09 | |
| *** haomaiwang has joined #openstack-cinder | 03:10 | |
| openstackgerrit | xing-yang proposed openstack/cinder: ScaleIO driver should use os-brick connector https://review.openstack.org/214420 | 03:15 |
| *** skylerberg has quit IRC | 03:15 | |
| *** BharatK has quit IRC | 03:19 | |
| openstackgerrit | JuPing proposed openstack/cinder: Fix description for "Barbarism of editting a file" https://review.openstack.org/216942 | 03:27 |
| *** garthb has joined #openstack-cinder | 03:31 | |
| *** jwcroppe_ is now known as jwcroppe | 03:35 | |
| *** baojg has joined #openstack-cinder | 03:39 | |
| *** haomaiwang has quit IRC | 03:40 | |
| *** hemna has joined #openstack-cinder | 03:41 | |
| *** hemna has quit IRC | 03:42 | |
| *** links has joined #openstack-cinder | 03:45 | |
| *** boris-42 has joined #openstack-cinder | 03:45 | |
| *** changbl has joined #openstack-cinder | 03:46 | |
| *** baojg has quit IRC | 03:52 | |
| *** rushil has quit IRC | 03:53 | |
| *** baojg has joined #openstack-cinder | 03:53 | |
| *** p0rtal has quit IRC | 03:54 | |
| *** dflorea has joined #openstack-cinder | 04:03 | |
| openstackgerrit | wanghao proposed openstack/python-cinderclient: Add to_dict method to Resource class https://review.openstack.org/207340 | 04:05 |
| *** haomaiwang has joined #openstack-cinder | 04:05 | |
| *** annegentle has quit IRC | 04:07 | |
| *** haomaiwang has quit IRC | 04:09 | |
| *** haomaiwang has joined #openstack-cinder | 04:10 | |
| *** hemna has joined #openstack-cinder | 04:13 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 04:15 |
| *** skylerberg has joined #openstack-cinder | 04:26 | |
| *** chenying has quit IRC | 04:27 | |
| *** humble__ has quit IRC | 04:28 | |
| *** sgotliv has joined #openstack-cinder | 04:35 | |
| *** deepakcs has joined #openstack-cinder | 04:35 | |
| openstackgerrit | Takeaki Matsumoto proposed openstack/cinder: Fix url in API response to get original https://review.openstack.org/216591 | 04:39 |
| *** timcl has quit IRC | 05:04 | |
| *** lpetrut has joined #openstack-cinder | 05:06 | |
| *** rbradbury has quit IRC | 05:07 | |
| *** haomaiwang has quit IRC | 05:09 | |
| *** haomaiwang has joined #openstack-cinder | 05:10 | |
| lixiaoy11 | winston-d: here? | 05:12 |
| *** hemna has quit IRC | 05:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/215440 | 05:14 |
| *** salv-orlando has joined #openstack-cinder | 05:16 | |
| *** zhongjun has joined #openstack-cinder | 05:16 | |
| openstackgerrit | Takeaki Matsumoto proposed openstack/cinder: Fix url in API response to get original https://review.openstack.org/216591 | 05:19 |
| *** salv-orlando has quit IRC | 05:21 | |
| *** sgotliv has quit IRC | 05:21 | |
| *** RaySun has joined #openstack-cinder | 05:21 | |
| *** asd112z has quit IRC | 05:35 | |
| *** p0rtal has joined #openstack-cinder | 05:36 | |
| openstackgerrit | Takeaki Matsumoto proposed openstack/cinder: Fix url in API response to get original https://review.openstack.org/216591 | 05:36 |
| *** dims has joined #openstack-cinder | 05:36 | |
| *** yrabl has quit IRC | 05:39 | |
| *** shausy has joined #openstack-cinder | 05:41 | |
| *** dims has quit IRC | 05:42 | |
| *** lpetrut has quit IRC | 05:43 | |
| *** shyama has joined #openstack-cinder | 05:46 | |
| openstackgerrit | Takeaki Matsumoto proposed openstack/cinder: Fix url in API response to get original https://review.openstack.org/216591 | 05:46 |
| *** zhongjun has quit IRC | 05:50 | |
| *** garthb has quit IRC | 05:52 | |
| *** shausy has quit IRC | 05:53 | |
| *** shausy has joined #openstack-cinder | 05:54 | |
| openstackgerrit | Takeaki Matsumoto proposed openstack/cinder: Fix url in API response to get original https://review.openstack.org/216591 | 06:02 |
| *** afazekas has joined #openstack-cinder | 06:08 | |
| *** haomaiwang has quit IRC | 06:09 | |
| *** lpetrut has joined #openstack-cinder | 06:09 | |
| *** haomaiwa_ has joined #openstack-cinder | 06:10 | |
| *** nkrinner has joined #openstack-cinder | 06:15 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 06:17 |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: Add class for dog command executor https://review.openstack.org/194532 | 06:17 |
| *** RaySun has quit IRC | 06:20 | |
| *** chenying has joined #openstack-cinder | 06:24 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add hypermetro support for Huawei driver https://review.openstack.org/202023 | 06:27 |
| *** lpetrut has quit IRC | 06:27 | |
| *** p0rtal has quit IRC | 06:27 | |
| *** p0rtal has joined #openstack-cinder | 06:28 | |
| *** skylerberg has quit IRC | 06:28 | |
| wilson2 | Zhang: what's the problem with 18000 storage? | 06:29 |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 06:30 |
| Zhang | create volume size 1Gb, but lookup the size 5Gb when attach volume. | 06:31 |
| *** p0rtal has quit IRC | 06:32 | |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 06:33 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/214468 | 06:34 |
| *** agarciam has joined #openstack-cinder | 06:37 | |
| *** salv-orlando has joined #openstack-cinder | 06:38 | |
| *** Zhang has quit IRC | 06:40 | |
| *** bluex-pl has joined #openstack-cinder | 06:43 | |
| *** bluex-pl has quit IRC | 06:43 | |
| *** vincent_hou has joined #openstack-cinder | 06:46 | |
| geguileo | jgriffith: I am now | 06:46 |
| vincent_hou | geguileo: Morning. | 06:46 |
| geguileo | vincent_hou: Morning | 06:46 |
| vincent_hou | geguileo: Thank you for your review. | 06:47 |
| *** ociuhandu has quit IRC | 06:47 | |
| geguileo | vincent_hou: No problem | 06:47 |
| vincent_hou | geguileo: I still have some minor questions for you. Is it OK? | 06:48 |
| geguileo | vincent_hou: Sure | 06:48 |
| geguileo | vincent_hou: Did you write them in the comments or are you going to ask them here? | 06:48 |
| vincent_hou | geguileo: I am putting some of them there, but there are something else here. | 06:49 |
| geguileo | vincent_hou: Ok | 06:49 |
| vincent_hou | geguileo: why is if a in ('a', 'b') better than if a in ['a', 'b']? | 06:49 |
| geguileo | vincent_hou: [] is a list, which is a "costly" operation | 06:50 |
| geguileo | vincent_hou: () is a immutable tuple, which is cheaper | 06:50 |
| geguileo | vincent_hou: And a dictionary is more expensive as well | 06:50 |
| geguileo | vincent_hou: Because it has to build the hash table | 06:50 |
| vincent_hou | geguileo: [] takes more space and time. | 06:50 |
| geguileo | vincent_hou: If you are not going to be modifying the items you should use a tuple | 06:51 |
| geguileo | vincent_hou: Exactly | 06:51 |
| geguileo | vincent_hou: And you can modify it by "mistake" | 06:51 |
| *** ronis has joined #openstack-cinder | 06:51 | |
| geguileo | vincent_hou: So [] should only be used when you are going to be adding/removing items | 06:51 |
| *** e0ne has joined #openstack-cinder | 06:52 | |
| vincent_hou | geguileo: Wow, you did awesome in python. Thank you for letting me know. | 06:53 |
| geguileo | vincent_hou: No problem | 06:53 |
| vincent_hou | geguileo: Then you said webob.exc.HTTPServiceUnavailable is not correct, what could be the right one for the exception raised for 'maintenance' volumes? | 06:53 |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: Cinder Nested Quota Driver https://review.openstack.org/205369 | 06:53 |
| *** ronis_ has joined #openstack-cinder | 06:53 | |
| geguileo | vincent_hou: I would send 409 Conflict | 06:56 |
| *** ronis has quit IRC | 06:57 | |
| vincent_hou | geguileo: HTTPConflict? | 06:58 |
| geguileo | vincent_hou: That's what I would return | 06:58 |
| vincent_hou | geguileo: I will put them in another patch . | 06:58 |
| geguileo | vincent_hou: According to the RFC: The 409 (Conflict) status code indicates that the request could not | 06:59 |
| geguileo | be completed due to a conflict with the current state of the target | 06:59 |
| geguileo | resource. This code is used in situations where the user might be | 06:59 |
| geguileo | able to resolve the conflict and resubmit the request. | 06:59 |
| *** ociuhandu has joined #openstack-cinder | 06:59 | |
| vincent_hou | Hmm, it sounds more reasonable. | 06:59 |
| geguileo | vincent_hou: But that error only for the maintenance status error | 06:59 |
| geguileo | vincent_hou: Not for all the errors where the input is not valid | 07:00 |
| vincent_hou | I see. | 07:00 |
| vincent_hou | geguileo: Thanks. | 07:00 |
| geguileo | vincent_hou: No problem | 07:01 |
| *** ankit_ag has joined #openstack-cinder | 07:01 | |
| *** yrabl has joined #openstack-cinder | 07:03 | |
| *** alexschm has joined #openstack-cinder | 07:04 | |
| openstackgerrit | XinXiaohui proposed openstack/cinder: Calculate virtual free capacity and notify https://review.openstack.org/206923 | 07:07 |
| *** dflorea has quit IRC | 07:07 | |
| *** baojg has quit IRC | 07:08 | |
| *** haomaiwa_ has quit IRC | 07:09 | |
| *** Zhang has joined #openstack-cinder | 07:09 | |
| *** haomaiwang has joined #openstack-cinder | 07:10 | |
| *** baojg has joined #openstack-cinder | 07:10 | |
| *** yrabl has quit IRC | 07:15 | |
| *** yrabl has joined #openstack-cinder | 07:15 | |
| *** Zhang has quit IRC | 07:16 | |
| geguileo | e0ne: ping | 07:16 |
| e0ne | geguileo: hi | 07:19 |
| geguileo | e0ne: Hi, thanks for your review on the snapshot pagination | 07:19 |
| geguileo | e0ne: Any reason why you didn't +1 the workflow? | 07:20 |
| e0ne | geguileo: you're welcome! thanks for addressing my comments | 07:20 |
| e0ne | geguileo: only because it was too late (about midnight) | 07:20 |
| e0ne | geguileo: i didn't want to miss anything:) | 07:20 |
| geguileo | e0ne: Thank you for the comment, I actually missed that method :-) | 07:20 |
| geguileo | e0ne: That's usually a good idea at those hours XD | 07:21 |
| e0ne | geguileo: yep, i did some mistakes in the past, so i try to put only +2 late | 07:21 |
| e0ne | geguileo: i'll be at the office in ~2 hours and +A your patch if it is still needed | 07:22 |
| geguileo | e0ne: Smart decission, I will take note XD | 07:22 |
| e0ne | :) | 07:22 |
| geguileo | e0ne: Awesome! | 07:22 |
| geguileo | e0ne: Is it in your review list the patch that fixes the backup imports? | 07:22 |
| e0ne | geguileo: it's on a top of my list | 07:23 |
| geguileo | e0ne: Awesome!! | 07:23 |
| geguileo | e0ne: Thanks | 07:23 |
| e0ne | geguileo: yesterday was my first day after vacation, i've got very long review and todos lists | 07:24 |
| *** Zhang has joined #openstack-cinder | 07:24 | |
| geguileo | e0ne: I hope you enjoyed your vacation, and good luck with your backlog | 07:24 |
| *** dims has joined #openstack-cinder | 07:24 | |
| *** dims has quit IRC | 07:29 | |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: Add unit test for backup get_all https://review.openstack.org/217009 | 07:30 |
| *** rushiagr_away is now known as rushiagr | 07:31 | |
| *** e0ne has quit IRC | 07:45 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 07:46 |
| *** ronis has joined #openstack-cinder | 07:52 | |
| *** ronis_ has quit IRC | 07:54 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 07:55 |
| openstackgerrit | LisaLi proposed openstack/cinder: Raise HTTPBadRequest once invalid filters are given https://review.openstack.org/217015 | 07:55 |
| openstackgerrit | LisaLi proposed openstack/cinder: Raise HTTPBadRequest when invalid filters are given https://review.openstack.org/217015 | 07:56 |
| openstackgerrit | LisaLi proposed openstack/cinder: Raise HTTPBadRequest when invalid filters are given https://review.openstack.org/217015 | 07:56 |
| *** IanGovett has joined #openstack-cinder | 07:56 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 07:58 |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: Sheepdog: improve create and delete operation https://review.openstack.org/197393 | 08:00 |
| *** ndipanov has quit IRC | 08:05 | |
| *** ndipanov has joined #openstack-cinder | 08:06 | |
| *** ndipanov has quit IRC | 08:06 | |
| *** chenying has quit IRC | 08:09 | |
| *** haomaiwang has quit IRC | 08:09 | |
| *** haomaiwang has joined #openstack-cinder | 08:10 | |
| *** jwcroppe has quit IRC | 08:16 | |
| *** geguileo has quit IRC | 08:16 | |
| *** geguileo has joined #openstack-cinder | 08:17 | |
| *** chenying has joined #openstack-cinder | 08:20 | |
| *** bluex-pl has joined #openstack-cinder | 08:24 | |
| *** bluex-pl has quit IRC | 08:24 | |
| *** bluex-pl has joined #openstack-cinder | 08:24 | |
| *** anshul has joined #openstack-cinder | 08:25 | |
| *** stefan_amann has joined #openstack-cinder | 08:26 | |
| *** yuriy_n17 has joined #openstack-cinder | 08:28 | |
| yuriy_n17 | geguileo: Please review https://review.openstack.org/#/c/216661/ | 08:30 |
| stefan_amann | thingee: Mike, may I ask you for your help with bug #1488233? System z / s390 will not be able to attach volumes when the FC LUN ID is >255. I proposed a fix for os-brick for Liberty. We need to back port to Kilo (Cinder, Nova, and os-brick since os-brick is not integrated in Kilo). But there is no option to nominate it for Kilo. At least not for Cinder, and os-brick. Any help would be appreciated! | 08:31 |
| openstack | bug 1488233 in os-brick "FC with LUN ID >255 not recognized" [Undecided,In progress] https://launchpad.net/bugs/1488233 - Assigned to Stefan Amann (stefan-amann) | 08:31 |
| openstackgerrit | Huang Zhiteng proposed openstack/os-brick: iSCSI Connector: force disconnect iscsi session https://review.openstack.org/217032 | 08:32 |
| *** jistr has joined #openstack-cinder | 08:39 | |
| geguileo | yuriy_n17: Done | 08:41 |
| *** Zhang has quit IRC | 08:43 | |
| *** jordanP has joined #openstack-cinder | 08:49 | |
| *** lixiaoy11 has quit IRC | 08:56 | |
| yuriy_n17 | geguileo: Thanks a lot. | 08:57 |
| yuriy_n17 | geguileo: What about to please review https://review.openstack.org/#/c/215666/ Thanks for advance. | 09:00 |
| *** enterprisedc has joined #openstack-cinder | 09:02 | |
| yuriy_n17 | jgriffith: Hi! Please review https://review.openstack.org/#/c/215666/ | 09:06 |
| openstackgerrit | Huang Zhiteng proposed openstack/cinder: Update volume status AFTER terminate_connection is done https://review.openstack.org/199015 | 09:06 |
| *** e0ne has joined #openstack-cinder | 09:08 | |
| *** haomaiwang has quit IRC | 09:09 | |
| openstackgerrit | lifeless proposed openstack/cinder: Remove unused dependency discover. https://review.openstack.org/217046 | 09:09 |
| *** enterprisedc has quit IRC | 09:09 | |
| *** haomaiwang has joined #openstack-cinder | 09:10 | |
| *** Zhang has joined #openstack-cinder | 09:12 | |
| *** h00327910__ has joined #openstack-cinder | 09:14 | |
| *** Zhang has quit IRC | 09:14 | |
| openstackgerrit | Jordan Pittier proposed openstack/cinder: Rework Scality SOFS driver to use RemoteFS class https://review.openstack.org/200610 | 09:14 |
| *** agarciam has quit IRC | 09:16 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add hypermetro support for Huawei driver https://review.openstack.org/202023 | 09:16 |
| *** agarciam has joined #openstack-cinder | 09:16 | |
| *** vincent_hou has quit IRC | 09:17 | |
| *** e0ne has quit IRC | 09:18 | |
| *** agarciam has quit IRC | 09:19 | |
| *** Zhang has joined #openstack-cinder | 09:19 | |
| *** Zhang has quit IRC | 09:22 | |
| *** e0ne has joined #openstack-cinder | 09:23 | |
| *** wilson1 has joined #openstack-cinder | 09:24 | |
| *** agarciam has joined #openstack-cinder | 09:24 | |
| *** daemontool_ has joined #openstack-cinder | 09:25 | |
| *** ndipanov has joined #openstack-cinder | 09:26 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 09:26 |
| *** gnarld_ has joined #openstack-cinder | 09:27 | |
| *** jgriffit1 has joined #openstack-cinder | 09:28 | |
| *** nug has quit IRC | 09:28 | |
| *** wilson2 has quit IRC | 09:28 | |
| *** geguileo has quit IRC | 09:28 | |
| *** rushiagr has quit IRC | 09:28 | |
| *** jgriffith has quit IRC | 09:28 | |
| *** gnarld_ is now known as nug | 09:28 | |
| *** daemontool__ has quit IRC | 09:28 | |
| *** geguileo has joined #openstack-cinder | 09:29 | |
| *** zhongjun has joined #openstack-cinder | 09:33 | |
| *** goodygum has joined #openstack-cinder | 09:38 | |
| openstackgerrit | zhaohua proposed openstack/cinder: Huawei: fix multi REST-URLs bug https://review.openstack.org/214500 | 09:42 |
| *** rushiagr has joined #openstack-cinder | 09:47 | |
| *** dims has joined #openstack-cinder | 09:48 | |
| *** davechen has left #openstack-cinder | 09:50 | |
| e0ne | geguileo: are you around? | 09:52 |
| geguileo | e0ne: Yep | 09:52 |
| e0ne | geguileo: few questions about import backup patches | 09:53 |
| geguileo | e0ne: I hope I still remember what it does ;-) | 09:53 |
| e0ne | :) | 09:53 |
| e0ne | it fixe a bug, i hope | 09:53 |
| geguileo | e0ne: That was the idea, yes XD | 09:54 |
| e0ne | geguileo: it wil never be merged until depended on patch get -1 from jenkins | 09:55 |
| geguileo | e0ne: We have a cyclic dependency | 09:55 |
| geguileo | e0ne: Tests in tempest are "wrong", since they assume current behavior is desired one | 09:56 |
| e0ne | geguileo: oh... | 09:56 |
| geguileo | e0ne: And I have a tempest patch to fix that | 09:56 |
| geguileo | e0ne: But with current code it doesn't pass either | 09:56 |
| e0ne | geguileo: why we've got a cyclic deps there> | 09:56 |
| geguileo | e0ne: Because Cinder patch needs tempest patch and viceversa | 09:56 |
| geguileo | e0ne: Cyclic dependency and Zuul doesn't like that | 09:57 |
| e0ne | geguileo: can we try to add dependency inly for cinder patch? | 09:57 |
| e0ne | geguileo: afair, tests for import backup were disabled | 09:57 |
| geguileo | e0ne: Not anymore | 09:58 |
| geguileo | e0ne: Cinder patch has dependency in the commit message | 09:58 |
| e0ne | geguileo: we need to ask infra guys how to fix it | 09:59 |
| e0ne | geguileo: i can't +2 w/o jenkins vote:( | 10:00 |
| geguileo | e0ne: I'll get Jenkins to give it -1 (with the error I mention) | 10:01 |
| *** agarciam has quit IRC | 10:01 | |
| e0ne | geguileo: but we can't merge with -1 from jenkins | 10:01 |
| *** agarciam has joined #openstack-cinder | 10:01 | |
| geguileo | e0ne: Yeah, we need to disable that test first in tempest | 10:02 |
| geguileo | e0ne: Then merge this code in Cinder | 10:02 |
| geguileo | e0ne: And then update the test and re-enable it | 10:02 |
| e0ne | geguileo: looks like a good plan to get it fixed and merged | 10:02 |
| geguileo | e0ne: Ok, I'll look into disabling that test | 10:02 |
| e0ne | geguileo: thanks! i'll be happy to +2 once jenkins passed | 10:03 |
| geguileo | e0ne: Ok, I'll ping you again when I have it sorted out and passing Jenkins | 10:03 |
| e0ne | geguileo: thanks! | 10:03 |
| *** baojg has quit IRC | 10:06 | |
| *** ociuhandu_ has joined #openstack-cinder | 10:06 | |
| *** ociuhandu has quit IRC | 10:08 | |
| *** ociuhandu_ is now known as ociuhandu | 10:08 | |
| *** haomaiwang has quit IRC | 10:09 | |
| *** shausy has quit IRC | 10:10 | |
| *** haomaiwang has joined #openstack-cinder | 10:10 | |
| *** Zhang has joined #openstack-cinder | 10:16 | |
| *** zhongjun has quit IRC | 10:24 | |
| *** vgridnev has joined #openstack-cinder | 10:26 | |
| vgridnev | is there someone who can review that change: https://review.openstack.org/#/c/214194/ ? | 10:28 |
| yuriy_n17 | smcginnis: Hi! Please review https://review.openstack.org/#/c/216661/ | 10:29 |
| *** chenying has quit IRC | 10:34 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: WIP:Add rootwrap daemon mode support https://review.openstack.org/149677 | 10:37 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: WIP: Switch to use execute from cinder utils https://review.openstack.org/214595 | 10:37 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add rootwrap daemon mode support https://review.openstack.org/149677 | 10:40 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Switch to use execute from cinder utils https://review.openstack.org/214595 | 10:40 |
| *** lpetrut has joined #openstack-cinder | 10:41 | |
| *** wilson2 has joined #openstack-cinder | 10:48 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add rootwrap daemon mode support https://review.openstack.org/149677 | 10:51 |
| *** wilson1 has quit IRC | 10:52 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix backup metadata import missing fields https://review.openstack.org/183222 | 10:55 |
| aarefiev | DuncanT: hi, are u around ? | 10:58 |
| nikeshm | hi | 10:59 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: Replace urllib.unquote with urllib.parse.unquote https://review.openstack.org/217089 | 10:59 |
| dulek | Guys, I'm working on a bugfix and made changes in db.sqlalchemy.api. Where should I put unit tests? | 11:00 |
| dulek | I'm changing snapshot_get_all methods, so test_volume.py doesn't seem like a proper place - it's testing volume manager mostly. | 11:01 |
| dulek | And tests.unit.db also doesn't hold such tests... | 11:02 |
| aarefiev | dulek: tests.unit.test_db_api ? | 11:02 |
| dulek | aarefiev: Ah, that should be it. Thanks! | 11:03 |
| aarefiev | dulek: np | 11:05 |
| *** haomaiwang has quit IRC | 11:09 | |
| *** agarciam1 has joined #openstack-cinder | 11:10 | |
| *** agarciam has quit IRC | 11:10 | |
| *** EinstCrazy has joined #openstack-cinder | 11:21 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Efficient volume copy for generic volume migration https://review.openstack.org/183701 | 11:36 |
| mtanino | Do we still have pthon27 error for VMwareVcVmdkDriverTestCase on the CI? | 11:42 |
| *** Zhang has quit IRC | 11:42 | |
| *** EinstCrazy has quit IRC | 11:43 | |
| mtanino | geguileo: Hi, I added Get Volume Driver Capabilities patch (mtanino) on meeting topics. | 11:43 |
| geguileo | mtanino: I saw it :-) | 11:44 |
| mtanino | If my understanding is different from your proposal, please let me know. | 11:44 |
| mtanino | :) | 11:44 |
| *** dims_ has joined #openstack-cinder | 11:48 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Fix way of get LUN id in Huawei driver https://review.openstack.org/209749 | 11:50 |
| *** bswartz has quit IRC | 11:51 | |
| *** mewald has joined #openstack-cinder | 11:52 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Encode username and password in Huawei driver https://review.openstack.org/216085 | 11:52 |
| *** dims has quit IRC | 11:52 | |
| *** h00327910__ has quit IRC | 11:53 | |
| DuncanT | aarefiev: Hi. I'm around right now | 11:56 |
| *** agarciam1 has quit IRC | 11:56 | |
| aarefiev | DuncanT: hi, just want to know your thoughts about rootwrap daemon mode testing | 11:58 |
| *** agarciam has joined #openstack-cinder | 11:59 | |
| aarefiev | DuncanT: and specifically about the sheepdog backup | 11:59 |
| *** marcusvrn_ has joined #openstack-cinder | 12:01 | |
| aarefiev | DuncanT: it should be separate job or what? | 12:02 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Fix way of get LUN id in Huawei driver https://review.openstack.org/209749 | 12:04 |
| *** mewald has left #openstack-cinder | 12:07 | |
| DuncanT | aarefiev: A separate job, at least initially, will help narrow down the problems faster | 12:08 |
| DuncanT | aarefiev: At some point, it can probably be rolled into one, since non-daemon mode is tested by lots of jobs | 12:08 |
| *** Zhang has joined #openstack-cinder | 12:08 | |
| *** mewald_ has joined #openstack-cinder | 12:09 | |
| *** EinstCrazy has joined #openstack-cinder | 12:10 | |
| *** simondodsley has quit IRC | 12:10 | |
| aarefiev | DuncanT : yep, we are worrying about the number of jobs) | 12:10 |
| aarefiev | they are a lot | 12:10 |
| mewald_ | I need help understanding how cinder-backup works: Will cinder-backup connect to the volume backends and backup backends and take care of copying? In other words: Will data flow through the cinder-backend service? | 12:11 |
| *** raildo-afk is now known as raildo | 12:14 | |
| *** bswartz has joined #openstack-cinder | 12:14 | |
| dulek | mewald_: http://specs.openstack.org/openstack/cinder-specs/specs/liberty/non-disruptive-backup.html | 12:17 |
| dulek | mewald_: Here is extensive documentation of current state of how backups work and the state we want to achieve in Liberty. | 12:18 |
| dulek | mewald_: Ah, sorry, wrong link. | 12:18 |
| dulek | mewald_: http://specs.openstack.org/openstack/cinder-specs/specs/liberty/scalable-backup-service.html | 12:18 |
| dulek | That's the right one. | 12:18 |
| mewald_ | thx! | 12:18 |
| *** mewald_ has quit IRC | 12:19 | |
| *** nikeshm has quit IRC | 12:19 | |
| *** nikeshm has joined #openstack-cinder | 12:19 | |
| *** erlon has joined #openstack-cinder | 12:23 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Validate filters to snapshot_get_all* in db.api https://review.openstack.org/217110 | 12:27 |
| DuncanT | aarefiev: Infra will have opinions on that I guess. Combining them is ok, just makes debugging slightly harder | 12:27 |
| *** annegentle has joined #openstack-cinder | 12:27 | |
| *** adrianofr has joined #openstack-cinder | 12:28 | |
| aarefiev | DuncanT: what do you mean 'Combining them...' ? | 12:28 |
| *** smoriya has quit IRC | 12:28 | |
| *** aix has quit IRC | 12:29 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Validate filters in snapshot_get_all* in db.api https://review.openstack.org/217110 | 12:29 |
| *** timcl has joined #openstack-cinder | 12:32 | |
| DuncanT | aarefiev: Just test everything in one job | 12:35 |
| *** akerr has joined #openstack-cinder | 12:35 | |
| DuncanT | aarefiev: as long as daemon mode is tested in one job somewhere, and non-daemon mode somewhere else, and every driver gets tested, we still get reasonable coverage | 12:36 |
| DuncanT | aarefiev: A vendor with interest in a specific scenario is welcome to add more CI on their own hardware | 12:37 |
| aarefiev | DuncanT: I'm not quite understand about 'test everything in one job', how do you see it ? | 12:38 |
| aarefiev | what job will run | 12:39 |
| *** edmondsw has joined #openstack-cinder | 12:40 | |
| yuriy_n17 | smcginnis: Thanks a lot. | 12:43 |
| smcginnis | yuriy_n17: no | 12:43 |
| yuriy_n17 | smcginnis: I mean thanks for your reviewing. | 12:44 |
| smcginnis | yuriy_n17: Hah! I meant to type np. No problem reviewing. Looked good to me. :) | 12:44 |
| *** EinstCrazy has quit IRC | 12:46 | |
| *** EinstCrazy has joined #openstack-cinder | 12:49 | |
| *** annasort has joined #openstack-cinder | 12:50 | |
| DuncanT | aarefiev: Combine daemon mode into e.g. the sheepdog job | 12:50 |
| *** simondodsley has joined #openstack-cinder | 12:50 | |
| aarefiev | DuncanT: ahh , I see :) , sorry, you mean daemon mode and sheepdog | 12:51 |
| DuncanT | aarefiev: In general, we can probably test more config options by mixing up the jobs a bit | 12:52 |
| aarefiev | DuncanT: I think it's good idea, the number of jobs have been increasing very fast | 12:54 |
| aarefiev | DuncanT: thanks for your time | 12:55 |
| *** deepakcs has quit IRC | 12:56 | |
| *** aix has joined #openstack-cinder | 12:58 | |
| *** mtanino has quit IRC | 12:59 | |
| *** diablo_rojo has joined #openstack-cinder | 13:00 | |
| *** eharney has joined #openstack-cinder | 13:00 | |
| *** diablo_rojo has left #openstack-cinder | 13:00 | |
| *** jwcroppe has joined #openstack-cinder | 13:00 | |
| *** mriedem has joined #openstack-cinder | 13:03 | |
| *** takedakn has joined #openstack-cinder | 13:04 | |
| jordanP | smcginnis, hi :) | 13:04 |
| *** Zhang has quit IRC | 13:04 | |
| smcginnis | jordanP: Hey. Sorry I didn't catch that earlier. | 13:05 |
| *** merooney has joined #openstack-cinder | 13:05 | |
| jordanP | smcginnis, no problem. | 13:05 |
| *** ericksonsantos has quit IRC | 13:05 | |
| jordanP | here's the thing. You can see the logs of the job if you click on the "console" link in the left panel of the jenkins screen. All the logs are available if you click on the "build artifacats" link. I am using gerrit trigger for now. I am not sure how customizable that is | 13:07 |
| *** porrua has joined #openstack-cinder | 13:07 | |
| *** xyang1 has joined #openstack-cinder | 13:07 | |
| jordanP | smcginnis, I agree it's less convenient, you have to click on 2 links but it was like this since the start | 13:07 |
| smcginnis | jordanP: I don't use Jenkins myself, but most third party CIs do. You need to collect all of those artifacts and publish them on a log server. Not just make your jenkins server accessible. | 13:08 |
| smcginnis | jordanP: Which really doesn't seem like a good idea to me anyway. :) | 13:08 |
| jordanP | (https://jenkins07.openstack.org/ is public) | 13:08 |
| jordanP | smcginnis, I agree but then I have another log server to maintain | 13:09 |
| smcginnis | Welcome to third party CI> :) | 13:09 |
| eikke | smcginnis: that jenkins instance is meant to be public, consciously | 13:09 |
| smcginnis | eikke: That's fine. That's not really an issue. | 13:09 |
| smcginnis | Just commenting that I don't have enough confidence in my own admin skills that I would feel comfortable doing that. | 13:10 |
| smcginnis | But the issue is the logs are not collected as required. | 13:10 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add rootwrap daemon mode support https://review.openstack.org/149677 | 13:10 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Switch to use execute from cinder utils https://review.openstack.org/214595 | 13:10 |
| *** diablo_rojo has joined #openstack-cinder | 13:10 | |
| eikke | smcginnis: which logs that aren't on e.g. https://37.187.159.67:5443/job/cinder-sofs-validate/8269/artifact/jenkins-logs/ ? | 13:11 |
| *** martyturner has joined #openstack-cinder | 13:11 | |
| smcginnis | eikke: I'll look. But that's part of the point. I shouldn't have to go digging all through your stuff to find them. | 13:11 |
| *** agarciam has quit IRC | 13:12 | |
| *** martyturner has quit IRC | 13:13 | |
| *** agarciam has joined #openstack-cinder | 13:13 | |
| smcginnis | eikke: How are you addressing the requirement to not have to download the file to view it. These are all gzipped, but via Jenkins it doesn't allow me to view in browser. | 13:13 |
| *** martyturner has joined #openstack-cinder | 13:13 | |
| smcginnis | eikke: It does look like at least all of the files are there. You just need to get them to an accessible place that allows viewing. | 13:13 |
| *** dustins has joined #openstack-cinder | 13:14 | |
| eikke | smcginnis: ok, that's something we'll work on (with some nginx filter or so) | 13:14 |
| eikke | we'll also let subunit dump an xunit xml so the test results can be listed/visualized/... in the jenkins UI | 13:15 |
| *** dustins has quit IRC | 13:15 | |
| *** links has quit IRC | 13:15 | |
| *** martyturner has quit IRC | 13:18 | |
| *** dustins has joined #openstack-cinder | 13:18 | |
| *** merooney has quit IRC | 13:20 | |
| *** diablo_rojo has quit IRC | 13:20 | |
| eikke | smcginnis: fwiw, the whole scality openstack team literally dropped whatever we were doing to handle this asap ;-) | 13:21 |
| smcginnis | eikke: Awesome. Hopefully it won't be that big of a deal for you. Glad to see your giving it the attention though. Thanks! | 13:22 |
| *** gouthamr has joined #openstack-cinder | 13:22 | |
| *** julim has joined #openstack-cinder | 13:22 | |
| stefan_amann | thingee: may I ask you for your help with bug #1488233? System z / s390 will not be able to attach volumes when the FC LUN ID is >255. I proposed a fix for os-brick for Liberty. We need to back port to Kilo (Cinder, Nova, and os-brick since os-brick is not integrated in Kilo). But there is no option to nominate it for Kilo. At least not for Cinder, and os-brick. Any help would be appreciated! | 13:22 |
| openstack | bug 1488233 in os-brick "FC with LUN ID >255 not recognized" [Undecided,In progress] https://launchpad.net/bugs/1488233 - Assigned to Stefan Amann (stefan-amann) | 13:22 |
| smcginnis | stefan_amann: thingee is on vacation right now. | 13:23 |
| stefan_amann | ok.. thanks for letting me know! Anybody else who may be able to help? | 13:23 |
| smcginnis | stefan_amann: I think jgriffit1 has the ability to do that. | 13:23 |
| * smcginnis throws jgriffit1 under the bus. | 13:23 | |
| *** Zhang has joined #openstack-cinder | 13:24 | |
| *** kvidvans has quit IRC | 13:25 | |
| stefan_amann | thanks! | 13:25 |
| *** ericksonsantos has joined #openstack-cinder | 13:25 | |
| *** annegentle has quit IRC | 13:27 | |
| *** eduardo has joined #openstack-cinder | 13:29 | |
| *** jgriffit1 is now known as jgriffith | 13:32 | |
| eharney | stefan_amann: i nominated the bug for kilo, but the main thing is just to get the patch submitted for kilo after it lands in master | 13:33 |
| *** diablo_rojo has joined #openstack-cinder | 13:34 | |
| *** Yogi1 has joined #openstack-cinder | 13:34 | |
| stefan_amann | thanks... did you do it for os-brick and cinder? I released it to Liberty/os-brick already. There is no need to release it to Liberty/Cinder. | 13:35 |
| jgriffith | stefan_amann: I'm mixed on it, but I think everyone will agree on and I've seen more questionable patches for IBM sail through Nova and Cinder | 13:35 |
| jgriffith | stefan_amann: You'll need to propose it and of course modify it as there's no brick in kilo | 13:35 |
| stefan_amann | It only affects System z because OpenStack has to use a special command to add each LUN. And the command is sensitive to the format. We discovered that the current implementation does not support LUN IDs >0xff. Hence OpenStack users can attach a max of 255 LUNs. | 13:37 |
| stefan_amann | System z partitions can be pretty large and powerful. And we have products integrating the Kilo code. That's why we would like to backport | 13:37 |
| *** nug is now known as cFouts | 13:38 | |
| stefan_amann | eharney: thanks for nominating! Can you also nominate it for os-brick / Kiloß | 13:38 |
| eharney | stefan_amann: there's no kilo release of os-brick, you have to fix it in Cinder | 13:40 |
| stefan_amann | ok :-) | 13:40 |
| stefan_amann | Thanks much, guys!! | 13:40 |
| *** merooney has joined #openstack-cinder | 13:41 | |
| *** annasort has quit IRC | 13:43 | |
| *** annasort_ has joined #openstack-cinder | 13:43 | |
| *** annasort_ is now known as annasort | 13:43 | |
| *** kvidvans has joined #openstack-cinder | 13:48 | |
| aarefiev | jgriffith, hi hcould you share your thoughts about https://review.openstack.org/#/c/200551/ | 13:50 |
| jordanP | smcginnis, you can now "view" a gzipped file without downloading it : https://37.187.159.67:5443/job/cinder-sofs-validate/8256/artifact/jenkins-logs/ (you have to click on the "view" link) | 13:52 |
| *** breitz has quit IRC | 13:52 | |
| *** breitz has joined #openstack-cinder | 13:52 | |
| *** annasort has quit IRC | 13:52 | |
| eikke | thanks to @jordanP's mad nginx skills ^^ | 13:53 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/215440 | 13:53 |
| jgriffith | aarefiev: seems ok | 13:56 |
| jgriffith | DuncanT: I would be curious to see why you're so interested in this particular patch? | 13:56 |
| jgriffith | aarefiev: DuncanT I believe that sometimes we give people too many options, and create things that we can't possibly support or maintain well :( | 13:56 |
| jgriffith | aarefiev: it would probably be good if we started being selective in our options if you ask me | 13:57 |
| smcginnis | jordanP, eikke: Cool! A little unconventional, but I think that probably meets most requirements. | 13:57 |
| smcginnis | jordanP, eikke: Can you update your comment posting to include the /artifact/jenkins-logs/ at the end of the URL. | 13:57 |
| smcginnis | jordanP, eikke: That way it's obvious and no need to click around to find your logs. | 13:58 |
| jordanP | smcginnis, yep I'll look at it | 13:58 |
| smcginnis | jordanP, eikke: I think that would be good enough (at least for now) to meet the requirements. | 13:58 |
| smcginnis | jordanP: Thanks! | 13:58 |
| *** mtanino has joined #openstack-cinder | 13:58 | |
| aarefiev | jgriffith: yep, but it's really nice option, especially for big envs with a lot concurrent access | 13:59 |
| smcginnis | aarefiev: I see value in it, even though I don't personally have need for it. | 13:59 |
| smcginnis | aarefiev: Seems like a good deployment option. | 13:59 |
| smcginnis | jgriffith: Good point though about more options to maintain. | 14:00 |
| jgriffith | aarefiev: :) | 14:00 |
| jgriffith | aarefiev: anybody looking at doing the same across the other services? | 14:00 |
| smcginnis | jgriffith: Keystone appears to have it. | 14:00 |
| jgriffith | smcginnis: I was mainly thinking Nova | 14:01 |
| aarefiev | jgriffith: unfortunately nova and ironic already did it( | 14:01 |
| *** diogogmt has quit IRC | 14:01 | |
| jgriffith | Well in that case.... | 14:01 |
| jgriffith | I'm just behind the times :) | 14:01 |
| aarefiev | I thought cinder will be the first one:) | 14:01 |
| *** eharney has quit IRC | 14:02 | |
| aarefiev | after keystone | 14:02 |
| *** thangp has joined #openstack-cinder | 14:03 | |
| *** rushiagr is now known as rushiagr_away | 14:03 | |
| *** annegentle has joined #openstack-cinder | 14:03 | |
| *** diogogmt has joined #openstack-cinder | 14:04 | |
| *** links has joined #openstack-cinder | 14:05 | |
| jgriffith | stefan_amann: Given this hasn't even landed in os-brick yet you might want to focus on that first :) | 14:09 |
| jgriffith | stefan_amann: https://review.openstack.org/#/c/216411/3 | 14:10 |
| jgriffith | hemnafk: ^^ when you get in this morning lemme know | 14:10 |
| stefan_amann | jgriffith: I'm confused now :-) isn't that the patch for os-brick? Or what do you mean with 'landed' | 14:12 |
| stefan_amann | does 'landed' mean merged? | 14:14 |
| stefan_amann | sorry for the dumb question :-) | 14:14 |
| *** annasort has joined #openstack-cinder | 14:15 | |
| *** yrabl has quit IRC | 14:16 | |
| *** annasort_ has joined #openstack-cinder | 14:18 | |
| jgriffith | stefan_amann: no worries :) | 14:18 |
| *** diogogmt has quit IRC | 14:18 | |
| jgriffith | stefan_amann: yes, by landed I mean merged | 14:18 |
| wilson2 | angela-s: are you Angela Smith? | 14:19 |
| stefan_amann | ok :-) | 14:19 |
| wilson2 | angela-s: :) | 14:19 |
| jgriffith | stefan_amann: and there's also now a tricky point because even after it merges in os-brick, it's still not picked up by Cinder until a new lib is released | 14:19 |
| jgriffith | stefan_amann: so it makes things rather sloowwwwww going :) | 14:19 |
| *** annasort has quit IRC | 14:20 | |
| *** annasort_ is now known as annasort | 14:20 | |
| *** takedakn has quit IRC | 14:22 | |
| *** rushil has joined #openstack-cinder | 14:22 | |
| openstackgerrit | Diem Tran proposed openstack/cinder: Local cache feature of Oracle ZFSSA drivers https://review.openstack.org/212155 | 14:22 |
| *** rushiagr_away is now known as rushiagr | 14:22 | |
| *** haomaiwang has joined #openstack-cinder | 14:22 | |
| *** lifeless1 has joined #openstack-cinder | 14:23 | |
| *** lifeless has quit IRC | 14:24 | |
| yuriy_n17 | jgriffith: Hi! | 14:25 |
| *** edtubill has joined #openstack-cinder | 14:27 | |
| *** ankit_ag has quit IRC | 14:27 | |
| *** EinstCrazy has quit IRC | 14:28 | |
| jordanP | kaisers, hi. are you silvain kaiser ? | 14:30 |
| jordanP | silvan sorry | 14:30 |
| jgriffith | yuriy_n17: What's up? | 14:31 |
| yuriy_n17 | jgriffith: sorry for persistence. what about to review https://review.openstack.org/#/c/215666/ Thanks a lot for advance.;) | 14:32 |
| jgriffith | yuriy_n17: sorry, didn't know you updated it | 14:36 |
| yuriy_n17 | jgriffith: Thanks a lot ;) | 14:37 |
| *** Zhang has quit IRC | 14:37 | |
| *** mtanino has quit IRC | 14:37 | |
| *** eharney has joined #openstack-cinder | 14:37 | |
| jgriffith | yuriy_n17: no problem | 14:43 |
| *** mtanino has joined #openstack-cinder | 14:43 | |
| DuncanT | jgriffith: My main interest with that patch is in API performance - it's currently pretty bad (in terms of requests per second), and running it under apache seems to give a noticable boost for little effort | 14:44 |
| *** tpsilva has joined #openstack-cinder | 14:45 | |
| jgriffith | DuncanT: but that patch doesn't run it under Apache :) | 14:45 |
| jgriffith | it's nginx | 14:45 |
| jgriffith | regardless I'm cool with the patch for sure | 14:45 |
| DuncanT | jgriffith: Bah, so I'm going senile, no surprise | 14:45 |
| jgriffith | I just wanted to see if somebody had some insight that I wasn't aware of as folks are pretty adament about merging it | 14:46 |
| jgriffith | and I personally haven't tried it | 14:46 |
| *** Zhang has joined #openstack-cinder | 14:46 | |
| stefan_amann | jgriffith: to your previous comment: is this related to Liberty only? We're less concerned about Liberty. Kilo is more pressing... | 14:46 |
| jgriffith | Was just curious is all | 14:46 |
| DuncanT | jgriffith: I tried it, seems to work fine and give up to 20% on the db-light API queries rate | 14:47 |
| e0ne | jgriffith: actually, it allows run c-api both under apache & nginx. may be adding sample config for nginx if a good idea | 14:47 |
| DuncanT | jgriffith: For team wide values of 'I' | 14:47 |
| jgriffith | stefan_amann: so now that we have os-brick library, it means that changes merge there.... then eventually a never version of the lib is pushed out for release, at which time it shows up in Cinder and no in Nova | 14:47 |
| jgriffith | DuncanT: cool | 14:47 |
| *** Zhang has quit IRC | 14:47 | |
| e0ne | jgriffith: i didn't put my +2 on it becasue of i was author of spec and helped a bit with patch. | 14:48 |
| jgriffith | e0ne: YES, we'll def need docs | 14:48 |
| e0ne | jgriffith: agree | 14:48 |
| jgriffith | e0ne: that should be proopsed to openstack-docs for config options etc | 14:48 |
| e0ne | jgriffith: we'll do it | 14:48 |
| jgriffith | e0ne: yeah, figured that migt be the case | 14:48 |
| *** Zhang has joined #openstack-cinder | 14:48 | |
| jgriffith | e0ne: I knew you were involved with it | 14:49 |
| jgriffith | e0ne: thanks! | 14:49 |
| *** tsekiyama has joined #openstack-cinder | 14:49 | |
| *** Zhang has quit IRC | 14:49 | |
| e0ne | jgriffith: also i'm going to post some banchmarks results once i'll have a time for it | 14:49 |
| jgriffith | e0ne: cool... always nice to have data | 14:49 |
| eikke | smcginnis: fwiw, our CI now posts a link straight to the artifacts/log page. I sent a recheck message to the review you commented on, but later runs (e.g. for 214595) already have the updated link | 14:49 |
| *** Zhang has joined #openstack-cinder | 14:49 | |
| e0ne | jgriffith: yea, i did it for keystone, it's time to make cinder perf tests :) | 14:50 |
| jgriffith | e0ne: DuncanT one thing I've been reading (true or not... it's on the internet so who knows) is that there's in some cases an equal trade off for the performance | 14:52 |
| jgriffith | e0ne: DuncanT in error rate | 14:52 |
| jgriffith | e0ne: DuncanT and mixed data on memory usage | 14:53 |
| e0ne | jgriffith: which cases do you mean? | 14:53 |
| jgriffith | e0ne: DuncanT in other words, "yeah" it's really fast using nginx, but it has a higher error-rate and larger memory footprint | 14:53 |
| jgriffith | e0ne: DuncanT take that with a grain of salt of course :) | 14:54 |
| DuncanT | jgriffith: Memory usage definitely goes up a bit, haven't tested nearly nearly enough to comment on errors - that worries me rather more | 14:54 |
| DuncanT | jgriffith: The memory usage compared to a modern service is still small, so that doesn't worry me at all | 14:54 |
| jgriffith | DuncanT: e0ne I certainly think it's a fine option to have, but as I commented on the patch the commit message is a bit biased :) | 14:54 |
| jgriffith | Would love to hear feedback from folks that are maybe running Nova with it in production | 14:55 |
| e0ne | :) | 14:55 |
| jgriffith | I'm not knocking the patch, I'm trying to figure out if I want to invest in using it or not :) | 14:55 |
| * thrawn01 is happy we are looking at preformance, I was planning on looking at it soonish, our cinder api in production currently preforms pretty badly | 14:55 | |
| *** anshul has quit IRC | 14:55 | |
| jgriffith | thrawn01: what version of Cinder? | 14:56 |
| jgriffith | thrawn01: frankly we've gone metadata crazy the last few releases and I think we're suffering from it | 14:56 |
| jgriffith | thrawn01: that and really "bad" programming in some of the API calls | 14:56 |
| smcginnis | eikke: Awesome! Thanks Scality team for the attention and quick turnaround. Nice work! | 14:57 |
| thrawn01 | lol, that is the best question! it's currently icehouse, but we are planning a quick upgrade cycle so we can get on the latest and start squashing bugs/performance issues that effect us. | 14:57 |
| jgriffith | thrawn01: like iterating over a list of all volumes and doing multiple fetches on each one :( | 14:57 |
| jgriffith | thrawn01: icehouse had a MAJOR problem in list-volumes | 14:57 |
| thrawn01 | yeah, icehouse is better than havana for sure. but it's still not great. | 14:57 |
| *** Zhang has quit IRC | 14:57 | |
| thrawn01 | oh yeah, I know it well | 14:58 |
| jgriffith | thrawn01: did a get on all volumes, then iterated as I described and did multiple fetches to get various metadata etc | 14:58 |
| *** topshare has joined #openstack-cinder | 14:58 | |
| jgriffith | thrawn01: :) | 14:58 |
| *** topshare has quit IRC | 14:58 | |
| *** topshare has joined #openstack-cinder | 14:58 | |
| thrawn01 | jgriffith: =( we had to disable some extensions as they made the situation worse. we counted 4 N+1 selects with all the extensions running | 14:58 |
| jgriffith | anywho... some day it would be awesome if we cleaned up our code :) | 14:59 |
| *** ronis has quit IRC | 14:59 | |
| *** links has quit IRC | 14:59 | |
| jgriffith | thrawn01: ouch!! | 14:59 |
| jgriffith | thrawn01: you should file a bug and list those out so folks can take a look at them | 14:59 |
| thrawn01 | once we get closer to liberty, I'll do just that! | 14:59 |
| jgriffith | thrawn01: it's hard to believe how badly we abuse the DB :( | 14:59 |
| thrawn01 | I'm currently working through some smaller easy to fix patches to get my feet wet. | 15:00 |
| jgriffith | thrawn01: just willie-nillie throw in some db calls anywhere you like... what could go wrong :) | 15:00 |
| thrawn01 | and trying to figure out thest testing stuff, I just now got tox working.... I think | 15:00 |
| jgriffith | thrawn01: :) Well let me know if I can help with tox | 15:01 |
| *** topshare has quit IRC | 15:01 | |
| jgriffith | thrawn01: I recently updated the devref docs in tree and hopefully that will help people at least a little | 15:01 |
| thrawn01 | oh.... | 15:01 |
| *** topshare has joined #openstack-cinder | 15:01 | |
| * thrawn01 looks | 15:01 | |
| *** kvidvans has quit IRC | 15:01 | |
| *** daneyon has joined #openstack-cinder | 15:01 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Support initialization state in Backup Manager https://review.openstack.org/216728 | 15:02 |
| *** asd112z has joined #openstack-cinder | 15:02 | |
| thrawn01 | jgriffith: in cinder or cinderclient source? | 15:02 |
| *** topshare has quit IRC | 15:02 | |
| jgriffith | thrawn01: cinder | 15:03 |
| *** topshare has joined #openstack-cinder | 15:03 | |
| *** vgridnev has quit IRC | 15:03 | |
| jgriffith | thrawn01: same work could/should be done in client | 15:03 |
| jgriffith | I think | 15:03 |
| *** topshare has quit IRC | 15:03 | |
| *** topshare has joined #openstack-cinder | 15:04 | |
| smcginnis | jgriffith, DuncanT, geguileo, e0ne, hemnafk, xyang1: Not sure what to think of this one: https://review.openstack.org/#/c/214194/ | 15:04 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Validate filters in snapshot*, backup* in db.api https://review.openstack.org/217110 | 15:04 |
| e0ne | smcginnis: just -1 on it! :) | 15:04 |
| smcginnis | e0ne: :) | 15:04 |
| e0ne | now, the long version.... | 15:05 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: utils.synchronized lock name formatting https://review.openstack.org/217258 | 15:05 |
| DuncanT | smcginnis: While I'd love to see the block device driver die a quick death | 15:05 |
| e0ne | that patch will be updated soon to support loopback devices | 15:05 |
| smcginnis | DuncanT: That's my dilemma. | 15:05 |
| *** crose has joined #openstack-cinder | 15:05 | |
| xyang1 | smcginnis: No snapshot support still? | 15:06 |
| e0ne | DuncanT: why? | 15:06 |
| *** topshare has quit IRC | 15:06 | |
| DuncanT | e0ne: There was talk at the midcycle 6 months ago about getting it fixed up to minimal feature support, not happened though | 15:07 |
| e0ne | xyang1: it's by design. we can't create snapshot from a whole devide | 15:07 |
| *** Zhang has joined #openstack-cinder | 15:07 | |
| e0ne | s/devide/device | 15:07 |
| jgriffith | smcginnis: see my comments on the patch: https://review.openstack.org/#/c/214194/ | 15:07 |
| *** r-daneel has joined #openstack-cinder | 15:07 | |
| DuncanT | e0ne: Do what rax do for snaps and copy the whole lot somewhere else? | 15:07 |
| smcginnis | DuncanT: I think that's another topic that has been discussed multiple times with no resolution. | 15:07 |
| e0ne | DuncanT: that's an option | 15:07 |
| guitarzan | that's gonna be difficult without dm snapshots helping out... | 15:07 |
| jgriffith | e0ne: see my comment in the patch | 15:07 |
| DuncanT | guitarzan: Offline only snaps? It will at least pass tempest | 15:08 |
| jgriffith | e0ne: DuncanT smcginnis my proposal was and still is, remove it from Cinder and have it be part of Sahara | 15:08 |
| guitarzan | DuncanT: I do suppose that's right | 15:08 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Python 3 incompatible expression fix https://review.openstack.org/215099 | 15:08 |
| *** hemnafk is now known as hemna | 15:09 | |
| smcginnis | jgriffith: I think I agree with that approach. | 15:09 |
| *** haomaiwang has quit IRC | 15:09 | |
| smcginnis | jgriffith: Either meet minimum requirements and have CI coverage, or move it out and maintain it in Sahara - the one and only place it's needed. | 15:09 |
| jgriffith | smcginnis: or if you want to do big-data *stuff* use Cinder storage that's capable of doing it | 15:09 |
| DuncanT | guitarzan: Having a driver in that only supports a fraction of our 'mandatory' features seems rather broken, and the performance .v. lvmthick seems to be lost in the noise according to my tests | 15:09 |
| smcginnis | jgriffith: Yeah, I don't think it was every proven the the block device driver is absolutely needed to meet the performance requirements needed. | 15:10 |
| *** boris-42 has quit IRC | 15:10 | |
| e0ne | what if we'll implement required features and ci? | 15:10 |
| hemna | jgriffith, +1 on your comment on that patch | 15:10 |
| *** edtubill has left #openstack-cinder | 15:10 | |
| smcginnis | What DuncanT said. | 15:10 |
| *** asd112z has quit IRC | 15:10 | |
| *** haomaiwang has joined #openstack-cinder | 15:10 | |
| guitarzan | DuncanT: yeah, I don't understand the "lvm is crippling" idea myself | 15:10 |
| DuncanT | e0ne: Then it becomes just another driver and most of us stop caring I think | 15:10 |
| smcginnis | jgriffith: Yes, good comment on patch. | 15:11 |
| e0ne | i'll propose patch for infra job once devstack integration will be finished | 15:11 |
| smcginnis | e0ne: Yeah, I'd be fine with it then. | 15:11 |
| *** asd112z has joined #openstack-cinder | 15:11 | |
| *** timcl1 has joined #openstack-cinder | 15:11 | |
| jgriffith | e0ne: FYI it won't run them currently | 15:11 |
| DuncanT | guitarzan: Even with ramdisks as the backing store straight R/W performance was withing 5% or so for me | 15:11 |
| hemna | but, if there is no CI on it, aren't we supposed to remove it ? | 15:11 |
| jgriffith | e0ne: it will fail miserable :( | 15:11 |
| hemna | that's the standard now anyway | 15:11 |
| e0ne | DuncanT, smcginnis: fair enough | 15:11 |
| smcginnis | jgriffith: Then we can remove it for failing CI. :) | 15:11 |
| e0ne | lol | 15:11 |
| guitarzan | DuncanT: I think you're preaching to the choir :) | 15:11 |
| DuncanT | hemna: It has been a special snowflake for a while now | 15:11 |
| jgriffith | hemna: I don't know if you noticed but there are a quite a few drivers that don't have CI right now :( | 15:11 |
| hemna | yup | 15:12 |
| DuncanT | guitarzan: :-) | 15:12 |
| smcginnis | We probably need to do an audit of that again. | 15:12 |
| hemna | jgriffith, which is ok for a time, as long as the CI is being worked on and communication with the driver maintainer | 15:12 |
| hemna | none of which is true for bdd ? | 15:12 |
| *** timcl has quit IRC | 15:12 | |
| smcginnis | What ever happened to the CI dashboard work? | 15:12 |
| *** jistr is now known as jistr|mtg | 15:12 | |
| jgriffith | Well here's a quick sample of just the last 48 hours: http://ec2-54-67-102-119.us-west-1.compute.amazonaws.com:5000/?project=openstack%2Fcinder&user=&timeframe=48&start=&end=&page_size=900 | 15:13 |
| e0ne | ok. we need working CI and minimum features set in bdd | 15:13 |
| *** jaypipes has joined #openstack-cinder | 15:13 | |
| jgriffith | I have no idea how patrickeast 's tool there actually works, but it seems pretty :) | 15:13 |
| e0ne | hemna: tbh, i didn't get any asks about adding new features to the bdd driver:(. it's my fault tooo | 15:14 |
| *** Zhang has quit IRC | 15:14 | |
| *** diogogmt has joined #openstack-cinder | 15:15 | |
| *** zhangjn has joined #openstack-cinder | 15:15 | |
| *** zhangjn has quit IRC | 15:16 | |
| *** zhangjn has joined #openstack-cinder | 15:16 | |
| *** bnemec has joined #openstack-cinder | 15:17 | |
| jgriffith | hemna: http://54.164.167.86/solidfire-ci-logs/refs-changes-95-214595-5/logs/screen-c-vol.log.txt, 2015-08-26 15:11:52.449 | 15:17 |
| jgriffith | hemna: looks suspect | 15:17 |
| *** zhangjn has quit IRC | 15:17 | |
| *** stefan_amann has quit IRC | 15:18 | |
| hemna | jgriffith, hrmm...wth is that | 15:18 |
| jgriffith | hemna: I have no idea :) | 15:18 |
| *** vgridnev has joined #openstack-cinder | 15:18 | |
| jgriffith | hemna: but didn't that rootwrap patch for brick land recently? | 15:19 |
| asselin | there's a new ci dashboard in town: http://ci-watch.tintri.com/project?project=cinder&time=48+hours | 15:19 |
| *** zhangjinnan has joined #openstack-cinder | 15:19 | |
| jgriffith | hemna: so I'm testing against os-brick master | 15:19 |
| *** zhangjinnan has quit IRC | 15:19 | |
| hemna | jgriffith, hasn't landed yet | 15:19 |
| hemna | https://review.openstack.org/#/c/207553/ | 15:19 |
| jgriffith | hemna: yeah, just saw that... wth.... | 15:20 |
| jgriffith | weird | 15:20 |
| *** zhangjn has joined #openstack-cinder | 15:20 | |
| hemna | no idea what that one is | 15:20 |
| hemna | never seen it before :( | 15:20 |
| jgriffith | I've hit it a few times now | 15:20 |
| jgriffith | looking into it | 15:20 |
| hemna | looks like the self.instream isn't exactly what they expected then | 15:21 |
| hemna | probably some other object that doesn't have read ? | 15:21 |
| *** changbl has quit IRC | 15:21 | |
| hemna | very strange | 15:21 |
| jgriffith | hemna: the thing is it's in the os-brick connector that it's puking it seems | 15:21 |
| *** BharatK has joined #openstack-cinder | 15:22 | |
| *** zhangjn has quit IRC | 15:22 | |
| hemna | yah, it looks like it's trying to read the initiator file to get the host iqn | 15:22 |
| *** zhangjn has joined #openstack-cinder | 15:22 | |
| *** rushil has quit IRC | 15:22 | |
| *** zhangjn has quit IRC | 15:22 | |
| jgriffith | hemna: yeah.. the call to 'cat' | 15:22 |
| *** jistr|mtg is now known as jistr | 15:22 | |
| jgriffith | meeeeooowwww | 15:22 |
| hemna | :) | 15:23 |
| *** zhangjn has joined #openstack-cinder | 15:23 | |
| jgriffith | but it's actually the shlex lib? | 15:23 |
| jgriffith | hmm | 15:23 |
| *** kvidvans has joined #openstack-cinder | 15:24 | |
| hemna | I that's the oslo processutils doing that | 15:25 |
| hemna | no idea what that is. | 15:25 |
| jgriffith | hemna: I think I might know... just a sec | 15:25 |
| *** zhangjn has quit IRC | 15:25 | |
| *** zhangjn has joined #openstack-cinder | 15:26 | |
| *** zhangjn has quit IRC | 15:26 | |
| *** zhangjn has joined #openstack-cinder | 15:26 | |
| *** zhangjn has quit IRC | 15:26 | |
| *** martyturner has joined #openstack-cinder | 15:27 | |
| *** zhangjn has joined #openstack-cinder | 15:27 | |
| *** zhangjn has quit IRC | 15:27 | |
| *** zhangjn has joined #openstack-cinder | 15:28 | |
| jgriffith | hmm...or not :( | 15:28 |
| *** zhangjn has quit IRC | 15:28 | |
| *** zhangjn has joined #openstack-cinder | 15:29 | |
| *** jdurgin1 has joined #openstack-cinder | 15:30 | |
| hemna | wish I knew how this was reproduced | 15:30 |
| jgriffith | hemna: looks like it's 100% for me right now | 15:31 |
| jgriffith | woner if shlex updated? | 15:31 |
| hemna | hrmm | 15:31 |
| jgriffith | and something broke.... | 15:31 |
| jgriffith | hmm... nope, that's part of python, not an external lib... weird | 15:32 |
| hemna | is shlex itself part of python? | 15:33 |
| hemna | oh heh | 15:33 |
| hemna | was just looking for it | 15:33 |
| jgriffith | :) | 15:33 |
| *** zhangjn has quit IRC | 15:33 | |
| hemna | oslo_concurrency 2.5.0 was released on 8/24 | 15:34 |
| *** haomaiwang has quit IRC | 15:34 | |
| *** edtubill has joined #openstack-cinder | 15:35 | |
| aarefiev | jgriffith: do you looking on ci on my patch? | 15:35 |
| jgriffith | aarefiev: I have no idea how to parse what you just said :) | 15:36 |
| *** bluex-pl has quit IRC | 15:36 | |
| aarefiev | jgriffith: if you have something like 'RootwrapProcessHelper' object has no attribute 'read' | 15:37 |
| hemna | jgriffith, but I don't see anything in processutils.py around that call that's changed recently | 15:37 |
| *** vgridnev has quit IRC | 15:37 | |
| jgriffith | aarefiev: that's the error I'm getting yes, when shlex tries to parse the tokenized root_helper that's passed in | 15:37 |
| jgriffith | I need to run down the hall before our meeting... bb in a few | 15:38 |
| aarefiev | jgriffith: don't worry, this is one https://review.openstack.org/#/c/214595/ causes it | 15:38 |
| hemna | is gerrit just horribly slow today? | 15:39 |
| smcginnis | hemna: Today? | 15:39 |
| hemna | smcginnis, :) | 15:39 |
| hemna | aarefiev, that hasn't landed though | 15:40 |
| *** afazekas has quit IRC | 15:40 | |
| Swanson | hemna: It certainly was yesterday. And review pages are taking forever to load today. I just assumed Dell added a few extra firewalls between here and the rest of humanity. | 15:41 |
| *** vgridnev has joined #openstack-cinder | 15:41 | |
| aarefiev | hemna: jgriffith told he get the same error, my patch introduce 'RootwrapProcessHelper' :) | 15:41 |
| hemna | Swanson, it's abnormally slow for me here today. I thought it just might be the super awesome HP network | 15:41 |
| thrawn01 | anyone here run tests on osx? | 15:42 |
| thrawn01 | ran into some trouble getting pycrypto installed for pypy | 15:42 |
| smcginnis | hemna: Is it down now? Just got a failure posting CI results. Network is unreachable. | 15:43 |
| hemna | doh | 15:44 |
| -openstackstatus- NOTICE: restarting gerrit due to a slow memory leak | 15:44 | |
| hemna | smcginnis, lol, I guess that answers your question. | 15:44 |
| *** alexschm has quit IRC | 15:44 | |
| smcginnis | Third times a charm. Worked now. | 15:44 |
| smcginnis | hemna: Darn internets. | 15:44 |
| hemna | smcginnis, openstackstatus/#openstack-cinder- NOTICE: restarting gerrit due to a slow memory leak | 15:44 |
| hemna | smcginnis, in case you missed that | 15:44 |
| *** garthb has joined #openstack-cinder | 15:45 | |
| smcginnis | hemna: Yep, didn't see that yet. | 15:45 |
| *** mtanino has quit IRC | 15:45 | |
| *** mtanino has joined #openstack-cinder | 15:48 | |
| *** dannywilson has joined #openstack-cinder | 15:48 | |
| *** scohen1 has joined #openstack-cinder | 15:49 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix backup metadata import missing fields https://review.openstack.org/183222 | 15:49 |
| *** agarciam has quit IRC | 15:50 | |
| *** dflorea has joined #openstack-cinder | 15:51 | |
| *** vincent_hou has joined #openstack-cinder | 15:52 | |
| *** p0rtal has joined #openstack-cinder | 15:53 | |
| *** vincent_hou_ has joined #openstack-cinder | 15:53 | |
| smcginnis | Anyone else going to be at a large vendor conference in San Francisco next week? | 15:54 |
| jgriffith | aarefiev: ahhh :) | 15:54 |
| hemna | smcginnis, ? | 15:54 |
| jgriffith | aarefiev: curious why it passes gate though ? | 15:54 |
| jgriffith | but yes, it was that one and it's dep | 15:54 |
| smcginnis | hemna: Going to be in San Fran next week through Thursday. Just wondering if anyone else here happens to be going. | 15:54 |
| aarefiev | jgriffith: there are still places where code uses processutils directly | 15:55 |
| *** adurbin_ has joined #openstack-cinder | 15:55 | |
| jgriffith | aarefiev: yeah, and you'll need to figure out some way to coordinate the with os-brick it appears | 15:55 |
| * hemna is confused how the RootwrapProcessHelper is already being used when it hasn't landed yet in Cinder | 15:56 | |
| Swanson | Got a simple patch that really should only impact our CI (as the function in question should really only happen in the tempest tests) https://review.openstack.org/#/c/216859/ | 15:56 |
| *** vincent_hou has quit IRC | 15:56 | |
| *** vincent_hou_ is now known as vincent_hou | 15:56 | |
| *** vgridnev has quit IRC | 15:56 | |
| aarefiev | jgriffith: yeah, a big patch, touches all drivers in diff way ( | 15:57 |
| *** rajinir has joined #openstack-cinder | 15:57 | |
| rhe00 | smcginnis: Are you going to promote OpenStack at that conference? ;) | 15:57 |
| hemna | jgriffith, oh, your CI log is from the solidfire CI test of that patch. | 15:57 |
| hemna | duh | 15:57 |
| *** mtanino_ has joined #openstack-cinder | 15:57 | |
| smcginnis | rhe00: Where I can. ;) | 15:57 |
| jgriffith | hemna: :) | 15:57 |
| smcginnis | rhe00: Are you going? | 15:58 |
| hemna | I catch up....eventually. | 15:58 |
| jgriffith | hemna: I was suspicious because I had two in a row... didn't notice the second was dependent on the first | 15:58 |
| rhe00 | smcginnis: No, didn't make it on the list. | 15:58 |
| jgriffith | hemna: haha... that's alright, luckily aarefiev was paying attention for both of us :) | 15:58 |
| jgriffith | hemna: aarefiev I knew I saw that rootwrap change, but confused it with the one in os-brick that hasn't merged yet | 15:59 |
| *** skylerberg has joined #openstack-cinder | 15:59 | |
| jgriffith | shall we have a meeting? | 15:59 |
| smcginnis | rhe00: Oh well. Not too excited about going. Been to enough of them now. | 15:59 |
| aarefiev | jgriffith: we should refactor the way how we execute cmds, all drivers do it in diff way, we need common approach | 15:59 |
| jgriffith | aarefiev: agreed!!!! | 15:59 |
| smcginnis | Meeting time. | 15:59 |
| hemna | jgriffith, looks like the Netapp cDOT-iSCSI CI is failing for the exact same reason | 16:00 |
| *** markstur has quit IRC | 16:00 | |
| *** mtanino has quit IRC | 16:00 | |
| *** p0rtal has quit IRC | 16:00 | |
| *** dims_ has quit IRC | 16:00 | |
| *** p0rtal has joined #openstack-cinder | 16:00 | |
| *** changbl has joined #openstack-cinder | 16:00 | |
| *** dims has joined #openstack-cinder | 16:01 | |
| anteaya | eharney: how did your patch make out after you removed the circular dependency? | 16:02 |
| *** timcl has joined #openstack-cinder | 16:04 | |
| *** _afazekas is now known as afazekas | 16:05 | |
| *** p0rtal has quit IRC | 16:05 | |
| *** DTadrzak has joined #openstack-cinder | 16:05 | |
| *** timcl1 has quit IRC | 16:05 | |
| *** markstur has joined #openstack-cinder | 16:07 | |
| *** nkrinner has quit IRC | 16:08 | |
| thrawn01 | anyone ever encounter an issue where you just can't install ANYTHING within a virtualenv? | 16:10 |
| thrawn01 | http://pastebin.com/mdk1REfQ | 16:10 |
| thrawn01 | I create a virtualenv with 2.6, and pip is unable to install anything. | 16:10 |
| *** jistr has quit IRC | 16:12 | |
| *** jordanP has quit IRC | 16:13 | |
| *** Apoorva has joined #openstack-cinder | 16:13 | |
| *** kvidvans has quit IRC | 16:14 | |
| *** yuriy_n17 has quit IRC | 16:15 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 16:20 |
| *** leeantho has joined #openstack-cinder | 16:21 | |
| *** agarciam has joined #openstack-cinder | 16:24 | |
| *** dubutante has joined #openstack-cinder | 16:24 | |
| *** _cjones_ has joined #openstack-cinder | 16:26 | |
| *** ronis has joined #openstack-cinder | 16:27 | |
| openstackgerrit | Huang Zhiteng proposed openstack/os-brick: iSCSI Connector: force disconnect iscsi session https://review.openstack.org/217032 | 16:29 |
| *** martyturner1 has joined #openstack-cinder | 16:30 | |
| *** martyturner has quit IRC | 16:32 | |
| *** skylerberg has quit IRC | 16:34 | |
| *** jdurgin1 has quit IRC | 16:34 | |
| *** dflorea has quit IRC | 16:35 | |
| bswartz | jgriffith: I'm trying to find the patch you referred to about capabilities during the meeting | 16:40 |
| *** p0rtal has joined #openstack-cinder | 16:40 | |
| *** p0rtal has quit IRC | 16:40 | |
| *** vincent_hou_ has joined #openstack-cinder | 16:45 | |
| *** vincent_hou has quit IRC | 16:46 | |
| *** vincent_hou_ is now known as vincent_hou | 16:46 | |
| *** rushil has joined #openstack-cinder | 16:48 | |
| *** nikeshm_ has joined #openstack-cinder | 16:49 | |
| *** nikeshm_ has quit IRC | 16:50 | |
| *** lpetrut has quit IRC | 16:51 | |
| *** navneet_ has joined #openstack-cinder | 16:52 | |
| navneet_ | experimental thing makes sense for feature with no wide acceptability | 16:52 |
| *** navneet_ has quit IRC | 16:53 | |
| *** simondodsley has quit IRC | 16:53 | |
| *** simondodsley has joined #openstack-cinder | 16:53 | |
| *** mudassirlatif has joined #openstack-cinder | 16:58 | |
| jbernard | vincent_hou: ping regarding in-use volume migration | 17:00 |
| *** martyturner1 has quit IRC | 17:01 | |
| hemna | mep | 17:02 |
| vincent_hou | HI | 17:02 |
| *** jseiler_ has quit IRC | 17:02 | |
| jgriffith | bswartz: https://review.openstack.org/#/c/201243/25 | 17:02 |
| bswartz | jgriffith: ty | 17:03 |
| *** dubutante has quit IRC | 17:03 | |
| jgriffith | vincent_hou: sorry we ran out of time | 17:03 |
| jgriffith | vincent_hou: but I think everybody is here | 17:03 |
| *** winston-d_ has joined #openstack-cinder | 17:03 | |
| vincent_hou | The good news with https://review.openstack.org/#/c/204953/ is that we can report the progress for the volume migration. | 17:03 |
| vincent_hou | No problem. | 17:03 |
| vincent_hou | However. | 17:03 |
| vincent_hou | folks have concerns that I make the driver too heavy. | 17:03 |
| *** tpsilva has quit IRC | 17:04 | |
| *** jseiler has joined #openstack-cinder | 17:04 | |
| vincent_hou | doing things like getting the process id, reading the files, calculating, etc. | 17:04 |
| eharney | the architecture here just isn't right... it's pushing too much stuff into the driver that shouldn't be there | 17:04 |
| jgriffith | eharney: I agree | 17:04 |
| vincent_hou | Right. | 17:04 |
| hemna | I like the feature and what it brings to Cinder. I just think it's in the wrong place. eharney +1 | 17:04 |
| eharney | same here | 17:04 |
| winston-d_ | process id? that's crazy, why does anybody cares about that. | 17:05 |
| vincent_hou | Shall I move them to manager? | 17:05 |
| jgriffith | winston-d_: hehe | 17:05 |
| eharney | it's a great idea, it just needs to be done more in the volume manager | 17:05 |
| jgriffith | I'd propose we fine tune it and fix thing up to use manager and bumpt it out to M at this point | 17:05 |
| vincent_hou | I need to get the process id for "dd" to check how much transferred. | 17:05 |
| hemna | jgriffith, +1 | 17:06 |
| eharney | vincent_hou: it's reasonable to add some driver hooks if we figure out that that's necessary... i'm not sure what the final interface would look like | 17:06 |
| *** mistermarkwest has joined #openstack-cinder | 17:06 | |
| jgriffith | vincent_hou: make a genrator that returns chunk completion :) | 17:06 |
| *** trree has joined #openstack-cinder | 17:06 | |
| guitarzan | it's a question of how much of the "default implementation" is a driver detail vs a manager detail | 17:06 |
| eharney | but because of questions like that, i'm also thinking what jgriffith said | 17:06 |
| *** Yogi1 has quit IRC | 17:06 | |
| *** boris-42 has joined #openstack-cinder | 17:07 | |
| *** jungleboyj has joined #openstack-cinder | 17:07 | |
| *** rushiagr is now known as rushiagr_away | 17:07 | |
| vincent_hou | For the migration with dd command, I think I can figure out a sort of interface. | 17:08 |
| *** rushiagr_away is now known as rushiagr | 17:08 | |
| jgriffith | vincent_hou: you want to take some time on it and postpone to M? | 17:08 |
| jgriffith | vincent_hou: I don't see it as critical for L at this point | 17:08 |
| jgriffith | vincent_hou: fact is there are still problems with migrate outside of progress reporting | 17:09 |
| jgriffith | vincent_hou: although you've fixed up a good deal of things | 17:09 |
| *** rushiagr is now known as rushiagr_away | 17:09 | |
| *** mistermarkwest has quit IRC | 17:09 | |
| vincent_hou | Well, it is part of the BP. frastrate me a bit. | 17:09 |
| jgriffith | vincent_hou: understand completely | 17:10 |
| vincent_hou | if it is delayed. | 17:10 |
| jgriffith | vincent_hou: but it's a pretty heavy change | 17:10 |
| *** rushiagr_away is now known as rushiagr | 17:10 | |
| jgriffith | vincent_hou: if you think you can redesign things in the next week I guess that's up to you | 17:10 |
| vincent_hou | Y. | 17:10 |
| *** diablo_rojo has quit IRC | 17:10 | |
| *** trree has quit IRC | 17:10 | |
| jgriffith | vincent_hou: the problem as I see it though is you have 7 other patches outstanding associated with that BP | 17:11 |
| jgriffith | vincent_hou: it doesn't seem like it's possible to get all of them finalized and stable in the next few days | 17:11 |
| *** skylerberg has joined #openstack-cinder | 17:11 | |
| vincent_hou | True. | 17:11 |
| openstackgerrit | Sonia Ghanekar proposed openstack/cinder: Add manage/unmanage volume support for Nimble https://review.openstack.org/214392 | 17:11 |
| jgriffith | vincent_hou: it's of course up to you | 17:11 |
| vincent_hou | Things getting much for me here | 17:11 |
| jgriffith | vincent_hou: you have the approved bp's and the reviews up | 17:11 |
| jgriffith | vincent_hou: but I don't think anybody is going to rush them through as they are now | 17:12 |
| vincent_hou | I see. | 17:12 |
| jgriffith | vincent_hou: and I don't know that it's worth you killing yourself writing code 24/7 for the next week | 17:12 |
| jgriffith | vincent_hou: but that's completely up to you | 17:12 |
| hemna | ok off to a meeting/beating. | 17:13 |
| vincent_hou | Thank you,, John. I know what you mean. | 17:13 |
| *** hemna is now known as hemnafk | 17:13 | |
| *** winston-d is now known as winston-d_zZZ | 17:13 | |
| *** trree has joined #openstack-cinder | 17:13 | |
| jgriffith | vincent_hou: it's great that you've stepped up and are working so hard on fixing this up | 17:13 |
| jgriffith | vincent_hou: I'm just saying you need to decide how to spend your time :) | 17:14 |
| vincent_hou | I will do my best, but will leave the option open for the patches. | 17:14 |
| jgriffith | vincent_hou: thanks!! | 17:14 |
| vincent_hou | :-) | 17:14 |
| vincent_hou | Yeah | 17:14 |
| *** e0ne has quit IRC | 17:15 | |
| vincent_hou | Thank you, John. Thanks everyone for the reviews. | 17:15 |
| jgriffith | patrickeast: kmartin hemnafk winston-d_ geguileo e0ne So I'm proposing that for the *next* release (M) we have an earlier deadline for core features | 17:15 |
| patrickeast | jgriffith: i like that idea | 17:15 |
| jgriffith | patrickeast: kmartin hemnafk winston-d_ geguileo e0ne at least two weeks before feature freeze | 17:16 |
| jgriffith | xyang1: ^^ | 17:16 |
| patrickeast | jgriffith: i had been wondering if it makes sense to push new drivers only into the third milestone and have 1 and 2 for core features | 17:16 |
| patrickeast | then do a similar milestone 2 freeze like nova | 17:16 |
| *** lpetrut has joined #openstack-cinder | 17:16 | |
| jgriffith | patrickeast: TBH we used to do it that way | 17:16 |
| jgriffith | patrickeast: well... not exactly, and not like Nova | 17:16 |
| jgriffith | patrickeast: but driver mods were open all the way to the third milestone | 17:17 |
| *** tpsilva has joined #openstack-cinder | 17:17 | |
| patrickeast | jgriffith: gotcha | 17:17 |
| vincent_hou | jbernard: hi | 17:17 |
| jgriffith | patrickeast: that was changed to stop the "new" driver submissions, then kinda extended itself to other things | 17:17 |
| jgriffith | patrickeast: but the key IMO is there should be a buffer there | 17:17 |
| jgriffith | patrickeast: and using Nova's model is not really the same in Cinder because of our plugin model | 17:18 |
| xyang1 | jgriffith: I thought we tried last time but difficult to get things merged that early | 17:18 |
| jgriffith | xyang1: I don't recall | 17:18 |
| jgriffith | xyang1: and by early I'm just saying a couple weeks | 17:18 |
| xyang1 | jgriffith: I'll have to check | 17:18 |
| jgriffith | xyang1: so even saying driver updates all the way to 3'rd milestone | 17:18 |
| jgriffith | xyang1: and stil the two week buffer for core features | 17:19 |
| jgriffith | xyang1: but I could be completely wrong... maybe we did try it | 17:19 |
| xyang1 | jgriffith: I think the 2 nd milestone seems difficult to meet, but not against trying it again | 17:19 |
| jgriffith | xyang1: I also think that maybe we should tie it to the feature itself | 17:19 |
| jgriffith | xyang1: oh, I'm certainly not proposing that!!! | 17:20 |
| jgriffith | xyang1: I'm saying a couple of weeks is all | 17:20 |
| jgriffith | xyang1: and I'm thinking what if you tied dates to the feature's release in core | 17:20 |
| xyang1 | jgriffith: sure | 17:20 |
| jgriffith | xyang1: so for example, if it landed in core early... well then you're doomed | 17:20 |
| jgriffith | xyang1: but if it lands at the last hour of the freeze date there's an amnisty period of sorts for drivers to implement it | 17:21 |
| jgriffith | xyang1: I think it should be that or nothing until the next release... not sure which I think is better | 17:21 |
| xyang1 | jgriffith: Why you are doomed if it land early? | 17:21 |
| jgriffith | xyang1: but that will be a great summit topic IMO | 17:22 |
| jgriffith | xyang1: I mean if it lands in core early, you have no excuses and no grace period | 17:22 |
| xyang1 | jgriffith: Ya, good to discuss at the summit | 17:22 |
| xyang1 | jgriffith: Oh, I see:) | 17:22 |
| xyang1 | jgriffith: So what does the voting suggest? Any action now? | 17:24 |
| dannywilson | that sounds good to discuss for next cycle but why are you trying to change the rules for L too? Changing rules mid cycle seems like a slippery slope | 17:25 |
| *** stefan_amann has joined #openstack-cinder | 17:27 | |
| jgriffith | xyang1: dannywilson bswartz I did the voting all wrong :( | 17:27 |
| dannywilson | oh noes | 17:27 |
| jgriffith | I should have proposed it a choice between the two options in the vote itself | 17:27 |
| *** daneyon has quit IRC | 17:27 | |
| xyang1 | jgriffith: oh:( | 17:27 |
| jgriffith | but it seems like the majority were in favor of extending deadline for driver to implement late landing features | 17:28 |
| xyang1 | jgriffith: You can still count manually:) | 17:28 |
| jgriffith | xyang1: yeah, that's what I did :) | 17:28 |
| jgriffith | but it would've solved the issue of people voting the same on both options (like I did) | 17:29 |
| xyang1 | jgriffith: :) | 17:29 |
| jgriffith | xyang1: I'm out of practice using meetbot after the past year :) | 17:29 |
| jgriffith | or | 17:30 |
| xyang1 | jgriffith: :) | 17:30 |
| jgriffith | vote-bot at any rate | 17:30 |
| *** agarciam has quit IRC | 17:31 | |
| *** kvidvans has joined #openstack-cinder | 17:33 | |
| bswartz | jgriffith: at least you don't have to deal with the parliamentary differences between "yea", "aye", "no", and "nay". | 17:33 |
| *** stefan_amann has quit IRC | 17:34 | |
| *** martyturner has joined #openstack-cinder | 17:34 | |
| xyang1 | bswartz: We should count those as well:) | 17:35 |
| jgriffith | bswartz: LOL | 17:35 |
| *** dflorea has joined #openstack-cinder | 17:38 | |
| *** martyturner has quit IRC | 17:39 | |
| *** rushiagr is now known as rushiagr_away | 17:41 | |
| *** dims_ has joined #openstack-cinder | 17:41 | |
| *** p0rtal has joined #openstack-cinder | 17:41 | |
| swamireddy | HI | 17:41 |
| *** dims has quit IRC | 17:42 | |
| swamireddy | Any one working cinder active-active functionality support ? | 17:42 |
| *** winston-d_ has quit IRC | 17:43 | |
| *** aix has quit IRC | 17:44 | |
| *** hemnafk is now known as hemna | 17:44 | |
| hemna | swamireddy, there are notes on that from the mid cycle meetup | 17:44 |
| scottda | swamireddy: https://etherpad.openstack.org/p/cinder-meetup-summer-2015 | 17:44 |
| *** dflorea has quit IRC | 17:46 | |
| *** trree has quit IRC | 17:47 | |
| *** stefan_amann has joined #openstack-cinder | 17:47 | |
| *** trree has joined #openstack-cinder | 17:47 | |
| *** diablo_rojo has joined #openstack-cinder | 17:48 | |
| *** jdurgin has quit IRC | 17:49 | |
| *** Yogi1 has joined #openstack-cinder | 17:49 | |
| *** rushiagr_away is now known as rushiagr | 17:53 | |
| openstackgerrit | Ryan Lucio proposed openstack/cinder: VMEM v7000: support storage pools / volume types https://review.openstack.org/195773 | 17:57 |
| *** andreykurilin__ has joined #openstack-cinder | 18:07 | |
| *** trree has quit IRC | 18:08 | |
| *** jdurgin has joined #openstack-cinder | 18:09 | |
| *** annasort has quit IRC | 18:10 | |
| *** yrabl has joined #openstack-cinder | 18:10 | |
| *** annegentle has quit IRC | 18:14 | |
| *** openstackgerrit has quit IRC | 18:17 | |
| *** openstackgerrit has joined #openstack-cinder | 18:17 | |
| *** zhenguo has quit IRC | 18:23 | |
| *** e0ne has joined #openstack-cinder | 18:25 | |
| *** tpsilva has quit IRC | 18:27 | |
| *** aix has joined #openstack-cinder | 18:30 | |
| *** jaypipes has quit IRC | 18:31 | |
| *** vincent_hou has quit IRC | 18:32 | |
| *** ociuhandu has quit IRC | 18:33 | |
| *** vincent_hou has joined #openstack-cinder | 18:34 | |
| *** shyama has quit IRC | 18:36 | |
| swamireddy | hemna: scottda: Thanks for the link.. | 18:37 |
| *** lifeless1 is now known as lifeless | 18:39 | |
| scottda | swamireddy: sure. Sorry that there's no quick, easy answer there... | 18:39 |
| *** lpetrut has quit IRC | 18:39 | |
| *** tpsilva has joined #openstack-cinder | 18:39 | |
| scottda | people are working on this, it's just a bunch of different stuff in different places. | 18:39 |
| swamireddy | One more Q is - I am upgrading the cinder from J ->K (or I->J). Here, after upgrading the binaries/pkgs, I will run the db_sync cmd to sync dB tables as per the upgrade. Is the db_sync is safe to run or do we need to take any sfaty steps before db_sync cmd? | 18:40 |
| *** jaypipes has joined #openstack-cinder | 18:40 | |
| *** esker has joined #openstack-cinder | 18:40 | |
| asselin_ | patrickeast, did find an issue. pushed a small change to fix | 18:41 |
| swamireddy | scottda: yep, thats correct...its discussion stage ATM | 18:41 |
| *** esker has quit IRC | 18:41 | |
| scottda | swamireddy: Sorry, I don't think I can help with the upgrade question. | 18:42 |
| *** esker has joined #openstack-cinder | 18:42 | |
| patrickeast | asselin_: sweet, thanks! i still haven't had any time to upgrade any of my systems :( | 18:42 |
| jgriffith | swamireddy: make a bckup of your db just to be safe | 18:43 |
| jgriffith | mysql-dump | 18:43 |
| jgriffith | swamireddy: You should be fine, but always recommend at least a mysql-dump prior to the migration | 18:44 |
| jgriffith | swamireddy: it's easy and a small precaution to take :) | 18:44 |
| swamireddy | scottda: Thanks. nop. | 18:44 |
| *** martyturner has joined #openstack-cinder | 18:44 | |
| *** tpsilva has quit IRC | 18:45 | |
| *** diegows has joined #openstack-cinder | 18:47 | |
| *** vincent_hou has quit IRC | 18:47 | |
| *** jdurgin has quit IRC | 18:48 | |
| *** e0ne has quit IRC | 18:50 | |
| *** jwcroppe has quit IRC | 18:50 | |
| *** asd112z_ has joined #openstack-cinder | 18:52 | |
| *** mtanino_ has quit IRC | 18:53 | |
| *** tsekiyam_ has joined #openstack-cinder | 18:53 | |
| *** dims_ has quit IRC | 18:53 | |
| *** dims has joined #openstack-cinder | 18:54 | |
| *** mtanino has joined #openstack-cinder | 18:54 | |
| *** asd112z has quit IRC | 18:56 | |
| *** tsekiyama has quit IRC | 18:57 | |
| *** tpsilva has joined #openstack-cinder | 18:58 | |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: Cinder Nested Quota Driver https://review.openstack.org/205369 | 19:01 |
| *** eduardo has quit IRC | 19:03 | |
| *** rushiagr is now known as rushiagr_away | 19:03 | |
| *** sidbhatt13 has joined #openstack-cinder | 19:05 | |
| *** jdurgin has joined #openstack-cinder | 19:07 | |
| *** lpetrut has joined #openstack-cinder | 19:07 | |
| *** jwcroppe has joined #openstack-cinder | 19:08 | |
| *** sidbhatt13 has quit IRC | 19:09 | |
| *** tpsilva has quit IRC | 19:12 | |
| *** e0ne has joined #openstack-cinder | 19:13 | |
| *** tsekiyam_ has quit IRC | 19:13 | |
| *** krtaylor has quit IRC | 19:14 | |
| *** esker has quit IRC | 19:15 | |
| *** lpetrut has quit IRC | 19:18 | |
| *** sidbhatt13 has joined #openstack-cinder | 19:19 | |
| *** yrabl has quit IRC | 19:20 | |
| *** BharatK has quit IRC | 19:21 | |
| *** Lee1092 has quit IRC | 19:23 | |
| *** stefan_amann has quit IRC | 19:24 | |
| openstackgerrit | Michael Price proposed openstack/cinder: Reduce runtime of E-Series iSCSI tests https://review.openstack.org/217361 | 19:25 |
| *** ronis has quit IRC | 19:28 | |
| *** daneyon has joined #openstack-cinder | 19:29 | |
| openstackgerrit | Derrick Wippler proposed openstack/python-cinderclient: Fixed test_password_prompted https://review.openstack.org/214704 | 19:31 |
| *** martyturner1 has joined #openstack-cinder | 19:31 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Implement manage/unmanage snapshot in Pure drivers https://review.openstack.org/213855 | 19:32 |
| *** martyturner has quit IRC | 19:33 | |
| *** daneyon has quit IRC | 19:33 | |
| *** adurbin_ has quit IRC | 19:34 | |
| *** tsekiyama has joined #openstack-cinder | 19:37 | |
| *** BharatK has joined #openstack-cinder | 19:37 | |
| *** mtanino_ has joined #openstack-cinder | 19:37 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Remove driver.set_execute() https://review.openstack.org/217364 | 19:38 |
| *** mtanino has quit IRC | 19:39 | |
| *** rushil has quit IRC | 19:41 | |
| *** lpetrut has joined #openstack-cinder | 19:41 | |
| *** ociuhandu has joined #openstack-cinder | 19:44 | |
| *** daneyon has joined #openstack-cinder | 19:48 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Remove driver.set_execute() https://review.openstack.org/217364 | 19:50 |
| *** daneyon has quit IRC | 19:54 | |
| *** daneyon has joined #openstack-cinder | 19:55 | |
| *** scohen1 has quit IRC | 19:58 | |
| *** sidbhatt13 has quit IRC | 20:00 | |
| *** scohen1 has joined #openstack-cinder | 20:01 | |
| *** scohen1 has quit IRC | 20:01 | |
| *** jwcroppe has quit IRC | 20:03 | |
| *** lpetrut1 has joined #openstack-cinder | 20:04 | |
| *** daneyon has quit IRC | 20:09 | |
| *** jwcroppe has joined #openstack-cinder | 20:10 | |
| *** krtaylor has joined #openstack-cinder | 20:11 | |
| *** Yogi1 has quit IRC | 20:12 | |
| *** jwcroppe has quit IRC | 20:14 | |
| *** merooney has quit IRC | 20:14 | |
| smcginnis | Looks like Summit presentation acceptance letters are starting to go out. | 20:15 |
| hemna | smcginnis, yup. Looks like I lucked out from doing preso's this summit :P | 20:18 |
| smcginnis | hemna: Dang it. Want to switch? :) | 20:18 |
| *** akerr has quit IRC | 20:20 | |
| *** porrua has quit IRC | 20:20 | |
| hemna | smcginnis, how many do you get to do? | 20:20 |
| *** martyturner1 has quit IRC | 20:21 | |
| cebruns | smcginnis: Everyone wants to hear the new core's song and dance. :) | 20:22 |
| e0ne | hemna: :). the next challenge - to get visa. you won it too | 20:24 |
| smcginnis | cebruns: :P | 20:24 |
| hemna | e0ne, heh | 20:24 |
| smcginnis | hemna: Just one. We've got the conversation with Cinder developers session again. One more as an alternate. | 20:24 |
| hemna | ah cool that one | 20:24 |
| *** daneyon has joined #openstack-cinder | 20:24 | |
| hemna | I'm not sure I was invited to that one | 20:25 |
| hemna | I'll sit in the audience and ask standardization and HA questions. | 20:25 |
| hemna | :P | 20:25 |
| smcginnis | hemna: I can see if I can add you as a speaker yet. ;) | 20:25 |
| e0ne | need to take a break to sleep. bb in ~12 hours:) | 20:27 |
| hemna | that sleep thing.... | 20:27 |
| *** e0ne has quit IRC | 20:28 | |
| *** rushil has joined #openstack-cinder | 20:29 | |
| *** rushil_ has joined #openstack-cinder | 20:29 | |
| *** sidbhatt13 has joined #openstack-cinder | 20:30 | |
| *** rushil has quit IRC | 20:34 | |
| *** simondodsley has quit IRC | 20:34 | |
| smcginnis | hemna: You ever see leftover scsi devices on the host after a volume is detached? | 20:41 |
| jgriffith | smcginnis: LOL | 20:41 |
| smcginnis | jgriffith: Is this a known thing? :) | 20:42 |
| *** bswartz has quit IRC | 20:42 | |
| jgriffith | smcginnis: I'll let hemna answer that one :) | 20:42 |
| hemna | oh man | 20:43 |
| hemna | smcginnis, https://review.openstack.org/#/c/203265/ | 20:43 |
| smcginnis | hemna: Oh, that's good. | 20:43 |
| smcginnis | hemna: This was reported to me by one of our test guys. | 20:43 |
| hemna | good :) | 20:44 |
| smcginnis | hemna: Wouldn't be at all surprised if he's running older code than that. | 20:44 |
| hemna | seems your test guys are good then :) | 20:44 |
| smcginnis | ;) | 20:44 |
| smcginnis | hemna: Has that been released in a new os-brick library? Or still pending? | 20:44 |
| hemna | not to pypi yet | 20:44 |
| smcginnis | hemna: Cool, awesome. I can at least let them know the status. Thanks! | 20:45 |
| hemna | np | 20:45 |
| smcginnis | hemna: I need to check, but it's possible they are still hitting Kilo. Is/was it an issue there, and if so has that been backported? | 20:47 |
| hemna | so Kilo is a different story, because Nova hasn't used os-brick until a few weeks ago | 20:48 |
| hemna | I think we switched Cinder to use os-brick in Juno | 20:48 |
| hemna | and prior to that it was the internal version of brick | 20:48 |
| smcginnis | Sounds right. | 20:49 |
| *** asd112z_ has quit IRC | 20:51 | |
| *** asd112z has joined #openstack-cinder | 20:52 | |
| *** asd112z has quit IRC | 20:52 | |
| *** asd112z has joined #openstack-cinder | 20:53 | |
| Swanson | smcginnis: test dude reported it on the 6th. So, well before the merge of that defect. | 20:57 |
| *** lpetrut1 has quit IRC | 20:57 | |
| *** dustins has quit IRC | 20:57 | |
| *** julim has quit IRC | 20:58 | |
| *** lpetrut has quit IRC | 20:58 | |
| *** dflorea has joined #openstack-cinder | 20:58 | |
| *** raildo is now known as raildo-afk | 20:59 | |
| *** jwcroppe has joined #openstack-cinder | 20:59 | |
| *** kambiz has quit IRC | 21:00 | |
| *** kvidvans has quit IRC | 21:01 | |
| *** agarciam has joined #openstack-cinder | 21:01 | |
| *** openstackgerrit has quit IRC | 21:01 | |
| *** openstackgerrit has joined #openstack-cinder | 21:02 | |
| *** winston-d_zZZ has quit IRC | 21:03 | |
| *** winston-d_zZZ has joined #openstack-cinder | 21:03 | |
| *** kambiz has joined #openstack-cinder | 21:03 | |
| *** garthb_ has joined #openstack-cinder | 21:03 | |
| *** jdandrea has joined #openstack-cinder | 21:05 | |
| *** r-daneel has quit IRC | 21:05 | |
| *** garthb_ has quit IRC | 21:06 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 21:06 |
| jdandrea | I seem to recall a presentation that illustrates how to use the QoS features in cinder. Is anyone aware of this and, if so, where to locate it? (Searching online. Found a NetApp presentation so far. Looking for others.) | 21:06 |
| *** garthb_ has joined #openstack-cinder | 21:06 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: Cleanup for cinder tests with CGSnapshot https://review.openstack.org/207866 | 21:06 |
| *** garthb has quit IRC | 21:06 | |
| *** sgotliv has joined #openstack-cinder | 21:07 | |
| *** dflorea has quit IRC | 21:10 | |
| *** crose has quit IRC | 21:11 | |
| *** daneyon_ has joined #openstack-cinder | 21:16 | |
| *** daneyon has quit IRC | 21:19 | |
| *** edtubill has left #openstack-cinder | 21:20 | |
| *** rushil_ has quit IRC | 21:22 | |
| *** timcl has quit IRC | 21:23 | |
| jungleboyj | hemna: You there? | 21:24 |
| *** agarciam has quit IRC | 21:25 | |
| jungleboyj | kmartin: You around? | 21:28 |
| *** IanGovett has quit IRC | 21:29 | |
| kmartin | jungleboyj, yeah | 21:30 |
| jungleboyj | kmartin: Hey man, I have a silly FC related question. Guess I could have pinged angela-s as well. | 21:31 |
| jungleboyj | kmartin: Where is it that the lookup service gets its credentials from for logging into the fabric? I can't find that in the code for some reason. | 21:32 |
| kmartin | jungleboyj, sure, is it how to configure the IBM FC driver? | 21:32 |
| jungleboyj | kmartin: No ... How do you guys set up the creds to talk to the FC switch ? | 21:33 |
| kmartin | in cinder.conf | 21:33 |
| kmartin | jungleboyj, just for you http://www8.hp.com/h20195/v2/GetPDF.aspx/4AA5-9485ENW.pdf | 21:33 |
| kmartin | jungleboyj, page 11 fc_fabric_user = admin | 21:35 |
| kmartin | fc_fabric_password = password | 21:35 |
| jungleboyj | kmartin: Thank you. Do they not use SSH keys at all? | 21:36 |
| *** jamielennox has quit IRC | 21:36 | |
| jungleboyj | kmartin: Only username/password? | 21:37 |
| *** jaypipes has quit IRC | 21:37 | |
| kmartin | jungleboyj, correct | 21:37 |
| jungleboyj | kmartin: That was the piece I was missing. Thank you! | 21:37 |
| *** esker has joined #openstack-cinder | 21:37 | |
| jungleboyj | I had assumed that they would also support SSH keys and couldn't figure out why I couldn't find a place to define that. | 21:38 |
| *** jamielennox has joined #openstack-cinder | 21:38 | |
| kmartin | yeah, angela-s was talking about using the REST interface on the switch and getting rid of SSH altogether, but not sure where that ended up | 21:39 |
| *** alejandrito has joined #openstack-cinder | 21:39 | |
| *** mriedem has quit IRC | 21:40 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port test_volume to Python 3 https://review.openstack.org/215729 | 21:45 |
| *** esker has quit IRC | 21:46 | |
| *** esker has joined #openstack-cinder | 21:46 | |
| *** krtaylor has quit IRC | 21:48 | |
| *** diablo_rojo has quit IRC | 21:49 | |
| *** alejandrito has quit IRC | 21:51 | |
| *** jaypipes has joined #openstack-cinder | 21:52 | |
| *** eharney has quit IRC | 21:53 | |
| *** jungleboyj has quit IRC | 21:55 | |
| *** esker has quit IRC | 21:56 | |
| *** dims_ has joined #openstack-cinder | 21:56 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port volume transfer to Python 3 https://review.openstack.org/215752 | 21:59 |
| *** edmondsw has quit IRC | 21:59 | |
| *** dims has quit IRC | 21:59 | |
| *** diogogmt has quit IRC | 22:08 | |
| *** sidbhatt13 has quit IRC | 22:09 | |
| *** merooney has joined #openstack-cinder | 22:12 | |
| *** merooney has joined #openstack-cinder | 22:12 | |
| *** merooney has quit IRC | 22:13 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: py3: Port pure driver test to Python 3 https://review.openstack.org/217408 | 22:16 |
| *** sgotliv has quit IRC | 22:19 | |
| *** dims_ has quit IRC | 22:24 | |
| *** dims has joined #openstack-cinder | 22:24 | |
| openstackgerrit | xing-yang proposed openstack/cinder: ScaleIO driver should use os-brick connector https://review.openstack.org/214420 | 22:26 |
| *** jwcroppe has quit IRC | 22:30 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Adds framework for get_capabilities() feature https://review.openstack.org/201243 | 22:32 |
| *** gouthamr has quit IRC | 22:32 | |
| *** jaypipes has quit IRC | 22:33 | |
| *** dims has quit IRC | 22:34 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Fix Python 3 issues in wsgi https://review.openstack.org/217417 | 22:35 |
| rhagarty__ | xyang1: hello - regarding consistency groups... is there any way to determine the assigned volume type that is set when the CG is created? | 22:35 |
| *** hemna is now known as hemnafk | 22:36 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Allow specified backend capabilities to be retrieved https://review.openstack.org/212861 | 22:37 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Efficient volume copy for generic volume migration https://review.openstack.org/183701 | 22:38 |
| *** krtaylor has joined #openstack-cinder | 22:39 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 22:40 |
| *** markvoelker has quit IRC | 22:46 | |
| *** thangp has quit IRC | 22:46 | |
| *** dflorea has joined #openstack-cinder | 22:46 | |
| *** gouthamr has joined #openstack-cinder | 22:53 | |
| *** markvoelker has joined #openstack-cinder | 22:54 | |
| *** dflorea has quit IRC | 23:06 | |
| *** dflorea has joined #openstack-cinder | 23:07 | |
| *** dflorea has quit IRC | 23:07 | |
| *** diegows has quit IRC | 23:10 | |
| angela-s | jungleboyj: hey, last year our team added support for SSH key, but looks like we never added it to the upstream driver. let me find out what happened and get back to you | 23:14 |
| angela-s | kmartin: we have a HTTPS connector in the VF code review: https://review.openstack.org/#/c/184799/ | 23:16 |
| *** esker has joined #openstack-cinder | 23:16 | |
| *** BharatK has quit IRC | 23:18 | |
| kmartin | angela-s, thanks, I knew I had seen it before. | 23:18 |
| *** bkopilov has quit IRC | 23:19 | |
| *** arif-ali has quit IRC | 23:19 | |
| *** bkopilov has joined #openstack-cinder | 23:19 | |
| *** arif-ali has joined #openstack-cinder | 23:19 | |
| angela-s | jungleboyj: the SSH key param addition is also in the VF code review ^^ | 23:19 |
| *** esker has quit IRC | 23:22 | |
| *** diegows has joined #openstack-cinder | 23:22 | |
| *** esker has joined #openstack-cinder | 23:23 | |
| *** diegows has quit IRC | 23:23 | |
| kmartin | angela-s, on launchpad defect https://bugs.launchpad.net/cinder/+bug/1486613 it seems if someone has attached volumes w/o FCZM then turns on FCZM any detach will fail? So would the process be detach all volumes from host that were pre-zoned, delete the pre-defined zone from the fabric, then enable FCZM and reattach all volumes? | 23:25 |
| openstack | Launchpad bug 1486613 in Cinder "Existing volume detach fails after modifying the zoning_mode=fabric from none" [Undecided,Invalid] | 23:25 |
| *** BharatK has joined #openstack-cinder | 23:30 | |
| angela-s | kmartin: it's drastic, but seems like the only option. Unless by some miracle the pre-defined zone names were formatted in the same way that we do for Openstack... | 23:30 |
| angela-s | kmartin: to support this scenario, FCZM would have to be reworked to not look up by zone name, but by zone members, which would increase the time required to process the call, somewhat drastically i think, as we'd have to process the entire defined zone database | 23:32 |
| kmartin | angela-s, yeah it's pretty ugly but these seems like a valid use case | 23:32 |
| kmartin | I'll bring it up in the FC meeting tomorrow | 23:33 |
| angela-s | sure, good venue to discuss this item | 23:34 |
| *** mriedem has joined #openstack-cinder | 23:36 | |
| kmartin | angela-s, I'll add a comment to the defect with the above. | 23:38 |
| angela-s | kmartin: ok, thanks! | 23:40 |
| tbarron | so I'm seeing cinder.tests.unit.test_vmware_vmdk.VMwareVcVmdkDriverTestCase.test_create_backing_from_stream_optimized_file fail on a bunch of unrelated changes. | 23:45 |
| tbarron | Does that ring a bell with anyone? | 23:45 |
| tbarron | ^^^^ http://paste.openstack.org/show/428909/ | 23:47 |
| *** martyturner has joined #openstack-cinder | 23:53 | |
| *** zhangjn has joined #openstack-cinder | 23:55 | |
| patrickeast | tbarron: i was seeing that on the image cache patchsets too | 23:56 |
| patrickeast | tbarron: couldn't reproduce it locally on my machine though | 23:56 |
| patrickeast | tbarron: i spent some time looking into what the test is doing... but couldn't really figure out what the problem is | 23:57 |
| *** asd112z has quit IRC | 23:57 | |
| patrickeast | the weird part is why a unit test would be reading from a file | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!