From d72770d5823dfc15c7ea08799caaf3fc2a45239c Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Tue, 24 Mar 2026 15:32:58 -0400 Subject: [PATCH 01/13] ci: Add fast profile that skips all maven lifecycle plugins --- .github/workflows/ci.yaml | 12 ++--- .github/workflows/sdk-platform-java-ci.yaml | 53 ++++++++++----------- google-cloud-pom-parent/pom.xml | 15 ++++++ sdk-platform-java/pom.xml | 15 ++++++ 4 files changed, 58 insertions(+), 37 deletions(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 2a310a374c6b..9a8bb73bd52d 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -67,7 +67,7 @@ jobs: - name: Install all modules using Java 11 shell: bash run: | - mvn -V -B -ntp clean install -DskipTests -Dfmt.skip -Dclirr.skip -Denforcer.skip -Dcheckstyle.skip -Dflatten.skip -Danimal-sniffer.skip -Djacoco.skip + mvn -V -B -ntp clean install -Pfast -DskipTests working-directory: sdk-platform-java - uses: actions/setup-java@v4 with: @@ -78,13 +78,9 @@ jobs: run: | mvn test \ -B -ntp \ + -Pfast \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ - -Dclirr.skip=true \ - -Denforcer.skip=true \ - -Dcheckstyle.skip=true \ - -Dflatten.skip=true \ - -Danimal.sniffer.skip=true \ -Dmaven.wagon.http.retryHandler.count=5 \ -T 1C env: @@ -138,7 +134,7 @@ jobs: - run: java -version - name: Install Maven modules to local Maven repository run: | - mvn install -B -ntp -T 1C -DskipTests -Dclirr.skip -Dcheckstyle.skip + mvn install -B -ntp -T 1C -Pfast -DskipTests - run: java -version - run: mvn -B -ntp enforcer:enforce@enforce -T 1C gapic-libraries-bom: @@ -152,7 +148,7 @@ jobs: cache: maven - name: Install Maven modules to local Maven repository run: | - mvn install -B -ntp -T 1C -DskipTests -Dclirr.skip -Dcheckstyle.skip -Denforcer.skip + mvn install -B -ntp -T 1C -Pfast -DskipTests - name: Validate gapic-libraries-bom uses: googleapis/java-cloud-bom/tests/validate-bom@v26.54.0 with: diff --git a/.github/workflows/sdk-platform-java-ci.yaml b/.github/workflows/sdk-platform-java-ci.yaml index c0e9904ea4f7..da50f12c51f1 100644 --- a/.github/workflows/sdk-platform-java-ci.yaml +++ b/.github/workflows/sdk-platform-java-ci.yaml @@ -34,14 +34,14 @@ jobs: - run: java -version - name: Unit Tests run: | - mvn test --batch-mode --no-transfer-progress -Dcheckstyle.skip \ - -Dfmt.skip -DenableTestCoverage + mvn test --batch-mode --no-transfer-progress -Pfast \ + -DenableTestCoverage working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable - name: Env Var Tests run: | - mvn test --batch-mode --no-transfer-progress -Dcheckstyle.skip \ - -Dfmt.skip -DenableTestCoverage -PenvVarTest + mvn test --batch-mode --no-transfer-progress -Pfast \ + -DenableTestCoverage -PenvVarTest # Set the Env Var for this step only env: GOOGLE_CLOUD_UNIVERSE_DOMAIN: random.com @@ -51,7 +51,7 @@ jobs: - run: bazelisk version - name: Install Maven modules run: | - mvn install -B -ntp -DskipTests -Dclirr.skip -Dcheckstyle.skip -Dfmt.skip + mvn install -B -ntp -Pfast -DskipTests working-directory: sdk-platform-java - name: Integration Tests run: | @@ -90,14 +90,14 @@ jobs: - run: java -version - name: Unit Tests run: | - mvn test --batch-mode --no-transfer-progress -Dcheckstyle.skip \ - -Dfmt.skip -DenableTestCoverage + mvn test --batch-mode --no-transfer-progress -Pfast \ + -DenableTestCoverage working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable - name: Env Var Tests run: | - mvn test --batch-mode --no-transfer-progress -Dcheckstyle.skip \ - -Dfmt.skip -DenableTestCoverage -PenvVarTest + mvn test --batch-mode --no-transfer-progress -Pfast \ + -DenableTestCoverage -PenvVarTest # Set the Env Var for this step only env: GOOGLE_CLOUD_UNIVERSE_DOMAIN: random.com @@ -107,7 +107,7 @@ jobs: - run: bazelisk version - name: Install Maven modules run: | - mvn install -B -ntp -DskipTests -Dclirr.skip -Dcheckstyle.skip + mvn install -B -ntp -Pfast -DskipTests working-directory: sdk-platform-java - name: Integration Tests run: | @@ -128,14 +128,14 @@ jobs: - run: java -version - name: Unit Tests run: | - mvn test --batch-mode --no-transfer-progress -Dcheckstyle.skip \ - -Dfmt.skip -DenableTestCoverage + mvn test --batch-mode --no-transfer-progress -Pfast \ + -DenableTestCoverage working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable - name: Env Var Tests run: | - mvn test --batch-mode --no-transfer-progress -Dcheckstyle.skip \ - -Dfmt.skip -DenableTestCoverage -PenvVarTest + mvn test --batch-mode --no-transfer-progress -Pfast \ + -DenableTestCoverage -PenvVarTest # Set the Env Var for this step only env: GOOGLE_CLOUD_UNIVERSE_DOMAIN: random.com @@ -145,7 +145,7 @@ jobs: - run: bazelisk version - name: Install Maven modules run: | - mvn install -B -ntp -DskipTests -Dclirr.skip -Dcheckstyle.skip -Dfmt.skip + mvn install -B -ntp -Pfast -DskipTests working-directory: sdk-platform-java - name: Integration Tests # note need to set shouldInstallTestSecurityManager=false due to https://github.com/bazelbuild/bazel/issues/24354 @@ -177,8 +177,7 @@ jobs: export PATH=${JAVA_HOME}/bin:$PATH # Maven surefire plugin lets us to specify the JVM when running tests via # the "jvm" system property. - mvn verify --batch-mode --no-transfer-progress -Dcheckstyle.skip \ - -Dfmt.skip \ + mvn verify --batch-mode --no-transfer-progress -Pfast \ -Djvm="${JAVA8_HOME}/bin/java" working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable @@ -191,8 +190,8 @@ jobs: # Maven surefire plugin lets us to specify the JVM when running tests via # the "jvm" system property. export GOOGLE_CLOUD_UNIVERSE_DOMAIN=random.com - mvn test --batch-mode --no-transfer-progress -Dcheckstyle.skip \ - -Dfmt.skip -DenableTestCoverage -Dsurefire.failIfNoSpecifiedTests=false \ + mvn test --batch-mode --no-transfer-progress -Pfast \ + -DenableTestCoverage -Dsurefire.failIfNoSpecifiedTests=false \ -PenvVarTest # Set the Env Var for this step only env: @@ -216,7 +215,7 @@ jobs: - name: Install all modules using Java 11 shell: bash run: | - mvn -V -B -ntp clean install -DskipTests -Dfmt.skip -Dcheckstyle.skip + mvn -V -B -ntp clean install -Pfast -DskipTests working-directory: sdk-platform-java - uses: actions/setup-java@v3 with: @@ -266,7 +265,7 @@ jobs: - name: Install all modules using Java 11 shell: bash run: | - mvn install -B -ntp -DskipTests -Dclirr.skip -Dcheckstyle.skip -Dfmt.skip + mvn install -B -ntp -Pfast -DskipTests working-directory: sdk-platform-java - uses: actions/setup-java@v3 with: @@ -340,7 +339,7 @@ jobs: - run: mvn -version - name: Install Maven modules run: | - mvn install -B -ntp -DskipTests -Dclirr.skip -Dcheckstyle.skip -Dfmt.skip + mvn install -B -ntp -Pfast -DskipTests working-directory: sdk-platform-java - name: Showcase golden tests working-directory: sdk-platform-java/java-showcase @@ -419,7 +418,7 @@ jobs: cache: maven - name: Install Maven modules run: | - mvn install -B -ntp -DskipTests -Dclirr.skip -Dcheckstyle.skip -Dfmt.skip + mvn install -B -ntp -Pfast -DskipTests working-directory: sdk-platform-java - name: Install showcase to local maven repository run: | @@ -454,14 +453,10 @@ jobs: run: | mvn install \ -B -ntp \ + -Pfast \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ -DskipTests \ - -Dclirr.skip=true \ - -Denforcer.skip=true \ - -Dcheckstyle.skip=true \ - -Dflatten.skip=true \ - -Danimal.sniffer.skip=true \ -Dmaven.wagon.http.retryHandler.count=5 \ -T 1C working-directory: sdk-platform-java @@ -489,7 +484,7 @@ jobs: working-directory: sdk-platform-java/java-shared-dependencies/unmanaged-dependency-check - name: Install Maven modules run: | - mvn install -B -ntp -DskipTests -Dclirr.skip -Dcheckstyle.skip -Dfmt.skip + mvn install -B -ntp -Pfast -DskipTests working-directory: sdk-platform-java - name: Unmanaged dependency check uses: ./sdk-platform-java/java-shared-dependencies/unmanaged-dependency-check diff --git a/google-cloud-pom-parent/pom.xml b/google-cloud-pom-parent/pom.xml index f536829ce561..90e8a67f8273 100644 --- a/google-cloud-pom-parent/pom.xml +++ b/google-cloud-pom-parent/pom.xml @@ -58,6 +58,21 @@ + + fast + + true + true + true + true + true + true + true + true + true + true + + checkstyle-tests diff --git a/sdk-platform-java/pom.xml b/sdk-platform-java/pom.xml index f6ac656163c0..bb050c68937a 100644 --- a/sdk-platform-java/pom.xml +++ b/sdk-platform-java/pom.xml @@ -50,6 +50,21 @@ + + fast + + true + true + true + true + true + true + true + true + true + true + + release-staging-repository From 74f40e5d336575e64400a2f05fa0d6cfb8a58c85 Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Tue, 24 Mar 2026 15:44:01 -0400 Subject: [PATCH 02/13] chore: Add checkstyle skip flag --- google-cloud-pom-parent/pom.xml | 1 + sdk-platform-java/gapic-generator-java-pom-parent/pom.xml | 1 + 2 files changed, 2 insertions(+) diff --git a/google-cloud-pom-parent/pom.xml b/google-cloud-pom-parent/pom.xml index 90e8a67f8273..fc85b487a805 100644 --- a/google-cloud-pom-parent/pom.xml +++ b/google-cloud-pom-parent/pom.xml @@ -93,6 +93,7 @@ ${checkstyle.header.file} + ${checkstyle.skip} diff --git a/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml b/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml index 8d0be132a5b5..0dc7d948d7fe 100644 --- a/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml +++ b/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml @@ -97,6 +97,7 @@ ${checkstyle.header.file} + ${checkstyle.skip} From fed45165bb45a524ec37652dc5c04a65db900da9 Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Tue, 24 Mar 2026 15:49:46 -0400 Subject: [PATCH 03/13] chore: Update kokoro CI to use fast profile --- .kokoro/build.sh | 6 +----- .kokoro/common.sh | 31 +++++-------------------------- .kokoro/dependencies.sh | 6 ++---- 3 files changed, 8 insertions(+), 35 deletions(-) diff --git a/.kokoro/build.sh b/.kokoro/build.sh index c20f1633a357..6af04fe3eea1 100755 --- a/.kokoro/build.sh +++ b/.kokoro/build.sh @@ -50,13 +50,9 @@ case ${JOB_TYPE} in retry_with_backoff 3 10 \ mvn install \ -B -ntp \ + -Pfast \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ - -Dclirr.skip=true \ - -Denforcer.skip=true \ - -Dcheckstyle.skip=true \ - -Dflatten.skip=true \ - -Danimal.sniffer.skip=true \ -Dmaven.wagon.http.retryHandler.count=5 \ --also-make \ ${SUREFIRE_JVM_OPT} diff --git a/.kokoro/common.sh b/.kokoro/common.sh index 24648d119932..5daad7413dbf 100644 --- a/.kokoro/common.sh +++ b/.kokoro/common.sh @@ -297,19 +297,13 @@ function run_integration_tests() { parse_all_submodules "$1" printf "Running integration tests for submodules:\n%s\n" "$all_submodules" - mvn verify -Penable-integration-tests --projects "$all_submodules" \ + mvn verify -Penable-integration-tests -Pfast --projects "$all_submodules" \ ${INTEGRATION_TEST_ARGS} \ -B -ntp -fae \ --also-make \ -DtrimStackTrace=false \ - -Dclirr.skip=true \ - -Denforcer.skip=true \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ - -Dcheckstyle.skip=true \ - -Dflatten.skip=true \ - -Danimal.sniffer.skip=true \ - -Djacoco.skip=true \ -DskipUnitTests=true \ -Dmaven.wagon.http.retryHandler.count=5 \ -T 1C @@ -323,17 +317,12 @@ function run_graalvm_tests() { parse_all_submodules "$1" printf "Running GraalVM ITs for submodules:\n%s\n" "$all_submodules" - mvn test -Pnative --projects "$all_submodules" \ + mvn test -Pnative -Pfast --projects "$all_submodules" \ ${INTEGRATION_TEST_ARGS} \ -B -ntp -fae \ -DtrimStackTrace=false \ - -Dclirr.skip=true \ - -Denforcer.skip=true \ -Dorg.slf4j.simpleLogger.showDateTime=true \ - -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ - -Dcheckstyle.skip=true \ - -Dflatten.skip=true \ - -Danimal.sniffer.skip=true + -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS RETURN_CODE=$? printf "Finished GraalVM ITs for modules:\n%s\n" "$1" @@ -398,16 +387,11 @@ function install_modules() { if [ -z "$1" ]; then mvn install \ -B -ntp \ + -Pfast \ -DtrimStackTrace=false \ - -Dclirr.skip=true \ - -Denforcer.skip=true \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ - -Dcheckstyle.skip=true \ - -Dflatten.skip=true \ - -Danimal.sniffer.skip=true \ -DskipTests=true \ - -Djacoco.skip=true \ -T 1C else printf "Installing modules:\n%s\n" "$1" @@ -451,16 +435,11 @@ function install_modules() { # Correctly builds dependencies without building dependents. mvn install --projects "$all_submodules,$always_install_deps" --also-make \ -B -ntp \ + -Pfast \ -DtrimStackTrace=false \ - -Dclirr.skip=true \ - -Denforcer.skip=true \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ - -Dcheckstyle.skip=true \ - -Dflatten.skip=true \ - -Danimal.sniffer.skip=true \ -DskipTests=true \ - -Djacoco.skip=true \ -T 1C fi } diff --git a/.kokoro/dependencies.sh b/.kokoro/dependencies.sh index b273094ff7d3..c7652805254d 100755 --- a/.kokoro/dependencies.sh +++ b/.kokoro/dependencies.sh @@ -60,11 +60,9 @@ fi # this should run maven enforcer retry_with_backoff 3 10 \ mvn install -B -V -ntp \ - -DskipTests=true \ - -Dmaven.javadoc.skip=true \ - -Dclirr.skip=true + -Pfast -DskipTests=true -Dmaven.javadoc.skip=true -mvn -B dependency:analyze -DfailOnWarning=true -Dcheckstyle.skip=true +mvn -B dependency:analyze -Pfast -DfailOnWarning=true if [[ -n "${BUILD_SUBDIR}" ]] then From 27f265ee52ec04c49963d5de99c3369e5dac3d91 Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Tue, 24 Mar 2026 15:55:45 -0400 Subject: [PATCH 04/13] chore: Add to gapic-generator-java-pom-parent --- .../gapic-generator-java-pom-parent/pom.xml | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml b/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml index 0dc7d948d7fe..cfee8876f551 100644 --- a/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml +++ b/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml @@ -77,6 +77,22 @@ + + fast + + true + true + true + true + true + true + true + true + true + true + + + checkstyle-tests From ba3c72fba8e93b5a44a0814cc58535c5b483b803 Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Tue, 24 Mar 2026 15:57:26 -0400 Subject: [PATCH 05/13] chore: Remove the redundant config in the aggregator pom --- sdk-platform-java/pom.xml | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/sdk-platform-java/pom.xml b/sdk-platform-java/pom.xml index bb050c68937a..f6ac656163c0 100644 --- a/sdk-platform-java/pom.xml +++ b/sdk-platform-java/pom.xml @@ -50,21 +50,6 @@ - - fast - - true - true - true - true - true - true - true - true - true - true - - release-staging-repository From bf7e2dff716a6c39fb512435fd8510b3d35d5981 Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Tue, 24 Mar 2026 16:38:59 -0400 Subject: [PATCH 06/13] chore: Address PR feedback --- .kokoro/dependencies.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.kokoro/dependencies.sh b/.kokoro/dependencies.sh index c7652805254d..7ee4915af22c 100755 --- a/.kokoro/dependencies.sh +++ b/.kokoro/dependencies.sh @@ -60,9 +60,9 @@ fi # this should run maven enforcer retry_with_backoff 3 10 \ mvn install -B -V -ntp \ - -Pfast -DskipTests=true -Dmaven.javadoc.skip=true + -Pfast -DskipTests=true -Dmaven.javadoc.skip=true -Denforcer.skip=false -mvn -B dependency:analyze -Pfast -DfailOnWarning=true +mvn -B dependency:analyze -Pfast -DfailOnWarning=true -Dmdep.analyze.skip=false if [[ -n "${BUILD_SUBDIR}" ]] then From 5cf488f8ed09fb87eba7c8e5e7159ef4e00770b7 Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Tue, 24 Mar 2026 21:51:40 -0400 Subject: [PATCH 07/13] Resolve merge conflicts and rename fast profile to quickBuild --- .github/workflows/ci.yaml | 4 +- .github/workflows/sdk-platform-java-ci.yaml | 38 ++++++++++--------- .kokoro/build.sh | 25 +++--------- .kokoro/common.sh | 12 +++++- .kokoro/dependencies.sh | 4 +- google-cloud-pom-parent/pom.xml | 2 +- .../gapic-generator-java-pom-parent/pom.xml | 2 +- 7 files changed, 42 insertions(+), 45 deletions(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 3b718218feb2..f01bef2198d9 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -216,7 +216,7 @@ jobs: - run: java -version - name: Install Maven modules to local Maven repository run: | - mvn install -B -ntp -T 1C -Pfast -DskipTests +mvn install -B -ntp -T 1C -PquickBuild -DskipTests - run: java -version - run: mvn -B -ntp enforcer:enforce@enforce -T 1C gapic-libraries-bom: @@ -230,7 +230,7 @@ jobs: cache: maven - name: Install Maven modules to local Maven repository run: | - mvn install -B -ntp -T 1C -Pfast -DskipTests +mvn install -B -ntp -T 1C -PquickBuild -DskipTests - name: Validate gapic-libraries-bom uses: googleapis/java-cloud-bom/tests/validate-bom@v26.54.0 with: diff --git a/.github/workflows/sdk-platform-java-ci.yaml b/.github/workflows/sdk-platform-java-ci.yaml index 5f6fe8d9aaa0..08c90c8f9b25 100644 --- a/.github/workflows/sdk-platform-java-ci.yaml +++ b/.github/workflows/sdk-platform-java-ci.yaml @@ -34,13 +34,13 @@ jobs: - run: java -version - name: Unit Tests run: | - mvn test --batch-mode --no-transfer-progress -Pfast \ +mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable - name: Env Var Tests run: | - mvn test --batch-mode --no-transfer-progress -Pfast \ +mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage -PenvVarTest # Set the Env Var for this step only env: @@ -51,7 +51,7 @@ jobs: - run: bazelisk version - name: Install Maven modules run: | - mvn install -B -ntp -Pfast -DskipTests +mvn install -B -ntp -PquickBuild -DskipTests working-directory: sdk-platform-java - name: Integration Tests run: | @@ -90,13 +90,13 @@ jobs: - run: java -version - name: Unit Tests run: | - mvn test --batch-mode --no-transfer-progress -Pfast \ +mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable - name: Env Var Tests run: | - mvn test --batch-mode --no-transfer-progress -Pfast \ +mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage -PenvVarTest # Set the Env Var for this step only env: @@ -107,7 +107,11 @@ jobs: - run: bazelisk version - name: Install Maven modules run: | +<<<<<<< Updated upstream mvn install -B -ntp -Pfast -DskipTests +======= + mvn install -B -ntp -PquickBuild -DskipTests +>>>>>>> Stashed changes working-directory: sdk-platform-java - name: Integration Tests run: | @@ -128,13 +132,13 @@ jobs: - run: java -version - name: Unit Tests run: | - mvn test --batch-mode --no-transfer-progress -Pfast \ +mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable - name: Env Var Tests run: | - mvn test --batch-mode --no-transfer-progress -Pfast \ +mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage -PenvVarTest # Set the Env Var for this step only env: @@ -145,7 +149,7 @@ jobs: - run: bazelisk version - name: Install Maven modules run: | - mvn install -B -ntp -Pfast -DskipTests +mvn install -B -ntp -PquickBuild -DskipTests working-directory: sdk-platform-java - name: Integration Tests # note need to set shouldInstallTestSecurityManager=false due to https://github.com/bazelbuild/bazel/issues/24354 @@ -177,7 +181,7 @@ jobs: export PATH=${JAVA_HOME}/bin:$PATH # Maven surefire plugin lets us to specify the JVM when running tests via # the "jvm" system property. - mvn verify --batch-mode --no-transfer-progress -Pfast \ + mvn verify --batch-mode --no-transfer-progress -PquickBuild \ -Djvm="${JAVA8_HOME}/bin/java" working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable @@ -190,7 +194,7 @@ jobs: # Maven surefire plugin lets us to specify the JVM when running tests via # the "jvm" system property. export GOOGLE_CLOUD_UNIVERSE_DOMAIN=random.com - mvn test --batch-mode --no-transfer-progress -Pfast \ +mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage -Dsurefire.failIfNoSpecifiedTests=false \ -PenvVarTest # Set the Env Var for this step only @@ -215,7 +219,7 @@ jobs: - name: Install all modules using Java 11 shell: bash run: | - mvn -V -B -ntp clean install -Pfast -DskipTests + mvn -V -B -ntp clean install -PquickBuild -DskipTests working-directory: sdk-platform-java - uses: actions/setup-java@v3 with: @@ -265,7 +269,7 @@ jobs: - name: Install all modules using Java 11 shell: bash run: | - mvn install -B -ntp -Pfast -DskipTests +mvn install -B -ntp -PquickBuild -DskipTests working-directory: sdk-platform-java - uses: actions/setup-java@v3 with: @@ -339,7 +343,7 @@ jobs: - run: mvn -version - name: Install Maven modules run: | - mvn install -B -ntp -Pfast -DskipTests +mvn install -B -ntp -PquickBuild -DskipTests working-directory: sdk-platform-java - name: Showcase golden tests working-directory: sdk-platform-java/java-showcase @@ -418,7 +422,7 @@ jobs: cache: maven - name: Install Maven modules run: | - mvn install -B -ntp -Pfast -DskipTests +mvn install -B -ntp -PquickBuild -DskipTests working-directory: sdk-platform-java - name: Install showcase to local maven repository run: | @@ -456,13 +460,13 @@ jobs: run: | mvn install \ -B -ntp \ - -Pfast \ + -PquickBuild \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ -DskipTests \ -Dmaven.wagon.http.retryHandler.count=5 \ -T 1C - working-directory: sdk-platform-java + working-directory: sdk-platform-java - name: Validate gapic-generator-java-bom uses: googleapis/java-cloud-bom/tests/validate-bom@47ad868794c5d38ee7eb95d69371a7a79f84d890 with: @@ -487,7 +491,7 @@ jobs: working-directory: sdk-platform-java/java-shared-dependencies/unmanaged-dependency-check - name: Install Maven modules run: | - mvn install -B -ntp -Pfast -DskipTests + mvn install -B -ntp -PquickBuild -DskipTests working-directory: sdk-platform-java - name: Unmanaged dependency check uses: ./sdk-platform-java/java-shared-dependencies/unmanaged-dependency-check diff --git a/.kokoro/build.sh b/.kokoro/build.sh index 4dd789d51459..c50b917987b5 100755 --- a/.kokoro/build.sh +++ b/.kokoro/build.sh @@ -52,7 +52,7 @@ case ${JOB_TYPE} in retry_with_backoff 3 10 \ mvn install \ -B -ntp \ - -Pfast \ + -PquickBuild \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ -Dmaven.wagon.http.retryHandler.count=5 \ @@ -76,13 +76,9 @@ case ${JOB_TYPE} in install_modules "sdk-platform-java" mvn install \ -B -ntp \ + -PquickBuild \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ - -Dclirr.skip=true \ - -Denforcer.skip=true \ - -Dcheckstyle.skip=true \ - -Dflatten.skip=true \ - -Danimal.sniffer.skip=true \ -Dmaven.wagon.http.retryHandler.count=5 \ -DskipTests=true \ --also-make \ @@ -139,19 +135,13 @@ case ${JOB_TYPE} in echo "SUREFIRE_JVM_OPT: ${SUREFIRE_JVM_OPT}" echo "INTEGRATION_TEST_ARGS: ${INTEGRATION_TEST_ARGS}" - mvn verify -Penable-integration-tests \ + mvn verify -Penable-integration-tests -PquickBuild \ --also-make \ ${INTEGRATION_TEST_ARGS} \ -B -ntp -fae \ -DtrimStackTrace=false \ - -Dclirr.skip=true \ - -Denforcer.skip=true \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ - -Dcheckstyle.skip=true \ - -Dflatten.skip=true \ - -Danimal.sniffer.skip=true \ - -Djacoco.skip=true \ -DskipUnitTests=true \ -Dmaven.wagon.http.retryHandler.count=5 \ ${SUREFIRE_JVM_OPT} @@ -198,18 +188,13 @@ case ${JOB_TYPE} in echo "Running in subdir: ${BUILD_SUBDIR}" pushd "${BUILD_SUBDIR}" echo "INTEGRATION_TEST_ARGS: ${INTEGRATION_TEST_ARGS}" - mvn test -Pnative \ + mvn test -Pnative -PquickBuild \ --also-make \ ${INTEGRATION_TEST_ARGS} \ -B -ntp -fae \ -DtrimStackTrace=false \ - -Dclirr.skip=true \ - -Denforcer.skip=true \ -Dorg.slf4j.simpleLogger.showDateTime=true \ - -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ - -Dcheckstyle.skip=true \ - -Dflatten.skip=true \ - -Danimal.sniffer.skip=true + -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS RETURN_CODE=$? popd diff --git a/.kokoro/common.sh b/.kokoro/common.sh index 5daad7413dbf..48927bb2581c 100644 --- a/.kokoro/common.sh +++ b/.kokoro/common.sh @@ -297,7 +297,7 @@ function run_integration_tests() { parse_all_submodules "$1" printf "Running integration tests for submodules:\n%s\n" "$all_submodules" - mvn verify -Penable-integration-tests -Pfast --projects "$all_submodules" \ + mvn verify -Penable-integration-tests -PquickBuild --projects "$all_submodules" \ ${INTEGRATION_TEST_ARGS} \ -B -ntp -fae \ --also-make \ @@ -317,7 +317,7 @@ function run_graalvm_tests() { parse_all_submodules "$1" printf "Running GraalVM ITs for submodules:\n%s\n" "$all_submodules" - mvn test -Pnative -Pfast --projects "$all_submodules" \ + mvn test -Pnative -PquickBuild --projects "$all_submodules" \ ${INTEGRATION_TEST_ARGS} \ -B -ntp -fae \ -DtrimStackTrace=false \ @@ -387,7 +387,11 @@ function install_modules() { if [ -z "$1" ]; then mvn install \ -B -ntp \ +<<<<<<< Updated upstream -Pfast \ +======= + -PquickBuild \ +>>>>>>> Stashed changes -DtrimStackTrace=false \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ @@ -435,7 +439,11 @@ function install_modules() { # Correctly builds dependencies without building dependents. mvn install --projects "$all_submodules,$always_install_deps" --also-make \ -B -ntp \ +<<<<<<< Updated upstream -Pfast \ +======= + -PquickBuild \ +>>>>>>> Stashed changes -DtrimStackTrace=false \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ diff --git a/.kokoro/dependencies.sh b/.kokoro/dependencies.sh index 7ee4915af22c..947422916e96 100755 --- a/.kokoro/dependencies.sh +++ b/.kokoro/dependencies.sh @@ -60,9 +60,9 @@ fi # this should run maven enforcer retry_with_backoff 3 10 \ mvn install -B -V -ntp \ - -Pfast -DskipTests=true -Dmaven.javadoc.skip=true -Denforcer.skip=false + -PquickBuild -DskipTests=true -Dmaven.javadoc.skip=true -Denforcer.skip=false -mvn -B dependency:analyze -Pfast -DfailOnWarning=true -Dmdep.analyze.skip=false +mvn -B dependency:analyze -PquickBuild -DfailOnWarning=true -Dmdep.analyze.skip=false if [[ -n "${BUILD_SUBDIR}" ]] then diff --git a/google-cloud-pom-parent/pom.xml b/google-cloud-pom-parent/pom.xml index fc85b487a805..4ba90fb08b60 100644 --- a/google-cloud-pom-parent/pom.xml +++ b/google-cloud-pom-parent/pom.xml @@ -59,7 +59,7 @@ - fast + quickBuild true true diff --git a/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml b/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml index f1fba2e60fd0..6778c89d5a65 100644 --- a/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml +++ b/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml @@ -78,7 +78,7 @@ - fast + quickBuild true true From 67f09f30c8f7c2ff5745bfc70735ec3ef46d592b Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Tue, 24 Mar 2026 21:55:33 -0400 Subject: [PATCH 08/13] Fix indentation in workflow files and resolve leftover conflicts --- .github/workflows/ci.yaml | 4 +-- .github/workflows/sdk-platform-java-ci.yaml | 28 +++++++++------------ 2 files changed, 14 insertions(+), 18 deletions(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index f01bef2198d9..9b9204eb43bd 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -216,7 +216,7 @@ jobs: - run: java -version - name: Install Maven modules to local Maven repository run: | -mvn install -B -ntp -T 1C -PquickBuild -DskipTests + mvn install -B -ntp -T 1C -PquickBuild -DskipTests - run: java -version - run: mvn -B -ntp enforcer:enforce@enforce -T 1C gapic-libraries-bom: @@ -230,7 +230,7 @@ mvn install -B -ntp -T 1C -PquickBuild -DskipTests cache: maven - name: Install Maven modules to local Maven repository run: | -mvn install -B -ntp -T 1C -PquickBuild -DskipTests + mvn install -B -ntp -T 1C -PquickBuild -DskipTests - name: Validate gapic-libraries-bom uses: googleapis/java-cloud-bom/tests/validate-bom@v26.54.0 with: diff --git a/.github/workflows/sdk-platform-java-ci.yaml b/.github/workflows/sdk-platform-java-ci.yaml index 08c90c8f9b25..0e6967025704 100644 --- a/.github/workflows/sdk-platform-java-ci.yaml +++ b/.github/workflows/sdk-platform-java-ci.yaml @@ -34,13 +34,13 @@ jobs: - run: java -version - name: Unit Tests run: | -mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable - name: Env Var Tests run: | -mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage -PenvVarTest # Set the Env Var for this step only env: @@ -51,7 +51,7 @@ mvn test --batch-mode --no-transfer-progress -PquickBuild \ - run: bazelisk version - name: Install Maven modules run: | -mvn install -B -ntp -PquickBuild -DskipTests + mvn install -B -ntp -PquickBuild -DskipTests working-directory: sdk-platform-java - name: Integration Tests run: | @@ -90,13 +90,13 @@ mvn install -B -ntp -PquickBuild -DskipTests - run: java -version - name: Unit Tests run: | -mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable - name: Env Var Tests run: | -mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage -PenvVarTest # Set the Env Var for this step only env: @@ -107,11 +107,7 @@ mvn test --batch-mode --no-transfer-progress -PquickBuild \ - run: bazelisk version - name: Install Maven modules run: | -<<<<<<< Updated upstream - mvn install -B -ntp -Pfast -DskipTests -======= mvn install -B -ntp -PquickBuild -DskipTests ->>>>>>> Stashed changes working-directory: sdk-platform-java - name: Integration Tests run: | @@ -132,13 +128,13 @@ mvn test --batch-mode --no-transfer-progress -PquickBuild \ - run: java -version - name: Unit Tests run: | -mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable - name: Env Var Tests run: | -mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage -PenvVarTest # Set the Env Var for this step only env: @@ -149,7 +145,7 @@ mvn test --batch-mode --no-transfer-progress -PquickBuild \ - run: bazelisk version - name: Install Maven modules run: | -mvn install -B -ntp -PquickBuild -DskipTests + mvn install -B -ntp -PquickBuild -DskipTests working-directory: sdk-platform-java - name: Integration Tests # note need to set shouldInstallTestSecurityManager=false due to https://github.com/bazelbuild/bazel/issues/24354 @@ -194,7 +190,7 @@ mvn install -B -ntp -PquickBuild -DskipTests # Maven surefire plugin lets us to specify the JVM when running tests via # the "jvm" system property. export GOOGLE_CLOUD_UNIVERSE_DOMAIN=random.com -mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -PquickBuild \ -DenableTestCoverage -Dsurefire.failIfNoSpecifiedTests=false \ -PenvVarTest # Set the Env Var for this step only @@ -269,7 +265,7 @@ mvn test --batch-mode --no-transfer-progress -PquickBuild \ - name: Install all modules using Java 11 shell: bash run: | -mvn install -B -ntp -PquickBuild -DskipTests + mvn install -B -ntp -PquickBuild -DskipTests working-directory: sdk-platform-java - uses: actions/setup-java@v3 with: @@ -343,7 +339,7 @@ mvn install -B -ntp -PquickBuild -DskipTests - run: mvn -version - name: Install Maven modules run: | -mvn install -B -ntp -PquickBuild -DskipTests + mvn install -B -ntp -PquickBuild -DskipTests working-directory: sdk-platform-java - name: Showcase golden tests working-directory: sdk-platform-java/java-showcase @@ -422,7 +418,7 @@ mvn install -B -ntp -PquickBuild -DskipTests cache: maven - name: Install Maven modules run: | -mvn install -B -ntp -PquickBuild -DskipTests + mvn install -B -ntp -PquickBuild -DskipTests working-directory: sdk-platform-java - name: Install showcase to local maven repository run: | From 6ab80cdf04af4d897f249f75a0385669d267b080 Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Tue, 24 Mar 2026 22:02:56 -0400 Subject: [PATCH 09/13] chore: Use quickBuild as maven profile name --- .github/workflows/ci.yaml | 2 +- .kokoro/common.sh | 8 -------- 2 files changed, 1 insertion(+), 9 deletions(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 9b9204eb43bd..3faacf5effaa 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -158,7 +158,7 @@ jobs: run: | mvn test \ -B -ntp \ - -Pfast \ + -PquickBuild \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ -Dmaven.wagon.http.retryHandler.count=5 \ diff --git a/.kokoro/common.sh b/.kokoro/common.sh index 675d9939e399..4ddf4a83d2b2 100644 --- a/.kokoro/common.sh +++ b/.kokoro/common.sh @@ -388,11 +388,7 @@ function install_modules() { if [ -z "$1" ]; then mvn install \ -B -ntp \ -<<<<<<< Updated upstream - -Pfast \ -======= -PquickBuild \ ->>>>>>> Stashed changes -DtrimStackTrace=false \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ @@ -440,11 +436,7 @@ function install_modules() { # Correctly builds dependencies without building dependents. mvn install --projects "$all_submodules,$always_install_deps" --also-make \ -B -ntp \ -<<<<<<< Updated upstream - -Pfast \ -======= -PquickBuild \ ->>>>>>> Stashed changes -DtrimStackTrace=false \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ From c0a55b6381e4c1a98066675eb4258bd3974fa106 Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Wed, 25 Mar 2026 12:30:37 -0400 Subject: [PATCH 10/13] Replace manual skip flags with -PquickBuild in remaining workflow files --- .../java-spanner-jdbc-integration-tests-against-emulator.yaml | 2 +- .../sdk-platform-java-dependency_compatibility_test.yaml | 2 +- .../workflows/sdk-platform-java-java_compatibility_check.yaml | 2 +- .github/workflows/sdk-platform-java-nightly.yaml | 4 ++-- .github/workflows/sdk-platform-java-shared_dependencies.yaml | 4 ++-- .github/workflows/sdk-platform-java-sonar.yaml | 4 ++-- .github/workflows/unmanaged_dependency_check.yaml | 2 +- 7 files changed, 10 insertions(+), 10 deletions(-) diff --git a/.github/workflows/java-spanner-jdbc-integration-tests-against-emulator.yaml b/.github/workflows/java-spanner-jdbc-integration-tests-against-emulator.yaml index cddffc081038..536a9db7aa18 100644 --- a/.github/workflows/java-spanner-jdbc-integration-tests-against-emulator.yaml +++ b/.github/workflows/java-spanner-jdbc-integration-tests-against-emulator.yaml @@ -42,7 +42,7 @@ jobs: run: .kokoro/build.sh env: JOB_TYPE: test - - run: mvn -B -Dspanner.testenv.instance="" -Penable-integration-tests -DtrimStackTrace=false -Dclirr.skip=true -Denforcer.skip=true -fae verify + - run: mvn -B -Dspanner.testenv.instance="" -Penable-integration-tests -DtrimStackTrace=false -PquickBuild -fae verify working-directory: java-spanner-jdbc env: SPANNER_EMULATOR_HOST: localhost:9010 diff --git a/.github/workflows/sdk-platform-java-dependency_compatibility_test.yaml b/.github/workflows/sdk-platform-java-dependency_compatibility_test.yaml index 733b64a7ec70..e5af610756bf 100644 --- a/.github/workflows/sdk-platform-java-dependency_compatibility_test.yaml +++ b/.github/workflows/sdk-platform-java-dependency_compatibility_test.yaml @@ -64,7 +64,7 @@ jobs: # Install the modules for showcase (Shared-Deps is required to run showcase) - name: Install sdk-platform-java's modules # gapic-generator-java requires Java 8 and is irrelevant for this CI - run: mvn -q -B -ntp install --projects '!gapic-generator-java' -Dcheckstyle.skip -Dfmt.skip -DskipTests -Dclirr.skip -T 1C + run: mvn -q -B -ntp install --projects '!gapic-generator-java' -PquickBuild -DskipTests -PquickBuild -T 1C working-directory: sdk-platform-java # Set up local showcase server to run the showcase ITs diff --git a/.github/workflows/sdk-platform-java-java_compatibility_check.yaml b/.github/workflows/sdk-platform-java-java_compatibility_check.yaml index ffd41e991db9..93cde1afba65 100644 --- a/.github/workflows/sdk-platform-java-java_compatibility_check.yaml +++ b/.github/workflows/sdk-platform-java-java_compatibility_check.yaml @@ -44,7 +44,7 @@ jobs: cache: maven - name: Install sdk-platform-modules to local Maven repository run: | - mvn install -B -ntp -T 1C -DskipTests -Dclirr.skip -Dcheckstyle.skip -Denforcer.skip + mvn install -B -ntp -T 1C -DskipTests -PquickBuild working-directory: sdk-platform-java - name: Check Java 8 compatibility for class files shell: bash diff --git a/.github/workflows/sdk-platform-java-nightly.yaml b/.github/workflows/sdk-platform-java-nightly.yaml index f262a9679fac..6a56c7fc5e8b 100644 --- a/.github/workflows/sdk-platform-java-nightly.yaml +++ b/.github/workflows/sdk-platform-java-nightly.yaml @@ -40,7 +40,7 @@ jobs: cache: maven - run: mvn -version - name: Install - run: mvn install --errors --batch-mode --no-transfer-progress -Dcheckstyle.skip -Dfmt.skip + run: mvn install --errors --batch-mode --no-transfer-progress -PquickBuild - name: Create issue if previous step fails if: ${{ failure() }} env: @@ -67,7 +67,7 @@ jobs: cache: maven - run: mvn -version - name: Install with Java 11 - run: mvn install --errors --batch-mode --no-transfer-progress -Dcheckstyle.skip -DskipTests -Dfmt.skip + run: mvn install --errors --batch-mode --no-transfer-progress -PquickBuild -DskipTests -PquickBuild - uses: actions/setup-java@v5 with: diff --git a/.github/workflows/sdk-platform-java-shared_dependencies.yaml b/.github/workflows/sdk-platform-java-shared_dependencies.yaml index ead0dbe6a476..d6d6a3af3c36 100644 --- a/.github/workflows/sdk-platform-java-shared_dependencies.yaml +++ b/.github/workflows/sdk-platform-java-shared_dependencies.yaml @@ -38,7 +38,7 @@ jobs: - run: java -version - name: Install maven modules run: | - mvn install -B -ntp -DskipTests -Dclirr.skip -Dcheckstyle.skip + mvn install -B -ntp -DskipTests -PquickBuild - name: Check the BOM content satisfies the upper-bound-check test case - run: mvn -B -V -ntp verify -Dcheckstyle.skip + run: mvn -B -V -ntp verify -PquickBuild working-directory: sdk-plaform-java/java-shared-dependencies/upper-bound-check diff --git a/.github/workflows/sdk-platform-java-sonar.yaml b/.github/workflows/sdk-platform-java-sonar.yaml index 998cf5d1d80c..51bd48e9b898 100644 --- a/.github/workflows/sdk-platform-java-sonar.yaml +++ b/.github/workflows/sdk-platform-java-sonar.yaml @@ -73,7 +73,7 @@ jobs: GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }} run: | - mvn -B verify -Dcheckstyle.skip \ + mvn -B verify -PquickBuild \ -DenableFullTestCoverage \ -Penable-integration-tests \ org.sonarsource.scanner.maven:sonar-maven-plugin:sonar \ @@ -85,7 +85,7 @@ jobs: GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }} run: | - mvn -B clean verify -Dcheckstyle.skip \ + mvn -B clean verify -PquickBuild \ -DskipUnitTests \ -Penable-integration-tests \ -DenableShowcaseTestCoverage \ diff --git a/.github/workflows/unmanaged_dependency_check.yaml b/.github/workflows/unmanaged_dependency_check.yaml index 2b6b1342374a..5a5d47f63050 100644 --- a/.github/workflows/unmanaged_dependency_check.yaml +++ b/.github/workflows/unmanaged_dependency_check.yaml @@ -12,7 +12,7 @@ jobs: java-version: 11 - name: Install modules shell: bash - run: mvn install -B -ntp -T 1C -DskipTests -Dclirr.skip -Dcheckstyle.skip -Denforcer.skip + run: mvn install -B -ntp -T 1C -DskipTests -PquickBuild - name: Unmanaged dependency check uses: ./sdk-platform-java/java-shared-dependencies/unmanaged-dependency-check with: From 93dbad9f3a38e92d8107e997783330388f68a2e5 Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Wed, 25 Mar 2026 13:51:50 -0400 Subject: [PATCH 11/13] chore: Remove duplicate quickBuild config --- .../sdk-platform-java-dependency_compatibility_test.yaml | 2 +- .github/workflows/sdk-platform-java-nightly.yaml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/sdk-platform-java-dependency_compatibility_test.yaml b/.github/workflows/sdk-platform-java-dependency_compatibility_test.yaml index e5af610756bf..473235784e4d 100644 --- a/.github/workflows/sdk-platform-java-dependency_compatibility_test.yaml +++ b/.github/workflows/sdk-platform-java-dependency_compatibility_test.yaml @@ -64,7 +64,7 @@ jobs: # Install the modules for showcase (Shared-Deps is required to run showcase) - name: Install sdk-platform-java's modules # gapic-generator-java requires Java 8 and is irrelevant for this CI - run: mvn -q -B -ntp install --projects '!gapic-generator-java' -PquickBuild -DskipTests -PquickBuild -T 1C + run: mvn -q -B -ntp install --projects '!gapic-generator-java' -PquickBuild -DskipTests -T 1C working-directory: sdk-platform-java # Set up local showcase server to run the showcase ITs diff --git a/.github/workflows/sdk-platform-java-nightly.yaml b/.github/workflows/sdk-platform-java-nightly.yaml index 6a56c7fc5e8b..3ddc5c4cbf1f 100644 --- a/.github/workflows/sdk-platform-java-nightly.yaml +++ b/.github/workflows/sdk-platform-java-nightly.yaml @@ -67,7 +67,7 @@ jobs: cache: maven - run: mvn -version - name: Install with Java 11 - run: mvn install --errors --batch-mode --no-transfer-progress -PquickBuild -DskipTests -PquickBuild + run: mvn install --errors --batch-mode --no-transfer-progress -PquickBuild -DskipTests - uses: actions/setup-java@v5 with: From 7f0458ab6d98b4b095e189b6898ee8cc95e0abd7 Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Wed, 25 Mar 2026 13:53:26 -0400 Subject: [PATCH 12/13] Revert automated cleanup for java-spanner-jdbc.yaml --- .../java-spanner-jdbc-integration-tests-against-emulator.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/java-spanner-jdbc-integration-tests-against-emulator.yaml b/.github/workflows/java-spanner-jdbc-integration-tests-against-emulator.yaml index 536a9db7aa18..cddffc081038 100644 --- a/.github/workflows/java-spanner-jdbc-integration-tests-against-emulator.yaml +++ b/.github/workflows/java-spanner-jdbc-integration-tests-against-emulator.yaml @@ -42,7 +42,7 @@ jobs: run: .kokoro/build.sh env: JOB_TYPE: test - - run: mvn -B -Dspanner.testenv.instance="" -Penable-integration-tests -DtrimStackTrace=false -PquickBuild -fae verify + - run: mvn -B -Dspanner.testenv.instance="" -Penable-integration-tests -DtrimStackTrace=false -Dclirr.skip=true -Denforcer.skip=true -fae verify working-directory: java-spanner-jdbc env: SPANNER_EMULATOR_HOST: localhost:9010 From cc3994e71acb17de0ef6cbf23b00950a09c5eff5 Mon Sep 17 00:00:00 2001 From: Lawrence Qiu Date: Wed, 25 Mar 2026 13:56:09 -0400 Subject: [PATCH 13/13] Rename quickBuild profile to quick-build project-wide --- .github/workflows/ci.yaml | 6 ++-- .github/workflows/sdk-platform-java-ci.yaml | 34 +++++++++---------- ...rm-java-dependency_compatibility_test.yaml | 2 +- ...latform-java-java_compatibility_check.yaml | 2 +- .../workflows/sdk-platform-java-nightly.yaml | 4 +-- ...sdk-platform-java-shared_dependencies.yaml | 4 +-- .../workflows/sdk-platform-java-sonar.yaml | 4 +-- .../workflows/unmanaged_dependency_check.yaml | 2 +- google-cloud-pom-parent/pom.xml | 2 +- .../gapic-generator-java-pom-parent/pom.xml | 2 +- 10 files changed, 31 insertions(+), 31 deletions(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 3faacf5effaa..099fbb7908f1 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -158,7 +158,7 @@ jobs: run: | mvn test \ -B -ntp \ - -PquickBuild \ + -Pquick-build \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ -Dmaven.wagon.http.retryHandler.count=5 \ @@ -216,7 +216,7 @@ jobs: - run: java -version - name: Install Maven modules to local Maven repository run: | - mvn install -B -ntp -T 1C -PquickBuild -DskipTests + mvn install -B -ntp -T 1C -Pquick-build -DskipTests - run: java -version - run: mvn -B -ntp enforcer:enforce@enforce -T 1C gapic-libraries-bom: @@ -230,7 +230,7 @@ jobs: cache: maven - name: Install Maven modules to local Maven repository run: | - mvn install -B -ntp -T 1C -PquickBuild -DskipTests + mvn install -B -ntp -T 1C -Pquick-build -DskipTests - name: Validate gapic-libraries-bom uses: googleapis/java-cloud-bom/tests/validate-bom@v26.54.0 with: diff --git a/.github/workflows/sdk-platform-java-ci.yaml b/.github/workflows/sdk-platform-java-ci.yaml index 0e6967025704..38653a474b24 100644 --- a/.github/workflows/sdk-platform-java-ci.yaml +++ b/.github/workflows/sdk-platform-java-ci.yaml @@ -34,13 +34,13 @@ jobs: - run: java -version - name: Unit Tests run: | - mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -Pquick-build \ -DenableTestCoverage working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable - name: Env Var Tests run: | - mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -Pquick-build \ -DenableTestCoverage -PenvVarTest # Set the Env Var for this step only env: @@ -51,7 +51,7 @@ jobs: - run: bazelisk version - name: Install Maven modules run: | - mvn install -B -ntp -PquickBuild -DskipTests + mvn install -B -ntp -Pquick-build -DskipTests working-directory: sdk-platform-java - name: Integration Tests run: | @@ -90,13 +90,13 @@ jobs: - run: java -version - name: Unit Tests run: | - mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -Pquick-build \ -DenableTestCoverage working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable - name: Env Var Tests run: | - mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -Pquick-build \ -DenableTestCoverage -PenvVarTest # Set the Env Var for this step only env: @@ -107,7 +107,7 @@ jobs: - run: bazelisk version - name: Install Maven modules run: | - mvn install -B -ntp -PquickBuild -DskipTests + mvn install -B -ntp -Pquick-build -DskipTests working-directory: sdk-platform-java - name: Integration Tests run: | @@ -128,13 +128,13 @@ jobs: - run: java -version - name: Unit Tests run: | - mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -Pquick-build \ -DenableTestCoverage working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable - name: Env Var Tests run: | - mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -Pquick-build \ -DenableTestCoverage -PenvVarTest # Set the Env Var for this step only env: @@ -145,7 +145,7 @@ jobs: - run: bazelisk version - name: Install Maven modules run: | - mvn install -B -ntp -PquickBuild -DskipTests + mvn install -B -ntp -Pquick-build -DskipTests working-directory: sdk-platform-java - name: Integration Tests # note need to set shouldInstallTestSecurityManager=false due to https://github.com/bazelbuild/bazel/issues/24354 @@ -177,7 +177,7 @@ jobs: export PATH=${JAVA_HOME}/bin:$PATH # Maven surefire plugin lets us to specify the JVM when running tests via # the "jvm" system property. - mvn verify --batch-mode --no-transfer-progress -PquickBuild \ + mvn verify --batch-mode --no-transfer-progress -Pquick-build \ -Djvm="${JAVA8_HOME}/bin/java" working-directory: sdk-platform-java # The `envVarTest` profile runs tests that require an environment variable @@ -190,7 +190,7 @@ jobs: # Maven surefire plugin lets us to specify the JVM when running tests via # the "jvm" system property. export GOOGLE_CLOUD_UNIVERSE_DOMAIN=random.com - mvn test --batch-mode --no-transfer-progress -PquickBuild \ + mvn test --batch-mode --no-transfer-progress -Pquick-build \ -DenableTestCoverage -Dsurefire.failIfNoSpecifiedTests=false \ -PenvVarTest # Set the Env Var for this step only @@ -215,7 +215,7 @@ jobs: - name: Install all modules using Java 11 shell: bash run: | - mvn -V -B -ntp clean install -PquickBuild -DskipTests + mvn -V -B -ntp clean install -Pquick-build -DskipTests working-directory: sdk-platform-java - uses: actions/setup-java@v3 with: @@ -265,7 +265,7 @@ jobs: - name: Install all modules using Java 11 shell: bash run: | - mvn install -B -ntp -PquickBuild -DskipTests + mvn install -B -ntp -Pquick-build -DskipTests working-directory: sdk-platform-java - uses: actions/setup-java@v3 with: @@ -339,7 +339,7 @@ jobs: - run: mvn -version - name: Install Maven modules run: | - mvn install -B -ntp -PquickBuild -DskipTests + mvn install -B -ntp -Pquick-build -DskipTests working-directory: sdk-platform-java - name: Showcase golden tests working-directory: sdk-platform-java/java-showcase @@ -418,7 +418,7 @@ jobs: cache: maven - name: Install Maven modules run: | - mvn install -B -ntp -PquickBuild -DskipTests + mvn install -B -ntp -Pquick-build -DskipTests working-directory: sdk-platform-java - name: Install showcase to local maven repository run: | @@ -456,7 +456,7 @@ jobs: run: | mvn install \ -B -ntp \ - -PquickBuild \ + -Pquick-build \ -Dorg.slf4j.simpleLogger.showDateTime=true \ -Dorg.slf4j.simpleLogger.dateTimeFormat=HH:mm:ss:SSS \ -DskipTests \ @@ -487,7 +487,7 @@ jobs: working-directory: sdk-platform-java/java-shared-dependencies/unmanaged-dependency-check - name: Install Maven modules run: | - mvn install -B -ntp -PquickBuild -DskipTests + mvn install -B -ntp -Pquick-build -DskipTests working-directory: sdk-platform-java - name: Unmanaged dependency check uses: ./sdk-platform-java/java-shared-dependencies/unmanaged-dependency-check diff --git a/.github/workflows/sdk-platform-java-dependency_compatibility_test.yaml b/.github/workflows/sdk-platform-java-dependency_compatibility_test.yaml index 473235784e4d..0b3f10bd3e82 100644 --- a/.github/workflows/sdk-platform-java-dependency_compatibility_test.yaml +++ b/.github/workflows/sdk-platform-java-dependency_compatibility_test.yaml @@ -64,7 +64,7 @@ jobs: # Install the modules for showcase (Shared-Deps is required to run showcase) - name: Install sdk-platform-java's modules # gapic-generator-java requires Java 8 and is irrelevant for this CI - run: mvn -q -B -ntp install --projects '!gapic-generator-java' -PquickBuild -DskipTests -T 1C + run: mvn -q -B -ntp install --projects '!gapic-generator-java' -Pquick-build -DskipTests -T 1C working-directory: sdk-platform-java # Set up local showcase server to run the showcase ITs diff --git a/.github/workflows/sdk-platform-java-java_compatibility_check.yaml b/.github/workflows/sdk-platform-java-java_compatibility_check.yaml index 93cde1afba65..893801d5ec08 100644 --- a/.github/workflows/sdk-platform-java-java_compatibility_check.yaml +++ b/.github/workflows/sdk-platform-java-java_compatibility_check.yaml @@ -44,7 +44,7 @@ jobs: cache: maven - name: Install sdk-platform-modules to local Maven repository run: | - mvn install -B -ntp -T 1C -DskipTests -PquickBuild + mvn install -B -ntp -T 1C -DskipTests -Pquick-build working-directory: sdk-platform-java - name: Check Java 8 compatibility for class files shell: bash diff --git a/.github/workflows/sdk-platform-java-nightly.yaml b/.github/workflows/sdk-platform-java-nightly.yaml index 3ddc5c4cbf1f..cf3fd6e8affa 100644 --- a/.github/workflows/sdk-platform-java-nightly.yaml +++ b/.github/workflows/sdk-platform-java-nightly.yaml @@ -40,7 +40,7 @@ jobs: cache: maven - run: mvn -version - name: Install - run: mvn install --errors --batch-mode --no-transfer-progress -PquickBuild + run: mvn install --errors --batch-mode --no-transfer-progress -Pquick-build - name: Create issue if previous step fails if: ${{ failure() }} env: @@ -67,7 +67,7 @@ jobs: cache: maven - run: mvn -version - name: Install with Java 11 - run: mvn install --errors --batch-mode --no-transfer-progress -PquickBuild -DskipTests + run: mvn install --errors --batch-mode --no-transfer-progress -Pquick-build -DskipTests - uses: actions/setup-java@v5 with: diff --git a/.github/workflows/sdk-platform-java-shared_dependencies.yaml b/.github/workflows/sdk-platform-java-shared_dependencies.yaml index d6d6a3af3c36..b1482fd2fef4 100644 --- a/.github/workflows/sdk-platform-java-shared_dependencies.yaml +++ b/.github/workflows/sdk-platform-java-shared_dependencies.yaml @@ -38,7 +38,7 @@ jobs: - run: java -version - name: Install maven modules run: | - mvn install -B -ntp -DskipTests -PquickBuild + mvn install -B -ntp -DskipTests -Pquick-build - name: Check the BOM content satisfies the upper-bound-check test case - run: mvn -B -V -ntp verify -PquickBuild + run: mvn -B -V -ntp verify -Pquick-build working-directory: sdk-plaform-java/java-shared-dependencies/upper-bound-check diff --git a/.github/workflows/sdk-platform-java-sonar.yaml b/.github/workflows/sdk-platform-java-sonar.yaml index 51bd48e9b898..9b29f31e7345 100644 --- a/.github/workflows/sdk-platform-java-sonar.yaml +++ b/.github/workflows/sdk-platform-java-sonar.yaml @@ -73,7 +73,7 @@ jobs: GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }} run: | - mvn -B verify -PquickBuild \ + mvn -B verify -Pquick-build \ -DenableFullTestCoverage \ -Penable-integration-tests \ org.sonarsource.scanner.maven:sonar-maven-plugin:sonar \ @@ -85,7 +85,7 @@ jobs: GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} # Needed to get PR information, if any SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }} run: | - mvn -B clean verify -PquickBuild \ + mvn -B clean verify -Pquick-build \ -DskipUnitTests \ -Penable-integration-tests \ -DenableShowcaseTestCoverage \ diff --git a/.github/workflows/unmanaged_dependency_check.yaml b/.github/workflows/unmanaged_dependency_check.yaml index 5a5d47f63050..1f3913856282 100644 --- a/.github/workflows/unmanaged_dependency_check.yaml +++ b/.github/workflows/unmanaged_dependency_check.yaml @@ -12,7 +12,7 @@ jobs: java-version: 11 - name: Install modules shell: bash - run: mvn install -B -ntp -T 1C -DskipTests -PquickBuild + run: mvn install -B -ntp -T 1C -DskipTests -Pquick-build - name: Unmanaged dependency check uses: ./sdk-platform-java/java-shared-dependencies/unmanaged-dependency-check with: diff --git a/google-cloud-pom-parent/pom.xml b/google-cloud-pom-parent/pom.xml index 4ba90fb08b60..168859b8d4dc 100644 --- a/google-cloud-pom-parent/pom.xml +++ b/google-cloud-pom-parent/pom.xml @@ -59,7 +59,7 @@ - quickBuild + quick-build true true diff --git a/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml b/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml index 6778c89d5a65..8aa38c67c5d2 100644 --- a/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml +++ b/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml @@ -78,7 +78,7 @@ - quickBuild + quick-build true true