ci: publish Rust coverage to Codecov#490
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 12cd2be58c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Signed-off-by: Jeremi Joslin <[email protected]>
Signed-off-by: Jeremi Joslin <[email protected]>
12cd2be to
884396f
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 884396fa52
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Pull Request
Summary
Integrate Codecov with the Rust coverage surfaces while keeping expensive coverage instrumentation off the pull-request path:
mainand manual runsplatformandnotary-postgresflags with carryforward enabledUploads use Codecov Action v7.0.0 pinned to commit
fb8b3582c8e4def4969c97caa2f19720cb33a72f, Codecov CLI v11.3.1, explicit report paths, and OIDC authentication. OIDC permission is scoped to the main/manual coverage job; pull-request conformance retains read-only repository permissions.Impact
This does not add another Rust build or test pass to pull requests. Platform LCOV export took about 0.2 seconds and the verified Codecov download/upload step took about 2.2 seconds in the initial run. Notary pull-request conformance no longer installs
cargo-llvm-cov, compiles with coverage instrumentation, produces LCOV, or uploads coverage.Comprehensive cross-workspace coverage is intentionally left for a follow-up nightly workflow so it does not consume the pull-request critical path.
Checks
actionlintpython3 .github/scripts/test_ci_changes.pyPOST https://api.codecov.io/validatecargo llvm-cov --locked -p registry-config-report -p 'registry-platform-*' --all-features --fail-under-lines 80cargo llvm-cov report --locked -p registry-config-report -p 'registry-platform-*' --lcov --output-path target/platform-coverage/platform.lcovgit diff --checkBefore splitting the PR path, GitHub Actions verified that the platform report and PostgreSQL 16, 17, and 18 reports were all accepted and merged by Codecov under OIDC. Initial flag baselines were platform 92.06% and PostgreSQL-only Notary 20.13%.
Notes
.github/workflowsand rootcodecov.yml).CODECOV_TOKENsecret is required.DCO
Signed-off-bytrailer.