Skip to content

Commit

Permalink
Remove redis save option so redis 7 can be used (#793)
Browse files Browse the repository at this point in the history
There is a difference in config format for redis 7, and since save are
it's defaults, it doesn't need supplying.

So remove the need for a workaround to supply save options if using
redis 7
  • Loading branch information
andytson-inviqa authored Oct 19, 2023
1 parent 246c6d7 commit 5b0b0ee
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions src/_base/harness/attributes/docker-base.yml
Original file line number Diff line number Diff line change
Expand Up @@ -159,10 +159,14 @@ attributes:
# Evict any least recently used key even if they don't have a TTL
maxmemory-policy: allkeys-lru
# Save snapshots every X changes have happened within Y seconds (these are the defaults in redis.conf)
save:
- 3600 1
- 300 100
- 60 10000
# Redis < 7
# save:
# - 3600 1
# - 300 100
# - 60 10000
# Redis >= 7
# save:
# - '3600 1 300 100 60 10000'
resources:
# 1.5 * maxmemory to allow copy on write snapshots
memory: "1.5Gi"
Expand Down

0 comments on commit 5b0b0ee

Please sign in to comment.