| *** huanan has quit IRC | 00:00 | |
| *** huyang has quit IRC | 00:00 | |
| *** Gao has quit IRC | 00:00 | |
| *** wanghao has quit IRC | 00:00 | |
| *** Lallana has joined #openstack-cinder | 00:01 | |
| *** huyang has joined #openstack-cinder | 00:01 | |
| *** wanghao has joined #openstack-cinder | 00:02 | |
| *** zhongjun_ has joined #openstack-cinder | 00:03 | |
| *** ducttape_ has quit IRC | 00:03 | |
| *** huanan has joined #openstack-cinder | 00:03 | |
| *** yhayashi has joined #openstack-cinder | 00:04 | |
| *** lixiaoy1 has quit IRC | 00:05 | |
| *** lixiaoy1 has joined #openstack-cinder | 00:06 | |
| *** huanan has quit IRC | 00:09 | |
| *** zhangjn has quit IRC | 00:11 | |
| *** bardia has quit IRC | 00:13 | |
| *** zhangjn has joined #openstack-cinder | 00:14 | |
| *** r-daneel has quit IRC | 00:19 | |
| *** suyash has quit IRC | 00:25 | |
| *** gouthamr_ has quit IRC | 00:30 | |
| *** mtanino has quit IRC | 00:33 | |
| *** ducttape_ has joined #openstack-cinder | 00:34 | |
| *** zhongjun_ has quit IRC | 00:35 | |
| *** jwcroppe has quit IRC | 00:36 | |
| *** takedakn has joined #openstack-cinder | 00:36 | |
| *** anteaya has quit IRC | 00:37 | |
| *** takedakn has quit IRC | 00:41 | |
| *** takedakn has joined #openstack-cinder | 00:42 | |
| *** takedakn has quit IRC | 00:42 | |
| *** sdake has quit IRC | 00:42 | |
| *** ducttape_ has quit IRC | 00:49 | |
| *** salv-orlando has quit IRC | 00:53 | |
| *** salv-orlando has joined #openstack-cinder | 00:53 | |
| *** mriedem has joined #openstack-cinder | 00:56 | |
| openstackgerrit | yuyafei proposed openstack/cinder: Size in tintri driver should be converted to integer https://review.openstack.org/325025 | 01:03 |
|---|---|---|
| *** jgregor has quit IRC | 01:03 | |
| *** raunak has quit IRC | 01:05 | |
| *** scottda has quit IRC | 01:05 | |
| *** Apoorva has quit IRC | 01:07 | |
| *** Apoorva has joined #openstack-cinder | 01:10 | |
| openstackgerrit | yuyafei proposed openstack/cinder: Huawei driver should return ceiling of volume size https://review.openstack.org/323699 | 01:11 |
| *** bardia has joined #openstack-cinder | 01:13 | |
| *** Apoorva has quit IRC | 01:17 | |
| *** jgregor has joined #openstack-cinder | 01:20 | |
| *** cknight has joined #openstack-cinder | 01:21 | |
| *** suyash has joined #openstack-cinder | 01:27 | |
| *** cknight has quit IRC | 01:34 | |
| *** cknight has joined #openstack-cinder | 01:34 | |
| *** cknight1 has joined #openstack-cinder | 01:38 | |
| *** cknight has quit IRC | 01:38 | |
| *** cknight has joined #openstack-cinder | 01:40 | |
| *** cknight1 has quit IRC | 01:40 | |
| *** cknight1 has joined #openstack-cinder | 01:43 | |
| *** cknight has quit IRC | 01:43 | |
| *** cknight has joined #openstack-cinder | 01:44 | |
| *** cknight1 has quit IRC | 01:44 | |
| *** cknight1 has joined #openstack-cinder | 01:47 | |
| *** cknight has quit IRC | 01:48 | |
| *** mtanino has joined #openstack-cinder | 01:48 | |
| *** cknight1 has quit IRC | 01:52 | |
| *** cknight has joined #openstack-cinder | 01:53 | |
| *** chris_morrell has quit IRC | 01:54 | |
| *** cknight1 has joined #openstack-cinder | 01:56 | |
| *** cknight has quit IRC | 01:56 | |
| *** cknight has joined #openstack-cinder | 01:59 | |
| *** cknight1 has quit IRC | 01:59 | |
| *** sticker has quit IRC | 02:02 | |
| *** savihou has quit IRC | 02:04 | |
| *** savihou has joined #openstack-cinder | 02:05 | |
| *** winston-d_ has quit IRC | 02:07 | |
| *** cknight has quit IRC | 02:09 | |
| *** cknight has joined #openstack-cinder | 02:09 | |
| *** cknight has quit IRC | 02:11 | |
| *** cknight has joined #openstack-cinder | 02:11 | |
| *** ducttape_ has joined #openstack-cinder | 02:13 | |
| *** mriedem has quit IRC | 02:14 | |
| *** cknight has quit IRC | 02:16 | |
| *** cknight has joined #openstack-cinder | 02:16 | |
| *** karthikp_ has joined #openstack-cinder | 02:20 | |
| *** cknight1 has joined #openstack-cinder | 02:20 | |
| *** cknight has quit IRC | 02:20 | |
| *** cknight has joined #openstack-cinder | 02:23 | |
| *** cknight1 has quit IRC | 02:24 | |
| *** jgregor has quit IRC | 02:27 | |
| *** gouthamr has joined #openstack-cinder | 02:31 | |
| *** sticker has joined #openstack-cinder | 02:34 | |
| *** jgriffith_away is now known as jgriffith | 02:42 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Add new attach/detach API calls https://review.openstack.org/327408 | 02:48 |
| *** cknight1 has joined #openstack-cinder | 02:48 | |
| *** cknight has quit IRC | 02:49 | |
| *** jgregor has joined #openstack-cinder | 02:51 | |
| *** cknight1 has quit IRC | 02:51 | |
| *** sheel has joined #openstack-cinder | 02:52 | |
| *** cknight has joined #openstack-cinder | 02:54 | |
| *** cknight1 has joined #openstack-cinder | 02:56 | |
| *** cknight has quit IRC | 02:56 | |
| *** adisky has joined #openstack-cinder | 02:59 | |
| *** raunak has joined #openstack-cinder | 03:01 | |
| *** sdake has joined #openstack-cinder | 03:10 | |
| *** cknight has joined #openstack-cinder | 03:13 | |
| *** cknight1 has quit IRC | 03:14 | |
| *** gouthamr has quit IRC | 03:21 | |
| *** cknight1 has joined #openstack-cinder | 03:24 | |
| *** cknight has quit IRC | 03:24 | |
| *** sdake has quit IRC | 03:26 | |
| *** ducttape_ has quit IRC | 03:28 | |
| *** lprice has joined #openstack-cinder | 03:34 | |
| *** lprice1 has quit IRC | 03:36 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/327362 | 03:38 |
| *** iceyao has joined #openstack-cinder | 03:38 | |
| *** karthikp_ has quit IRC | 03:39 | |
| *** bardia has quit IRC | 03:41 | |
| *** iceyao has quit IRC | 03:43 | |
| *** cknight1 has quit IRC | 03:44 | |
| *** links has joined #openstack-cinder | 03:46 | |
| *** jwcroppe has joined #openstack-cinder | 04:02 | |
| *** Poornima has joined #openstack-cinder | 04:09 | |
| *** jwcroppe has quit IRC | 04:14 | |
| *** Lee1092 has joined #openstack-cinder | 04:15 | |
| *** zhongjun_ has joined #openstack-cinder | 04:15 | |
| *** huanan has joined #openstack-cinder | 04:15 | |
| *** akshai has quit IRC | 04:24 | |
| *** salv-orl_ has joined #openstack-cinder | 04:25 | |
| *** salv-orlando has quit IRC | 04:28 | |
| *** akshai has joined #openstack-cinder | 04:33 | |
| *** akshai has quit IRC | 04:36 | |
| openstackgerrit | nikesh proposed openstack/cinder: Provide Kaminario K2 all-flash array iSCSI driver https://review.openstack.org/327285 | 04:40 |
| *** salv-orl_ has quit IRC | 04:51 | |
| *** salv-orlando has joined #openstack-cinder | 04:51 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:56 | |
| *** harlowja_at_home has quit IRC | 04:56 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:56 | |
| *** GB21 has joined #openstack-cinder | 04:57 | |
| *** salv-orlando has quit IRC | 05:00 | |
| *** karthikp_ has joined #openstack-cinder | 05:01 | |
| *** pgadiya has joined #openstack-cinder | 05:04 | |
| *** savihou has quit IRC | 05:04 | |
| *** harlowja_at_home has quit IRC | 05:08 | |
| *** rcernin|off has joined #openstack-cinder | 05:15 | |
| *** fmccrthy has quit IRC | 05:23 | |
| *** ameade has quit IRC | 05:24 | |
| *** rajinir has quit IRC | 05:24 | |
| *** tpeoples has quit IRC | 05:24 | |
| *** DuncanT has quit IRC | 05:24 | |
| *** sheel has quit IRC | 05:24 | |
| *** vmtyler has quit IRC | 05:24 | |
| *** serverascode has quit IRC | 05:25 | |
| *** cppforlife_ has quit IRC | 05:25 | |
| *** adisky has quit IRC | 05:25 | |
| *** rhefner has quit IRC | 05:25 | |
| *** Lee1092 has quit IRC | 05:25 | |
| *** aimeeu has quit IRC | 05:25 | |
| *** briancurtin has quit IRC | 05:25 | |
| *** boris-42 has quit IRC | 05:25 | |
| *** zhiyan has quit IRC | 05:25 | |
| *** sweston has quit IRC | 05:25 | |
| *** patrickeast has quit IRC | 05:25 | |
| *** zhenguo_ has quit IRC | 05:25 | |
| *** raunak has quit IRC | 05:26 | |
| *** rcernin|off has quit IRC | 05:26 | |
| *** rhefner has joined #openstack-cinder | 05:28 | |
| *** vmtyler has joined #openstack-cinder | 05:28 | |
| *** DuncanT has joined #openstack-cinder | 05:28 | |
| *** adisky has joined #openstack-cinder | 05:28 | |
| *** cppforlife_ has joined #openstack-cinder | 05:28 | |
| *** sheel has joined #openstack-cinder | 05:29 | |
| *** briancurtin has joined #openstack-cinder | 05:30 | |
| *** zhenguo_ has joined #openstack-cinder | 05:30 | |
| *** patrickeast has joined #openstack-cinder | 05:30 | |
| *** sweston has joined #openstack-cinder | 05:30 | |
| *** jwcroppe has joined #openstack-cinder | 05:31 | |
| *** aimeeu has joined #openstack-cinder | 05:32 | |
| *** Lee1092 has joined #openstack-cinder | 05:32 | |
| *** ameade has joined #openstack-cinder | 05:32 | |
| *** tpeoples has joined #openstack-cinder | 05:32 | |
| *** adisky has quit IRC | 05:33 | |
| *** vmtyler has quit IRC | 05:33 | |
| *** zhiyan has joined #openstack-cinder | 05:33 | |
| *** rajinir has joined #openstack-cinder | 05:33 | |
| *** serverascode has joined #openstack-cinder | 05:34 | |
| *** boris-42 has joined #openstack-cinder | 05:34 | |
| *** fmccrthy has joined #openstack-cinder | 05:34 | |
| *** vmtyler has joined #openstack-cinder | 05:35 | |
| *** suyash has quit IRC | 05:36 | |
| *** adisky has joined #openstack-cinder | 05:37 | |
| *** karthikp_ has quit IRC | 05:40 | |
| *** savihou has joined #openstack-cinder | 05:59 | |
| *** savihou has quit IRC | 06:01 | |
| *** savihou has joined #openstack-cinder | 06:01 | |
| *** GB21 has quit IRC | 06:03 | |
| *** adisky has quit IRC | 06:07 | |
| *** vmtyler has quit IRC | 06:08 | |
| *** liverpooler has quit IRC | 06:08 | |
| *** vmtyler has joined #openstack-cinder | 06:10 | |
| nikeshm | hi | 06:12 |
| nikeshm | any issue in jenkins | 06:12 |
| *** rcernin|off has joined #openstack-cinder | 06:15 | |
| *** tomoiaga has joined #openstack-cinder | 06:15 | |
| openstackgerrit | Abhishek Kekane proposed openstack/cinder: volume upload-to-image fails for iso disk-format https://review.openstack.org/287606 | 06:17 |
| *** GB21 has joined #openstack-cinder | 06:19 | |
| *** bbhavsar has joined #openstack-cinder | 06:20 | |
| *** adisky has joined #openstack-cinder | 06:20 | |
| *** rcernin|off is now known as rcernin | 06:21 | |
| nikeshm | not sure why jenkins give -1 on https://review.openstack.org/#/c/327285/3 | 06:21 |
| nikeshm | any idea | 06:21 |
| *** lpetrut has joined #openstack-cinder | 06:29 | |
| *** arecknag has joined #openstack-cinder | 06:32 | |
| *** laughterwym has joined #openstack-cinder | 06:37 | |
| *** belmoreira has joined #openstack-cinder | 06:38 | |
| *** GB21 has quit IRC | 06:42 | |
| *** ChubYann has quit IRC | 06:44 | |
| *** bardia has joined #openstack-cinder | 06:47 | |
| *** belmoreira has quit IRC | 06:48 | |
| *** bardia has quit IRC | 06:50 | |
| *** GB21 has joined #openstack-cinder | 07:00 | |
| *** mjura has joined #openstack-cinder | 07:01 | |
| *** bardia has joined #openstack-cinder | 07:02 | |
| *** belmoreira has joined #openstack-cinder | 07:07 | |
| sheel | nikeshm: I have issues recheck on it ... seems some env issue.. | 07:09 |
| dulek | nikeshm: http://logs.openstack.org/85/327285/3/check/gate-cinder-python27-db/f1dd3df/console.html#_2016-06-09_04_48_30_414 | 07:10 |
| dulek | nikeshm: Aren't you trying to execute new commands without rootwrap entries? | 07:11 |
| dulek | nikeshm: Like this systool and cat? | 07:11 |
| dulek | nikeshm: Or you just haven't mocked them? | 07:13 |
| nikeshm | dulek: i am not using any systool and cat,. | 07:14 |
| nikeshm | type commands | 07:15 |
| *** mtanino has quit IRC | 07:15 | |
| nikeshm | but i think internally it might be called | 07:15 |
| *** bardia has quit IRC | 07:16 | |
| dulek | nikeshm: Yup, this new lib maybe? You can for example get something named systool into your virtualenv and check again. | 07:18 |
| *** mjura has quit IRC | 07:28 | |
| *** mjura has joined #openstack-cinder | 07:31 | |
| *** itzdilip has quit IRC | 07:32 | |
| *** liverpooler has joined #openstack-cinder | 07:36 | |
| *** lpetrut has quit IRC | 07:39 | |
| *** salv-orlando has joined #openstack-cinder | 07:41 | |
| *** salv-orl_ has joined #openstack-cinder | 07:42 | |
| *** laughterwym has quit IRC | 07:45 | |
| *** avishay has joined #openstack-cinder | 07:46 | |
| *** salv-orlando has quit IRC | 07:46 | |
| *** lpetrut has joined #openstack-cinder | 07:46 | |
| *** hoonetorg has quit IRC | 07:47 | |
| *** liverpooler has quit IRC | 07:55 | |
| *** liverpooler has joined #openstack-cinder | 07:55 | |
| *** nkrinner has joined #openstack-cinder | 07:56 | |
| *** tesseract has joined #openstack-cinder | 07:56 | |
| *** hoonetorg has joined #openstack-cinder | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| *** adisky has quit IRC | 08:00 | |
| *** openstackgerrit has quit IRC | 08:03 | |
| *** openstackgerrit has joined #openstack-cinder | 08:04 | |
| *** zhipeng has joined #openstack-cinder | 08:06 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| *** lpetrut has quit IRC | 08:17 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder-specs: Update Job Distribution for A/A Specs https://review.openstack.org/327283 | 08:21 |
| flip214 | can anybody give a hint how to use test.testtools.skipIf(...) for a whole class of tests in a cinder unit test? | 08:21 |
| *** permalac has joined #openstack-cinder | 08:21 | |
| flip214 | I get "running None" "The test run didn't actually run any tests | 08:21 |
| *** nkrinner has quit IRC | 08:22 | |
| flip214 | although it shouldn't be skipped right now | 08:22 |
| *** zhipeng has quit IRC | 08:23 | |
| zigo | E: cinder source: source-is-missing api-ref/v1/source/_static/bootstrap.min.js | 08:23 |
| zigo | Cinder is now non-free. | 08:23 |
| zigo | Could someone address this, and replace the javascript blob by a free software version? | 08:24 |
| *** salv-orl_ has quit IRC | 08:25 | |
| *** itzdilip has joined #openstack-cinder | 08:25 | |
| dulek | zigo: There is a line about MIT license in that file. What exactly would you expect there? | 08:26 |
| * dulek isn't really keen on software licensing… :( | 08:26 | |
| zigo | dulek: I expect people to remember that we *cannot* include pre-compiled javascript blobs like this one: we must include the non-minified version. | 08:26 |
| zigo | It's only the 100th time I repeat it... :P | 08:26 |
| zigo | It's like if we were shipping a .so or a windows .dll pre-compiled without its corresponding source code. | 08:27 |
| *** nkrinner has joined #openstack-cinder | 08:27 | |
| dulek | zigo: Same goes for bootstrap.min.css? | 08:28 |
| zigo | Yup. | 08:28 |
| zigo | For everything. | 08:28 |
| dulek | sheel: Are you online maybe? | 08:28 |
| sheel | dulek: yep | 08:29 |
| sheel | dulek: tell me | 08:29 |
| dulek | sheel: Please read my conversation with zigo. | 08:29 |
| flip214 | hmmm, "run_tests.sh ..." doesn't find a python module - but "python cinder/tests/unit/..." does. Hrmpf. | 08:29 |
| zigo | Thanks ! :) | 08:29 |
| sheel | zigo: refer https://review.openstack.org/#/c/317283/ | 08:30 |
| dulek | sheel: If you could propose a commit to include non-minified versions of these bootstrap files I'll approve it right away. | 08:30 |
| zigo | Ah, super cool ! | 08:30 |
| sheel | dulek: I have already | 08:30 |
| sheel | dulek: refer https://review.openstack.org/#/c/317283/ | 08:30 |
| dulek | sheel: Oh. Even better! Reviewing. :) | 08:31 |
| sheel | dulek: but this depends on https://review.openstack.org/#/c/316587/ which is waiting for final +A only | 08:31 |
| zigo | FYI, we're not even packaging os-api-ref in Debian, it's not needed. | 08:31 |
| dulek | sheel, zigo: Okay, I'll review both now. | 08:32 |
| sheel | dulek: thank you.. | 08:32 |
| sheel | zigo: thanks for raising this.. this will help my patches to merge soon... :) | 08:32 |
| zigo | I'd very much appreciate a ping once all of this is merged, so I can upload Cinder Newton b1 (+ a few commits) to Debian experimental. | 08:32 |
| zigo | I'm putting this on hold in the mean while. | 08:32 |
| zigo | sheel: Awesome! :) | 08:32 |
| zigo | I was about to report the bug to launchpad, I guess it's not needed. | 08:33 |
| zigo | Should I? | 08:33 |
| sheel | zigo: not required.. | 08:33 |
| zigo | (I'm one click away, the form is already filled...) | 08:33 |
| dulek | zigo: We can do without that I think. | 08:33 |
| zigo | Ok. | 08:33 |
| *** salv-orlando has joined #openstack-cinder | 08:39 | |
| *** zhipeng has joined #openstack-cinder | 08:40 | |
| *** EinstCrazy has joined #openstack-cinder | 08:42 | |
| *** savihou has quit IRC | 08:48 | |
| *** savihou has joined #openstack-cinder | 08:49 | |
| *** laughterwym has joined #openstack-cinder | 08:49 | |
| *** yangyapeng has quit IRC | 08:51 | |
| dulek | sheel: I've added comment on 316587. Can you answer? | 08:53 |
| dulek | sheel: Are we right now just moving info from api-site and these aren't supposed to be 100% accurate yet? | 08:53 |
| sheel | dulek: ok, i am looking | 08:54 |
| sheel | dulek: no these must be 100% ok now | 08:54 |
| *** yangyapeng has joined #openstack-cinder | 08:56 | |
| *** salv-orlando has quit IRC | 08:56 | |
| *** guy has joined #openstack-cinder | 08:59 | |
| *** laughterwym has quit IRC | 09:01 | |
| *** laughterwym has joined #openstack-cinder | 09:02 | |
| sheel | dulek: I will have to update to remove one line... I | 09:03 |
| sheel | dulek: need clarification on one point | 09:03 |
| sheel | I'm just getting 302 Found as a response to /v2 endpoint. Why is that here? | 09:03 |
| sheel | dulek: I could not get meaning here.. | 09:04 |
| sheel | https://review.openstack.org/#/c/316587/5/api-ref/v2/source/api-versions.inc | 09:04 |
| *** e0ne has joined #openstack-cinder | 09:04 | |
| dulek | sheel: I've did GET on /v2 and received only 302 Found HTTP code. Without any body. And with your patch docs will say that in that case I will get same response as for / endpoint. | 09:05 |
| sheel | dulek: ok, let me check, will ping you back | 09:05 |
| dulek | e0ne: Can you read the scrollback starting from 10:23? | 09:06 |
| *** laughterwym has quit IRC | 09:07 | |
| *** locust has joined #openstack-cinder | 09:08 | |
| *** savihou has quit IRC | 09:08 | |
| *** savihou has joined #openstack-cinder | 09:09 | |
| dulek | e0ne: I think the best course of action here would be to get 317283 fixed (fails locally on my env), make it independent from 316587 and approve it to help zigo. What do you think? | 09:11 |
| *** mjura has quit IRC | 09:16 | |
| *** mjura has joined #openstack-cinder | 09:24 | |
| e0ne | dulek: sure. looking on it | 09:28 |
| *** locust has quit IRC | 09:29 | |
| *** guy has quit IRC | 09:29 | |
| *** nikeshm has quit IRC | 09:30 | |
| *** lpetrut has joined #openstack-cinder | 09:30 | |
| e0ne | dulek, zigo: what is the issue with mimified js files? they containt MIT licence headers | 09:32 |
| zigo | e0ne: This is an already built artifact, this is not source code. | 09:33 |
| zigo | e0ne: Therefore, it's non-free. | 09:33 |
| e0ne | zigo: it makes sense | 09:33 |
| zigo | e0ne: All downstream distros will refuse the whole package because it will be considered non-free. | 09:33 |
| e0ne | zigo: does any other openstack project have the same issue? | 09:36 |
| zigo | e0ne: Yeah, it's been years I'm constantly complaining about this kind of stuff... :) | 09:37 |
| e0ne | :) | 09:37 |
| zigo | Things get fixed on a case-by-case basis after I open bugs. | 09:37 |
| zigo | Generally speaking, people understand, though find it very annoying (me included, it annoys me but I follow the Debian rules...). | 09:38 |
| *** nikeshm has joined #openstack-cinder | 09:38 | |
| * zigo got to go for lunch | 09:39 | |
| e0ne | zigo: I'm agree with you | 09:45 |
| e0ne | sheel: do we really need that '_static' directory? what is the source of it? | 09:45 |
| *** mvk_ has quit IRC | 09:46 | |
| dulek | e0ne: That's os-api-site. sheel is removing that in 317283. | 09:47 |
| *** salv-orlando has joined #openstack-cinder | 09:48 | |
| flip214 | can "openstack volume list" return the IDs of servers the volume is attached to? I've got multiple servers with the same name... | 09:48 |
| nikeshm | dulex: how can i repeat the issue given by jenkins in http://logs.openstack.org/85/327285/3/check/gate-cinder-python27-db/3fba7cd/console.html | 09:49 |
| nikeshm | dulex: i tried to run unit tests on other machines, which do not have access to backend | 09:50 |
| nikeshm | they are passing | 09:50 |
| flip214 | ah, "openstack server show" returns the data | 09:52 |
| e0ne | dulek: so, everything we need is to merge that patch, right? | 09:52 |
| *** sgotliv_ has joined #openstack-cinder | 09:55 | |
| dulek | e0ne: That's right, but without the 316587 it will fail and there are mistakes in 316587. So i think we should fix 317283 by doing minimal required changes in it and then merge it. | 09:56 |
| *** takedakn has joined #openstack-cinder | 09:56 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Updated volume, version, limit, metadata API-ref https://review.openstack.org/316587 | 09:58 |
| sheel | dulek: mistakes are fixed ^^ | 10:00 |
| sheel | e0ne: we are updating that through 317283 | 10:00 |
| dulek | sheel: Sure, but I'm not sure if I'm comfortable enough to approve it without manually checking all the calls if they are compliant. I would prefer making 317283 independent of 316587 and including minimum required fixes in 316587. | 10:02 |
| dulek | That way we would just approve 317283 and continue review on 316587. | 10:03 |
| sheel | dulek: Let me find if i have command output with me which have details of manual output | 10:03 |
| sheel | dulek: give me 5 min | 10:03 |
| sheel | dulek: https://www.irccloud.com/pastebin/VDQ4tpyv/curl_calls.txt | 10:06 |
| dulek | sheel: What's that? | 10:08 |
| sheel | dulek: api calls | 10:08 |
| sheel | dulek: using curl | 10:08 |
| sheel | dulek: this contains output as well as commands with body | 10:08 |
| sheel | dulek: so will be easy to confirm | 10:08 |
| dulek | sheel: Suuure, but still to completely review 316587 it would take me hours. And I was able to find mistakes in it with just a superficial look, so I'm not comfortable with approving it without that diligent review. | 10:10 |
| dulek | And to help zigo we need 317283 as soon as possible. | 10:11 |
| sheel | dulek: thats ok, but believe me you found all what was missing... but yep its always good to fully sure... | 10:11 |
| sheel | dulek: ok, let me know what is expected from me now? | 10:11 |
| *** itzdilip has quit IRC | 10:11 | |
| sheel | dulek: happy to help | 10:11 |
| dulek | So I would say the best course of action is to make these to independent by including minimal set of needed fixes in 317283 and then continuing to review 316587. | 10:11 |
| dulek | (if "the minimal set of needed fixes" isn't really huge) | 10:12 |
| sheel | dulek: let me give a look.. | 10:12 |
| sheel | s/give/have | 10:12 |
| sheel | :) | 10:12 |
| sheel | dulek: ok, i think its easy.. let me submit it in next half an hour | 10:14 |
| *** huanan has quit IRC | 10:14 | |
| *** huanan has joined #openstack-cinder | 10:15 | |
| dulek | sheel: Thanks! :) | 10:16 |
| sheel | dulek: always welcome :) | 10:16 |
| *** zhipeng has quit IRC | 10:17 | |
| *** mvk_ has joined #openstack-cinder | 10:17 | |
| *** pgadiya has quit IRC | 10:27 | |
| sheel | api-ref: commands succeeded | 10:30 |
| sheel | congratulations :) | 10:30 |
| *** belmoreira has quit IRC | 10:31 | |
| dulek | sheel: Yay! Can you upload it? :) | 10:31 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: migrate to os-api-ref https://review.openstack.org/317283 | 10:31 |
| dulek | e0ne: ^ | 10:31 |
| sheel | dulek: here you go ^^ | 10:31 |
| dulek | sheel: Ah, you need also to remove Depends-On from the commit message. | 10:34 |
| sheel | dulek: oops | 10:34 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: migrate to os-api-ref https://review.openstack.org/317283 | 10:35 |
| sheel | dulek: done | 10:35 |
| dulek | sheel: Thank you. Let's wait for Jenkins and merge this. | 10:36 |
| sheel | dulek: np | 10:37 |
| *** kmartin has quit IRC | 10:38 | |
| *** yhayashi has quit IRC | 10:40 | |
| *** salv-orlando has quit IRC | 10:41 | |
| openstackgerrit | Pranali Deore proposed openstack/python-cinderclient: Removed 'u' from output of type-show command https://review.openstack.org/327583 | 10:44 |
| *** salv-orlando has joined #openstack-cinder | 10:49 | |
| *** iceyao has joined #openstack-cinder | 10:51 | |
| *** iceyao has quit IRC | 10:55 | |
| *** sdague has joined #openstack-cinder | 10:55 | |
| *** salv-orlando has quit IRC | 11:03 | |
| *** tpsilva has joined #openstack-cinder | 11:04 | |
| *** takedakn has quit IRC | 11:10 | |
| *** alyson_ has joined #openstack-cinder | 11:19 | |
| *** flip214 is now known as flip217 | 11:25 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Use oslo_middleware sizelimit https://review.openstack.org/314143 | 11:30 |
| *** salv-orlando has joined #openstack-cinder | 11:35 | |
| *** GB21 has quit IRC | 11:37 | |
| *** ducttape_ has joined #openstack-cinder | 11:40 | |
| *** bbhavsar is now known as bhavik | 11:50 | |
| *** sdake has joined #openstack-cinder | 11:56 | |
| *** ducttape_ has quit IRC | 11:56 | |
| *** GB21 has joined #openstack-cinder | 11:57 | |
| e0ne | tbarron: hi. are you around? | 12:00 |
| tbarron | e0ne: hi | 12:00 |
| *** scottda has joined #openstack-cinder | 12:01 | |
| e0ne | tbarron: looks like we're working on the same things for rbd connector | 12:01 |
| tbarron | e0ne: i am resubmitting 319553 as a parital fix this am | 12:01 |
| e0ne | tbarron: did you see my patch to os-brick? | 12:02 |
| tbarron | we absolutely need to pass that info in os_brick linuxrbd intiator | 12:02 |
| e0ne | tbarron: TBH, I didn't see your patch when submitted my | 12:02 |
| tbarron | it just isn't sufficient by itself to fix the general problem when ceph backup and volume use difft clusters | 12:02 |
| tbarron | e0ne: i will check your patch | 12:03 |
| e0ne | tbarron: here is it https://review.openstack.org/#/c/326696/4/os_brick/initiator/linuxrbd.py | 12:03 |
| tbarron | e0ne: looking | 12:03 |
| e0ne | tbarron: the problem is that cinder does not use rbdconnector now | 12:03 |
| *** EinstCrazy has quit IRC | 12:04 | |
| *** markvoelker has joined #openstack-cinder | 12:04 | |
| dulek | scottda: https://review.openstack.org/#/c/327619 = http://goo.gl/1xOL6j | 12:06 |
| *** salv-orlando has quit IRC | 12:06 | |
| dulek | scottda: But we need to start with something and I don't know how to test that locally. ;) | 12:06 |
| scottda | dulek: Cool.. | 12:07 |
| *** takedakn has joined #openstack-cinder | 12:09 | |
| scottda | dulek: I was reading the devstack-gate/README about "Simulating Devstack Gate Tests" and thought I might try to set that up. | 12:09 |
| dulek | scottda: I just need two VMs prepared like nodepool would do it, right? | 12:10 |
| dulek | scottda: And with proper networking. | 12:11 |
| scottda | I think so, yes. Although the docs had quite a few steps, but I think that's the gist of it. | 12:11 |
| scottda | dulek: It might be a bit easier for someone with a CI setup, since they probably already have that infrastructure setup. But I don't have CI... | 12:12 |
| dulek | scottda: I think they have just a single node, so it's different. | 12:13 |
| * dulek sets off on a journey to find a functional local OpenStack cloud. | 12:13 | |
| openstackgerrit | nikesh proposed openstack/cinder: Provide Kaminario K2 all-flash array iSCSI driver https://review.openstack.org/327285 | 12:13 |
| *** GB21 has quit IRC | 12:16 | |
| *** huyang_ has joined #openstack-cinder | 12:17 | |
| dulek | nikeshm: Ah, I've missed your ping. I think it's not tox checking that no commands are run, but some higher level infra stuff, so it's irreproducible locally. | 12:17 |
| tbarron | e0ne: i do see that the patches overlap. I'm going to go on and refresh mine in case reviewers end up agreeing with Hope-Marly that we should just first make the change to get backwards compatibility, then separatedly add new properties (e.g. cluster name and connect_timeout). | 12:18 |
| dulek | nikeshm: But as I've said - adding executable cat file with "exit 1" in your .tox/py27/bin should at least make tests explode where it's called. | 12:18 |
| tbarron | e0ne: but I'll happily abandon mine if reviewers want to do it all in one go | 12:18 |
| *** wanghao has quit IRC | 12:20 | |
| *** huyang has quit IRC | 12:20 | |
| *** huyang_ is now known as huyang | 12:20 | |
| *** belmoreira has joined #openstack-cinder | 12:20 | |
| *** wanghao has joined #openstack-cinder | 12:20 | |
| dulek | e0ne: https://review.openstack.org/#/c/317283/ - Jenkins just passed and I've tested it successfully locally. | 12:20 |
| tbarron | e0ne: if you are adding new properties, i think keyring path might be good too. | 12:21 |
| *** EinstCrazy has joined #openstack-cinder | 12:24 | |
| lennyb | Hi Cinder, after running tempest and devstack ./clean.sh I have some LVM volumes left 'in use' #http://paste.openstack.org/show/509186/ how can I clean them ? Thanks | 12:26 |
| *** yangyapeng has quit IRC | 12:26 | |
| *** yangyapeng has joined #openstack-cinder | 12:27 | |
| e0ne | tbarron: for now, I've just synced with cinder's code to remove duplication | 12:31 |
| e0ne | tbarron: I'm going to continue work on it after this on will be merged | 12:31 |
| sheel | lennyb: volumes can be easily removed through lvremove | 12:31 |
| sheel | lennyb: but i dont know what is use case on your end.. you are using existing environment or cleaned everything out of database.. | 12:32 |
| tbarron | e0ne: sure, I'll put myself as reviewer on your patch and follow it. I appreciate your calling my attention to it. | 12:32 |
| e0ne | tbarron: np. I'm always open to communications and good reviews | 12:32 |
| sheel | lennyb: I think clean.sh clears whole environment as well so there is no issue in running lvremove on local environment | 12:33 |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: Update provider_location during snapshot backup https://review.openstack.org/327375 | 12:35 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/os-brick: Make RBDImageMetadata and RBDVolumeIOWrapper re-usable https://review.openstack.org/326696 | 12:35 |
| tomoiaga | scottda: the yesterday issue, where deleting an instance did not release the volume happened because I used the cinder client attach and detach commands (I had my reasons). More info here, seems an old issue: http://www.florentflament.com/blog/openstack-volume-in-use-although-vm-doesnt-exist.html | 12:37 |
| scottda | tomoiaga: Ahhh..that would explain it. | 12:38 |
| scottda | tomoiaga: You can cleanup the cinder DB state with the 'cinder reset-state ....' command | 12:39 |
| tomoiaga | scottda: back when I used the cinderclient, nova required me to give it a device, and I didn't want to do that so I used the cinder client :) | 12:39 |
| openstackgerrit | Tom Barron proposed openstack/os-brick: Fix ceph backup failure to do incremental backup https://review.openstack.org/319553 | 12:39 |
| tomoiaga | scottda: indeed | 12:39 |
| scottda | tomoiaga: That need to use a device should be removed, as it is not really honored by Nova anyway. | 12:40 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/os-brick: Make RBDImageMetadata and RBDVolumeIOWrapper re-usable https://review.openstack.org/326696 | 12:40 |
| tomoiaga | scottda: it seems to be working fine in Nova in Mitaka now without the device (it is set to null, probably the guys using XenServer needed it) | 12:40 |
| sheel | dulek: https://review.openstack.org/#/c/325847/ looking for +A, its only 3 lines of change thats also cherry picked.. | 12:41 |
| scottda | tomoiaga: OK. I thought there was effort to remove the need for a dev. I guess it went in. | 12:41 |
| tomoiaga | scottda: yes, it's ok now | 12:42 |
| *** rlrossit has joined #openstack-cinder | 12:42 | |
| *** sdake has quit IRC | 12:48 | |
| *** JoseMello has joined #openstack-cinder | 12:48 | |
| dulek | sheel: I'm not in cinder-stable-maint, you need to ask someone from this list: https://review.openstack.org/#/admin/groups/534,members | 12:48 |
| lennyb | sheel: Cinder CI, same physical server running ./stack.sh; tempest; ./clean.sh . After about 100 runs some of the volumes are not being cleaned up, so that ./stack.sh starts to fail. # http://13.69.151.247/30/260930/30/check-cinder/Cinder-ISER-ISCSI/c0319a9/logs/stack.sh.log.gz since lvremove cant remove the volumes. | 12:49 |
| e0ne | tbarron: now, your patch has the same code as my :( | 12:49 |
| sheel | dulek: oh sure... I was not aware of this.. too many things to learn everyday.. | 12:49 |
| sheel | dulek: thanks | 12:49 |
| *** salv-orlando has joined #openstack-cinder | 12:49 | |
| tbarron | e0ne: not the same code, you are passing more properties | 12:50 |
| sheel | lennyb: oops.. any permission related error? | 12:50 |
| e0ne | tbarron: yes, I mean: the same behaviour | 12:50 |
| sheel | lennyb: ah...inuse...then may be lvremove force can be used | 12:51 |
| *** julim has joined #openstack-cinder | 12:51 | |
| tbarron | e0ne: it always had the same behavior by that definition | 12:51 |
| sheel | lvremove --force | 12:51 |
| tbarron | e0ne: are you asking me to abandon it? I don't really care. | 12:51 |
| sheel | lennyb: oh i see -f is also failing | 12:51 |
| sheel | sudo lvremove -f stack-volumes-lvmdriver-1 | 12:52 |
| sheel | 2016-06-09 07:19:24.119 | Logical volume stack-volumes-lvmdriver-1/volume-786f3310-ce5c-448b-b869-1a6762b2d973 in use. | 12:52 |
| sheel | e0ne: do we explicitly unmount volumes before removing them | 12:53 |
| *** akshai has joined #openstack-cinder | 12:53 | |
| e0ne | tbarron: TBH, I don't know what is the better. let's wait for reviews | 12:53 |
| sheel | e0ne: I am sure we dont do this way... but just asking if have experienced this kind of thing before... | 12:53 |
| tbarron | e0ne: sure, I just fixed mine up so that I wasn't ignoring the previous review comments. You guys let me know if I should abandon it. | 12:54 |
| e0ne | sheel: we can remove attached volumes onlu with 'force' flag | 12:54 |
| sheel | e0ne: but -f is also failing | 12:55 |
| tbarron | e0ne: we still have to address the harder problems that remain for multi-ceph-cluster backup, whichever way people decide to go on the brick patch | 12:55 |
| e0ne | sheel: I mean 'cinder delete vol --force' | 12:55 |
| e0ne | lvremove can't remove volume if it is used now | 12:56 |
| *** locust has joined #openstack-cinder | 12:56 | |
| sheel | lennyb: ^ | 12:56 |
| sheel | e0ne: thanks for pointer | 12:56 |
| locust | hello | 12:56 |
| *** links has quit IRC | 12:57 | |
| lennyb | sheel: is it possible to 'clean' or 'uninstall' the all database? No one use those volumes. | 12:57 |
| e0ne | sheel: you're welcome | 12:58 |
| e0ne | tbarron: I've commented your patch with my thoughts | 12:58 |
| e0ne | tbarron: btw, did you see this https://etherpad.openstack.org/p/cinder-newton-rbd-refactoring etherpad? | 12:59 |
| tbarron | e0ne: no, thanks for the pointer! | 13:00 |
| e0ne | tbarron: what issues with multi-ceph-cluster do you mean? I even didn't test cinder with such configuration | 13:00 |
| sheel | lennyb: there could be other problems in system if we clean whole database | 13:00 |
| *** avishay has quit IRC | 13:00 | |
| sheel | lennyb: as it will delete all things not only volumes | 13:00 |
| sheel | lennyb: database can be recreated though but i dont know what will be aftermaths of that... | 13:01 |
| tbarron | e0ne: make two devstack nodes, run c-vol one one and c-bak on the other. configure each to work with different ceph clusters (not just separate pools). Run cinder backup. Bang! | 13:01 |
| lennyb | sheel: what things? This is a CI server | 13:01 |
| tbarron | e0ne: this should be in our backup tests. | 13:01 |
| tbarron | this scenario. | 13:01 |
| e0ne | tbarron: +2 | 13:01 |
| lennyb | sheel: btw, reboot 'fixes' the issue | 13:02 |
| sheel | lennyb: great | 13:02 |
| *** salv-orlando has quit IRC | 13:03 | |
| *** salv-orlando has joined #openstack-cinder | 13:04 | |
| *** akerr has joined #openstack-cinder | 13:07 | |
| lennyb | sheel: this issue is reproduces every 2 weeks or so, so I prefer solving it with a script :). Maybe there are some processes that are still running? What should I look for? None of the python procs are running | 13:07 |
| sheel | lennyb: did you tried solution suggested by e0ne : to use "cinder delete vol --force"? | 13:08 |
| sheel | lennyb: if that works may be you can create script around that | 13:08 |
| sheel | lennyb: also volumes comes in-use when they are attached with some instance... so, may be some tests are attaching volumes with instance but not passing delete on detach flag with that | 13:09 |
| *** zhonghua-lee has quit IRC | 13:10 | |
| sheel | lennyb: this might be the reason for this problem | 13:10 |
| lennyb | sheel: e0ne: thanks. I will try and check it | 13:10 |
| sheel | sheel: sure, please let me know in case you stuck... it seems interesting to fix this... | 13:10 |
| *** zhonghua-lee has joined #openstack-cinder | 13:10 | |
| *** merooney has joined #openstack-cinder | 13:11 | |
| *** rlrossit has quit IRC | 13:13 | |
| *** salv-orlando has quit IRC | 13:15 | |
| *** erlon has joined #openstack-cinder | 13:17 | |
| *** links has joined #openstack-cinder | 13:17 | |
| flip217 | "openstack volume list" => "Attached to None on None". Rrrrrright! | 13:20 |
| *** porrua has joined #openstack-cinder | 13:20 | |
| *** edmondsw has joined #openstack-cinder | 13:22 | |
| *** links has quit IRC | 13:23 | |
| locust | hello everybody, I have a question about behaviour of cinder driver | 13:23 |
| locust | for instance: I have a volume and its snapshot | 13:24 |
| *** gouthamr has joined #openstack-cinder | 13:24 | |
| dulek | scottda: \o/ | 13:24 |
| dulek | scottda: Looks like in multinode with that patch to devstack-gate we have two c-vols on different nodes and volumes are placed on both. | 13:24 |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Updated volume, version, limit, metadata API-ref https://review.openstack.org/316587 | 13:25 |
| *** laughterwym has joined #openstack-cinder | 13:25 | |
| locust | then I am creating a volume from the clone | 13:25 |
| locust | and after that I am deleting source snapshot | 13:25 |
| locust | some drivers failed to do this, some ok | 13:26 |
| locust | but what the "original driver do"? | 13:26 |
| locust | but what shuold the "original driver do"? | 13:27 |
| *** ducttape_ has joined #openstack-cinder | 13:27 | |
| scottda | dulek: Sweet! | 13:28 |
| locust | may be you can give me doc to dig in? | 13:28 |
| scottda | dulek: I had done some brainstorming with PatrickE about how to Fake a common backend, maybe with NFS....maybe later when he is online (or some other day) we can talk about that. | 13:29 |
| *** nkrinner has quit IRC | 13:29 | |
| *** laughterwym has quit IRC | 13:29 | |
| *** laughterwym has joined #openstack-cinder | 13:29 | |
| *** mriedem has joined #openstack-cinder | 13:29 | |
| scottda | Something like that would be good/required for HA testing | 13:29 |
| sheel | locust: AFAIK, there should not be dependency for deleting snapshot after cloning volume from it.. | 13:30 |
| *** rlrossit has joined #openstack-cinder | 13:30 | |
| dulek | scottda: But… Why not ceph? | 13:30 |
| scottda | dulek: Yes, that's a good idea... | 13:31 |
| scottda | dulek: I'm just a former NFS guy, but Ceph is probably a better choice. | 13:31 |
| locust | sheel --- it is because of usege zfs | 13:31 |
| locust | and its native cloning features | 13:31 |
| dulek | scottda: I mean NFS explodes harder when we're messing with it without synchronization, but it will be crazy-non-deterministic to fail. | 13:32 |
| sheel | locust: ohk... so thats driver specific behavior... | 13:32 |
| sheel | locust: I think lvm will allow to delete that.. | 13:32 |
| sheel | locust: may be you can try this ... | 13:32 |
| scottda | dulek: Yeah, NFS syncing is unreliable. It's not the best choice, just the first thing that popped into my head. | 13:32 |
| dulek | scottda: And for HA we would need to setup redis/zookeeper for DLM. Fortunately that's supported in DevStack, so we should be fine. | 13:33 |
| *** nkrinner has joined #openstack-cinder | 13:33 | |
| *** jgregor has quit IRC | 13:33 | |
| *** laughterwym has quit IRC | 13:33 | |
| locust | sheel: yep, I am developing driver for hardware and I do not know what I am supposed to do --- like what is "RIGHT" | 13:34 |
| sheel | locust: well as per general behavior, it should not prohibit to delete source snapshot | 13:35 |
| sheel | locust: there seems no usage of keeping source snapshot for volume been cloned untill unless we dont have any specific use case for that | 13:35 |
| savihou | 026 | 13:36 |
| locust | sheel: yes, according to lvm driver as a standard | 13:37 |
| sheel | locust: may be you can replicate lvm behavior for features where you dont have any specific requirements.. | 13:38 |
| openstackgerrit | Merged openstack/cinder: migrate to os-api-ref https://review.openstack.org/317283 | 13:38 |
| sheel | locust: will be fast enough for you to develop base driver | 13:38 |
| sheel | locust: later on you can update that once it get merged | 13:38 |
| sheel | locust: as per specific requirements | 13:38 |
| *** sdake has joined #openstack-cinder | 13:39 | |
| smcginnis | locust: Which driver are you working on? | 13:39 |
| *** liverpooler has quit IRC | 13:42 | |
| *** eharney has joined #openstack-cinder | 13:43 | |
| dulek | zigo: https://review.openstack.org/317283 is merged now. | 13:43 |
| locust | it is new Open-e driver | 13:47 |
| locust | liverpooler: for their product JovianDSS | 13:48 |
| smcginnis | Oh, interesting. New one for me. Thanks! | 13:48 |
| zigo | dulek: Awesome, thanks! | 13:48 |
| smcginnis | locust: If you are planning on trying to get this in Newton yet, make sure you get a blueprint filed and follow all the requirements here https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver | 13:49 |
| *** ametts has joined #openstack-cinder | 13:49 | |
| locust | sheel: I have taken nexenta driver as base reference and now implementing all "must" features | 13:49 |
| sheel | smcginnis: ^ | 13:49 |
| *** laughterwym has joined #openstack-cinder | 13:50 | |
| smcginnis | That's probably a good one to follow. IIRC, that is ZFS based as well. | 13:50 |
| locust | smcginnis: as I understand I can release driver only with openstack version release | 13:51 |
| locust | ? | 13:51 |
| smcginnis | locust: Yes, to be included as an official Cinder driver. | 13:52 |
| *** lprice has quit IRC | 13:54 | |
| *** jgregor has joined #openstack-cinder | 13:55 | |
| locust | smcginnis: thank you. I am reading now "contribute requirements" pretty much staff. | 13:55 |
| locust | As I understand I can be helped with here #openstack-dev | 13:57 |
| *** Nag_ has joined #openstack-cinder | 13:58 | |
| locust | sheel: thank you | 13:59 |
| locust | sheel: you mean than I can contribute driver to mitaka step by step? | 14:00 |
| smcginnis | I have a couple easy patches I wouldn't mind getting out of the way if anyone has a moment. :) | 14:01 |
| smcginnis | https://review.openstack.org/#/q/owner:sean.mcginnis%2540gmail.com+status:open | 14:01 |
| smcginnis | locust: New drivers would just go in for Newton. We don't backport new drivers to stable branches. | 14:01 |
| smcginnis | locust: So if you are able to get everything done in time, the soonest an official driver would be available would be with the next OpenStack release at the beginning of October. | 14:02 |
| smcginnis | http://releases.openstack.org/newton/schedule.html | 14:02 |
| sheel | locust: currently its newton, so target for newton | 14:03 |
| locust | smcginnis: I see, so I fallow all instructions for https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver with Newton as source | 14:03 |
| *** merooney_ has joined #openstack-cinder | 14:03 | |
| sheel | locust: and yes, its good to break in parts | 14:04 |
| smcginnis | locust: Yes, just use the current "master" branch. That is what will become the Newton release. | 14:04 |
| locust | sheel: oh, I see | 14:05 |
| *** meroone__ has joined #openstack-cinder | 14:05 | |
| locust | smcginnis: ok just cinder master | 14:05 |
| *** merooney has quit IRC | 14:06 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Refactor sqlalchemy service methods https://review.openstack.org/286598 | 14:07 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Example of RPC calls moved to HA A-A https://review.openstack.org/318573 | 14:07 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add cluster table and related methods https://review.openstack.org/318572 | 14:07 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Update Versioned Objects with Cluster object https://review.openstack.org/327686 | 14:07 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add cluster job distribution https://review.openstack.org/327687 | 14:07 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Update manage with cluster related commands https://review.openstack.org/327688 | 14:07 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Modify API to include cluster related operations https://review.openstack.org/327689 | 14:07 |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: Add cluster related commands https://review.openstack.org/327692 | 14:08 |
| *** merooney_ has quit IRC | 14:08 | |
| locust | Thenk you! Gues! sheel smcginnis you are extreamly helpfull!!! | 14:10 |
| sheel | locust: my pleasure | 14:11 |
| *** GB21 has joined #openstack-cinder | 14:11 | |
| smcginnis | locust: No problem. Feel free to ask any questions if you need clarification on anything. | 14:11 |
| smcginnis | geguileo: How's that stuff going? ^^ | 14:11 |
| geguileo | smcginnis: Slow :-( | 14:11 |
| geguileo | smcginnis: That's the stuff that I had to redo | 14:12 |
| smcginnis | geguileo: Yeah, you're pushing it along though! | 14:12 |
| smcginnis | geguileo: Oh, from the discussion in Austin? | 14:12 |
| geguileo | smcginnis: Yes | 14:12 |
| geguileo | smcginnis: I had to redo them completely | 14:12 |
| locust | may be you can advise me any openstack meating from the schedull? where I can find cinder developers in flash? | 14:13 |
| geguileo | Now I hope I can reuse most of what I have from the cleanup patches | 14:13 |
| smcginnis | geguileo: Hopefully we can get some movement now. Would be nice to have that done. | 14:13 |
| smcginnis | geguileo: And hopefully no more rewrites! :) | 14:13 |
| geguileo | lol | 14:13 |
| geguileo | Yeah, I keep crossing my fingers for no more rewrites | 14:13 |
| geguileo | XD | 14:13 |
| smcginnis | locust: This channel usually has someone around. | 14:14 |
| smcginnis | geguileo: Hopefully rebases are minimal too. ;) | 14:14 |
| smcginnis | locust: Most activity is around US timezones, but we have folks around the world. | 14:14 |
| locust | I am from Belarus | 14:14 |
| geguileo | smcginnis: I know there are some changes that I need to do to those patches, I just hope that there are no hidden stuff I need to change | 14:15 |
| smcginnis | locust: We have a meeting once a week that would catch most people online: https://wiki.openstack.org/wiki/CinderMeetings | 14:15 |
| locust | but interested in Germany Poland | 14:15 |
| *** laughterwym has quit IRC | 14:15 | |
| locust | or any other central Europe | 14:15 |
| geguileo | locust: I am in Germany TZ | 14:15 |
| geguileo | And there are other people working around this TZ | 14:16 |
| locust | geguileo: nice to heare this | 14:17 |
| locust | so I am gona be following weekly mittings | 14:18 |
| geguileo | locust: That's a good idea | 14:19 |
| locust | geguileo: (^-^)/ | 14:20 |
| geguileo | locust: I would recommend you look at the topics beforehand so you can try to look at the topics before the meeting | 14:20 |
| geguileo | It's easier to follow the conversations that way | 14:20 |
| *** GB21 has quit IRC | 14:21 | |
| eharney | smcginnis: are there any particular things to know about backporting reno notes to a stable branch? | 14:23 |
| *** takedakn has quit IRC | 14:24 | |
| eharney | hadn't really thought about it, i guess they end up being used the same way there? | 14:24 |
| smcginnis | eharney: Yep, it just does it's git history inspection magic to put the release notes in the right place. | 14:25 |
| smcginnis | eharney: So should be able to just backport without any changes. | 14:25 |
| eharney | smcginnis: cool | 14:25 |
| smcginnis | eharney: Yeah, glad that didn't have any complication caveats to be aware of. We have enough of those already. | 14:25 |
| eharney | haha | 14:25 |
| *** lprice has joined #openstack-cinder | 14:27 | |
| *** tongli has joined #openstack-cinder | 14:28 | |
| *** ccesario has quit IRC | 14:29 | |
| *** arecknag has quit IRC | 14:34 | |
| *** ccesario has joined #openstack-cinder | 14:34 | |
| *** laughterwym has joined #openstack-cinder | 14:35 | |
| *** diablo_rojo has joined #openstack-cinder | 14:37 | |
| *** laughterwym has quit IRC | 14:39 | |
| *** wanghao has quit IRC | 14:41 | |
| *** dustins has joined #openstack-cinder | 14:41 | |
| *** wanghao has joined #openstack-cinder | 14:42 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Handle API NotFound exceptions at WSGI level https://review.openstack.org/314193 | 14:42 |
| *** jungleboyj has joined #openstack-cinder | 14:42 | |
| *** Poornima has quit IRC | 14:42 | |
| *** akshai has quit IRC | 14:56 | |
| *** suyash has joined #openstack-cinder | 14:58 | |
| *** cknight has joined #openstack-cinder | 14:59 | |
| *** jgregor has quit IRC | 14:59 | |
| *** mtanino has joined #openstack-cinder | 15:00 | |
| *** jgregor has joined #openstack-cinder | 15:00 | |
| *** akshai has joined #openstack-cinder | 15:01 | |
| *** tomoiaga has quit IRC | 15:02 | |
| *** nkrinner has quit IRC | 15:03 | |
| *** jgregor1 has joined #openstack-cinder | 15:04 | |
| *** jgregor1 has quit IRC | 15:04 | |
| *** jgregor1 has joined #openstack-cinder | 15:05 | |
| *** EinstCrazy has quit IRC | 15:05 | |
| *** jgregor has quit IRC | 15:06 | |
| *** akshai_ has joined #openstack-cinder | 15:07 | |
| *** julim has quit IRC | 15:09 | |
| *** akshai has quit IRC | 15:11 | |
| *** Nag_ has quit IRC | 15:20 | |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: Move test from test_glusterfs and test_quobyte to test_remotefs https://review.openstack.org/261549 | 15:23 |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: vzstorage: add ploop volume format support https://review.openstack.org/262246 | 15:23 |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: vzstorage: fix create/delete snapshots of in-use volumes https://review.openstack.org/276465 | 15:23 |
| openstackgerrit | Evgeny Antyshev proposed openstack/cinder: Use versionedobjects in remotefs.py https://review.openstack.org/261558 | 15:23 |
| *** hemnafk is now known as hemna | 15:27 | |
| *** tongli has quit IRC | 15:28 | |
| *** belmoreira has quit IRC | 15:29 | |
| *** tesseract has quit IRC | 15:30 | |
| *** xinli has joined #openstack-cinder | 15:31 | |
| *** rcernin has quit IRC | 15:32 | |
| *** asselin_ has quit IRC | 15:33 | |
| *** GB21 has joined #openstack-cinder | 15:33 | |
| *** arch-nemesis has joined #openstack-cinder | 15:34 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Retry unhandled exception REST Gets https://review.openstack.org/325506 | 15:36 |
| *** suyash has quit IRC | 15:38 | |
| *** jdillaman has quit IRC | 15:41 | |
| *** ebalduf has joined #openstack-cinder | 15:45 | |
| *** mjura has quit IRC | 15:53 | |
| *** merooney has joined #openstack-cinder | 15:54 | |
| *** suyash has joined #openstack-cinder | 15:55 | |
| *** avishay has joined #openstack-cinder | 15:56 | |
| *** meroone__ has quit IRC | 15:56 | |
| *** Apoorva has joined #openstack-cinder | 15:56 | |
| *** Apoorva has quit IRC | 15:57 | |
| *** locust has quit IRC | 15:58 | |
| *** Apoorva has joined #openstack-cinder | 15:58 | |
| *** raunak has joined #openstack-cinder | 16:01 | |
| *** jgregor has joined #openstack-cinder | 16:04 | |
| *** jgregor1 has quit IRC | 16:05 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/327362 | 16:07 |
| *** avishay has quit IRC | 16:10 | |
| raunak | Hi, Is there a way to directly run host side commands from within the cinder driver of a vendor ? For eg. iscsi logout ? | 16:11 |
| *** GB21 has quit IRC | 16:12 | |
| *** merooney_ has joined #openstack-cinder | 16:12 | |
| *** karthikp_ has joined #openstack-cinder | 16:13 | |
| smcginnis | raunak: That shouldn't be done from within the driver. | 16:15 |
| *** leeantho has joined #openstack-cinder | 16:15 | |
| *** merooney has quit IRC | 16:15 | |
| raunak | scmginnis: In terminate connection of the driver with multipath enabled, syslogs keeps printing iscsid: conn 0 login rejected: | 16:17 |
| raunak | initiator error (02/04). This is because volume is unmapped from the host but the iscsi transport keeps polling for the connection to be recovered | 16:17 |
| smcginnis | raunak: If you need some special handling when attaching a volume you may need to implement a vendor specific connector. http://docs.openstack.org/developer/os-brick/api/os_brick/initiator/connector.html | 16:17 |
| smcginnis | raunak: Was it not detached first? | 16:18 |
| raunak | The usecase is copy image to volume . Does that involves detach as well ? | 16:19 |
| smcginnis | raunak: It should... | 16:20 |
| *** savihou has quit IRC | 16:20 | |
| raunak | In either case will look at os_brick for our specific implementation. Thanks | 16:21 |
| smcginnis | raunak: np | 16:21 |
| *** avishay has joined #openstack-cinder | 16:21 | |
| *** jsheeren has joined #openstack-cinder | 16:23 | |
| *** laughterwym has joined #openstack-cinder | 16:24 | |
| jsheeren | hi all, i have a question about cinder and nfs shared storage permissions | 16:24 |
| jsheeren | when i create a volume, it is owend by cinder:cinder | 16:24 |
| jsheeren | i can extend the volume without issues | 16:24 |
| jsheeren | when i attach the volume to an instance, ownership changes to libvirt-qemu:kvm | 16:24 |
| jgriffith | smcginnis: unique connectors are EViL | 16:25 |
| *** timcl has quit IRC | 16:25 | |
| jsheeren | i can mount it, create filesystems etc | 16:25 |
| jgriffith | They should die in a fire | 16:25 |
| *** sdake has quit IRC | 16:25 | |
| *** sheel has quit IRC | 16:25 | |
| jsheeren | when i unmount it, then detach it from the instance; the ownership changes to nobody:nogroup (i think nogroup, it's a ridiculously large gid) | 16:25 |
| smcginnis | jgriffith: Yeah, but unfortunately a necessary evil. | 16:25 |
| jsheeren | i cannot extend this volume anymore, permission denied, which is correct 'cause cinder user cannot access it | 16:26 |
| jsheeren | but! i can still attach it to instances and format it, create filesystems, etc | 16:26 |
| jgriffith | smcginnis: true statement | 16:26 |
| jsheeren | how can i fix this so i can still extend detached volumes | 16:26 |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Add test type clarification to devref https://review.openstack.org/327806 | 16:26 |
| jsheeren | how can i fix the ownership? | 16:27 |
| jsheeren | anyone seen this before? | 16:27 |
| jsheeren | i'm on the mitaka release btw | 16:27 |
| smcginnis | jsheeren: Sorry, haven't done much with NFS myself. There's usually a few folks around here that have some pretty good knowledge on permissions. | 16:27 |
| jgriffith | bswartz: weren't you working on the permissions swapping for NFS at one point? | 16:28 |
| jgriffith | bswartz: or maybe it was eharney ? | 16:28 |
| jsheeren | smcginnis: thanks, i was hopeing someone could point me in the right direction. i'm thinking it's the nfs export settings, but not sure (and cannot play with the settings, our storage guy must do this) | 16:28 |
| smcginnis | jsheeren: Yeah, that's what stuff like that usually comes down to. | 16:29 |
| smcginnis | jgriffith: I'm hoping to play around with your attach workflow this afternoon. Just at a high level glance, I like the direction so far. | 16:29 |
| jgriffith | smcginnis: cool... I'm working out the nova side of it right now, so maybe have something for you to test with other than manual calls later today | 16:30 |
| smcginnis | jgriffith: That would be huge, IMO. If we can show a working end to end solution that's going to be a lot easier. | 16:30 |
| jgriffith | smcginnis: I'll hollar when I push that | 16:30 |
| smcginnis | jgriffith: Awesome | 16:30 |
| jgriffith | it'll be later today or tomorrow | 16:30 |
| jgriffith | my goal is before the week-end if I can avoid interruptions/meetings :) | 16:31 |
| smcginnis | [jgriffith translator: it'll be tomorrow or some time next week] | 16:31 |
| smcginnis | :D | 16:31 |
| jgriffith | smcginnis: touchet | 16:31 |
| jsheeren | smcginnis: i'm looking at: https://blueprints.launchpad.net/cinder/+spec/secure-nfs where you marked it as obsolete. if i read point 1, a) "root squash is enabled on the NFS server" i think this is what i'm missing | 16:31 |
| smcginnis | hehe | 16:31 |
| smcginnis | jsheeren: Hmm, that was a batch bug close for old bugs sitting out there. :/ | 16:32 |
| smcginnis | jsheeren: But hopefully that has the clues you need. | 16:33 |
| smcginnis | jsheeren: But from my limited NFS experience, root squash has been a thing impacting others. | 16:34 |
| jsheeren | smcginnis: ah ok | 16:34 |
| *** avishay has quit IRC | 16:34 | |
| smcginnis | Ah, looks like the bug just didn't get linked properly. The patch merged: https://review.openstack.org/#/c/107693/ | 16:34 |
| *** porrua has quit IRC | 16:35 | |
| Anticimex | I just upgraded cinder in our test-deployment (centos EL7, rdo rpms) from liberty to mitaka, and ran into this: https://ask.openstack.org/en/question/93243/mitaka-cinder-ssl-issue/ | 16:36 |
| Anticimex | same issue, cinder-api ignoring SSL options and starting wsgi event listener in cleartext only | 16:36 |
| *** crose has joined #openstack-cinder | 16:37 | |
| *** sdake has joined #openstack-cinder | 16:38 | |
| *** jwcroppe has quit IRC | 16:38 | |
| *** ntpttr_ has quit IRC | 16:39 | |
| jgriffith | Anticimex: I'm not certain, but I don't think we've ever pulled the SSL options in Cinder's wsgi layer | 16:40 |
| *** ntpttr has quit IRC | 16:40 | |
| jgriffith | Anticimex: the only place we're honoring them I think is the glance image setup and a number of drivers | 16:41 |
| jgriffith | Anticimex: I can try and fix that | 16:41 |
| *** ntpttr has joined #openstack-cinder | 16:42 | |
| jgriffith | Anticimex: I could be wrong about that... maybe we pick it up from the oslo.wsgi code | 16:44 |
| *** sdake_ has joined #openstack-cinder | 16:45 | |
| *** sdake has quit IRC | 16:45 | |
| *** eharney has quit IRC | 16:45 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Retry unhandled exception REST Gets https://review.openstack.org/325506 | 16:47 |
| Swanson | @$#%#$!!!!! | 16:47 |
| jgriffith | Anticimex: hmm... actually we are trying to pull it directly via httplib2 | 16:48 |
| *** e0ne has quit IRC | 16:48 | |
| jgriffith | Anticimex: but I'm unclear how it would work as we don't check the config on any init routines to feed it back in | 16:49 |
| *** eharney has joined #openstack-cinder | 16:49 | |
| jgriffith | Anticimex: ahh.... https://bugs.launchpad.net/devstack/+bug/1533813 | 16:51 |
| openstack | Launchpad bug 1533813 in devstack "Cinder API server disagrees with USE_SSL=True" [Undecided,In progress] - Assigned to Guang Yee (guang-yee) | 16:51 |
| jgriffith | Anticimex: you may want to have a look at this: http://lists.openstack.org/pipermail/openstack/2015-August/013626.html | 16:51 |
| *** sdake_ has quit IRC | 16:52 | |
| Anticimex | jgriffith: it works in the liberty setup | 16:54 |
| jgriffith | Anticimex: interesting | 16:55 |
| Anticimex | but if mod_wsgi is the supported way to go, i guess puppet-cinder can roll that out | 16:55 |
| Anticimex | though others will run into this as well (as shown by ask.openstack.org above :) ) | 16:55 |
| jgriffith | Anticimex: Yeah, my point was I tihnk it's a Cinder bug that needs addressed perhaps new in M | 16:56 |
| Anticimex | so it just works by accident in liberty sort of? :) | 16:57 |
| *** diogogmt has joined #openstack-cinder | 16:57 | |
| jgriffith | Anticimex: haha... yeah | 16:57 |
| jgriffith | so there was a change to eventlet_server.py in M that removed it | 16:57 |
| jgriffith | tracking down the commit now | 16:57 |
| *** laughterwym has quit IRC | 16:57 | |
| *** laughterwym has joined #openstack-cinder | 16:58 | |
| Anticimex | cool | 16:58 |
| *** mvk_ has quit IRC | 16:59 | |
| Anticimex | liberty: 2016-06-09 18:54:21.663 4708 INFO eventlet.wsgi.server [-] (4708) wsgi starting up on https://0.0.0.0:8776 | 16:59 |
| jgriffith | Anticimex: I think this is the culprit: https://github.com/openstack/cinder/commit/b4c8bb3912972a4417c9efa93cb209f5efe76a14 | 16:59 |
| Anticimex | mitaka: 2016-06-09 18:25:28.911 26920 INFO eventlet.wsgi.server [-] (26920) wsgi starting up on http://0.0.0.0:8776 | 16:59 |
| Anticimex | identical config file | 16:59 |
| Anticimex | jgriffith: ah, let's see | 17:00 |
| jgriffith | Anticimex: so that pushed the eventlet server up into the oslo code and removed the ssl portions | 17:00 |
| jgriffith | Anticimex: I think what we need to do is write an init for wsgi in Cinder that takes the option and passes it up | 17:00 |
| jgriffith | Anticimex: probably the right thing to do, just not quite *done* | 17:01 |
| *** timcl has joined #openstack-cinder | 17:01 | |
| Anticimex | allright, oslo.service doesn't have generic support for ssl options? | 17:01 |
| Anticimex | or is that what you mean, to feed *that* with appropriate options? | 17:02 |
| *** laughterwym has quit IRC | 17:02 | |
| *** porrua has joined #openstack-cinder | 17:04 | |
| *** angela-s has joined #openstack-cinder | 17:06 | |
| karthikp_ | Please could somebody review my change in their free time..highly appreicate it : https://review.openstack.org/#/c/322977/ | 17:09 |
| *** jwcroppe has joined #openstack-cinder | 17:09 | |
| *** diablo_rojo has quit IRC | 17:09 | |
| openstackgerrit | Merged openstack/cinder: Add docstrings to CG and CG Snapshot filters https://review.openstack.org/326323 | 17:10 |
| openstackgerrit | Merged openstack/cinder: Add error messages to conditional updates devref https://review.openstack.org/326324 | 17:11 |
| *** julim has joined #openstack-cinder | 17:13 | |
| *** jungleboyj has quit IRC | 17:16 | |
| *** timcl has quit IRC | 17:23 | |
| openstackgerrit | Clinton Knight proposed openstack/cinder: NetApp: Replace SSC for cDOT block & file drivers https://review.openstack.org/327831 | 17:23 |
| *** kfarr has joined #openstack-cinder | 17:26 | |
| *** porrua has quit IRC | 17:27 | |
| *** jungleboyj has joined #openstack-cinder | 17:28 | |
| *** permalac has quit IRC | 17:31 | |
| *** sdake has joined #openstack-cinder | 17:31 | |
| jgriffith | Anticimex: yeah, I mean I think oslo wsgi will support it, but we're not turning it on during init | 17:31 |
| jgriffith | Anticimex: I'll have to look at it when I finish up the attach stuff | 17:32 |
| *** salv-orlando has joined #openstack-cinder | 17:32 | |
| jgriffith | for now on M the workarounds appear to be your only option... unless somebody more familiar with the SSL stuff knows another solution that I don't see | 17:32 |
| Anticimex | that's ok, i'm hopefully past halfway done with turning up the wsgi. we thus far already run keystone under wsgi and the puppet-openstack things are fairly similar, so i could copy/paste a bit | 17:34 |
| *** salv-orlando has quit IRC | 17:35 | |
| *** sdake_ has joined #openstack-cinder | 17:35 | |
| smcginnis | jgriffith, Anticimex: Do we have a cinder bug filed for this? | 17:35 |
| *** salv-orlando has joined #openstack-cinder | 17:36 | |
| *** sdake has quit IRC | 17:37 | |
| *** lpetrut has quit IRC | 17:37 | |
| Anticimex | or hrmpf. puppet-cinder workaround foiled. stable/mitaka release of it doesn't do the wsgi at all, it's post stable/mitaka, where i ran into some puppet lib dependency fail. so i'm back to unbreaking that then i guess | 17:38 |
| Anticimex | smcginnis: i haven't filed any and there was a ref to something somewhat related above, https://bugs.launchpad.net/devstack/+bug/1533813 | 17:39 |
| openstack | Launchpad bug 1533813 in devstack "Cinder API server disagrees with USE_SSL=True" [Undecided,In progress] - Assigned to Guang Yee (guang-yee) | 17:39 |
| *** mtanino has quit IRC | 17:39 | |
| Anticimex | that's on devstack options though | 17:39 |
| Anticimex | so, no | 17:39 |
| Anticimex | i can file it | 17:39 |
| smcginnis | Anticimex: That would be great if you have the time. | 17:40 |
| Anticimex | done | 17:44 |
| Anticimex | https://bugs.launchpad.net/cinder/+bug/1590901 | 17:45 |
| openstack | Launchpad bug 1590901 in Cinder "cinder-api lost SSL in oslo.service wsgi move for M" [Undecided,New] | 17:45 |
| *** mriedem1 has joined #openstack-cinder | 17:47 | |
| *** mriedem has quit IRC | 17:47 | |
| *** mriedem1 has quit IRC | 17:48 | |
| *** mriedem has joined #openstack-cinder | 17:49 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Fix delete volume when online clone https://review.openstack.org/327840 | 17:49 |
| *** vgridnev has joined #openstack-cinder | 17:50 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Fix delete volume when online clone https://review.openstack.org/327840 | 17:51 |
| *** bhavik has quit IRC | 17:54 | |
| *** enriquetaso has joined #openstack-cinder | 17:55 | |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: 3PAR: Fix delete volume when online clone https://review.openstack.org/327840 | 17:56 |
| *** tpeoples has quit IRC | 17:57 | |
| *** tpsilva has quit IRC | 17:57 | |
| *** serverascode has quit IRC | 17:58 | |
| *** flip217 has quit IRC | 17:58 | |
| *** flip217 has joined #openstack-cinder | 17:58 | |
| *** JoseMello has quit IRC | 17:59 | |
| *** serverascode has joined #openstack-cinder | 18:00 | |
| *** bardia has joined #openstack-cinder | 18:00 | |
| *** tpeoples has joined #openstack-cinder | 18:00 | |
| *** mmotiani- has quit IRC | 18:00 | |
| *** tpsilva has joined #openstack-cinder | 18:01 | |
| *** JoseMello has joined #openstack-cinder | 18:01 | |
| *** mtreinish has quit IRC | 18:03 | |
| *** mtreinish has joined #openstack-cinder | 18:03 | |
| *** mmotiani_ has joined #openstack-cinder | 18:04 | |
| *** jsheeren has quit IRC | 18:05 | |
| akerr | so i guess brick locks for attach/detach only work if nova and cinder are actually using the same lock directory | 18:07 |
| *** scottda has quit IRC | 18:07 | |
| * akerr bangs head on wall | 18:07 | |
| *** kwathore has quit IRC | 18:08 | |
| *** tpsilva has quit IRC | 18:09 | |
| *** scottda has joined #openstack-cinder | 18:09 | |
| *** mtreinish has quit IRC | 18:10 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/327362 | 18:10 |
| *** tpsilva has joined #openstack-cinder | 18:10 | |
| hemna | akerr, wassup? | 18:12 |
| *** jungleboyj has quit IRC | 18:12 | |
| akerr | hemna: oh just debugging why we seem to have iscsi sessions getting logged out | 18:12 |
| *** mtreinish has joined #openstack-cinder | 18:13 | |
| akerr | hemna: i think its because nova and cinder are using different lock paths in our CI and so the brick locks around that aren't really working | 18:13 |
| hemna | :( | 18:13 |
| akerr | just been a recurring intermittent problem | 18:13 |
| hemna | do you have parallel tests running ? | 18:14 |
| akerr | hemna: yea, and nova and cinder are on the same node | 18:14 |
| *** diablo_rojo has joined #openstack-cinder | 18:14 | |
| ameade | it can happen without parallel tests | 18:15 |
| ameade | one sec | 18:15 |
| *** mtanino has joined #openstack-cinder | 18:15 | |
| *** mgagne has quit IRC | 18:16 | |
| hemna | akerr, https://github.com/openstack/oslo.concurrency/blob/master/oslo_concurrency/lockutils.py#L44 | 18:16 |
| ameade | https://bugs.launchpad.net/cinder/+bug/1429684 | 18:16 |
| openstack | ameade: Error: Could not gather data from Launchpad for bug #1429684 (https://launchpad.net/bugs/1429684). The error has been logged | 18:16 |
| ameade | guess we thought it was fixed but didnt think about directory | 18:16 |
| akerr | hemna: ya, but nova and cinder each have a [oslo_concurrency] lock_path config param... is that not what's used? | 18:16 |
| *** erlon has quit IRC | 18:17 | |
| hemna | looking | 18:17 |
| *** mgagne has joined #openstack-cinder | 18:17 | |
| hemna | so my devstack install has | 18:18 |
| hemna | nova.cinf lock_path = /opt/stack/data/nova | 18:18 |
| hemna | nova.conf | 18:18 |
| *** erlon has joined #openstack-cinder | 18:18 | |
| akerr | and cinder uses /opt/stack/data/cinder | 18:19 |
| akerr | i'm testing out changing both /opt/stack/data/locks | 18:19 |
| hemna | http://paste.openstack.org/show/509288/ | 18:19 |
| *** salv-orlando has quit IRC | 18:21 | |
| akerr | otherwise you have nova putting a brick lock into different path than cinder so they don't actually lock each other | 18:21 |
| *** salv-orlando has joined #openstack-cinder | 18:22 | |
| *** lcurtis has joined #openstack-cinder | 18:22 | |
| *** asselin has quit IRC | 18:23 | |
| patrickeast | ohh yea, i remember having issues with that at some point too | 18:24 |
| *** asselin_ has joined #openstack-cinder | 18:25 | |
| akerr | i think its mostly user error on my part (if this is the actual problem) | 18:25 |
| akerr | just an easy error to make given the defaults | 18:25 |
| *** salv-orlando has quit IRC | 18:26 | |
| *** salv-orlando has joined #openstack-cinder | 18:26 | |
| ameade | +1 | 18:26 |
| *** lprice1 has joined #openstack-cinder | 18:32 | |
| hemna | akerr, well unfortunately, it's an oslo-concurrency wide path | 18:32 |
| *** jungleboyj has joined #openstack-cinder | 18:32 | |
| *** lprice has quit IRC | 18:34 | |
| Swanson | Got some dell bug fixes and retry code here ... https://review.openstack.org/#/c/325506/ | 18:37 |
| *** xinli has quit IRC | 18:37 | |
| *** xinli has joined #openstack-cinder | 18:38 | |
| *** sdake_ has quit IRC | 18:38 | |
| *** savihou has joined #openstack-cinder | 18:39 | |
| *** timcl has joined #openstack-cinder | 18:41 | |
| tbarron | akerr: in real life nova and cinder may not be running on the same node anyways, so even if it's the same path that might not be sufficient, right? | 18:41 |
| akerr | tbarron: ya i'm not saying its necessarily a problem, just venting that i didn't see this earlier | 18:42 |
| tbarron | akerr: but are the locks supposed to coordinate between nova and cinder, or only on a single node? | 18:42 |
| hemna | it's only an issue if you have n-cpu and c-vol on the same host | 18:43 |
| akerr | tbarron: the brick locks are specifically synchronized to a single namespace so that nova and cinder can coordinate... and multinode locking has the whole dlm issue | 18:43 |
| hemna | and I guess cinder backup now too | 18:43 |
| *** timcl1 has joined #openstack-cinder | 18:44 | |
| tbarron | hemna: ok | 18:44 |
| akerr | we just hit it because we're running devstack all-in-one | 18:44 |
| * tbarron goes away, relieved | 18:44 | |
| hemna | it might be nice to be able to set the os-brick lock dir specifically | 18:44 |
| hemna | in nova and cinder | 18:44 |
| hemna | which would be separate from the other lock_dir that nova/cinder set | 18:44 |
| hemna | would require some decorator magic | 18:45 |
| *** timcl has quit IRC | 18:45 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Add test type clarification to devref https://review.openstack.org/327806 | 18:48 |
| *** lpetrut has joined #openstack-cinder | 18:50 | |
| *** lpetrut has quit IRC | 18:50 | |
| *** pauloewerton has joined #openstack-cinder | 18:55 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Add test type clarification to devref https://review.openstack.org/327806 | 18:56 |
| *** rlrossit has quit IRC | 18:57 | |
| *** sdake has joined #openstack-cinder | 18:59 | |
| *** e0ne has joined #openstack-cinder | 18:59 | |
| *** zhonghua has joined #openstack-cinder | 19:00 | |
| *** e0ne has quit IRC | 19:02 | |
| *** raunak has quit IRC | 19:03 | |
| *** zhonghua-lee has quit IRC | 19:03 | |
| *** lprice1 has quit IRC | 19:07 | |
| *** e0ne has joined #openstack-cinder | 19:07 | |
| *** darosale has joined #openstack-cinder | 19:07 | |
| *** akerr has quit IRC | 19:07 | |
| *** xinli has quit IRC | 19:08 | |
| *** harlowja has quit IRC | 19:12 | |
| *** lprice has joined #openstack-cinder | 19:14 | |
| *** itlinux has joined #openstack-cinder | 19:18 | |
| *** sdake has quit IRC | 19:19 | |
| *** timcl1 has quit IRC | 19:19 | |
| *** timcl has joined #openstack-cinder | 19:20 | |
| *** timcl has quit IRC | 19:24 | |
| *** timcl has joined #openstack-cinder | 19:25 | |
| *** martyturner has joined #openstack-cinder | 19:27 | |
| *** diablo_rojo has quit IRC | 19:28 | |
| *** diablo_rojo has joined #openstack-cinder | 19:30 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Add test type clarification to devref https://review.openstack.org/327806 | 19:32 |
| *** Apoorva has quit IRC | 19:34 | |
| *** jungleboyj has quit IRC | 19:39 | |
| *** tyrola has joined #openstack-cinder | 19:44 | |
| *** tyrola has quit IRC | 19:45 | |
| *** timcl has quit IRC | 19:46 | |
| *** martyturner has quit IRC | 19:47 | |
| *** xinli has joined #openstack-cinder | 19:47 | |
| *** sgotliv_ has quit IRC | 19:48 | |
| *** savihou has quit IRC | 19:48 | |
| *** harlowja has joined #openstack-cinder | 19:49 | |
| *** e0ne has quit IRC | 19:49 | |
| *** rlrossit has joined #openstack-cinder | 19:50 | |
| *** Lee1092 has quit IRC | 19:53 | |
| *** timcl has joined #openstack-cinder | 19:54 | |
| *** lprice1 has joined #openstack-cinder | 19:57 | |
| *** lprice has quit IRC | 19:58 | |
| *** timcl has quit IRC | 19:59 | |
| *** xyang1 has joined #openstack-cinder | 20:00 | |
| *** crose has quit IRC | 20:00 | |
| *** timcl has joined #openstack-cinder | 20:00 | |
| *** Apoorva has joined #openstack-cinder | 20:03 | |
| ccesario | hello, does someone use ibm storwize driver !? | 20:04 |
| *** timcl has quit IRC | 20:06 | |
| *** akerr has joined #openstack-cinder | 20:15 | |
| *** akerr_ has joined #openstack-cinder | 20:18 | |
| *** akerr has quit IRC | 20:21 | |
| *** kfarr has quit IRC | 20:21 | |
| *** raunak_ has joined #openstack-cinder | 20:21 | |
| scottda | Is there any difference between "Fixes-Bug" and "Closes-Bug" in a commit message? | 20:21 |
| *** chris_morrell has joined #openstack-cinder | 20:21 | |
| *** akerr has joined #openstack-cinder | 20:21 | |
| smcginnis | scottda: I think it's just Closes-bug. | 20:22 |
| eharney | scottda: iirc, gerrit doesn't link Fixes-Bug right so it looks funny... but launchpad still gets updated etc | 20:23 |
| scottda | smcginnis: Thanks. I see use of Fixes-Bug but wasn't sure if that was just plain wrong, or a synonym | 20:23 |
| eharney | Closes-Bug is preferred for sure | 20:23 |
| scottda | ok. Closes-Bug it is. Thanks. | 20:23 |
| smcginnis | Closes is the what's listed as the "official" mechanism: https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references | 20:23 |
| *** akerr_ has quit IRC | 20:24 | |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: Update provider_location during snapshot backup https://review.openstack.org/327375 | 20:24 |
| xyang1 | scottda: looks like we are working on the same bug? https://review.openstack.org/#/c/322294/ | 20:30 |
| angela-s | any IBMer here that works with Yucong Feng? | 20:31 |
| scottda | xyang1: Yes, I think so....looking... | 20:31 |
| openstackgerrit | Jon Bernard proposed openstack/cinder: RBD: support driver-assisted volume migration https://review.openstack.org/256091 | 20:32 |
| *** arch-nemesis has quit IRC | 20:35 | |
| scottda | xyang1: Yup. I've abandoned mine in favour of yours. | 20:36 |
| *** rcernin has joined #openstack-cinder | 20:36 | |
| xyang1 | scottda: I can add you as co-author. mine is not merged yet | 20:36 |
| scottda | sure, either way. I mostly want the bug fix as it causes an issue for us. | 20:37 |
| xyang1 | scottda: sure | 20:37 |
| *** timcl has joined #openstack-cinder | 20:37 | |
| _alastor_ | In os_brick connector classes the connect_volume function returns a device_info dict that has a 'type' key that appears to either be set to 'block' or 'local'. Dose anyone know where the options for this key are documented? Also, what do they do? | 20:38 |
| _alastor_ | Does* | 20:38 |
| scottda | Where is the "Close as duplicate" button for a bug? or do I just close as invalid? | 20:38 |
| hemna | _alastor_, that's basically it | 20:38 |
| _alastor_ | hemna: Ok, is there a place I can look for where they dictate a change in behavior? I've been trying to follow the call stack, but it seems to jump repositories | 20:39 |
| smcginnis | scottda: Which one? I forget exactly where, looking now... | 20:39 |
| *** gouthamr has quit IRC | 20:40 | |
| hemna | _alastor_, I believe nova is the only one that consumes that value at all for any reason. | 20:40 |
| smcginnis | scottda: Left hand side, third box down for me, link labeled "Mark as duplicate" | 20:40 |
| scottda | smcginnis: This: https://bugs.launchpad.net/cinder/+bug/1590563 is a duplicate of https://bugs.launchpad.net/cinder/+bug/1575888 | 20:40 |
| openstack | Launchpad bug 1590563 in Cinder "Backup with create_temp_volume_from_snapshot can fail if provider_location is updated" [Undecided,In progress] - Assigned to Scott DAngelo (scott-dangelo) | 20:40 |
| openstack | Launchpad bug 1575888 in Cinder "Create temp volume in backup needs to return model_update from driver" [Medium,In progress] - Assigned to Xing Yang (xing-yang) | 20:40 |
| *** jwcroppe_ has joined #openstack-cinder | 20:40 | |
| openstackgerrit | xing-yang proposed openstack/cinder: model_update for temp volume or snapshot in backup https://review.openstack.org/322294 | 20:41 |
| hemna | _alastor_, looking in Nova, it doesn't seem like it's even being used there either | 20:41 |
| smcginnis | scottda: Marked it. | 20:41 |
| _alastor_ | hemna: ok, I was starting to feel crazy that I couldn't find any real usage | 20:41 |
| *** jwcropp__ has joined #openstack-cinder | 20:42 | |
| *** timcl has quit IRC | 20:42 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: WIP: Update LVM code so nova can use it https://review.openstack.org/308054 | 20:42 |
| *** timcl has joined #openstack-cinder | 20:42 | |
| xyang1 | smcginnis: hi, you and jungleboy reviewed this one earlier: https://review.openstack.org/#/c/322294 | 20:42 |
| xyang1 | smcginnis: I justed added scottda as co-author | 20:43 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Add return parameter info for manage_existing_get_size https://review.openstack.org/327937 | 20:43 |
| *** jwcroppe has quit IRC | 20:44 | |
| smcginnis | xyang1: OK, I'll take another quick look in a bit. | 20:44 |
| xyang1 | smcginnis: thanks! | 20:44 |
| *** jwcroppe_ has quit IRC | 20:46 | |
| *** jungleboyj has joined #openstack-cinder | 20:48 | |
| *** cknight has quit IRC | 20:49 | |
| *** lprice has joined #openstack-cinder | 20:49 | |
| *** arch-nemesis has joined #openstack-cinder | 20:51 | |
| *** timcl has quit IRC | 20:51 | |
| *** lprice1 has quit IRC | 20:51 | |
| *** neophy has joined #openstack-cinder | 20:52 | |
| *** ametts has quit IRC | 20:52 | |
| *** iceyao has joined #openstack-cinder | 20:55 | |
| *** dustins has quit IRC | 20:55 | |
| *** timcl has joined #openstack-cinder | 20:58 | |
| *** pauloewerton has quit IRC | 20:58 | |
| *** iceyao has quit IRC | 21:00 | |
| *** jwcropp__ has quit IRC | 21:01 | |
| *** eharney has quit IRC | 21:02 | |
| *** jwcroppe has joined #openstack-cinder | 21:10 | |
| *** vgridnev has quit IRC | 21:11 | |
| *** rcernin has quit IRC | 21:15 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Add fast8 to quickly test pep8 changes https://review.openstack.org/327964 | 21:19 |
| *** timcl has quit IRC | 21:20 | |
| *** wN has quit IRC | 21:22 | |
| *** arch-nemesis has quit IRC | 21:26 | |
| *** wN has joined #openstack-cinder | 21:27 | |
| *** wN has quit IRC | 21:27 | |
| *** wN has joined #openstack-cinder | 21:27 | |
| *** edmondsw has quit IRC | 21:29 | |
| *** rlrossit has quit IRC | 21:32 | |
| *** jgregor has quit IRC | 21:32 | |
| *** diablo_rojo has quit IRC | 21:33 | |
| *** zul has quit IRC | 21:39 | |
| *** akerr has quit IRC | 21:45 | |
| *** akerr has joined #openstack-cinder | 21:45 | |
| *** lprice has quit IRC | 21:45 | |
| *** arch-nemesis has joined #openstack-cinder | 21:46 | |
| *** akerr has quit IRC | 21:48 | |
| *** akerr_ has joined #openstack-cinder | 21:49 | |
| openstackgerrit | Merged openstack/cinder: List manageable volumes and snapshots https://review.openstack.org/285296 | 21:49 |
| *** zul has joined #openstack-cinder | 21:54 | |
| *** karthikp_ has quit IRC | 21:55 | |
| *** karthikp_ has joined #openstack-cinder | 21:56 | |
| *** xinli has quit IRC | 22:02 | |
| *** jungleboyj has quit IRC | 22:04 | |
| *** mriedem has quit IRC | 22:05 | |
| *** jgregor has joined #openstack-cinder | 22:06 | |
| *** mriedem has joined #openstack-cinder | 22:06 | |
| *** mriedem has quit IRC | 22:06 | |
| *** tpsilva has quit IRC | 22:07 | |
| *** jamielennox is now known as jamielennox|away | 22:09 | |
| *** angela-s has quit IRC | 22:13 | |
| *** darosale has quit IRC | 22:15 | |
| *** takedakn has joined #openstack-cinder | 22:26 | |
| *** erlon has quit IRC | 22:26 | |
| *** jwcroppe has quit IRC | 22:28 | |
| *** akerr_ is now known as akerr_away | 22:30 | |
| *** takedakn has quit IRC | 22:30 | |
| *** ducttape_ has quit IRC | 22:31 | |
| *** neophy has quit IRC | 22:37 | |
| *** lcurtis has quit IRC | 22:38 | |
| *** karthikp_ has quit IRC | 22:39 | |
| *** hemna is now known as hemnafk | 22:40 | |
| *** chris_morrell has quit IRC | 22:55 | |
| *** mtanino has quit IRC | 22:58 | |
| openstackgerrit | Merged openstack/cinder: Add sample config file to cinder docs https://review.openstack.org/314220 | 23:00 |
| *** xyang1 has quit IRC | 23:04 | |
| *** cknight has joined #openstack-cinder | 23:05 | |
| *** sdague has quit IRC | 23:10 | |
| *** gouthamr has joined #openstack-cinder | 23:23 | |
| *** karthikp_ has joined #openstack-cinder | 23:26 | |
| *** salv-orl_ has joined #openstack-cinder | 23:27 | |
| *** salv-orl_ has quit IRC | 23:28 | |
| *** jwcroppe has joined #openstack-cinder | 23:28 | |
| *** salv-orl_ has joined #openstack-cinder | 23:29 | |
| *** Apoorva has quit IRC | 23:30 | |
| *** salv-orlando has quit IRC | 23:31 | |
| *** jwcroppe has quit IRC | 23:32 | |
| openstackgerrit | Merged openstack/cinder: model_update for temp volume or snapshot in backup https://review.openstack.org/322294 | 23:34 |
| *** jamielennox|away is now known as jamielennox | 23:37 | |
| *** Apoorva has joined #openstack-cinder | 23:43 | |
| *** chlong has quit IRC | 23:45 | |
| *** akerr_away is now known as akerr_ | 23:46 | |
| *** akerr_ is now known as akerr_away | 23:47 | |
| *** akerr_away is now known as akerr_ | 23:48 | |
| *** akerr_ is now known as akerr_away | 23:49 | |
| *** yhayashi has joined #openstack-cinder | 23:50 | |
| *** akerr_away is now known as akerr_ | 23:51 | |
| *** akerr_ is now known as akerr_away | 23:51 | |
| *** akerr_away is now known as akerr_ | 23:51 | |
| *** akerr_ is now known as akerr_away | 23:52 | |
| *** raunak_ has quit IRC | 23:52 | |
| *** akerr_away is now known as akerr_ | 23:52 | |
| *** akerr_ is now known as akerr_away | 23:53 | |
| *** akerr_away is now known as akerr_ | 23:54 | |
| *** akerr has joined #openstack-cinder | 23:55 | |
| *** akerr_ has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!