Abacn commented on code in PR #38753:
URL: https://github.com/apache/beam/pull/38753#discussion_r3329137298
##########
buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy:
##########
@@ -1218,6 +1218,17 @@ class BeamModulePlugin implements Plugin<Project> {
useJUnit {}
// default maxHeapSize on gradle 5 is 512m, lets increase to handle
more demanding tests
maxHeapSize = '2g'
+
+ // Develocity Gradle plugin (applied in settings.gradle.kts) provides
test retry
+ // natively. Configure it in CI to retry flaky integration tests.
+ def isCI = System.getenv("GITHUB_ACTIONS") != null ||
System.getenv("JENKINS_HOME") != null
+ if (isCI) {
+ develocity.testRetry {
Review Comment:
Retry just make test pass and make underlying cause of flakiness even less
visible. We don't do it here.
The real flaky tests e.g.
`testBundleFinalizationOccursOnBoundedSplittableDoFn`
`testBundleFinalizationOccursOnUnboundedSplittableDoFn` needs to be resolved
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
