squakez commented on code in PR #4021: URL: https://github.com/apache/camel-k/pull/4021#discussion_r1109420702
########## e2e/support/test_support.go: ########## @@ -93,10 +93,10 @@ const kubeConfigEnvVar = "KUBECONFIG" var TestTimeoutShort = 1 * time.Minute var TestTimeoutMedium = 5 * time.Minute -var TestTimeoutLong = 10 * time.Minute +var TestTimeoutLong = 15 * time.Minute // TestTimeoutVeryLong should be used only for testing native builds. -var TestTimeoutVeryLong = 90 * time.Minute +var TestTimeoutVeryLong = 60 * time.Minute Review Comment: I was more concerned about `TestTimeoutLong` which is increased a little bit. I wonder if we could instead introduce a different timeout variables for native (ie, `NativeTestTimeoutLong`) or either just use the quantity of minutes. But we refine this in a following iteration, not a blocker for this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
