Helm config spec cannot be set because the some paths are incorrect

[kentakozuka]

Description

In the zookeeper helm chart, some config specs cannot be set because the paths are incorrect.
They should be;

    {{- if .Values.config.xxxx}}
    xxxx: {{ .Values.config.xxxx }}
    {{- end }}

But in some lines, they are;

    {{- if .Values.config.xxxx}}
    xxxx: {{ .Values.xxxx }}
    {{- end }}

Location

zookeeper-operator/charts/zookeeper/templates/zookeeper.yaml

Lines 91 to 137 in b50eaa4

	config:
	{{- if .Values.config.initLimit }}
	initLimit: {{ .Values.config.initLimit }}
	{{- end }}
	{{- if .Values.config.tickTime }}
	tickTime: {{ .Values.config.tickTime }}
	{{- end }}
	{{- if .Values.config.syncLimit }}
	syncLimit: {{ .Values.config.syncLimit }}
	{{- end }}
	{{- if .Values.config.globalOutstandingLimit }}
	globalOutstandingLimit: {{ .Values.config.globalOutstandingLimit }}
	{{- end }}
	{{- if .Values.config.preAllocSize }}
	preAllocSize: {{ .Values.config.preAllocSize }}
	{{- end }}
	{{- if .Values.config.snapCount }}
	snapCount: {{ .Values.config.snapCount }}
	{{- end }}
	{{- if .Values.config.commitLogCount }}
	commitLogCount: {{ .Values.config.commitLogCount }}
	{{- end }}
	{{- if .Values.config.snapSizeLimitInKb }}
	snapSizeLimitInKb: {{ .Values.snapSizeLimitInKb }}
	{{- end }}
	{{- if .Values.config.maxCnxns }}
	maxCnxns: {{ .Values.maxCnxns }}
	{{- end }}
	{{- if .Values.config.maxClientCnxns }}
	maxClientCnxns: {{ .Values.maxClientCnxns }}
	{{- end }}
	{{- if .Values.config.minSessionTimeout }}
	minSessionTimeout: {{ .Values.minSessionTimeout }}
	{{- end }}
	{{- if .Values.config.maxSessionTimeout }}
	maxSessionTimeout: {{ .Values.maxSessionTimeout }}
	{{- end }}
	{{- if .Values.config.autoPurgeSnapRetainCount }}
	autoPurgeSnapRetainCount: {{ .Values.autoPurgeSnapRetainCount }}
	{{- end }}
	{{- if .Values.config.autoPurgePurgeInterval }}
	autoPurgePurgeInterval: {{ .Values.config.autoPurgePurgeInterval }}
	{{- end }}
	{{- if .Values.config.quorumListenOnAllIPs }}
	quorumListenOnAllIPs: {{ .Values.config.quorumListenOnAllIPs }}
	{{- end }}
	{{- end }}

Suggestions for an improvement

It is necessary to set correct paths.

[SrishT]

@kentakozuka this would require a minor fix. Would u like to send out a PR in order to fix this issue?