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

Take into account all possible states of c.Authenticator.admin_users … #2299

Merged
merged 1 commit into from
Mar 3, 2023

Conversation

GeorgianaElena
Copy link
Member

@GeorgianaElena GeorgianaElena commented Mar 3, 2023

…list in a hub config

Fixes #1671 based on @sgibson91 's awesome detective work ✨

With this, we can remove redundant config that specifies "admin_users = []" in the config, but won't do it in this PR yet, because it might cause conflicts for #2277
Will do all necersary simplification of the auth config in that big PR instead.

This came up when playing with CILogon on the staging hubs, when I realized that the 2i2c dask-staging hub had no admin users.

@GeorgianaElena GeorgianaElena self-assigned this Mar 3, 2023
@github-actions
Copy link

github-actions bot commented Mar 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-uk No Yes Core infrastructure has been modified
gcp linked-earth No Yes Core infrastructure has been modified
gcp pangeo-hubs No Yes Core infrastructure has been modified
aws nasa-veda No Yes Core infrastructure has been modified
gcp callysto No Yes Core infrastructure has been modified
aws victor No Yes Core infrastructure has been modified
gcp leap No Yes Core infrastructure has been modified
aws ubc-eoas No Yes Core infrastructure has been modified
gcp 2i2c No Yes Core infrastructure has been modified
gcp cloudbank No Yes Core infrastructure has been modified
aws openscapes No Yes Core infrastructure has been modified
aws gridsst No Yes Core infrastructure has been modified
aws nasa-cryo No Yes Core infrastructure has been modified
aws 2i2c-aws-us No Yes Core infrastructure has been modified
aws carbonplan No Yes Core infrastructure has been modified
gcp m2lines No Yes Core infrastructure has been modified
gcp awi-ciroh No Yes Core infrastructure has been modified
gcp meom-ige No Yes Core infrastructure has been modified
kubeconfig utoronto No Yes Core infrastructure has been modified

Production deployments

Cloud Provider Cluster Name Hub Name Reason for Redeploy
gcp 2i2c-uk lis Core infrastructure has been modified
gcp linked-earth prod Core infrastructure has been modified
gcp pangeo-hubs prod Core infrastructure has been modified
aws nasa-veda prod Core infrastructure has been modified
gcp callysto prod Core infrastructure has been modified
aws victor prod Core infrastructure has been modified
gcp leap prod Core infrastructure has been modified
aws ubc-eoas prod Core infrastructure has been modified
gcp 2i2c demo Core infrastructure has been modified
gcp 2i2c ohw Core infrastructure has been modified
gcp 2i2c pfw Core infrastructure has been modified
gcp 2i2c peddie Core infrastructure has been modified
gcp 2i2c catalyst-cooperative Core infrastructure has been modified
gcp 2i2c earthlab Core infrastructure has been modified
gcp 2i2c paleohack2021 Core infrastructure has been modified
gcp 2i2c aup Core infrastructure has been modified
gcp 2i2c temple Core infrastructure has been modified
gcp 2i2c ucmerced Core infrastructure has been modified
gcp cloudbank ccsf Core infrastructure has been modified
gcp cloudbank csm Core infrastructure has been modified
gcp cloudbank elcamino Core infrastructure has been modified
gcp cloudbank glendale Core infrastructure has been modified
gcp cloudbank howard Core infrastructure has been modified
gcp cloudbank miracosta Core infrastructure has been modified
gcp cloudbank skyline Core infrastructure has been modified
gcp cloudbank demo Core infrastructure has been modified
gcp cloudbank fresno Core infrastructure has been modified
gcp cloudbank lassen Core infrastructure has been modified
gcp cloudbank sbcc Core infrastructure has been modified
gcp cloudbank lacc Core infrastructure has been modified
gcp cloudbank mills Core infrastructure has been modified
gcp cloudbank palomar Core infrastructure has been modified
gcp cloudbank pasadena Core infrastructure has been modified
gcp cloudbank sjcc Core infrastructure has been modified
gcp cloudbank tuskegee Core infrastructure has been modified
gcp cloudbank avc Core infrastructure has been modified
gcp cloudbank csu Core infrastructure has been modified
aws openscapes prod Core infrastructure has been modified
aws gridsst prod Core infrastructure has been modified
aws nasa-cryo prod Core infrastructure has been modified
aws 2i2c-aws-us researchdelight Core infrastructure has been modified
aws carbonplan prod Core infrastructure has been modified
gcp m2lines prod Core infrastructure has been modified
gcp awi-ciroh prod Core infrastructure has been modified
gcp meom-ige prod Core infrastructure has been modified
gcp meom-ige drakkar-demo Core infrastructure has been modified
kubeconfig utoronto prod Core infrastructure has been modified
kubeconfig utoronto r-prod Core infrastructure has been modified

@GeorgianaElena GeorgianaElena requested a review from a team March 3, 2023 11:39
Copy link
Member

@sgibson91 sgibson91 left a comment

Choose a reason for hiding this comment

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

Awesome! 🚀

@GeorgianaElena GeorgianaElena merged commit 94bcaef into 2i2c-org:master Mar 3, 2023
@GeorgianaElena GeorgianaElena deleted the fix-empty-admin branch March 3, 2023 11:44
@github-actions
Copy link

github-actions bot commented Mar 3, 2023

🎉🎉🎉🎉

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

consideRatio added a commit to consideRatio/pilot-hubs that referenced this pull request Sep 10, 2023
consideRatio added a commit to consideRatio/pilot-hubs that referenced this pull request Sep 13, 2023
consideRatio added a commit to consideRatio/pilot-hubs that referenced this pull request Sep 15, 2023
consideRatio added a commit to consideRatio/pilot-hubs that referenced this pull request Sep 30, 2023
consideRatio added a commit to consideRatio/pilot-hubs that referenced this pull request Oct 4, 2023
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
Archived in project
2 participants