Friday, 2019-03-15

*** dpawlik has joined #openstack-cinder00:10
*** sapd1 has joined #openstack-cinder00:14
*** dpawlik has quit IRC00:15
*** sapd1 has quit IRC00:18
*** enriquetaso has joined #openstack-cinder00:20
*** enriquetaso has quit IRC00:25
*** jgriffith has quit IRC00:26
*** markvoelker has joined #openstack-cinder00:34
*** markvoelker has quit IRC00:39
*** dpawlik has joined #openstack-cinder00:52
*** dpawlik has quit IRC00:57
*** markvoelker has joined #openstack-cinder01:05
*** ruffian_sheep has joined #openstack-cinder01:12
*** whoami-rajat has joined #openstack-cinder01:17
ruffian_sheepe0ne:I have read the link you set,but ...I still dont know what the job our driver should do.The driver patch is this :https://review.openstack.org/#/c/612311/01:27
ruffian_sheepe0ne:We have run the Running Python Unit Tests.But I dont know what is the job should be set in the CI01:28
*** markvoelker has quit IRC01:32
*** zhubx has quit IRC01:33
*** zhubx has joined #openstack-cinder01:33
openstackgerritMerged openstack/os-brick stable/pike: Handle multiple errors in multipath -l parsing  https://review.openstack.org/64312001:35
*** ruffian_sheep has quit IRC01:46
*** dpawlik has joined #openstack-cinder02:07
*** penick has quit IRC02:09
*** dpawlik has quit IRC02:12
*** dpawlik has joined #openstack-cinder02:18
*** dpawlik has quit IRC02:22
openstackgerritfutaotao proposed openstack/cinder master: Modify the configuration mode of FusionStorage Cinder Driver  https://review.openstack.org/63298302:47
*** psachin has joined #openstack-cinder03:02
*** zhubx has quit IRC03:12
*** zhubx has joined #openstack-cinder03:13
LiangFanghi cores, anybody know how to get project id from project name?03:33
LiangFangit may need to query keystone to get project id03:34
LiangFangin cinder, is there any api to do this?03:37
*** udesale has joined #openstack-cinder03:50
openstackgerritMerged openstack/os-brick master: Fix ScaleIO KeyError after upgrade  https://review.openstack.org/64309204:06
whoami-rajatLiangFang:  Hi, why do you want to do that from cinder? could you explain the usecase.04:14
*** psachin has quit IRC04:30
*** dave-mccowan has quit IRC04:30
LiangFangwhoami-rajat: in order to enable the raw cache feature, the internal_project_id and internal_user_id need to be set in cinder.conf04:32
LiangFangbut in container environment, cinder.conf is filled by openstack_helm repo04:32
LiangFangand we find it's hard to get project_id at the time of deploy cinder container04:33
LiangFangso want to hardcode an internal_project_name and internal_user_name in values.yaml, so these hard coded info can be filled in cinder.conf04:35
LiangFangin cinder, a patch will be submitted, get_internal_tenant_context() will be modified. if there's internal_project_name and internal_user_name specified in cinder.conf, then it translate name to id04:37
*** lpetrut has joined #openstack-cinder04:47
*** dpawlik has joined #openstack-cinder04:52
*** dpawlik has quit IRC04:57
*** sapd1 has joined #openstack-cinder05:04
*** dpawlik has joined #openstack-cinder05:04
*** dpawlik has quit IRC05:08
*** lpetrut has quit IRC05:19
*** chhagarw has joined #openstack-cinder05:21
*** lpetrut has joined #openstack-cinder05:29
*** zhubx007 has joined #openstack-cinder05:34
*** zhubx has quit IRC05:34
*** dpawlik has joined #openstack-cinder05:42
*** lpetrut has quit IRC05:43
*** dpawlik has quit IRC05:47
*** rcernin has quit IRC05:58
*** rcernin has joined #openstack-cinder06:12
*** e0ne has joined #openstack-cinder06:24
*** e0ne has quit IRC06:27
whoami-rajatLiangFang:  AFAIK cinder has cinder_internal_tenant_project_id and cinder_internal_tenant_user_id parameters in cinder.conf and we fetch it to generate the RequestContext for internal requests.06:29
whoami-rajatI'm not much familiar with helm repo and how it deploys containers so i've a few queries,06:29
whoami-rajat1) what are the contents and purpose of values.yaml06:29
whoami-rajat2) from where you will get internal_project_name to hardcode it in values.yaml, if values.yaml copies the same parameters in cinder.conf and we've project info, can't we directly supply internal_project_id?06:29
*** yury-kulaz has joined #openstack-cinder06:43
LiangFangwhoami-rajat: to 1) values.yaml is in openstacl-helm repo, openstack-helm/cinder/values.yaml, you can clone the repo and see. It contains the default values for configure cinder containers. internal_project_name and internal_user_name want to be put there.06:56
LiangFangwhoami-rajat: 2) just hardcode a name, such as "internal_cinder_project" and "internal_cinder_user" in values.yaml. user can easily override the hardcode values via giving a parameter to helm ...06:58
*** Luzi has joined #openstack-cinder07:00
LiangFangat the time of deploying cinder containers, no cinder services are ready. helm then try to create project and user in a pod, use the name specified in values.yaml. But helm cannot get the project_id an user_id from pod, because when pod is running, configmap is already generated.07:03
LiangFangit is configmap which will fill cinder.conf07:03
LiangFangso configmap should get the internal_project_name and internal_user_name from values.yaml at beginning, then fill in cinder.conf, the run pod to create project and user using the name in values.yaml07:05
*** ruffian_sheep has joined #openstack-cinder07:06
ruffian_sheepwhoami-rajat:Hi,whoami-rajat.Do u know what the job should I set to the CI?07:07
*** yury-kulaz has quit IRC07:08
*** stakeda has joined #openstack-cinder07:14
*** dpawlik has joined #openstack-cinder07:15
*** sahid has joined #openstack-cinder07:35
*** pcaruana has joined #openstack-cinder07:37
openstackgerrithjy proposed openstack/cinder master: Add MacroSAN cinder driver  https://review.openstack.org/64351907:40
openstackgerrithjy proposed openstack/cinder master: Add MacroSAN cinder driver  https://review.openstack.org/64351907:57
*** rcernin has quit IRC07:59
*** dpawlik has quit IRC08:10
openstackgerritzhufl proposed openstack/cinder master: Fix kwargs passed to exception to get better format of error message  https://review.openstack.org/64352808:11
*** geguileo has quit IRC08:15
*** tkajinam has quit IRC08:18
whoami-rajatruffian_sheep:  i think the CI should run tempest tests.08:19
*** pcaruana has quit IRC08:24
*** pcaruana|afk| has joined #openstack-cinder08:24
*** mcgiggler has joined #openstack-cinder08:24
*** markvoelker has joined #openstack-cinder08:27
*** LiangFang has quit IRC08:31
whoami-rajatruffian_sheep:  you can take reference from other CI's like here08:32
whoami-rajathttps://openstackci.oracle.com/zfssa_ci/refs-changes-77-640077-3-iscsi/stestr_results.html08:32
*** e0ne has joined #openstack-cinder08:35
*** raghavendrat has quit IRC08:56
*** mszwed has joined #openstack-cinder08:59
openstackgerritRyan Liang proposed openstack/cinder master: VNX: delete the LUN from VNX backend  https://review.openstack.org/60590409:07
ruffian_sheepwhoami-rajat:Not yet09:07
*** dpawlik has joined #openstack-cinder09:13
*** lpetrut has joined #openstack-cinder09:20
*** logan- has quit IRC09:23
*** logan- has joined #openstack-cinder09:27
*** Dinesh_Bhor has joined #openstack-cinder09:30
*** Dinesh__Bhor has joined #openstack-cinder09:30
*** Dinesh__Bhor has quit IRC09:30
*** Dinesh_Bhor has quit IRC09:30
*** Dinesh_Bhor has joined #openstack-cinder09:31
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Fix cinderlib Python package  https://review.openstack.org/64301309:38
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add DevStack plugin  https://review.openstack.org/64301409:38
*** geguileo has joined #openstack-cinder09:38
*** dpawlik has quit IRC09:45
*** dpawlik has joined #openstack-cinder09:48
*** lpetrut has quit IRC10:02
*** martinkennelly has joined #openstack-cinder10:05
*** stakeda has quit IRC10:06
*** dpawlik has quit IRC10:07
openstackgerritRajat Dhasmana proposed openstack/cinder master: Gate Fix: Bumping versions in lower-constraints.txt  https://review.openstack.org/64355510:24
whoami-rajatgeguileo:  ^ making your patch dependent on this may pass it.10:26
*** lpetrut has joined #openstack-cinder10:27
geguileowhoami-rajat: thanks, I'll update the patch now  :-)10:28
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix VolumeAttachment is not bound to a Session  https://review.openstack.org/63254910:29
geguileowhoami-rajat: fingers crossed ^  :-)10:30
whoami-rajatgeguileo:  :)10:33
*** bal has joined #openstack-cinder10:41
*** dviroel_ has joined #openstack-cinder10:42
openstackgerritRajat Dhasmana proposed openstack/cinder master: Gate Fix: Bumping versions in lower-constraints.txt  https://review.openstack.org/64355510:43
*** dpawlik has joined #openstack-cinder10:45
*** mriedem has joined #openstack-cinder10:53
*** chhagarw has quit IRC10:53
*** udesale has quit IRC10:56
*** udesale has joined #openstack-cinder10:56
*** e0ne has quit IRC11:03
*** bal has quit IRC11:03
*** chhagarw has joined #openstack-cinder11:07
*** lemko has joined #openstack-cinder11:28
*** BlackDex has quit IRC11:28
*** thgcorrea has joined #openstack-cinder11:33
*** dpawlik has quit IRC11:33
*** abishop has joined #openstack-cinder11:36
*** ruffian_sheep has quit IRC11:52
*** BlackDex has joined #openstack-cinder11:57
*** dpawlik has joined #openstack-cinder11:58
efriedwhoami-rajat: os-brick ready for release?12:10
*** e0ne has joined #openstack-cinder12:12
*** abhishekk has joined #openstack-cinder12:13
*** FlorianFa has quit IRC12:14
*** sapd1_x has joined #openstack-cinder12:17
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Fix cinderlib Python package  https://review.openstack.org/64301312:19
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add DevStack plugin  https://review.openstack.org/64301412:19
*** markvoelker has quit IRC12:25
*** dcdamien has quit IRC12:26
*** pcaruana has joined #openstack-cinder12:27
*** dcdamien has joined #openstack-cinder12:27
*** pcaruana|afk| has quit IRC12:28
*** dpawlik has quit IRC12:33
*** chhagarw has quit IRC12:34
whoami-rajatefried: I think so, jungleboyj ?12:38
*** chhagarw has joined #openstack-cinder12:39
*** tobias-urdin has joined #openstack-cinder12:40
*** abhishekk has quit IRC12:53
*** dave-mccowan has joined #openstack-cinder13:02
efriedwhoami-rajat: I thought maybe prometheanfire was going to get impatient, but it sounds like he's not.13:04
efriedPretty sure jungleboyj said he was going to propose it, so I don't want to step on toes; I'm sure it's okay to wait for him.13:05
*** chhagarw has quit IRC13:11
*** chhagarw has joined #openstack-cinder13:11
*** lseki has joined #openstack-cinder13:11
*** eharney has quit IRC13:15
whoami-rajatefried: he will be available in a while, I'm sure he won't mind it but it's better to ensure everything's alright this time.13:19
efriedwhoami-rajat: He spoke up in -requirements - he's on it as soon as he gets to his desk.13:19
*** chhagarw has quit IRC13:20
*** dpawlik has joined #openstack-cinder13:20
*** enriquetaso has joined #openstack-cinder13:33
*** pcaruana has quit IRC13:36
*** lemko has quit IRC13:37
*** dpawlik has quit IRC13:52
*** dpawlik has joined #openstack-cinder13:54
*** e0ne has quit IRC13:58
*** eharney has joined #openstack-cinder13:59
*** TheJulia is now known as needssleep13:59
*** efried is now known as fried_rice14:00
*** e0ne has joined #openstack-cinder14:06
*** pcaruana has joined #openstack-cinder14:17
*** sahid has quit IRC14:29
*** sahid has joined #openstack-cinder14:29
*** dpawlik has quit IRC14:39
*** dpawlik has joined #openstack-cinder14:51
*** Luzi has quit IRC15:14
*** dpawlik has quit IRC15:22
*** zhubx007 has quit IRC15:33
*** zhubx007 has joined #openstack-cinder15:35
*** zhubx007 has quit IRC15:36
*** zhubx007 has joined #openstack-cinder15:37
smcginnisSo I missed this, but why are we doing all this busy work to revert that connector name change?15:38
smcginnisjungleboyj ^15:38
fried_ricesmcginnis: http://eavesdrop.openstack.org/irclogs/%23openstack-cinder/%23openstack-cinder.2019-03-12.log.html#t2019-03-12T16:01:0915:40
smcginnisfried_rice: Still not following why it's a critical issue that required a revert and FFE.15:41
smcginnisBecause we didn't want a warning message in the log saying the name was changing when it's going to be changing?15:42
fried_ricesmcginnis: It's not going to be changing.15:42
smcginnisIt will be.15:42
fried_riceIIUC the original rename was a bad idea that never should have landed.15:42
smcginnisNo, they are changing the name. That's not the name of their product any more, so whether it's in stein (not any more) or train, it's changing.15:42
whoami-rajateharney: due to bionic migration our gates are failing at lower constraints jobs, that's why i bumped the package versions.15:43
smcginnisWe can drop the warning message, but such a trivial thing that I don't follow why this blew up into such a big thing.15:43
fried_ricesmcginnis: I wasn't involved in those decisions, don't really understand why the rename was originally proposed, or the rationale behind reverting. But on the surface, it looks like "rename my product" doesn't affect "name of the internal connector" and we don't want the latter to be following corporate rebrandings of the former.15:43
jungleboyjsmcginnis:  Right, but the connector name is not user visible so we don't want to make changes like this.15:44
smcginnisWhy not.15:44
jungleboyjsmcginnis:  hemna_ can best explain why.15:44
smcginnisConfusing in the long run to have a connector that is named for something that no longer exists.15:44
smcginnisKeep the mapping in there.15:44
smcginnisDrop the warning log and just do this as a stable release after requirements freeze is over.15:45
jungleboyjBecause os-brick is not intended to have vendor naming in it.15:45
smcginnisThen it's in there and they can finish renaming their drivers in train without needing to do this whole song and dance again.15:45
smcginnisIntended or not, it does.15:45
jungleboyjUgh, ok so we spent a good part of the week spinning around this.  hemna_  felt this was critical, so we pushed it.15:46
jungleboyjI am sure that I am missing a detail in the explanation.15:46
jungleboyjAlso discovered that the connector for NVME shouldn't have been merged in as what Cinder actually does is NVMe-oF .15:47
whoami-rajatjungleboyj:  smcginnis is there a requirements freeze exception too?15:47
smcginnisYeah15:47
jungleboyjDidn't understand how NVME worked at the time.15:47
whoami-rajathttps://review.openstack.org/64355515:48
whoami-rajatThis is for our gate fix at lower constraints,15:48
*** e0ne has quit IRC15:51
eharneywhoami-rajat: right, i got that part..15:54
*** chhagarw has joined #openstack-cinder15:54
*** lpetrut has quit IRC15:56
*** lpetrut has joined #openstack-cinder15:56
smcginnisThe requirements freeze is really the openstack/requirements repo freeze. As long as we are using versions of packages that are already listed in there, we should be safe.15:58
*** udesale has quit IRC15:59
eharneysmcginnis: cool, makes sense16:02
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Fix RBD connector  https://review.openstack.org/64301516:13
*** lpetrut has quit IRC16:13
openstackgerritGorka Eguileor proposed openstack/cinderlib master: WIP/DROP: Test cross-repo on cinderlib  https://review.openstack.org/64362816:23
*** woojay has quit IRC16:24
*** kmalloc is now known as needscoffee16:25
martinkennellyhey all (hemna especially), any update on the review of the tempest tests/logs for the RSD CI? No pressure, just curious so I could resolve any issues you may have ASAP. Thank you for taking the time to do this.16:27
*** sapd1_x has quit IRC16:35
*** henriqueof has quit IRC16:48
*** walshh__ has joined #openstack-cinder16:53
whoami-rajatsmcginnis: we did the FFE due to avishay's revert on the wwn part, also hemna proposed a case when the os-brick renaming will cause a failure since it wasnt renamed in cinder and nova.16:53
*** mcgiggler has quit IRC17:01
jungleboyjwhoami-rajat:  I had a feeling I was forgetting a case that justified the change more.17:10
jungleboyjThanks for sharing that.  I have had too many fires burning this week.17:10
*** e0ne has joined #openstack-cinder17:10
*** e0ne has quit IRC17:11
*** itlinux has quit IRC17:18
*** woojay has joined #openstack-cinder17:20
*** sahid has quit IRC17:26
whoami-rajatjungleboyj: I thought it was missed in the above convo so just for info :)17:27
jungleboyjwhoami-rajat:  Yep.  Very helpful.17:28
*** zhubx007 has quit IRC17:31
*** zhubx007 has joined #openstack-cinder17:32
*** dcdamien has quit IRC17:38
*** tomtom001 has joined #openstack-cinder17:40
tomtom001hello, running OpenStack queens and running into an issue where volumes go into a creating state and never finish.  I see the request go into the cinder-api, but the scheduler nor the the volume service ever see the vol id or request id.  Does anyone know where I could start looking for the root cause of this issue?17:41
tomtom001Also, after checking logs there are no error messages reported.17:42
tomtom001Some volumes have been stuck in this state for two days now.17:42
*** walshh__ has quit IRC17:45
jungleboyjtomtom001:  It sounds like it is not finding a host for the volumes.17:46
*** jdillaman has quit IRC17:47
jungleboyjThe error message for that isn't as obvious.  What logging level to you have set?17:47
eharneyif the create volume message never reaches the scheduler due to a problem with amqp, you wouldn't see an error message -- probably need to examine that layer17:49
jungleboyjIt would be a message like this:17:49
jungleboyjhttps://ask.openstack.org/en/question/96614/no-valid-host-was-found-cinder-volume-create/17:49
jungleboyjeharney:  Oh good thought.17:49
tomtom001jungleboyj: have debug turned on, is there a loglevel setting for cinder?17:49
tomtom001eharney: I'll doublecheck rabbit, I've done that, and it seems ok, but I'll check again.17:50
jungleboyjYes, in cinder.conf you can set debug = True and verbose = True17:50
jungleboyjNeed to restart your services to set that.17:51
tomtom001jungleboyj: no problem will do17:51
jungleboyjtomtom001:  Cool.  Hope that might help you find something.17:52
tomtom001hmmm.... turning on verbose didn't generate any new log entries18:00
tomtom001so I have debug and verbose set to True18:00
jungleboyjYou restarted the services.18:06
tomtom001Yes18:07
tomtom001debug was already turned on18:07
openstackgerritMerged openstack/cinder master: Gate Fix: Bumping versions in lower-constraints.txt  https://review.openstack.org/64355518:10
jungleboyjtomtom001:  And AMQP looked ok?18:10
tomtom001jungleboyj: yeah, this is the strangest thing I've seen in a while....  How about this: http://paste.openstack.org/show/747901/  is empty body meaningful from the cinder-api log?18:12
tomtom001jungleboyj: i'm not seeing any refused connections/channel/vhost errors in rabbit, is there any specific rabbit steps I could take to ensure that the cinder messaging is functioning correctly?18:14
*** chhagarw has quit IRC18:14
tomtom001The rabbit cluster looks healthy.18:14
jungleboyjI think the empty body is normal.18:18
*** eharney has quit IRC18:38
*** bal has joined #openstack-cinder18:45
*** bal has quit IRC18:46
*** e0ne has joined #openstack-cinder18:53
*** martinkennelly has quit IRC19:05
*** gouthamr has quit IRC19:17
*** dave-mccowan has quit IRC19:18
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick master: Add generate_connector_list  https://review.openstack.org/64366719:31
*** eharney has joined #openstack-cinder19:33
*** dave-mccowan has joined #openstack-cinder19:33
*** mchlumsky_ has quit IRC19:34
*** e0ne has quit IRC19:37
*** carlos_silva has quit IRC19:37
*** openstack has joined #openstack-cinder19:45
*** ChanServ sets mode: +o openstack19:45
*** gouthamr has joined #openstack-cinder19:53
*** pcaruana has quit IRC19:59
*** woojay has quit IRC20:12
*** abishop has quit IRC20:20
*** thgcorrea has quit IRC20:24
*** gouthamr_ has joined #openstack-cinder20:29
*** jgriffith has joined #openstack-cinder20:30
*** jgriffith has quit IRC20:33
*** jgriffith has joined #openstack-cinder20:34
*** whoami-rajat has quit IRC20:46
*** martinkennelly has joined #openstack-cinder20:50
*** dviroel_ has quit IRC21:21
*** gouthamr has quit IRC21:22
*** dcdamien has joined #openstack-cinder21:42
*** woojay has joined #openstack-cinder21:48
*** KeithMnemonic has quit IRC21:51
*** dcdamien has quit IRC22:15
*** lseki has quit IRC22:49
*** markvoelker has joined #openstack-cinder22:51
*** _alastor_ has quit IRC23:09
*** rosmaita has left #openstack-cinder23:12
*** mriedem has quit IRC23:13
*** _alastor_ has joined #openstack-cinder23:21
*** martinkennelly has quit IRC23:27
openstackgerritMerged openstack/cinder master: NetApp SolidFire: Fix misbehavior on account creation  https://review.openstack.org/63359623:50

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