From f590c59930ee922521db0168d40ae1dbb67920ab Mon Sep 17 00:00:00 2001 From: Karthik Chowdary <21139050+Karthik-Chowdary@users.noreply.github.com> Date: Wed, 26 Aug 2026 09:22:13 +0000 Subject: [PATCH] fix(cmd): suppress wait progress during dry runs Signed-off-by: Karthik Chowdary <21139050+Karthik-Chowdary@users.noreply.github.com> --- pkg/cmd/install.go | 6 +++--- pkg/cmd/upgrade.go | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/pkg/cmd/install.go b/pkg/cmd/install.go index b9affaaaa..ff8a3de08 100644 --- a/pkg/cmd/install.go +++ b/pkg/cmd/install.go @@ -158,7 +158,7 @@ func newInstallCmd(cfg *action.Configuration, out io.Writer) *cobra.Command { } client.DryRunStrategy = dryRunStrategy - printWaitMessage(out, outfmt, client.WaitStrategy, client.Timeout) + printWaitMessage(out, outfmt, client.WaitStrategy, client.DryRunStrategy, client.Timeout) rel, err := runInstall(args, client, valueOpts, out) if err != nil { @@ -373,8 +373,8 @@ func checkIfInstallable(ch chart.Accessor) error { return fmt.Errorf("%s charts are not installable", meta["Type"]) } -func printWaitMessage(out io.Writer, outfmt output.Format, strategy kube.WaitStrategy, timeout time.Duration) { - if outfmt != output.Table || strategy == kube.HookOnlyStrategy { +func printWaitMessage(out io.Writer, outfmt output.Format, strategy kube.WaitStrategy, dryRun action.DryRunStrategy, timeout time.Duration) { + if outfmt != output.Table || strategy == kube.HookOnlyStrategy || dryRun != action.DryRunNone { return } fmt.Fprintf(out, "Waiting for resources to become ready (timeout: %s)\n", timeout) diff --git a/pkg/cmd/upgrade.go b/pkg/cmd/upgrade.go index 3df130b3c..e8bc834ac 100644 --- a/pkg/cmd/upgrade.go +++ b/pkg/cmd/upgrade.go @@ -163,7 +163,7 @@ func newUpgradeCmd(cfg *action.Configuration, out io.Writer) *cobra.Command { instClient.Replace = true } - printWaitMessage(out, outfmt, instClient.WaitStrategy, instClient.Timeout) + printWaitMessage(out, outfmt, instClient.WaitStrategy, instClient.DryRunStrategy, instClient.Timeout) rel, err := runInstall(args, instClient, valueOpts, out) if err != nil { @@ -259,7 +259,7 @@ func newUpgradeCmd(cfg *action.Configuration, out io.Writer) *cobra.Command { cancel() }() - printWaitMessage(out, outfmt, client.WaitStrategy, client.Timeout) + printWaitMessage(out, outfmt, client.WaitStrategy, client.DryRunStrategy, client.Timeout) rel, err := client.RunWithContext(ctx, args[0], ch, vals) if err != nil {