Skip to content

Limit Redis memory usage of system RAM #3907

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.

Sign up for GitHub

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Limit Redis memory usage of system RAM #3907

Changes from 5 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
9007d31
Optimize Redis configuration in `docker-compose.yml` by dynamically s…
kixorz Aug 26, 2025
5333261
Refactor Redis command in `docker-compose.yml` for improved readabili…
kixorz Aug 26, 2025
e15c731
Remove redundant YAML code block delimiter in `docker-compose.yml`.
kixorz Aug 26, 2025
8df1aea
fix
kixorz Aug 26, 2025
8ef00ee
limit memory in the redis config
kixorz Sep 7, 2025
8360839
updated docker compose
kixorz Sep 29, 2025
3d3f880
updated docker compose
kixorz Oct 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion redis.conf
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
# maxmemory 0 # Unlimited memory usage
# maxmemory 15gb # Limit memory usage to 15 GB

maxmemory 0
maxmemory 2gb
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is absolutely crucial for maxmemory-policy to kick in.

https://redis.io/docs/latest/develop/reference/eviction/ :

Use the maxmemory-policy configuration directive to select the eviction policy you want to use when the limit set by maxmemory is reached.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I would argue that 2gb is not enough. I would prefer 8gb of 12gb.

Copy link
Author

@kixorz kixorz Oct 9, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

8 or 12g requires a large and more expensive instance.

We have a relatively busy instance and 2g works fine for us.

Copy link
Collaborator

@aldy505 aldy505 Oct 9, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

Redis is just a cache, the system may only be slightly slower in certain situations.

Surprisingly that's not how Sentry uses Redis.

Memcached is being used just for caching. But Redis, it's being used as (1) lock contention store, (2) storing uptime monitor configuration, (3) storing project configs that is used for validating incoming SDK envelopes, and much more that I can't remember. These are not really "caches" for me though.

Although, I understand that this shouldn't how you use Redis, but it's a live software anyway, we can only work around it.

Copy link
Author

Choose a reason for hiding this comment

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

Are those use cases subject to cache eviction?


# This setting determines how Redis evicts keys when it reaches the memory limit.
# `allkeys-lru` evicts the least recently used keys from all keys stored in Redis,
Expand Down
Loading