Prioritize resource deletion over accessing connection info, add the …

…missing fmt.Sprintf inside the logger

internal/client/ssh.go

@@ -133,7 +133,7 @@ func NewSSHClient(ctx context.Context, data []byte) (*ssh.Client, error) { // no
 
 	if err != nil {
 		// Final failure after all attempts
-		logger.Info("All %d attempts to connect to %s failed.\n", maxAttempts, remoteHost)
+		logger.Info(fmt.Sprintf("All %d attempts to connect to %s failed.", maxAttempts, remoteHost))
 		return nil, err
 	}
 

internal/controller/script/script.go

@@ -107,6 +107,13 @@ func (c *connector) Connect(ctx context.Context, mg resource.Managed) (managed.E
 		return nil, errors.Wrap(err, errGetPC)
 	}
 
+	// if the resource is being deleted, we just return
+	cond := cr.GetCondition(xpv1.Deleting().Type)
+	if cond.Type == xpv1.TypeReady && cond.Status == "False" && cond.Reason == xpv1.ReasonDeleting {
+		logger.Info(fmt.Sprintf("[%s] Resource is being deleted. Skip the connection.", mg.GetName()))
+		return &external{}, nil
+	}
+
 	cd := pc.Spec.Credentials
 	data, err := resource.CommonCredentialExtractor(ctx, cd.Source, c.kube, cd.CommonCredentialSelectors)
 	if err != nil {