Skip to content

fix(seer-infra-telemetry): Switch Datadog whoami from MCP to API - #123283

Merged
srest2021 merged 3 commits into
masterfrom
srest2021/CW-1983
Sep 1, 2026
Merged

fix(seer-infra-telemetry): Switch Datadog whoami from MCP to API#123283
srest2021 merged 3 commits into
masterfrom
srest2021/CW-1983

Conversation

@srest2021

@srest2021 srest2021 commented Aug 31, 2026

Copy link
Copy Markdown
Member

fixes CW-1983

I was getting Datadog returned an unexpected response while validating credentials. meaning the Datadog whoami MCP changed response formats. The Datadog MCP is currently marked as unstable and under development, so switch the whoami on Datadog integration installation from MCP to API.

Use the https://docs.datadoghq.com/api/latest/users/get-current-user/ endpoint which has no additional scope requirements besides a pair of valid keys.

@linear-code

linear-code Bot commented Aug 31, 2026

Copy link
Copy Markdown

CW-1983

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 31, 2026
@srest2021 srest2021 changed the title switch from mcp whoami to api ref(seer-infra-telemetry): Switch Datadog whoami from MCP to API Aug 31, 2026
"ap2.datadoghq.com": "AP2",
"ddog-gov.com": "US1-FED",
"us2.ddog-gov.com": "US2-FED",
}

@srest2021 srest2021 Aug 31, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Preemptively move this thing over since this file is for Datadog user-level identity and therefore defunct.

@srest2021
srest2021 marked this pull request as ready for review August 31, 2026 23:48
@srest2021
srest2021 requested review from a team as code owners August 31, 2026 23:48
@srest2021
srest2021 requested review from shashjar and removed request for a team August 31, 2026 23:48
Comment thread src/sentry/integrations/datadog/client.py
@srest2021 srest2021 changed the title ref(seer-infra-telemetry): Switch Datadog whoami from MCP to API fix(seer-infra-telemetry): Switch Datadog whoami from MCP to API Sep 1, 2026
@srest2021
srest2021 enabled auto-merge (squash) September 1, 2026 17:14
@srest2021
srest2021 disabled auto-merge September 1, 2026 17:18
@srest2021
srest2021 requested a review from a team as a code owner September 1, 2026 17:19
@srest2021
srest2021 removed the request for review from a team September 1, 2026 17:19
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Sep 1, 2026
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

@srest2021

Copy link
Copy Markdown
Member Author

The frontend change is only a comment. So deploying both is safe

@srest2021
srest2021 enabled auto-merge (squash) September 1, 2026 17:21
@srest2021
srest2021 merged commit 13029e5 into master Sep 1, 2026
74 checks passed
@srest2021
srest2021 deleted the srest2021/CW-1983 branch September 1, 2026 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants