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

Puma-legacy not properly controlling memory resource usage #219

Open
dvalfre opened this issue Aug 30, 2023 · 0 comments
Open

Puma-legacy not properly controlling memory resource usage #219

dvalfre opened this issue Aug 30, 2023 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@dvalfre
Copy link
Contributor

dvalfre commented Aug 30, 2023

The ey-puma_legacy recipe configures monit for watching over its processes and workers.
But in doing so, it misses to calculate the memory usage of the whole process tree (main process + child workers). Due to Puma's own architecture, the child processes carry the brunt of the work, thus prone to use more memory, but still not checked by monit.

We want to extend the relevant check to use if total mem, so taking into account the resource usage of the entire process tree.

Alternatively, as an improvement, we would like the monit control file for ey-puma_legacy to be aligned with the best practices of the ey-unicorn one

@dvalfre dvalfre added the bug Something isn't working label Aug 30, 2023
@dvalfre dvalfre self-assigned this Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant