From 41ebb0b485556417a0820d1624ac0d5e1d37cd1b Mon Sep 17 00:00:00 2001 From: Matt Dainty Date: Tue, 30 Jan 2024 21:19:58 +0000 Subject: [PATCH] fix: Add revisionHistoryLimit override (#3486) Signed-off-by: Matt Dainty --- helm/aws-load-balancer-controller/README.md | 1 + helm/aws-load-balancer-controller/templates/deployment.yaml | 1 + helm/aws-load-balancer-controller/values.yaml | 2 ++ 3 files changed, 4 insertions(+) diff --git a/helm/aws-load-balancer-controller/README.md b/helm/aws-load-balancer-controller/README.md index 02b47ea15..dbb5aaf04 100644 --- a/helm/aws-load-balancer-controller/README.md +++ b/helm/aws-load-balancer-controller/README.md @@ -246,6 +246,7 @@ The default values set by the application itself can be confirmed [here](https:/ | `extraVolumes` | Extra volumes for the pod | `[]` | | `defaultTags` | Default tags to apply to all AWS resources managed by this controller | `{}` | | `replicaCount` | Number of controller pods to run, only one will be active due to leader election | `2` | +| `revisionHistoryLimit` | Number of revisions to keep | `10` | | `podDisruptionBudget` | Limit the disruption for controller pods. Require at least 2 controller replicas and 3 worker nodes | `{}` | | `updateStrategy` | Defines the update strategy for the deployment | `{}` | | `enableCertManager` | If enabled, cert-manager issues the webhook certificates instead of the helm template, requires cert-manager and it's CRDs to be installed | `false` | diff --git a/helm/aws-load-balancer-controller/templates/deployment.yaml b/helm/aws-load-balancer-controller/templates/deployment.yaml index 73967b5cd..3984bf450 100644 --- a/helm/aws-load-balancer-controller/templates/deployment.yaml +++ b/helm/aws-load-balancer-controller/templates/deployment.yaml @@ -11,6 +11,7 @@ metadata: {{- include "aws-load-balancer-controller.labels" . | nindent 4 }} spec: replicas: {{ .Values.replicaCount }} + revisionHistoryLimit: {{ .Values.revisionHistoryLimit }} selector: matchLabels: {{- include "aws-load-balancer-controller.selectorLabels" . | nindent 6 }} diff --git a/helm/aws-load-balancer-controller/values.yaml b/helm/aws-load-balancer-controller/values.yaml index b4fbab46c..da7c6e039 100644 --- a/helm/aws-load-balancer-controller/values.yaml +++ b/helm/aws-load-balancer-controller/values.yaml @@ -4,6 +4,8 @@ replicaCount: 2 +revisionHistoryLimit: 10 + image: repository: public.ecr.aws/eks/aws-load-balancer-controller tag: v2.6.1