revert lostcancel analyzer

Signed-off-by: Karol Szwaj <karol.szwaj@gmail.com>

Author: cnvergence

backend/provider/kcp/provider.go

@@ -83,11 +83,11 @@ type awareWrapper struct {
 }
 
 func (a *awareWrapper) Engage(ctx context.Context, name multicluster.ClusterName, cluster cluster.Cluster) error {
-	ctx, cancel := context.WithCancel(ctx)
-	defer cancel()
+	ctx, cancel := context.WithCancel(ctx) //nolint:govet // cancel is called in the error case only.
 
 	err := a.Aware.Engage(ctx, name, cluster)
 	if err != nil {
+		cancel()
 		return err
 	}
 
@@ -108,6 +108,7 @@ func (a *awareWrapper) Engage(ctx context.Context, name multicluster.ClusterName
 		}
 
 		if !apierrors.IsForbidden(err) {
+			cancel()
 			return err
 		}
 
@@ -116,15 +117,18 @@ func (a *awareWrapper) Engage(ctx context.Context, name multicluster.ClusterName
 			select {
 			case <-time.After(backoff):
 			case <-ctx.Done():
+				cancel()
 				return ctx.Err()
 			}
 		}
 	}
 
 	if err != nil && !apierrors.IsAlreadyExists(err) {
+		cancel()
 		return err
 	}
-	return nil
+
+	return nil //nolint:govet // cancel is called in the error case only.
 }
 
 // NewKCPExternalAddressGenerator returns an ExternalAddressGeneratorFunc