Don't cleanup forklift entities on remote cluster

When MTV is not installed on the remote OCP cluster and we try to
cleanup we won't succeed. This patch won't try to remove or edit
forklift entities, as if MTV doesn't exist there, it will fail - causing
the plan to fail.

Signed-off-by: Liran Rotenberg <[email protected]>

pkg/controller/plan/migration.go

@@ -469,8 +469,10 @@ func (r *Migration) cleanup(vm *plan.VMStatus, failOnErr func(error) bool) error
 	if err := r.kubevirt.DeleteHookJobs(vm); failOnErr(err) {
 		return err
 	}
-	if err := r.destinationClient.DeletePopulatorDataSource(vm); failOnErr(err) {
-		return err
+	if r.Plan.Provider.Destination.IsHost() {
+		if err := r.destinationClient.DeletePopulatorDataSource(vm); failOnErr(err) {
+			return err
+		}
 	}
 	if err := r.kubevirt.DeletePopulatorPods(vm); failOnErr(err) {
 		return err
@@ -829,10 +831,12 @@ func (r *Migration) execute(vm *plan.VMStatus) (err error) {
 			}
 		}
 		// set ownership to populator Crs
-		err = r.destinationClient.SetPopulatorCrOwnership()
-		if err != nil {
-			err = liberr.Wrap(err)
-			return
+		if r.Plan.Provider.Destination.IsHost() {
+			err = r.destinationClient.SetPopulatorCrOwnership()
+			if err != nil {
+				err = liberr.Wrap(err)
+				return
+			}
 		}
 		// set ownership to populator pods
 		err = r.kubevirt.SetPopulatorPodOwnership(vm)