(pre-flight check) Improve error message

anik120 · anik120 · commit 1513098a8d46 · 2025-06-03T15:57:05.000-04:00
The line "failed to get release state using client-only dry-run" is
too jargon heavy, and meaningless for the end user.
This PR replaces the error message with a more user friendly text.
diff --git a/internal/operator-controller/applier/helm.go b/internal/operator-controller/applier/helm.go
@@ -145,7 +145,7 @@ func (h *Helm) Apply(ctx context.Context, contentFS fs.FS, ext *ocv1.ClusterExte
 
 	rel, desiredRel, state, err := h.getReleaseState(ac, ext, chrt, values, post)
 	if err != nil {
-		return nil, "", fmt.Errorf("failed to get release state using server-side dry-run: %w", err)
+		return nil, "", fmt.Errorf("error rendering bundle for pre-authorization checks: %w", err)
 	}
 
 	for _, preflight := range h.Preflights {

Original file line number	Diff line number	Diff line change
`@@ -145,7 +145,7 @@ func (h Helm) Apply(ctx context.Context, contentFS fs.FS, ext ocv1.ClusterExte`
`145`	`145`
`146`	`146`	`rel, desiredRel, state, err := h.getReleaseState(ac, ext, chrt, values, post)`
`147`	`147`	`if err != nil {`
`148`		`- return nil, "", fmt.Errorf("failed to get release state using server-side dry-run: %w", err)`
	`148`	`+ return nil, "", fmt.Errorf("error rendering bundle for pre-authorization checks: %w", err)`
`149`	`149`	`}`
`150`	`150`
`151`	`151`	`for _, preflight := range h.Preflights {`