feat(seer-infra-telemetry): Reauthenticate monitoring provider in explorer#118592
Draft
srest2021 wants to merge 14 commits into
Draft
feat(seer-infra-telemetry): Reauthenticate monitoring provider in explorer#118592srest2021 wants to merge 14 commits into
srest2021 wants to merge 14 commits into
Conversation
Relocate the Datadog PAT connect modal from getsentry into open-source sentry/components/seer/ so it can be reused outside the billing-gated Seer Automation settings page (e.g. the Seer Explorer reauth flow). Pure relocation: the modal depends only on sentry/* and @sentry/scraps/*, so it works anywhere the open-source monitoring-provider endpoints do. The settings section (MonitoringProvidersSection) stays in getsentry and now imports the modal from its new location.
…1/explorer-reauth-monitoring-provider
…1/explorer-reauth-monitoring-provider
Contributor
📊 Type Coverage Diff
🔍 1 new type safety issue introducedType assertions (
This is informational only and does not block the PR. |
srest2021
added a commit
that referenced
this pull request
Jun 26, 2026
…ider endpoint (#118577) Adds `PUT` to the monitoring provider details endpoint to re-authenticate an existing connection. Looks up the user's existing connection for the provider and reuses its stored `site`. **Why:** the Seer Explorer can pause mid-run when a monitoring provider requires reauth (refresh token or PAT expired). For providers like Datadog that require site selection, the backend reuses the existing site stored in the identity, and user doesn't need to pick a site from the dropdown. Followup: #118592
Base automatically changed from
srest2021/monitoring-provider-reauth-put
to
master
June 26, 2026 21:15
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes CW-1546
Depends on #118577, https://github.com/getsentry/seer/pull/7133, #118616
Render a reconnect block, reauthenticate, and resume the Explorer run when a connected monitoring provider's refresh fails mid-run.
Datadog PAT reconnects in-place via the modal and auto-resumes the run. OAuth providers do a full-window redirect and (for now) need the explicit Resume button on return, a stopgap until CW-1557 redirects back into the Explorer and resumes automatically.
Expired at the beginning of the run:
Screen.Recording.2026-06-26.at.6.55.28.PM.mov
Expired in the middle of the run:
Screen.Recording.2026-06-26.at.6.56.56.PM.mov