Merge pull request #39 from syseleven/os-project-name

Make it possible to configure OS_PROJECT_NAME in helm chart
syseleven · Nov 5, 2020 · f6501c4 · f6501c4
2 parents 3ebebaa + 9725123
commit f6501c4
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/helm/designate-certmanager-webhook/Chart.yaml b/helm/designate-certmanager-webhook/Chart.yaml
@@ -1,5 +1,5 @@
 apiVersion: v1
-appVersion: "0.2.8"
+appVersion: "0.2.9"
 description: ACME webhook Implementation for OpenStack Designate
 name: designate-certmanager-webhook
-version: "0.2.9"
+version: "0.2.10"
diff --git a/helm/designate-certmanager-webhook/templates/secret.yaml b/helm/designate-certmanager-webhook/templates/secret.yaml
@@ -18,6 +18,10 @@ data:
  {{- end }}
  {{- if .Values.openstack.project_id }}
  OS_PROJECT_ID: {{ .Values.openstack.project_id | b64enc | quote }}
+ {{- else if .Values.openstack.project_name }}
+ OS_PROJECT_NAME: {{ .Values.openstack.project_name | b64enc | quote }}
+ {{- else }}
+ {{- fail "project_id or project_name is needed!" }}
  {{- end }}
  {{- if .Values.openstack.region_name }}
  OS_REGION_NAME: {{ .Values.openstack.region_name | b64enc | quote }}

diff --git a/helm/designate-certmanager-webhook/values.yaml b/helm/designate-certmanager-webhook/values.yaml
@@ -22,6 +22,7 @@ openstack:
  username: ""
  password: ""
  project_id: ""
+ project_name: "" # Use project_id OR project_name
  region_name: ""
  auth_url: ""
  domain_name: ""