-
Notifications
You must be signed in to change notification settings - Fork 65
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 hub image #1849
Update hub image #1849
Conversation
Merging this PR will trigger the following deployment actions. Support and Staging deployments
Production deployments
|
I think for dependabot to pick up these files, we should remove this line from the config: infrastructure/.github/dependabot.yaml Line 35 in 74b06d5
I believe then dependabot will look for any requirements files anywhere in the repo, as opposed to just the root |
Also remember that merging this will overwrite what was done manually in #1848 We should probably not wait for Yuvi to merge that and just do it now |
Hmm, I actually thought that the dependabot/dependabot-core#3951 has more context about specifying multiple directories for dependabot. It looks like we need to list each individual location for the requirements file |
Good point! I will merge it before this one |
RE dependabot: dang. Let's just merge as-is and I'll open a follow-up PR for dependabot |
🎉🎉🎉🎉 Monitor the deployment of the hubs here 👉 https://github.com/2i2c-org/infrastructure/actions/runs/3378915289 |
Thanks a lot @sgibson91! Should we note this somewhere to don't forget? Hmm, maybe re-open #1589 ? |
@GeorgianaElena I just went straight for the PR because I had the commit ready |
Thanks a lot for catching #1848 dependency, @sgibson91! |
This updates the hub image with the following:
oauthenticator
from the commit it was pinned, to the latest15.1.0
releasekubespawner
to this commit jupyterhub/kubespawner@fa19acdIt also puts the custom packages we're installing alogside their pinned versions inside a
requirement.txt
to be picked up by dependabot.Closes #1589 and closes #1825