Skip to content

Commit 177580d

Browse files
authored
Use Min Available not Max unavailable (#58)
Signed-off-by: 5st7 <[email protected]> Signed-off-by: 5st7 <[email protected]>
1 parent c08af65 commit 177580d

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

pkg/controller/poddisruptionbudget_control.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,9 @@ func (s *PodDisruptionBudgetsControl) CreateRedisClusterPodDisruptionBudget(redi
8383
if err != nil {
8484
return nil, err
8585
}
86-
maxUnavailable := intstr.FromInt(1)
86+
87+
minAvailable := intstr.FromInt(int(*redisCluster.Spec.NumberOfPrimaries*(1+*redisCluster.Spec.ReplicationFactor) - 1))
88+
8789
labelSelector := metav1.LabelSelector{
8890
MatchLabels: desiredLabels,
8991
}
@@ -96,8 +98,8 @@ func (s *PodDisruptionBudgetsControl) CreateRedisClusterPodDisruptionBudget(redi
9698
OwnerReferences: []metav1.OwnerReference{pod.BuildOwnerReference(redisCluster)},
9799
},
98100
Spec: policyv1.PodDisruptionBudgetSpec{
99-
MaxUnavailable: &maxUnavailable,
100-
Selector: &labelSelector,
101+
MinAvailable: &minAvailable,
102+
Selector: &labelSelector,
101103
},
102104
}
103105
err = s.KubeClient.Create(context.Background(), newPodDisruptionBudget)

0 commit comments

Comments
 (0)