Merge pull request #4136 from heliapb/feat/allow_config_timeout

[rollout-operator] feat: allow config of `timeoutSeconds` in rollout operator

Author: tcp13equals2

charts/rollout-operator/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: rollout-operator
 description: "Grafana rollout-operator"
 type: application
-version: 0.44.0
+version: 0.45.0
 appVersion: v0.35.0
 home: https://github.com/grafana/rollout-operator
 kubeVersion: ^1.10.0-0

charts/rollout-operator/README.md

@@ -4,7 +4,7 @@ Helm chart for deploying [Grafana rollout-operator](https://github.com/grafana/r
 
 # rollout-operator
 
-![Version: 0.44.0](https://img.shields.io/badge/Version-0.44.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v0.35.0](https://img.shields.io/badge/AppVersion-v0.35.0-informational?style=flat-square)
+![Version: 0.45.0](https://img.shields.io/badge/Version-0.45.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v0.35.0](https://img.shields.io/badge/AppVersion-v0.35.0-informational?style=flat-square)
 
 Grafana rollout-operator
 
@@ -89,3 +89,4 @@ Manually applying these CRDs is only required if upgrading from a chart <= v0.32
 | webhooks.failurePolicy | string | `"Fail"` | Validating and mutating webhook failure policy. `Ignore` or `Fail`. |
 | webhooks.objectSelector | object | `{}` | objectSelector to filter which objects the webhooks apply to. |
 | webhooks.selfSignedCertSecretName | string | `"certificate"` | Secret resource name for the TLS certificate to be used with the webhooks |
+| webhooks.timeoutSeconds | int | `10` | Timeout in seconds for the prepare-downscale mutating webhook. Must be between 1 and 30. |

charts/rollout-operator/templates/pod-eviction-webhook.yaml

@@ -46,5 +46,6 @@ webhooks:
       {{- toYaml . | nindent 6 }}
     {{- end }}
     sideEffects: None
+    timeoutSeconds: {{.Values.webhooks.timeoutSeconds}}
     failurePolicy: {{.Values.webhooks.failurePolicy}}
 {{- end -}}

charts/rollout-operator/templates/prepare-downscale-webhook.yaml

@@ -48,6 +48,6 @@ webhooks:
     {{- end }}
     sideEffects: NoneOnDryRun
     matchPolicy: Equivalent
-    timeoutSeconds: 10
+    timeoutSeconds: {{.Values.webhooks.timeoutSeconds}}
     failurePolicy: {{.Values.webhooks.failurePolicy}}
 {{- end -}}

charts/rollout-operator/templates/zone-aware-pod-disruption-budget-validating-webhook.yaml

@@ -46,5 +46,6 @@ webhooks:
       {{- toYaml . | nindent 6 }}
     {{- end }}
     sideEffects: None
+    timeoutSeconds: {{.Values.webhooks.timeoutSeconds}}
     failurePolicy: {{.Values.webhooks.failurePolicy}}
 {{- end -}}

charts/rollout-operator/values.yaml

@@ -108,6 +108,8 @@ webhooks:
   # -- Enable the rollout-operator webhooks. See https://github.com/grafana/rollout-operator/#webhooks.
   # Note that the webhooks require custom resource definitions. If upgrading, manually apply the files in the `crds` directory.
   enabled: true
+  # -- Timeout in seconds for the prepare-downscale mutating webhook. Must be between 1 and 30.
+  timeoutSeconds: 10
   # -- Validating and mutating webhook failure policy. `Ignore` or `Fail`.
   failurePolicy: "Fail"
   # -- Secret resource name for the TLS certificate to be used with the webhooks