Skip to content

Fix YAML error when lacksK8sScript is true#2

Open
funkypenguin wants to merge 1 commit intoiMartyn:masterfrom
funkypenguin:patch-1
Open

Fix YAML error when lacksK8sScript is true#2
funkypenguin wants to merge 1 commit intoiMartyn:masterfrom
funkypenguin:patch-1

Conversation

@funkypenguin
Copy link

Fixes the overly-aggressive trimming of whitespace in an if statement, which produced invalid YAML under true condition:

I.e, when lacksK8sScript is true, the deployment would fail a helm lint as follows:

helm lint .
==> Linting .
[INFO] Chart.yaml: icon is recommended
[ERROR] templates/deployment.yaml: unable to parse YAML: error converting YAML to JSON: yaml: line 70: did not find expected key

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

Fixes the overly-aggressive trimming of whitespace in an if statement, which produced invalid YAML under true condition
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant