Skip to content

ci: Add Maven quick-build profile to skip maven lifecycle plugins #12188

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
lqiu96 merged 20 commits into from
Apr 8, 2026
Merged

ci: Add Maven quick-build profile to skip maven lifecycle plugins #12188

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
d72770d
ci: Add fast profile that skips all maven lifecycle plugins
lqiu96 Mar 24, 2026
74f40e5
chore: Add checkstyle skip flag
lqiu96 Mar 24, 2026
fed4516
chore: Update kokoro CI to use fast profile
lqiu96 Mar 24, 2026
27f265e
chore: Add to gapic-generator-java-pom-parent
lqiu96 Mar 24, 2026
ba3c72f
chore: Remove the redundant config in the aggregator pom
lqiu96 Mar 24, 2026
bf7e2df
chore: Address PR feedback
lqiu96 Mar 24, 2026
79f2562
Merge branch 'main' into maven-fast-profile
lqiu96 Mar 24, 2026
5cf488f
Resolve merge conflicts and rename fast profile to quickBuild
lqiu96 Mar 25, 2026
8493218
Merge branch 'main' of github.com:googleapis/google-cloud-java into m…
lqiu96 Mar 25, 2026
67f09f3
Fix indentation in workflow files and resolve leftover conflicts
lqiu96 Mar 25, 2026
6ab80cd
chore: Use quickBuild as maven profile name
lqiu96 Mar 25, 2026
c0a55b6
Replace manual skip flags with -PquickBuild in remaining workflow files
lqiu96 Mar 25, 2026
93dbad9
chore: Remove duplicate quickBuild config
lqiu96 Mar 25, 2026
7f0458a
Revert automated cleanup for java-spanner-jdbc.yaml
lqiu96 Mar 25, 2026
cc3994e
Rename quickBuild profile to quick-build project-wide
lqiu96 Mar 25, 2026
43fe441
Merge branch 'main' into maven-fast-profile
lqiu96 Mar 27, 2026
55ae26e
Merge branch 'main' into maven-fast-profile
lqiu96 Apr 6, 2026
24a0a62
chore: Add comment for the quick-build profile
lqiu96 Apr 6, 2026
46cf801
chore: Update from quickBuild to quick-build
lqiu96 Apr 6, 2026
ed95dcd
Merge branch 'main' into maven-fast-profile
lqiu96 Apr 8, 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
6 changes: 1 addition & 5 deletions .github/workflows/ci.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -187,13 +187,9 @@ jobs:
run: |
mvn test \
-B -ntp \
-Pquick-build \
-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 \
-T 1C
Expand Down
11 changes: 5 additions & 6 deletions .github/workflows/sdk-platform-java-ci.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,8 @@ jobs:
# 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 -Pquick-build \
-DenableTestCoverage -PenvVarTest
# Set the Env Var for this step only
env:
GOOGLE_CLOUD_UNIVERSE_DOMAIN: random.com
Expand Down Expand Up @@ -191,8 +191,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 -Pquick-build \
-Djvm="${JAVA8_HOME}/bin/java"
working-directory: sdk-platform-java
# The `envVarTest` profile runs tests that require an environment variable
Expand All @@ -205,8 +204,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 -Pquick-build \
-DenableTestCoverage -Dsurefire.failIfNoSpecifiedTests=false \
-PenvVarTest
# Set the Env Var for this step only
env:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/sdk-platform-java-nightly.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ jobs:
BUILD_SUBDIR: sdk-platform-java
JOB_TYPE: install
- name: Install
run: mvn install --errors --batch-mode --no-transfer-progress -Dcheckstyle.skip -Dfmt.skip
run: mvn install --errors --batch-mode --no-transfer-progress -Pquick-build
working-directory: sdk-platform-java
- name: Create issue if previous step fails
if: ${{ failure() }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/sdk-platform-java-shared_dependencies.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
- run: java -version
- name: Install maven modules
run: |
mvn install -B -ntp -DskipTests -Dclirr.skip -Dcheckstyle.skip
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 -Dcheckstyle.skip
run: mvn -B -V -ntp verify -Pquick-build
working-directory: sdk-plaform-java/java-shared-dependencies/upper-bound-check
4 changes: 2 additions & 2 deletions .github/workflows/sdk-platform-java-sonar.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,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 -Pquick-build \
-DenableFullTestCoverage \
-Penable-integration-tests \
org.sonarsource.scanner.maven:sonar-maven-plugin:sonar \
Expand All @@ -84,7 +84,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 -Pquick-build \
-DskipUnitTests \
-Penable-integration-tests \
-DenableShowcaseTestCoverage \
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/unmanaged_dependency_check.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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 -Pquick-build
- name: Unmanaged dependency check
uses: ./sdk-platform-java/java-shared-dependencies/unmanaged-dependency-check
with:
Expand Down
29 changes: 5 additions & 24 deletions .kokoro/build.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,13 +52,9 @@ case ${JOB_TYPE} in
retry_with_backoff 3 10 \
mvn install \
-B -ntp \
-Pquick-build \
-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} "${EXTRA_PROFILE_OPTS[@]}"
Expand All @@ -80,13 +76,9 @@ case ${JOB_TYPE} in
install_modules "sdk-platform-java"
mvn install \
-B -ntp \
-Pquick-build \
-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 \
Expand Down Expand Up @@ -143,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 -Pquick-build \
--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}
Expand Down Expand Up @@ -202,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 -Pquick-build \
--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
Expand Down
31 changes: 5 additions & 26 deletions .kokoro/common.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -303,20 +303,14 @@ 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 -Pquick-build --projects "$all_submodules" \
${INTEGRATION_TEST_ARGS} \
-B -ntp -fae \
--also-make \
-PbulkTests \
-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
Expand All @@ -330,17 +324,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 -Pquick-build --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"
Expand Down Expand Up @@ -405,16 +394,11 @@ function install_modules() {
if [ -z "$1" ]; then
mvn install \
-B -ntp \
-Pquick-build \
-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"
Expand Down Expand Up @@ -472,16 +456,11 @@ function install_modules() {
# Correctly builds dependencies without building dependents.
mvn install --projects "$all_submodules,$always_install_deps" --also-make \
-B -ntp \
-Pquick-build \
-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
}
Expand Down
6 changes: 2 additions & 4 deletions .kokoro/dependencies.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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
-Pquick-build -DskipTests=true -Dmaven.javadoc.skip=true -Denforcer.skip=false

mvn -B dependency:analyze -DfailOnWarning=true -Dcheckstyle.skip=true
mvn -B dependency:analyze -Pquick-build -DfailOnWarning=true -Dmdep.analyze.skip=false

if [[ -n "${BUILD_SUBDIR}" ]]
then
Expand Down
17 changes: 17 additions & 0 deletions google-cloud-pom-parent/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,22 @@
</licenses>

<profiles>
<profile>
<!-- skips all plugins that would have been activated by the Maven lifecycle -->
<id>quick-build</id>
<properties>
<checkstyle.skip>true</checkstyle.skip>
<enforcer.skip>true</enforcer.skip>
<jacoco.skip>true</jacoco.skip>
<clirr.skip>true</clirr.skip>
<spotbugs.skip>true</spotbugs.skip>
<pmd.skip>true</pmd.skip>
<animal.sniffer.skip>true</animal.sniffer.skip>
<fmt.skip>true</fmt.skip>
<flatten.skip>true</flatten.skip>
<mdep.analyze.skip>true</mdep.analyze.skip>
</properties>
</profile>
<profile>
<!-- Only run checkstyle plugin on Java 11+ (checkstyle artifact only supports Java 11+) -->
<id>checkstyle-tests</id>
Expand All @@ -78,6 +94,7 @@
</goals>
<configuration>
<headerLocation>${checkstyle.header.file}</headerLocation>
<skip>${checkstyle.skip}</skip>
</configuration>
</execution>
</executions>
Expand Down
18 changes: 18 additions & 0 deletions sdk-platform-java/gapic-generator-java-pom-parent/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,23 @@
</licenses>

<profiles>
<profile>
<!-- skips all plugins that would have been activated by the Maven lifecycle -->
<id>quick-build</id>
Copy link
Copy Markdown
Contributor

@blakeli0 blakeli0 Apr 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a comment describing what this profile does?

Copy link
Copy Markdown
Member Author

@lqiu96 lqiu96 Apr 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated!

<properties>
<checkstyle.skip>true</checkstyle.skip>
<enforcer.skip>true</enforcer.skip>
<jacoco.skip>true</jacoco.skip>
<clirr.skip>true</clirr.skip>
<spotbugs.skip>true</spotbugs.skip>
<pmd.skip>true</pmd.skip>
<animal.sniffer.skip>true</animal.sniffer.skip>
<fmt.skip>true</fmt.skip>
<flatten.skip>true</flatten.skip>
<mdep.analyze.skip>true</mdep.analyze.skip>
</properties>
</profile>
Comment on lines +79 to +94
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot Mar 24, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

This fast profile is identical to the one defined in google-cloud-pom-parent/pom.xml. To improve maintainability and avoid code duplication, this profile should be defined in a common parent POM and inherited in both places. Please remove this duplicated profile and ensure it's inherited from a single source.


<profile>
<!-- Only run checkstyle plugin on Java 11+ (checkstyle artifact only supports Java 11+) -->
<id>checkstyle-tests</id>
Expand All @@ -96,6 +113,7 @@
</goals>
<configuration>
<headerLocation>${checkstyle.header.file}</headerLocation>
<skip>${checkstyle.skip}</skip>
</configuration>
</execution>
</executions>
Expand Down
Loading