| *** salv-orl_ has quit IRC | 00:00 | |
| *** dannywilson has quit IRC | 00:01 | |
| *** _cjones_ has quit IRC | 00:01 | |
| *** martyturner has quit IRC | 00:03 | |
| *** rushil has quit IRC | 00:05 | |
| *** angela-s has quit IRC | 00:05 | |
| *** smoriya has joined #openstack-cinder | 00:11 | |
| *** tellesnobrega_ has joined #openstack-cinder | 00:22 | |
| *** tellesnobrega_ has quit IRC | 00:23 | |
| *** tellesnobrega_ has joined #openstack-cinder | 00:23 | |
| *** tellesnobrega_ has quit IRC | 00:24 | |
| *** mtanino has quit IRC | 00:30 | |
| *** zul has joined #openstack-cinder | 00:34 | |
| *** zhenguo has joined #openstack-cinder | 00:43 | |
| *** takedakn has quit IRC | 00:47 | |
| *** Zhang has joined #openstack-cinder | 00:58 | |
| *** leeantho has quit IRC | 00:58 | |
| *** salv-orlando has joined #openstack-cinder | 01:00 | |
| asselin_ | patrickeast, I pushed the changes to my repo. | 01:01 |
|---|---|---|
| asselin_ | I have 3 of my ci systems using that | 01:01 |
| *** lixiaoy1 has joined #openstack-cinder | 01:02 | |
| patrickeast | asselin_: sweet | 01:02 |
| asselin_ | there's one more change you might like, but requires this to merge first: https://review.openstack.org/#/c/216451/ | 01:02 |
| patrickeast | asselin_: oooh nice, i had wondered about that | 01:03 |
| asselin_ | patrickeast, https://github.com/rasselin/os-ext-testing/commit/d0a3491f4ad141744a4624229140fa819d5a71eb | 01:04 |
| asselin_ | finally...like 1 year later :) | 01:04 |
| patrickeast | haha | 01:04 |
| *** julim has quit IRC | 01:06 | |
| *** xyang has joined #openstack-cinder | 01:10 | |
| *** 64MADOHP4 has joined #openstack-cinder | 01:12 | |
| *** xyang has quit IRC | 01:12 | |
| *** gouthamr has quit IRC | 01:13 | |
| *** jungleboyj has joined #openstack-cinder | 01:18 | |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 01:20 |
| *** sghanekar has quit IRC | 01:23 | |
| *** 64MADOHP4 has quit IRC | 01:24 | |
| *** Apoorva has quit IRC | 01:25 | |
| asselin_ | patrickeast, http://lists.openstack.org/pipermail/openstack-dev/2015-August/072789.html | 01:26 |
| patrickeast | lol | 01:27 |
| patrickeast | asselin_: i guess more options are always better? | 01:28 |
| asselin_ | patrickeast, yes, we can cross correlate status via 4 different dashboards | 01:28 |
| patrickeast | oh man | 01:29 |
| patrickeast | this weekend im going to make a ci dashboard aggregator dashboard | 01:29 |
| asselin_ | well hopefully we can converge | 01:29 |
| asselin_ | lol | 01:29 |
| mtreinish | asselin_: man that's a lot of check marks on that dashboard | 01:30 |
| *** salv-orlando has quit IRC | 01:30 | |
| *** davechen has joined #openstack-cinder | 01:30 | |
| *** salv-orlando has joined #openstack-cinder | 01:31 | |
| mtreinish | asselin_: like: http://ci-watch.tintri.com/project?project=nova&time=7+days I'm not sure how useful that view of things actually is | 01:32 |
| mtreinish | patrickeast: what does yours look like? :) | 01:32 |
| patrickeast | mtreinish: http://ec2-54-67-102-119.us-west-1.compute.amazonaws.com:5000/?project=openstack%2Fcinder&user=&timeframe=24 | 01:32 |
| patrickeast | mtreinish: thats my demo/dev server | 01:32 |
| mtreinish | patrickeast: ec2, really... | 01:33 |
| patrickeast | shh | 01:33 |
| patrickeast | thats why the spec to get infra hosting it is important | 01:33 |
| *** Lee1092 has joined #openstack-cinder | 01:33 | |
| patrickeast | mtreinish: for my company all our external facing dev hosts are ec2 | 01:33 |
| mtreinish | patrickeast: ok cool, you're showing an aggregate view to visualize the failure rate | 01:34 |
| mtreinish | that's what's actually useful as top level info | 01:34 |
| patrickeast | mtreinish: yea its the view i found most helpful in keeping an eye on my ci system | 01:35 |
| *** asd112z has joined #openstack-cinder | 01:35 | |
| *** salv-orlando has quit IRC | 01:36 | |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 01:41 |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 01:43 |
| *** gouthamr has joined #openstack-cinder | 01:44 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Register RPC and object versions https://review.openstack.org/209701 | 01:45 |
| *** tsekiyam_ has joined #openstack-cinder | 01:47 | |
| patrickeast | ugh i don't understand, i keep getting this failure from jenkins http://logs.openstack.org/95/195795/44/check/gate-cinder-python27/7455352/testr_results.html.gz but can't repro it locally | 01:48 |
| patrickeast | i feel like unit tests aren't supposed to be platform specific : / | 01:48 |
| *** tsekiyama has quit IRC | 01:50 | |
| *** tsekiyam_ has quit IRC | 01:51 | |
| *** skylerberg has quit IRC | 01:54 | |
| asselin | patrickeast, mtreinish the part I do like about the other one is that you can see all the jobs. Maybe too much info, but definitely useful to see if one of them is failing | 01:55 |
| *** dims__ has quit IRC | 01:55 | |
| patrickeast | asselin: heh yea, i have been meaning to look into changing that for mine... at the time of hacking it together i only had a single job :p | 01:56 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Use cinder internal tenant to create the destination volume https://review.openstack.org/210237 | 01:57 |
| patrickeast | asselin: the biggest problem i had with it was figuring out a good way to show them without cluttering everywhere... was thinking maybe a click to expand kind of deal so you could (if desired) only see one at a time or maybe one ci system at a time like that | 01:57 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: WIP: Adds migration abortion https://review.openstack.org/207754 | 02:04 |
| *** haomaiwang has joined #openstack-cinder | 02:05 | |
| *** zhongjun has joined #openstack-cinder | 02:07 | |
| *** akerr has joined #openstack-cinder | 02:08 | |
| *** akerr_ has joined #openstack-cinder | 02:09 | |
| *** julim has joined #openstack-cinder | 02:11 | |
| *** zhonghua-lee has joined #openstack-cinder | 02:11 | |
| *** akerr has quit IRC | 02:12 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 02:14 |
| *** ebalduf has quit IRC | 02:14 | |
| *** ebalduf has joined #openstack-cinder | 02:15 | |
| *** xiaohui has quit IRC | 02:19 | |
| *** xiaohui has joined #openstack-cinder | 02:19 | |
| *** jwang has joined #openstack-cinder | 02:21 | |
| *** akerr_ has quit IRC | 02:32 | |
| *** hemna has joined #openstack-cinder | 02:34 | |
| *** bkopilov has quit IRC | 02:34 | |
| *** gouthamr has quit IRC | 02:43 | |
| *** p0rtal has quit IRC | 02:44 | |
| openstackgerrit | chenying proposed openstack/cinder: Fix order of arguments in assertEqual https://review.openstack.org/216502 | 02:44 |
| *** p0rtal has joined #openstack-cinder | 02:44 | |
| *** p0rtal has quit IRC | 02:49 | |
| *** hakimo_ has joined #openstack-cinder | 02:52 | |
| *** zhongjun has quit IRC | 02:53 | |
| *** hakimo has quit IRC | 02:55 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:02 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Fix the virtual port support in VNX driver https://review.openstack.org/214044 | 03:19 |
| *** skylerberg has joined #openstack-cinder | 03:22 | |
| *** gouthamr has joined #openstack-cinder | 03:22 | |
| *** links has joined #openstack-cinder | 03:24 | |
| *** asd112z has quit IRC | 03:31 | |
| *** hemna has quit IRC | 03:31 | |
| *** diogogmt has quit IRC | 03:34 | |
| *** skylerberg has quit IRC | 03:46 | |
| *** zhongjun has joined #openstack-cinder | 03:50 | |
| *** p0rtal has joined #openstack-cinder | 03:55 | |
| *** zhongjun has quit IRC | 04:00 | |
| *** zhongjun has joined #openstack-cinder | 04:01 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** 17WAAILMU has joined #openstack-cinder | 04:01 | |
| *** bkopilov has joined #openstack-cinder | 04:18 | |
| *** garthb has joined #openstack-cinder | 04:29 | |
| *** zhongjun has quit IRC | 04:40 | |
| *** gouthamr has quit IRC | 04:42 | |
| *** bnemec has quit IRC | 04:42 | |
| *** Zhang has quit IRC | 04:47 | |
| *** shyama has joined #openstack-cinder | 04:47 | |
| *** Zhang has joined #openstack-cinder | 04:48 | |
| *** Zhang has quit IRC | 04:51 | |
| *** Zhang has joined #openstack-cinder | 04:51 | |
| *** shyama has quit IRC | 05:00 | |
| *** 17WAAILMU has quit IRC | 05:01 | |
| *** garthb has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-cinder | 05:02 | |
| *** BharatK has joined #openstack-cinder | 05:07 | |
| *** ebalduf has quit IRC | 05:08 | |
| *** nkrinner has joined #openstack-cinder | 05:16 | |
| *** hemna has joined #openstack-cinder | 05:17 | |
| *** hemna has quit IRC | 05:22 | |
| *** dannywilson has joined #openstack-cinder | 05:23 | |
| jgriffith | patrickeast: that's nice! | 05:24 |
| patrickeast | jgriffith: whats nice? :o | 05:25 |
| jgriffith | patrickeast: your summary page for CI | 05:25 |
| *** shausy has joined #openstack-cinder | 05:25 | |
| patrickeast | jgriffith: ahh gotcha | 05:25 |
| jgriffith | patrickeast: so is that just looking at items that merged? | 05:26 |
| patrickeast | jgriffith: the scoreboard tracks just ci postings in general, at most one per patchset | 05:27 |
| patrickeast | jgriffith: so like rechecks can change them | 05:28 |
| jgriffith | patrickeast: hmm, I see | 05:28 |
| jgriffith | patrickeast: I'm trying to figure out the delta in numbers | 05:28 |
| jgriffith | patrickeast: so if I look over 24 hours I have 30+ runs, but even a search on 96 hours on yours (which should be 100's) doesn't show much | 05:29 |
| jgriffith | Not sure that's how it works or not... re the score | 05:29 |
| patrickeast | hehe mine was off all weekend | 05:29 |
| patrickeast | i turned it back to not-quiet mode around 11 | 05:30 |
| patrickeast | am* | 05:30 |
| patrickeast | ohh wait | 05:30 |
| patrickeast | you mean solidfire? | 05:30 |
| patrickeast | did you adjust the page size? | 05:30 |
| jgriffith | patrickeast: yeah, so for example: http://ec2-54-67-102-119.us-west-1.compute.amazonaws.com:5000/?project=openstack%2Fcinder&user=sfci&timeframe=700&start=&end=&page_size= | 05:30 |
| patrickeast | the default is kinda small | 05:30 |
| jgriffith | Ahhhhhh | 05:31 |
| jgriffith | Oh, interesting... that changes things drastically :) | 05:31 |
| *** asd112z has joined #openstack-cinder | 05:32 | |
| patrickeast | haha yea, at some point i wanna bug one of my designer friends to suggest some better ui layout for the page... its not super intuitive | 05:32 |
| patrickeast | i dunno how to make nice ui's | 05:32 |
| * jgriffith doesn't know how to make UI's at all :) | 05:32 | |
| *** dannywilson has quit IRC | 05:34 | |
| *** dannywilson has joined #openstack-cinder | 05:35 | |
| *** deepakcs has joined #openstack-cinder | 05:48 | |
| *** asd112z has quit IRC | 05:51 | |
| *** zhenguo has quit IRC | 05:53 | |
| *** links has quit IRC | 05:57 | |
| *** links has joined #openstack-cinder | 06:00 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-cinder | 06:01 | |
| *** zhenguo has joined #openstack-cinder | 06:09 | |
| *** baojg has joined #openstack-cinder | 06:10 | |
| *** boris-42 has quit IRC | 06:12 | |
| *** serverascode has quit IRC | 06:12 | |
| *** DuncanT has quit IRC | 06:12 | |
| *** rhefner has quit IRC | 06:12 | |
| *** patrickeast has quit IRC | 06:12 | |
| openstackgerrit | chenying proposed openstack/cinder: Fix order of arguments in assertEqual https://review.openstack.org/216502 | 06:13 |
| *** agarciam has joined #openstack-cinder | 06:13 | |
| *** serverascode has joined #openstack-cinder | 06:14 | |
| *** DuncanT has joined #openstack-cinder | 06:15 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Transifex https://review.openstack.org/214468 | 06:15 |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Move CloneableVD to common functions https://review.openstack.org/203593 | 06:16 |
| *** rhefner has joined #openstack-cinder | 06:16 | |
| *** baojg has quit IRC | 06:16 | |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Move CloneableVD to common functions https://review.openstack.org/203593 | 06:17 |
| *** patrickeast has joined #openstack-cinder | 06:17 | |
| *** boris-42 has joined #openstack-cinder | 06:17 | |
| *** baojg has joined #openstack-cinder | 06:19 | |
| *** zhongjun has joined #openstack-cinder | 06:21 | |
| *** anshul has joined #openstack-cinder | 06:24 | |
| *** skylerberg has joined #openstack-cinder | 06:26 | |
| *** zhongjun has quit IRC | 06:27 | |
| openstackgerrit | Abhishek Shrivastava proposed openstack/cinder: Adding delete-wait-loop for CloudByte Volumes https://review.openstack.org/201059 | 06:29 |
| *** p0rtal has quit IRC | 06:42 | |
| *** baojg has quit IRC | 06:42 | |
| *** baojg has joined #openstack-cinder | 06:43 | |
| *** yrabl has joined #openstack-cinder | 06:51 | |
| *** ankit_ag has joined #openstack-cinder | 06:53 | |
| *** alexschm has joined #openstack-cinder | 06:53 | |
| *** pots has quit IRC | 07:00 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:02 | |
| *** shausy has quit IRC | 07:02 | |
| *** shausy has joined #openstack-cinder | 07:03 | |
| *** nkrinner has quit IRC | 07:05 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/215440 | 07:06 |
| *** nkrinner has joined #openstack-cinder | 07:09 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Data transformation betweeen encrypted volumes and images https://review.openstack.org/216567 | 07:09 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/216569 | 07:10 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/210918 | 07:11 |
| *** ronis_ has joined #openstack-cinder | 07:12 | |
| *** ronis__ has joined #openstack-cinder | 07:13 | |
| *** ronis_ has quit IRC | 07:17 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Add consistency group tests to test_volume_rpcapi https://review.openstack.org/202496 | 07:19 |
| *** bluex-pl has joined #openstack-cinder | 07:32 | |
| openstackgerrit | chenying proposed openstack/cinder: Fix order of arguments in assertEqual https://review.openstack.org/216502 | 07:32 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/215440 | 07:34 |
| *** ociuhandu has quit IRC | 07:35 | |
| *** ndipanov has quit IRC | 07:38 | |
| *** skylerberg has quit IRC | 07:39 | |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 07:40 |
| *** salv-orlando has joined #openstack-cinder | 07:47 | |
| *** lpetrut has joined #openstack-cinder | 07:51 | |
| *** IanGovett has joined #openstack-cinder | 07:52 | |
| *** jordanP has joined #openstack-cinder | 07:52 | |
| *** bkopilov has quit IRC | 07:54 | |
| openstackgerrit | wanghao proposed openstack/python-cinderclient: Add to_dict method to Resource class https://review.openstack.org/207340 | 07:55 |
| openstackgerrit | Takeaki Matsumoto proposed openstack/cinder: Fix url in API response to get original https://review.openstack.org/216591 | 07:57 |
| *** Zhang has quit IRC | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-cinder | 08:01 | |
| *** asd112z has joined #openstack-cinder | 08:03 | |
| *** asd112z has quit IRC | 08:08 | |
| *** jistr has joined #openstack-cinder | 08:12 | |
| *** Zhang has joined #openstack-cinder | 08:14 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 08:16 |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: Cleanup for cinder tests with CGSnapshot https://review.openstack.org/207866 | 08:16 |
| openstackgerrit | Eli Qiao proposed openstack/cinder: Handler KeyManager exception when delete an volume https://review.openstack.org/209828 | 08:16 |
| *** lpetrut has quit IRC | 08:17 | |
| *** e0ne has joined #openstack-cinder | 08:17 | |
| *** markus_z has joined #openstack-cinder | 08:21 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 08:23 |
| openstackgerrit | Stefan Amann proposed openstack/os-brick: Handle FC LUN IDs greater 255 correctly on s390x architectures https://review.openstack.org/216411 | 08:34 |
| *** sgotliv has joined #openstack-cinder | 08:36 | |
| *** sgotliv has quit IRC | 08:37 | |
| *** sgotliv has joined #openstack-cinder | 08:37 | |
| openstackgerrit | Danny Wilson proposed openstack/cinder: Replication V2 for Pure Storage https://review.openstack.org/214825 | 08:41 |
| *** haomaiwang has quit IRC | 08:41 | |
| *** haomaiwang has joined #openstack-cinder | 08:42 | |
| *** dannywilson has quit IRC | 08:46 | |
| *** yrabl has quit IRC | 08:48 | |
| *** yrabl has joined #openstack-cinder | 08:48 | |
| *** ociuhandu has joined #openstack-cinder | 09:04 | |
| *** haomaiwang has quit IRC | 09:09 | |
| *** haomaiwang has joined #openstack-cinder | 09:10 | |
| *** yuriy_n17 has joined #openstack-cinder | 09:19 | |
| yuriy_n17 | smcginnis: Hi! Please review https://review.openstack.org/#/c/215666/ Thanks for advance. | 09:20 |
| yuriy_n17 | jgriffith: Hi! Please review https://review.openstack.org/#/c/215666/ Thanks for advance. | 09:21 |
| *** BharatK has quit IRC | 09:27 | |
| *** lpetrut has joined #openstack-cinder | 09:28 | |
| *** Zhang has quit IRC | 09:30 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds the migration progress support for migration https://review.openstack.org/204953 | 09:31 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds migration abortion https://review.openstack.org/207754 | 09:31 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 09:31 |
| *** jordanP has quit IRC | 09:36 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds the migration progress support for migration https://review.openstack.org/204953 | 09:43 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds migration abortion https://review.openstack.org/207754 | 09:43 |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Move CloneableVD to common functions https://review.openstack.org/203593 | 09:47 |
| *** dims__ has joined #openstack-cinder | 09:48 | |
| *** dims__ has quit IRC | 09:48 | |
| *** dims__ has joined #openstack-cinder | 09:49 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: WIP: Testing to investigate test issue https://review.openstack.org/216643 | 09:49 |
| *** aix has joined #openstack-cinder | 09:50 | |
| *** ndipanov has joined #openstack-cinder | 09:51 | |
| *** lixiaoy1 has quit IRC | 09:52 | |
| *** aix has quit IRC | 09:52 | |
| *** aix has joined #openstack-cinder | 09:53 | |
| *** davechen has left #openstack-cinder | 09:55 | |
| *** BharatK has joined #openstack-cinder | 09:56 | |
| *** igajsin_ has quit IRC | 10:03 | |
| *** Zhang has joined #openstack-cinder | 10:05 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: WIP: Testing to investigate test issue https://review.openstack.org/216643 | 10:05 |
| *** haomaiwang has quit IRC | 10:09 | |
| *** haomaiwa_ has joined #openstack-cinder | 10:10 | |
| *** yrabl has quit IRC | 10:10 | |
| *** baojg has quit IRC | 10:11 | |
| *** Zhang has quit IRC | 10:11 | |
| *** Zhang has joined #openstack-cinder | 10:13 | |
| *** lpetrut has quit IRC | 10:13 | |
| *** Zhang has quit IRC | 10:15 | |
| *** lpetrut has joined #openstack-cinder | 10:17 | |
| *** btran has quit IRC | 10:18 | |
| *** salv-orlando has quit IRC | 10:20 | |
| *** Zhang has joined #openstack-cinder | 10:26 | |
| *** Zhang has quit IRC | 10:27 | |
| openstackgerrit | Marc Koderer proposed openstack/cinder: Move CloneableVD to common functions https://review.openstack.org/203593 | 10:32 |
| *** shyama has joined #openstack-cinder | 10:37 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add atomic conditional updates to objects https://review.openstack.org/205834 | 10:38 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: WIP: Testing to investigate test issue https://review.openstack.org/216643 | 10:38 |
| *** Zhang has joined #openstack-cinder | 10:38 | |
| *** Zhang has quit IRC | 10:39 | |
| *** Zhang has joined #openstack-cinder | 10:42 | |
| *** salv-orlando has joined #openstack-cinder | 10:43 | |
| *** yrabl has joined #openstack-cinder | 10:54 | |
| *** Zhang has quit IRC | 10:56 | |
| *** smoriya has quit IRC | 11:00 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add instructions on how to deploy API under Apache https://review.openstack.org/207020 | 11:00 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add Cinder API wsgi application https://review.openstack.org/200551 | 11:00 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Parameter osapi_max_limit is always used by default https://review.openstack.org/216661 | 11:01 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix backup list all_tenants=0 filtering for admin https://review.openstack.org/214086 | 11:01 |
| *** ganso_ has joined #openstack-cinder | 11:03 | |
| *** asd112z has joined #openstack-cinder | 11:03 | |
| *** Zhang has joined #openstack-cinder | 11:03 | |
| yuriy_n17 | geguileo: Hi! Please review https://review.openstack.org/#/c/215666/ | 11:04 |
| *** asd112z has quit IRC | 11:08 | |
| *** haomaiwa_ has quit IRC | 11:09 | |
| *** Zhang has quit IRC | 11:09 | |
| *** Zhang has joined #openstack-cinder | 11:10 | |
| *** haomaiwa_ has joined #openstack-cinder | 11:10 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Remove v1 server support https://review.openstack.org/145703 | 11:18 |
| openstackgerrit | Jordan Pittier proposed openstack/cinder: Rework Scality SOFS driver to use RemoteFS class https://review.openstack.org/200610 | 11:24 |
| *** ccesario has quit IRC | 11:24 | |
| *** timcl has joined #openstack-cinder | 11:24 | |
| *** bswartz has quit IRC | 11:29 | |
| *** tpsilva has joined #openstack-cinder | 11:34 | |
| *** ccesario has joined #openstack-cinder | 11:36 | |
| *** Zhang has quit IRC | 11:37 | |
| *** Zhang has joined #openstack-cinder | 11:39 | |
| openstackgerrit | Brent Roskos proposed openstack/cinder: adds user_id to check_is_admin https://review.openstack.org/213501 | 11:46 |
| *** erlon has joined #openstack-cinder | 11:48 | |
| *** marcusvrn_ has joined #openstack-cinder | 11:53 | |
| *** zhenguo has quit IRC | 11:55 | |
| openstackgerrit | Takeaki Matsumoto proposed openstack/cinder: Fix url in API response to get original https://review.openstack.org/216591 | 11:55 |
| nikeshm_ | hi | 11:57 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Add LC_ALL=C to lvcreate, lvextend and pvresize https://review.openstack.org/216675 | 12:02 |
| *** nikeshm has joined #openstack-cinder | 12:04 | |
| *** nikeshm_ has quit IRC | 12:06 | |
| *** haomaiwa_ has quit IRC | 12:09 | |
| *** haomaiwang has joined #openstack-cinder | 12:10 | |
| *** deepakcs has quit IRC | 12:14 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add functional tests for python-cinderclient https://review.openstack.org/215666 | 12:15 |
| *** Zhang has quit IRC | 12:16 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from delete methods https://review.openstack.org/205835 | 12:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add atomic conditional updates to objects https://review.openstack.org/205834 | 12:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: WIP: Remove more API races https://review.openstack.org/216378 | 12:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Improve metadata update operations https://review.openstack.org/216376 | 12:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from attach and detach methods https://review.openstack.org/216377 | 12:18 |
| *** Zhang has joined #openstack-cinder | 12:19 | |
| *** annegentle has joined #openstack-cinder | 12:28 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from delete methods https://review.openstack.org/205835 | 12:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add atomic conditional updates to objects https://review.openstack.org/205834 | 12:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: WIP: Remove more API races https://review.openstack.org/216378 | 12:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Improve metadata update operations https://review.openstack.org/216376 | 12:31 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from attach and detach methods https://review.openstack.org/216377 | 12:31 |
| *** bill_az has joined #openstack-cinder | 12:31 | |
| *** jordanP has joined #openstack-cinder | 12:32 | |
| *** kjnelson has joined #openstack-cinder | 12:34 | |
| *** akerr has joined #openstack-cinder | 12:35 | |
| *** edmondsw has joined #openstack-cinder | 12:36 | |
| *** takedakn has joined #openstack-cinder | 12:36 | |
| *** bswartz has joined #openstack-cinder | 12:37 | |
| *** porrua has joined #openstack-cinder | 12:40 | |
| *** cbader has quit IRC | 12:41 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Remove v1 server support https://review.openstack.org/145703 | 12:42 |
| *** gouthamr has joined #openstack-cinder | 12:45 | |
| *** bluex-pl has quit IRC | 12:46 | |
| *** bluex-pl has joined #openstack-cinder | 12:47 | |
| *** annegentle has quit IRC | 12:50 | |
| *** simondodsley has joined #openstack-cinder | 12:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/215440 | 12:50 |
| *** merooney has joined #openstack-cinder | 12:51 | |
| *** dustins has joined #openstack-cinder | 12:52 | |
| *** jungleboyj has quit IRC | 12:54 | |
| *** martyturner has joined #openstack-cinder | 12:54 | |
| smcginnis | yuriy_n17: Any idea why I get this when trying to run the functional tests? | 13:00 |
| smcginnis | yuriy_n17: http://paste.openstack.org/show/427031/ | 13:00 |
| smcginnis | yuriy_n17: brb | 13:00 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 13:06 |
| *** Zhang has quit IRC | 13:06 | |
| *** xyang1 has joined #openstack-cinder | 13:06 | |
| *** haomaiwang has quit IRC | 13:09 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add hypermetro support for Huawei driver https://review.openstack.org/202023 | 13:10 |
| yuriy_n17 | smcginnis: Hi! Check your credentials: user in functional_creds.conf and in your DevStack | 13:10 |
| *** haomaiwang has joined #openstack-cinder | 13:10 | |
| *** annegentle has joined #openstack-cinder | 13:11 | |
| *** annegentle has quit IRC | 13:12 | |
| smcginnis | yuriy_n17: Yeah, that looks good. I don't think I'm getting that far though. Think I'll spin up a fresh machine and see how that goes. | 13:12 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Fix way of get LUN id in Huawei driver https://review.openstack.org/209749 | 13:12 |
| *** raildo-afk is now known as raildo | 13:13 | |
| *** annegentle has joined #openstack-cinder | 13:14 | |
| smcginnis | yuriy_n17: Yeah, looking better on a new machine. At least so far. ;) | 13:15 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Encode username and password in Huawei driver https://review.openstack.org/216085 | 13:15 |
| *** zhenguo has joined #openstack-cinder | 13:16 | |
| *** Yogi11 has joined #openstack-cinder | 13:17 | |
| yuriy_n17 | smcginnis: Or remove .tox to create new virtual env | 13:17 |
| smcginnis | yuriy_n17: That hadn't helped either. I think I need to reload that machine. Or do some serious cleanup. | 13:17 |
| yuriy_n17 | smcginnis: Try to remove .tox | 13:18 |
| smcginnis | Yes, that hadn't helped either. | 13:18 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/215440 | 13:18 |
| *** martyturner has quit IRC | 13:19 | |
| *** kjnelson has quit IRC | 13:19 | |
| *** Zhang has joined #openstack-cinder | 13:19 | |
| yuriy_n17 | smcginnis: Just a minute. Let me think about it;) | 13:19 |
| *** breitz has joined #openstack-cinder | 13:19 | |
| *** thangp has joined #openstack-cinder | 13:20 | |
| smcginnis | yuriy_n17: The new machine could run it. I got a lot of errors though. Restacking. Don't worry about it, I'm sure it's just my environment. | 13:20 |
| *** changbl has quit IRC | 13:21 | |
| *** Zhang has quit IRC | 13:22 | |
| *** kvidvans has quit IRC | 13:22 | |
| yuriy_n17 | smcginnis: According to your traceback please delete virtualenv /home/smcginnis/Documents/dev/repos/OpenStack/python-cinderclient/.tox/functional | 13:23 |
| *** links has quit IRC | 13:23 | |
| *** mtanino has joined #openstack-cinder | 13:23 | |
| *** dims__ has quit IRC | 13:24 | |
| smcginnis | yuriy_n17: I had deleted the entire .tox directory, so that wasn't it. | 13:24 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds the migration progress support for migration https://review.openstack.org/204953 | 13:25 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds migration abortion https://review.openstack.org/207754 | 13:25 |
| *** dims__ has joined #openstack-cinder | 13:25 | |
| mtanino | geguileo: hi | 13:26 |
| geguileo | mtanino: Hi | 13:26 |
| mtanino | geguileo: Thank you for your review for https://review.openstack.org/#/c/201243/24/cinder/volume/driver.py | 13:26 |
| *** bill_az has quit IRC | 13:26 | |
| mtanino | may I have a question? | 13:26 |
| geguileo | mtanino: Sure, fire away | 13:27 |
| *** jwcroppe has quit IRC | 13:27 | |
| mtanino | geguileo: so, about the separator, | 13:27 |
| *** IanGovett1 has joined #openstack-cinder | 13:27 | |
| mtanino | geguileo: Where did you find the separator "." and not "_"? | 13:28 |
| geguileo | mtanino: If I wrote . it was my mistake, I wanted to write : | 13:29 |
| mtanino | geguileo: Ah, you mean you recomment to use "." instead of "_", right? | 13:29 |
| geguileo | mtanino: No, that the specs say it should be : | 13:30 |
| *** jwcroppe has joined #openstack-cinder | 13:30 | |
| *** diogogmt has joined #openstack-cinder | 13:30 | |
| *** IanGovett has quit IRC | 13:30 | |
| *** jgregor has joined #openstack-cinder | 13:30 | |
| *** kvidvans has joined #openstack-cinder | 13:30 | |
| mtanino | ":" | 13:31 |
| *** IanGovett has joined #openstack-cinder | 13:31 | |
| *** Zhang has joined #openstack-cinder | 13:31 | |
| geguileo | mtanino: https://review.openstack.org/#/c/127646/12/specs/kilo/get-vol-type-extra-specs.rst,cm | 13:32 |
| geguileo | mtanino: That's the spec in the BP you refer in your patch | 13:32 |
| *** IanGovett1 has quit IRC | 13:32 | |
| mtanino | geguileo: This seems old spec. | 13:33 |
| geguileo | mtanino: Yeah, I'm just looking at the newer L one | 13:33 |
| *** IanGovett1 has joined #openstack-cinder | 13:33 | |
| mtanino | geguileo: Ah, so the original spec was once agreed to use ":" as a separater. | 13:34 |
| *** jwcroppe has quit IRC | 13:34 | |
| geguileo | mtanino: Apparently, but I'm reading L spec to see what it says | 13:34 |
| *** breitz has quit IRC | 13:34 | |
| mtanino | geguileo: L spec is here. https://review.openstack.org/#/c/183947/5/specs/liberty/get-vol-type-extra-specs.rst | 13:35 |
| geguileo | mtanino: Yeah, reading it now | 13:35 |
| *** kjnelson has joined #openstack-cinder | 13:35 | |
| geguileo | mtanino: But according to that spec, your implementation is wrong | 13:36 |
| *** dims has joined #openstack-cinder | 13:36 | |
| geguileo | mtanino: You implemented the K version | 13:36 |
| *** kjnelson is now known as diablo_rojo | 13:36 | |
| geguileo | mtanino: With the exception of the separator | 13:36 |
| *** dims__ has quit IRC | 13:36 | |
| *** IanGovett has quit IRC | 13:36 | |
| mtanino | geguileo: just a moment. | 13:36 |
| -mtanino- Mitsuhiro TaninoAug 17 5:58 PM | 13:37 | |
| -mtanino- Patch Set 18: | 13:37 | |
| -mtanino- Based on the discussion at IRC, we changed policy of properties name. | 13:37 | |
| -mtanino- Cinder well defined key should be "bare"(without prefix). | 13:37 | |
| -mtanino- Vendor unique properties should have vendor_name as a prefix(like Hitachi_minIOPS). | 13:37 | |
| -mtanino- I will post a follow up patch for cinder-spec. | 13:37 | |
| *** mriedem has joined #openstack-cinder | 13:37 | |
| *** Zhang has quit IRC | 13:38 | |
| mtanino | geguileo: we discussed about prefix with Mike and John on the IRC, and we changed the prefix policy. | 13:38 |
| mtanino | as above. | 13:38 |
| mtanino | geguileo: now I'm proposing folloup patch for the current spec too. | 13:39 |
| geguileo | mtanino: Or at least add it to the BP as a comment | 13:39 |
| *** dims_ has joined #openstack-cinder | 13:39 | |
| mtanino | geguileo: agree. | 13:39 |
| mtanino | geguileo: proposed spec change. https://review.openstack.org/#/c/216394/ | 13:40 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 13:40 |
| *** dims__ has joined #openstack-cinder | 13:41 | |
| mtanino | geguileo: I think we did not decide which separater is best at that time. so ":" might be reasonable. What do you think? | 13:42 |
| *** dims has quit IRC | 13:42 | |
| geguileo | mtanino: You should look at vendor names and see if anyone is using _ in the name | 13:42 |
| geguileo | mtanino: If they are we should change it to : | 13:42 |
| mtanino | geguileo: I think vendor name will be decide freely, so that name might include. | 13:44 |
| geguileo | mtanino: But you could set some rules now | 13:44 |
| *** mriedem is now known as mriedem_class | 13:44 | |
| geguileo | mtanino: Like it must be lowercase and no spaces, or whatever | 13:44 |
| *** bapalm has quit IRC | 13:44 | |
| mtanino | geguileo: ah, I see. by checking it. | 13:45 |
| *** Zhang has joined #openstack-cinder | 13:45 | |
| geguileo | mtanino: Otherwise driver could just return the same they are returning in the dictionary | 13:45 |
| *** dims_ has quit IRC | 13:45 | |
| geguileo | mtanino: And you would have things like "Open Source_specifickey" | 13:45 |
| mtanino | geguileo: I agree. | 13:45 |
| geguileo | mtanino: I would let them pass whatever they want except a : and use the : as the separator | 13:46 |
| geguileo | mtanino: But we should agree to that | 13:46 |
| geguileo | mtanino: You should add it to tomorrow's agenda | 13:47 |
| *** jwcroppe has joined #openstack-cinder | 13:47 | |
| mtanino | geguileo: I understand. | 13:47 |
| mtanino | geguileo: After agreeing the separator, I will add check for "vendor name" for prefix. | 13:48 |
| *** cbader has joined #openstack-cinder | 13:49 | |
| guitarzan | do you have another way of knowing what the vendor name is? | 13:49 |
| geguileo | mtanino: Ok | 13:49 |
| geguileo | guitarzan: Yes we do | 13:49 |
| guitarzan | then you don't need to reject any characters | 13:49 |
| geguileo | guitarzan: But the vendor name we can get from the dictionary | 13:49 |
| guitarzan | and in the same case, it doesn't matter what the separator is either | 13:50 |
| geguileo | guitarzan: But only if we use the one they send in the properties | 13:50 |
| guitarzan | I thought that was what you were saying in the first place | 13:50 |
| geguileo | guitarzan: That's why I asked if there were any restrictions | 13:51 |
| geguileo | guitarzan: He is not using that one right now | 13:51 |
| geguileo | guitarzan: And I agree we should use that one | 13:51 |
| geguileo | guitarzan: But then we have things like 'Open Source' as the vendor name | 13:51 |
| *** bapalm_ is now known as bapalm | 13:51 | |
| *** breitz has joined #openstack-cinder | 13:52 | |
| mtanino | geguileo: like LVM driver | 13:52 |
| geguileo | guitarzan: And it doesn't look very nice to have a property key called 'Open Source_fireproof' | 13:52 |
| guitarzan | yeah, this is kind of ugly :) | 13:52 |
| geguileo | mtanino: Yep | 13:52 |
| geguileo | guitarzan: But I think we should use it anyway | 13:52 |
| *** cvstealth has joined #openstack-cinder | 13:52 | |
| guitarzan | probably so | 13:52 |
| geguileo | guitarzan: To make it easier to know which part is the vendor prefix | 13:52 |
| geguileo | guitarzan: Or we could just do the classic "to lowercase and replace spaces with _" | 13:53 |
| mtanino | geguileo: I hope only LVM driver is ugly one... | 13:53 |
| geguileo | mtanino: No idea, you'll have to fish in the code for those vendor names ;-) | 13:53 |
| mtanino | yes... | 13:54 |
| *** jungleboyj has joined #openstack-cinder | 13:56 | |
| mtanino | geguileo: In this classic case "to lowercase and replace spaces with _", separater for prefix should be ":" instead of "_". | 13:56 |
| geguileo | mtanino: Yes | 13:56 |
| geguileo | mtanino: But I think you should wait for an agreement on the meeting | 13:57 |
| mtanino | geguileo: I got it. | 13:57 |
| mtanino | yes. | 13:57 |
| geguileo | mtanino: Unless you don't mind working and then redoing it | 13:57 |
| mtanino | geguileo: Unfortunately, my patch is in merge conflict... | 13:57 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 13:57 |
| mtanino | I need to rebase it anyway. | 13:57 |
| *** daneyon_ has quit IRC | 13:58 | |
| *** jaypipes has joined #openstack-cinder | 13:58 | |
| *** daneyon has joined #openstack-cinder | 13:58 | |
| *** martyturner has joined #openstack-cinder | 14:01 | |
| *** martyturner has left #openstack-cinder | 14:01 | |
| *** diogogmt has quit IRC | 14:04 | |
| *** pots has joined #openstack-cinder | 14:08 | |
| *** haomaiwang has quit IRC | 14:09 | |
| *** haomaiwang has joined #openstack-cinder | 14:10 | |
| *** Zhang has quit IRC | 14:12 | |
| *** mtanino has quit IRC | 14:14 | |
| *** raildo is now known as raildo-afk | 14:16 | |
| *** raildo-afk is now known as raildo | 14:17 | |
| *** annasort_ has joined #openstack-cinder | 14:21 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Support initialization state in Backup Manager https://review.openstack.org/216728 | 14:22 |
| smcginnis | Note to self. If you test the WSGI under apache patch, make sure you shut down apache before trying to restack. :) | 14:23 |
| *** annasort has quit IRC | 14:24 | |
| *** annasort_ is now known as annasort | 14:24 | |
| *** nkrinner has quit IRC | 14:26 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Use cinder internal tenant to create the destination volume https://review.openstack.org/210237 | 14:26 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Parameter osapi_max_limit is always used by default https://review.openstack.org/216661 | 14:28 |
| DuncanT | smcginnis: Can that be wired into unstack? | 14:29 |
| *** annasort has quit IRC | 14:30 | |
| *** ankit_ag has quit IRC | 14:31 | |
| smcginnis | DuncanT: Ooh, not a bad idea. | 14:31 |
| *** tsekiyama has joined #openstack-cinder | 14:32 | |
| *** ronis_ has joined #openstack-cinder | 14:32 | |
| dulek | jungleboyj: Thanks for your recheck, but I don't think it will help. The errors in logs are clearly related to the patch - new volume.filters isn't copied after update. | 14:32 |
| *** asd112z has joined #openstack-cinder | 14:32 | |
| *** asd112z has quit IRC | 14:33 | |
| *** asd112z has joined #openstack-cinder | 14:33 | |
| dulek | jungleboyj: I wonder what a correct approach would be here. I can simply leave old entries in volume.filters, but that seems messy. | 14:33 |
| jungleboyj | dulek: Doh, sorry. Glanced at it too quickly. | 14:33 |
| e0ne | smcginnis: did you test it with patch to devstack? | 14:34 |
| *** asd112z has quit IRC | 14:34 | |
| *** asd112z has joined #openstack-cinder | 14:34 | |
| *** ronis__ has quit IRC | 14:35 | |
| smcginnis | e0ne: No, just manually walking through the config instructions. | 14:37 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: TemporaryImages to inspect image before conversion https://review.openstack.org/213346 | 14:37 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Efficient image transfer for Glance cinder store https://review.openstack.org/201754 | 14:37 |
| smcginnis | Might not have been my issue though. Still having trouble. Might need to spend a little time today cleaning up things. | 14:38 |
| e0ne | smcginnis: patch for devstack restarts apache | 14:38 |
| *** edtubill has joined #openstack-cinder | 14:38 | |
| smcginnis | e0ne: Remembered that now. Looks like my issue is I need to remove some of the custom configuration I did for cinder in apache. Forgot keystone uses it. | 14:39 |
| *** bluex-pl has quit IRC | 14:39 | |
| e0ne | smcginnis: welcome to the manual deployment world:) | 14:39 |
| smcginnis | e0ne: :) | 14:40 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: TemporaryImages to inspect image before conversion https://review.openstack.org/213346 | 14:40 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Efficient image transfer for Glance cinder store https://review.openstack.org/201754 | 14:40 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: WIP:Add rootwrap daemon mode support https://review.openstack.org/149677 | 14:41 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: WIP: Switch to use execute from cinder utils https://review.openstack.org/214595 | 14:41 |
| *** annegentle has quit IRC | 14:41 | |
| *** annegentle has joined #openstack-cinder | 14:41 | |
| *** links has joined #openstack-cinder | 14:43 | |
| openstackgerrit | Diem Tran proposed openstack/cinder: Local cache feature of Oracle ZFSSA drivers https://review.openstack.org/212155 | 14:46 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Continue conversion to volume object https://review.openstack.org/201404 | 14:52 |
| *** diogogmt has joined #openstack-cinder | 14:53 | |
| *** Yogi11 has quit IRC | 14:57 | |
| kvidvans | smcginnis: whenever you get time, https://review.openstack.org/#/c/210608/12 has been updated to address your comments. I appreciate your feedback. Thank you. | 14:58 |
| *** timcl1 has joined #openstack-cinder | 14:58 | |
| *** timcl has quit IRC | 15:00 | |
| *** Yogi1 has joined #openstack-cinder | 15:02 | |
| *** alexschm has quit IRC | 15:02 | |
| *** mtanino has joined #openstack-cinder | 15:02 | |
| *** Yogi11 has joined #openstack-cinder | 15:03 | |
| *** hemnafk is now known as hemna | 15:03 | |
| *** rushil has joined #openstack-cinder | 15:04 | |
| smcginnis | kvidvans: Sure, I'll take a look. | 15:04 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Register RPC and object versions https://review.openstack.org/209701 | 15:07 |
| *** Yogi1 has quit IRC | 15:07 | |
| *** dustins has quit IRC | 15:08 | |
| *** haomaiwang has quit IRC | 15:09 | |
| *** annasort has joined #openstack-cinder | 15:09 | |
| yuriy_n17 | jgriffith: Hi! Please review https://review.openstack.org/#/c/215666/ | 15:09 |
| *** p0rtal has joined #openstack-cinder | 15:09 | |
| *** p0rtal has quit IRC | 15:09 | |
| *** p0rtal has joined #openstack-cinder | 15:10 | |
| *** haomaiwang has joined #openstack-cinder | 15:10 | |
| *** timcl1 has quit IRC | 15:12 | |
| *** annasort_ has joined #openstack-cinder | 15:13 | |
| jordanP | smcginnis, thanks for your encouraging comment :) | 15:13 |
| smcginnis | jordanP: np | 15:13 |
| *** diablo_rojo has left #openstack-cinder | 15:13 | |
| *** annasort has quit IRC | 15:14 | |
| *** annasort_ is now known as annasort | 15:14 | |
| e0ne | jgriffith: morning, John. are you around? i've got a question about solidfire 3rd party ci | 15:17 |
| e0ne | maybe, it's a common question about all 3rd party ci, not sure yet | 15:18 |
| hemna | e0ne, sup | 15:21 |
| e0ne | hemna: morning | 15:21 |
| *** dannywilson has joined #openstack-cinder | 15:22 | |
| e0ne | hemna: i can't say anything about HP CI because it didn't voted yet :) | 15:22 |
| hemna | heh ok | 15:22 |
| e0ne | hemna: does your ci follow Depends-On flag? | 15:23 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Parameter osapi_max_limit is always used by default https://review.openstack.org/216661 | 15:23 |
| asselin | e0ne, it does for some projects | 15:23 |
| smcginnis | e0ne: Ours does not. | 15:23 |
| smcginnis | That's actually been one of my concerns. | 15:23 |
| e0ne | i've got a patch (https://review.openstack.org/#/c/145703/ - removing api v1) | 15:23 |
| asselin | e0ne, which patch? | 15:23 |
| *** ronis__ has joined #openstack-cinder | 15:24 | |
| e0ne | it depends on tempest, destack and grenade (rally dependency will be added soon) | 15:24 |
| asselin | e0ne, no, ours won't pull those in. | 15:24 |
| e0ne | if 3rd party ci is voting, such patches won't be merged:( | 15:24 |
| jlk | Can anybody help me figure out what's next with https://review.openstack.org/#/c/215860/ ? | 15:25 |
| smcginnis | It would be nice if devstack can detect that and do the right thing. | 15:25 |
| asselin | back later | 15:25 |
| *** Zhang has joined #openstack-cinder | 15:25 | |
| e0ne | smcginnis: it can, but not in gates | 15:25 |
| smcginnis | e0ne: Ah, OK. | 15:26 |
| *** sgotliv has quit IRC | 15:26 | |
| *** p0rtal has quit IRC | 15:26 | |
| e0ne | can anyone help me with link to 3rd party CIs requirements page? can't find it | 15:27 |
| *** ronis_ has quit IRC | 15:27 | |
| smcginnis | jlk: Commented on the patch. Almost there. | 15:27 |
| hemna | xyang1, ping | 15:27 |
| smcginnis | e0ne: This one? http://docs.openstack.org/infra/system-config/third_party.html | 15:28 |
| *** markus_z has quit IRC | 15:28 | |
| hemna | e0ne, good question. I think it depends, the Depends-On flag needs to be supported by the upsteam codebase | 15:28 |
| hemna | which I believe it is, but isn't perfect | 15:28 |
| jgriffith | e0ne: hey... saw your ping... what's up? | 15:29 |
| e0ne | smcginnis: may be. do we have the same for cinder? | 15:29 |
| smcginnis | e0ne: We just like there: https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver#Third_Party_CI_Requirement_Policy | 15:29 |
| e0ne | hemna: what do you mean? didn't get your point | 15:29 |
| *** shausy has quit IRC | 15:30 | |
| e0ne | smcginnis: great! thanks | 15:30 |
| smcginnis | e0ne: I guess a little more detail in our link. | 15:30 |
| hemna | e0ne, I've seen cases where the Depends-On points to a dependant library patch (a la os-brick), and that libary patch gets pulled down, but isn't the basis for the install, because it's already installed via pip. | 15:30 |
| hemna | so it's not perfect | 15:30 |
| *** jdurgin1 has joined #openstack-cinder | 15:30 | |
| jgriffith | e0ne: smcginnis cbader We had a rather *nasty* discussion in here one day WRT documenting requirements... | 15:30 |
| *** thangp has quit IRC | 15:31 | |
| jgriffith | e0ne: smcginnis it ended with hemna and asselin saying something like "griffith can't be a cowboy anymore it's not the wild west" :) | 15:31 |
| hemna | yeeehaw! | 15:31 |
| jgriffith | hemna: LOL!! | 15:31 |
| smcginnis | jgriffith: Guess you have to sell your horses. | 15:31 |
| jgriffith | smcginnis: no way :) | 15:31 |
| e0ne | jgriffith: actually, i've got 2 questions already | 15:32 |
| jgriffith | e0ne: ok, lay them on me | 15:32 |
| smcginnis | jgriffith: Had a nice trail ride in Estes after the midcycle by the way. What a great area to have horses. Anyway... | 15:32 |
| e0ne | jgriffith: looks like your ci is voting and dosn't support Depends-On flag. am i right? | 15:32 |
| jgriffith | smcginnis: Nice!!! | 15:32 |
| jgriffith | smcginnis: you were in my backyard :) | 15:32 |
| *** ronis__ has quit IRC | 15:32 | |
| *** Zhang has quit IRC | 15:32 | |
| xyang1 | hemna: Hi | 15:33 |
| jgriffith | e0ne: correct | 15:33 |
| smcginnis | jgriffith: Next time we'll book the trail ride at your place. ;) | 15:33 |
| jgriffith | e0ne: you're referring to the depends-on image-xyz | 15:33 |
| e0ne | jgriffith: so... it will post -1 for any patch with depend-on :( | 15:33 |
| openstackgerrit | Jesse Keating proposed openstack/cinder: Fix nimble storage volume stats reporting https://review.openstack.org/215860 | 15:33 |
| hemna | xyang1, hey | 15:33 |
| jgriffith | e0ne: actually, NO I don't do voting | 15:33 |
| jlk | smcginnis: thanks, updated. | 15:33 |
| jgriffith | and it seems we're talking about something different | 15:34 |
| smcginnis | jlk: Cool, I'll take another look in a bit. | 15:34 |
| hemna | xyang1, so in doing the review for our LeftHand driver's support for CG, I noticed that the code was fetching things from the DB. :( | 15:34 |
| hemna | xyang1, and I'm wondering the reason for that and if other CG supported drivers are doing the same thing. | 15:34 |
| hemna | xyang1, I -1'd the review for doing that. | 15:34 |
| e0ne | jgriffith: oh... so i was wrong. it's good for now:) | 15:34 |
| jgriffith | e0ne: Ceph is the only job other than *regular* CI jobs that's voting | 15:34 |
| jgriffith | e0ne: for now? | 15:34 |
| xyang1 | hemna: Correct, patrickeast and I just talk about changing those apis | 15:34 |
| hemna | xyang1, ok, because I think that's bad mmmmkay | 15:35 |
| e0ne | jgriffith: i mean it's good for me that you ci is not voting now | 15:35 |
| jgriffith | e0ne: just curious what are you submitting? | 15:35 |
| hemna | xyang1, specifically: https://review.openstack.org/#/c/214386/6/cinder/volume/drivers/san/hp/hp_lefthand_rest_proxy.py | 15:35 |
| e0ne | jgriffith: https://review.openstack.org/#/c/145703/ | 15:35 |
| *** zhenguo has quit IRC | 15:36 | |
| hemna | xyang1, it does self.db. queries as well as objects.SnapshotList() queries (both of which go to the DB) | 15:36 |
| *** Zhang has joined #openstack-cinder | 15:36 | |
| jgriffith | e0ne: that shouldn't fail my CI that I know of | 15:36 |
| *** rushil has quit IRC | 15:36 | |
| jgriffith | e0ne: but I won't pick it up til it passes Jenkins | 15:36 |
| e0ne | jgriffith: i've found that at least yours ci doesn't support Depends-On flad. and now i want to understand is it required for 3rd party ci or not | 15:36 |
| xyang1 | hemna: When I designed APIs for the first version, I wa trying to make them look simple, also at that point there were two different point of views | 15:36 |
| *** rushil has joined #openstack-cinder | 15:37 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR update driver to store stats https://review.openstack.org/216771 | 15:37 |
| jgriffith | e0ne: well, if you want to teach me about that depends on flag, I can surely fix it up :) | 15:37 |
| xyang1 | hemna: The 2nd set of APIs have all info so driver does not have to fetch them | 15:37 |
| e0ne | jgriffith: just deleted lofs. it doesn't pull my devstack patch | 15:37 |
| *** p0rtal has joined #openstack-cinder | 15:37 | |
| hemna | xyang1, ok so where are we with changing the APIs then? | 15:37 |
| xyang1 | hemna: It is going to be M | 15:37 |
| hemna | xyang1, is that out for L? or do we have a patch up already that fixes it? | 15:37 |
| hemna | :( | 15:37 |
| xyang1 | hemna: Too late for now | 15:38 |
| hemna | ouch | 15:38 |
| jgriffith | e0ne: Where's the flag you're talking about? I'm happy to take a look, I'm not sure I know about it | 15:39 |
| *** thangp has joined #openstack-cinder | 15:39 | |
| e0ne | jgriffith: it's everything i know http://docs.openstack.org/infra/manual/developers.html#cross-repository-dependencies | 15:39 |
| jgriffith | e0ne: OHHHHH!!!! | 15:39 |
| jgriffith | e0ne: You mean pulling in dependencies for a patch? | 15:40 |
| e0ne | jgriffith: 'Depends-On: gerrit-cachange-id' in a commit message. e.g. it will test your patch in cinder and devstck | 15:40 |
| e0ne | jgriffith: yes | 15:40 |
| jgriffith | e0ne: sorry, I had no idea that's what you meant | 15:40 |
| jgriffith | e0ne: hmm... so I thought gerrit just did that for us? | 15:40 |
| e0ne | jgriffith: not gerrit, but infra does | 15:40 |
| mtanino | jgriffith: hi, | 15:41 |
| jgriffith | e0ne: what I mean is when I do a "git review -d xyz" doesn't it pull the deps as well? | 15:41 |
| jgriffith | ahh... but i use git checkout.... hmmm | 15:41 |
| e0ne | jgriffith: no. these deps will be on the other project | 15:41 |
| jgriffith | e0ne: ohhh! | 15:42 |
| jgriffith | yeah, I don't support that :) | 15:42 |
| e0ne | jgriffith: it's cross-project dependencies | 15:42 |
| jgriffith | e0ne: and I didn't know infra supported that now either... :( | 15:42 |
| hemna | xyang1, ok, so I guess we don't have a choice at the moment. I'll have him put # TODO() comment blocks above each of the DB queries. | 15:42 |
| xyang1 | hemna: Sure | 15:43 |
| *** links has quit IRC | 15:44 | |
| hemna | jgriffith, fwiw, the upstream infra scripts only slightly support it. | 15:44 |
| hemna | some times it doesn't really work for libraries | 15:44 |
| *** diablo_rojo has joined #openstack-cinder | 15:44 | |
| *** timcl has joined #openstack-cinder | 15:44 | |
| mtanino | jgriffith: May I take a few minutes about this spec? https://review.openstack.org/#/c/216394/2/specs/liberty/get-vol-type-extra-specs.rst | 15:45 |
| e0ne | hemna, jgriffith: it doesn't work for new infra jobs too | 15:46 |
| hemna | e0ne, :( | 15:46 |
| jgriffith | hemna: e0ne in that case I'm not nearly as interested any longer ;) | 15:46 |
| *** thangp has quit IRC | 15:46 | |
| hemna | jgriffith, I think that stuff should get fixed though | 15:46 |
| *** timcl has quit IRC | 15:47 | |
| hemna | in which case you'd benefit when it does fwiw. | 15:47 |
| jgriffith | hemna: it's not a trivial problem, been discussed for years | 15:47 |
| *** merooney has quit IRC | 15:47 | |
| jgriffith | hemna: so what happens when change-a passes, but change-b fails? | 15:47 |
| jgriffith | hemna: neither merge right? | 15:47 |
| hemna | good question | 15:48 |
| e0ne | jgriffith: yes. only 2 patches could be merged | 15:48 |
| jgriffith | hemna: but if one runs in nova-dsvm and the other in cidner-dsvm and they give different results... then what? | 15:48 |
| smcginnis | jgriffith: I don't believe it's allowed to have both depend on the other. | 15:48 |
| smcginnis | jgriffith: It can only be one way. | 15:48 |
| hemna | I don't think you can have circular deps | 15:48 |
| e0ne | smcginnis, hemna: you're right | 15:49 |
| jgriffith | smcginnis: sure, but that's not a circular depend | 15:49 |
| jgriffith | smcginnis: hemna it has to run through both gates | 15:49 |
| smcginnis | SO if change-a depended on change-b and b failed neither would go in. | 15:49 |
| smcginnis | But if b depended on a, a would still go in but b wouldn't. | 15:49 |
| jgriffith | smcginnis: hemna I get what you're saying, I'm just pointing out there are some little tricks in there because of how we currently process changes | 15:49 |
| hemna | sure | 15:49 |
| smcginnis | True | 15:49 |
| jgriffith | smcginnis: sure, but there's not logic to handle that cross-communication/voting... and that's kind hard | 15:50 |
| *** Yogi12 has joined #openstack-cinder | 15:50 | |
| smcginnis | I do think it's good theirs at least a way to do cross-repo now. | 15:50 |
| jgriffith | smcginnis: I'm mixed to be honest | 15:50 |
| *** garthb has joined #openstack-cinder | 15:50 | |
| jgriffith | smcginnis: it's not really Continious integration any more :) | 15:50 |
| jgriffith | smcginnis: it's coordinated integration :) | 15:50 |
| smcginnis | :) | 15:50 |
| *** Yogi11 has quit IRC | 15:51 | |
| smcginnis | We can at least do things like have a python-cinderclient patch for new functionality not go in until the cinder side merges. | 15:51 |
| e0ne | jgriffith: :) | 15:51 |
| * jgriffith gets more coffee | 15:53 | |
| *** p0rtal_ has joined #openstack-cinder | 15:54 | |
| xyang1 | hemna: By the way, we tested your FC patch with VNX and it works | 15:54 |
| xyang1 | hemna: Any plan for similar change for iSCSI? | 15:55 |
| hemna | xyang1, yah I'd like to work on the iSCSI side next | 15:55 |
| xyang1 | hemna: Cool | 15:55 |
| hemna | xyang1, thanks for looking at it and testing it. | 15:55 |
| *** changbl has joined #openstack-cinder | 15:55 | |
| xyang1 | hemna: Np | 15:56 |
| hemna | xyang1, I'd like to consolidate some of the mpath code between iSCSI and FC | 15:56 |
| *** crose has joined #openstack-cinder | 15:56 | |
| hemna | this new code should be a bit faster | 15:56 |
| xyang1 | hemna: Sure, sounds good | 15:56 |
| *** p0rtal has quit IRC | 15:58 | |
| *** p0rtal_ has quit IRC | 15:59 | |
| *** jordanP has quit IRC | 16:00 | |
| *** leeantho has joined #openstack-cinder | 16:01 | |
| hemna | mtanino, ping | 16:01 |
| mtanino | hemna: pong | 16:01 |
| openstackgerrit | Michael Price proposed openstack/cinder: NetApp E-Series over-subscription support https://review.openstack.org/215801 | 16:02 |
| hemna | mtanino, https://review.openstack.org/#/c/201243/ is in merge conflict FYI | 16:02 |
| mtanino | hemna: ah, yes. I need to change RPC version of the patch. | 16:03 |
| *** takedakn has quit IRC | 16:03 | |
| *** _cjones_ has joined #openstack-cinder | 16:03 | |
| hemna | mtanino, ok just wanted to make sure you were aware of it. I'm doing my review rounds :P | 16:03 |
| mtanino | hemna: Also I had a comment for spec of vendor name prefix from geguileo and so I need to address it. | 16:04 |
| mtanino | hemna: sure. | 16:04 |
| *** Apoorva has joined #openstack-cinder | 16:04 | |
| dannywilson | hey everyone, is there an easy way to loop through the other backends defined in cinder.conf in driver code and get settings from them? (san-ip, etc.) | 16:05 |
| jgriffith | dannywilson: haha! :) | 16:06 |
| dannywilson | jgriffith: ;) | 16:06 |
| jgriffith | dannywilson: I know what you're looking at :) | 16:06 |
| hemna | ! | 16:06 |
| dannywilson | jgriffith: was going to ask you but figured anyone might know | 16:06 |
| dannywilson | backup manager has this nice stuff but alas https://github.com/openstack/cinder/blob/master/cinder/backup/manager.py#L152 | 16:06 |
| openstackgerrit | Zhang Jinnan proposed openstack/cinder: Volume enters error_deleting state when unmanage https://review.openstack.org/215494 | 16:06 |
| jgriffith | dannywilson: we can just add the same thing :) | 16:07 |
| *** kvidvans has quit IRC | 16:07 | |
| dannywilson | jgriffith: in L? | 16:08 |
| *** agarciam has quit IRC | 16:08 | |
| *** aix has quit IRC | 16:08 | |
| jgriffith | dannywilson: kinda depends, the replication thing is technically dead for everyone but you | 16:08 |
| jgriffith | dannywilson: but I'd like to discuss that and some other things in tomorrows meeting | 16:08 |
| *** Zhang has quit IRC | 16:09 | |
| dannywilson | jgriffith: oh | 16:09 |
| *** haomaiwang has quit IRC | 16:09 | |
| dannywilson | jgriffith: okay, is there a brute force way to parse the other managed back ends? | 16:09 |
| jgriffith | dannywilson: I was screwing around with getting hosts from backend and stuff, but it didn't really work | 16:10 |
| *** haomaiwa_ has joined #openstack-cinder | 16:10 | |
| jgriffith | dannywilson: I think the answer is in fact to add something similar to what you pointed out | 16:10 |
| dannywilson | jgriffith: gotcha, worst case I guess we can just support unmanaged targets at first | 16:11 |
| *** martyturner has joined #openstack-cinder | 16:13 | |
| *** jistr has quit IRC | 16:17 | |
| *** martyturner has quit IRC | 16:17 | |
| jgriffith | hemna: https://review.openstack.org/#/c/214731/ | 16:19 |
| jgriffith | hemna: ^^ Because get_volume_types is a valid call for any tenant | 16:19 |
| jgriffith | hemna: the only thing that's admin-only is extra-specs | 16:19 |
| jgriffith | geguileo: https://review.openstack.org/#/c/214731/ | 16:19 |
| jgriffith | geguileo: responses to your comments inline | 16:20 |
| hemna | ok I wasn't sure. it seemed that we were going through a lot of effort to return nothing | 16:20 |
| *** e0ne has quit IRC | 16:20 | |
| *** takedakn has joined #openstack-cinder | 16:20 | |
| hemna | if it wasn't the admin context | 16:20 |
| jgriffith | hemna: it still returns the voume-types | 16:20 |
| hemna | ok gotcha. thanks | 16:20 |
| jgriffith | hemna: name/id etc | 16:20 |
| geguileo | jgriffith: Looking now | 16:20 |
| *** timcl has joined #openstack-cinder | 16:21 | |
| jgriffith | geguileo: I'm not sure about changing it to del; seems like it shouldn't be a big deal, but I can change it if you feel strongly | 16:21 |
| jgriffith | geguileo: but the first time we get a failure for DNE I'm going to be really bummed out :( | 16:21 |
| jgriffith | geguileo: I don't know of any reason technically that pop/None is "weird" or "unacceptable" | 16:22 |
| geguileo | jgriffith: No big deal, just weird | 16:22 |
| *** takedakn has quit IRC | 16:22 | |
| jgriffith | geguileo: why is it *weird* ? | 16:22 |
| *** martyturner has joined #openstack-cinder | 16:22 | |
| geguileo | jgriffith: I was just writing it in the patch | 16:22 |
| jgriffith | geguileo: sure, I'm open minded... maybe you can educate me on why it's weird? | 16:22 |
| openstackgerrit | Michael Price proposed openstack/cinder: Add support for volume groups and netapp_raid_type https://review.openstack.org/203300 | 16:22 |
| DuncanT | jgriffith: FWIW I am fine with pop, but my code is know to be weird | 16:22 |
| geguileo | jgriffith: Because you access it directly 3 lines below | 16:22 |
| geguileo | jgriffith: So if it can me missing, we shouldn't access it directly 3 lines below | 16:23 |
| openstackgerrit | Michael Price proposed openstack/cinder: Implement AutoSupport for NetApp E-Series driver https://review.openstack.org/215807 | 16:23 |
| jgriffith | geguileo: hmm... but we create it down below | 16:23 |
| *** diogogmt has quit IRC | 16:24 | |
| jgriffith | geguileo: I think I see what you're saying | 16:24 |
| openstackgerrit | Michael Price proposed openstack/cinder: Add additional SSC extra specs to E-Series driver https://review.openstack.org/215808 | 16:24 |
| jgriffith | geguileo: if anybody cares enough about it I can certainly change it | 16:24 |
| geguileo | jgriffith: It's not wrong, the code is just fine, and it will work, like you said | 16:24 |
| openstackgerrit | Michael Price proposed openstack/cinder: Implement thin provisioning support for E-Series https://review.openstack.org/215833 | 16:24 |
| jgriffith | geguileo: ok... just want to make sure you're happy :) | 16:25 |
| * jgriffith is a "people pleaser" :) | 16:25 | |
| *** e0ne has joined #openstack-cinder | 16:25 | |
| *** diogogmt has joined #openstack-cinder | 16:25 | |
| geguileo | jgriffith: Downvote was just for function name and docstring | 16:25 |
| * jgriffith hears the groaning and laughter | 16:25 | |
| *** bnemec has joined #openstack-cinder | 16:25 | |
| *** sgotliv has joined #openstack-cinder | 16:25 | |
| geguileo | XD | 16:25 |
| jgriffith | geguileo: cool, I'll fix the docstring | 16:26 |
| geguileo | jgriffith: But not the method name? | 16:26 |
| jgriffith | geguileo: I disagree with you regarding the name | 16:26 |
| jgriffith | geguileo: what woul you like "_dict_with_extra_specs_if_admin_else_empty_dict() | 16:27 |
| geguileo | XD XD | 16:28 |
| jgriffith | :) | 16:28 |
| *** ByPasS has quit IRC | 16:28 | |
| geguileo | jgriffith: _query_to_dict | 16:28 |
| hemna | jgriffith, :P | 16:28 |
| geguileo | jgriffith: _result_query_to_dict | 16:28 |
| *** jdurgin1 has quit IRC | 16:29 | |
| jgriffith | geguileo: LOL | 16:29 |
| hemna | _something_other_than_what_it_was_previously_named_dict() | 16:29 |
| geguileo | jgriffith: But ok, if docstring is update it's ok | 16:29 |
| geguileo | XD | 16:29 |
| *** e0ne has quit IRC | 16:29 | |
| *** yuriy_n17 has quit IRC | 16:30 | |
| *** ByPasS has joined #openstack-cinder | 16:31 | |
| *** bkopilov has joined #openstack-cinder | 16:32 | |
| *** DTadrzak has quit IRC | 16:34 | |
| *** ronis__ has joined #openstack-cinder | 16:35 | |
| *** rushil has quit IRC | 16:37 | |
| *** Apoorva has quit IRC | 16:37 | |
| *** sghanekar has joined #openstack-cinder | 16:43 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: LeftHand: Adding Consistency Group Support https://review.openstack.org/214386 | 16:44 |
| *** skylerberg has joined #openstack-cinder | 16:51 | |
| *** rushil has joined #openstack-cinder | 16:53 | |
| *** Apoorva has joined #openstack-cinder | 16:53 | |
| *** rushil has quit IRC | 16:54 | |
| *** rushil has joined #openstack-cinder | 16:54 | |
| hemna | jgriffith, ping | 16:57 |
| *** sgotliv has quit IRC | 16:57 | |
| jgriffith | hemna: what's up? | 16:57 |
| hemna | question for you about this guy: https://review.openstack.org/#/c/204953 | 16:57 |
| hemna | which is about tracking progress for migration | 16:58 |
| hemna | I like the idea | 16:58 |
| *** sghanekar has quit IRC | 16:58 | |
| hemna | but it seems like that patch is adding a lot of complex code to the base volume driver class | 16:58 |
| hemna | just for tracking progress | 16:58 |
| jgriffith | hemna: yeah, I started looking at this last week... honestly I'm trying to understand if there's real value or not | 16:59 |
| jgriffith | hemna: and it seems kinda convoluted | 16:59 |
| hemna | I don't really like the idea of the drivers getting this complex code for things other than 'driver' like stuffs re: provisioning | 16:59 |
| hemna | changes the idea of what a driver is IMHO | 17:00 |
| jgriffith | hemna: meh... maybe; honestly I left it as "I need to load this up and see how it actually works" | 17:00 |
| jgriffith | hemna: because honestly I'm not sure I follow all of the code | 17:00 |
| hemna | yah there is that as well | 17:00 |
| hemna | I dunno, it seems like the volume manager should be doing some of this | 17:01 |
| hemna | to me a driver should be simple | 17:01 |
| hemna | for the most part, and the base driver shouldn't be overloaded with lots of complex things for managing processes | 17:01 |
| hemna | which this seems to make even worse | 17:01 |
| hemna | and by 'worse' I mean more non provisioning type of things | 17:02 |
| *** martyturner has quit IRC | 17:02 | |
| jgriffith | hemna: yeah, the only problem is that if you don't put it in the base you can't get the impl for free in all the sub-class drivers | 17:02 |
| hemna | maybe i'm stuck on the philosophical idea in my head of what a driver is supposed to do. | 17:02 |
| jgriffith | hemna: well, that's kinda what the driver needs to do in that case; because nobody else can report it's status | 17:03 |
| hemna | I could be off base here, just wanted to throw it out there and see what others thought. | 17:03 |
| jgriffith | hemna: but to your point there might be a better/lighter-weight way to go about it | 17:03 |
| jgriffith | hemna: I think your gut feel is probably right... "it's icky" | 17:04 |
| hemna | yah, it's icky is probably a good term for it :) | 17:04 |
| eharney | hemna: i think you make a good point... some of that does look like it should be handled in the manager... why would a driver send notifications, for instance? | 17:04 |
| hemna | same thing with all of the volume data copying stuff in there as well. | 17:04 |
| hemna | I'm not sure what the right answer is | 17:05 |
| eharney | having to import config opts into the driver from the manager is a tip-off that something isn't structured right to me | 17:05 |
| hemna | I don't really want to derail the effort on reporting progress, I just want to make sure that we are ok with the direction of the drivers and especially the BaseVD and what's supposed to be in there or isn't. | 17:05 |
| hemna | eharney, +1 | 17:06 |
| jgriffith | hemna: eharney honestly I don't think this really *fixes* the issues with migration, in fact I think it kinda makes things worse | 17:06 |
| patrickeast | any vmware folks around that maintain the vmdk driver stuff? | 17:06 |
| hemna | jgriffith, +1 | 17:06 |
| patrickeast | the image cache patch is failing on http://logs.openstack.org/95/195795/45/check/gate-cinder-python27/40b5028/testr_results.html.gz | 17:06 |
| patrickeast | and i have no idea whats going on with it... can't even repro on my local machine | 17:06 |
| jgriffith | hemna: eharney I'm inclined to punt on it | 17:06 |
| hemna | re: get_pid_for_host_based_migration() | 17:07 |
| hemna | really, drivers are getting a process id? | 17:07 |
| hemna | that's just wrong to me. | 17:07 |
| *** merooney has joined #openstack-cinder | 17:07 | |
| eharney | hemna: i agree | 17:07 |
| *** e0ne has joined #openstack-cinder | 17:08 | |
| *** Yogi12 has quit IRC | 17:08 | |
| *** haomaiwa_ has quit IRC | 17:09 | |
| *** julim has quit IRC | 17:09 | |
| *** rbradbury has joined #openstack-cinder | 17:09 | |
| *** haomaiwang has joined #openstack-cinder | 17:10 | |
| hemna | ok I put a -2 on it for the time being and asked for this to be discussed in the Cinder meeting. | 17:10 |
| *** wilson2 has joined #openstack-cinder | 17:11 | |
| eharney | hemna: cool | 17:11 |
| jgriffith | hemna: thanks! | 17:11 |
| hemna | np. I don't like being the 'bad guy (-2)', but I think it's worth discussing | 17:12 |
| *** wilson1 has quit IRC | 17:13 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 17:13 |
| *** Yogi1 has joined #openstack-cinder | 17:16 | |
| *** mudassirlatif has joined #openstack-cinder | 17:17 | |
| *** diablo_rojo has quit IRC | 17:19 | |
| *** jgregor has quit IRC | 17:19 | |
| *** kvidvans has joined #openstack-cinder | 17:20 | |
| *** esker has joined #openstack-cinder | 17:24 | |
| *** leeantho has quit IRC | 17:25 | |
| *** leeantho has joined #openstack-cinder | 17:25 | |
| *** p0rtal has joined #openstack-cinder | 17:27 | |
| *** angela-s has joined #openstack-cinder | 17:31 | |
| *** Apoorva has quit IRC | 17:33 | |
| *** esker has quit IRC | 17:37 | |
| *** BharatK has quit IRC | 17:39 | |
| *** timcl has quit IRC | 17:41 | |
| *** timcl has joined #openstack-cinder | 17:41 | |
| *** dustins has joined #openstack-cinder | 17:43 | |
| *** abhilash has joined #openstack-cinder | 17:43 | |
| *** lpetrut has quit IRC | 17:48 | |
| *** Apoorva has joined #openstack-cinder | 17:48 | |
| *** Apoorva has quit IRC | 17:49 | |
| *** dustins has quit IRC | 17:50 | |
| *** mudassirlatif has quit IRC | 17:51 | |
| *** Apoorva has joined #openstack-cinder | 17:51 | |
| *** diemt has joined #openstack-cinder | 17:52 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: LeftHand: Adding Consistency Group Support https://review.openstack.org/214386 | 17:58 |
| *** BharatK has joined #openstack-cinder | 18:00 | |
| *** dannywilson has quit IRC | 18:02 | |
| *** dannywilson has joined #openstack-cinder | 18:02 | |
| *** rs007 has joined #openstack-cinder | 18:06 | |
| *** haomaiwang has quit IRC | 18:09 | |
| *** haomaiwang has joined #openstack-cinder | 18:10 | |
| *** lpetrut has joined #openstack-cinder | 18:10 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Extend unit tests for backup get_all https://review.openstack.org/216408 | 18:10 |
| *** rushil has quit IRC | 18:11 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Adds framework for get_capabilities() feature https://review.openstack.org/201243 | 18:15 |
| *** rs007 has left #openstack-cinder | 18:17 | |
| * hemna scratches his head | 18:17 | |
| *** rs007 has joined #openstack-cinder | 18:20 | |
| *** rushil has joined #openstack-cinder | 18:20 | |
| *** jgregor has joined #openstack-cinder | 18:20 | |
| *** diablo_rojo has joined #openstack-cinder | 18:22 | |
| e0ne | Cores, please review https://review.openstack.org/#/c/194180/ (Add more Rally scenarios to run) - lets increase our test coverage | 18:23 |
| *** ociuhandu_ has joined #openstack-cinder | 18:28 | |
| *** ociuhandu has quit IRC | 18:29 | |
| *** ociuhandu_ is now known as ociuhandu | 18:29 | |
| *** mtanino_ has joined #openstack-cinder | 18:32 | |
| *** dustins has joined #openstack-cinder | 18:33 | |
| *** mtanino has quit IRC | 18:34 | |
| *** shyama has quit IRC | 18:41 | |
| *** haomaiwang has quit IRC | 18:42 | |
| *** haomaiwang has joined #openstack-cinder | 18:45 | |
| *** porrua has quit IRC | 18:45 | |
| *** changbl has quit IRC | 18:49 | |
| *** jgregor has quit IRC | 18:51 | |
| *** dflorea has joined #openstack-cinder | 18:52 | |
| *** daneyon_ has joined #openstack-cinder | 18:53 | |
| *** porrua has joined #openstack-cinder | 18:55 | |
| *** daneyon has quit IRC | 18:55 | |
| *** mtanino has joined #openstack-cinder | 18:58 | |
| *** changbl has joined #openstack-cinder | 18:59 | |
| openstackgerrit | Qiu Yu proposed openstack/python-cinderclient: Make sure --bypass-url honored if specified https://review.openstack.org/216836 | 18:59 |
| *** sgotliv has joined #openstack-cinder | 18:59 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Allow specified backend capabilities to be retrieved https://review.openstack.org/212861 | 18:59 |
| *** mtanino_ has quit IRC | 19:01 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/cinder: Update NetApp Drivers to use netapp_lib https://review.openstack.org/215700 | 19:03 |
| *** dustins has quit IRC | 19:04 | |
| *** pots1 has joined #openstack-cinder | 19:05 | |
| *** pots has quit IRC | 19:05 | |
| *** pots has joined #openstack-cinder | 19:08 | |
| *** dflorea has quit IRC | 19:08 | |
| *** haomaiwang has quit IRC | 19:09 | |
| *** haomaiwa_ has joined #openstack-cinder | 19:10 | |
| *** pots1 has quit IRC | 19:11 | |
| *** daneyon_ has quit IRC | 19:12 | |
| *** daneyon has joined #openstack-cinder | 19:12 | |
| *** daneyon has quit IRC | 19:12 | |
| *** merooney has quit IRC | 19:17 | |
| *** tpsilva has quit IRC | 19:17 | |
| *** rushil has quit IRC | 19:19 | |
| *** lpetrut has quit IRC | 19:24 | |
| *** dims__ has quit IRC | 19:25 | |
| *** dims has joined #openstack-cinder | 19:26 | |
| *** ociuhandu has quit IRC | 19:27 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Continue conversion to volume object https://review.openstack.org/201404 | 19:27 |
| *** ociuhandu has joined #openstack-cinder | 19:29 | |
| *** dannywilson has quit IRC | 19:32 | |
| *** Lee1092 has quit IRC | 19:33 | |
| *** ronis__ has quit IRC | 19:34 | |
| openstackgerrit | Sonia Ghanekar proposed openstack/cinder: Add manage/unmanage volume support for Nimble https://review.openstack.org/214392 | 19:36 |
| *** lpetrut has joined #openstack-cinder | 19:38 | |
| *** martyturner has joined #openstack-cinder | 19:43 | |
| *** lpetrut has quit IRC | 19:45 | |
| *** haomaiw__ has joined #openstack-cinder | 19:46 | |
| *** mriedem_class is now known as mriedem | 19:46 | |
| *** haomaiwa_ has quit IRC | 19:46 | |
| *** martyturner has quit IRC | 19:50 | |
| *** dflorea has joined #openstack-cinder | 19:50 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Parameter osapi_max_limit is always used by default https://review.openstack.org/216661 | 19:51 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Parameter osapi_max_limit is always used by default https://review.openstack.org/216661 | 19:52 |
| *** dustins has joined #openstack-cinder | 19:56 | |
| *** dflorea has quit IRC | 20:00 | |
| *** diemt has quit IRC | 20:00 | |
| *** kvidvans has quit IRC | 20:02 | |
| *** merooney has joined #openstack-cinder | 20:05 | |
| *** julim has joined #openstack-cinder | 20:06 | |
| *** porrua has quit IRC | 20:06 | |
| *** rushil has joined #openstack-cinder | 20:06 | |
| *** rushil_ has joined #openstack-cinder | 20:08 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add pagination to snapshots https://review.openstack.org/195071 | 20:08 |
| *** lpetrut has joined #openstack-cinder | 20:08 | |
| *** dflorea has joined #openstack-cinder | 20:08 | |
| *** haomaiw__ has quit IRC | 20:09 | |
| *** haomaiwang has joined #openstack-cinder | 20:10 | |
| *** rushil has quit IRC | 20:12 | |
| *** lpetrut has quit IRC | 20:15 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: Fixes delete cgsnapshot to catch SnapshotIsBusy https://review.openstack.org/216858 | 20:16 |
| *** diemt has joined #openstack-cinder | 20:17 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: init_volume stale volume info fix https://review.openstack.org/216859 | 20:20 |
| *** dustins has quit IRC | 20:27 | |
| *** kvidvans has joined #openstack-cinder | 20:28 | |
| kvidvans | hemna, smcginnis: For patch: https://review.openstack.org/#/c/210608/12, Oracle CI for both drivers has posted a success | 20:30 |
| smcginnis | kvidvans: OK, I'll take a look. Thanks for the heads up. | 20:32 |
| kvidvans | hemna: thanks! | 20:32 |
| hemna | kvidvans, np | 20:32 |
| smcginnis | Just not as fast as hemna! :) | 20:32 |
| *** anshul has quit IRC | 20:32 | |
| *** ociuhandu has quit IRC | 20:32 | |
| kvidvans | thank you smcginnis | 20:32 |
| smcginnis | np | 20:32 |
| *** Apoorva has quit IRC | 20:33 | |
| *** Apoorva has joined #openstack-cinder | 20:34 | |
| *** e0ne has quit IRC | 20:35 | |
| *** gouthamr has quit IRC | 20:40 | |
| *** akerr has quit IRC | 20:42 | |
| *** diemt has quit IRC | 20:42 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: NetApp DOT block driver over-subscription support https://review.openstack.org/215865 | 20:44 |
| *** ociuhandu has joined #openstack-cinder | 20:45 | |
| *** IanGovett1 has quit IRC | 20:49 | |
| *** Yogi1 has quit IRC | 20:52 | |
| *** changbl has quit IRC | 20:53 | |
| *** daneyon has joined #openstack-cinder | 20:56 | |
| *** IanGovett has joined #openstack-cinder | 20:57 | |
| *** bswartz has quit IRC | 20:59 | |
| *** lprice1 has joined #openstack-cinder | 21:01 | |
| ameade | lprice1, tbarron, thingee: https://blueprints.launchpad.net/cinder/+spec/netapp-e-series-asup appears to not be approved | 21:02 |
| *** rs007_ has joined #openstack-cinder | 21:02 | |
| *** raildo is now known as raildo-afk | 21:03 | |
| *** rs007_ has quit IRC | 21:03 | |
| *** bnemec has quit IRC | 21:05 | |
| *** rs007 has quit IRC | 21:06 | |
| openstackgerrit | Michael Price proposed openstack/cinder: Add support for volume groups and netapp_raid_type https://review.openstack.org/203300 | 21:06 |
| *** bnemec has joined #openstack-cinder | 21:06 | |
| *** gouthamr has joined #openstack-cinder | 21:09 | |
| *** haomaiwang has quit IRC | 21:09 | |
| *** haomaiwang has joined #openstack-cinder | 21:10 | |
| *** diablo_rojo has quit IRC | 21:15 | |
| ameade | i think it should have been, not sure why it isn't but I cannot tweak it at all either since rushil is no longer on the team | 21:16 |
| *** jungleboyj has quit IRC | 21:24 | |
| *** merooney has quit IRC | 21:28 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: Fixes delete cgsnapshot to catch SnapshotIsBusy https://review.openstack.org/216858 | 21:33 |
| *** IlyaG has joined #openstack-cinder | 21:33 | |
| *** ekoojr has joined #openstack-cinder | 21:35 | |
| *** ekoojr has quit IRC | 21:36 | |
| *** annasort has quit IRC | 21:39 | |
| *** ekoojr has joined #openstack-cinder | 21:46 | |
| *** ekoojr has quit IRC | 21:47 | |
| *** e0ne has joined #openstack-cinder | 21:48 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: init_volume stale volume info fix https://review.openstack.org/216859 | 21:49 |
| *** mriedem has quit IRC | 21:49 | |
| *** edtubill has quit IRC | 21:50 | |
| *** bswartz has joined #openstack-cinder | 21:53 | |
| *** jaypipes has quit IRC | 21:55 | |
| mfisch | thingee: just to follow up on the "cinder wont make volumes anymore" question I asked yesterday, turns out its a ceph bug exacerbated by kilo | 21:57 |
| *** edmondsw has quit IRC | 21:58 | |
| *** jaypipes has joined #openstack-cinder | 21:59 | |
| openstackgerrit | Jon Bernard proposed openstack/cinder: Add support for file I/O volume migration https://review.openstack.org/187270 | 22:01 |
| *** gouthamr has quit IRC | 22:02 | |
| *** gouthamr has joined #openstack-cinder | 22:03 | |
| *** xyang1 has quit IRC | 22:04 | |
| *** rushil_ has quit IRC | 22:05 | |
| *** rushil has joined #openstack-cinder | 22:05 | |
| *** aix has joined #openstack-cinder | 22:07 | |
| *** bnemec has quit IRC | 22:07 | |
| *** bnemec has joined #openstack-cinder | 22:09 | |
| *** haomaiwang has quit IRC | 22:09 | |
| *** haomaiwa_ has joined #openstack-cinder | 22:10 | |
| *** eharney has quit IRC | 22:11 | |
| *** e0ne has quit IRC | 22:16 | |
| *** crose has quit IRC | 22:20 | |
| *** gouthamr has quit IRC | 22:23 | |
| tbarron | ameade: lprice1, tbarron, thingee: jgriffith: that looks like an inadvertent omission. | 22:32 |
| tbarron | ameade: lprice1, thingee, jgriffith: our batch of BPs for L3 was approved 7/29 | 22:33 |
| tbarron | http://eavesdrop.openstack.org/meetings/cinder/2015/cinder.2015-07-29-16.03.log.txt | 22:33 |
| tbarron | and this bp https://blueprints.launchpad.net/cinder/+spec/netapp-e-series-asup was filed 7/10 | 22:34 |
| tbarron | it's not a case of trying to sneak something in | 22:34 |
| *** RaySun has joined #openstack-cinder | 22:35 | |
| jgriffith | tbarron: yeah, although I have not way of knowing what was or wasn't intended to be approved or why | 22:36 |
| jgriffith | tbarron: personally I don't have a problem with it, bp was filed, code was submitted etc | 22:36 |
| tbarron | jgriffith: understood. | 22:36 |
| jgriffith | tbarron: you'll have to convince jungleboy if you can catch him however :) | 22:37 |
| jgriffith | tbarron: I plan to talk about feature freeze etc in tomorrows meeting | 22:37 |
| tbarron | jgriffith: well, I made the mistake of asking in the meeting about this stuff :-) so I backed away and guess I didn't go through the whole list that we asked for to see that | 22:37 |
| tbarron | they were all approved | 22:37 |
| jgriffith | tbarron: I'd like to make decisions as a team | 22:37 |
| tbarron | so my fault too | 22:37 |
| tbarron | or mostly | 22:37 |
| jgriffith | tbarron: nah... it's just the nature of a release | 22:38 |
| jgriffith | tbarron: I think it's fair for us to go through these as a team at this point | 22:38 |
| tbarron | I didn't like having to nag about this stuff in the fist place. | 22:38 |
| tbarron | jgriffith: sure, that's fine. I'm not goint to put it on the agenda myself though :-) | 22:38 |
| tbarron | jgriffith: or maybe you just mean you and jungleboyj or you and other cores. Whatever you folks decide ;-) | 22:39 |
| asselin_ | patrickeast, all my ci systems are migrated to use project-config | 22:39 |
| patrickeast | asselin_: awesome, so far so good? | 22:40 |
| asselin_ | patrickeast, yes, all no-ops | 22:40 |
| jgriffith | tbarron: I added a topic | 22:42 |
| tbarron | jgriffith: thanks. I've done my bit. | 22:43 |
| *** rushil has quit IRC | 22:43 | |
| *** sgotliv has quit IRC | 22:47 | |
| *** jaypipes has quit IRC | 22:48 | |
| *** gouthamr has joined #openstack-cinder | 22:48 | |
| *** gouthamr has quit IRC | 22:54 | |
| jgriffith | tbarron: https://blueprints.launchpad.net/cinder/+spec/netapp-e-series-asup | 22:54 |
| tbarron | jgriffith: thanks! Now on to more rewarding stuff. | 22:56 |
| *** RaySun has quit IRC | 22:58 | |
| *** asd112z has quit IRC | 22:59 | |
| *** dflorea has quit IRC | 23:01 | |
| *** dflorea has joined #openstack-cinder | 23:06 | |
| *** dflorea has quit IRC | 23:08 | |
| *** haomaiwa_ has quit IRC | 23:09 | |
| *** 64MADOR5X has joined #openstack-cinder | 23:10 | |
| *** dflorea has joined #openstack-cinder | 23:10 | |
| *** dflorea has quit IRC | 23:11 | |
| *** akerr has joined #openstack-cinder | 23:13 | |
| *** akerr has quit IRC | 23:13 | |
| *** dflorea_ has joined #openstack-cinder | 23:15 | |
| *** dflorea_ has quit IRC | 23:16 | |
| *** daneyon has quit IRC | 23:18 | |
| *** geguileo has quit IRC | 23:19 | |
| *** simondodsley has quit IRC | 23:20 | |
| *** bitblt has joined #openstack-cinder | 23:28 | |
| *** bitblt has quit IRC | 23:28 | |
| *** changbl has joined #openstack-cinder | 23:29 | |
| *** geguileo has joined #openstack-cinder | 23:29 | |
| *** jseiler_ has quit IRC | 23:30 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Add rootwrap filters https://review.openstack.org/207553 | 23:30 |
| *** jseiler_ has joined #openstack-cinder | 23:30 | |
| *** hemna is now known as hemnafk | 23:31 | |
| *** dflorea has joined #openstack-cinder | 23:32 | |
| *** dannywilson has joined #openstack-cinder | 23:33 | |
| *** dflorea has quit IRC | 23:35 | |
| *** dannywilson has quit IRC | 23:38 | |
| *** diogogmt has quit IRC | 23:38 | |
| *** IlyaG has quit IRC | 23:39 | |
| *** annasort has joined #openstack-cinder | 23:40 | |
| *** annasort has quit IRC | 23:41 | |
| *** annegent_ has joined #openstack-cinder | 23:42 | |
| jgriffith | geguileo: still around? | 23:42 |
| *** marcusvrn_ has quit IRC | 23:45 | |
| *** annegentle has quit IRC | 23:45 | |
| *** IlyaG has joined #openstack-cinder | 23:47 | |
| *** erlon has quit IRC | 23:48 | |
| *** IlyaG has quit IRC | 23:51 | |
| *** leeantho has quit IRC | 23:52 | |
| *** takedakn has joined #openstack-cinder | 23:53 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Filter out extra-specs from type get for non-admin https://review.openstack.org/214731 | 23:57 |
| *** dflorea has joined #openstack-cinder | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!