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