From b5ce8ac9f5c58805c45085b840dd1341b34a4391 Mon Sep 17 00:00:00 2001
From: Matthias Baur <baurmatt@users.noreply.github.com>
Date: Mon, 3 May 2021 11:03:54 +0200
Subject: [PATCH] Use release namespace by default for ClusterRoleBinding (#60)

This switches the default of the cert-manager namespace to the namespace
this release is going to be installed while still maintaing configurability.
This makes it easier to use a custom namespace for cert-manager because
designate-certmanager-webhook will most certainly just be installed in
the same namespace as cert-manager.
---
 helm/designate-certmanager-webhook/Chart.yaml          | 2 +-
 helm/designate-certmanager-webhook/templates/rbac.yaml | 2 +-
 helm/designate-certmanager-webhook/values.yaml         | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/helm/designate-certmanager-webhook/Chart.yaml b/helm/designate-certmanager-webhook/Chart.yaml
index e8310af3..a339edcc 100644
--- a/helm/designate-certmanager-webhook/Chart.yaml
+++ b/helm/designate-certmanager-webhook/Chart.yaml
@@ -2,4 +2,4 @@ apiVersion: v1
 appVersion: "0.2.12"
 description: ACME webhook Implementation for OpenStack Designate
 name: designate-certmanager-webhook
-version: "0.2.12"
+version: "0.2.13"
diff --git a/helm/designate-certmanager-webhook/templates/rbac.yaml b/helm/designate-certmanager-webhook/templates/rbac.yaml
index f7fdeb44..12999d98 100644
--- a/helm/designate-certmanager-webhook/templates/rbac.yaml
+++ b/helm/designate-certmanager-webhook/templates/rbac.yaml
@@ -93,7 +93,7 @@ subjects:
   - apiGroup: ""
     kind: ServiceAccount
     name: {{ .Values.certManager.serviceAccountName }}
-    namespace: {{ .Values.certManager.namespace }}
+    namespace: {{ .Values.certManager.namespace | default .Release.Namespace }}
   - apiGroup: ""
     kind: ServiceAccount
     name: {{ include "designate-certmanager-webhook.fullname" . }}
diff --git a/helm/designate-certmanager-webhook/values.yaml b/helm/designate-certmanager-webhook/values.yaml
index 9955b1a4..13539444 100644
--- a/helm/designate-certmanager-webhook/values.yaml
+++ b/helm/designate-certmanager-webhook/values.yaml
@@ -1,7 +1,7 @@
 replicaCount: 1
 
 certManager:
-  namespace: cert-manager
+  namespace: ~
   serviceAccountName: cert-manager
 
 image: