Skip to content

Conversation

yuvipanda
Copy link
Member

Both the images in use at UCMerced don't have nbgitpuller installed in them, while the previous combined image did.

I've been working on getting nbgitpuller into upstream jupyter docker-stacks (jupyter/docker-stacks#2000) for a while, but it looks like it'll take a bit longer. So I've temporarily created a repo (https://github.com/2i2c-org/scipy-notebook-with-nbgitpuller/) to use here. It's based on the same tag as before, but with nbgitpuller installed. We can get rid of this once nbgitpuller lands upstream.

I have created https://github.com/2i2c-org/rocker-with-nbgitpuller/ to do the same for use with rocker. This too is a temporary repo. I hopefully will have a better longer term solution speced out next week.

This unblocks ucmerced currently teaching.

Ref https://2i2c.freshdesk.com/a/tickets/1089

Both the images in use at UCMerced don't have nbgitpuller
installed in them, while the previous combined image did.

I've been working on getting nbgitpuller into upstream
jupyter docker-stacks (jupyter/docker-stacks#2000)
for a while, but it looks like it'll take a bit longer. So
I've temporarily created a repo (https://github.com/2i2c-org/scipy-notebook-with-nbgitpuller/)
to use here. It's based on the same tag as before, but with
nbgitpuller installed. We can get rid of this once nbgitpuller
lands upstream.

I have created https://github.com/2i2c-org/rocker-with-nbgitpuller/
to do the same for use with rocker. This too is a temporary repo.
I hopefully will have a better longer term solution speced out
next week.

This unblocks ucmerced currently teaching.

Ref https://2i2c.freshdesk.com/a/tickets/1089
@yuvipanda yuvipanda requested a review from a team as a code owner November 3, 2023 13:54
@yuvipanda
Copy link
Member Author

I've test deployed this onto the staging hub and verified that it works

Copy link

github-actions bot commented Nov 3, 2023

Merging this PR will trigger the following deployment actions.

Support and Staging deployments

Cloud Provider Cluster Name Upgrade Support? Reason for Support Redeploy Upgrade Staging? Reason for Staging Redeploy
gcp 2i2c No Yes Following helm chart values files were modified: ucmerced-staging.values.yaml

Production deployments

Cloud Provider Cluster Name Hub Name Reason for Redeploy
gcp 2i2c ucmerced Following helm chart values files were modified: ucmerced.values.yaml

Copy link
Contributor

@consideRatio consideRatio left a comment

Choose a reason for hiding this comment

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

I appreciate this solution and that the name is "scipy-notebook-with-nbgitpuller" as that is narrowly scoped.

It would been a plus to use date tags as the base images does, but its not worth blocking on!

@yuvipanda
Copy link
Member Author

Thanks for the review, @consideRatio!

@yuvipanda yuvipanda merged commit afb72db into 2i2c-org:master Nov 3, 2023
Copy link

github-actions bot commented Nov 3, 2023

🎉🎉🎉🎉

Monitor the deployment of the hubs here 👉 https://github.com/2i2c-org/infrastructure/actions/runs/6747479809

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done 🎉
Development

Successfully merging this pull request may close these issues.

2 participants