adoroszlai opened a new pull request, #1079: URL: https://github.com/apache/ratis/pull/1079
## What changes were proposed in this pull request? Ratis build sets Surefire fork timeout to 10 minutes: https://github.com/apache/ratis/blob/58671923e78a10d5c6a7d49a06e2d95690d5e9bb/pom.xml#L637 Yet, sometimes Surefire fork is not killed, check runs until Github workflow is cancelled: ``` Tue, 12 Mar 2024 19:43:11 GMT [INFO] Running org.apache.ratis.netty.TestRaftWithNetty Wed, 13 Mar 2024 01:40:25 GMT Error: The operation was canceled. ``` The problem is that: > Since 3.0.0-M4 the process checkers are disabled. ([doc](https://maven.apache.org/surefire/maven-surefire-plugin/test-mojo.html#enableProcessChecker)) Changes in this PR (and links to similar change in Ozone): 1. Enable process checkers to apply the fork timeout ([HDDS-10174](https://issues.apache.org/jira/browse/HDDS-10174)) 2. Downgrade to Surefire 3.0.0-M4 due to: [SUREFIRE-1722](https://issues.apache.org/jira/browse/SUREFIRE-1722) (affects M6 and newer), [SUREFIRE-1815](https://issues.apache.org/jira/browse/SUREFIRE-1815) (affects M5) ([HDDS-10522](https://issues.apache.org/jira/browse/HDDS-10522)) https://issues.apache.org/jira/browse/RATIS-2073 ## How was this patch tested? Created a small repro to test various Surefire versions, see https://github.com/apache/ozone/pull/6075 for details. Regular CI: https://github.com/adoroszlai/ratis/actions/runs/8972109715 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
