Skip to content

Commit 8bce316

Browse files
authored
Fix ENABLE_LEADER_ELECTION env variable name (#466)
Description of changes: This fixes that setting and use of env variable had different names. Without this fix leader election is always enabled in controller, but permissions might not be given. So if leader election isn't enabled in the values you get constant messages like: `E0911 06:54:26.919325 1 leaderelection.go:330] error retrieving resource lock alb-ingress/ack-acm.services.k8s.aws: leases.coordination.k8s.io "ack-acm.services.k8s.aws" is forbidden: User "system:serviceaccount:alb-ingress:ack-acm-controller" cannot get resource "leases" in API group "coordination.k8s.io" in the namespace "alb-ingress"` By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent 2f2b5e9 commit 8bce316

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

templates/helm/templates/deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ spec:
9191
value: {{ include "watch-namespace" . }}
9292
- name: DELETION_POLICY
9393
value: {{ .Values.deletionPolicy }}
94-
- name: ENABLED_LEADER_ELECTION
94+
- name: ENABLE_LEADER_ELECTION
9595
value: {{ .Values.leaderElection.enabled | quote }}
9696
- name: LEADER_ELECTION_NAMESPACE
9797
value: {{ .Values.leaderElection.namespace | quote }}

0 commit comments

Comments
 (0)