Skip to content

Bump neurohackademy to g4dn.2xlarge #6537

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

Merged
merged 2 commits into from
Aug 7, 2025
Merged

Conversation

yuvipanda
Copy link
Member

They need same amount of GPU but more RAM

Replaces #6536

They need same amount of GPU but more RAM
@arokem
Copy link
Contributor

arokem commented Aug 7, 2025

Let's do 4x (see recent change on the other PR)

Copy link

github-actions bot commented Aug 7, 2025

Merging this PR will trigger the following deployment actions.

Support deployments

No support upgrades will be triggered

Staging deployments

Cloud Provider Cluster Name Hub Name Reason for Redeploy
aws neurohackademy staging Following helm chart values files were modified: common.values.yaml

Production deployments

Cloud Provider Cluster Name Hub Name Reason for Redeploy
aws neurohackademy prod Following helm chart values files were modified: common.values.yaml

@yuvipanda
Copy link
Member Author

@arokem per https://aws.amazon.com/ec2/instance-types/g4/, 2x has 32G of RAM and 1 GPU, while 4x has 64G of RAM and 1 GPU. Given there's only 1 GPU and we don't have GPU sharing enabled, do you want 2x with 30G of RAM or 4x with 61G of RAM? We shouldn't do 4x with 30G of RAM

@arokem
Copy link
Contributor

arokem commented Aug 7, 2025

Oh - OK, let's do 2x with 30G then. Thanks!

@yuvipanda
Copy link
Member Author

@arokem great, already deployed! I did provision (but not use) 4xlarge as well, so if you want more RAM (~61G) you can bump up straightforward with a PR (eksctl changes require manual work, which I've just done)

@yuvipanda yuvipanda merged commit 7343011 into 2i2c-org:main Aug 7, 2025
11 checks passed
Copy link

github-actions bot commented Aug 7, 2025

🎉🎉🎉🎉

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

@yuvipanda
Copy link
Member Author

Hang on, turns out 30G is not right. Fixing

yuvipanda added a commit to yuvipanda/pilot-hubs that referenced this pull request Aug 7, 2025
@yuvipanda
Copy link
Member Author

@arokem should be fixed now

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.

2 participants