*** erlon has joined #openstack-cinder | 00:01 | |
*** _alastor_ has quit IRC | 00:18 | |
*** dave-mccowan has joined #openstack-cinder | 00:36 | |
openstackgerrit | caoyuan proposed openstack/cinder master: Update json module to jsonutils https://review.openstack.org/637428 | 00:44 |
---|---|---|
openstackgerrit | caoyuan proposed openstack/cinder master: Update json module to jsonutils https://review.openstack.org/637428 | 00:44 |
*** markvoelker has joined #openstack-cinder | 00:50 | |
*** erlon has quit IRC | 00:51 | |
*** stakeda has joined #openstack-cinder | 00:51 | |
*** bhagyashris has joined #openstack-cinder | 01:01 | |
*** whoami-rajat has joined #openstack-cinder | 01:07 | |
*** dave-mccowan has quit IRC | 01:18 | |
*** markvoelker has quit IRC | 01:22 | |
*** erlon has joined #openstack-cinder | 01:30 | |
*** rcernin has quit IRC | 01:31 | |
*** rcernin has joined #openstack-cinder | 01:32 | |
*** rchurch has joined #openstack-cinder | 01:36 | |
*** rchurch_ has quit IRC | 01:38 | |
*** erlon has quit IRC | 01:50 | |
*** mriedem is now known as mriedem_afk | 01:51 | |
openstackgerrit | Matt Riedemann proposed openstack/cinder master: docs: mark rbd driver support for extending an in-use volume https://review.openstack.org/641539 | 01:57 |
openstackgerrit | Matt Riedemann proposed openstack/cinder master: docs: mark rbd driver support for extending an in-use volume https://review.openstack.org/641539 | 01:58 |
*** mriedem_afk has quit IRC | 01:59 | |
*** dave-mccowan has joined #openstack-cinder | 02:17 | |
*** wanghao has joined #openstack-cinder | 02:19 | |
*** markvoelker has joined #openstack-cinder | 02:20 | |
*** dviroel_ has quit IRC | 02:42 | |
*** psachin has joined #openstack-cinder | 02:42 | |
*** markvoelker has quit IRC | 02:52 | |
*** udesale has joined #openstack-cinder | 03:24 | |
*** markvoelker has joined #openstack-cinder | 03:49 | |
*** vishakha has joined #openstack-cinder | 03:50 | |
*** dave-mccowan has quit IRC | 04:00 | |
openstackgerrit | futaotao proposed openstack/cinder master: Amend: Modify the configuration mode and add the release notes on FusionStorage Driver Refactor https://review.openstack.org/632983 | 04:03 |
*** alkhodos has quit IRC | 04:09 | |
*** _alastor_ has joined #openstack-cinder | 04:18 | |
*** markvoelker has quit IRC | 04:22 | |
*** _alastor_ has quit IRC | 04:22 | |
*** sdinescu has quit IRC | 04:26 | |
*** strigazi has quit IRC | 04:26 | |
*** arunkant has quit IRC | 04:26 | |
*** kukacz has quit IRC | 04:26 | |
*** cburgess has quit IRC | 04:26 | |
*** penick has quit IRC | 04:26 | |
*** gary_perkins has quit IRC | 04:26 | |
*** bswartz has quit IRC | 04:26 | |
*** sorrison has quit IRC | 04:26 | |
*** kambiz has quit IRC | 04:26 | |
*** abhishekk has joined #openstack-cinder | 04:29 | |
*** sdinescu has joined #openstack-cinder | 04:53 | |
*** strigazi has joined #openstack-cinder | 04:53 | |
*** arunkant has joined #openstack-cinder | 04:53 | |
*** kukacz has joined #openstack-cinder | 04:53 | |
*** cburgess has joined #openstack-cinder | 04:53 | |
*** penick has joined #openstack-cinder | 04:53 | |
*** bswartz has joined #openstack-cinder | 04:53 | |
*** gary_perkins has joined #openstack-cinder | 04:53 | |
*** sorrison has joined #openstack-cinder | 04:53 | |
*** kambiz has joined #openstack-cinder | 04:53 | |
*** rcernin has quit IRC | 04:56 | |
*** rcernin has joined #openstack-cinder | 05:02 | |
*** sdinescu has quit IRC | 05:05 | |
*** markvoelker has joined #openstack-cinder | 05:19 | |
*** Bhujay has joined #openstack-cinder | 05:34 | |
*** lemko has joined #openstack-cinder | 05:35 | |
*** avishay has joined #openstack-cinder | 05:40 | |
openstackgerrit | Merged openstack/cinder stable/ocata: Fix for auth version change in Brcd HTTP https://review.openstack.org/640567 | 05:53 |
*** markvoelker has quit IRC | 05:53 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: DNM: Testing hpe3par driver against rally-openstack jobs https://review.openstack.org/641567 | 06:10 |
raghavendrat | whoami-rajat: hemna: Thanks for bringing up various points for discussion | 06:27 |
raghavendrat | for https://review.openstack.org/#/c/634119/ if i understand correctly, i should change default value of extra-spec to False | 06:31 |
*** e0ne has joined #openstack-cinder | 06:32 | |
openstackgerrit | Merged openstack/cinder master: docs: mark rbd driver support for extending an in-use volume https://review.openstack.org/641539 | 06:40 |
*** avishay has quit IRC | 06:45 | |
*** markvoelker has joined #openstack-cinder | 06:50 | |
*** Luzi has joined #openstack-cinder | 06:54 | |
*** dpawlik has joined #openstack-cinder | 07:05 | |
*** Bhujay has quit IRC | 07:08 | |
*** Bhujay has joined #openstack-cinder | 07:09 | |
*** Bhujay has quit IRC | 07:10 | |
*** Bhujay has joined #openstack-cinder | 07:10 | |
*** Bhujay has quit IRC | 07:11 | |
*** Bhujay has joined #openstack-cinder | 07:12 | |
*** Bhujay has quit IRC | 07:13 | |
*** Bhujay has joined #openstack-cinder | 07:13 | |
*** Bhujay has quit IRC | 07:14 | |
*** Bhujay has joined #openstack-cinder | 07:15 | |
*** Bhujay has quit IRC | 07:16 | |
*** Bhujay has joined #openstack-cinder | 07:16 | |
*** e0ne has quit IRC | 07:17 | |
*** Bhujay has quit IRC | 07:17 | |
*** Bhujay has joined #openstack-cinder | 07:18 | |
*** Bhujay has quit IRC | 07:19 | |
*** Bhujay has joined #openstack-cinder | 07:19 | |
*** Bhujay has quit IRC | 07:20 | |
*** Bhujay has joined #openstack-cinder | 07:21 | |
*** gkadam__ has joined #openstack-cinder | 07:21 | |
*** e0ne has joined #openstack-cinder | 07:21 | |
*** Bhujay has quit IRC | 07:22 | |
*** markvoelker has quit IRC | 07:22 | |
*** Bhujay has joined #openstack-cinder | 07:23 | |
*** pcaruana has joined #openstack-cinder | 07:25 | |
*** rcernin has quit IRC | 07:27 | |
*** david-lyle has joined #openstack-cinder | 07:36 | |
*** dklyle has quit IRC | 07:37 | |
*** dpawlik has quit IRC | 07:39 | |
*** dklyle has joined #openstack-cinder | 07:41 | |
*** david-lyle has quit IRC | 07:42 | |
*** lemko has quit IRC | 07:45 | |
*** dpawlik has joined #openstack-cinder | 07:52 | |
*** dklyle has quit IRC | 07:55 | |
*** dklyle has joined #openstack-cinder | 07:55 | |
*** e0ne has quit IRC | 08:04 | |
*** avishay has joined #openstack-cinder | 08:05 | |
*** sdinescu has joined #openstack-cinder | 08:09 | |
*** tkajinam has quit IRC | 08:12 | |
*** markvoelker has joined #openstack-cinder | 08:19 | |
*** markvoelker has quit IRC | 08:53 | |
openstackgerrit | zhulingjie proposed openstack/cinder master: Update json module to jsonutils https://review.openstack.org/637428 | 08:53 |
*** Bhujay has quit IRC | 08:55 | |
*** Bhujay has joined #openstack-cinder | 08:56 | |
*** Bhujay has quit IRC | 08:57 | |
*** Bhujay has joined #openstack-cinder | 08:58 | |
*** Bhujay has quit IRC | 08:59 | |
*** Bhujay has joined #openstack-cinder | 08:59 | |
*** Bhujay has quit IRC | 09:00 | |
*** Bhujay has joined #openstack-cinder | 09:01 | |
*** Bhujay has quit IRC | 09:02 | |
*** Bhujay has joined #openstack-cinder | 09:02 | |
*** Bhujay has quit IRC | 09:03 | |
*** Bhujay has joined #openstack-cinder | 09:04 | |
*** Bhujay has quit IRC | 09:05 | |
*** Bhujay has joined #openstack-cinder | 09:05 | |
*** Bhujay has quit IRC | 09:06 | |
*** Bhujay has joined #openstack-cinder | 09:07 | |
*** Bhujay has quit IRC | 09:08 | |
*** Bhujay has joined #openstack-cinder | 09:08 | |
*** Bhujay has quit IRC | 09:09 | |
*** Bhujay has joined #openstack-cinder | 09:10 | |
*** Bhujay has quit IRC | 09:11 | |
*** Bhujay has joined #openstack-cinder | 09:11 | |
*** Bhujay has quit IRC | 09:12 | |
*** Bhujay has joined #openstack-cinder | 09:13 | |
*** Bhujay has quit IRC | 09:14 | |
*** Bhujay has joined #openstack-cinder | 09:14 | |
*** Bhujay has quit IRC | 09:15 | |
*** Bhujay has joined #openstack-cinder | 09:16 | |
*** Bhujay has quit IRC | 09:17 | |
*** Bhujay has joined #openstack-cinder | 09:17 | |
openstackgerrit | zhulingjie proposed openstack/cinder master: Update json module to jsonutils https://review.openstack.org/637428 | 09:17 |
*** Bhujay has quit IRC | 09:18 | |
*** Bhujay has joined #openstack-cinder | 09:19 | |
*** Bhujay has quit IRC | 09:20 | |
*** Bhujay has joined #openstack-cinder | 09:20 | |
*** Bhujay has quit IRC | 09:21 | |
*** Bhujay has joined #openstack-cinder | 09:22 | |
*** Bhujay has quit IRC | 09:23 | |
*** Bhujay has joined #openstack-cinder | 09:23 | |
*** Bhujay has quit IRC | 09:24 | |
*** Bhujay has joined #openstack-cinder | 09:25 | |
*** Bhujay has quit IRC | 09:26 | |
*** Bhujay has joined #openstack-cinder | 09:26 | |
*** Bhujay has quit IRC | 09:27 | |
*** Bhujay has joined #openstack-cinder | 09:28 | |
*** Bhujay has quit IRC | 09:29 | |
*** Bhujay has joined #openstack-cinder | 09:29 | |
*** Bhujay has quit IRC | 09:30 | |
*** Bhujay has joined #openstack-cinder | 09:31 | |
*** Bhujay has quit IRC | 09:32 | |
*** Bhujay has joined #openstack-cinder | 09:32 | |
*** Bhujay has quit IRC | 09:33 | |
*** Bhujay has joined #openstack-cinder | 09:34 | |
*** Bhujay has quit IRC | 09:35 | |
*** Bhujay has joined #openstack-cinder | 09:35 | |
*** Bhujay has quit IRC | 09:36 | |
*** Bhujay has joined #openstack-cinder | 09:37 | |
*** Bhujay has quit IRC | 09:38 | |
*** Bhujay has joined #openstack-cinder | 09:38 | |
*** Bhujay has quit IRC | 09:39 | |
*** Bhujay has joined #openstack-cinder | 09:40 | |
*** Bhujay has quit IRC | 09:41 | |
*** Bhujay has joined #openstack-cinder | 09:41 | |
whoami-rajat | raghavendrat: hemna I put up a patch with convert_to_base =False as default[1], tested cinder rally job [2] (specifically create_volume_from_snaphot [3]), and it passed (if my approach to test it was correct). | 09:42 |
whoami-rajat | The bug reported for it's failure[4] | 09:42 |
whoami-rajat | [1] https://review.openstack.org/#/c/641567/ | 09:42 |
whoami-rajat | [2] https://review.openstack.org/#/c/641573/ | 09:42 |
whoami-rajat | [3] http://logs.openstack.org/73/641573/1/check/rally-task-cinder/c86477a/results/report.html.gz#/CinderVolumes.create_volume_from_snapshot | 09:42 |
whoami-rajat | [4] https://bugs.launchpad.net/cinder/+bug/1656186 | 09:42 |
openstack | Launchpad bug 1656186 in Cinder "Error running rally test scenario 'create-volume-from-snapshot'" [Undecided,Fix released] - Assigned to Vijay Ladani (vijay-ladani) | 09:42 |
*** Bhujay has quit IRC | 09:42 | |
*** Bhujay has joined #openstack-cinder | 09:43 | |
*** Bhujay has quit IRC | 09:44 | |
*** Bhujay has joined #openstack-cinder | 09:44 | |
raghavendrat | whoami-rajat: thanks. did you test deletion of volume v1 and snapshot s1 ? | 09:44 |
*** Bhujay has quit IRC | 09:45 | |
*** Bhujay has joined #openstack-cinder | 09:46 | |
*** Bhujay has quit IRC | 09:47 | |
*** Bhujay has joined #openstack-cinder | 09:47 | |
whoami-rajat | raghavendrat: yes, this is the case tested above in the rally gate. https://github.com/openstack/rally-openstack/blob/master/rally_openstack/scenarios/cinder/volumes.py#L834-L835 | 09:48 |
*** Bhujay has quit IRC | 09:48 | |
*** Bhujay has joined #openstack-cinder | 09:49 | |
*** Bhujay has quit IRC | 09:50 | |
*** markvoelker has joined #openstack-cinder | 09:50 | |
*** Bhujay has joined #openstack-cinder | 09:50 | |
*** Bhujay has quit IRC | 09:51 | |
*** Bhujay has joined #openstack-cinder | 09:52 | |
*** bhagyashris has quit IRC | 09:52 | |
*** ianw is now known as ianw_pto | 09:52 | |
*** Bhujay has quit IRC | 09:53 | |
*** Bhujay has joined #openstack-cinder | 09:53 | |
raghavendrat | whoami-rajat: thanks. | 09:54 |
*** Bhujay has quit IRC | 09:54 | |
*** Bhujay has joined #openstack-cinder | 09:55 | |
*** Bhujay has quit IRC | 09:56 | |
*** Bhujay has joined #openstack-cinder | 09:56 | |
*** Bhujay has quit IRC | 09:57 | |
*** Bhujay has joined #openstack-cinder | 09:58 | |
*** stakeda has quit IRC | 09:59 | |
*** Bhujay has quit IRC | 09:59 | |
*** Bhujay has joined #openstack-cinder | 09:59 | |
*** Bhujay has quit IRC | 10:00 | |
*** Bhujay has joined #openstack-cinder | 10:01 | |
*** Bhujay has quit IRC | 10:02 | |
*** Bhujay has joined #openstack-cinder | 10:02 | |
*** Bhujay has quit IRC | 10:03 | |
*** Bhujay has joined #openstack-cinder | 10:04 | |
*** Bhujay has quit IRC | 10:05 | |
*** Bhujay has joined #openstack-cinder | 10:05 | |
*** Bhujay has quit IRC | 10:06 | |
*** Bhujay has joined #openstack-cinder | 10:07 | |
*** Bhujay has quit IRC | 10:08 | |
*** Bhujay has joined #openstack-cinder | 10:08 | |
*** Bhujay has quit IRC | 10:09 | |
*** Bhujay has joined #openstack-cinder | 10:10 | |
*** Bhujay has quit IRC | 10:11 | |
*** Bhujay has joined #openstack-cinder | 10:11 | |
*** Bhujay has quit IRC | 10:12 | |
*** Bhujay has joined #openstack-cinder | 10:13 | |
*** Bhujay has quit IRC | 10:14 | |
*** Bhujay has joined #openstack-cinder | 10:14 | |
*** Bhujay has quit IRC | 10:15 | |
*** Bhujay has joined #openstack-cinder | 10:16 | |
*** Bhujay has quit IRC | 10:17 | |
*** Bhujay has joined #openstack-cinder | 10:17 | |
*** Bhujay has quit IRC | 10:18 | |
*** Bhujay has joined #openstack-cinder | 10:19 | |
*** Bhujay has quit IRC | 10:20 | |
*** Bhujay has joined #openstack-cinder | 10:20 | |
*** Bhujay has quit IRC | 10:21 | |
*** Bhujay has joined #openstack-cinder | 10:22 | |
*** Bhujay has quit IRC | 10:23 | |
*** markvoelker has quit IRC | 10:23 | |
*** Bhujay has joined #openstack-cinder | 10:23 | |
*** wanghao has quit IRC | 10:24 | |
*** Bhujay has quit IRC | 10:24 | |
*** Bhujay has joined #openstack-cinder | 10:25 | |
*** e0ne has joined #openstack-cinder | 10:25 | |
*** Bhujay has quit IRC | 10:26 | |
*** Bhujay has joined #openstack-cinder | 10:26 | |
*** abhishekk has quit IRC | 10:27 | |
*** Bhujay has quit IRC | 10:27 | |
*** Bhujay has joined #openstack-cinder | 10:28 | |
*** Bhujay has quit IRC | 10:29 | |
*** Bhujay has joined #openstack-cinder | 10:29 | |
*** dviroel_ has joined #openstack-cinder | 10:30 | |
*** sapd1 has joined #openstack-cinder | 10:30 | |
*** Bhujay has quit IRC | 10:30 | |
*** Bhujay has joined #openstack-cinder | 10:31 | |
*** Bhujay has quit IRC | 10:32 | |
*** Bhujay has joined #openstack-cinder | 10:32 | |
*** Bhujay has quit IRC | 10:33 | |
*** Bhujay has joined #openstack-cinder | 10:34 | |
*** Bhujay has quit IRC | 10:35 | |
*** Bhujay has joined #openstack-cinder | 10:35 | |
*** Bhujay has quit IRC | 10:36 | |
*** luizbag has joined #openstack-cinder | 10:36 | |
*** Bhujay has joined #openstack-cinder | 10:37 | |
*** Bhujay has quit IRC | 10:38 | |
*** Bhujay has joined #openstack-cinder | 10:38 | |
*** Bhujay has quit IRC | 10:39 | |
*** Bhujay has joined #openstack-cinder | 10:40 | |
*** Bhujay has quit IRC | 10:41 | |
*** Bhujay has joined #openstack-cinder | 10:41 | |
*** Bhujay has quit IRC | 10:42 | |
*** Bhujay has joined #openstack-cinder | 10:43 | |
*** Bhujay has quit IRC | 10:44 | |
*** Bhujay has joined #openstack-cinder | 10:44 | |
*** Bhujay has quit IRC | 10:45 | |
*** Bhujay has joined #openstack-cinder | 10:46 | |
*** bal has joined #openstack-cinder | 10:51 | |
*** carlos_silva has joined #openstack-cinder | 10:53 | |
*** odyssey4me has quit IRC | 10:53 | |
*** odyssey4me has joined #openstack-cinder | 10:53 | |
*** udesale has quit IRC | 10:59 | |
*** udesale has joined #openstack-cinder | 10:59 | |
avishay | geguileo: Hey, how's it going? I have two questions please - (1) I am seeing CleanableInUse errors with a single cinder-volume (Ocata) and a new version of pymysql any idea? (2) I put a -1 on my patch to clean defunct devices because Nova is unnecessarily calling connect_volume in post VM migration (the volume is already connected in pre-migration). Would you say this is a bug in Nova or should connect_volume work if | 11:08 |
avishay | called repeatedly? | 11:08 |
geguileo | avishay: (1): This looks like a pymysql thing, even if you have a newer version of the code... r:-??? | 11:16 |
geguileo | avishay: it should work, but aren't you checking if the id matches what you already have? | 11:16 |
geguileo | avishay: because even with multiple calls it should always match | 11:17 |
geguileo | last 2 lines are for (2) | 11:17 |
avishay | geguileo, No, we said that even if it's the same WWN, the volume may have been connected to another node in the meantime, so it should still be cleaned up. I could change that but it weakens the check a little. | 11:18 |
geguileo | avishay: unfortunately the operation should be idempotent, and nova code can do that | 11:19 |
avishay | geguileo, For (1) I guess I need to figure out a reliable way to reproduce the issue and file a bug report. I'm surprised it isn't coming up in cinder CIs. | 11:19 |
avishay | OK | 11:19 |
*** markvoelker has joined #openstack-cinder | 11:20 | |
geguileo | avishay: check your DB version and compare it with CI jobs | 11:20 |
geguileo | in case it's not the library but the DB itself the one that's doing something weird | 11:20 |
avishay | geguileo, Could it be the fact that I'm using galera (though always accessing the master)? | 11:21 |
geguileo | avishay: dunno | 11:21 |
avishay | OK thank you! | 11:21 |
*** erlon has joined #openstack-cinder | 11:24 | |
*** thgcorrea has joined #openstack-cinder | 11:28 | |
*** udesale has quit IRC | 11:28 | |
*** abishop has quit IRC | 11:33 | |
*** udesale has joined #openstack-cinder | 11:45 | |
*** raghavendrat has quit IRC | 11:52 | |
*** markvoelker has quit IRC | 11:53 | |
*** sapd1 has quit IRC | 11:55 | |
*** mdbooth has joined #openstack-cinder | 11:58 | |
*** anks2k has joined #openstack-cinder | 12:06 | |
openstackgerrit | Yachika_Ralhan proposed openstack/cinder master: Add new OSNEXUS Quantastor Cinder driver https://review.openstack.org/617503 | 12:06 |
*** sapd1 has joined #openstack-cinder | 12:08 | |
*** henriqueof has joined #openstack-cinder | 12:21 | |
openstackgerrit | Merged openstack/cinder master: Revert "Use native python truncate for privsep" https://review.openstack.org/641413 | 12:22 |
openstackgerrit | Merged openstack/cinder master: Revert "Remove truncate from rootwrap filters" https://review.openstack.org/641414 | 12:22 |
*** dave-mccowan has joined #openstack-cinder | 12:22 | |
*** gkadam__ has quit IRC | 12:32 | |
*** gkadam__ has joined #openstack-cinder | 12:32 | |
*** sapd1 has quit IRC | 12:36 | |
*** raghavendrat has joined #openstack-cinder | 12:50 | |
*** markvoelker has joined #openstack-cinder | 12:50 | |
*** enriquetaso has joined #openstack-cinder | 12:52 | |
*** udesale has quit IRC | 13:02 | |
*** udesale has joined #openstack-cinder | 13:08 | |
*** abishop has joined #openstack-cinder | 13:08 | |
*** markvoelker has quit IRC | 13:22 | |
enriquetaso | abishop, good morning! Would you mind reviewing this https://review.openstack.org/#/c/640863/ ? It has already +2. The author is an Outreachy possible interns :) | 13:23 |
abishop | enriquetaso: yes, happy to do so (and it was on my to-do list). Bear in mind, though, that I am not a core reviewer for that project | 13:24 |
* enriquetaso facepalm | 13:27 | |
enriquetaso | abishop, hahaha Thanks! | 13:28 |
enriquetaso | I thought you were core :P | 13:28 |
abishop | enriquetaso: not for puppet stuff | 13:28 |
*** mriedem has joined #openstack-cinder | 13:30 | |
anks2k | my patchset received review priority a month back. and since then all changes have been done and submitted. I wanted to know how this affects this patch being considered to be reviewed ? | 13:33 |
whoami-rajat | anks2k: could you please paste the link here? | 13:36 |
anks2k | https://review.openstack.org/#/c/617503/ | 13:36 |
anks2k | Thanks ! | 13:37 |
whoami-rajat | anks2k: we've already freezed all driver merge. you may continue to work on it but merging it will be considered in next release (Train). | 13:39 |
anks2k | Okay, Thanks. | 13:46 |
*** jmlowe has quit IRC | 13:51 | |
*** sapd1 has joined #openstack-cinder | 13:54 | |
*** BlackDex has quit IRC | 13:59 | |
*** BlackDex has joined #openstack-cinder | 14:02 | |
*** avishay has quit IRC | 14:06 | |
hemna | doink | 14:11 |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder master: Add get_driver_options method https://review.openstack.org/635255 | 14:16 |
*** _alastor_ has joined #openstack-cinder | 14:17 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder master: Add get_driver_options method https://review.openstack.org/635255 | 14:18 |
hemna | friggin finally | 14:18 |
hemna | git was messin w/ me. kept reinjecting those @utils.trace lines | 14:18 |
*** markvoelker has joined #openstack-cinder | 14:20 | |
*** mchlumsky has joined #openstack-cinder | 14:21 | |
*** _alastor_ has quit IRC | 14:22 | |
*** psachin has quit IRC | 14:24 | |
*** jmlowe has joined #openstack-cinder | 14:30 | |
*** lseki has joined #openstack-cinder | 14:32 | |
*** mriedem is now known as mriedem_afk | 14:34 | |
*** jgriffith has joined #openstack-cinder | 14:35 | |
*** imacdonn has quit IRC | 14:39 | |
*** mriedem_afk is now known as mriedem | 14:46 | |
whoami-rajat | hemna: Hey, could you please check if my testing was done correctly. | 14:46 |
*** sapd1 has quit IRC | 14:51 | |
*** eharney has joined #openstack-cinder | 14:53 | |
*** markvoelker has quit IRC | 14:53 | |
*** Bhujay has quit IRC | 14:56 | |
*** gnufied has joined #openstack-cinder | 15:04 | |
*** h3xagon has joined #openstack-cinder | 15:14 | |
h3xagon | Hi all. Running a two-node cluster, cinder -> lvm as storage on compute. The array is 6 x Samsung 845DC Pro 800 SSD's in a Raid 5 with an LSI MegaRAID 2208 controller (Dell box). Some IOPS calculators are showing we should be seeing like 200k+ IOPS with this setup and with fio (4k block) sequential read tests we're only seeing like 54k. Are the calculators wrong (which I'm doubtful of) or is something up with what we have? | 15:19 |
h3xagon | Had someone basically cite cinder as a performance problem on this, but the test are being run on the bare metal os (which is Ubuntu 16.04) | 15:20 |
smcginnis | h3xagon: When talking IO performance, Cinder isn't even in the picture. | 15:21 |
*** thgcorrea has quit IRC | 15:22 | |
h3xagon | smcginnis, That's kind of what I was thinking... I've dealt with things along these lines for a while so it seemed an odd suggestion to me. | 15:22 |
smcginnis | h3xagon: I think you would have to look at different Linux tuning parameters to figure out why things are slow. | 15:24 |
smcginnis | Maybe some tips in - https://cromwell-intl.com/open-source/performance-tuning/disks.html | 15:24 |
*** ondrejme has quit IRC | 15:25 | |
smcginnis | If there's some kind of set up thing Cinder can do to improve default settings, we could definitely look in to that. | 15:25 |
h3xagon | Understood and appreciated, will definitely look into that. For my own sanity seeing 54k read iops on a raid5 with disks that claim 90k capable "IS" indeed off? I.e. I have a problem to fix at that level first. | 15:26 |
smcginnis | h3xagon: If you have the ability, might be best to take OpenStack out of the picture and just see what kind of benchmarks you get just with Linux and a drive. Then see if adding things like LVM on top introduces the performance degradation. | 15:29 |
openstackgerrit | Rajat Dhasmana proposed openstack/python-cinderclient master: Add 'is_public' option for cinder type-list command https://review.openstack.org/641698 | 15:31 |
*** thgcorrea has joined #openstack-cinder | 15:31 | |
h3xagon | smcginnis, great suggestion and will likely go that route to determine the bottleneck. Mainly we have some folks questioning wither 10k iops is a "legit" number for 6x800GB SSD drives. I'm saying seems low, by a lot. =) | 15:32 |
h3xagon | Thank you. =) | 15:32 |
*** Luzi has quit IRC | 15:35 | |
smcginnis | Good luck! | 15:35 |
hemna | whoami-rajat: sup | 15:45 |
whoami-rajat | hemna: hey, I tried the rally cinder jobs with convert_to_base=False as default, I've provided all the links above, could you please verify that I didn't miss anything while testing it. | 15:48 |
hemna | whoami-rajat: File Not Found | 15:49 |
whoami-rajat | Which one ? | 15:50 |
*** markvoelker has joined #openstack-cinder | 15:50 | |
hemna | http://logs.openstack.org/73/641573/1/check/rally-task-cinder/c86477a/result | 15:55 |
hemna | that one | 15:55 |
*** _alastor_ has joined #openstack-cinder | 16:00 | |
*** udesale has quit IRC | 16:01 | |
whoami-rajat | hemna: i'm not sure where that link came from. | 16:04 |
whoami-rajat | this is my original msg. | 16:04 |
whoami-rajat | I put up a patch with convert_to_base =False as default[1], tested cinder rally job [2] (specifically create_volume_from_snaphot [3]), and it passed (if my approach to test it was correct). | 16:04 |
whoami-rajat | 3:12 PM The bug reported for it's failure[4] | 16:04 |
whoami-rajat | 3:12 PM [1] https://review.openstack.org/#/c/641567/ | 16:04 |
whoami-rajat | 3:12 PM [2] https://review.openstack.org/#/c/641573/ | 16:04 |
whoami-rajat | 3:12 PM [3] http://logs.openstack.org/73/641573/1/check/rally-task-cinder/c86477a/results/report.html.gz#/CinderVolumes.create_volume_from_snapshot | 16:04 |
whoami-rajat | 3:12 PM [4] https://bugs.launchpad.net/cinder/+bug/1656186 | 16:04 |
openstack | Launchpad bug 1656186 in Cinder "Error running rally test scenario 'create-volume-from-snapshot'" [Undecided,Fix released] - Assigned to Vijay Ladani (vijay-ladani) | 16:04 |
*** enriquetaso has quit IRC | 16:10 | |
*** alkhodos has joined #openstack-cinder | 16:14 | |
*** anks2k has quit IRC | 16:20 | |
*** markvoelker has quit IRC | 16:23 | |
*** jmlowe has quit IRC | 16:36 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Handling filters in Generalized Filtering API https://review.openstack.org/641717 | 16:37 |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Handling filters in Generalized Filtering API https://review.openstack.org/641717 | 16:43 |
whoami-rajat | eharney smcginnis minimal code change that i could do to make these filters work ^ | 16:47 |
smcginnis | whoami-rajat: Thanks | 16:47 |
whoami-rajat | smcginnis: :) | 16:47 |
eharney | whoami-rajat: it looks about right at a glance to me | 16:48 |
whoami-rajat | eharney: i tested out with few commands, works fine in my env. :) | 16:49 |
eharney | whoami-rajat: i need to think a little on how to validate the "do all of the expected paths perform the right checking w/ all_tenants and admin-ness etc" concern | 16:50 |
whoami-rajat | eharney: do you mean how handling is done for 'all_tenants=0/1' in every resource (after passing filtering) | 16:56 |
eharney | yes | 16:56 |
eharney | especially if any resources were added after the generic filtering code was | 16:56 |
whoami-rajat | eharney: good point. i'm not sure this filtering check is added to resources added after 3.31 | 16:57 |
whoami-rajat | eharney: will take a look at that part. Thanks! | 16:58 |
*** e0ne has quit IRC | 17:07 | |
*** markvoelker has joined #openstack-cinder | 17:20 | |
openstackgerrit | Eric Harney proposed openstack/python-cinderclient master: Add bash completion for groups https://review.openstack.org/641733 | 17:26 |
*** enriquetaso has joined #openstack-cinder | 17:28 | |
*** senrique_ has joined #openstack-cinder | 17:30 | |
*** enriquetaso has quit IRC | 17:33 | |
*** imacdonn has joined #openstack-cinder | 17:37 | |
*** senrique_ has quit IRC | 17:42 | |
*** senrique_ has joined #openstack-cinder | 17:43 | |
*** jmlowe has joined #openstack-cinder | 17:49 | |
*** markvoelker has quit IRC | 17:53 | |
*** gkadam__ has quit IRC | 17:53 | |
*** mriedem is now known as mriedem_afk | 17:58 | |
alkhodos | smcginnis: Hi Sean! Please take a look at the latest patchset here https://review.openstack.org/#/c/586239/, addressed all your comments. | 18:00 |
smcginnis | alkhodos: Thanks, I'll try to get to it a little later. | 18:01 |
smcginnis | jungleboyj: You have a -2 on that from awhile back that should probably be removed. ^ | 18:01 |
*** irclogbot_0 has joined #openstack-cinder | 18:02 | |
whoami-rajat | in merge conflict ^ | 18:04 |
*** senrique_ has quit IRC | 18:18 | |
*** e0ne has joined #openstack-cinder | 18:19 | |
*** psachin has joined #openstack-cinder | 18:23 | |
*** e0ne has quit IRC | 18:24 | |
*** e0ne has joined #openstack-cinder | 18:29 | |
*** e0ne has quit IRC | 18:33 | |
*** mriedem_afk is now known as mriedem | 18:40 | |
*** pcaruana has quit IRC | 18:46 | |
erlon | eharney, ping | 18:49 |
eharney | erlon: hey | 18:50 |
*** markvoelker has joined #openstack-cinder | 18:51 | |
erlon | eharney, hey Eric, about your comment on: https://review.openstack.org/#/c/633596/5/cinder/volume/drivers/solidfire.py@a1124 | 18:51 |
erlon | eharney, it was working before, but on python2. For python 3 there was an error | 18:51 |
eharney | erlon: are you sure it wasn't just broken on python2 in general? | 18:53 |
*** vishakha has quit IRC | 18:54 | |
*** bal has quit IRC | 18:57 | |
erlon | eharney, the code was wrong but the path was not being hit, and CI was passing in all test. We only hit the error after trying to run on python3. | 18:58 |
erlon | eharney, but the fix address the wrong behaviour too. | 18:59 |
eharney | not worried about CI passing... i was more worried about someone trying to backport a fix later that's an actual bug but is hidden in a patch that just says "fix python3" | 18:59 |
eharney | but if it never actually gets hit, i guess it doesn't matter much | 19:00 |
erlon | eharney, I see, there would be hard to find out that this fix would also fix the other problem | 19:03 |
whoami-rajat | erlon: in py2 it was working but not correctly. | 19:04 |
erlon | eharney, may be making the commit message more generic also including that it fixes that hidden issue | 19:04 |
erlon | whoami-rajat, ^ | 19:04 |
*** pcaruana has joined #openstack-cinder | 19:05 | |
whoami-rajat | erlon: was gonna suggest the same. Updating commit msg would be good. | 19:05 |
erlon | whoami-rajat, ok | 19:06 |
*** senrique_ has joined #openstack-cinder | 19:14 | |
*** markvoelker has quit IRC | 19:23 | |
*** luizbag has quit IRC | 19:28 | |
*** pcaruana has quit IRC | 19:33 | |
*** senrique__ has joined #openstack-cinder | 19:39 | |
*** senrique_ has quit IRC | 19:41 | |
*** thgcorrea has quit IRC | 20:02 | |
*** eharney has quit IRC | 20:06 | |
openstackgerrit | Alexey Khodos proposed openstack/cinder master: Refactored NexentaStor5 driver https://review.openstack.org/586239 | 20:08 |
*** rchurch_ has joined #openstack-cinder | 20:15 | |
*** rchurch has quit IRC | 20:17 | |
*** psachin has quit IRC | 20:17 | |
*** markvoelker has joined #openstack-cinder | 20:20 | |
*** e0ne has joined #openstack-cinder | 20:35 | |
*** senrique__ has quit IRC | 20:36 | |
*** senrique__ has joined #openstack-cinder | 20:39 | |
*** hemna has quit IRC | 20:41 | |
openstackgerrit | Tejdeep Kautharam proposed openstack/os-brick master: _get_system_uuid function in the os_brick/initiator/connectors/nvme.py line#64 is returning uuid with trailing newline character. removed new line before function return. https://review.openstack.org/641796 | 20:49 |
*** markvoelker has quit IRC | 20:53 | |
*** eharney has joined #openstack-cinder | 20:54 | |
*** erlon has quit IRC | 21:05 | |
*** e0ne has quit IRC | 21:14 | |
*** whoami-rajat has quit IRC | 21:17 | |
*** carlos_silva has quit IRC | 21:33 | |
*** markvoelker has joined #openstack-cinder | 21:50 | |
*** e0ne has joined #openstack-cinder | 22:01 | |
*** dave-mccowan has quit IRC | 22:02 | |
*** lseki has quit IRC | 22:10 | |
*** abishop has quit IRC | 22:10 | |
*** markvoelker has quit IRC | 22:23 | |
*** e0ne has quit IRC | 22:34 | |
*** e0ne has joined #openstack-cinder | 22:34 | |
*** e0ne has quit IRC | 22:36 | |
*** rcernin has joined #openstack-cinder | 22:41 | |
openstackgerrit | Merged openstack/cinder master: Add get_driver_options method https://review.openstack.org/635255 | 22:41 |
*** sapd1 has joined #openstack-cinder | 22:51 | |
*** tkajinam has joined #openstack-cinder | 22:58 | |
*** mriedem is now known as mriedem_away | 23:02 | |
*** senrique__ has quit IRC | 23:18 | |
*** dviroel_ has quit IRC | 23:19 | |
*** markvoelker has joined #openstack-cinder | 23:20 | |
*** irclogbot_0 has quit IRC | 23:26 | |
*** jmlowe has quit IRC | 23:46 | |
*** jmlowe has joined #openstack-cinder | 23:46 | |
*** efried has joined #openstack-cinder | 23:46 | |
efried | o/ | 23:47 |
efried | I noticed there's no os-brick release proposed. Shouldn't there be? | 23:48 |
efried | If nothing else to pick up https://review.openstack.org/#/c/635530/ | 23:48 |
efried | smcginnis, jungleboyj: ^ | 23:48 |
efried | proposed: https://review.openstack.org/641863 | 23:50 |
*** markvoelker has quit IRC | 23:52 | |
smcginnis | efried: There should have been last week. | 23:53 |
efried | smcginnis: Maybe I missed it? | 23:56 |
smcginnis | efried: No, jungleboyj did. :) | 23:56 |
smcginnis | efried: Now technically we should get a requirements FFE, but we'll see if requirements PTL is OK with it. | 23:56 |
efried | heh, okay. I didn't know whether os-brick was a subsidiary of nova or cinder. | 23:56 |
efried | anyway, it's up, and Thode seems okay with it. | 23:57 |
smcginnis | Yeah, owned by the Cinder team. | 23:57 |
efried | okay. | 23:57 |
smcginnis | Oh good, quick response there. | 23:57 |
efried | though it's too late for the scaleio rename to help, cause I dropped the ball on https://review.openstack.org/#/c/634866/ | 23:58 |
efried | I should have realized I might be Fred. | 23:58 |
smcginnis | efried: Eh, at least it will be ready for train and we won't have to wait for lib updates. | 23:58 |
efried | ++ | 23:58 |
smcginnis | Just a rebranding, so I don't get too overly concerned about those. | 23:58 |
efried | okay, I've done enough damage for one day. o/ | 23:59 |
smcginnis | o/ | 23:59 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!