Fix helper template variables

[rvmiller89]

Description

In case we can't get #266 to work

Testing performed

• ci/build.sh
• Redeploy fluentd and fluentd-events pods
• Confirm events, logs, and metrics are coming in

[djsly]

This goes with #265, since nindent is now used, the strings should now have leading spaces

[djsly]

@rvmiller89 when you get a chance to merge this, could we release an alpha tag to allow a public chart to be accessible? our logic is currently based on your public helm REPO and I don't have an easy way to inject local charts into my pipelines.

[rvmiller89]

@djsly I'll bring it up to the team about releasing Helm charts for alpha tag versions, right now I think our concern is that by publishing the alpha versions in https://github.com/SumoLogic/sumologic-kubernetes-collection/blob/gh-pages/index.yaml the default helm install sumologic/sumologic would download and install the latest alpha chart, rather than the stable tested release.

[djsly]

Thanks. I haven’t tried it for say but it looks like helm will not fetch pre release version in a repo anymore unless the -version is used to explicitly chose one. helm/helm#2407 My gut feeling is that it should be safe to push some -alpha as long as they follow the pre-release format from SemVer2

…

On Oct 30, 2019, at 5:33 PM, Ryan Miller ***@***.***> wrote:  @djsly I'll bring it up to the team about releasing Helm charts for alpha tag versions, right now I think our concern is that by publishing the alpha versions in https://github.com/SumoLogic/sumologic-kubernetes-collection/blob/gh-pages/index.yaml the default helm install sumologic/sumologic would download and install the latest alpha chart, rather than the stable tested release. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.