From e7ed856e20c5a9f8156f4cd7e3b173e330bf8dd7 Mon Sep 17 00:00:00 2001
From: James Munnelly <james.munnelly@jetstack.io>
Date: Wed, 9 May 2018 16:54:25 +0100
Subject: [PATCH] Add leaderElection.namespace option to helm chart

---
 contrib/charts/cert-manager/Chart.yaml                | 2 +-
 contrib/charts/cert-manager/README.md                 | 1 +
 contrib/charts/cert-manager/templates/deployment.yaml | 5 +++++
 contrib/charts/cert-manager/values.yaml               | 4 ++++
 4 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/contrib/charts/cert-manager/Chart.yaml b/contrib/charts/cert-manager/Chart.yaml
index 1d47fa0ff..706e9c7a4 100644
--- a/contrib/charts/cert-manager/Chart.yaml
+++ b/contrib/charts/cert-manager/Chart.yaml
@@ -1,5 +1,5 @@
 name: cert-manager
-version: v0.3.0-alpha.5
+version: v0.3.0-alpha.6
 appVersion: v0.3.0-alpha.2
 description: A Helm chart for cert-manager
 home: https://github.com/jetstack/cert-manager
diff --git a/contrib/charts/cert-manager/README.md b/contrib/charts/cert-manager/README.md
index 24856b3d4..5d42f2d1d 100644
--- a/contrib/charts/cert-manager/README.md
+++ b/contrib/charts/cert-manager/README.md
@@ -59,6 +59,7 @@ The following table lists the configurable parameters of the cert-manager chart
 | `replicaCount`  | Number of cert-manager replicas  | `1` |
 | `createCustomResource` | Create CRD/TPR with this release | `true` |
 | `clusterResourceNamespace` | Override the namespace used to store DNS provider credentials etc. for ClusterIssuer resources | Same namespace as cert-manager pod
+| `leaderElection.Namespace` | Override the namespace used to store the ConfigMap for leader election | Same namespace as cert-manager pod
 | `certificateResourceShortNames` | Custom aliases for Certificate CRD | `["cert", "certs"]` |
 | `extraArgs` | Optional flags for cert-manager | `[]` |
 | `rbac.create` | If `true`, create and use RBAC resources | `true`
diff --git a/contrib/charts/cert-manager/templates/deployment.yaml b/contrib/charts/cert-manager/templates/deployment.yaml
index 886050e9c..2d25e870b 100644
--- a/contrib/charts/cert-manager/templates/deployment.yaml
+++ b/contrib/charts/cert-manager/templates/deployment.yaml
@@ -35,6 +35,11 @@ spec:
         {{- else }}
           - --cluster-resource-namespace=$(POD_NAMESPACE)
         {{- end }}
+        {{- if .Values.leaderElection.namespace }}
+          - --leader-election-namespace={{ .Values.leaderElection.namespace }}
+        {{- else }}
+          - --leader-election-namespace=$(POD_NAMESPACE)
+        {{- end }}
         {{- if .Values.extraArgs }}
 {{ toYaml .Values.extraArgs | indent 10 }}
         {{- end }}
diff --git a/contrib/charts/cert-manager/values.yaml b/contrib/charts/cert-manager/values.yaml
index 95e3bd84e..8b82b6595 100644
--- a/contrib/charts/cert-manager/values.yaml
+++ b/contrib/charts/cert-manager/values.yaml
@@ -15,6 +15,10 @@ createCustomResource: true
 # used. This namespace will not be automatically created by the Helm chart.
 clusterResourceNamespace: ""
 
+leaderElection:
+  # Override the namespace used to store the ConfigMap for leader election
+  namespace: ""
+
 certificateResourceShortNames: ["cert", "certs"]
 
 rbac: