[SPARK-55286][INFRA] Add test summary to GitHub Actions for better failure visibility#54069
Closed
yaooqinn wants to merge 1 commit intoapache:masterfrom
Closed
[SPARK-55286][INFRA] Add test summary to GitHub Actions for better failure visibility#54069yaooqinn wants to merge 1 commit intoapache:masterfrom
yaooqinn wants to merge 1 commit intoapache:masterfrom
Conversation
…ilure visibility Add test-summary/action@v2 to display test failures directly in the GitHub Actions job summary. This makes it easier to identify failures without scrolling through verbose logs. Jobs updated: build, pyspark, sparkr, tpcds, docker-integration-tests
JIRA Issue Information=== Improvement SPARK-55286 === This comment was automatically generated by GitHub Actions |
Member
Author
|
https://github.com/yaooqinn/spark/actions/runs/21505022749#summary-61961633818 After this change, you can view the test failure in summary instead of searching the console log. |
There was a problem hiding this comment.
Pull request overview
This PR enhances GitHub Actions CI visibility by adding the test-summary/action@v2 to display test failures directly in workflow job summaries, eliminating the need to scroll through verbose logs.
Changes:
- Added
test-summary/action@v2step to 5 test jobs (build, pyspark, sparkr, tpcds-1g, docker-integration-tests) - Configured action to parse JUnit XML reports from
**/target/test-reports/*.xmland**/target/surefire-reports/*.xml - Used
if: always()condition to ensure summaries are generated even when tests fail
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
HyukjinKwon
approved these changes
Feb 1, 2026
Member
|
Merged to master. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changes were proposed in this pull request?
This PR adds
test-summary/action@v2to GitHub Actions workflows to display test failures directly in the job summary.Jobs updated:
build- Scala/Java unit testspyspark- Python testssparkr- R teststpcds- TPC-DS benchmark testsdocker-integration-tests- Docker integration testsThe action parses JUnit XML test reports and generates a summary table showing:
Why are the changes needed?
Currently, GitHub Actions CI generates verbose logs that make it hard to find test failures quickly. Developers have to scroll through extensive log output to identify which tests failed.
With this change, test failures appear directly in the GitHub Actions workflow summary, making it easy to identify failures at a glance.
Example output:
Does this PR introduce any user-facing change?
No. This only affects the CI/CD workflow display.
How was this patch tested?
test-summary/actionis a well-maintained GitHub Action with 1000+ starsif: always()to ensure summary is generated even when tests failWas this patch authored or co-authored using generative AI tooling?
Generated-by: GitHub Copilot