Skip to content

Tbavelier/contp 1596 autopilot#2983

Draft
tbavelier wants to merge 21 commits intomainfrom
tbavelier/contp-1596-autopilot
Draft

Tbavelier/contp 1596 autopilot#2983
tbavelier wants to merge 21 commits intomainfrom
tbavelier/contp-1596-autopilot

Conversation

@tbavelier
Copy link
Copy Markdown
Member

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)

tbavelier added 5 commits May 4, 2026 09:24
Bump the default from v1.0.1 to v1.0.3, which includes system-probe / NPM
exemptions required for the NPM feature on GKE Autopilot.

Users can override the version via the
experimental.agent.datadoghq.com/autopilot-allowlist-version annotation on
the DatadogAgent. Malformed versions are rejected with a log message and fall
back to the default.
…n override

On GKE Autopilot the kubelet endpoint is not reachable, so the Agent must use
the Kubernetes API server to discover pods. Inject DD_KUBELET_USE_API_SERVER=true
on all node-agent containers when the autopilot annotation is set.

Expose a new annotation experimental.agent.datadoghq.com/autopilot-allowlist-version
so users can point at a specific WorkloadAllowlist version instead of the default.

Add a regression test asserting that NPM volumes/mounts and HostPID survive the
autopilot overrides (system-probe relies on them when the WorkloadAllowlist grants
the required exemptions).
…PI server mode

When autopilot is enabled:
- Fine-grained kubelet authorization is turned on by default (users can still
  opt in explicitly via the existing annotation on non-autopilot clusters).
- A pods get/list ClusterRole rule is added so the Agent can list pods via the
  API server when DD_KUBELET_USE_API_SERVER=true.
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 6, 2026

Codecov Report

❌ Patch coverage is 70.29703% with 30 lines in your changes missing coverage. Please review.
✅ Project coverage is 41.18%. Comparing base (d3652e6) to head (f8dc2c7).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
.../controller/datadogagent/experimental/autopilot.go 67.56% 20 Missing and 4 partials ⚠️
pkg/allowlistsynchronizer/allowlistsynchronizer.go 71.42% 4 Missing ⚠️
internal/controller/datadogagent/common/volumes.go 0.00% 1 Missing ⚠️
...nal/controller/datadogagent/global/dependencies.go 75.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2983      +/-   ##
==========================================
+ Coverage   40.75%   41.18%   +0.42%     
==========================================
  Files         332      332              
  Lines       28197    28288      +91     
==========================================
+ Hits        11493    11651     +158     
+ Misses      15929    15850      -79     
- Partials      775      787      +12     
Flag Coverage Δ
unittests 41.18% <70.29%> (+0.42%) ⬆️

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

Files with missing lines Coverage Δ
...controller/datadogagent/component/agent/default.go 44.13% <ø> (+0.06%) ⬆️
...al/controller/datadogagent/component/agent/rbac.go 81.48% <100.00%> (+81.48%) ⬆️
...ller/datadogagent/feature/logcollection/feature.go 84.14% <ø> (ø)
internal/controller/datadogagent/common/volumes.go 0.00% <0.00%> (ø)
...nal/controller/datadogagent/global/dependencies.go 19.52% <75.00%> (+0.64%) ⬆️
pkg/allowlistsynchronizer/allowlistsynchronizer.go 58.06% <71.42%> (+58.06%) ⬆️
.../controller/datadogagent/experimental/autopilot.go 59.84% <67.56%> (+59.84%) ⬆️

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 d3652e6...f8dc2c7. Read the comment docs.

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

@datadog-prod-us1-4

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants