Skip to content

Commit 5dff9df

Browse files
authored
Merge pull request #5099 from justinsb/dont_swallow_helm_errors
Don't swallow helm execution errors
2 parents 84682a1 + c151147 commit 5dff9df

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

api/internal/builtins/HelmChartInflationGenerator.go

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

plugin/builtin/helmchartinflationgenerator/HelmChartInflationGenerator.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,8 +161,8 @@ func (p *plugin) runHelmCommand(
161161
helm := p.h.GeneralConfig().HelmConfig.Command
162162
err = errors.WrapPrefixf(
163163
fmt.Errorf(
164-
"unable to run: '%s %s' with env=%s (is '%s' installed?)",
165-
helm, strings.Join(args, " "), env, helm),
164+
"unable to run: '%s %s' with env=%s (is '%s' installed?): %w",
165+
helm, strings.Join(args, " "), env, helm, err),
166166
stderr.String(),
167167
)
168168
}

0 commit comments

Comments
 (0)