Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use helm 2.15.1 #1453

Merged
merged 1 commit into from
Oct 24, 2019
Merged

Conversation

consideRatio
Copy link
Member

Helm 2.15.0 stopped using extensions/v1beta1 for tiller, which was
required for installation on k8s 1.16 as we have testing for. So now we
don't have to maintain a separate Helm version for k8s 1.16 tests.

@consideRatio
Copy link
Member Author

consideRatio commented Oct 19, 2019

I debugged this locally, and conclude that the error on running helm upgrade to install something fails, but if you run the command again it works... I did not find any error in the logs, and --debug didn't help me much.

Installing/upgrading the Helm chart on the Kubernetes cluster.
$ helm upgrade jh-dev ./jupyterhub --install --debug --namespace jh-dev --kube-context kubernetes-admin@jh-dev --values dev-config.yaml
[debug] Created tunnel using local port: '37061'

[debug] SERVER: "127.0.0.1:37061"

Release "jh-dev" does not exist. Installing it now.
[debug] CHART PATH: /home/erik/dev/contrib/jupyterhub/zero-to-jupyterhub-k8s/jupyterhub

Error: unsupported format 

@consideRatio
Copy link
Member Author

Fixed in Helm 2.15.1 I think:
helm/helm@41a697e

Helm 2.15.0 stopped using extensions/v1beta1 for tiller, which was
required for installation on k8s 1.16 as we have testing for. So now we
don't have to maintain a separate Helm version for k8s 1.16 tests.

Helm 2.15.1 reverted a breaking change in 2.15.0 that made helm upgrade
--install fail.
@consideRatio consideRatio changed the title Use helm 2.15.0 Use helm 2.15.1 Oct 24, 2019
@consideRatio consideRatio merged commit 8675406 into jupyterhub:master Oct 24, 2019
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant