Merge pull request #1777 from swetharepakula/extend-test-timeout

Increase timeouts to reflect GCE API
kubernetes · Aug 13, 2022 · 09c54ff · 09c54ff
2 parents 9a7a6aa + e141aeb
commit 09c54ff
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/cmd/e2e-test/draining_test.go b/cmd/e2e-test/draining_test.go
@@ -35,7 +35,7 @@ import (
 )
 
 const (
-	drainingTransitionPollTimeout = 10 * time.Minute
+	drainingTransitionPollTimeout = 15 * time.Minute
 	drainingTansitionPollInterval = 30 * time.Second
 )
 

diff --git a/pkg/e2e/helpers.go b/pkg/e2e/helpers.go
@@ -71,7 +71,7 @@ const (
 
 	negPollInterval  = 5 * time.Second
 	negPollTimeout   = 3 * time.Minute
-	negGCPollTimeout = 5 * time.Minute
+	negGCPollTimeout = 6 * time.Minute
 
 	k8sApiPoolInterval = 10 * time.Second
 	k8sApiPollTimeout  = 30 * time.Minute
@@ -85,6 +85,9 @@ const (
 
 	redirectURLMapPollTimeout = 10 * time.Minute
 
+	// timeout for ILB creation + SA creation
+	saPollTimeout = 10 * time.Minute
+
 	healthyState = "HEALTHY"
 )
 
@@ -962,7 +965,7 @@ func DeleteNegCR(s *Sandbox, negName string) error {
 // created and properly configured
 func WaitForServiceAttachment(s *Sandbox, saName string) (string, error) {
 	var gceSAURL string
-	err := wait.Poll(negPollInterval, negPollTimeout, func() (bool, error) {
+	err := wait.Poll(negPollInterval, saPollTimeout, func() (bool, error) {
 		saCR, err := s.f.SAClient.Get(s.Namespace, saName)
 		if saCR == nil || err != nil {
 			return false, fmt.Errorf("failed to get service attachment %s/%s: %v", s.Namespace, saName, err)
-Original file line number
+Diff line change
@@ Expand Up / @@ -35,7 +35,7 @@ import ( @@
     )
     const (
-    	drainingTransitionPollTimeout = 10 * time.Minute
+    	drainingTransitionPollTimeout = 15 * time.Minute
     	drainingTansitionPollInterval = 30 * time.Second
     )
@@ Expand Down @@