-
Notifications
You must be signed in to change notification settings - Fork 130
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
Move from DockerHub to quay.io #508
Conversation
Companion PR to jupyterhub/zero-to-jupyterhub-k8s#3254. See that PR for more information.
@@ -68,7 +68,7 @@ jobs: | |||
# https://github.com/jupyterhub/configurable-http-proxy/settings/secrets/actions | |||
if: startsWith(github.ref, 'refs/tags/') | |||
run: | | |||
docker login -u "${{ secrets.DOCKERHUB_USERNAME }}" -p "${{ secrets.DOCKERHUB_TOKEN }}" | |||
docker login -u "${{ secrets.QUAY_USERNAME }}" -p "${{ secrets.QUAY_PASSWORD }}" quay.io |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This changes the creds, but doesn't change where it's pushed (next step in gettags needs quay.io
added, I think)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@minrk how about now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that's right, thanks.
Thanks @minrk. I'll use workflow dispatch to run this just in case. |
I just re-pushed 4.6.0 but realized that won't actually trigger the new workflow, only the old one. I don't want to make a fresh release, so I've just re-run |
Companion PR to
jupyterhub/zero-to-jupyterhub-k8s#3254. See that PR for more information.