You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

waterWang e35f6bb312 Fix #32530: preserve null values in top-level chart values.yaml ... Since Helm 4.2.0, null-valued keys defined directly in the top-level chart's own values.yaml were being removed from `.Values` during coalescing. For example: ```yaml stuff: foo: bar baz: null foobar: null ``` would render `{"stuff":{"foo":"bar"}}` instead of `{"foobar":null,"stuff":{"baz":null,"foo":"bar"}}`. The root cause is in `coalesceValues`: the nil-skipping and `cleanNilValues` cleanup applied to `val == nil` keys was being applied unconditionally to the top-level chart's own defaults, not just to subchart defaults. This change restricts the nil cleanup to subchart coalescing (`prefix != ""`), so null values defined directly in the top-level chart's values.yaml are now preserved. Subchart default nils are still cleaned as before, retaining the in-place delete behavior from the `cleanNilValues` path. Adds a regression test covering both a top-level null key and a nested null key. Restores the pre-4.2.0 behavior described in the issue. Signed-off-by: waterWang		1 month ago
..
testdata	…
coalesce.go	Fix #32530: preserve null values in top-level chart values.yaml	1 month ago
coalesce_test.go	Fix #32530: preserve null values in top-level chart values.yaml	1 month ago
jsonschema.go	chore: enable contextcheck, fatcontext and noctx linters	3 months ago
jsonschema_test.go	chore(pkg): refactor: convert tests to testify assert/require part 9	2 months ago
values.go	…
values_test.go	chore(pkg): refactor: convert tests to testify assert/require part 20	2 months ago