[kafka] 01/01: MINOR: Retry on test failure for branch builds and increase max test retry to 10

[ij...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

ijuma pushed a commit to branch ijuma/retry-branch-builds
in repository https://gitbox.apache.org/repos/asf/kafka.git

commit 5c2aea8e5d36f0cb244ae50afde4e55073c5160c
Author: Ismael Juma <is...@juma.me.uk>
AuthorDate: Wed Sep 7 09:53:03 2022 -0700

    MINOR: Retry on test failure for branch builds and increase max test retry to 10
    
    Originally, we only enabled retries for PR builds to avoid hiding timing
    related issues. In practice, however, the results are too noisy without
    any retry due to various environmental issues.
    
    Enable 1 retry for all builds and increase the max test retry to 10.
---
 Jenkinsfile | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/Jenkinsfile b/Jenkinsfile
index 272e67544a..aedcc08d15 100644
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -29,15 +29,10 @@ def isChangeRequest(env) {
   env.CHANGE_ID != null && !env.CHANGE_ID.isEmpty()
 }
 
-def retryFlagsString(env) {
-    if (isChangeRequest(env)) " -PmaxTestRetries=1 -PmaxTestRetryFailures=5"
-    else ""
-}
-
 def doTest(env, target = "unitTest integrationTest") {
   sh """./gradlew -PscalaVersion=$SCALA_VERSION ${target} \
       --profile --no-daemon --continue -PtestLoggingEvents=started,passed,skipped,failed \
-      -PignoreFailures=true -PmaxParallelForks=2""" + retryFlagsString(env)
+      -PignoreFailures=true -PmaxParallelForks=2 -PmaxTestRetries=1 -PmaxTestRetryFailures=10"""
   junit '**/build/test-results/**/TEST-*.xml'
 }