linkyard / concourse-helm-resource

Deploy to kubernetes helm from your concourse.ci.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

helm version times out in Concourse in Kubernetes

andyhd opened this issue · comments

We have Concourse deployed in our Kubernetes cluster and when the concourse-helm-resource sets up helm, it hangs when running helm version:

Initializing kubectl...
Cluster "default" set.
User "admin" set.
Context "default" created.
Switched to context "default".
Client Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.6", GitCommit:"9f8ebd171479bec0ada837d7ee641dec2f8c6dd1", GitTreeState:"clean", BuildDate:"2018-03-21T15:21:50Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.9", GitCommit:"3fb1aafdafa3d33bc698930095db1e56c0f76452", GitTreeState:"clean", BuildDate:"2018-03-12T16:13:32Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}
Initializing helm...
Client: &version.Version{SemVer:"v2.8.2", GitCommit:"a80231648a1473929271764b920a8e346f6de844", GitTreeState:"clean"}
Kubernetes: &version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.9", GitCommit:"3fb1aafdafa3d33bc698930095db1e56c0f76452", GitTreeState:"clean", BuildDate:"2018-03-12T16:13:32Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}
[debug] context deadline exceeded
Error: cannot connect to Tiller

We tracked this down to setupConnection() in helm.go, which creates a tunnel to Tiller, unless HELM_HOST is set.

Setting HELM_HOST to tiller-deploy.kube-system:44134 solved our problem.

I have added a source configuration setting for HELM_HOST, so that this can be set in the pipeline, if you would like me to make a PR.

I have the same issue and I suspect we are seing this: helm/helm#2409 which should be fixed by helm/helm#3715. This has not made it into the released version of helm yet...

I am running into this issue. Setting HELM_HOST on my worker container does not seem to help. I'm not sure how I can work around this at this point.

helm/helm#3715 has been merged. Will make it into 2.9.0-RC4 which will be released tomorrow...

I’ll release a new version as soon as the final 2.9.0 is released

I have verified that 2.9.0-RC4 does not fix the issue. Setting the helm host manually does however. @msiegenthaler: would you consider accepting the PR that was suggested by @andyhd? The helm host seems like a reasonable parameter to be able to set even if this bug did not require it.

yes, I'd accept a PR that sets HELM_HOST. Can you open that PR @andyhd ?

Here it is, sorry for the delay!