Update profiler to 5.14.0 (#7815)

szegedi · claude · web-flow · commit 5e82809f30e0 · 2026-03-18T17:25:17.000+01:00
* Update profiler to 5.14.0

* test(profiler): fix wall profiler context telemetry tests on Ubuntu+Node24

With pprof 5.14.0, SetContext() in CPED mode only creates context holders
when inside an ALS run() scope. On Ubuntu (more aggressive GC), the transient
holders created by the profiler exporter's storage.run() were collected before
getMetrics() was called, causing the gauge to always report 0.

Fix by wrapping busyWait in tracer.trace() in the profiler test fixture so
there's a long-lived span context keeping CPED holders alive throughout the
test duration.

Additionally, use resolveAtFirstSuccess=true in both profiler telemetry
assertTelemetryReceived calls: with an active span, dd-trace emits 'tracers'
namespace generate-metrics events before the 'profilers' ones, so the handler
must keep scanning rather than stopping at the first event.

Also removes the macOS skip from the wall profiler context telemetry test and
increases TEST_DURATION_MS from 1500 to 3000 to avoid timing races.

Co-authored-by: Claude Sonnet 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/integration-tests/profiler/index.js b/integration-tests/profiler/index.js
@@ -21,5 +21,6 @@ function busyWait (ms) {
   }))
 }
 
+const tracer = require('dd-trace')
 const durationMs = Number.parseInt(process.env.TEST_DURATION_MS ?? '500')
-setImmediate(async () => busyWait(durationMs))
+setImmediate(() => tracer.trace('test-operation', () => busyWait(durationMs)))
diff --git a/integration-tests/profiler/profiler.spec.js b/integration-tests/profiler/profiler.spec.js
@@ -720,7 +720,7 @@ describe('profiler', () => {
 
         // Same number of requests and responses
         assert.strictEqual(responses.points[0][1], requestCount)
-      }, 'generate-metrics', timeout)
+      }, 'generate-metrics', timeout, 1, true)
 
       const checkDistributions = agent.assertTelemetryReceived(({ _, payload }) => {
         const pp = payload.payload
@@ -748,9 +748,6 @@ describe('profiler', () => {
       if (process.platform === 'win32') {
         this.skip() // Wall profiler context count telemetry is not supported on Windows
       }
-      if (process.platform === 'darwin') {
-        this.skip() // Test is flaky on macOS
-      }
       proc = fork(profilerTestFile, {
         cwd,
         env: {
@@ -759,7 +756,7 @@ describe('profiler', () => {
           DD_PROFILING_UPLOAD_PERIOD: '1',
           DD_PROFILING_ASYNC_CONTEXT_FRAME_ENABLED: '1',
           DD_TELEMETRY_HEARTBEAT_INTERVAL: '1', // every second
-          TEST_DURATION_MS: 1500,
+          TEST_DURATION_MS: 3000,
         },
       })
 
@@ -772,7 +769,7 @@ describe('profiler', () => {
           assert.strictEqual(sampleContexts.type, 'gauge')
           assert.ok(sampleContexts.points[0][1] >= 1)
         })
-      }, 'generate-metrics', timeout)
+      }, 'generate-metrics', timeout, 1, true)
 
       await Promise.all([checkProfiles(agent, proc, timeout), checkMetrics])
     })
diff --git a/package.json b/package.json
@@ -143,7 +143,7 @@
     "@datadog/native-iast-taint-tracking": "4.1.0",
     "@datadog/native-metrics": "3.1.1",
     "@datadog/openfeature-node-server": "^1.1.0",
-    "@datadog/pprof": "5.13.5",
+    "@datadog/pprof": "5.14.0",
     "@datadog/wasm-js-rewriter": "5.0.1",
     "@opentelemetry/api": ">=1.0.0 <1.10.0",
     "@opentelemetry/api-logs": "<1.0.0",
diff --git a/yarn.lock b/yarn.lock
@@ -231,10 +231,10 @@
   dependencies:
     "@datadog/flagging-core" "1.1.0"
 
-"@datadog/pprof@5.13.5":
-  version "5.13.5"
-  resolved "https://registry.yarnpkg.com/@datadog/pprof/-/pprof-5.13.5.tgz#36eea628f321ed512fe4ce440de4cffa2e056569"
-  integrity sha512-W0dvo91ff2EMQI9Vhv8PNM+w1ZWuClm1pBVdLB6y0bMB3+E+wEGg0VD1iNJxsuPbwDt5+yV0u3e4WkqK12Lzlg==
+"@datadog/pprof@5.14.0":
+  version "5.14.0"
+  resolved "https://registry.yarnpkg.com/@datadog/pprof/-/pprof-5.14.0.tgz#a5a43fde3ca9369345994cc5cf82d5dcf743fb1c"
+  integrity sha512-bXYptMf0BY3NdKcAmX5aAOnozj7GFdNYE4SIdPF0/Q/Yfb2fYBIYZaN/Le6BhiDIepBvvH/H75d3EIFhB64kOw==
   dependencies:
     node-gyp-build "<4.0"
     pprof-format "^2.2.1"

Original file line number	Diff line number	Diff line change
`@@ -21,5 +21,6 @@ function busyWait (ms) {`
`21`	`21`	`}))`
`22`	`22`	`}`
`23`	`23`
	`24`	`+const tracer = require('dd-trace')`
`24`	`25`	`const durationMs = Number.parseInt(process.env.TEST_DURATION_MS ?? '500')`
`25`		`-setImmediate(async () => busyWait(durationMs))`
	`26`	`+setImmediate(() => tracer.trace('test-operation', () => busyWait(durationMs)))`