Thursday, 2019-03-07

*** erlon has joined #openstack-cinder00:01
*** _alastor_ has quit IRC00:18
*** dave-mccowan has joined #openstack-cinder00:36
openstackgerritcaoyuan proposed openstack/cinder master: Update json module to jsonutils  https://review.openstack.org/63742800:44
openstackgerritcaoyuan proposed openstack/cinder master: Update json module to jsonutils  https://review.openstack.org/63742800:44
*** markvoelker has joined #openstack-cinder00:50
*** erlon has quit IRC00:51
*** stakeda has joined #openstack-cinder00:51
*** bhagyashris has joined #openstack-cinder01:01
*** whoami-rajat has joined #openstack-cinder01:07
*** dave-mccowan has quit IRC01:18
*** markvoelker has quit IRC01:22
*** erlon has joined #openstack-cinder01:30
*** rcernin has quit IRC01:31
*** rcernin has joined #openstack-cinder01:32
*** rchurch has joined #openstack-cinder01:36
*** rchurch_ has quit IRC01:38
*** erlon has quit IRC01:50
*** mriedem is now known as mriedem_afk01:51
openstackgerritMatt Riedemann proposed openstack/cinder master: docs: mark rbd driver support for extending an in-use volume  https://review.openstack.org/64153901:57
openstackgerritMatt Riedemann proposed openstack/cinder master: docs: mark rbd driver support for extending an in-use volume  https://review.openstack.org/64153901:58
*** mriedem_afk has quit IRC01:59
*** dave-mccowan has joined #openstack-cinder02:17
*** wanghao has joined #openstack-cinder02:19
*** markvoelker has joined #openstack-cinder02:20
*** dviroel_ has quit IRC02:42
*** psachin has joined #openstack-cinder02:42
*** markvoelker has quit IRC02:52
*** udesale has joined #openstack-cinder03:24
*** markvoelker has joined #openstack-cinder03:49
*** vishakha has joined #openstack-cinder03:50
*** dave-mccowan has quit IRC04:00
openstackgerritfutaotao proposed openstack/cinder master: Amend: Modify the configuration mode and add the release notes on FusionStorage Driver Refactor  https://review.openstack.org/63298304:03
*** alkhodos has quit IRC04:09
*** _alastor_ has joined #openstack-cinder04:18
*** markvoelker has quit IRC04:22
*** _alastor_ has quit IRC04:22
*** sdinescu has quit IRC04:26
*** strigazi has quit IRC04:26
*** arunkant has quit IRC04:26
*** kukacz has quit IRC04:26
*** cburgess has quit IRC04:26
*** penick has quit IRC04:26
*** gary_perkins has quit IRC04:26
*** bswartz has quit IRC04:26
*** sorrison has quit IRC04:26
*** kambiz has quit IRC04:26
*** abhishekk has joined #openstack-cinder04:29
*** sdinescu has joined #openstack-cinder04:53
*** strigazi has joined #openstack-cinder04:53
*** arunkant has joined #openstack-cinder04:53
*** kukacz has joined #openstack-cinder04:53
*** cburgess has joined #openstack-cinder04:53
*** penick has joined #openstack-cinder04:53
*** bswartz has joined #openstack-cinder04:53
*** gary_perkins has joined #openstack-cinder04:53
*** sorrison has joined #openstack-cinder04:53
*** kambiz has joined #openstack-cinder04:53
*** rcernin has quit IRC04:56
*** rcernin has joined #openstack-cinder05:02
*** sdinescu has quit IRC05:05
*** markvoelker has joined #openstack-cinder05:19
*** Bhujay has joined #openstack-cinder05:34
*** lemko has joined #openstack-cinder05:35
*** avishay has joined #openstack-cinder05:40
openstackgerritMerged openstack/cinder stable/ocata: Fix for auth version change in Brcd HTTP  https://review.openstack.org/64056705:53
*** markvoelker has quit IRC05:53
openstackgerritRajat Dhasmana proposed openstack/cinder master: DNM: Testing hpe3par driver against rally-openstack jobs  https://review.openstack.org/64156706:10
raghavendratwhoami-rajat: hemna: Thanks for bringing up various points for discussion06:27
raghavendratfor https://review.openstack.org/#/c/634119/ if i understand correctly, i should change default value of extra-spec to False06:31
*** e0ne has joined #openstack-cinder06:32
openstackgerritMerged openstack/cinder master: docs: mark rbd driver support for extending an in-use volume  https://review.openstack.org/64153906:40
*** avishay has quit IRC06:45
*** markvoelker has joined #openstack-cinder06:50
*** Luzi has joined #openstack-cinder06:54
*** dpawlik has joined #openstack-cinder07:05
*** Bhujay has quit IRC07:08
*** Bhujay has joined #openstack-cinder07:09
*** Bhujay has quit IRC07:10
*** Bhujay has joined #openstack-cinder07:10
*** Bhujay has quit IRC07:11
*** Bhujay has joined #openstack-cinder07:12
*** Bhujay has quit IRC07:13
*** Bhujay has joined #openstack-cinder07:13
*** Bhujay has quit IRC07:14
*** Bhujay has joined #openstack-cinder07:15
*** Bhujay has quit IRC07:16
*** Bhujay has joined #openstack-cinder07:16
*** e0ne has quit IRC07:17
*** Bhujay has quit IRC07:17
*** Bhujay has joined #openstack-cinder07:18
*** Bhujay has quit IRC07:19
*** Bhujay has joined #openstack-cinder07:19
*** Bhujay has quit IRC07:20
*** Bhujay has joined #openstack-cinder07:21
*** gkadam__ has joined #openstack-cinder07:21
*** e0ne has joined #openstack-cinder07:21
*** Bhujay has quit IRC07:22
*** markvoelker has quit IRC07:22
*** Bhujay has joined #openstack-cinder07:23
*** pcaruana has joined #openstack-cinder07:25
*** rcernin has quit IRC07:27
*** david-lyle has joined #openstack-cinder07:36
*** dklyle has quit IRC07:37
*** dpawlik has quit IRC07:39
*** dklyle has joined #openstack-cinder07:41
*** david-lyle has quit IRC07:42
*** lemko has quit IRC07:45
*** dpawlik has joined #openstack-cinder07:52
*** dklyle has quit IRC07:55
*** dklyle has joined #openstack-cinder07:55
*** e0ne has quit IRC08:04
*** avishay has joined #openstack-cinder08:05
*** sdinescu has joined #openstack-cinder08:09
*** tkajinam has quit IRC08:12
*** markvoelker has joined #openstack-cinder08:19
*** markvoelker has quit IRC08:53
openstackgerritzhulingjie proposed openstack/cinder master: Update json module to jsonutils  https://review.openstack.org/63742808:53
*** Bhujay has quit IRC08:55
*** Bhujay has joined #openstack-cinder08:56
*** Bhujay has quit IRC08:57
*** Bhujay has joined #openstack-cinder08:58
*** Bhujay has quit IRC08:59
*** Bhujay has joined #openstack-cinder08:59
*** Bhujay has quit IRC09:00
*** Bhujay has joined #openstack-cinder09:01
*** Bhujay has quit IRC09:02
*** Bhujay has joined #openstack-cinder09:02
*** Bhujay has quit IRC09:03
*** Bhujay has joined #openstack-cinder09:04
*** Bhujay has quit IRC09:05
*** Bhujay has joined #openstack-cinder09:05
*** Bhujay has quit IRC09:06
*** Bhujay has joined #openstack-cinder09:07
*** Bhujay has quit IRC09:08
*** Bhujay has joined #openstack-cinder09:08
*** Bhujay has quit IRC09:09
*** Bhujay has joined #openstack-cinder09:10
*** Bhujay has quit IRC09:11
*** Bhujay has joined #openstack-cinder09:11
*** Bhujay has quit IRC09:12
*** Bhujay has joined #openstack-cinder09:13
*** Bhujay has quit IRC09:14
*** Bhujay has joined #openstack-cinder09:14
*** Bhujay has quit IRC09:15
*** Bhujay has joined #openstack-cinder09:16
*** Bhujay has quit IRC09:17
*** Bhujay has joined #openstack-cinder09:17
openstackgerritzhulingjie proposed openstack/cinder master: Update json module to jsonutils  https://review.openstack.org/63742809:17
*** Bhujay has quit IRC09:18
*** Bhujay has joined #openstack-cinder09:19
*** Bhujay has quit IRC09:20
*** Bhujay has joined #openstack-cinder09:20
*** Bhujay has quit IRC09:21
*** Bhujay has joined #openstack-cinder09:22
*** Bhujay has quit IRC09:23
*** Bhujay has joined #openstack-cinder09:23
*** Bhujay has quit IRC09:24
*** Bhujay has joined #openstack-cinder09:25
*** Bhujay has quit IRC09:26
*** Bhujay has joined #openstack-cinder09:26
*** Bhujay has quit IRC09:27
*** Bhujay has joined #openstack-cinder09:28
*** Bhujay has quit IRC09:29
*** Bhujay has joined #openstack-cinder09:29
*** Bhujay has quit IRC09:30
*** Bhujay has joined #openstack-cinder09:31
*** Bhujay has quit IRC09:32
*** Bhujay has joined #openstack-cinder09:32
*** Bhujay has quit IRC09:33
*** Bhujay has joined #openstack-cinder09:34
*** Bhujay has quit IRC09:35
*** Bhujay has joined #openstack-cinder09:35
*** Bhujay has quit IRC09:36
*** Bhujay has joined #openstack-cinder09:37
*** Bhujay has quit IRC09:38
*** Bhujay has joined #openstack-cinder09:38
*** Bhujay has quit IRC09:39
*** Bhujay has joined #openstack-cinder09:40
*** Bhujay has quit IRC09:41
*** Bhujay has joined #openstack-cinder09:41
whoami-rajatraghavendrat:  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-rajatThe 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_snapshot09:42
whoami-rajat[4] https://bugs.launchpad.net/cinder/+bug/165618609:42
openstackLaunchpad 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 IRC09:42
*** Bhujay has joined #openstack-cinder09:43
*** Bhujay has quit IRC09:44
*** Bhujay has joined #openstack-cinder09:44
raghavendratwhoami-rajat: thanks. did you test deletion of volume v1 and snapshot s1 ?09:44
*** Bhujay has quit IRC09:45
*** Bhujay has joined #openstack-cinder09:46
*** Bhujay has quit IRC09:47
*** Bhujay has joined #openstack-cinder09:47
whoami-rajatraghavendrat: 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-L83509:48
*** Bhujay has quit IRC09:48
*** Bhujay has joined #openstack-cinder09:49
*** Bhujay has quit IRC09:50
*** markvoelker has joined #openstack-cinder09:50
*** Bhujay has joined #openstack-cinder09:50
*** Bhujay has quit IRC09:51
*** Bhujay has joined #openstack-cinder09:52
*** bhagyashris has quit IRC09:52
*** ianw is now known as ianw_pto09:52
*** Bhujay has quit IRC09:53
*** Bhujay has joined #openstack-cinder09:53
raghavendratwhoami-rajat: thanks.09:54
*** Bhujay has quit IRC09:54
*** Bhujay has joined #openstack-cinder09:55
*** Bhujay has quit IRC09:56
*** Bhujay has joined #openstack-cinder09:56
*** Bhujay has quit IRC09:57
*** Bhujay has joined #openstack-cinder09:58
*** stakeda has quit IRC09:59
*** Bhujay has quit IRC09:59
*** Bhujay has joined #openstack-cinder09:59
*** Bhujay has quit IRC10:00
*** Bhujay has joined #openstack-cinder10:01
*** Bhujay has quit IRC10:02
*** Bhujay has joined #openstack-cinder10:02
*** Bhujay has quit IRC10:03
*** Bhujay has joined #openstack-cinder10:04
*** Bhujay has quit IRC10:05
*** Bhujay has joined #openstack-cinder10:05
*** Bhujay has quit IRC10:06
*** Bhujay has joined #openstack-cinder10:07
*** Bhujay has quit IRC10:08
*** Bhujay has joined #openstack-cinder10:08
*** Bhujay has quit IRC10:09
*** Bhujay has joined #openstack-cinder10:10
*** Bhujay has quit IRC10:11
*** Bhujay has joined #openstack-cinder10:11
*** Bhujay has quit IRC10:12
*** Bhujay has joined #openstack-cinder10:13
*** Bhujay has quit IRC10:14
*** Bhujay has joined #openstack-cinder10:14
*** Bhujay has quit IRC10:15
*** Bhujay has joined #openstack-cinder10:16
*** Bhujay has quit IRC10:17
*** Bhujay has joined #openstack-cinder10:17
*** Bhujay has quit IRC10:18
*** Bhujay has joined #openstack-cinder10:19
*** Bhujay has quit IRC10:20
*** Bhujay has joined #openstack-cinder10:20
*** Bhujay has quit IRC10:21
*** Bhujay has joined #openstack-cinder10:22
*** Bhujay has quit IRC10:23
*** markvoelker has quit IRC10:23
*** Bhujay has joined #openstack-cinder10:23
*** wanghao has quit IRC10:24
*** Bhujay has quit IRC10:24
*** Bhujay has joined #openstack-cinder10:25
*** e0ne has joined #openstack-cinder10:25
*** Bhujay has quit IRC10:26
*** Bhujay has joined #openstack-cinder10:26
*** abhishekk has quit IRC10:27
*** Bhujay has quit IRC10:27
*** Bhujay has joined #openstack-cinder10:28
*** Bhujay has quit IRC10:29
*** Bhujay has joined #openstack-cinder10:29
*** dviroel_ has joined #openstack-cinder10:30
*** sapd1 has joined #openstack-cinder10:30
*** Bhujay has quit IRC10:30
*** Bhujay has joined #openstack-cinder10:31
*** Bhujay has quit IRC10:32
*** Bhujay has joined #openstack-cinder10:32
*** Bhujay has quit IRC10:33
*** Bhujay has joined #openstack-cinder10:34
*** Bhujay has quit IRC10:35
*** Bhujay has joined #openstack-cinder10:35
*** Bhujay has quit IRC10:36
*** luizbag has joined #openstack-cinder10:36
*** Bhujay has joined #openstack-cinder10:37
*** Bhujay has quit IRC10:38
*** Bhujay has joined #openstack-cinder10:38
*** Bhujay has quit IRC10:39
*** Bhujay has joined #openstack-cinder10:40
*** Bhujay has quit IRC10:41
*** Bhujay has joined #openstack-cinder10:41
*** Bhujay has quit IRC10:42
*** Bhujay has joined #openstack-cinder10:43
*** Bhujay has quit IRC10:44
*** Bhujay has joined #openstack-cinder10:44
*** Bhujay has quit IRC10:45
*** Bhujay has joined #openstack-cinder10:46
*** bal has joined #openstack-cinder10:51
*** carlos_silva has joined #openstack-cinder10:53
*** odyssey4me has quit IRC10:53
*** odyssey4me has joined #openstack-cinder10:53
*** udesale has quit IRC10:59
*** udesale has joined #openstack-cinder10:59
avishaygeguileo: 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 if11:08
avishaycalled repeatedly?11:08
geguileoavishay: (1): This looks like a pymysql thing, even if you have a newer version of the code... r:-???11:16
geguileoavishay: it should work, but aren't you checking if the id matches what you already have?11:16
geguileoavishay: because even with multiple calls it should always match11:17
geguileolast 2 lines are for (2)11:17
avishaygeguileo, 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
geguileoavishay: unfortunately the operation should be idempotent, and nova code can do that11:19
avishaygeguileo, 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
avishayOK11:19
*** markvoelker has joined #openstack-cinder11:20
geguileoavishay: check your DB version and compare it with CI jobs11:20
geguileoin case it's not the library but the DB itself the one that's doing something weird11:20
avishaygeguileo, Could it be the fact that I'm using galera (though always accessing the master)?11:21
geguileoavishay: dunno11:21
avishayOK thank you!11:21
*** erlon has joined #openstack-cinder11:24
*** thgcorrea has joined #openstack-cinder11:28
*** udesale has quit IRC11:28
*** abishop has quit IRC11:33
*** udesale has joined #openstack-cinder11:45
*** raghavendrat has quit IRC11:52
*** markvoelker has quit IRC11:53
*** sapd1 has quit IRC11:55
*** mdbooth has joined #openstack-cinder11:58
*** anks2k has joined #openstack-cinder12:06
openstackgerritYachika_Ralhan proposed openstack/cinder master: Add new OSNEXUS Quantastor Cinder driver  https://review.openstack.org/61750312:06
*** sapd1 has joined #openstack-cinder12:08
*** henriqueof has joined #openstack-cinder12:21
openstackgerritMerged openstack/cinder master: Revert "Use native python truncate for privsep"  https://review.openstack.org/64141312:22
openstackgerritMerged openstack/cinder master: Revert "Remove truncate from rootwrap filters"  https://review.openstack.org/64141412:22
*** dave-mccowan has joined #openstack-cinder12:22
*** gkadam__ has quit IRC12:32
*** gkadam__ has joined #openstack-cinder12:32
*** sapd1 has quit IRC12:36
*** raghavendrat has joined #openstack-cinder12:50
*** markvoelker has joined #openstack-cinder12:50
*** enriquetaso has joined #openstack-cinder12:52
*** udesale has quit IRC13:02
*** udesale has joined #openstack-cinder13:08
*** abishop has joined #openstack-cinder13:08
*** markvoelker has quit IRC13:22
enriquetasoabishop, 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
abishopenriquetaso: 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 project13:24
* enriquetaso facepalm 13:27
enriquetasoabishop, hahaha Thanks!13:28
enriquetasoI thought you were core :P13:28
abishopenriquetaso: not for puppet stuff13:28
*** mriedem has joined #openstack-cinder13:30
anks2kmy 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-rajatanks2k: could you please paste the link here?13:36
anks2khttps://review.openstack.org/#/c/617503/13:36
anks2kThanks !13:37
whoami-rajatanks2k: 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
anks2kOkay, Thanks.13:46
*** jmlowe has quit IRC13:51
*** sapd1 has joined #openstack-cinder13:54
*** BlackDex has quit IRC13:59
*** BlackDex has joined #openstack-cinder14:02
*** avishay has quit IRC14:06
hemnadoink14:11
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: Add get_driver_options method  https://review.openstack.org/63525514:16
*** _alastor_ has joined #openstack-cinder14:17
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: Add get_driver_options method  https://review.openstack.org/63525514:18
hemnafriggin finally14:18
hemnagit was messin w/ me.  kept reinjecting those @utils.trace lines14:18
*** markvoelker has joined #openstack-cinder14:20
*** mchlumsky has joined #openstack-cinder14:21
*** _alastor_ has quit IRC14:22
*** psachin has quit IRC14:24
*** jmlowe has joined #openstack-cinder14:30
*** lseki has joined #openstack-cinder14:32
*** mriedem is now known as mriedem_afk14:34
*** jgriffith has joined #openstack-cinder14:35
*** imacdonn has quit IRC14:39
*** mriedem_afk is now known as mriedem14:46
whoami-rajathemna: Hey, could you please check if my testing was done correctly.14:46
*** sapd1 has quit IRC14:51
*** eharney has joined #openstack-cinder14:53
*** markvoelker has quit IRC14:53
*** Bhujay has quit IRC14:56
*** gnufied has joined #openstack-cinder15:04
*** h3xagon has joined #openstack-cinder15:14
h3xagonHi 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
h3xagonHad 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
smcginnish3xagon: When talking IO performance, Cinder isn't even in the picture.15:21
*** thgcorrea has quit IRC15:22
h3xagonsmcginnis, 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
smcginnish3xagon: I think you would have to look at different Linux tuning parameters to figure out why things are slow.15:24
smcginnisMaybe some tips in - https://cromwell-intl.com/open-source/performance-tuning/disks.html15:24
*** ondrejme has quit IRC15:25
smcginnisIf there's some kind of set up thing Cinder can do to improve default settings, we could definitely look in to that.15:25
h3xagonUnderstood 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
smcginnish3xagon: 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
openstackgerritRajat Dhasmana proposed openstack/python-cinderclient master: Add 'is_public' option for cinder type-list command  https://review.openstack.org/64169815:31
*** thgcorrea has joined #openstack-cinder15:31
h3xagonsmcginnis, 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
h3xagonThank you. =)15:32
*** Luzi has quit IRC15:35
smcginnisGood luck!15:35
hemnawhoami-rajat: sup15:45
whoami-rajathemna: 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
hemnawhoami-rajat: File Not Found15:49
whoami-rajatWhich one ?15:50
*** markvoelker has joined #openstack-cinder15:50
hemnahttp://logs.openstack.org/73/641573/1/check/rally-task-cinder/c86477a/result15:55
hemnathat one15:55
*** _alastor_ has joined #openstack-cinder16:00
*** udesale has quit IRC16:01
whoami-rajathemna: i'm not sure where that link came from.16:04
whoami-rajatthis is my original msg.16:04
whoami-rajatI 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-rajat3:12 PM The bug reported for it's failure[4]16:04
whoami-rajat3:12 PM [1] https://review.openstack.org/#/c/641567/16:04
whoami-rajat3:12 PM [2] https://review.openstack.org/#/c/641573/16:04
whoami-rajat3:12 PM [3] http://logs.openstack.org/73/641573/1/check/rally-task-cinder/c86477a/results/report.html.gz#/CinderVolumes.create_volume_from_snapshot16:04
whoami-rajat3:12 PM [4] https://bugs.launchpad.net/cinder/+bug/165618616:04
openstackLaunchpad 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 IRC16:10
*** alkhodos has joined #openstack-cinder16:14
*** anks2k has quit IRC16:20
*** markvoelker has quit IRC16:23
*** jmlowe has quit IRC16:36
openstackgerritRajat Dhasmana proposed openstack/cinder master: Handling filters in Generalized Filtering API  https://review.openstack.org/64171716:37
openstackgerritRajat Dhasmana proposed openstack/cinder master: Handling filters in Generalized Filtering API  https://review.openstack.org/64171716:43
whoami-rajateharney smcginnis minimal code change that i could do to make these filters work ^16:47
smcginniswhoami-rajat: Thanks16:47
whoami-rajatsmcginnis:  :)16:47
eharneywhoami-rajat: it looks about right at a glance to me16:48
whoami-rajateharney: i tested out with few commands, works fine in my env. :)16:49
eharneywhoami-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" concern16:50
whoami-rajateharney: do you mean how handling is done for 'all_tenants=0/1' in every resource (after passing filtering)16:56
eharneyyes16:56
eharneyespecially if any resources were added after the generic filtering code was16:56
whoami-rajateharney: good point. i'm not sure this filtering check is added to resources added after 3.3116:57
whoami-rajateharney: will take a look at that part. Thanks!16:58
*** e0ne has quit IRC17:07
*** markvoelker has joined #openstack-cinder17:20
openstackgerritEric Harney proposed openstack/python-cinderclient master: Add bash completion for groups  https://review.openstack.org/64173317:26
*** enriquetaso has joined #openstack-cinder17:28
*** senrique_ has joined #openstack-cinder17:30
*** enriquetaso has quit IRC17:33
*** imacdonn has joined #openstack-cinder17:37
*** senrique_ has quit IRC17:42
*** senrique_ has joined #openstack-cinder17:43
*** jmlowe has joined #openstack-cinder17:49
*** markvoelker has quit IRC17:53
*** gkadam__ has quit IRC17:53
*** mriedem is now known as mriedem_afk17:58
alkhodossmcginnis: Hi Sean! Please take a look at the latest patchset here https://review.openstack.org/#/c/586239/, addressed all your comments.18:00
smcginnisalkhodos: Thanks, I'll try to get to it a little later.18:01
smcginnisjungleboyj: You have a -2 on that from awhile back that should probably be removed. ^18:01
*** irclogbot_0 has joined #openstack-cinder18:02
whoami-rajatin merge conflict ^18:04
*** senrique_ has quit IRC18:18
*** e0ne has joined #openstack-cinder18:19
*** psachin has joined #openstack-cinder18:23
*** e0ne has quit IRC18:24
*** e0ne has joined #openstack-cinder18:29
*** e0ne has quit IRC18:33
*** mriedem_afk is now known as mriedem18:40
*** pcaruana has quit IRC18:46
erloneharney, ping18:49
eharneyerlon: hey18:50
*** markvoelker has joined #openstack-cinder18:51
erloneharney, hey Eric, about your comment on: https://review.openstack.org/#/c/633596/5/cinder/volume/drivers/solidfire.py@a112418:51
erloneharney, it was working before, but on python2. For python 3 there was an error18:51
eharneyerlon: are you sure it wasn't just broken on python2 in general?18:53
*** vishakha has quit IRC18:54
*** bal has quit IRC18:57
erloneharney, 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
erloneharney, but the fix address the wrong behaviour too.18:59
eharneynot 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
eharneybut if it never actually gets hit, i guess it doesn't matter much19:00
erloneharney, I see, there would be hard to find out that this fix would also fix the other problem19:03
whoami-rajaterlon: in py2 it was working but not correctly.19:04
erloneharney, may be making the commit message more generic also including that it fixes that hidden issue19:04
erlonwhoami-rajat, ^19:04
*** pcaruana has joined #openstack-cinder19:05
whoami-rajaterlon: was gonna suggest the same. Updating commit msg would be good.19:05
erlonwhoami-rajat, ok19:06
*** senrique_ has joined #openstack-cinder19:14
*** markvoelker has quit IRC19:23
*** luizbag has quit IRC19:28
*** pcaruana has quit IRC19:33
*** senrique__ has joined #openstack-cinder19:39
*** senrique_ has quit IRC19:41
*** thgcorrea has quit IRC20:02
*** eharney has quit IRC20:06
openstackgerritAlexey Khodos proposed openstack/cinder master: Refactored NexentaStor5 driver  https://review.openstack.org/58623920:08
*** rchurch_ has joined #openstack-cinder20:15
*** rchurch has quit IRC20:17
*** psachin has quit IRC20:17
*** markvoelker has joined #openstack-cinder20:20
*** e0ne has joined #openstack-cinder20:35
*** senrique__ has quit IRC20:36
*** senrique__ has joined #openstack-cinder20:39
*** hemna has quit IRC20:41
openstackgerritTejdeep 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/64179620:49
*** markvoelker has quit IRC20:53
*** eharney has joined #openstack-cinder20:54
*** erlon has quit IRC21:05
*** e0ne has quit IRC21:14
*** whoami-rajat has quit IRC21:17
*** carlos_silva has quit IRC21:33
*** markvoelker has joined #openstack-cinder21:50
*** e0ne has joined #openstack-cinder22:01
*** dave-mccowan has quit IRC22:02
*** lseki has quit IRC22:10
*** abishop has quit IRC22:10
*** markvoelker has quit IRC22:23
*** e0ne has quit IRC22:34
*** e0ne has joined #openstack-cinder22:34
*** e0ne has quit IRC22:36
*** rcernin has joined #openstack-cinder22:41
openstackgerritMerged openstack/cinder master: Add get_driver_options method  https://review.openstack.org/63525522:41
*** sapd1 has joined #openstack-cinder22:51
*** tkajinam has joined #openstack-cinder22:58
*** mriedem is now known as mriedem_away23:02
*** senrique__ has quit IRC23:18
*** dviroel_ has quit IRC23:19
*** markvoelker has joined #openstack-cinder23:20
*** irclogbot_0 has quit IRC23:26
*** jmlowe has quit IRC23:46
*** jmlowe has joined #openstack-cinder23:46
*** efried has joined #openstack-cinder23:46
efriedo/23:47
efriedI noticed there's no os-brick release proposed. Shouldn't there be?23:48
efriedIf nothing else to pick up https://review.openstack.org/#/c/635530/23:48
efriedsmcginnis, jungleboyj: ^23:48
efriedproposed: https://review.openstack.org/64186323:50
*** markvoelker has quit IRC23:52
smcginnisefried: There should have been last week.23:53
efriedsmcginnis: Maybe I missed it?23:56
smcginnisefried: No, jungleboyj did. :)23:56
smcginnisefried: Now technically we should get a requirements FFE, but we'll see if requirements PTL is OK with it.23:56
efriedheh, okay. I didn't know whether os-brick was a subsidiary of nova or cinder.23:56
efriedanyway, it's up, and Thode seems okay with it.23:57
smcginnisYeah, owned by the Cinder team.23:57
efriedokay.23:57
smcginnisOh good, quick response there.23:57
efriedthough it's too late for the scaleio rename to help, cause I dropped the ball on https://review.openstack.org/#/c/634866/23:58
efriedI should have realized I might be Fred.23:58
smcginnisefried: Eh, at least it will be ready for train and we won't have to wait for lib updates.23:58
efried++23:58
smcginnisJust a rebranding, so I don't get too overly concerned about those.23:58
efriedokay, I've done enough damage for one day. o/23:59
smcginniso/23:59

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!