search

OT-CONTAINER-KIT / redis

A production optimized redis docker image
https://opstree.github.io
23 stars 51 forks source link

fix: cluster-announce-hostname only support in redis 7 #69

Closed drivebyer closed 10 months ago

drivebyer commented 10 months ago

see: https://github.com/redis/redis/blob/7.0/redis.conf

When build image with redis 6.2.*, the startup log shows:

 Running without TLS mode
 ACL_MODE is not true, skipping ACL file modification
 Starting redis service in cluster mode.....

 *** FATAL CONFIG FILE ERROR (Redis 6.2.12) ***
 Reading the configuration file, at line 25
 >>> 'cluster-announce-hostname "new-redis-cluster111-leader-0"'
 Bad directive or wrong number of arguments

So we should remove the param setting when REDIS_MAJOR_VERSION settting to v7