feat: Add instrumentation controller crd feature#3002
feat: Add instrumentation controller crd feature#3002Mathew-Estafanous wants to merge 4 commits into
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2b15c1b4da
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| func (f *instrumentationCRDFeature) ManageSingleContainerNodeAgent(feature.PodTemplateManagers, string) error { | ||
| return nil |
There was a problem hiding this comment.
Configure instrumentation env var for single-container agents
The new feature never configures node-agent env vars when SingleContainerStrategy is used because ManageSingleContainerNodeAgent is a no-op. In that mode the reconciler calls ManageSingleContainerNodeAgent instead of ManageNodeAgent, so DD_INSTRUMENTATION_CRD_CONTROLLER_ENABLED is not added to the node agent container and the instrumentation CRD flow is only partially enabled.
Useful? React with 👍 / 👎.
2b15c1b to
c216052
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #3002 +/- ##
==========================================
- Coverage 40.76% 40.59% -0.17%
==========================================
Files 332 334 +2
Lines 28197 28272 +75
==========================================
- Hits 11494 11477 -17
- Misses 15928 16018 +90
- Partials 775 777 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
🛑 Gate Violations
ℹ️ Info🎯 Code Coverage (details) Useful? React with 👍 / 👎 This comment will be updated automatically if new data arrives.🔗 Commit SHA: d8fdb7f | Docs | Datadog PR Page | Give us feedback! |
What does this PR do?
Adds a
feature.instrumentationCRDfeature that'll be used to enable the DatadogInstrumentation CRD controller in the cluster agent and node agent's AD provider.The feature depends on:
Motivation
Without this feature, enabling instrumentation CRD controller requires manually adding
DD_INSTRUMENTATION_CRD_CONTROLLER_ENABLEDto both cluster and node agents. Additionally there'd be no enforcement of admission controller or rbac pre-requisites.Minimum Agent Versions
Are there minimum versions of the Datadog Agent and/or Cluster Agent required?
Describe your test plan
instrumentationcrd/feature_test.go.Manual QA
DD_INSTRUMENTATION_CRD_CONTROLLER_ENABLED: "true"Checklist
bug,enhancement,refactoring,documentation,tooling, and/ordependenciesqa/skip-qalabel