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

Update JupyterHub helm chart version #468

Merged
merged 1 commit into from
Feb 23, 2018

Conversation

betatim
Copy link
Member

@betatim betatim commented Feb 23, 2018

The new version of the chart brings in a new culler that is better at
handling users that are slow to stop. See
jupyterhub/zero-to-jupyterhub-k8s#531

@@ -1,4 +1,4 @@
dependencies:
- name: jupyterhub
version: "0.6.0"
version: "v0.7-091be8d"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"v0.7..." or "0.7..."? Checking.

@betatim betatim force-pushed the update-jhub-chart-version branch 3 times, most recently from a79cd56 to 274654f Compare February 23, 2018 10:59
@minrk
Copy link
Member

minrk commented Feb 23, 2018

There's a bug in our build.py when building multiple images. It's reusing the last commit from the first image as the tag for subsequent images. This is fixed in the new standalone chartpress, which is picked up in jupyterhub/zero-to-jupyterhub-k8s#533 . If we merge that PR, then the next build of the jupyterhub chart should have the right images.

@betatim betatim force-pushed the update-jhub-chart-version branch from 274654f to 83095c6 Compare February 23, 2018 12:58
The new version of the chart brings in a new culler that is better at
handling users that are slow to stop. See
jupyterhub/zero-to-jupyterhub-k8s#531
@betatim betatim force-pushed the update-jhub-chart-version branch from 83095c6 to b7e6711 Compare February 23, 2018 13:18
@minrk minrk merged commit 3648dfa into jupyterhub:master Feb 23, 2018
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.

2 participants