review comments incorporated

Signed-off-by: rashmi_kh <[email protected]>

Author: rashmi43

internal/authentication/tokengetter.go

@@ -22,11 +22,12 @@ type TokenGetter struct {
 }
 
 type ServiceAccountNotFoundError struct {
-	ServiceAccountName string
+	ServiceAccountName string // The name of the missing ServiceAccount.
 }
 
+// Error implements the error interface for ServiceAccountNotFoundError.
 func (e *ServiceAccountNotFoundError) Error() string {
-	return fmt.Sprintf("Unable to authenticate with Kubernetes cluster using ServiceAccount \"%s\": ServiceAccount \"%s\" not found.", e.ServiceAccountName, e.ServiceAccountName)
+	return fmt.Sprintf("ServiceAccount \"%s\" not found: Unable to authenticate with the Kubernetes cluster.", e.ServiceAccountName)
 }
 
 type TokenGetterOption func(*TokenGetter)
@@ -97,8 +98,7 @@ func (t *TokenGetter) getToken(ctx context.Context, key types.NamespacedName) (*
 		}, metav1.CreateOptions{})
 	if err != nil {
 		if errors.IsNotFound(err) {
-			saErr := &ServiceAccountNotFoundError{key.Name}
-			return nil, saErr
+			return nil, &ServiceAccountNotFoundError{ServiceAccountName: key.Name}
 		}
 		return nil, err
 	}

internal/controllers/clusterextension_controller.go

@@ -47,10 +47,10 @@ import (
 	"github.com/operator-framework/api/pkg/operators/v1alpha1"
 	catalogd "github.com/operator-framework/catalogd/api/v1"
 	helmclient "github.com/operator-framework/helm-operator-plugins/pkg/client"
-	tokengetter "github.com/operator-framework/operator-controller/internal/authentication"
 	"github.com/operator-framework/operator-registry/alpha/declcfg"
 
 	ocv1 "github.com/operator-framework/operator-controller/api/v1"
+	"github.com/operator-framework/operator-controller/internal/authentication"
 	"github.com/operator-framework/operator-controller/internal/bundleutil"
 	"github.com/operator-framework/operator-controller/internal/conditionsets"
 	"github.com/operator-framework/operator-controller/internal/contentmanager"
@@ -207,7 +207,7 @@ func (r *ClusterExtensionReconciler) reconcile(ctx context.Context, ext *ocv1.Cl
 	installedBundle, err := r.InstalledBundleGetter.GetInstalledBundle(ctx, ext)
 	if err != nil {
 		setInstallStatus(ext, nil)
-		var saerr *tokengetter.ServiceAccountNotFoundError
+		var saerr *authentication.ServiceAccountNotFoundError
 		if errors.As(err, &saerr) {
 			l.Info("ServiceAccount defined in the ClusterExtension is not found")
 			err = saerr