feat: allow changing the default revisionHistoryLimit (#2920)

Co-authored-by: Sertaç Özercan <[email protected]>
open-policy-agent · Aug 2, 2023 · db4f346 · db4f346
1 parent 8d72e8d
commit db4f346
Show file tree

Hide file tree

Showing 6 changed files with 8 additions and 0 deletions.
diff --git a/cmd/build/helmify/kustomize-for-helm.yaml b/cmd/build/helmify/kustomize-for-helm.yaml
@@ -62,6 +62,7 @@ metadata:
   namespace: gatekeeper-system
 spec:
   replicas: HELMSUBST_DEPLOYMENT_REPLICAS
+  revisionHistoryLimit: HELMSUBST_DEPLOYMENT_REVISION_HISTORY_LIMIT
   strategy:
     type: HELMSUBST_DEPLOYMENT_CONTROLLER_MANAGER_STRATEGY_TYPE
   template:
@@ -147,6 +148,7 @@ metadata:
   name: gatekeeper-audit
   namespace: gatekeeper-system
 spec:
+  revisionHistoryLimit: HELMSUBST_DEPLOYMENT_REVISION_HISTORY_LIMIT
   template:
     metadata:
       annotations:

diff --git a/cmd/build/helmify/replacements.go b/cmd/build/helmify/replacements.go
@@ -73,6 +73,8 @@ var replacements = map[string]string{
 
 	"HELMSUBST_DEPLOYMENT_REPLICAS": `{{ .Values.replicas }}`,
 
+	"HELMSUBST_DEPLOYMENT_REVISION_HISTORY_LIMIT": `{{ .Values.revisionHistoryLimit }}`,
+
 	`HELMSUBST_ANNOTATIONS: ""`: `{{- if .Values.podAnnotations }}
         {{- toYaml .Values.podAnnotations | trim | nindent 8 }}
         {{- end }}`,

diff --git a/cmd/build/helmify/static/values.yaml b/cmd/build/helmify/static/values.yaml
@@ -1,4 +1,5 @@
 replicas: 3
+revisionHistoryLimit: 10
 auditInterval: 60
 metricsBackends: ["prometheus"]
 auditMatchKindOnly: false

diff --git a/manifest_staging/charts/gatekeeper/templates/gatekeeper-audit-deployment.yaml b/manifest_staging/charts/gatekeeper/templates/gatekeeper-audit-deployment.yaml
@@ -13,6 +13,7 @@ metadata:
   namespace: '{{ .Release.Namespace }}'
 spec:
   replicas: 1
+  revisionHistoryLimit: {{ .Values.revisionHistoryLimit }}
   selector:
     matchLabels:
       app: '{{ template "gatekeeper.name" . }}'

diff --git a/manifest_staging/charts/gatekeeper/templates/gatekeeper-controller-manager-deployment.yaml b/manifest_staging/charts/gatekeeper/templates/gatekeeper-controller-manager-deployment.yaml
@@ -13,6 +13,7 @@ metadata:
   namespace: '{{ .Release.Namespace }}'
 spec:
   replicas: {{ .Values.replicas }}
+  revisionHistoryLimit: {{ .Values.revisionHistoryLimit }}
   selector:
     matchLabels:
       app: '{{ template "gatekeeper.name" . }}'

diff --git a/manifest_staging/charts/gatekeeper/values.yaml b/manifest_staging/charts/gatekeeper/values.yaml
@@ -1,4 +1,5 @@
 replicas: 3
+revisionHistoryLimit: 10
 auditInterval: 60
 metricsBackends: ["prometheus"]
 auditMatchKindOnly: false