From 5bd8aa1943776a42bd55b511682dd50a1ba11c61 Mon Sep 17 00:00:00 2001 From: Julian Siebert Date: Tue, 30 Jun 2026 09:49:41 +0200 Subject: [PATCH] Apply Copilot suggestion to avoid confusion in doc Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Julian Siebert --- pkg/chart/v2/util/dependencies.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/pkg/chart/v2/util/dependencies.go b/pkg/chart/v2/util/dependencies.go index f00a5ecd1..e92eb2348 100644 --- a/pkg/chart/v2/util/dependencies.go +++ b/pkg/chart/v2/util/dependencies.go @@ -215,11 +215,11 @@ Loop: // recursively call self to process sub dependencies for _, t := range cd { - // When a dependency uses an alias, cvals[path][alias] may be missing keys - // that were coalesced into cvals[originalName] by ancestor CoalesceValues - // calls (which used the original chart name, not the alias). To correct - // this, we backfill any missing keys from the top-level entry into the - // nested path entry. The nested entry is authoritative: if a key exists + // When processing nested dependencies, CoalesceValues is invoked with the full values tree, + // so defaults for this dependency can be coalesced into cvals[t.Metadata.Name] at the + // current values root instead of into the path-scoped table (under `path`). + // Backfill any missing keys from the root entry into the path-scoped entry so condition + // checks see chart defaults. // in both, the nested (alias-keyed) value wins. if path != "" { if pt, err := cvals.Table(strings.TrimSuffix(path, ".")); err == nil {