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

cloudbank: Ensure add"staff"user_ids_of_type matches authentication method used #4914

Merged

Conversation

sgibson91
Copy link
Member

These cloudbank hubs are using GitHub auth, but 2i2c staff IDs being added were Google IDs which are not compatible

Copy link

github-actions bot commented Oct 2, 2024

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 cloudbank No Yes Following prod hubs require redeploy: lacc, merced, riohondo, tuskegee

Production deployments

Cloud Provider Cluster Name Hub Name Reason for Redeploy
gcp cloudbank lacc Following helm chart values files were modified: lacc.values.yaml
gcp cloudbank merced Following helm chart values files were modified: merced.values.yaml
gcp cloudbank riohondo Following helm chart values files were modified: riohondo.values.yaml
gcp cloudbank tuskegee Following helm chart values files were modified: tuskegee.values.yaml

@sgibson91 sgibson91 force-pushed the cloudbank/update-staff-admin-user-type branch from 31b14f8 to d8a175f Compare October 2, 2024 08:58
@sgibson91 sgibson91 merged commit 3048a62 into 2i2c-org:main Oct 2, 2024
8 checks passed
@sgibson91 sgibson91 deleted the cloudbank/update-staff-admin-user-type branch October 2, 2024 09:02
Copy link

github-actions bot commented Oct 2, 2024

🎉🎉🎉🎉

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant