extend pull image response timeout to 10min (openkruise#1288)

Co-authored-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com>
ppbits · May 18, 2023 · 3a80fd2 · 3a80fd2
1 parent a975d12
commit 3a80fd2
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/pkg/controller/nodeimage/nodeimage_controller.go b/pkg/controller/nodeimage/nodeimage_controller.go
@@ -69,7 +69,7 @@ var (
 const (
 	controllerName     = "nodeimage-controller"
 	minRequeueDuration = 3 * time.Second
-	responseTimeout    = time.Minute
+	responseTimeout    = 10 * time.Minute
 
 	// Allow fake NodeImage with no Node related, just for tests
 	fakeLabelKey = "apps.kruise.io/fake-nodeimage"

diff --git a/test/e2e/apps/pullimages.go b/test/e2e/apps/pullimages.go
@@ -221,13 +221,13 @@ var _ = SIGDescribe("PullImage", func() {
 			}, 120*time.Second, 3*time.Second).Should(gomega.Equal(int32(len(nodes))))
 			gomega.Expect(job.Status.CompletionTime == nil).To(gomega.Equal(true))
 
-			ginkgo.By("Wait 1 failed in 80s")
-			gomega.Eventually(func() int32 {
-				job, err = testerForImagePullJob.GetJob(job)
-				gomega.Expect(err).NotTo(gomega.HaveOccurred())
-				return job.Status.Failed
-			}, 80*time.Second, 3*time.Second).Should(gomega.Equal(int32(1)))
-			gomega.Expect(len(job.Status.FailedNodes)).To(gomega.Equal(1))
+			//ginkgo.By("Wait 1 failed in 80s")
+			//gomega.Eventually(func() int32 {
+			//	job, err = testerForImagePullJob.GetJob(job)
+			//	gomega.Expect(err).NotTo(gomega.HaveOccurred())
+			//	return job.Status.Failed
+			//}, 80*time.Second, 3*time.Second).Should(gomega.Equal(int32(1)))
+			//gomega.Expect(len(job.Status.FailedNodes)).To(gomega.Equal(1))
 		})
 
 		framework.ConformanceIt("create two jobs to pull a same image", func() {