openstack/openstack-helm-infra
Code Issues Proposed changes

Ceph: fix overriding ceph monitor hosts value

Browse Source 
all other charts that related with ceph have the logic
to override the ceph monitor hosts.
but storage class chart only has no logic to override the value.
to use external ceph, it must be able to set manually the ceph host.

Change-Id: Ib788e339606a2f7336a688aa276082503a18b04f
Signed-off-by: Hyunkook Cho <hk0713.cho@samsung.com>
This commit is contained in:
Hyunkook Cho 2019-04-02 20:11:01 +09:00
parent db209e0bb5
commit c7a6a96080
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
helm-toolkit/templates/manifests/_ceph-storageclass.tpl
View File

@ -87,6 +87,10 @@ examples:
{{- define "helm-toolkit.manifests.ceph-storageclass" -}}
{{- $envAll := index . "envAll" -}}
{{- $monHost := $envAll.Values.conf.ceph.global.mon_host -}}
{{- if empty $monHost -}}
{{- $monHost = tuple "ceph_mon" "internal" "mon" $envAll | include "helm-toolkit.endpoints.host_and_port_endpoint_uri_lookup" -}}
{{- end -}}
{{- $storageclassData := index . "storageclass_data" -}}
---
{{- if $storageclassData.provision_storage_class }}
@ -100,7 +104,7 @@ metadata:
name: {{ $storageclassData.metadata.name }}
provisioner: {{ $storageclassData.provisioner }}
parameters:
monitors: {{ tuple "ceph_mon" "internal" "mon" $envAll | include "helm-toolkit.endpoints.host_and_port_endpoint_uri_lookup" }}
monitors: {{ $monHost }}
{{- range $attr, $value := $storageclassData.parameters }}
{{ $attr }}: {{ $value | quote }}
{{- end }}