From 754867a3b252e24d5d2364d5ea65a7c1e38a796d Mon Sep 17 00:00:00 2001 From: Tin Lam Date: Tue, 31 Oct 2017 13:34:30 -0500 Subject: [PATCH] Change .to_oslo_conf to .to_ini Currently, openstackhelm-toolkit provides two utility methods: .to_oslo_conf and .to_ini whose only difference at this time is the ability to handle multistring in .to_oslo_conf as it is an openstack concept that is not found in .ini files. As logging.conf and paste.ini do not support multistring, they do not need to be render by .to_oslo_conf, and while the code is not incorrect, using .to_ini would be more appropriate. Trivial fix Change-Id: I20ae77842a3406ea64bf40d951adef2e519cbfcb --- charts/deckhand/templates/configmap-etc.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/charts/deckhand/templates/configmap-etc.yaml b/charts/deckhand/templates/configmap-etc.yaml index 641ee9f2..88766fe8 100644 --- a/charts/deckhand/templates/configmap-etc.yaml +++ b/charts/deckhand/templates/configmap-etc.yaml @@ -65,8 +65,8 @@ data: deckhand.conf: |+ {{ include "helm-toolkit.utils.to_oslo_conf" .Values.conf.deckhand | indent 4 }} logging.conf: |+ -{{ include "helm-toolkit.utils.to_oslo_conf" .Values.conf.logging | indent 4 }} +{{ include "helm-toolkit.utils.to_ini" .Values.conf.logging | indent 4 }} deckhand-paste.ini: |+ -{{ include "helm-toolkit.utils.to_oslo_conf" .Values.conf.paste | indent 4 }} +{{ include "helm-toolkit.utils.to_ini" .Values.conf.paste | indent 4 }} policy.yaml: |+ {{ toYaml .Values.conf.policy | indent 4 }}