This is an automated email from the ASF dual-hosted git repository.
gurwls223 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 71f514774bb1 [SPARK-55383][INFRA] Only send test report to codecov in
coverage run
71f514774bb1 is described below
commit 71f514774bb19b08331da863cd4c3ea34fc9a97b
Author: Tian Gao <[email protected]>
AuthorDate: Mon Feb 9 07:12:11 2026 +0900
[SPARK-55383][INFRA] Only send test report to codecov in coverage run
### 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.
Closes #54168 from gaogaotiantian/revert-test-report.
Authored-by: Tian Gao <[email protected]>
Signed-off-by: Hyukjin Kwon <[email protected]>
---
.github/workflows/build_and_test.yml | 2 +-
.github/workflows/build_main.yml | 2 --
.github/workflows/python_hosted_runner_test.yml | 10 ----------
3 files changed, 1 insertion(+), 13 deletions(-)
diff --git a/.github/workflows/build_and_test.yml
b/.github/workflows/build_and_test.yml
index 996124ddccc5..8c332d1fae9b 100644
--- a/.github/workflows/build_and_test.yml
+++ b/.github/workflows/build_and_test.yml
@@ -674,7 +674,7 @@ jobs:
verbose: true
- name: Upload test results to Codecov
env: ${{ fromJSON(inputs.envs) }}
- if: (!cancelled()) && github.repository == 'apache/spark'
+ if: fromJSON(inputs.envs).PYSPARK_CODECOV == 'true'
uses: codecov/codecov-action@v5
with:
report_type: 'test_results'
diff --git a/.github/workflows/build_main.yml b/.github/workflows/build_main.yml
index 97c19d92eeae..9ef52f326375 100644
--- a/.github/workflows/build_main.yml
+++ b/.github/workflows/build_main.yml
@@ -30,5 +30,3 @@ jobs:
packages: write
name: Run
uses: ./.github/workflows/build_and_test.yml
- secrets:
- codecov_token: ${{ secrets.CODECOV_TOKEN }}
diff --git a/.github/workflows/python_hosted_runner_test.yml
b/.github/workflows/python_hosted_runner_test.yml
index 7ab56ce5d7e5..b32d2d1a5628 100644
--- a/.github/workflows/python_hosted_runner_test.yml
+++ b/.github/workflows/python_hosted_runner_test.yml
@@ -167,16 +167,6 @@ jobs:
echo "Python Packaging Tests Enabled!"
fi
./dev/run-tests --parallelism 1 --modules "$MODULES_TO_TEST"
--python-executables "$PYTHON_TO_TEST"
- - name: Upload test results to Codecov
- env: ${{ fromJSON(inputs.envs) }}
- if: (!cancelled()) && github.repository == 'apache/spark'
- uses: codecov/codecov-action@v5
- with:
- report_type: 'test_results'
- files: '**/target/test-reports/*.xml'
- flags: ${{ env.PYTHON_TO_TEST }}-${{ inputs.branch }}-${{ inputs.os
}}
- name: PySpark-Test-Results
- token: ${{ secrets.codecov_token }}
- name: Upload test results to report
env: ${{ fromJSON(inputs.envs) }}
if: always()
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
