Skip to content

ci: stop running RAT in the parallel packaging-check build #20527

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
FrankChen021 merged 1 commit into from
Oct 9, 2026
Merged

ci: stop running RAT in the parallel packaging-check build #20527

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
16 changes: 9 additions & 7 deletions .github/scripts/packaging-check.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,16 @@ set -x

./.github/scripts/setup_generate_license.sh
# This job is the single place in CI that validates everything a release build
# produces: RAT license headers, javadoc and source jars, the binary and source
# distribution assemblies, and the license dependency reports. The apache-release
# profile is enabled here so that the Docker test job only needs to build the
# binary tarball. GPG signing and the OWASP dependency check are skipped as they
# are not meaningful in CI.
./mvnw -B clean install -Prat -Papache-release --fail-at-end \
# produces: javadoc and source jars, the binary and source distribution
# assemblies, and the license dependency reports. The apache-release profile is
# enabled here so that the Docker test job only needs to build the binary
# tarball. GPG signing and the OWASP dependency check are skipped as they are not
# meaningful in CI. RAT is not run here: apache-rat-plugin is not safe to run in
# a parallel (-T) build, so the single-threaded RAT pass in static-checks-maven.sh
# covers it instead.
./mvnw -B clean install -Papache-release --fail-at-end \
-pl '!benchmarks, !distribution' -P skip-tests -Dweb.console.skip=false -T1C \
-Dgpg.skip -Ddependency-check.skip
./mvnw -B install -Prat -Papache-release -Pdist -Pbundle-contrib-exts --fail-at-end \
./mvnw -B install -Papache-release -Pdist -Pbundle-contrib-exts --fail-at-end \
-pl 'distribution' -P skip-tests -Dweb.console.skip=false -T1C \
-Dgpg.skip -Ddependency-check.skip
5 changes: 3 additions & 2 deletions .github/scripts/static-checks-maven.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,9 @@ echo 'Running Maven install...'

./mvnw -B checkstyle:checkstyle --fail-at-end

# Repo-wide RAT check. packaging-check also runs RAT, but it excludes the
# benchmarks module from its reactor, so keep this standalone pass here.
# Repo-wide RAT check. This is the only RAT pass in CI. It must run without -T:
# apache-rat-plugin 0.17+ shares per-run state across modules, so parallel runs
# can drop configured excludes or throw NPEs (RAT-553, fixed upstream by RAT-573).
./mvnw -B apache-rat:check -Prat --fail-at-end \
-Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn \
-Drat.consoleOutput=true
Expand Down
Loading