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

Set the default vm_memory_high_watermark to 0.6 (backport #12161) #12162

Merged
merged 1 commit into from
Aug 29, 2024

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Aug 29, 2024

The default of 0.4 was very conservative even when it was set years ago. Since then:

  • we moved to CQv2, which have much more predictable memory usage than (non-lazy) CQv1 used to
  • we removed CQ mirroring which caused large sudden memory spikes in some situations
  • we removed the option to store message payload in memory in quorum queues

For the past two years or so, we've been running all our internal tests and benchmarks using the value of 0.8 with no OOMkills at all (note: we do this on Kubernetes where the Cluster Operators overrides the available memory leaving some additional headroom, but effectively we are still using more than 0.6 of memory).

Corresponding docs PR: rabbitmq/rabbitmq-website#2032


This is an automatic backport of pull request #12161 done by Mergify.

The default of 0.4 was very conservative even when it was
set years ago. Since then:
- we moved to CQv2, which have much more predictable memory usage than (non-lazy) CQv1 used to
- we removed CQ mirroring which caused large sudden memory spikes in some situations
- we removed the option to store message payload in memory in quorum queues

For the past two years or so, we've been running all our internal tests and benchmarks
using the value of 0.8 with no OOMkills at all (note: we do this on
Kubernetes where the Cluster Operators overrides the available memory
levaing some additional headroom, but effectively we are still using  more than
0.6 of memory).

(cherry picked from commit 8a03975)
@mkuratczyk mkuratczyk merged commit 9407f5d into v4.0.x Aug 29, 2024
197 checks passed
@mkuratczyk mkuratczyk deleted the mergify/bp/v4.0.x/pr-12161 branch August 29, 2024 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant