Add Datadog code coverage upload#799
Merged
Merged
Conversation
|
🎯 Code Coverage (details) 🔗 Commit SHA: 22e0b39 | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback! |
kakkoyun
approved these changes
Mar 12, 2026
github-merge-queue Bot
pushed a commit
that referenced
this pull request
Mar 13, 2026
## Summary Adds `code-coverage.datadog.yml` to configure Datadog Code Coverage ignore rules. This is a prerequisite for the coverage upload PR (#799) — the Datadog coverage reducer discovers this config file from the default branch via GitDB, so it must be merged to `main` before coverage uploads will apply ignore rules correctly. ### Ignore rules Excludes the following from coverage reporting (matching the existing Codecov behavior): - `samples/` and `_samples/` — test fixture directories - `_tools/` — build tooling - `dot_ast.proxies.go` — generated proxy file Includes both repo-relative paths and glob patterns to handle the Go module path resolver's suffix-matching limitation (where `_samples` Go module maps to `samples/` directory on disk). ### Context Part of the Codecov → Datadog Code Coverage migration. See #799 for the full upload integration. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
ManuelPalenzuelaDD
force-pushed
the
add-datadog-coverage
branch
from
March 16, 2026 16:52
fa3de83 to
eb84a0b
Compare
Mirror Codecov upload exactly: same per-flag matrix, same files, same flags. Simplify code-coverage.datadog.yml to match codecov.yml ignore paths directly. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
nikita-tkachenko-datadog
force-pushed
the
add-datadog-coverage
branch
from
March 20, 2026 13:03
10a77f5 to
22e0b39
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #799 +/- ##
==========================================
+ Coverage 65.72% 69.60% +3.88%
==========================================
Files 113 116 +3
Lines 7926 6900 -1026
==========================================
- Hits 5209 4803 -406
+ Misses 2192 1547 -645
- Partials 525 550 +25
🚀 New features to boost your workflow:
|
nikita-tkachenko-datadog
force-pushed
the
add-datadog-coverage
branch
from
March 20, 2026 14:23
6082620 to
22e0b39
Compare
nikita-tkachenko-datadog
marked this pull request as ready for review
March 20, 2026 14:36
nikita-tkachenko-datadog
enabled auto-merge
March 20, 2026 14:37
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
We're migrating Datadog repositories from Codecov to Datadog Code Coverage for tracking test coverage. This PR is the first step: it adds a Datadog coverage upload alongside the existing Codecov upload so we can run both systems in parallel and verify parity before switching over.
Changes
DataDog/coverage-upload-github-action@v1upload steps to thevalidate.ymlworkflow, mirroring the per-flag matrix strategy used by Codecovcode-coverage.datadog.ymlwith ignore paths matchingcodecov.yml(samples/,_tools/, auto-generated proxy files)DD_CI_API_KEYsecret (already available in this repo)continue-on-error: trueso neither blocks CIWhy are we doing this?
As part of a company-wide effort, we're consolidating code coverage reporting into Datadog's own Code Coverage product. This gives us:
Validation
CI needs to run to produce coverage data. Previous comparison on commit
e708754:The 2.11% gap is likely due to Go coverprofile statement-block-to-line mapping differences between the two systems. Still under investigation.
Next steps (not in this PR)
Once this PR is merged and we've confirmed Datadog coverage is stable over several commits:
CODECOV_TOKENsecretcodecov.ymlcode-coverage.datadog.ymlNo action needed from reviewers beyond normal review
This is a low-risk, additive change. The new upload steps run independently of the existing CI pipeline and cannot cause test failures.