Resources that only contain comments after processing cause issues

[michaelgeorgeattard]

Given the following resource for example:

# ConfigMap with extra configuration specified directly to the chart
# for client agents only.
{{- if (or (and (ne (.Values.client.enabled | toString) "-") .Values.client.enabled) (and (eq (.Values.client.enabled | toString) "-") .Values.global.enabled)) }}
apiVersion: v1
kind: ConfigMap
metadata:
  name: {{ template "consul.fullname" . }}-client-config
  namespace: {{ .Release.Namespace }}
  labels:
    app: {{ template "consul.name" . }}
    chart: {{ template "consul.chart" . }}
    heritage: {{ .Release.Service }}
    release: {{ .Release.Name }}
data:
  extra-from-values.json: |-
{{ tpl .Values.client.extraConfig . | trimAll "\"" | indent 4 }}
{{- end }}

If through configuration, only the comments are left after processing:

# ConfigMap with extra configuration specified directly to the chart
# for client agents only.

helm-diff complains with the following error:

2019/03/28 10:40:34 Error: Found duplicate key "foo, , ()" in manifest

[andytom]

I think this might be a duplicate of #48 which was fixed in #127.

[michaelgeorgeattard]

@andytom Can you release a new version of the plugin please?

[stefanandres]

Yes, please release a new version. We'd really like to deploy this in our CI

[andytom]

@andytom Can you release a new version of the plugin please?

Sorry but I'm not a contributor, I just noticed that this issue was the same as one I had seem before. I think only @databus23 can release a new version.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.