helm lint fails due to extra hyphen in if statement

[cmontemuino]

What steps did you take and what happened:

git clone [email protected]:openebs/lvm-localpv.git
git checkout lvm-localpv-1.3.0
cd deploy/helm/charts
helm lint .

Output:

==> Linting .
[ERROR] templates/rbac.yaml: unable to parse YAML: invalid Yaml document separator: apiVersion: v1

Error: 1 chart(s) linted, 1 chart(s) failed

What did you expect to happen:

I would expect this output instead:

==> Linting .

1 chart(s) linted, 0 chart(s) failed

Anything else you would like to add:

Offending code:

lvm-localpv/deploy/helm/charts/templates/rbac.yaml

Line 126 in 400159d

{{- if .Values.serviceAccount.lvmNode.create -}}

{{- if .Values.serviceAccount.lvmNode.create -}}

The hyphen before the closing brackets is chomping all the whitespace after the brackets.

I'd like to open a PR with the fix.

Environment:

Not applicable

• LVM Driver version
• Kubernetes version (use kubectl version):
• Kubernetes installer & version:
• Cloud provider or hardware configuration:
• OS (e.g. from /etc/os-release):

[abhilashshetty04]

Hi @cmontemuino , Thanks for creating the issue. Please go ahead and raise the PR.

[cmontemuino]

Hi @cmontemuino , Thanks for creating the issue. Please go ahead and raise the PR.

Here it is: #272