chore: Add revisionHistoryLimit to helm chart (#1615)

Author: flbla

manifest_staging/charts/workload-identity-webhook/README.md

@@ -56,6 +56,7 @@ helm upgrade -n azure-workload-identity-system [RELEASE_NAME] azure-workload-ide
 | mutatingWebhookNamespaceSelector   | The namespace selector to further refine which namespaces will be selected by the webhook.                                        | `{}`                                                    |
 | podDisruptionBudget.minAvailable   | The minimum number of pods that must be available for the webhook to be considered available                                      | `1`                                                     |
 | podDisruptionBudget.maxUnavailable | The maximum number of pods that may be unavailable for the webhook to be considered available                                     | `nil`                                                   |
+| revisionHistoryLimit               | The number of old ReplicaSets to retain for the webhook deployment                                                                | `10`                                                    |
 
 ## Contributing Changes
 

manifest_staging/charts/workload-identity-webhook/templates/azure-wi-webhook-controller-manager-deployment.yaml

@@ -9,6 +9,9 @@ metadata:
   name: azure-wi-webhook-controller-manager
   namespace: '{{ .Release.Namespace }}'
 spec:
+  {{- if .Values.revisionHistoryLimit }}
+  revisionHistoryLimit: {{ .Values.revisionHistoryLimit }}
+  {{- end }}
   replicas: {{ .Values.replicaCount }}
   selector:
     matchLabels:

third_party/open-policy-agent/gatekeeper/helmify/kustomize-for-helm.yaml

@@ -29,6 +29,7 @@ metadata:
   namespace: azure-workload-identity-system
 spec:
   replicas: HELMSUBST_DEPLOYMENT_REPLICAS
+  HELMSUBST_DEPLOYMENT_REVISION_HISTORY_LIMIT: ""
   selector:
     matchLabels:
       azure-workload-identity.io/system: "true"

third_party/open-policy-agent/gatekeeper/helmify/replacements.go

@@ -5,6 +5,10 @@ var replacements = map[string]string{
 
 	`HELMSUBST_DEPLOYMENT_NODE_SELECTOR: ""`: `{{- toYaml .Values.nodeSelector | nindent 8 }}`,
 
+	`HELMSUBST_DEPLOYMENT_REVISION_HISTORY_LIMIT: ""`: `{{- if .Values.revisionHistoryLimit }}
+  revisionHistoryLimit: {{ .Values.revisionHistoryLimit }}
+  {{- end }}`,
+
 	"HELMSUBST_DEPLOYMENT_REPLICAS": `{{ .Values.replicaCount }}`,
 
 	`HELMSUBST_DEPLOYMENT_AFFINITY: ""`: `{{- toYaml .Values.affinity | nindent 8 }}`,

third_party/open-policy-agent/gatekeeper/helmify/static/README.md

@@ -56,6 +56,7 @@ helm upgrade -n azure-workload-identity-system [RELEASE_NAME] azure-workload-ide
 | mutatingWebhookNamespaceSelector   | The namespace selector to further refine which namespaces will be selected by the webhook.                                        | `{}`                                                    |
 | podDisruptionBudget.minAvailable   | The minimum number of pods that must be available for the webhook to be considered available                                      | `1`                                                     |
 | podDisruptionBudget.maxUnavailable | The maximum number of pods that may be unavailable for the webhook to be considered available                                     | `nil`                                                   |
+| revisionHistoryLimit               | The number of old ReplicaSets to retain for the webhook deployment                                                                | `10`                                                    |
 
 ## Contributing Changes