attempt address AfterSuite and generate-verify failures

Signed-off-by: Michael Henriksen <mhenriks@redhat.com>
kubevirt · Jun 22, 2023 · 99bf0dd · 99bf0dd
1 parent 75e1297
commit 99bf0dd
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/pkg/controller/datavolume/BUILD.bazel b/pkg/controller/datavolume/BUILD.bazel
@@ -72,7 +72,9 @@ go_test(
     embed = [":go_default_library"],
     deps = [
         "//pkg/common:go_default_library",
+        "//pkg/controller/clone:go_default_library",
         "//pkg/controller/common:go_default_library",
+        "//pkg/controller/populators:go_default_library",
         "//pkg/feature-gates:go_default_library",
         "//pkg/token:go_default_library",
         "//staging/src/kubevirt.io/containerized-data-importer-api/pkg/apis/core/v1beta1:go_default_library",

diff --git a/pkg/controller/datavolume/pvc-clone-controller.go b/pkg/controller/datavolume/pvc-clone-controller.go
@@ -191,7 +191,7 @@ func (r *PvcCloneReconciler) cleanup(syncState *dvSyncState) error {
 		return err
 	}
 
-	if dv.DeletionTimestamp == nil || dv.Status.Phase != cdiv1.Succeeded {
+	if dv.DeletionTimestamp == nil && dv.Status.Phase != cdiv1.Succeeded {
 		return nil
 	}