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

Adjust memory requests so largest instances spawn on r5.4xlarge #3573

Merged
merged 1 commit into from
Jan 5, 2024

Conversation

yuvipanda
Copy link
Member

Brings in the new memory / cpu limits set up for
#3572 - without that, the largest set up size doesn't actually spawn on r5.4xlarge due to insufficient memory.

Brings in the new memory / cpu limits set up for
2i2c-org#3572 - without
that, the largest set up size doesn't actually spawn on r5.4xlarge
due to insufficient memory.
Copy link

github-actions bot commented Jan 4, 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
aws nasa-esdis No Yes Following helm chart values files were modified: common.values.yaml
aws nasa-veda No Yes Following helm chart values files were modified: common.values.yaml
aws openscapes No Yes Following helm chart values files were modified: staging.values.yaml

Production deployments

Cloud Provider Cluster Name Hub Name Reason for Redeploy
aws nasa-esdis prod Following helm chart values files were modified: common.values.yaml
aws nasa-veda prod Following helm chart values files were modified: common.values.yaml
aws openscapes prod Following helm chart values files were modified: prod.values.yaml

@yuvipanda
Copy link
Member Author

I'll self merge this in about 2h if nobody is around to review, as this means the veda hub will not be able to launch the largest instance size.

@yuvipanda
Copy link
Member Author

I'm actually just going to wait for review, I think these have been broken for a little bit now, so no point rushing this fix.

@yuvipanda yuvipanda merged commit 7d0527d into 2i2c-org:master Jan 5, 2024
11 checks passed
@yuvipanda
Copy link
Member Author

Thanks @sgibson91!

Copy link

github-actions bot commented Jan 5, 2024

🎉🎉🎉🎉

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

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.

2 participants