Skip to content

ci: authenticate Codecov uploads with OIDC - #929

Merged
paleolimbot merged 1 commit into
apache:mainfrom
Query-farm:ci/codecov-oidc
Sep 2, 2026
Merged

ci: authenticate Codecov uploads with OIDC#929
paleolimbot merged 1 commit into
apache:mainfrom
Query-farm:ci/codecov-oidc

Conversation

@rustyconover

Copy link
Copy Markdown
Contributor

The coverage workflow on main currently completes successfully even though Codecov rejects its upload with Token required because branch is protected. This leaves the repository coverage baseline stale and causes PR patch reports to compare against an old commit.

Configure the Codecov action to use GitHub OIDC, grant the required permissions, and fail the job when an upload is rejected. Fork pull requests continue to use Codecov tokenless uploads because the action detects fork events and skips OIDC for them.

The failed protected-branch upload is visible in https://github.com/apache/arrow-nanoarrow/actions/runs/32611406823.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.30%. Comparing base (f11d517) to head (8b8671e).
⚠️ Report is 81 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #929      +/-   ##
==========================================
- Coverage   79.96%   79.30%   -0.67%     
==========================================
  Files         105      106       +1     
  Lines       15461    16223     +762     
  Branches     1738     1882     +144     
==========================================
+ Hits        12364    12865     +501     
- Misses       1998     2177     +179     
- Partials     1099     1181      +82     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@paleolimbot paleolimbot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this...the codecov setup is wildly out of date.

@paleolimbot
paleolimbot merged commit f55f85b into apache:main Sep 2, 2026
6 checks passed
@rustyconover
rustyconover deleted the ci/codecov-oidc branch September 2, 2026 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants