Add Datadog code coverage upload#324
Draft
ManuelPalenzuelaDD wants to merge 1 commit intoDataDog:masterfrom
Draft
Add Datadog code coverage upload#324ManuelPalenzuelaDD wants to merge 1 commit intoDataDog:masterfrom
ManuelPalenzuelaDD wants to merge 1 commit intoDataDog:masterfrom
Conversation
Side-by-side coverage reporting: adds Datadog coverage upload to the GitLab CI coverage job alongside the existing Codecov upload. Uses datadog-ci CLI to upload the LCOV report to Datadog Code Coverage. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
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.
Context
We're migrating Datadog repositories from Codecov to Datadog Code Coverage — our own code coverage solution. This is part of an org-wide effort to dogfood our product and consolidate coverage reporting into Datadog.
This PR does not remove Codecov. It adds Datadog coverage upload alongside the existing Codecov upload so we can run both in parallel and verify that coverage numbers match before making any switch.
What this PR does
Adds a single step to the GitLab CI
coveragejob (in.gitlab/build-and-test-fast.yml) that uploads the existing LCOV report to Datadog after the Codecov upload:datadog-ciCLI.musl-build/coverage.lcovto Datadog Code CoverageThe existing Codecov upload and all other CI behavior is completely untouched.
Risk
Very low. The upload step:
make coverage(including the existing Codecov upload) completes successfully|| trueso a Datadog upload failure will never break the pipelineRequirements
DD_API_KEYneeds to be available as a CI/CD variable in the GitLab project. If it's not set yet, the upload will silently fail (due to|| true) with no impact on the pipeline.Next steps
Once this lands and a coverage pipeline runs on
master:If you have any questions about the migration, feel free to ask — happy to provide more context!
🤖 Generated with Claude Code