Skip to content

feat(otel-thread-ctx): Node.js OTEP-4947 thread-context writer #9210

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
szegedi merged 6 commits into from
Aug 13, 2026
Merged

feat(otel-thread-ctx): Node.js OTEP-4947 thread-context writer #9210

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
4b10aef
Bump @datadog/libdatadog to 0.12.1
szegedi Jul 3, 2026
7e2fa88
Extract storage-channels module from wall profiler
szegedi Jun 23, 2026
b2e7128
Extract shared web-tags cache from wall profiler
szegedi Jul 13, 2026
8808ab3
Add OTEP-4947 thread context writer
szegedi Jun 23, 2026
4733cb3
Publish OTEP-4947 process-context metadata via process discovery
szegedi Jun 23, 2026
f66073e
CI: run test:otel-thread-ctx explicitly in profiling.yml
szegedi Jul 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .github/CODEOWNERS
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -369,8 +369,13 @@
/integration-tests/profiler/ @DataDog/profiling-js

/packages/dd-trace/*/profiling/ @DataDog/profiling-js
/packages/dd-trace/src/otel-thread-ctx.js @DataDog/profiling-js
/packages/dd-trace/src/profiler.js @DataDog/profiling-js
/packages/dd-trace/src/storage-channels.js @DataDog/profiling-js @DataDog/apm-sdk-capabilities-js
/packages/dd-trace/src/web-tags-cache.js @DataDog/profiling-js
/packages/dd-trace/test/exporters/common/form-data.spec.js @DataDog/profiling-js
/packages/dd-trace/test/otel-thread-ctx.spec.js @DataDog/profiling-js
/packages/dd-trace/test/web-tags-cache.spec.js @DataDog/profiling-js

# Language Platform
/* @DataDog/lang-platform-js
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/profiling.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ jobs:
- uses: ./.github/actions/install
- run: npm run test:profiler:ci
- run: npm run test:integration:profiler:coverage
- run: npm run test:otel-thread-ctx
- uses: ./.github/actions/coverage
with:
flags: profiling-macos
Expand Down Expand Up @@ -60,6 +61,7 @@ jobs:
- uses: ./.github/actions/node/latest
- run: npm run test:profiler:ci
- run: npm run test:integration:profiler:coverage
- run: npm run test:otel-thread-ctx
- uses: ./.github/actions/coverage
with:
flags: profiling-ubuntu
Expand Down Expand Up @@ -118,6 +120,7 @@ jobs:
) | ForEach-Object { "$_=" | Out-File -FilePath $env:GITHUB_ENV -Append -Encoding utf8 }
- run: npm run test:profiler:ci
- run: npm run test:integration:profiler:coverage
- run: npm run test:otel-thread-ctx
- uses: ./.github/actions/node-crash-report
if: failure()
# Upload any WER minidumps that landed during this job, even on
Expand Down
4 changes: 3 additions & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,8 @@
"test:plugins:upstream": "node ./packages/dd-trace/test/plugins/suite.js",
"test:profiler": "node scripts/mocha-parallel-files.js --expose-gc --timeout 30000 -- \"packages/dd-trace/test/profiling/**/*.spec.js\"",
"test:profiler:ci": "node scripts/c8-ci.js test:profiler",
"test:otel-thread-ctx": "mocha --timeout 60000 packages/dd-trace/test/otel-thread-ctx.spec.js",
"test:docker:otel-thread-ctx": "./scripts/docker/run-otel-thread-ctx-spec.sh",
Comment thread
szegedi marked this conversation as resolved.
"test:integration": "mocha --timeout 60000 \"integration-tests/*.spec.js\"",
"test:integration:coverage": "node ./integration-tests/coverage/run-suite.js --timeout 60000 \"integration-tests/*.spec.js\"",
"test:integration:aiguard": "mocha --timeout 60000 \"integration-tests/aiguard/*.spec.js\"",
Expand Down Expand Up @@ -180,7 +182,7 @@
"opentracing": ">=0.14.7"
},
"optionalDependencies": {
"@datadog/libdatadog": "0.9.4",
"@datadog/libdatadog": "0.12.1",
"@datadog/native-appsec": "11.0.1",
"@datadog/native-iast-taint-tracking": "4.2.0",
"@datadog/native-metrics": "3.1.2",
Expand Down
2 changes: 2 additions & 0 deletions packages/dd-trace/src/config/generated-config-types.d.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -340,6 +340,7 @@ export interface GeneratedConfig {
DD_TRACE_OPENSEARCH_PROJECT_OPENSEARCH_ENABLED: boolean;
DD_TRACE_OPENTELEMETRY_SDK_TRACE_NODE_ENABLED: boolean;
DD_TRACE_ORACLEDB_ENABLED: boolean;
DD_TRACE_OTEL_CTX_ENABLED: boolean;
DD_TRACE_OTEL_ENABLED: boolean;
DD_TRACE_OTEL_SEMANTICS_ENABLED: boolean;
DD_TRACE_PASSPORT_ENABLED: boolean;
Expand Down Expand Up @@ -1040,6 +1041,7 @@ export interface GeneratedEnvVarConfig {
DD_TRACE_OPENSEARCH_PROJECT_OPENSEARCH_ENABLED: boolean;
DD_TRACE_OPENTELEMETRY_SDK_TRACE_NODE_ENABLED: boolean;
DD_TRACE_ORACLEDB_ENABLED: boolean;
DD_TRACE_OTEL_CTX_ENABLED: boolean;
DD_TRACE_OTEL_ENABLED: boolean;
DD_TRACE_OTEL_SEMANTICS_ENABLED: boolean;
DD_TRACE_PARTIAL_FLUSH_MIN_SPANS: number;
Expand Down
7 changes: 7 additions & 0 deletions packages/dd-trace/src/config/supported-configurations.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3518,6 +3518,13 @@
"default": "false"
}
],
"DD_TRACE_OTEL_CTX_ENABLED": [
{
"implementation": "A",
"type": "boolean",
"default": "false"
}
],
Comment on lines +3521 to +3527

@ivoanjo ivoanjo Jul 17, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fingers crossed we can enable by default (on supported node.js versions) soon ;)

"DD_TRACE_PARTIAL_FLUSH_MIN_SPANS": [
{
"implementation": "B",
Expand Down
Loading
Loading