From 973abf99783767cdf5efdb64ad87b734d2d1d48a Mon Sep 17 00:00:00 2001 From: Brett Tofel Date: Wed, 9 Aug 2023 12:12:52 +0700 Subject: [PATCH] Improve actuator AZident field validation err msg Along the same lines as OCPBUGS-16684 changes --- pkg/azure/actuator.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/pkg/azure/actuator.go b/pkg/azure/actuator.go index d834edf84..1c1724408 100644 --- a/pkg/azure/actuator.go +++ b/pkg/azure/actuator.go @@ -557,7 +557,12 @@ func (a *Actuator) syncWorkloadIdentitySecret( } err := validateAzureProviderSpec(azureProviderSpec) if err != nil { - return err + logger.WithError(err).Error("error determining whether a credentials update is needed") + msg := "error validating credentials request Azure AD Workload Identity fields" + return &actuatoriface.ActuatorError{ + ErrReason: minterv1.CredentialsProvisionFailure, + Message: fmt.Sprintf("%v: %v", msg, err), + } } op, err := controllerutil.CreateOrPatch(ctx, a.client, secret, func() error { if secret.Labels == nil {