datadog-operator icon indicating copy to clipboard operation
datadog-operator copied to clipboard

[BARX-1009] Use the ddot-collector image by default for the operator

Open AliDatadog opened this issue 5 months ago • 1 comments

What does this PR do?

Configure the otel-agent to use the ddot-collector image by default.

Motivation

ddot-collector is the recommended way to deploy the Agent.

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

AliDatadog avatar Jun 27 '25 14:06 AliDatadog

Codecov Report

Attention: Patch coverage is 84.72222% with 11 lines in your changes missing coverage. Please review.

Project coverage is 47.13%. Comparing base (5dff879) to head (78a8a7c).

Files with missing lines Patch % Lines
...ller/datadogagent/feature/otelcollector/feature.go 91.93% 4 Missing and 1 partial :warning:
...controller/datadogagent/component/agent/default.go 0.00% 3 Missing :warning:
pkg/images/images.go 0.00% 3 Missing :warning:
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2012      +/-   ##
==========================================
+ Coverage   47.00%   47.13%   +0.13%     
==========================================
  Files         255      255              
  Lines       25302    25356      +54     
==========================================
+ Hits        11892    11951      +59     
+ Misses      12794    12790       -4     
+ Partials      616      615       -1     
Flag Coverage Δ
unittests 47.13% <84.72%> (+0.13%) :arrow_up:

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

Files with missing lines Coverage Δ
pkg/testutils/builder.go 93.00% <100.00%> (+0.03%) :arrow_up:
...controller/datadogagent/component/agent/default.go 87.24% <0.00%> (+0.12%) :arrow_up:
pkg/images/images.go 96.36% <0.00%> (-1.79%) :arrow_down:
...ller/datadogagent/feature/otelcollector/feature.go 89.18% <91.93%> (+5.77%) :arrow_up:

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 5dff879...78a8a7c. Read the comment docs.

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

codecov-commenter avatar Jun 27 '25 15:06 codecov-commenter