Skip to content

Set the Datadog site in the Kubernetes Metrics setup secret - #39309

Draft
brett0000FF wants to merge 1 commit into
masterfrom
brett.blue/otel-k8s-metrics-site
Draft

Set the Datadog site in the Kubernetes Metrics setup secret#39309
brett0000FF wants to merge 1 commit into
masterfrom
brett.blue/otel-k8s-metrics-site

Conversation

@brett0000FF

Copy link
Copy Markdown
Collaborator

What does this PR do? What is the motivation?

The reference Collector configurations in the Kubernetes Metrics setup read the Datadog site from a dd-site secret key that falls back to datadoghq.com:

site: ${env:DD_SITE:-datadoghq.com}

The setup step created the secret with only an API key, so customers on any other site had no instruction to set it and their data would go to US1.

This PR adds the site to the secret creation step and links to the Datadog site reference.

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

Found and verified with Claude Code, which checked the secret key name and site value format against the reference configurations in DataDog/opentelemetry-examples.

Additional notes

Independent of the other two Kubernetes Explorer PRs; this one is a standalone correctness fix.

The reference Collector configurations read the Datadog site from a
dd-site secret key that falls back to datadoghq.com, but the setup step
created the secret with only an API key. Customers on other sites had no
instruction to set it.
@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant