-
Notifications
You must be signed in to change notification settings - Fork 109
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[examples] Fix "datadog-agent-secret-backend" #467
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pull request does not contain a valid label. Please add one of the following labels: bug, enhancement, refactoring, documentation, tooling
Codecov Report
@@ Coverage Diff @@
## main #467 +/- ##
=======================================
Coverage 60.60% 60.60%
=======================================
Files 3 3
Lines 132 132
=======================================
Hits 80 80
Misses 40 40
Partials 12 12
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
* release v0.8.0-rc.1 * [defaulting/images] Update agent to 7.35.0 (#465) * [controllers/datadogagent/utils] Fix process collection nil check (#466) * [examples] Fix datadog-agent-secret-backend (#467) * Add port names to agent service (#468) * Update default DCA image to 1.19.0 (#470) * release v0.8.0-rc.2 * fix: k8s version detection for agent service (#472) (#473) * Add NodeLabelsAsTags and NamespaceLabelsAsTags, remove default Pod env vars (#471) (#475) Remove POD_X_AS_TAGS as no longer defaulting to {} Test cases for various metadata as tags configs Move the xyzAsTags to the common set of environment variables Revert "Move the xyzAsTags to the common set of environment variables" This reverts commit 11a8cfa. Update tests to use NodeAgentConfig/EnvVar rather than generics Add port names to agent service (#468) Update default DCA image to 1.19.0 (#470) Share common structs in common/v1 and adds missing fields to v2alpha1 (#438) fix: k8s version detection for agent service (#472) Rename Test_getEnvVarsForMetadataAsTags function Co-authored-by: Celene Chang <[email protected]> Co-authored-by: Celene Chang <[email protected]> * release v0.8.0-rc.3 * Default Agent version to 7.35.2 (#480) * release v0.8.0-rc.4 * release v0.8.0 Co-authored-by: David Ortiz <[email protected]> Co-authored-by: Cedric Lamoriniere <[email protected]> Co-authored-by: JacksonDavenport <[email protected]> Co-authored-by: Celene Chang <[email protected]> Co-authored-by: Paul Cacheux <[email protected]>
* release v0.8.0-rc.1 * [defaulting/images] Update agent to 7.35.0 (#465) * [controllers/datadogagent/utils] Fix process collection nil check (#466) * [examples] Fix datadog-agent-secret-backend (#467) * Add port names to agent service (#468) * Update default DCA image to 1.19.0 (#470) * release v0.8.0-rc.2 * fix: k8s version detection for agent service (#472) (#473) * Add NodeLabelsAsTags and NamespaceLabelsAsTags, remove default Pod env vars (#471) (#475) Remove POD_X_AS_TAGS as no longer defaulting to {} Test cases for various metadata as tags configs Move the xyzAsTags to the common set of environment variables Revert "Move the xyzAsTags to the common set of environment variables" This reverts commit 11a8cfa. Update tests to use NodeAgentConfig/EnvVar rather than generics Add port names to agent service (#468) Update default DCA image to 1.19.0 (#470) Share common structs in common/v1 and adds missing fields to v2alpha1 (#438) fix: k8s version detection for agent service (#472) Rename Test_getEnvVarsForMetadataAsTags function Co-authored-by: Celene Chang <[email protected]> Co-authored-by: Celene Chang <[email protected]> * release v0.8.0-rc.3 * Default Agent version to 7.35.2 (#480) * release v0.8.0-rc.4 * release v0.8.0 Co-authored-by: David Ortiz <[email protected]> Co-authored-by: Cedric Lamoriniere <[email protected]> Co-authored-by: JacksonDavenport <[email protected]> Co-authored-by: Celene Chang <[email protected]> Co-authored-by: Paul Cacheux <[email protected]>
What does this PR do?
Fixes the example manifest in
examples/datadogagent/datadog-agent-secret-backend.yaml
.Currently it fails with this error:
critical: unable to decrypt secret from datadog.yaml: an error occurred while decrypting 'api_key': secret does not exist
It's because the process container is created by default (because the orchestrator explorer is enabled by default), so we need to mount the volume with the credentials.
Describe your test plan
Deploy the example and check that the agent pods start.