diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index c3e8df0be851..cf5896bff2cd 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -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 diff --git a/.github/workflows/sdk-platform-java-ci.yaml b/.github/workflows/sdk-platform-java-ci.yaml index 06b94a09c6ee..c9e2b264f0b7 100644 --- a/.github/workflows/sdk-platform-java-ci.yaml +++ b/.github/workflows/sdk-platform-java-ci.yaml @@ -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 @@ -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 @@ -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: diff --git a/.github/workflows/sdk-platform-java-nightly.yaml b/.github/workflows/sdk-platform-java-nightly.yaml index 5afefbda41c6..97d52681862c 100644 --- a/.github/workflows/sdk-platform-java-nightly.yaml +++ b/.github/workflows/sdk-platform-java-nightly.yaml @@ -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() }} diff --git a/.github/workflows/sdk-platform-java-shared_dependencies.yaml b/.github/workflows/sdk-platform-java-shared_dependencies.yaml index 0a92d836a2fd..19e0fe320921 100644 --- a/.github/workflows/sdk-platform-java-shared_dependencies.yaml +++ b/.github/workflows/sdk-platform-java-shared_dependencies.yaml @@ -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 diff --git a/.github/workflows/sdk-platform-java-sonar.yaml b/.github/workflows/sdk-platform-java-sonar.yaml index bbe558e168b4..24133d69110d 100644 --- a/.github/workflows/sdk-platform-java-sonar.yaml +++ b/.github/workflows/sdk-platform-java-sonar.yaml @@ -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 \ @@ -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 \ diff --git a/.github/workflows/unmanaged_dependency_check.yaml b/.github/workflows/unmanaged_dependency_check.yaml index 2b6b1342374a..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 -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: diff --git a/.kokoro/build.sh b/.kokoro/build.sh index b8524fbb94d8..305c16edf5b0 100755 --- a/.kokoro/build.sh +++ b/.kokoro/build.sh @@ -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[@]}" @@ -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 \ @@ -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} @@ -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 diff --git a/.kokoro/common.sh b/.kokoro/common.sh index aac5159433d7..bfcef516e43e 100644 --- a/.kokoro/common.sh +++ b/.kokoro/common.sh @@ -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 @@ -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" @@ -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" @@ -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 } diff --git a/.kokoro/dependencies.sh b/.kokoro/dependencies.sh index b273094ff7d3..008b72d8134b 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 + -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 diff --git a/google-cloud-pom-parent/pom.xml b/google-cloud-pom-parent/pom.xml index b6f48164e8b1..43fd7f7a577a 100644 --- a/google-cloud-pom-parent/pom.xml +++ b/google-cloud-pom-parent/pom.xml @@ -58,6 +58,22 @@ + + + quick-build + + true + true + true + true + true + true + true + true + true + true + + checkstyle-tests @@ -78,6 +94,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 c976f0d643f2..71a5be665c73 100644 --- a/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml +++ b/sdk-platform-java/gapic-generator-java-pom-parent/pom.xml @@ -76,6 +76,23 @@ + + + quick-build + + true + true + true + true + true + true + true + true + true + true + + + checkstyle-tests @@ -96,6 +113,7 @@ ${checkstyle.header.file} + ${checkstyle.skip}