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: Explicitly specify what nodepools users should go to #3535

Merged
merged 1 commit into from
Dec 18, 2023

Conversation

yuvipanda
Copy link
Member

Otherwise, they still end up on the old 'user' nodepool, and it can not be decomissioned. With this, they will end up on the equivalent 'new' nodepool, and then I can get rid of the old one.

I have also had to adjust the CPU limit, as at some point I suppose we switched from 8 CPU nodes to 4 but did not adjust this.

Otherwise, they still end up on the old 'user' nodepool, and it
can not be decomissioned. With this, they will end up on the
equivalent 'new' nodepool, and then I can get rid of the old one.

I have also had to adjust the CPU limit, as at some point I suppose
we switched from 8 CPU nodes to 4 but did not adjust this.
@yuvipanda yuvipanda requested a review from a team as a code owner December 16, 2023 02:56
Copy link

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 helm chart values files were modified: common.values.yaml

Production deployments

Cloud Provider Cluster Name Hub Name Reason for Redeploy
gcp cloudbank bcc Following helm chart values files were modified: common.values.yaml
gcp cloudbank ccsf Following helm chart values files were modified: common.values.yaml
gcp cloudbank csm Following helm chart values files were modified: common.values.yaml
gcp cloudbank dvc Following helm chart values files were modified: common.values.yaml
gcp cloudbank elcamino Following helm chart values files were modified: common.values.yaml
gcp cloudbank evc Following helm chart values files were modified: common.values.yaml
gcp cloudbank glendale Following helm chart values files were modified: common.values.yaml
gcp cloudbank howard Following helm chart values files were modified: common.values.yaml
gcp cloudbank miracosta Following helm chart values files were modified: common.values.yaml
gcp cloudbank skyline Following helm chart values files were modified: common.values.yaml
gcp cloudbank demo Following helm chart values files were modified: common.values.yaml
gcp cloudbank fresno Following helm chart values files were modified: common.values.yaml
gcp cloudbank humboldt Following helm chart values files were modified: common.values.yaml
gcp cloudbank laney Following helm chart values files were modified: common.values.yaml
gcp cloudbank sbcc Following helm chart values files were modified: common.values.yaml
gcp cloudbank sbcc-dev Following helm chart values files were modified: common.values.yaml
gcp cloudbank lacc Following helm chart values files were modified: common.values.yaml
gcp cloudbank lamission Following helm chart values files were modified: common.values.yaml
gcp cloudbank mills Following helm chart values files were modified: common.values.yaml
gcp cloudbank mission Following helm chart values files were modified: common.values.yaml
gcp cloudbank norco Following helm chart values files were modified: common.values.yaml
gcp cloudbank palomar Following helm chart values files were modified: common.values.yaml
gcp cloudbank pasadena Following helm chart values files were modified: common.values.yaml
gcp cloudbank sjcc Following helm chart values files were modified: common.values.yaml
gcp cloudbank sacramento Following helm chart values files were modified: common.values.yaml
gcp cloudbank srjc Following helm chart values files were modified: common.values.yaml
gcp cloudbank saddleback Following helm chart values files were modified: common.values.yaml
gcp cloudbank santiago Following helm chart values files were modified: common.values.yaml
gcp cloudbank sjsu Following helm chart values files were modified: common.values.yaml
gcp cloudbank tuskegee Following helm chart values files were modified: common.values.yaml
gcp cloudbank wlac Following helm chart values files were modified: common.values.yaml
gcp cloudbank csulb Following helm chart values files were modified: common.values.yaml
gcp cloudbank csum Following helm chart values files were modified: common.values.yaml

@yuvipanda
Copy link
Member Author

This also ensures users don't end up accidentally on one of the large nodepools, which would cost a lot more!

@yuvipanda
Copy link
Member Author

Self-merging as there aren't any other engineers around until end of year.

@yuvipanda yuvipanda merged commit 3e0911e into 2i2c-org:master Dec 18, 2023
7 checks passed
Copy link

🎉🎉🎉🎉

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

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
Status: Done 🎉
Development

Successfully merging this pull request may close these issues.

1 participant