Skip to content

Make LLM Obs MCP setup commands site-aware - #38485

Merged
mehulsonowal merged 1 commit into
masterfrom
dd/mehul.sonowal/llmobs-mcp-site-aware-endpoint
Jul 29, 2026
Merged

Make LLM Obs MCP setup commands site-aware#38485
mehulsonowal merged 1 commit into
masterfrom
dd/mehul.sonowal/llmobs-mcp-site-aware-endpoint

Conversation

@mehulsonowal

Copy link
Copy Markdown
Contributor

What does this PR do? What is the motivation?

The in-product MCP setup commands were changed by https://github.com/DataDog/web-ui/pull/330414 from a hardcoded US1-only URL (https://mcp.datadoghq.com/api/unstable/mcp-server/mcp?toolsets=llmobs,core) to a site-aware dynamic URL pattern (https://mcp.{site}/v1/mcp?toolsets=llmobs,core), and the UI now always emits the core toolset alongside llmobs. This updates the docs to match.

Changes

  • content/en/llm_observability/mcp_server.md (Agent skills → Install): replaced the hardcoded US1 claude mcp add command with a {{< region-param key="mcp_server_endpoint" >}}-based command wrapped in a {{< site-region region="us,us3,us5,eu,ap1,ap2,uk1" >}} block, updated the toolsets query string to llmobs,core, and added a {{< site-region region="gov,gov2" >}} unsupported-site alert — consistent with the rest of the page.
  • content/en/llm_observability/guide/claude_code_skills.md (Datadog MCP server): applied the same site-region–aware command, llmobs,core toolsets, and gov/gov2 unsupported-site alert.

Testing / validation

  • Verified the new blocks use the exact <pre><code> + {{< region-param >}} + {{< site-region >}} shortcode convention already used elsewhere on mcp_server.md (so the endpoint renders per selected Datadog site rather than being hardcoded).
  • Confirmed no remaining hardcoded mcp.datadoghq.com MCP URLs in the two edited sections.

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

Drafted with Bits Code (Datadog's AI coding agent), reviewed manually.

Additional notes


PR by Bits - View session in Datadog

Comment @DataDog to request changes

@datadog-official

datadog-official Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

View session in Datadog

Bits Code status: ✅ Done

Comment @DataDog to request changes

@mehulsonowal
mehulsonowal requested a review from a team as a code owner July 27, 2026 15:55
@datadog-datadog-prod-us1

Copy link
Copy Markdown
Contributor

I can only run on private repositories.

@github-actions github-actions Bot added the Guide Content impacting a guide label Jul 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

@mehulsonowal
mehulsonowal merged commit 70dfe32 into master Jul 29, 2026
22 of 26 checks passed
@mehulsonowal
mehulsonowal deleted the dd/mehul.sonowal/llmobs-mcp-site-aware-endpoint branch July 29, 2026 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bits AI Guide Content impacting a guide

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants