[PROF-14062] Rename the profiler in the datadog-agent, helm, operator#2901
[PROF-14062] Rename the profiler in the datadog-agent, helm, operator#2901
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2901 +/- ##
=======================================
Coverage 40.05% 40.05%
=======================================
Files 319 319
Lines 28052 28052
=======================================
Hits 11237 11237
Misses 15993 15993
Partials 822 822
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5a725f9c0c
ℹ️ 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".
| // Note: Need to override image via annotation | ||
| Image: agentImage(), | ||
| Command: []string{ | ||
| "/opt/datadog-agent/embedded/bin/full-host-profiler", |
There was a problem hiding this comment.
What agent version was this rename made? Is there a fallback mechanism for customers using a newer operator version but an older agent version?
There was a problem hiding this comment.
No customers are using this yet, and the image is only configurable via experimental.agent.datadoghq.com/image-override-config currently. Only place this could break things is in internal deploys, @wehzzz lets check the image there contains the change before merging this.
What does this PR do?
Renames references from
full-host-profilertohost-profilerfollowing the recent binary renaming.Motivation
To match the new binary name.
Additional Notes
Minimum Agent Versions
Describe your test plan
Checklist
bug,enhancement,refactoring,documentation,tooling, and/ordependenciesqa/skip-qalabel