-
-
Notifications
You must be signed in to change notification settings - Fork 232
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: redis-cluster add podAntiAffinity(#1174 ) #1180
base: master
Are you sure you want to change the base?
feat: redis-cluster add podAntiAffinity(#1174 ) #1180
Conversation
Signed-off-by: xiaozhuang <[email protected]>
Signed-off-by: xiaozhuang <[email protected]>
Signed-off-by: xiaozhuang <[email protected]>
Signed-off-by: xiaozhuang <[email protected]>
webhook MutatingWebhookConfiguration need to add it to the redis-operator charts template? |
Signed-off-by: xiaozhuang <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1180 +/- ##
==========================================
- Coverage 30.20% 29.80% -0.40%
==========================================
Files 55 56 +1
Lines 6294 6488 +194
==========================================
+ Hits 1901 1934 +33
- Misses 4200 4356 +156
- Partials 193 198 +5 ☔ View full report in Codecov by Sentry. |
Signed-off-by: xiaozhuang <[email protected]>
Signed-off-by: xiaozhuang <[email protected]>
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 |
BTW, we need add some test case to it. |
I think the 'add annotation' solution is very good, and I will implement it |
get |
Description
support leader and follower anti affinity of pods with the same order
Fixes #ISSUE
Type of change
Checklist
Additional Context