Monday, 2018-12-03

*** eharney has quit IRC00:07
*** lixiaoy1 has joined #openstack-cinder00:29
*** brinzhang has joined #openstack-cinder01:15
brinzhangsmcginnis, jungleboyi: Is this need the prefix of the "os-backup-user-attr" with backup details? Can I can remove it?01:26
openstackgerritYingxin Cheng proposed openstack/os-brick master: Support RSD scenario of nvme connector  https://review.openstack.org/62025001:37
openstackgerritBrin Zhang proposed openstack/cinder-specs master: Add user_id attribute to the backup response  https://review.openstack.org/61549401:46
openstackgerritBrin Zhang proposed openstack/cinder master: Add user_id in backup list and show API  https://review.openstack.org/61512902:02
openstackgerritYingxin Cheng proposed openstack/cinder master: [wip] Cinder RSD Driver  https://review.openstack.org/62146502:18
*** bhagyashris_ has joined #openstack-cinder02:29
*** mhen has quit IRC02:38
*** mhen has joined #openstack-cinder02:39
*** brinzhang has quit IRC02:40
*** brinzhang has joined #openstack-cinder02:40
*** psachin has joined #openstack-cinder02:42
*** imacdonn has joined #openstack-cinder02:52
openstackgerritwanghao proposed openstack/cinder master: Support to query volume filter by changes since/before  https://review.openstack.org/35268803:08
openstackgerritwanghao proposed openstack/cinder master: Support to query volume filter by changes since/before  https://review.openstack.org/35268803:43
*** lixiaoy1 has quit IRC03:46
*** Nel1x has quit IRC04:39
*** brinzhang has quit IRC05:05
*** brinzhang has joined #openstack-cinder05:06
*** faizy98 has joined #openstack-cinder05:31
*** faizy98 has quit IRC05:32
*** faizy98 has joined #openstack-cinder05:32
*** abhishekk has joined #openstack-cinder05:36
*** gkadam has joined #openstack-cinder05:45
openstackgerritwanghao proposed openstack/cinder master: Support to query volume filter by changes since/before  https://review.openstack.org/35268805:47
*** faizy_ has joined #openstack-cinder05:50
*** faizy98 has quit IRC05:52
*** faizy_ has quit IRC05:59
*** elbragstad has quit IRC06:03
*** zul has quit IRC06:04
openstackgerritBrin Zhang proposed openstack/cinder master: Add user_id in backup list and show API  https://review.openstack.org/61512906:10
*** whoami-rajat has joined #openstack-cinder06:13
openstackgerritRajat Dhasmana proposed openstack/cinder master: Automating generation of volumes sample files  https://review.openstack.org/62147806:14
*** ircuser-1 has joined #openstack-cinder06:29
*** arne_wiebalck_ has joined #openstack-cinder06:41
*** arne_wiebalck_ has quit IRC06:46
*** rcernin has quit IRC06:57
*** Luzi has joined #openstack-cinder07:04
*** dpawlik has joined #openstack-cinder07:15
*** dpawlik has quit IRC07:20
*** dpawlik_ has joined #openstack-cinder07:20
*** pcaruana has joined #openstack-cinder07:25
*** lemko has joined #openstack-cinder07:40
*** Dinesh_Bhor has joined #openstack-cinder07:43
openstackgerritSilvan Kaiser proposed openstack/cinder stable/rocky: Stop cleaning images to be deleted in remotefs driver  https://review.openstack.org/62149607:47
*** Dinesh_Bhor has quit IRC07:55
*** e0ne has joined #openstack-cinder08:04
*** Dinesh_Bhor has joined #openstack-cinder08:04
*** Liang__ has joined #openstack-cinder08:17
*** Liang__ is now known as LiangFang08:18
*** trident has quit IRC08:28
*** trident has joined #openstack-cinder08:30
*** mszwed has quit IRC08:49
openstackgerritRajat Dhasmana proposed openstack/cinder master: Automating generation of volumes sample files  https://review.openstack.org/62147808:59
*** josecastroleon has joined #openstack-cinder09:07
josecastroleonhi, could you please have a look on https://review.openstack.org/#/c/589925/?09:09
*** geguileo has joined #openstack-cinder09:28
*** sri_ has joined #openstack-cinder09:31
openstackgerritRajat Dhasmana proposed openstack/cinder master: Adding missing Volume state to VolumeStatus enum  https://review.openstack.org/62153309:42
*** Dinesh_Bhor has quit IRC09:46
*** LiangFang has quit IRC09:46
*** erlon has joined #openstack-cinder09:50
openstackgerritGorka Eguileor proposed openstack/cinder master: Add cinderlib docs  https://review.openstack.org/62067009:54
openstackgerritGorka Eguileor proposed openstack/cinder master: Add cinderlib functional tests  https://review.openstack.org/62067109:54
*** bhagyashris_ has quit IRC09:57
*** ganso has joined #openstack-cinder10:05
*** Dinesh_Bhor has joined #openstack-cinder10:14
*** Dinesh_Bhor has quit IRC10:50
*** luizbag has joined #openstack-cinder10:50
*** mhen has quit IRC11:23
*** mhen has joined #openstack-cinder11:24
*** abhishekk has quit IRC11:27
*** arne_wiebalck has quit IRC11:31
*** arne_wiebalck has joined #openstack-cinder11:32
*** brinzhang has quit IRC11:37
*** whoami-rajat has quit IRC11:43
*** whoami-rajat has joined #openstack-cinder11:44
*** erlon has quit IRC11:46
*** erlon has joined #openstack-cinder12:03
*** lpetrut has joined #openstack-cinder12:52
*** erlon_ has joined #openstack-cinder12:52
*** lpetrut has quit IRC12:55
*** lpetrut has joined #openstack-cinder12:55
*** erlon has quit IRC12:56
*** dave-mccowan has joined #openstack-cinder12:56
*** abishop has joined #openstack-cinder13:06
*** tpsilva has joined #openstack-cinder13:15
erlon_kukacz, hey, was just about to leave last Friday13:20
erlon_Im checking the bug13:20
*** mszwed has joined #openstack-cinder13:22
mszwedsmcginnis: Hi, I'm not sure what tests are you talking about in your last comment here: https://review.openstack.org/#/c/572759/22 Could to explain it please?13:22
kukaczerlon_: sure, no worries. thanks for checking it!13:22
*** udesale has joined #openstack-cinder13:30
*** dave-mccowan has quit IRC13:32
*** zul has joined #openstack-cinder13:35
francoishi there, still waiting for a Workflow +1 on https://review.openstack.org/#/c/612393/13:36
francoisI'll propose the change for stable branches once it's merged in master13:37
*** jroll has quit IRC13:38
*** jroll has joined #openstack-cinder13:38
openstackgerritGorka Eguileor proposed openstack/cinder master: Return all iSCSI targets-portals  https://review.openstack.org/62157513:43
*** whoami-rajat has quit IRC13:50
*** jdillaman has joined #openstack-cinder13:52
*** josecastroleon has quit IRC13:56
openstackgerritGorka Eguileor proposed openstack/cinder master: Add cinderlib functional tests  https://review.openstack.org/62067113:57
*** finix has joined #openstack-cinder13:58
*** mriedem has joined #openstack-cinder14:07
*** Nel1x has joined #openstack-cinder14:08
*** psachin has quit IRC14:24
*** jmlowe has quit IRC14:25
*** jmlowe has joined #openstack-cinder14:26
*** SteelyDan is now known as dansmith14:28
*** arne_wiebalck_ has joined #openstack-cinder14:34
smcginnismszwed: Looking again, but if I remember right when I looked at the CI before it was only running 20-some tests rather than the 250+ expected to be run.14:45
smcginnisconsole log doesn't have test run output.14:46
smcginnisHeh, only 15 tests run.14:46
mszwedsmcginnis: where can I find the list of the tests that shold be run? lennyb_ is actually working on enabling these tests, so I'm not fully aware what are the requirements.14:48
smcginnismszwed: It should be running "tox -e all volume" for running tempest.14:48
mszwedok14:49
*** lbragstad has joined #openstack-cinder14:49
smcginnisIt appears all the Mellanox CI runs are only running 15-23 tests (< jungleboyj)14:49
*** dave-mccowan has joined #openstack-cinder14:55
*** lbragstad has quit IRC14:58
*** lbragstad has joined #openstack-cinder15:00
*** finix has quit IRC15:06
*** finix has joined #openstack-cinder15:07
lennyb_mszwed: smcginnis you can tempest that are currently run on SPDK Mellanox HW #link http://13.74.249.42/59/572759/22/check-cinder/SPDK/55d08a8/stestr_results.html.gz15:09
jungleboyjsmcginnis:  Yeah, I think we knew that was running a limited number of tests.15:12
*** gnufied has joined #openstack-cinder15:16
*** Luzi has quit IRC15:21
*** mchlumsky has joined #openstack-cinder15:24
jungleboyjsmcginnis: Cisco is running one test15:27
*** mchlumsky has quit IRC15:28
*** mchlumsky has joined #openstack-cinder15:29
*** dpawlik_ has quit IRC15:31
*** itlinux has quit IRC15:34
smcginnisHah!15:38
*** rosmaita has joined #openstack-cinder15:48
*** finix has quit IRC15:51
spotzhey jungleboyj smcginnis!15:53
*** rtjure has quit IRC15:55
*** lennyb_ has quit IRC15:55
smcginnisHowdy spotz15:55
*** lennyb has joined #openstack-cinder15:58
jungleboyjspotz:  Morning.16:02
jungleboyjsmcginnis:  Yeah, they are just doing the attach volume test.  So ....16:04
jungleboyjsmcginnis:  I have a topic for the mid-cycle to talk about what we do going forward with the CIs.16:04
*** jmlowe has quit IRC16:04
smcginnisThe set of tests Brocade was running (around 13 tests I think) is what Angela and I had pared down looking at what actually exercised the FCZM code.16:05
smcginnisAnd I could have sworn I had noticed that Cisco was not running much and told them those tests were what was needed a long time ago.16:05
*** rtjure has joined #openstack-cinder16:10
*** arne_wiebalck_ has quit IRC16:11
*** dpawlik has joined #openstack-cinder16:11
erlon_kukacz, hey, so, did a quick attempt to reproduce here16:15
*** dklyle has joined #openstack-cinder16:15
erlon_kukacz, the driver does not error, but the copy volume stays there16:16
*** dpawlik has quit IRC16:16
jungleboyjsmcginnis:  Brocade doesn't run anymore.  Is that what Mellanox replaced?16:18
erlon_ow, actually no, was just messing with a volume that was already there, the volume does not uses the generic process16:19
smcginnisNo. But when Brocade was running, I had referred them to those tests.16:19
*** dpawlik has joined #openstack-cinder16:20
kukaczerlon_: hmm, and you tested it with non-admin context and 2.0.7 driver? strange. what could be the difference then? I assume the Element OS release is not relevant here16:22
jungleboyjOk, so we have Brocade still missing, we have Cisco running one test and we have Mellanox running fewer tests than we expected.16:23
jungleboyjThe last Brocade test that passed just ran the attach/detach test.  http://144.49.208.28:8000/build_logs/2017-06-26_09-11-03/console.html.gz16:24
jungleboyj:-(16:24
erlon_kukacz, no it shouldn't. I tested with a non admin user, but the last version of the driver16:24
kukaczerlon_: with admin context (OS_PROJECT_NAME=admin, OS_USERNAME=admin) it works to me too. and of courses deletes the volume, as there's still the same one on the storage system16:24
*** arne_wiebalck_ has joined #openstack-cinder16:25
erlon_kukacz, you mean that even without the error poping, the aditional volume is created and left over?16:25
erlon_declare -x OS_AUTH_URL="http://192.168.10.15/identity/v3"16:26
erlon_declare -x OS_IDENTITY_API_VERSION="3"16:26
erlon_declare -x OS_PASSWORD="tijolo22"16:26
erlon_declare -x OS_PROJECT_DOMAIN_ID="default"16:26
erlon_declare -x OS_PROJECT_ID="ac57db7bc0184dbf9f6f9fd941a89131"16:26
erlon_declare -x OS_PROJECT_NAME="demo"16:26
erlon_declare -x OS_REGION_NAME="RegionOne"16:26
erlon_declare -x OS_USERNAME="erlon"16:26
erlon_declare -x OS_USER_DOMAIN_ID="default"16:26
erlon_declare -x OS_USER_DOMAIN_NAME="Default"16:26
kukaczerlon_: no, I mean it behaves correctly when run as admin. it just modifies the qos parameters on storage without creating new volume.16:26
erlon_kukacz, this generic fall back happens because of the first error16:30
kukaczerlon_: I'm commonly not using ...DOMAIN_ID in openrc, using ...DOMAIN_NAME only. that does not matter perhaps16:31
*** arne_wiebalck_ has quit IRC16:31
erlon_kukacz, no, it shouldn't. Im using that because of something in this last version that requires it to run as user16:32
*** itlinux has joined #openstack-cinder16:33
*** itlinux has quit IRC16:33
*** itlinux has joined #openstack-cinder16:34
*** lpetrut has quit IRC16:36
kukaczerlon_: what did you mean by the first error?16:36
erlon_kukacz, the admin context error. When it happens, the volume manager will try to next approach which is the generic16:38
kukaczerlon_: ahh, ok. is there some relevant difference between 2.0.7 and your version which could explain the issue?16:40
*** dpawlik has quit IRC16:42
erlon_kukacz, you said you are running ocata right?16:42
kukaczerlon_: yes16:42
*** e0ne has quit IRC16:48
*** itlinux_ has joined #openstack-cinder16:50
erlon_kukacz, yes, i think that was fixed here: https://github.com/openstack/cinder/blob/57669939789229d3efefed92b41b7f402f10f7b0/cinder/volume/drivers/solidfire.py#L87016:50
erlon_kukacz, did you tried to add ctxt = context.get_admin_context() before the qos_specs.get_qos_specs() call?16:51
erlon_if you have access to that code16:51
*** dpawlik has joined #openstack-cinder16:52
erlon_kukacz, https://review.openstack.org/#/c/529395/16:53
*** itlinux has quit IRC16:54
*** itlinux_ has quit IRC16:55
*** itlinux has joined #openstack-cinder16:56
kukaczerlon_: no, I did not try that. I can do now. should be safe, right?16:59
*** udesale has quit IRC17:02
erlon_kukacz, yes,, that is safe17:03
*** _hemna has quit IRC17:17
*** _hemna has joined #openstack-cinder17:18
*** dpawlik has quit IRC17:19
kukaczerlon_: this solved it! it converts without fall back now. thank you a lot!17:33
*** dpawlik has joined #openstack-cinder17:35
erlon_kukacz, cool! I think that patching that yourself is the quickest (may be only) way to fix it17:36
erlon_as ocata is no longer supported by the OS distros17:37
*** dpawlik has quit IRC17:39
kukaczerlon_: in our case it's still supported (Mirantis MCP). but it's not such a big deal to keep the patch tracked and applied manually through updates17:41
*** luizbag has quit IRC17:44
*** erlon_ has quit IRC17:44
*** hedvig_01 has joined #openstack-cinder17:48
openstackgerritGorka Eguileor proposed openstack/cinder master: Add cinderlib functional tests  https://review.openstack.org/62067117:55
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax driver - rebranding VMAX to PowerMax  https://review.openstack.org/62065917:59
*** erlon has joined #openstack-cinder18:01
*** e0ne has joined #openstack-cinder18:03
geguileoerlon: hi, I submitted a patch to fix the NetApp iSCSI drivers issue when the selected IP is not accessible18:20
geguileoerlon: but I don't see the NetApp CI kicking in, is it running fine? https://review.openstack.org/#/c/62157518:20
erlongeguileo, hey, yeah, we are having some issues these last weeks18:21
geguileoerlon: anything I can do to see it running for that patch?18:21
erlongeguileo, Ill keep an eye in you patch and re-enqueue when CI is up18:21
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax driver - rebranding VMAX to PowerMax  https://review.openstack.org/62065918:22
geguileoerlon: THANKS!!!18:23
jgriffitherlon: hey ya.. did you see the conversation around this one:  https://bugs.launchpad.net/cinder/+bug/180608418:23
openstackLaunchpad bug 1806084 in Cinder "Retyped SolidFire volume not being deleted from backend after fallback to generic mechanism" [Undecided,New]18:23
jgriffithOh, hahah, never mind18:23
* jgriffith needs to read scroll back more often18:24
geguileoerlon: I'll neet someone from NetApp to review that  ;-)18:24
erlongeguileo, your patch it is actually half way through :)18:25
geguileoerlon: Awesome!!!  :-)18:25
erlonjgriffith, hey18:26
erlonjgriffith, yes I talked to today18:26
erlonjgriffith, you have actually fixed that issue18:26
*** udesale has joined #openstack-cinder18:26
jgriffithYeah, just not back in O :(18:26
erlonow my! did I just ping all j* ?18:27
erlonlol, sorry18:27
erloncrazy IRC client18:27
jgriffithhaha... I don't know; I'd be surprised if that works.  I'm amazed I've never done that (accidentally)18:28
erlongeguileo, sure, I will18:28
erlonjgriffith, ow, no, my client is showing people that has left with the same color/appearance as my message, so18:29
geguileoerlon: thanks :-)18:30
erlongeguileo, are you on a hush on that? we will probably test it with the multipathing18:31
jgriffithphewww :)18:31
erlonour CI is not doing that right now IIRC18:31
geguileoerlon: ouch, I assumed your CI would run it multipathing18:35
geguileoerlon: although I may be able to confirm that the patch is working even without multipathing enabled (if there are multiple interfaces in your backend)18:35
erlongeguileo, it use to run. Then was a time when something got broken in os-brick and we changed it in order to fix, so, not sure if we have re-added it18:36
*** jmlowe has joined #openstack-cinder18:36
geguileolol18:36
*** jmlowe has quit IRC18:39
*** arne_wiebalck has quit IRC18:44
erlongeguileo, when the logs gets posted, if you need to see them, please replace 10.63.*.* ip by -> 13.57.58.9618:49
*** hedvig_01 has quit IRC18:55
*** erlon has quit IRC19:03
*** e0ne has quit IRC19:04
*** jmlowe has joined #openstack-cinder19:07
*** luizbag has joined #openstack-cinder19:12
*** e0ne has joined #openstack-cinder19:17
*** luizbag has quit IRC19:23
openstackgerritDavid Hill proposed openstack/cinder master: Add secret=true to fixed_key configuration parameter  https://review.openstack.org/62168619:43
*** e0ne has quit IRC19:51
*** e0ne has joined #openstack-cinder19:52
*** tpsilva has quit IRC20:01
*** ganso has quit IRC20:02
*** e0ne has quit IRC20:02
*** e0ne has joined #openstack-cinder20:03
*** e0ne has quit IRC20:10
*** david-lyle has joined #openstack-cinder20:16
*** e0ne has joined #openstack-cinder20:17
*** dklyle has quit IRC20:18
*** mszwed has quit IRC20:19
*** e0ne has quit IRC20:21
*** jmlowe has quit IRC20:23
*** udesale has quit IRC20:30
*** jmlowe has joined #openstack-cinder20:39
*** jmlowe has quit IRC20:41
*** e0ne has joined #openstack-cinder20:42
*** e0ne has quit IRC20:42
*** udesale has joined #openstack-cinder20:58
*** udesale has quit IRC21:23
openstackgerritSofia Enriquez proposed openstack/cinder-tempest-plugin master: Tempest coverage for volume retype+migrate  https://review.openstack.org/61402221:32
openstackgerritSofia Enriquez proposed openstack/cinder-tempest-plugin master: Tempest coverage for volume retype+migrate  https://review.openstack.org/61402221:40
*** hedvig_01 has joined #openstack-cinder21:57
*** abishop has quit IRC22:13
*** rcernin has joined #openstack-cinder22:18
*** pcaruana has quit IRC22:18
*** udesale has joined #openstack-cinder22:23
*** udesale has quit IRC22:25
*** udesale has joined #openstack-cinder22:26
*** itlinux has quit IRC22:38
*** mriedem is now known as mriedem_away22:40
*** udesale has quit IRC22:46
*** irclogbot_0 has quit IRC22:47
*** lbragstad has quit IRC22:51
*** lbragstad has joined #openstack-cinder22:52
*** irclogbot_0 has joined #openstack-cinder23:09
*** lemko has quit IRC23:29
*** Liang__ has joined #openstack-cinder23:40
*** gkadam has quit IRC23:48
*** Liang__ has quit IRC23:57

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