search

spotahome / redis-operator

Redis Operator creates/configures/manages high availability redis with sentinel automatic failover atop Kubernetes.
Apache License 2.0
1.51k stars 359 forks source link

Fix sentinel health check #556

Closed bwrobc closed 1 year ago

bwrobc commented 1 year ago

Currently, a RedisFailover defined with with more redis than sentinel replicas will never start. This is because the health check compares the number of running sentinel pods against the number of specified redis replicas, due to a copy-paste error in 1e46182984c0dee22b95382acd7eb3c1c7cac6d8

Also corrects a minor typo in the redis readiness check script.

samof76 commented 1 year ago

@ese is this someone else who you could tag to take these forward?

kmcrawford commented 1 year ago

This PR will fix my issue: https://github.com/spotahome/redis-operator/issues/589

Prudhvi0717 commented 1 year ago

Issue still persists even after pulling 3.2.8 charts

drivebyer commented 8 months ago

This need release new operator image.