starlingx/openstack-armada-app
Code Issues Proposed changes

Fix secret tls for placement helm chart

Browse Source 
When HTTPS is enabled, the public endpoint of placement service
is set with https and fqdn. The placement helm chart would fail
to apply.
This commit is to correct the syntax of the secret tls section
in the values.yaml and secret-ingress-tls.yaml.

Upstream (openstack-helm) review:
https://review.opendev.org/#/c/662229

Partial-Bug: 1826583

Change-Id: I64e7c1ff1972b0640145d18cafe2ea4e505db465
Signed-off-by: Teresa Ho <teresa.ho@windriver.com>
This commit is contained in:
Teresa Ho 2019-07-12 13:37:53 -04:00
parent e03692c512
commit c89d718248
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
openstack-helm/files/0015-Add-Placement-Chart.patch
View File

@ -610,7 +610,7 @@ index 0000000..3413b5b
+*/}}
+
+{{- if .Values.manifests.secret_ingress_tls }}
+{{ include "helm-toolkit.manifests.secret_ingress_tls" ( dict "envAll" . "backendService" "placement" "backendServiceType" "placement" ) }}
+{{ include "helm-toolkit.manifests.secret_ingress_tls" ( dict "envAll" . "backendServiceType" "placement" ) }}
+{{- end }}
diff --git a/placement/templates/secret-keystone.yaml b/placement/templates/secret-keystone.yaml
new file mode 100644
@ -1041,7 +1041,7 @@ index 0000000..33139f0
+ placement: placement-db-user
+ tls:
+ placement:
+ placement:
+ api:
+ public: placement-tls-public
+
+dependencies: