vectordotdev / helm-charts

Helm charts for Vector.
https://vector.dev
Mozilla Public License 2.0
114 stars 89 forks source link

fix(vector): Fix indentation for haproxy.initContainers #390

Closed ludwighansson closed 5 months ago

ludwighansson commented 6 months ago

Hello friends!

I just encountered a small issue with the vector chart when trying to add an optional init-container for the HAProxy deployment. The line that expands .Values.haproxy.initContainers doesn't trim the initial whitespaces, causing the first line of the rendered YAML to have 16 spaces instead of the desired 8.

Added a {{- to trim the initial whitespaces (and newline), and used nindent to add the newline again. This aligns with other expansions within this template :)

This is my first PR, so please let me know if you need any additional info or if I need to change the commit message etc. Tried to keep it similar to the existing ones ;)

Cheers!