valdar commented on code in PR #5415: URL: https://github.com/apache/camel-k/pull/5415#discussion_r1577779400
########## e2e/support/test_support.go: ########## @@ -128,9 +128,9 @@ const ExpectedOSClusterRoles = 1 var TestDefaultNamespace = "default" -var TestTimeoutShort = 1 * time.Minute -var TestTimeoutMedium = 5 * time.Minute -var TestTimeoutLong = 15 * time.Minute +var TestTimeoutShort = 5 * time.Minute Review Comment: According to my empirical tests these are the values that minimize test runs failing due to small perturbations in load or network speed. Honestly I don't see may cons in bumping them, the only one is that actually failing tests will take longer. But that is a condition that should happens rarely since most of the runs should be of passing tests. -- 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
