Skip to content

[improve][ci] Disable detailed console logging for integration tests in CI #24266

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
lhotari merged 4 commits into from
May 7, 2025
Merged

[improve][ci] Disable detailed console logging for integration tests in CI #24266

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
28c95b8
[improve][ci] Disable detailed console logging for integration test
lhotari May 7, 2025
4da2f35
Use unique names for uploaded files in integration tests
lhotari Feb 11, 2025
d0e2f4b
Merge branch 'master' into lh-disable-log-output-for-integration-test…
lhotari May 7, 2025
15f51d0
Merge branch 'master' into lh-disable-log-output-for-integration-test…
lhotari May 7, 2025
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
8 changes: 6 additions & 2 deletions .github/workflows/pulsar-ci.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -572,24 +572,28 @@ jobs:

- name: Shade on Java 8
group: SHADE_RUN
upload_name: SHADE_RUN_8
runtime_jdk: 8
setup: ./build/run_integration_group.sh SHADE_BUILD
no_coverage: true

- name: Shade on Java 11
group: SHADE_RUN
upload_name: SHADE_RUN_11
runtime_jdk: 11
setup: ./build/run_integration_group.sh SHADE_BUILD
no_coverage: true

- name: Shade on Java 17
group: SHADE_RUN
upload_name: SHADE_RUN_17
runtime_jdk: 17
setup: ./build/run_integration_group.sh SHADE_BUILD
no_coverage: true

- name: Shade on Java 21
group: SHADE_RUN
upload_name: SHADE_RUN_21
runtime_jdk: 21
setup: ./build/run_integration_group.sh SHADE_BUILD
no_coverage: true
Expand Down Expand Up @@ -693,7 +697,7 @@ jobs:
uses: actions/upload-artifact@v4
if: ${{ !success() }}
with:
name: Integration-${{ matrix.group }}-surefire-reports
name: Integration-${{ matrix.upload_name || matrix.group }}-surefire-reports
path: surefire-reports
retention-days: 7

Expand All @@ -702,7 +706,7 @@ jobs:
if: ${{ !success() }}
continue-on-error: true
with:
name: Integration-${{ matrix.group }}-container-logs
name: Integration-${{ matrix.upload_name || matrix.group }}-container-logs
path: tests/integration/target/container-logs
retention-days: 7

Expand Down
2 changes: 1 addition & 1 deletion build/run_integration_group.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ mvn_run_integration_test() {
if [[ $build_only -ne 1 ]]; then
echo "::group::Run tests for " "$@"
# use "verify" instead of "test"
mvn -B -ntp -pl "$modules" $failfast_args $coverage_args -DskipDocker -DskipSourceReleaseAssembly=true -Dspotbugs.skip=true -Dlicense.skip=true -Dcheckstyle.skip=true -Drat.skip=true -DredirectTestOutputToFile=false $clean_arg verify "$@"
mvn -B -ntp -pl "$modules" $failfast_args $coverage_args -DskipDocker -DskipSourceReleaseAssembly=true -Dspotbugs.skip=true -Dlicense.skip=true -Dcheckstyle.skip=true -Drat.skip=true $clean_arg verify "$@"
echo "::endgroup::"
set +x
"$SCRIPT_DIR/pulsar_ci_tool.sh" move_test_reports || true
Expand Down
Loading