elek commented on issue #750: HDDS-3309. Add timeout to all integration tests URL: https://github.com/apache/hadoop-ozone/pull/750#issuecomment-608398011 Thanks the patch @smengcl This timeouts (and especially the root cause) the biggest problem with the integration tests right now... > This helps ruling out flaky long-running tests (e.g. TestRandomKeyGenerator.bigFileThan2GB) that are taking a very long time to run in Github actions. Did you see any long running integration tests? It seems that we have a global timeout: In the main `pom.xml`: ``` <surefire.fork.timeout>900</surefire.fork.timeout> ... <forkedProcessTimeoutInSeconds>${surefire.fork.timeout}</forkedProcessTimeoutInSeconds> ``` Isn't it easier to decrease this number? (If I understand well it does the same as we fork the JVM) Did you check the current execution time of all the integration tests? Is the proposed value is significant bigger than the expected time of the slowest test?
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
