You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by js...@apache.org on 2022/09/08 15:01:50 UTC

[kafka] branch 3.3 updated: MINOR; Retry on test failure for branch builds and increase max test retry to 10 (#12601)

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

jsancio pushed a commit to branch 3.3
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/3.3 by this push:
     new fa3834618c MINOR; Retry on test failure for branch builds and increase max test retry to 10 (#12601)
fa3834618c is described below

commit fa3834618cd88ddddb8df8b8f5771458e6678232
Author: Ismael Juma <is...@juma.me.uk>
AuthorDate: Thu Sep 8 08:00:16 2022 -0700

    MINOR; Retry on test failure for branch builds and increase max test retry to 10 (#12601)
    
    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.
    
    Reviewers: José Armando García Sancio <js...@users.noreply.github.com>
---
 Jenkinsfile | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/Jenkinsfile b/Jenkinsfile
index 7946f9a7fb..e3a7154c16 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'
 }