e2e: add no-kubernetes-in-docker label

This label means the e2e tests are not suitable to test in containerized k8s env (kind provider). In this commit, we label two tests because the large size images would lead to OOM of cdi-uploader. For more detail, please refer to: kubevirt#3577 (comment) Signed-off-by: howard zhang <[email protected]>
zhlhahaha · Jan 8, 2025 · c5c84af · c5c84af
1 parent 573ce9c
commit c5c84af
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/tests/import_test.go b/tests/import_test.go
@@ -57,7 +57,7 @@ var _ = Describe("[rfe_id:1115][crit:high][vendor:[email protected]][level:compo
 		ns = f.Namespace.Name
 	})
 
-	DescribeTable("[test_id:2329] Should fail to import images that require too much space", func(uploadURL string) {
+	DescribeTable("[test_id:2329] Should fail to import images that require too much space", Label("no-kubernetes-in-docker"), func(uploadURL string) {
 		imageURL := fmt.Sprintf(uploadURL, f.CdiInstallNs)
 
 		By(imageURL)

diff --git a/tests/upload_test.go b/tests/upload_test.go
@@ -307,7 +307,7 @@ var _ = Describe("[rfe_id:138][crit:high][vendor:[email protected]][level:compon
 			Entry("fail given a large physical size QCOW2 file", utils.UploadFileLargePhysicalDiskQcow),
 		)
 
-		DescribeTable("[posneg:negative][test_id:2330]Verify failure on sync upload if virtual size > pvc size", Serial, func(filename string) {
+		DescribeTable("[posneg:negative][test_id:2330]Verify failure on sync upload if virtual size > pvc size", Label("no-kubernetes-in-docker"), Serial, func(filename string) {
 			By("Verify PVC annotation says ready")
 			found, err := utils.WaitPVCPodStatusReady(f.K8sClient, pvc)
 			Expect(err).ToNot(HaveOccurred())