dd-trace-go icon indicating copy to clipboard operation
dd-trace-go copied to clipboard

feat(tracer): add support for dynamically enabling and disabling Live Debugger

Open andreimatei opened this issue 1 month ago • 3 comments

Add support for the "multiconfig" remote config convention, and add support for dynamically enabling dynamic instrumentation. Please see individual commits.

  • [ ] Changed code has unit tests for its functionality at or near 100% coverage.
  • [ ] System-Tests covering this feature have been added and enabled with the va.b.c-dev version tag.
  • [ ] There is a benchmark for any new code, or changes to existing code.
  • [ ] If this interacts with the agent in a new way, a system test has been added.
  • [ ] New code is free of linting errors. You can check this by running ./scripts/lint.sh locally.
  • [ ] Add an appropriate team label so this PR gets put in the right place for the release notes.
  • [ ] Non-trivial go.mod changes, e.g. adding new modules, are reviewed by @DataDog/dd-trace-go-guild.

andreimatei avatar Dec 10 '25 23:12 andreimatei

Benchmarks

Benchmark execution time: 2025-12-18 18:00:51

Comparing candidate commit 24b2c68d899282332d2e6333cfe5e302775e690c in PR branch andrei/remote-enablement with baseline commit 6cd3a068e4f8fd46b67fc145d60438f578433053 in branch main.

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

pr-commenter[bot] avatar Dec 10 '25 23:12 pr-commenter[bot]

Codecov Report

:x: Patch coverage is 84.96732% with 23 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 54.62%. Comparing base (abc3656) to head (24b2c68). :warning: Report is 113 commits behind head on main.

Files with missing lines Patch % Lines
ddtrace/tracer/remote_config.go 84.89% 13 Missing and 8 partials :warning:
ddtrace/tracer/option.go 85.71% 1 Missing and 1 partial :warning:
Additional details and impacted files
Files with missing lines Coverage Δ
ddtrace/tracer/tracer.go 86.40% <ø> (ø)
internal/remoteconfig/remoteconfig.go 65.10% <ø> (+5.38%) :arrow_up:
ddtrace/tracer/option.go 85.06% <85.71%> (+0.45%) :arrow_up:
ddtrace/tracer/remote_config.go 89.20% <84.89%> (-0.27%) :arrow_down:

... and 80 files with indirect coverage changes

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

codecov[bot] avatar Dec 10 '25 23:12 codecov[bot]

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 24b2c68 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@codex review

darccio avatar Dec 16 '25 08:12 darccio