Skip to content

chore(ci): improve JUnit upload speed by using the standalone binary#1376

Merged
dd-mergequeue[bot] merged 1 commit into
mainfrom
juan-fernandez/improve-datadog-ci-upload
Dec 1, 2025
Merged

chore(ci): improve JUnit upload speed by using the standalone binary#1376
dd-mergequeue[bot] merged 1 commit into
mainfrom
juan-fernandez/improve-datadog-ci-upload

Conversation

@juan-fernandez
Copy link
Copy Markdown
Contributor

@juan-fernandez juan-fernandez commented Nov 28, 2025

What does this PR do?

Use datadog-ci's standalone binaries to upload JUnit XML reports to Test Optimization, rather than using datadog/junit-upload-github-action, which uses npx under the hood, which can be slow, especially in windows machines.

The version is pinned to 4.2.2 and I've added the checksum verification. See binaries and their checksums in https://github.com/DataDog/datadog-ci/releases/tag/v4.2.2

Motivation

Same as DataDog/dd-trace-rs#125

Before:
From ~2 min 40 secs https://github.com/DataDog/libdatadog/actions/runs/19766310102/job/56639862627

After:
To ~3 secs https://github.com/DataDog/libdatadog/actions/runs/19766740267/job/56641239510?pr=1376

How to test the change?

  • Check that jobs are passing
  • Check that test results are reported to Test Optimization
  • Check that job is faster

@juan-fernandez juan-fernandez marked this pull request as ready for review November 28, 2025 14:46
@juan-fernandez juan-fernandez requested a review from a team as a code owner November 28, 2025 14:46
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.22%. Comparing base (5079362) to head (d13f5d5).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1376      +/-   ##
==========================================
+ Coverage   71.20%   71.22%   +0.02%     
==========================================
  Files         395      395              
  Lines       63292    63292              
==========================================
+ Hits        45066    45079      +13     
+ Misses      18226    18213      -13     
Components Coverage Δ
libdd-crashtracker 59.90% <ø> (-0.02%) ⬇️
libdd-crashtracker-ffi 15.31% <ø> (ø)
libdd-alloc 98.73% <ø> (ø)
libdd-data-pipeline 86.00% <ø> (ø)
libdd-data-pipeline-ffi 77.63% <ø> (ø)
libdd-common 80.93% <ø> (ø)
libdd-common-ffi 73.90% <ø> (ø)
libdd-telemetry 60.02% <ø> (ø)
libdd-telemetry-ffi 21.24% <ø> (ø)
libdd-dogstatsd-client 83.75% <ø> (ø)
datadog-ipc 82.61% <ø> (ø)
libdd-profiling 79.73% <ø> (ø)
libdd-profiling-ffi 62.13% <ø> (ø)
datadog-sidecar 36.98% <ø> (ø)
datdog-sidecar-ffi 16.40% <ø> (ø)
spawn-worker 55.18% <ø> (ø)
libdd-tinybytes 92.44% <ø> (ø)
libdd-trace-normalization 98.24% <ø> (ø)
datadog-trace-obfuscation 94.17% <ø> (ø)
libdd-trace-protobuf 59.65% <ø> (ø)
libdd-trace-utils 90.06% <ø> (ø)
datadog-tracer-flare 62.06% <ø> (ø)
libdd-log 75.57% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bantonsson
Copy link
Copy Markdown
Contributor

/merge

@dd-devflow-routing-codex
Copy link
Copy Markdown

dd-devflow-routing-codex Bot commented Dec 1, 2025

View all feedbacks in Devflow UI.

2025-12-01 14:57:07 UTC ℹ️ Start processing command /merge


2025-12-01 14:57:12 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in main is approximately 34m (p90).


2025-12-01 15:20:43 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue dd-mergequeue Bot merged commit dea3604 into main Dec 1, 2025
38 of 39 checks passed
@dd-mergequeue dd-mergequeue Bot deleted the juan-fernandez/improve-datadog-ci-upload branch December 1, 2025 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants