jertel / elastalert2

ElastAlert 2 is a continuation of the original yelp/elastalert project. Pull requests are appreciated!
https://elastalert2.readthedocs.org
Apache License 2.0
895 stars 282 forks source link

Correct deployment commonLabels indentation for pod template #1250

Closed dan-duffy closed 1 year ago

dan-duffy commented 1 year ago

Description

I noticed when using the commonLabels value that the deployment was unable to render additional labels correctly. It looks like the pod template specific template's indentation was off by 4 spaces.

Checklist

Questions or Comments

As this is a minor indentation change specific to the Helm chart, the testing didn't seem to be relevant but please correct me if I'm wrong! I've tested the indentation locally and the manifests seem to be rendering correctly now when adding commonLabels.