spotahome / redis-operator

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

Redis-operator wont update statefulset with new port configuration (v1.2.0) #633

Closed itayvolo closed 8 months ago

itayvolo commented 1 year ago

Expected behaviour

What do you want to achieve? Answer: (Argocd) After syncing redisFailover with a custom port key change, redis stateful-set should roll/restart and change the container port to the new custom port

Actual behaviour

What is happening? Are all the pieces created? Can you access to the service? Answer: After syncing redisFailover with a new custom port, redis stateful-set healthcheck fails and pod is stuck in progressing "Readiness probe failed: unespected Could not connect to Redis at 127.0.0.1:6379: Connection refused" 6379 - new port 7777 - old port

In the stateful-set yaml, I can see that the new port was not updated since the updateStrategy is set to 'OnDelete' image

This is the container's port configuration in the rfr-redis-0 pod's yaml: image

This is the pod's error message: image

Steps to reproduce the behaviour

Describe step by step what you've have done to get to this point Answer: Synced the redisFailover resource with new custom port + new redis image tag (to support the custom port key)

Environment

How are the pieces configured?

Logs

Please, add the debugging logs. In order to be able to gather them, add -debug flag when running the operator.

ebuildy commented 10 months ago

well, changing port is really a bad idea!

github-actions[bot] commented 9 months ago

This issue is stale because it has been open for 45 days with no activity.

github-actions[bot] commented 8 months ago

This issue was closed because it has been inactive for 14 days since being marked as stale.