Skip to content

Conversation

@tbavelier
Copy link
Member

What does this PR do?

As stated in PR title, remove duplicate imports

Motivation

Reduce number of staticcheck linter (disabled for now) violations

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

@tbavelier tbavelier requested a review from a team as a code owner December 19, 2025 16:53
@tbavelier tbavelier added enhancement New feature or request qa/skip-qa labels Dec 19, 2025
@codecov-commenter
Copy link

codecov-commenter commented Dec 19, 2025

Codecov Report

❌ Patch coverage is 37.50000% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 37.28%. Comparing base (517b3d1) to head (9d3c477).

Files with missing lines Patch % Lines
...dogagentinternal/controller_reconcile_v2_common.go 0.00% 3 Missing ⚠️
...ler/datadogagent/controller_reconcile_v2_common.go 75.00% 1 Missing ⚠️
.../controller/datadogagent/feature/test/testsuite.go 0.00% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (37.50%) 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           @@
##             main    #2420   +/-   ##
=======================================
  Coverage   37.28%   37.28%           
=======================================
  Files         290      290           
  Lines       24708    24708           
=======================================
  Hits         9212     9212           
  Misses      14783    14783           
  Partials      713      713           
Flag Coverage Δ
unittests 37.28% <37.50%> (ø)

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

Files with missing lines Coverage Δ
...ler/datadogagent/controller_reconcile_v2_common.go 30.80% <75.00%> (ø)
.../controller/datadogagent/feature/test/testsuite.go 0.00% <0.00%> (ø)
...dogagentinternal/controller_reconcile_v2_common.go 5.39% <0.00%> (ø)

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 517b3d1...9d3c477. Read the comment docs.

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

@tbavelier tbavelier merged commit 4b3db1f into main Jan 6, 2026
32 checks passed
@tbavelier tbavelier deleted the tbavelier/duplicate-imports branch January 6, 2026 14:46
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.

4 participants