Skip to content

Commit

Permalink
Gradle check retry (opensearch-project#2638)
Browse files Browse the repository at this point in the history
* Add retry plugin support for Test implementations

Signed-off-by: Kunal Kotwani <[email protected]>

* Update test retry parameters

Signed-off-by: Kunal Kotwani <[email protected]>

* Remove CI environment check for test retries

Signed-off-by: Kunal Kotwani <[email protected]>

* Update retry count for tests

Signed-off-by: Kunal Kotwani <[email protected]>
  • Loading branch information
kotwanikunal authored Mar 30, 2022
1 parent 8ea246e commit 65cc56e
Showing 1 changed file with 17 additions and 4 deletions.
21 changes: 17 additions & 4 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ plugins {
id 'opensearch.docker-support'
id 'opensearch.global-build-info'
id "com.diffplug.spotless" version "6.3.0" apply false
id "org.gradle.test-retry" version "1.3.1" apply false
}

apply from: 'gradle/build-complete.gradle'
Expand Down Expand Up @@ -232,7 +233,7 @@ allprojects {
tasks.withType(JavaCompile).configureEach { JavaCompile compile ->
// See please https://bugs.openjdk.java.net/browse/JDK-8209058
if (BuildParams.runtimeJavaVersion > JavaVersion.VERSION_11) {
compile.options.compilerArgs << '-Werror'
compile.options.compilerArgs << '-Werror'
}
compile.options.compilerArgs << '-Xlint:auxiliaryclass'
compile.options.compilerArgs << '-Xlint:cast'
Expand Down Expand Up @@ -386,6 +387,18 @@ gradle.projectsEvaluated {
}
}

// test retry configuration
subprojects {
apply plugin: "org.gradle.test-retry"
tasks.withType(Test).configureEach {
retry {
failOnPassedAfterRetry = false
maxRetries = 3
maxFailures = 10
}
}
}

// eclipse configuration
allprojects {
apply plugin: 'eclipse'
Expand Down Expand Up @@ -445,9 +458,9 @@ allprojects {
tasks.named('eclipse') { dependsOn 'cleanEclipse', 'copyEclipseSettings' }

afterEvaluate {
tasks.findByName("eclipseJdt")?.configure {
dependsOn 'copyEclipseSettings'
}
tasks.findByName("eclipseJdt")?.configure {
dependsOn 'copyEclipseSettings'
}
}
}

Expand Down

0 comments on commit 65cc56e

Please sign in to comment.