feat(vpa): add configuration for revisionHistoryLimit in helm chart (kubernetes#9307)

* fix(vpa): add configuration for revisionHistoryLimit in helm chart

Signed-off-by: t3mi <t3mi@users.noreply.github.com>

* fix: update docs

Signed-off-by: t3mi <t3mi@users.noreply.github.com>

---------

Signed-off-by: t3mi <t3mi@users.noreply.github.com>

Author: t3mi

vertical-pod-autoscaler/charts/vertical-pod-autoscaler/README.md

@@ -116,6 +116,7 @@ helm upgrade <release-name> <chart> \
 | admissionController.registerWebhook | bool | `false` | Whether to register webhook via the application itself or via Helm. Set to false when using Helm-managed webhook. Security issue: granting delete on mutatingwebhookconfigurations is a potential security risk as it allows the admission controller to remove any webhook configurations. |
 | admissionController.replicas | int | `2` |  |
 | admissionController.resources | object | `{}` |  |
+| admissionController.revisionHistoryLimit | int | `10` |  |
 | admissionController.service.annotations | object | `{}` |  |
 | admissionController.service.name | string | `"vpa-webhook"` |  |
 | admissionController.service.ports[0].port | int | `443` |  |
@@ -178,6 +179,7 @@ helm upgrade <release-name> <chart> \
 | recommender.priorityClassName | string | `nil` |  |
 | recommender.replicas | int | `2` |  |
 | recommender.resources | object | `{}` |  |
+| recommender.revisionHistoryLimit | int | `10` |  |
 | recommender.serviceAccount.annotations | object | `{}` |  |
 | recommender.serviceAccount.create | bool | `true` |  |
 | recommender.serviceAccount.labels | object | `{}` |  |
@@ -208,6 +210,7 @@ helm upgrade <release-name> <chart> \
 | updater.priorityClassName | string | `nil` |  |
 | updater.replicas | int | `2` |  |
 | updater.resources | object | `{}` |  |
+| updater.revisionHistoryLimit | int | `10` |  |
 | updater.serviceAccount.annotations | object | `{}` |  |
 | updater.serviceAccount.create | bool | `true` |  |
 | updater.serviceAccount.labels | object | `{}` |  |

vertical-pod-autoscaler/charts/vertical-pod-autoscaler/templates/admission-controller-deployment.yaml

@@ -15,6 +15,7 @@ metadata:
   {{- end }}
 spec:
   replicas: {{ .Values.admissionController.replicas }}
+  revisionHistoryLimit: {{ .Values.admissionController.revisionHistoryLimit }}
   {{- with .Values.admissionController.updateStrategy }}
   strategy:
     {{- toYaml . | nindent 4 }}

vertical-pod-autoscaler/charts/vertical-pod-autoscaler/templates/recommender-deployment.yaml

@@ -12,6 +12,7 @@ metadata:
   {{- end }}
 spec:
   replicas: {{ .Values.recommender.replicas }}
+  revisionHistoryLimit: {{ .Values.recommender.revisionHistoryLimit }}
   {{- with .Values.recommender.updateStrategy }}
   strategy:
     {{- toYaml . | nindent 4 }}

vertical-pod-autoscaler/charts/vertical-pod-autoscaler/templates/updater-deployment.yaml

@@ -12,6 +12,7 @@ metadata:
   {{- end }}
 spec:
   replicas: {{ .Values.updater.replicas }}
+  revisionHistoryLimit: {{ .Values.updater.revisionHistoryLimit }}
   selector:
     matchLabels:
       {{- include "vertical-pod-autoscaler.updater.selectorLabels" . | nindent 6 }}

vertical-pod-autoscaler/charts/vertical-pod-autoscaler/values.yaml

@@ -56,6 +56,9 @@ admissionController:
   # Number of Admission Controller replicas to create.
   replicas: 2
 
+  # The number of old history revisions to retain to allow rollback.
+  revisionHistoryLimit: 10
+
   # Labels to add to the Admission Controller pod.
   podLabels: {}
 
@@ -212,6 +215,9 @@ recommender:
   # Number of Recommender replicas to create.
   replicas: 2
 
+  # The number of old history revisions to retain to allow rollback.
+  revisionHistoryLimit: 10
+
   # Labels to add to the Recommender pod.
   podLabels: {}
 
@@ -304,6 +310,9 @@ updater:
   # Number of Updater replicas to create.
   replicas: 2
 
+  # The number of old history revisions to retain to allow rollback.
+  revisionHistoryLimit: 10
+
   # Labels to add to the Updater pod.
   podLabels: {}