fix(test-optimization): disable telemetry in test workers - #9687
Conversation
Overall package sizeSelf size: 7.83 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 3.3.3 | 125.43 kB | 441.68 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 kB | | dc-polyfill | 0.1.11 | 25.74 kB | 25.74 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
🎉 All green!🧪 All tests passed 🎯 Code Coverage (details) 🔗 Commit SHA: 33a29e5 | Docs | Datadog PR Page | Give us feedback! |
BenchmarksBenchmark execution time: 2026-08-05 09:55:47 Comparing candidate commit 33a29e5 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 2316 metrics, 42 unstable metrics.
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #9687 +/- ##
==========================================
+ Coverage 98.52% 98.53% +0.01%
==========================================
Files 957 957
Lines 135106 135234 +128
Branches 12015 11787 -228
==========================================
+ Hits 133109 133257 +148
+ Misses 1997 1977 -20
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
@codex review |
There was a problem hiding this comment.
More details
The worker exporter mapping consistently disables instrumentation telemetry for Cucumber, Jest, Mocha, Playwright, and Vitest workers, including when environment or remote configuration requests enablement. Non-worker CI initialization remains configurable, and the unsupported telemetry initialization option no longer produces the observed warning.
📊 Validated against 14 scenarios · Open Bits AI session
🤖 Datadog Autotest · Commit 30656d8 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 30656d8a80
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Bits Code status: ✅ Done Comment @DataDog to request changes |
|
@codex review |
There was a problem hiding this comment.
More details
The worker telemetry changes preserve Test Optimization metrics across the new Cucumber, Mocha, Playwright, and Vitest IPC paths while keeping instrumentation telemetry disabled in worker configurations. Real parallel-worker runs, including Cucumber 7, delivered test telemetry successfully; no additional tests recommended: existing and added integration coverage already exercises the relevant worker input classes.
📊 Validated against 48 scenarios · Open Bits AI session
🤖 Datadog Autotest · Commit 50dbd2c · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 50dbd2c5a4
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. 🚀 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
What does this PR do?
Unconditionally disables instrumentation telemetry in Test Optimization test workers. Worker telemetry remains disabled even when
DD_INSTRUMENTATION_TELEMETRY_ENABLED=true; non-worker processes remain configurable.It also removes the unsupported
telemetry: { enabled: false }initialization option and uses the existing worker exporter identifiers to recognize Cucumber, Jest, Mocha, Playwright, and Vitest workers.Test Optimization metrics produced in workers are forwarded to the coordinator for Cucumber, Jest, Mocha, Playwright, Vitest, and WebdriverIO, so disabling each worker's instrumentation telemetry client does not drop worker metrics. The worker paths now include Cucumber 7's callback-based parallel coordinator and Mocha 8 through 9.1, before Mocha supplied
MOCHA_WORKER_IDitself.Motivation
The worker bootstrap passed a programmatic telemetry option that the current configuration schema does not support. With debug logging enabled, every Vitest worker therefore logged
Unknown option telemetry with value { enabled: false }, and the ignored option did not disable telemetry.