[helm/charts] [stable/datadog] Fix a bunch of nil defaults (#22999)

From: notifications@github.com
Domain: IP info github.com
MX-server: IP info out-20.smtp.github.com
Size: 5466 Bytes
Create: 2020-06-29
Update: 2020-06-29
Score: 0
Safe: Yes

Outbound domains: github.com |

This PR is a follow-up to #2243 to set defaults for all values in values.yaml.

In particular, it fixes the following error I was experiencing, but I didn't see a reason to not set defaults for all values to avoid this problem in future:

Merging destination map for chart 'datadog'. The destination item 'podAnnotations' is a table and ignoring the podAnnotations 'confd' as it has a non-table value of: <nil>

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • DCO signed
  • Chart Version bumped
  • Variables are documented in the README.md
  • Title of the PR starts with chart name (e.g. [stable/mychartname])

@ahmed-mez @celenechang @CharlyF @clamoriniere @irabinovitch @L3n41c @vboulineau @xornivore


You can view, comment on, or merge this pull request online at:

  https://github.com/helm/charts/pull/22999

Commit Summary

  • [stable/datadog] Fix a bunch of nil defaults

File Changes

  • M stable/datadog/CHANGELOG.md (4)
  • M stable/datadog/values.yaml (77)

Patch Links:

  • https://github.com/helm/charts/pull/22999.patch
  • https://github.com/helm/charts/pull/22999.diff


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Want to protect your real email from messages like this? Use TempM email and be more secure on the internet.