Adding the ability to customize the percentages of maxUnavailable and maxSurge should be added into the helm values file for the sentinal deployment.
Actual behavior
Default values are auto-populated at 25% for both maxUnavailable and maxSurge.
Steps to reproduce the behavior
Complete a base sentinal deployment and describe the deployment to see those values set.
Environment
Redis Operator version - 1.2.2
Kubernetes version - 1.23.9
Kubernetes configuration used (eg: Is RBAC active?)
Recommendation
Given the normal cloud standard 3 zone architecture of data centers (AWS = 1a, 1b, 1c) when the topologySpreadContraint requires zone distribution on scheduling and the rollingUpdate strategy only allows for 25% scale up and unavailable pods are unable to be scheduled because they don't meet the rolling update requirement (when running 3 replicas, 1 per zone).
Enabling the helm values to customize the deployment's maxUnavailable and maxSurge parameters would allow teams to define those values at 35% and enable rolling updates for 3 replica zone distributed systems.
Expected behavior
Adding the ability to customize the percentages of maxUnavailable and maxSurge should be added into the helm values file for the sentinal deployment.
Actual behavior
Default values are auto-populated at 25% for both maxUnavailable and maxSurge.
Steps to reproduce the behavior
Complete a base sentinal deployment and describe the deployment to see those values set.
Environment
Recommendation
Given the normal cloud standard 3 zone architecture of data centers (AWS = 1a, 1b, 1c) when the topologySpreadContraint requires zone distribution on scheduling and the rollingUpdate strategy only allows for 25% scale up and unavailable pods are unable to be scheduled because they don't meet the rolling update requirement (when running 3 replicas, 1 per zone).
Enabling the helm values to customize the deployment's maxUnavailable and maxSurge parameters would allow teams to define those values at 35% and enable rolling updates for 3 replica zone distributed systems.