This is an automated email from the ASF dual-hosted git repository.
gaogaotiantian pushed a commit to branch branch-4.x
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/branch-4.x by this push:
new 9002a1ce1289 [SPARK-58852][INFRA] Replace archived test report action
9002a1ce1289 is described below
commit 9002a1ce12899278b0fdbcf948d5c83c9ffbd7f5
Author: Tian Gao <[email protected]>
AuthorDate: Tue Aug 25 00:33:39 2026 -0700
[SPARK-58852][INFRA] Replace archived test report action
### What changes were proposed in this pull request?
Replace `scacap/action-surefire-report` with
`EnricoMi/publish-unit-test-result-action` for test reporting.
### Why are the changes needed?
`scacap/action-surefire-report` is
[archived](https://github.com/ScaCap/action-surefire-report) and removed from
apache's allowlist. It has a
[descendent](https://github.com/ScaCap/action-surefire-report) but none of them
are used as much as `EnricoMi/publish-unit-test-result-action` , which is
already listed on apache's allowlist. We should use the popular projects which
is better maintained and tested. Also it saves us trouble to create a new entry
for the allowed action.
We considered `dorny/test-reporter` but we realized that the latest
released version (v3.0.0) does not have the `list-files` options, which means
it will always list **all** the files in the test suite and it will always
overflow the buffer github allows.
### Does this PR introduce _any_ user-facing change?
No, CI only.
### How was this patch tested?
CI should show this report. `test report` action on `apache/spark` has been
failing for a while.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #58098 from gaogaotiantian/replace-report-action.
Authored-by: Tian Gao <[email protected]>
Signed-off-by: Tian Gao <[email protected]>
(cherry picked from commit 05c2c126560698e84e7cc64fce88378ef5bedbc8)
Signed-off-by: Tian Gao <[email protected]>
---
.github/workflows/test_report.yml | 21 ++++++++-------------
1 file changed, 8 insertions(+), 13 deletions(-)
diff --git a/.github/workflows/test_report.yml
b/.github/workflows/test_report.yml
index 2640803bf9f6..548f6c99f1e7 100644
--- a/.github/workflows/test_report.yml
+++ b/.github/workflows/test_report.yml
@@ -41,20 +41,15 @@ jobs:
github-token: ${{ secrets.GITHUB_TOKEN }}
run-id: ${{ github.event.workflow_run.id }}
pattern: "test-*"
- - name: Check if test results exist
- id: check
- run: |
- if find . -path '*/target/test-reports/*.xml' -print -quit | grep -q
.; then
- echo "has_results=true" >> $GITHUB_OUTPUT
- else
- echo "No test result files found. Skipping report."
- echo "has_results=false" >> $GITHUB_OUTPUT
- fi
- name: Publish test report
- if: steps.check.outputs.has_results == 'true'
- uses:
scacap/action-surefire-report@5609ce4db72c09db044803b344a8968fd1f315da
+ uses:
EnricoMi/publish-unit-test-result-action@d0a4676d0e0b938bc201470d88276b7c74c712b3
# v2.24.0
with:
check_name: Report test results
- github_token: ${{ secrets.GITHUB_TOKEN }}
- report_paths: "**/target/test-reports/*.xml"
+ files: "**/target/test-reports/*.xml"
+ comment_mode: off
+ large_files: true
commit: ${{ github.event.workflow_run.head_commit.id }}
+ github_token: ${{ secrets.GITHUB_TOKEN }}
+ compare_to_earlier_commit: false
+ test_changes_limit: 0
+ check_run_annotations: none
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
