-
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
Upgrade oauthenticator version and docs #1583
Conversation
for more information, see https://pre-commit.ci
Support and Staging deployments
Production deployments
|
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
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.
LGTM! Thanks for this work!
Co-authored-by: Sarah Gibson <[email protected]>
Co-authored-by: Sarah Gibson <[email protected]>
Co-authored-by: Sarah Gibson <[email protected]>
🎉🎉🎉🎉 Monitor the deployment of the hubs here 👉 https://github.com/2i2c-org/infrastructure/actions/workflows/deploy-hubs.yaml?query=branch%3Amaster |
This PR is awesome, thanks for working on this one @GeorgianaElena.
Can we create an issue to follow up with a released version when it is released? I am always worried about using unreleased "versions" of upstream dependencies 😉 (sorry to be annoying with these recurrent requests!). |
Upgrade and pin
oauthenticator
version to latest commitUpdate docs about building hub image on an M1 arch
Update docs about cilogon oathenticator config to match current version of
oauthenticator
we're usingUpdate two hubs (
dask-staging
and2i2c-demo
) to use latest feature ofoauthenticator
, i.e.allowed_idps.idp.allowed_domains
.There are other hubs that could use this instead of the
username_pattern
we were using (esp thecloudbank
hubs), but I wasn't sure which identity providers they wished to use specifically, so I didn't change them. But I suggest using this pattern from now on.Ref: #1579 (comment)