Skip to content

Commit

Permalink
Merge pull request #852 from pb82/fix-381
Browse files Browse the repository at this point in the history
change maxUnavailable and MaxSurge to use percentages as opposed to int
  • Loading branch information
pb82 committed Nov 29, 2022
2 parents 4e92d62 + 5add9c2 commit 06458e2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions controllers/model/grafanaDeployment.go
Expand Up @@ -149,8 +149,8 @@ func getDeploymentAnnotations(cr *v1alpha1.Grafana, existing map[string]string)
}

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,
Expand Down

0 comments on commit 06458e2

Please sign in to comment.