-
Notifications
You must be signed in to change notification settings - Fork 39
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
Add cpu/memory request for mongo component of deployment #20
Comments
@mdbishop it makes total sense. For now, we don't have enough metrics about the usage of Mongo so we decided to not add limit for now but we should for sure. Do you have any suggestions? |
@arnaud-elasticbox The database components in K8S as a whole (speaking very broadly nothing specific to this use case) strike me as little un-tuned icebergs we're leaving for our future selves unless we allow for some self tuning based on telemetry. It's so easy to deploy redis, mongo, even DB2 these days and forget about it until it blows up -- sometimes not even realizing you just HELM installed some clustered database for WordPress or whatever. 💣 Sorry, gotten way off topic, apologies. In order of ideal to still fine:
3 May be the easiest and most practical to get kicked off with as you may not hit mongo tuning issues right off the bat (unless Google starts using this themselves 😄 ) so could be deferred. I'll keep an eye on mongo's metrics and can update this with such once I have a stable installation running. |
…ignment template sorter alignment
Unlike the other components in the elastickube manifest, mongo seems to not have any resource asks defined for it. Not sure if this was a design decision, but it would seem prudent to at least ask for some floor for a few reasons:
The text was updated successfully, but these errors were encountered: