djjudas21 / charts

Collection of Helm charts
15 stars 6 forks source link

[graphite-exporter] remove duplicate `ports:` in deployment.yaml #34

Closed mateuszdrab closed 1 year ago

mateuszdrab commented 1 year ago

What this PR does / why we need it

Remove duplicate ports: in deployment of graphite-exporter

Which issue this PR fixes

Fluxv2 can't handle duplicate entries in YAML as per the following issue https://github.com/fluxcd/flux2/issues/1522 Warning error 3m55s (x44 over 8h) helm-controller reconciliation failed: Helm upgrade failed: error while running post render on files: map[string]interface {}(nil): yaml: unmarshal errors: line 38: mapping key "ports" already defined at line 37

Special notes for your reviewer

Checklist

djjudas21 commented 1 year ago

Thanks @mateuszdrab. I've merged and published the new release 0.1.6 now

mateuszdrab commented 1 year ago

Thanks, the release already deployed in my cluster with no errors this time 😉