Skip to content

Pin datadog-ci for junit upload#8078

Merged
BridgeAR merged 2 commits intomasterfrom
dd/fix/test-optimization-junit-action
Apr 23, 2026
Merged

Pin datadog-ci for junit upload#8078
BridgeAR merged 2 commits intomasterfrom
dd/fix/test-optimization-junit-action

Conversation

@juan-fernandez
Copy link
Copy Markdown
Collaborator

@juan-fernandez juan-fernandez commented Apr 23, 2026

What does this PR do?

Pins the datadog-ci version used by DataDog/junit-upload-github-action to 5.13.1 in .github/actions/push_to_test_optimization/action.yml and .github/workflows/platform.yml.

Motivation

Some jobs intermittently fail in the push_to_test_optimization step after the upgrade to DataDog/junit-upload-github-action v3.0.0. Example in https://github.com/DataDog/dd-trace-js/actions/runs/24780644771/job/72510523178?pr=8060.
The failure happens inside DataDog/install-datadog-ci-github-action while resolving the floating datadog-ci version v5, where jq receives a non-release JSON payload and crashes with Cannot index string with string "prerelease". Pinning datadog-ci to 5.13.1 avoids that floating-version GitHub release lookup while preserving the v3 junit upload action behavior.

Co-authored-by: juan-fernandez <22798219+juan-fernandez@users.noreply.github.com>
@datadog-datadog-prod-us1-2
Copy link
Copy Markdown

datadog-datadog-prod-us1-2 Bot commented Apr 23, 2026

View session in Datadog

Bits Dev status: ✅ Done

Comment @DataDog to request changes

@datadog-prod-us1-4
Copy link
Copy Markdown

I can only run on private repositories.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 23, 2026

Overall package size

Self size: 5.56 MB
Deduped: 6.4 MB
No deduping: 6.4 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 3.0.1 | 82.56 kB | 817.39 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |

🤖 This report was automatically generated by heaviest-objects-in-the-universe

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.74%. Comparing base (8cd7c26) to head (4a3bf8b).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8078      +/-   ##
==========================================
- Coverage   73.77%   73.74%   -0.04%     
==========================================
  Files         782      782              
  Lines       36339    36354      +15     
==========================================
- Hits        26809    26808       -1     
- Misses       9530     9546      +16     
Flag Coverage Δ
aiguard-macos 36.58% <ø> (-0.09%) ⬇️
aiguard-ubuntu 36.69% <ø> (-0.09%) ⬇️
aiguard-windows 36.49% <ø> (-0.09%) ⬇️
apm-capabilities-tracing-macos 48.42% <ø> (ø)
apm-capabilities-tracing-ubuntu 48.46% <ø> (ø)
apm-capabilities-tracing-windows 48.21% <ø> (-0.04%) ⬇️
apm-integrations-child-process 36.17% <ø> (-0.10%) ⬇️
apm-integrations-couchbase-18 35.17% <ø> (-0.09%) ⬇️
apm-integrations-couchbase-eol 35.23% <ø> (-0.12%) ⬇️
apm-integrations-oracledb 35.22% <ø> (-0.09%) ⬇️
appsec-express 52.89% <ø> (-0.07%) ⬇️
appsec-fastify 49.37% <ø> (-0.07%) ⬇️
appsec-graphql 49.63% <ø> (-0.17%) ⬇️
appsec-kafka 42.16% <ø> (-0.09%) ⬇️
appsec-ldapjs 41.48% <ø> (-0.08%) ⬇️
appsec-lodash 41.51% <ø> (-0.08%) ⬇️
appsec-macos 56.96% <ø> (-0.07%) ⬇️
appsec-mongodb-core 45.82% <ø> (-0.07%) ⬇️
appsec-mongoose 46.70% <ø> (-0.07%) ⬇️
appsec-mysql 48.84% <ø> (-0.08%) ⬇️
appsec-node-serialize 40.69% <ø> (-0.08%) ⬇️
appsec-passport 44.67% <ø> (-0.08%) ⬇️
appsec-postgres 48.47% <ø> (-0.07%) ⬇️
appsec-sourcing 40.18% <ø> (-0.08%) ⬇️
appsec-stripe 42.40% <ø> (-0.20%) ⬇️
appsec-template 40.85% <ø> (-0.08%) ⬇️
appsec-ubuntu 57.03% <ø> (-0.07%) ⬇️
appsec-windows 56.83% <ø> (-0.07%) ⬇️
instrumentations-instrumentation-bluebird 29.86% <ø> (-0.10%) ⬇️
instrumentations-instrumentation-body-parser 37.76% <ø> (-0.09%) ⬇️
instrumentations-instrumentation-child_process 35.54% <ø> (-0.10%) ⬇️
instrumentations-instrumentation-cookie-parser 31.78% <ø> (-0.09%) ⬇️
instrumentations-instrumentation-express 32.00% <ø> (-0.09%) ⬇️
instrumentations-instrumentation-express-mongo-sanitize 31.91% <ø> (-0.09%) ⬇️
instrumentations-instrumentation-express-session 37.39% <ø> (-0.09%) ⬇️
instrumentations-instrumentation-fs 29.53% <ø> (-0.10%) ⬇️
instrumentations-instrumentation-generic-pool 30.91% <ø> (ø)
instrumentations-instrumentation-http 37.00% <ø> (-0.09%) ⬇️
instrumentations-instrumentation-knex 29.83% <ø> (-0.09%) ⬇️
instrumentations-instrumentation-light-my-request 37.33% <ø> (-0.09%) ⬇️
instrumentations-instrumentation-mongoose 30.94% <ø> (-0.09%) ⬇️
instrumentations-instrumentation-multer 37.53% <ø> (-0.09%) ⬇️
instrumentations-instrumentation-mysql2 35.52% <ø> (-0.09%) ⬇️
instrumentations-instrumentation-passport 41.27% <ø> (-0.08%) ⬇️
instrumentations-instrumentation-passport-http 40.96% <ø> (-0.08%) ⬇️
instrumentations-instrumentation-passport-local 41.46% <ø> (-0.08%) ⬇️
instrumentations-instrumentation-pg 35.04% <ø> (-0.09%) ⬇️
instrumentations-instrumentation-promise 29.80% <ø> (-0.10%) ⬇️
instrumentations-instrumentation-promise-js 29.80% <ø> (-0.10%) ⬇️
instrumentations-instrumentation-q 29.84% <ø> (-0.10%) ⬇️
instrumentations-instrumentation-url 29.79% <ø> (-0.10%) ⬇️
instrumentations-instrumentation-when 29.82% <ø> (-0.10%) ⬇️
llmobs-ai 38.49% <ø> (-0.09%) ⬇️
llmobs-anthropic 37.94% <ø> (-0.09%) ⬇️
llmobs-bedrock 37.17% <ø> (-0.08%) ⬇️
llmobs-google-genai 37.61% <ø> (-0.08%) ⬇️
llmobs-langchain 37.07% <ø> (-0.18%) ⬇️
llmobs-openai 41.31% <ø> (-0.08%) ⬇️
llmobs-vertex-ai 37.80% <ø> (-0.09%) ⬇️
platform-core 29.90% <ø> (ø)
platform-esbuild 32.72% <ø> (ø)
platform-instrumentations-misc 40.08% <ø> (ø)
platform-shimmer 35.62% <ø> (ø)
platform-unit-guardrails 31.28% <ø> (ø)
platform-webpack 20.79% <ø> (-0.02%) ⬇️
plugins-azure-durable-functions 25.25% <ø> (ø)
plugins-azure-event-hubs 25.40% <ø> (ø)
plugins-azure-service-bus 24.82% <ø> (ø)
plugins-bullmq 40.83% <ø> (+0.03%) ⬆️
plugins-cassandra 35.36% <ø> (-0.09%) ⬇️
plugins-cookie 26.37% <ø> (ø)
plugins-cookie-parser 26.17% <ø> (ø)
plugins-crypto 25.56% <ø> (ø)
plugins-dd-trace-api 35.48% <ø> (-0.09%) ⬇️
plugins-express-mongo-sanitize 26.31% <ø> (ø)
plugins-express-session 26.13% <ø> (ø)
plugins-fastify 39.44% <ø> (-0.09%) ⬇️
plugins-fetch 35.87% <ø> (-0.09%) ⬇️
plugins-fs 35.77% <ø> (-0.10%) ⬇️
plugins-generic-pool 25.29% <ø> (ø)
plugins-google-cloud-pubsub 43.12% <ø> (-0.08%) ⬇️
plugins-grpc 38.14% <ø> (-0.09%) ⬇️
plugins-handlebars 26.35% <ø> (ø)
plugins-hapi 37.38% <ø> (-0.09%) ⬇️
plugins-hono 37.62% <ø> (-0.09%) ⬇️
plugins-ioredis 35.82% <ø> (-0.09%) ⬇️
plugins-knex 26.04% <ø> (ø)
plugins-langgraph 35.15% <ø> (-0.09%) ⬇️
plugins-ldapjs 23.92% <ø> (ø)
plugins-light-my-request 25.78% <ø> (ø)
plugins-limitd-client 30.10% <ø> (-0.10%) ⬇️
plugins-lodash 25.36% <ø> (ø)
plugins-mariadb 36.74% <ø> (-0.05%) ⬇️
plugins-memcached 35.47% <ø> (-0.09%) ⬇️
plugins-microgateway-core 36.46% <ø> (-0.09%) ⬇️
plugins-modelcontextprotocol-sdk 34.40% <ø> (-0.09%) ⬇️
plugins-moleculer 38.16% <ø> (-0.09%) ⬇️
plugins-mongodb 36.54% <ø> (-0.09%) ⬇️
plugins-mongodb-core 36.28% <ø> (-0.09%) ⬇️
plugins-mongoose 36.15% <ø> (-0.17%) ⬇️
plugins-multer 26.13% <ø> (ø)
plugins-mysql 36.55% <ø> (-0.09%) ⬇️
plugins-mysql2 36.53% <ø> (-0.09%) ⬇️
plugins-node-serialize 26.40% <ø> (ø)
plugins-opensearch 35.12% <ø> (-0.09%) ⬇️
plugins-passport-http 26.19% <ø> (ø)
plugins-postgres 34.62% <ø> (+0.02%) ⬆️
plugins-process 25.56% <ø> (ø)
plugins-pug 26.37% <ø> (ø)
plugins-redis 36.02% <ø> (-0.09%) ⬇️
plugins-router 40.03% <ø> (-0.09%) ⬇️
plugins-sequelize 25.07% <ø> (ø)
plugins-test-and-upstream-amqp10 35.79% <ø> (-0.09%) ⬇️
plugins-test-and-upstream-amqplib 40.97% <ø> (-0.09%) ⬇️
plugins-test-and-upstream-apollo 36.62% <ø> (-0.08%) ⬇️
plugins-test-and-upstream-avsc 35.62% <ø> (-0.09%) ⬇️
plugins-test-and-upstream-bunyan 31.25% <ø> (-0.10%) ⬇️
plugins-test-and-upstream-connect 37.97% <ø> (-0.09%) ⬇️
plugins-test-and-upstream-graphql 37.32% <ø> (-0.09%) ⬇️
plugins-test-and-upstream-koa 37.58% <ø> (-0.09%) ⬇️
plugins-test-and-upstream-protobufjs 35.84% <ø> (-0.09%) ⬇️
plugins-test-and-upstream-rhea 41.05% <ø> (-0.12%) ⬇️
plugins-undici 36.63% <ø> (-0.09%) ⬇️
plugins-url 25.56% <ø> (ø)
plugins-valkey 35.50% <ø> (-0.09%) ⬇️
plugins-vm 25.56% <ø> (ø)
plugins-winston 31.55% <ø> (-0.22%) ⬇️
plugins-ws 39.10% <ø> (-0.09%) ⬇️
profiling-macos 38.03% <ø> (-0.09%) ⬇️
profiling-ubuntu 38.19% <ø> (-0.09%) ⬇️
profiling-windows 39.56% <ø> (-0.09%) ⬇️
serverless-azure-functions-client 25.14% <ø> (ø)
serverless-azure-functions-eventhubs 25.14% <ø> (ø)
serverless-azure-functions-servicebus 25.14% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@datadog-datadog-prod-us1-2
Copy link
Copy Markdown

datadog-datadog-prod-us1-2 Bot commented Apr 23, 2026

Tests

Fix all issues with BitsAI or with Cursor

⚠️ Warnings

🧪 2 Tests failed

Child process plugin context maintenance "before all" hook for "should preserve context around execSync calls" from context maintenance   View in Datadog   (Fix with Cursor)
Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/dd-trace-js/dd-trace-js/packages/datadog-plugin-child_process/test/index.spec.js)

Error: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/dd-trace-js/dd-trace-js/packages/datadog-plugin-child_process/test/index.spec.js)
Plugin couchbase "before all" hook in "couchbase" from couchbase   View in Datadog   (Fix with Cursor)
Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/dd-trace-js/dd-trace-js/packages/datadog-plugin-couchbase/test/index.spec.js)

Error: Timeout of 5000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/home/runner/work/dd-trace-js/dd-trace-js/packages/datadog-plugin-couchbase/test/index.spec.js)
    at processImmediate (node:internal/timers:466:21)

ℹ️ Info

No other issues found (see more)

❄️ No new flaky tests detected

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 68.11% (-0.03%)

Useful? React with 👍 / 👎

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

@pr-commenter
Copy link
Copy Markdown

pr-commenter Bot commented Apr 23, 2026

Benchmarks

Benchmark execution time: 2026-04-23 12:48:34

Comparing candidate commit 4a3bf8b in PR branch dd/fix/test-optimization-junit-action with baseline commit 8cd7c26 in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 1343 metrics, 101 unstable metrics.

Copy link
Copy Markdown
Member

@BridgeAR BridgeAR left a comment

Choose a reason for hiding this comment

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

I can see that this fixes the immediate issue.

I believe it just causes a new issue, which is that the version would not automatically be updated anymore and we would for example not receive the latest security updates and dependabot would also not know to update this.

What about changing junit-upload-github-action to use a specific pinned version, if not specified and we just update that each time there is a new CI release with that version being the new default. That way the benefit would also exist for our users the same as for us :)

Co-authored-by: juan-fernandez <22798219+juan-fernandez@users.noreply.github.com>
@BridgeAR BridgeAR enabled auto-merge (squash) April 23, 2026 12:39
@BridgeAR BridgeAR merged commit 9ef3cb5 into master Apr 23, 2026
933 of 938 checks passed
@BridgeAR BridgeAR deleted the dd/fix/test-optimization-junit-action branch April 23, 2026 12:54
dd-octo-sts Bot pushed a commit that referenced this pull request Apr 24, 2026
@dd-octo-sts dd-octo-sts Bot mentioned this pull request Apr 24, 2026
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.

2 participants