This is an automated email from the ASF dual-hosted git repository.
sarutak pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new f5e990005dde [SPARK-57692][INFRA] Skip test report when no test
results exist
f5e990005dde is described below
commit f5e990005dde3aa24241b97f4498449ca68e94a5
Author: Kousuke Saruta <[email protected]>
AuthorDate: Sat Jun 27 18:55:56 2026 +0900
[SPARK-57692][INFRA] Skip test report when no test results exist
### What changes were proposed in this pull request?
This PR adds a check in `test_report.yml` to skip the test report
publishing step when no test result XML files are found after downloading
artifacts.
### Why are the changes needed?
When a commit only modifies files in the `dev-tools` module (e.g.,
`LICENSE-binary`), no build or tests are executed. However, the "Report test
results" workflow still triggers on the Build workflow's completion and fails
with "No test results found!" because there are no test artifacts to report on.
* https://github.com/apache/spark/runs/83376805611
* https://github.com/apache/spark/runs/83376713136
* https://github.com/apache/spark/runs/83376067066
SPARK-55594 (78ab6e2e2f) previously addressed the same "No test results
found!" error for the `pages.yml` case by filtering out that specific workflow
path. However, the `dev-tools` module case was not covered. This PR takes a
more general approach by checking for the actual presence of test result files,
which handles all cases where tests are legitimately not executed, including
any future ones.
Note: The existing `pages.yml` filter in the job's `if` condition is
intentionally kept. It avoids spinning up the job entirely for a known no-op
case, saving runner resources.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Self review. Also, this will be validated by observing that commits which
only modify files in the `dev-tools` module (e.g., `LICENSE-binary`,
`README.md`, `.gitignore`) no longer produce a failed "Report test results"
check.
### Was this patch authored or co-authored using generative AI tooling?
Kiro CLI / Claude
Closes #56769 from sarutak/fix-test-report-no-results.
Authored-by: Kousuke Saruta <[email protected]>
Signed-off-by: Kousuke Saruta <[email protected]>
---
.github/workflows/test_report.yml | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/.github/workflows/test_report.yml
b/.github/workflows/test_report.yml
index bcc6b5220c23..80e57b38d275 100644
--- a/.github/workflows/test_report.yml
+++ b/.github/workflows/test_report.yml
@@ -41,7 +41,17 @@ 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
with:
check_name: Report test results
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
