Do not use direct apireader client and instead make f.etcdSecretPresent as part of existing calls

[tbavelier]

What does this PR do?

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

Motivation

Review of #3179

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)

[datadog-datadog-prod-us1-2]

 

✨ Fix all issues with BitsAI

🛑 Gate Violations

🎯 1 Code Coverage issue detected

A Patch coverage percentage gate may be blocking this PR.

• Patch coverage: 50.00% (threshold: 80.00%)

⚠️ Warnings

🚦 2 Pipeline jobs failed

pull request linter | Check Milestone     

pull request linter | build     

ℹ️ Info

🎯 Code Coverage (details)
• Patch Coverage: 50.00%
• Overall Coverage: 44.65%

Useful? React with 👍 / 👎

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 0286a7d | Docs | Datadog PR Page | Give us feedback!}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0286a7d906

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Restore etcd secret detection for profile agents

When DatadogAgentProfile is enabled, non-default profile DDAIs skip manageFeatureDependencies (controller_reconcile_v2.go lines 68-83) but still run reconcileV2Agent (line 106), so this field is never set for those feature instances. On OpenShift with control plane monitoring and a profile-selected node pool, the default DDAI can create/retain etcd-metric-client, but profile DaemonSets will see false here and omit the etcd cert volume/mount, preventing the etcd check from authenticating on those nodes.

Useful? React with 👍 / 👎.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 50.00000% with 3 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (levan-m/introspection-ddai-fix-podstartup@09b8ef8). Learn more about missing BASE report.

Files with missing lines	Patch %	Lines
...dogagent/feature/controlplanemonitoring/feature.go	50.00%	2 Missing and 1 partial ⚠️

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

Additional details and impacted files

@@                             Coverage Diff                              @@
##             levan-m/introspection-ddai-fix-podstartup    #3180   +/-   ##
============================================================================
  Coverage                                             ?   44.40%           
============================================================================
  Files                                                ?      378           
  Lines                                                ?    30912           
  Branches                                             ?        0           
============================================================================
  Hits                                                 ?    13726           
  Misses                                               ?    16290           
  Partials                                             ?      896           

Flag	Coverage Δ
unittests	44.40% <50.00%> (?)

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

Files with missing lines	Coverage Δ
...dogagent/feature/controlplanemonitoring/feature.go	52.41% <50.00%> (ø)

---

Continue to review full report in Codecov by Harness.

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.