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

reduce memory usage #317

Open
wants to merge 2 commits into
base: stable/2023.1
Choose a base branch
from

Conversation

msherman64
Copy link
Contributor

This PR reduces memory usage by memcached and mariadb.

However, we may want to put more effort into the new default values used, and/or make it more easily configurable by the site operator.

This has been contributing to OOM errors on chi@edge, and should be set
by site operators relative to total available memory.

This reverts commit 68b2d55.
kolla has mariadb default to using 40% of available memory, up to a max
of 8GB. This has been causing OOM errors on chi@edge, so this drops the
ratio to 20%.
Copy link
Member

@codyhammock codyhammock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@Mark-Powers Mark-Powers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, I agree regarding easier configuration of these values, but this seems fine as default.

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.

3 participants