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

Callysto core nodepool rightsizing #2005

Closed
ianabc opened this issue Dec 14, 2022 · 4 comments
Closed

Callysto core nodepool rightsizing #2005

ianabc opened this issue Dec 14, 2022 · 4 comments
Assignees

Comments

@ianabc
Copy link
Contributor

ianabc commented Dec 14, 2022

Context

The callysto hub infrastructure is currently running on a pair of n1-highmem-4 (GCE) nodes which are relatively expensive, and I was asked to check that these are the right size for the job. Poking around in our grafana stats, it looks like they might be a bit underutilized, would it make sense to resize these to n1-highmem-2 nodes? I think the default cluster config uses n1-highmen-2 with the caveat

For single-tenant clusters, a single n1-highmem-2 node seems enough - if network policy and config connector are not on. For others, please experiment to see what fits.

but we override that in our current terraform config. Are there any reasons for us not to resize to n1-highmem-2 nodes?

If it is possible for us to do this, do we just propose the change to the terraform config? I guess applying the change would tear down our current cluster and redeploy it; that would be OK, but I would the user data (FileStore volume) be preserved and re-attached?

Proposal

Explore the possibility of resizing the core node pool of the Callysto cluster. If it is feasible, redeploy.

@yuvipanda
Copy link
Member

@ianabc we can try it out! However, I am curious if that is enough for the prometheus nodes.

@ianabc
Copy link
Contributor Author

ianabc commented Jan 11, 2023

Thanks, I think we might give it a go and see what happens. My understanding is we would make the update in terraform and then issue a PR, but I'm a little fuzzy on what will happen when we merge the PR. I think it has to destroy the existing code nodepool and create a new one, but can you confirm that it would leave the user storage intact? i.e. when it adds the new core nodepool and spins things up would existing users see their existing files? Or should we arrange to have them backed up somewhere then reimported?

@sgibson91
Copy link
Member

@ianabc User's home directories are stored in a Google Filestore instance that is mounted to the user servers via NFS. I don't foresee why terraform would delete the filestore just to replace one nodepool, so I think we are ok - but you never truly know until you run terraform plan!

@consideRatio
Copy link
Member

Closing via:

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
Development

No branches or pull requests

4 participants