Skip to content

Reduce Gradle build and test log spam in Java CI workflows #40458

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Abacn merged 2 commits into from
Oct 8, 2026
Merged

Reduce Gradle build and test log spam in Java CI workflows #40458

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
7a78ac7
Reduce Gradle build and test log spam in Java CI workflows
Abacn Oct 7, 2026
3b6c9a9
Address comments
Abacn Oct 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 7 additions & 8 deletions buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1265,15 +1265,14 @@ class BeamModulePlugin implements Plugin<Project> {
if (System.getProperty("os.name").toLowerCase().contains("windows")) {
systemProperty 'org.xerial.snappy.tempdir', System.getProperty('org.xerial.snappy.tempdir') ?: "${project.rootDir.absolutePath}/build/snappy_bin"
}
// Report each test as it runs, plus a per-task summary. Gradle logs only
// failures by default, so a passing run prints nothing but "BUILD SUCCESSFUL"
// and gives no indication of which tests --tests actually selected.
// Test stdout/stderr stays off unless asked for, since it is noisy:
// -PshowTestOutput also forward test stdout/stderr
// A cached or up-to-date test task forks no JVM and prints nothing;
// add --no-build-cache --rerun to force real execution.
// Log skipped and failed tests, plus a per-task summary.
// Pass -PshowTestOutput to also forward test stdout/stderr.
// Pass -PshowTestPassed to also log passed tests.
testLogging {
events 'passed', 'skipped', 'failed'
Comment thread
Abacn marked this conversation as resolved.
events 'skipped', 'failed'
if (project.hasProperty('showTestPassed')) {
events 'passed'
}
exceptionFormat = 'full'
showExceptions = true
showCauses = true
Expand Down
2 changes: 1 addition & 1 deletion buildSrc/src/main/groovy/org/apache/beam/gradle/Repositories.groovy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ class Repositories {
def useMirror = isCi && mirrorUrl

if (useMirror) {
project.logger.lifecycle("Running in CI. Mirroring Maven Central repositories via Google Maven Mirror.")
project.logger.debug("Running in CI. Mirroring Maven Central repositories via Google Maven Mirror.")
}

project.repositories {
Expand Down
7 changes: 0 additions & 7 deletions runners/direct-java/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,6 @@ task needsRunnerTests(type: Test) {
description = "Runs tests that require a runner to validate that pipelines/transforms work correctly"

maxParallelForks 4
testLogging.showStandardStreams = true

String[] pipelineOptions = ["--runner=DirectRunner", "--runnerDeterminedSharding=false"]
systemProperty "beamTestPipelineOptions", pipelineOptionsStringCrossPlatformHandling(pipelineOptions)
Expand All @@ -148,10 +147,6 @@ task needsRunnerTests(type: Test) {
excludeCategories "org.apache.beam.sdk.testing.UsesExternalService"
excludeCategories 'org.apache.beam.sdk.testing.UsesBundleFinalizer'
}
testLogging {
outputs.upToDateWhen {false}
showStandardStreams = true
}
}

// NOTE: This will also run 'NeedsRunner' tests, which are run in the :needsRunnerTests task as well.
Expand Down Expand Up @@ -223,8 +218,6 @@ createJavaExamplesArchetypeValidationTask(type: 'MobileGaming',
task examplesIntegrationTest(type: Test) {
description = "Runs examples tests on DirectRunner"

testLogging.showStandardStreams = true

String[] pipelineOptions = [
"--runner=DirectRunner",
"--runnerDeterminedSharding=false",
Expand Down
6 changes: 0 additions & 6 deletions sdks/java/io/debezium/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -113,13 +113,7 @@ def configureTestJvmArgs(Task task) {
}

test {
testLogging {
outputs.upToDateWhen {false}
showStandardStreams = true
exceptionFormat = 'full'
}
configureTestJvmArgs(delegate)

}

task integrationTest(type: Test, dependsOn: processTestResources) {
Expand Down
2 changes: 1 addition & 1 deletion settings.gradle.kts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ pluginManagement {
val useMirror = isCi && !mavenCentralMirrorUrl.isNullOrBlank()

if (useMirror) {
logger.lifecycle("Running in CI. Mirroring Maven Central repositories via Google Maven Mirror.")
logger.debug("Running in CI. Mirroring Maven Central repositories via Google Maven Mirror.")
}

repositories {
Expand Down
Loading