ml-hub icon indicating copy to clipboard operation
ml-hub copied to clipboard

upgrade to adopt the latest jupyterhub feature

Open comeUpWithItLater opened this issue 4 years ago • 2 comments

Feature description:

Upgrade the helm chart to adopt the latest feature that jupyterhub offer , such as Customizing User Storage

Problem and motivation:

This will ensure that the latest feature in jupyterhub is always available in ml-hub.

Is this something you're interested in working on?

Yes, l'd love to . But I need some guideline on where to start.

comeUpWithItLater avatar Jul 28 '21 08:07 comeUpWithItLater

I was going to make some changes in the helm chart but saw that most of what I'm trying to solve is already fixed in https://github.com/jupyterhub/zero-to-jupyterhub-k8s/tree/main/jupyterhub

@raethlein : would you consider removing all jupyterhub related things from this repository and use the helm chart from jupyterhub as dependency?

ClementGautier avatar Dec 22 '21 16:12 ClementGautier

Hey @comeUpWithItLater and @ClementGautier, thanks for looking into this! I think using the official jupyterhub helm chart would be great as this would be one less dependency to care about. Back then I think the decision to copy the helm chart / parts of it to this repo was that we needed to make some slight modifications (e.g. setting some environment variables, opening another port for ssh access etc.) that was not directly configurable in the jupyterhub helm chart. I have to look into the newest helm chart version to see whether it could not be easily configured without the need to copy it here

raethlein avatar Dec 23 '21 13:12 raethlein