Skip to content

ci: remove Codecov coverage upload#2094

Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
mainfrom
dd/ci/remove-codecov
Jun 8, 2026
Merged

ci: remove Codecov coverage upload#2094
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
mainfrom
dd/ci/remove-codecov

Conversation

@ManuelPalenzuelaDD

@ManuelPalenzuelaDD ManuelPalenzuelaDD commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Removes the Codecov coverage upload now that the migration to Datadog Code Coverage is complete.

  • Removed the "Upload coverage to Codecov" step (and CODECOV_TOKEN usage) from the coverage workflow, leaving the Datadog coverage upload as the sole uploader.
  • Deleted .codecov.yml.
  • Removed the .codecov.yml entry from .github/CODEOWNERS.

Motivation

The Datadog Code Coverage migration (started in #1718) is done and verified, and a recent coverage workflow run failed on the Codecov GPG key. Codecov is no longer needed, and the team has no remaining attachment to it.

Additional Notes

The Datadog upload step is left unchanged. The CODECOV_TOKEN secret can be removed separately at the repository settings level.

How to test the change?

Trigger the Coverage workflow (PR/push/schedule) and confirm it runs without the Codecov step and that the Datadog coverage upload still succeeds.

PR by Bits - View session in Datadog

Comment @DataDog to request changes

@datadog-datadog-prod-us1 @ManuelPalenzuelaDD
ci: remove Codecov coverage upload
88c0243 
Co-authored-by: ManuelPalenzuelaDD <144776385+ManuelPalenzuelaDD@users.noreply.github.com>
@datadog-datadog-prod-us1-2

datadog-datadog-prod-us1-2 Bot commented Jun 8, 2026 •
edited
Loading

Copy link
Copy Markdown

View session in Datadog

Bits Code status: ✅ Done

Comment @DataDog to request changes

@ManuelPalenzuelaDD ManuelPalenzuelaDD requested a review from a team as a code owner June 8, 2026 13:18
@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

I can only run on private repositories.

chatgpt-codex-connector[bot]
chatgpt-codex-connector Bot reviewed Jun 8, 2026
View reviewed changes

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 88c0243f5d

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread .github/workflows/coverage.yml
Comment on lines 58 to 60
- name: Upload coverage to Datadog
if: always()
continue-on-error: true

@chatgpt-codex-connector chatgpt-codex-connector Bot Jun 8, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Preserve a failing signal for coverage uploads

With the Codecov step removed, the only remaining publisher in this workflow is the Datadog step below, and it is still marked continue-on-error: true. In the inspected coverage.yml workflow, that means pushes, PRs, and scheduled runs can now succeed after generating lcov.info even when the Datadog upload fails or the DATADOG_API_KEY secret is missing, so coverage can silently stop being reported. If Datadog is now the sole uploader, make this step required or add another required verification of the upload.

Useful? React with 👍 / 👎.

@datadog-datadog-prod-us1-2

datadog-datadog-prod-us1-2 Bot commented Jun 8, 2026

Copy link
Copy Markdown

Tests

🎉 All green!

🧪 All tests passed
❄️ No new flaky tests detected

🎯 Code Coverage (details)
• Patch Coverage: 100.00%
• Overall Coverage: 73.57% (+0.01%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 88c0243 | Docs | Datadog PR Page | Give us feedback!

@dd-octo-sts

dd-octo-sts Bot commented Jun 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Artifact Size Benchmark Report

aarch64-alpine-linux-musl
Artifact Baseline Commit Change
/aarch64-alpine-linux-musl/lib/libdatadog_profiling.so 7.70 MB 7.70 MB 0% (0 B) 👌
/aarch64-alpine-linux-musl/lib/libdatadog_profiling.a 83.48 MB 83.48 MB 0% (0 B) 👌
aarch64-unknown-linux-gnu
Artifact Baseline Commit Change
/aarch64-unknown-linux-gnu/lib/libdatadog_profiling.a 94.59 MB 94.59 MB 0% (0 B) 👌
/aarch64-unknown-linux-gnu/lib/libdatadog_profiling.so 10.34 MB 10.34 MB 0% (0 B) 👌
libdatadog-x64-windows
Artifact Baseline Commit Change
/libdatadog-x64-windows/debug/dynamic/datadog_profiling_ffi.dll 24.71 MB 24.71 MB 0% (0 B) 👌
/libdatadog-x64-windows/debug/dynamic/datadog_profiling_ffi.lib 86.89 KB 86.89 KB 0% (0 B) 👌
/libdatadog-x64-windows/debug/dynamic/datadog_profiling_ffi.pdb 179.83 MB 179.85 MB +0% (+16.00 KB) 👌
/libdatadog-x64-windows/debug/static/datadog_profiling_ffi.lib 923.21 MB 923.21 MB 0% (0 B) 👌
/libdatadog-x64-windows/release/dynamic/datadog_profiling_ffi.dll 8.09 MB 8.09 MB 0% (0 B) 👌
/libdatadog-x64-windows/release/dynamic/datadog_profiling_ffi.lib 86.89 KB 86.89 KB 0% (0 B) 👌
/libdatadog-x64-windows/release/dynamic/datadog_profiling_ffi.pdb 23.91 MB 23.91 MB 0% (0 B) 👌
/libdatadog-x64-windows/release/static/datadog_profiling_ffi.lib 47.73 MB 47.73 MB 0% (0 B) 👌
libdatadog-x86-windows
Artifact Baseline Commit Change
/libdatadog-x86-windows/debug/dynamic/datadog_profiling_ffi.dll 21.42 MB 21.42 MB 0% (0 B) 👌
/libdatadog-x86-windows/debug/dynamic/datadog_profiling_ffi.lib 88.26 KB 88.26 KB 0% (0 B) 👌
/libdatadog-x86-windows/debug/dynamic/datadog_profiling_ffi.pdb 183.83 MB 183.85 MB +.01% (+24.00 KB) 🔍
/libdatadog-x86-windows/debug/static/datadog_profiling_ffi.lib 915.97 MB 915.97 MB 0% (0 B) 👌
/libdatadog-x86-windows/release/dynamic/datadog_profiling_ffi.dll 6.25 MB 6.25 MB 0% (0 B) 👌
/libdatadog-x86-windows/release/dynamic/datadog_profiling_ffi.lib 88.26 KB 88.26 KB 0% (0 B) 👌
/libdatadog-x86-windows/release/dynamic/datadog_profiling_ffi.pdb 25.63 MB 25.63 MB 0% (0 B) 👌
/libdatadog-x86-windows/release/static/datadog_profiling_ffi.lib 45.35 MB 45.35 MB 0% (0 B) 👌
x86_64-alpine-linux-musl
Artifact Baseline Commit Change
/x86_64-alpine-linux-musl/lib/libdatadog_profiling.a 74.45 MB 74.45 MB 0% (0 B) 👌
/x86_64-alpine-linux-musl/lib/libdatadog_profiling.so 8.58 MB 8.58 MB 0% (0 B) 👌
x86_64-unknown-linux-gnu
Artifact Baseline Commit Change
/x86_64-unknown-linux-gnu/lib/libdatadog_profiling.a 89.87 MB 89.87 MB 0% (0 B) 👌
/x86_64-unknown-linux-gnu/lib/libdatadog_profiling.so 10.43 MB 10.43 MB 0% (0 B) 👌

@gh-worker-dd-mergequeue-cf854d gh-worker-dd-mergequeue-cf854d Bot merged commit bb528a2 into main Jun 8, 2026
125 checks passed
@gh-worker-dd-mergequeue-cf854d gh-worker-dd-mergequeue-cf854d Bot deleted the dd/ci/remove-codecov branch June 8, 2026 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chatgpt-codex-connector chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments

@iunanua iunanua iunanua approved these changes

Assignees

No one assigned

Labels

Bits AI ci-build mergequeue-status: done

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ManuelPalenzuelaDD @iunanua