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

awi-ciroh: upgrade k8s to 1.25, and adjust node pools following event #2554

Merged
merged 1 commit into from
May 19, 2023

Conversation

consideRatio
Copy link
Contributor

@consideRatio consideRatio commented May 19, 2023

Prepared for node sharing

The awi-ciroh cluster had a workshop where we ran into limitations from using 1:1 user:node. Due to that we did a behind the scenes quick migration to a node sharing setup.

This node sharing setup is now complemented, allowing us to reconfigure the awi-ciroh hubs without further terraform changes based on their wishes.

K8s cluster upgrade

I made an opportunistic upgrade of the k8s cluster while adjusting node pools. I also gained some experience which helps create GKE k8s cluster upgrade docs. Those notes are captured and linked from #2157.

Future work

They now have a UI with small / medium / large / huge, but they will just start on the n2-highmem-16 machine no matter what with a pod requesting memory based on the legacy 1:1 user:node profile list options.

I'll reach out in order to align their setup to a typical node sharing setup, which from this point onwards is just a hub config change.

@github-actions
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 awi-ciroh No Yes Following helm chart values files were modified: common.values.yaml

Production deployments

Cloud Provider Cluster Name Hub Name Reason for Redeploy
gcp awi-ciroh prod Following helm chart values files were modified: common.values.yaml

@consideRatio
Copy link
Contributor Author

Thanks @yuvipanda! I didn't finish the PR description in time for your review, but there is now one :)

@consideRatio consideRatio merged commit 86659aa into 2i2c-org:master May 19, 2023
@github-actions
Copy link

🎉🎉🎉🎉

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

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
Development

Successfully merging this pull request may close these issues.

2 participants