Skip to content

added env vars + DDA spans#2744

Draft
nlchung wants to merge 6 commits intokhewonc/ddai-tracingfrom
nlchung/dd-trace-go
Draft

added env vars + DDA spans#2744
nlchung wants to merge 6 commits intokhewonc/ddai-tracingfrom
nlchung/dd-trace-go

Conversation

@nlchung
Copy link

@nlchung nlchung commented Mar 11, 2026

What does this PR do?

A brief description of the change being made with this pull request.

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

Minimum Agent Versions

Are there minimum versions of the Datadog Agent and/or Cluster Agent required?

  • Agent: vX.Y.Z
  • Cluster Agent: vX.Y.Z

Describe your test plan

Write there any instructions and details you may have to test your PR.

Checklist

  • PR has at least one valid label: bug, enhancement, refactoring, documentation, tooling, and/or dependencies
  • PR has a milestone or the qa/skip-qa label
  • All commits are signed (see: signing commits)

@nlchung nlchung requested review from a team as code owners March 11, 2026 15:25
@codecov-commenter
Copy link

codecov-commenter commented Mar 11, 2026

Codecov Report

❌ Patch coverage is 55.64516% with 55 lines in your changes missing coverage. Please review.
✅ Project coverage is 38.87%. Comparing base (6823e7f) to head (b3e575a).

Files with missing lines Patch % Lines
...controller/datadogagent/controller_reconcile_v2.go 53.84% 40 Missing and 14 partials ⚠️
cmd/main.go 0.00% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (55.64%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@               Coverage Diff               @@
##           dd-trace-go    #2744      +/-   ##
===============================================
+ Coverage        38.79%   38.87%   +0.08%     
===============================================
  Files              309      309              
  Lines            26741    26818      +77     
===============================================
+ Hits             10373    10425      +52     
- Misses           15589    15614      +25     
  Partials           779      779              
Flag Coverage Δ
unittests 38.87% <55.64%> (+0.08%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
internal/controller/datadogagent_controller.go 68.36% <100.00%> (+1.70%) ⬆️
cmd/main.go 6.54% <0.00%> (ø)
...controller/datadogagent/controller_reconcile_v2.go 61.74% <53.84%> (+1.32%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6823e7f...b3e575a. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nlchung nlchung changed the title added DD_OPERATOR_TRACING_ENABLED env var added DD_OPERATOR_TRACING_ENABLED env var + DDA spans Mar 12, 2026
@nlchung nlchung marked this pull request as draft March 12, 2026 14:53
@nlchung nlchung added the enhancement New feature or request label Mar 12, 2026
@nlchung nlchung force-pushed the nlchung/dd-trace-go branch from b3e575a to 84029df Compare March 13, 2026 13:59
@nlchung nlchung changed the title added DD_OPERATOR_TRACING_ENABLED env var + DDA spans added env vars + DDA spans Mar 13, 2026
@nlchung nlchung changed the base branch from dd-trace-go to khewonc/ddai-tracing March 13, 2026 14:02
@nlchung nlchung changed the base branch from khewonc/ddai-tracing to main March 16, 2026 13:59
@nlchung nlchung changed the base branch from main to khewonc/ddai-tracing March 16, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request qa/skip-qa

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants