kokuwaio / helm-charts

helm charts maintained by the kokuwa project
GNU General Public License v3.0
42 stars 70 forks source link

[fluentd-elasticsearch] Remove YAML syntax cruft #54

Closed stpierre closed 3 years ago

stpierre commented 3 years ago

Which chart

fluentd-elasticsearch

What this PR does / why we need it

Removes cruft from converting the list of arguments for the AWS sidecar from an inline list. YAML is remarkably flexible and can actually deal with most of this, but the stray ] caused issues with extra arguments.

Sorry this has taken so many PRs; I'm new to all of this and it's taken me a while to figure out how to effectively test parts of it.

Checklist