From 65bd9a7a04d7c8eb57432a8bbaabc541bed1a472 Mon Sep 17 00:00:00 2001 From: jlego229-kim Date: Wed, 17 Apr 2019 13:08:32 +0900 Subject: [PATCH] Change default of ironic.conductor.api_url from internal to public Currently a pxe-client has to contact with ironic-api through it's internal endpoint during lookup() period. However, the pxe-client cannot access kube-dns by ifself, so it needs additional dns server. (or Using NodePort) With additional dns server, it will be safer for pxe-client to contact with ironic-api through it's public endpoint(passing by ingress) rather than internal endpoint directly becuase internal pod's IPs would be changed frequently. Also, I refered to {Values.conf.ironic.glance.swift_endpoint_url}. (swift is also accessed by the pxe-client and swift_url's default is public endpoint) Change-Id: I0ad97f3ed608973d7e5a4a11d87595fe258a0db5 --- ironic/templates/configmap-etc.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ironic/templates/configmap-etc.yaml b/ironic/templates/configmap-etc.yaml index 613e4bfe1d..bf92433ff2 100644 --- a/ironic/templates/configmap-etc.yaml +++ b/ironic/templates/configmap-etc.yaml @@ -165,7 +165,7 @@ limitations under the License. {{- end -}} {{- if empty .Values.conf.ironic.conductor.api_url -}} -{{- $_ := tuple "baremetal" "internal" "api" . | include "helm-toolkit.endpoints.keystone_endpoint_uri_lookup"| set .Values.conf.ironic.conductor "api_url" -}} +{{- $_ := tuple "baremetal" "public" "api" . | include "helm-toolkit.endpoints.keystone_endpoint_uri_lookup"| set .Values.conf.ironic.conductor "api_url" -}} {{- end -}} {{- if empty .Values.conf.ironic.api.public_endpoint -}}