feat: redis-cluster add podAntiAffinity(#1174 )

[xiaozhuang-a]

Description

support leader and follower anti affinity of pods with the same order

Fixes #ISSUE

Type of change

• New feature (non-breaking change which adds functionality)

Checklist

• Tests have been added/modified and all tests pass.
• Functionality/bugs have been confirmed to be unchanged or fixed.
• I have performed a self-review of my own code.
• Documentation has been updated or added where necessary.

Additional Context

[xiaozhuang-a]

webhook MutatingWebhookConfiguration need to add it to the redis-operator charts template？

[codecov]

Codecov Report

Attention: Patch coverage is 63.82979% with 34 lines in your changes missing coverage. Please review.

Project coverage is 29.70%. Comparing base (60ef5a2) to head (858fb42).
Report is 16 commits behind head on master.

Files with missing lines	Patch %	Lines
pkg/webhook/pod_webhook.go	65.21%	24 Missing and 8 partials ⚠️
main.go	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1180      +/-   ##
==========================================
- Coverage   30.20%   29.70%   -0.51%     
==========================================
  Files          55       57       +2     
  Lines        6294     6579     +285     
==========================================
+ Hits         1901     1954      +53     
- Misses       4200     4440     +240     
+ Partials      193      185       -8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[drivebyer]

Nice idea.

How about trigger mutation by adding an annotation to the RedisCluster resource, such as rediscluster.redis.redis.opstreelabs.in/role-anti-affinity. @xiaozhuang-a

[drivebyer]

BTW, we need add some test case to it.

[xiaozhuang-a]

Nice idea.

How about trigger mutation by adding an annotation to the RedisCluster resource, such as rediscluster.redis.redis.opstreelabs.in/role-anti-affinity. @xiaozhuang-a

I think the 'add annotation' solution is very good, and I will implement it

[xiaozhuang-a]

BTW, we need add some test case to it.

get

[drivebyer]

lgtm

[radsto]

Hi, could you please give us a hint on how you generated the certificate for the webhook webhook-server-cert ?

[xiaozhuang-a]

Hi, could you please give us a hint on how you generated the certificate for the webhook webhook-server-cert ?

reference https://github.com/OT-CONTAINER-KIT/redis-operator/blob/master/charts/redis-operator/values.yaml

[xiaozhuang-a]

Hi, could you please give us a hint on how you generated the certificate for the webhook webhook-server-cert ?

If you want to use this feature, just use this chart and set the parameter to true

[radsto]

@xiaozhuang-a thank you I have managed to do it.