-
Notifications
You must be signed in to change notification settings - Fork 418
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
Outdated information about memory limits #621
Comments
Ah, good catch! I won't be able to get to it right away myself, but that's maintained at https://github.com/docker-library/docs/blob/master/rabbitmq/content.md if you (or someone else reading this) wanted to take a stab at a PR. 👍 |
Turns out this can still happen. A user created an issue today (using up to date version of RabbitMQ and Erlang) and I noticed:
I'm fairly sure this is the host's memory, not container memory limit. I wonder what could be causing this in some situations. |
Erlang's imperfect detection of the limits using cgroups? cgroup version differences (IIRC some distribution have started rolling out v2 recently)? |
In rabbitmq cluster operator, we set |
Is there any solution for this? 🙏 |
@luarx no, it never did work reliably because the runtime's ability to detect the amount of available memory is only limited to some OSes and likely cgroups v1 for now. The docs have been updated to provide Kubernetes-specific recommendations. |
@tianon @yosifkit folks, can we integrate these recommendations from the RabbitMQ docs into this image, and then hopefully consider this issue resolved? I would be happy to contribute if you tell me where to look in the source, and how to test a preview of my edits locally. Thank you! |
That's in https://github.com/docker-library/docs/blob/master/rabbitmq/content.md 👀 (as linked above 😅) If you want to test the Docker Hub rendering, you can just create a repository and there's a preview in the private "edit" page, but I think markdown is probably limited enough that you can probably guess how it'll render pretty well (Hub's rendering is pretty vanilla). 😇 |
Any chance there is an update on this issue? I'd like to set the free disk space but this is not supported atm AFAIK. |
@JefSeys this issue has nothing to do with free disk space limits or how this image approaches configuration. You can set the free disk space alarm threshold using Any further comments or questions about free disk space limits will be deleted. This repository has discussions enabled, please start a new one instead of commenting on an open issue that is about a different limit in a specific context. |
I would argue that the docs as they are now are sufficient. It provides a brief overview and a link to an authoritative source for more details. |
The
Memory Limits
section of https://hub.docker.com/_/rabbitmq is incorrect (perhaps was true a long time ago). Container limits are taken into account, sovm_memory_high_watermark.relative
will be calculated based on container memory, not host memory. I think the whole section can be simply removed.The text was updated successfully, but these errors were encountered: