Skip to content

Commit 5524471

Browse files
gaogaotiantianhuangxiaopingRD
authored andcommitted
[SPARK-54458][INFRA] Use official download-artifact to fix the test_report action
### What changes were proposed in this pull request? We should fix the `test_report` workflow - it barely works. * Used the official download-artifacts to replace the 3rd party one * Unpinned the specific version of test report action * Used explicit permissions for the action ### Why are the changes needed? The problem is the 3rd party action used can't deal with non-zip files. In the orignal [PR](apache#29333) it was mentioned that the official `download-artifact` action should be used when it can download from other workflows. We should try to use that. ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? CI ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#53167 from gaogaotiantian/fix-report-action. Authored-by: Tian Gao <gaogaotiantian@hotmail.com> Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
1 parent ee34122 commit 5524471

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

.github/workflows/test_report.yml

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,16 +28,19 @@ jobs:
2828
test_report:
2929
if: github.event.workflow_run.conclusion != 'skipped'
3030
runs-on: ubuntu-latest
31+
permissions:
32+
actions: read
33+
checks: write
34+
contents: read
3135
steps:
3236
- name: Download test results to report
33-
uses: dawidd6/action-download-artifact@bf251b5aa9c2f7eeb574a96ee720e24f801b7c11 # pin @v6
37+
uses: actions/download-artifact@v5
3438
with:
35-
github_token: ${{ secrets.GITHUB_TOKEN }}
36-
workflow: ${{ github.event.workflow_run.workflow_id }}
37-
commit: ${{ github.event.workflow_run.head_commit.id }}
38-
workflow_conclusion: completed
39+
github-token: ${{ secrets.GITHUB_TOKEN }}
40+
run-id: ${{ github.event.workflow_run.id }}
41+
pattern: "test-*"
3942
- name: Publish test report
40-
uses: scacap/action-surefire-report@a2911bd1a4412ec18dde2d93b1758b3e56d2a880 # pin @v1.8.0
43+
uses: scacap/action-surefire-report@v1
4144
with:
4245
check_name: Report test results
4346
github_token: ${{ secrets.GITHUB_TOKEN }}

0 commit comments

Comments
 (0)