Find Email

[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.

Feedback