Skip to content

feat: add Datadog continuous profiler gated by DD_PROFILING_ENABLED#296

Merged
worstell merged 1 commit intomainfrom
blox/dd-profiler
May 6, 2026
Merged

feat: add Datadog continuous profiler gated by DD_PROFILING_ENABLED#296
worstell merged 1 commit intomainfrom
blox/dd-profiler

Conversation

@worstell
Copy link
Copy Markdown
Contributor

@worstell worstell commented May 6, 2026

Adds internal/profiling that starts the dd-trace-go continuous profiler when DD_PROFILING_ENABLED=true. All other configuration (DD_AGENT_HOST, DD_SERVICE, DD_ENV, DD_VERSION) is read from standard env vars wired in by the caller's deployment manifest.

Default is no-op — flipping DD_PROFILING_ENABLED=true in the environment turns it on without a code change.

@worstell worstell force-pushed the blox/dd-profiler branch from adb62d3 to 538b2e4 Compare May 6, 2026 23:03
@worstell worstell marked this pull request as ready for review May 6, 2026 23:10
@worstell worstell requested a review from a team as a code owner May 6, 2026 23:10
@worstell worstell requested review from stuartwdouglas and removed request for a team May 6, 2026 23:10
@worstell worstell force-pushed the blox/dd-profiler branch from 538b2e4 to bb2f5fe Compare May 6, 2026 23:15
@worstell worstell force-pushed the blox/dd-profiler branch 2 times, most recently from 5634d15 to c6eb412 Compare May 6, 2026 23:19
@worstell worstell enabled auto-merge (squash) May 6, 2026 23:20
Adds internal/profiling that starts the dd-trace-go continuous profiler
when DD_PROFILING_ENABLED=true. All other configuration (DD_AGENT_HOST,
DD_SERVICE, DD_ENV, DD_VERSION) is read from env vars wired in by the
caller's deployment manifest.

Default is no-op.

Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019dfedf-4ab9-753d-a2b8-6c459756099f
@worstell worstell force-pushed the blox/dd-profiler branch from c6eb412 to 2444edf Compare May 6, 2026 23:21
@worstell worstell merged commit 3ae9f03 into main May 6, 2026
8 checks passed
@worstell worstell deleted the blox/dd-profiler branch May 6, 2026 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants