From 5b0b0eee4e8c7db3cc81d7460b9e718b92f89106 Mon Sep 17 00:00:00 2001 From: andytson-inviqa Date: Thu, 19 Oct 2023 14:08:49 +0100 Subject: [PATCH] Remove redis save option so redis 7 can be used (#793) 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 --- src/_base/harness/attributes/docker-base.yml | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/_base/harness/attributes/docker-base.yml b/src/_base/harness/attributes/docker-base.yml index 97fe5baf1..cd710ea22 100644 --- a/src/_base/harness/attributes/docker-base.yml +++ b/src/_base/harness/attributes/docker-base.yml @@ -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"