[kafka] branch ijuma/retry-branch-builds created (now 5c2aea8e5d)

[ij...@apache.org]

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

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


      at 5c2aea8e5d MINOR: Retry on test failure for branch builds and increase max test retry to 10

This branch includes the following new commits:

     new 5c2aea8e5d MINOR: Retry on test failure for branch builds and increase max test retry to 10

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

[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'
 }