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

Support multiple images #10

Open
yuvipanda opened this issue Jan 23, 2019 · 2 comments
Open

Support multiple images #10

yuvipanda opened this issue Jan 23, 2019 · 2 comments

Comments

@yuvipanda
Copy link
Collaborator

We currently only support a single image per deployment. We should support multiple images instead, and integrate that with kubespawner profiles somehow.

@rabernat
Copy link
Collaborator

I was thinking about something related in pangeo-data/pangeo-cloud-federation#56.

For a production cluster, it seems important to have a certain level of persistence and versioning of the images. Like, if I am using a container named gcr.io/pangeo-181919/ocean-pangeo-io-notebook:327e7f6, I should always have the option to launch that same container, even if new ones come out. This allows to maintain long-term backwards compatibility.

An alternative would be to address this at the jupyter kernel level. This is how we do it on standard, non-cloud-based jupyterhubs. We just keep making new kernels, but don't delete the old ones.

@salvis2
Copy link
Collaborator

salvis2 commented Aug 14, 2020

Looks like this issue will get some newfound attention. So hubploy.yaml defines images as a top-level section of the configuration. It would be easy to change that to a map and have several entries.

The image building would probably need another parameter to figure out which image to build. It could also have an option to build all of them in order.

I'm guessing the harder part will be how to inject this into the JupyterHub config. @yuvipanda do you have some suggestions on how to fiddle with this? I'm seeing that hubploy currently defines the path jupyterhub.singleuser.image in class LocalImage of config.py. We could make a new class / extend that class to structure a map that is assigned as the value of the profileList.

We might also want to setup some way to pass in prebuilt Docker images that hubploy doesn't have to build but still needs to put in profileLIst.

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

No branches or pull requests

3 participants