Skip to content

feat(seer-infra-telemetry): Show GCP connection status on the integration settings page - #123258

Open
shashjar wants to merge 3 commits into
masterfrom
shashjar/show-gcp-connection-status-on-integration-settings-page
Open

feat(seer-infra-telemetry): Show GCP connection status on the integration settings page#123258
shashjar wants to merge 3 commits into
masterfrom
shashjar/show-gcp-connection-status-on-integration-settings-page

Conversation

@shashjar

@shashjar shashjar commented Aug 31, 2026

Copy link
Copy Markdown
Member

PR 2 for https://linear.app/getsentry/issue/CW-1668/build-gcp-mcp-connected-state-error-state-ui.

Adds a connection status block to the GCP integration settings page: the current status, the failure message and remediation when it isn't connected, when it was last checked, and a Re-test button.

Re-test posts to verify-connection and refreshes. The endpoint records its own result on the organization integration config, so nothing else is needed.

@github-actions github-actions Bot added Scope: Frontend Automatically applied to PRs that change frontend components Scope: Backend Automatically applied to PRs that change backend components labels Aug 31, 2026
@github-actions

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.

@shashjar
shashjar force-pushed the shashjar/show-gcp-connection-status-on-integration-settings-page branch from c65c192 to fa85016 Compare August 31, 2026 22:07
@sentry

sentry Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Sentry Snapshot Testing

Name Added Removed Changed Renamed Unchanged Skipped Status
sentry-frontend
sentry-frontend
0 0 0 0 481 0 ✅ Unchanged

⚙️ sentry-frontend Snapshot Settings

@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

✅ no issues found

@shashjar shashjar changed the title feat(seer-infra-telemetry): Make GCP integration settings editable and re-verify on change feat(seer-infra-telemetry): Show GCP connection status on the integration settings page Aug 31, 2026
@shashjar
shashjar requested a review from srest2021 August 31, 2026 22:15
@shashjar
shashjar marked this pull request as ready for review August 31, 2026 22:15
@shashjar
shashjar requested review from a team as code owners August 31, 2026 22:15
@shashjar
shashjar removed request for a team August 31, 2026 22:16
organization: Organization;
}

export function GcpConnectionStatus({

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.

will get some screenshots of what this looks like from preview deployment and then add to PR

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit fa85016. Configure here.

Comment thread static/app/views/settings/organizationIntegrations/gcpConnectionStatus.tsx Outdated
Base automatically changed from shashjar/expose-gcp-connection-status-in-integration-config-data to master August 31, 2026 22:21
@shashjar
shashjar requested a review from a team as a code owner August 31, 2026 22:21
@shashjar
shashjar force-pushed the shashjar/show-gcp-connection-status-on-integration-settings-page branch from fa85016 to ea3761a Compare August 31, 2026 22:21
@linear-code

linear-code Bot commented Aug 31, 2026

Copy link
Copy Markdown

CW-1668

@shashjar
shashjar removed the request for review from a team August 31, 2026 22:50
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.

1 participant