Merge pull request #383 from HubertStefanski/rolling-update-fix

change maxUnavailable and MaxSurge to use percentages as opposed to int
grafana · Apr 8, 2021 · ca8ba68 · ca8ba68
2 parents 8e599a8 + 40adcb2
commit ca8ba68
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/controller/model/grafanaDeployment.go b/pkg/controller/model/grafanaDeployment.go
@@ -116,8 +116,8 @@ func getReplicas(cr *v1alpha1.Grafana) *int32 {
 }
 
 func getRollingUpdateStrategy() *v1.RollingUpdateDeployment {
-	var maxUnaval intstr.IntOrString = intstr.FromInt(25)
-	var maxSurge intstr.IntOrString = intstr.FromInt(25)
+	var maxUnaval intstr.IntOrString = intstr.FromString("25%")
+	var maxSurge intstr.IntOrString = intstr.FromString("25%")
 	return &v1.RollingUpdateDeployment{
 		MaxUnavailable: &maxUnaval,
 		MaxSurge:       &maxSurge,