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

basehub: allow shared-volume-reporter to be disabled, and disable it on 2i2c-aws-us and catalystproject-africa #4000

Merged
merged 2 commits into from
May 7, 2024

Conversation

consideRatio
Copy link
Member

@consideRatio consideRatio commented Apr 29, 2024

Technically why this is acceptable is detailed in #3999.

2i2c-aws-us runs two core nodes, but only requires one for its CPU/memory needs. What makes us have two is the fact that we need room for 66 pods, but each core node only allows for 57 per node.

With this change, we reduce the pods to run on the core nodes with one pod per hub we have deployed --- 7 pods. And that in turn combined with other pod reductions like #3869 will get us below 57 pods to run on the core nodes, allowing us to cut cost for communities on 2i2c-aws-us (I think with almost 200USD total) and help us demonstrate more cost effective cloud operations.

Related

This comment was marked as resolved.

Specifically this cluster with its many hubs runs out of available pods
per node on its core nodes, forcing two core nodes instead of one to be
used. That in turn increase costs for the communities on this cluster
with tens of percent.

Without these pods that reports metrics only used for a graphana
dashboard to conclude 100% of disk space is available on EFS without a
storage limit - we could together with other optimizations become able
to get down to using a single core node.
@consideRatio consideRatio changed the title basehub: allow shared-volume-reporter to be disabled, and disable it on 2i2c-aws-us basehub: allow shared-volume-reporter to be disabled, and disable it on 2i2c-aws-us and catalystproject-africa May 7, 2024
@consideRatio
Copy link
Member Author

I'll go for a self-merge here, it should be safe and can help me verify that we are indeed able to cut costs in 2i2c-aws-us and catalystproject-africa by reducing node pools form 2 to 1 nodes.

@consideRatio consideRatio merged commit 8d61006 into 2i2c-org:main May 7, 2024
34 checks passed
Copy link

github-actions bot commented May 7, 2024

🎉🎉🎉🎉

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

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.

None yet

1 participant