diff --git a/tests/cloner_test.go b/tests/cloner_test.go index f867cd0d38..3885ad209d 100644 --- a/tests/cloner_test.go +++ b/tests/cloner_test.go @@ -965,7 +965,7 @@ var _ = Describe("all clone tests", func() { targetDvs = nil }) - It("[rfe_id:1277][test_id:1899][crit:High][vendor:cnv-qe@redhat.com][level:component] Should allow multiple cloning operations in parallel", func() { + FIt("[rfe_id:1277][test_id:1899][crit:High][vendor:cnv-qe@redhat.com][level:component] Should allow multiple cloning operations in parallel", func() { const NumOfClones int = 3 By("Creating a source from a real image") @@ -999,7 +999,7 @@ var _ = Describe("all clone tests", func() { targetDvs = append(targetDvs, targetDv) } - podsNodeName := make(map[string]string) + podsNodeName := make(map[string]bool) for _, dv := range targetDvs { By("Waiting for clone to be completed") err = utils.WaitForDataVolumePhaseWithTimeout(f, f.Namespace.Name, cdiv1.Succeeded, dv.Name, 3*90*time.Second) @@ -1032,7 +1032,7 @@ var _ = Describe("all clone tests", func() { By(fmt.Sprintf("Getting pod %s/%s", dv.Namespace, cloneSourcePod)) pod, err := f.K8sClient.CoreV1().Pods(dv.Namespace).Get(context.TODO(), cloneSourcePod, metav1.GetOptions{}) Expect(err).ToNot(HaveOccurred()) - podsNodeName[dv.Name] = pod.Spec.NodeName + podsNodeName[pod.Spec.NodeName] = true } By("Deleting verifier pod") @@ -1042,14 +1042,13 @@ var _ = Describe("all clone tests", func() { Expect(err).ToNot(HaveOccurred()) } - // All pods should be in the same node + // All pods should be in the same node except when the map is empty in smart clone if len(podsNodeName) > 0 { - Expect(podsNodeName[targetDvs[0].Name]).To(Equal(podsNodeName[targetDvs[1].Name])) - Expect(podsNodeName[targetDvs[1].Name]).To(Equal(podsNodeName[targetDvs[2].Name])) + Expect(podsNodeName).To(HaveLen(1)) } }) - It("[rfe_id:1277][test_id:1899][crit:High][vendor:cnv-qe@redhat.com][level:component] Should allow multiple cloning operations in parallel for block devices", func() { + FIt("[rfe_id:1277][test_id:1899][crit:High][vendor:cnv-qe@redhat.com][level:component] Should allow multiple cloning operations in parallel for block devices", func() { const NumOfClones int = 3 if !f.IsBlockVolumeStorageClassAvailable() {