Skip to content

[SPARK-55383][INFRA] Only send test report to codecov in coverage run#54168

Open
gaogaotiantian wants to merge 3 commits intoapache:masterfrom
gaogaotiantian:revert-test-report
Open

[SPARK-55383][INFRA] Only send test report to codecov in coverage run#54168
gaogaotiantian wants to merge 3 commits intoapache:masterfrom
gaogaotiantian:revert-test-report

Conversation

@gaogaotiantian
Copy link
Contributor

What changes were proposed in this pull request?

Instead of trying to send test report on all main branch commits, we only send it for coverage run.

Also removed an unused coverage token.

Why are the changes needed?

We tried the test dashboard of codecov - https://app.codecov.io/github/apache/spark/tests but it's not super great.

  • It can't parse the xml file to locate the failed test case. Instead it records a failure for the full test suite.
  • It does not have a timeline so we don't really know if the test failed recently.
  • The deduplication is bad.
  • It messed up our coverage report - because the backend can't split the test and coverage report into their own category.

However it's not completely useless. The average time of tests actually helps us to locate slowest tests. It does not mess up with coverage report, as long as for each commit we have both the coverage report and test report.

So, we can do a report only on coverage run - single report each day for both coverage and test speed.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

CI should pass, hopefully coverage can recover.

Was this patch authored or co-authored using generative AI tooling?

No.

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

JIRA Issue Information

=== Improvement SPARK-55383 ===
Summary: Only send test report to codecov in coverage run
Assignee: None
Status: Open
Affected: ["4.2.0"]


This comment was automatically generated by GitHub Actions

@github-actions github-actions bot added the INFRA label Feb 6, 2026
@Yicong-Huang
Copy link
Contributor

need coverage back!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants