Skip to content

chore(API): Remove unused grouping configs endpoint#104303

Merged
ceorourke merged 3 commits intomasterfrom
ceorourke/rm-old-grouping-configs-endpoint
Dec 3, 2025
Merged

chore(API): Remove unused grouping configs endpoint#104303
ceorourke merged 3 commits intomasterfrom
ceorourke/rm-old-grouping-configs-endpoint

Conversation

@ceorourke
Copy link
Copy Markdown
Member

As far as I can tell this endpoint is not used in favor of ProjectGroupingConfigsEndpoint https://github.com/getsentry/sentry/blob/master/src/sentry/issues/endpoints/project_grouping_configs.py

It isn't published so we don't have to go through the deprecation process, and the front end only uses the project scoped one.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 3, 2025
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 3, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 3, 2025

🚨 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.

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 3, 2025

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
30135 2 30133 240
View the top 2 failed test(s) by shortest run time
tests.sentry.hybridcloud.apigateway.test_apigateway.ApiGatewayTest__InMonolithMode::test_region_pinned_urls_are_defined
Stack Traces | 0.467s run time
#x1B[1m#x1B[.../hybridcloud/apigateway/test_apigateway.py#x1B[0m:72: in test_region_pinned_urls_are_defined
    assert (
#x1B[1m#x1B[31mE   AssertionError: REGION_PINNED_URL_NAMES contains sentry-api-0-grouping-configs, but no route is registered with that name#x1B[0m
#x1B[1m#x1B[31mE   assert None#x1B[0m
tests.sentry.hybridcloud.apigateway.test_apigateway.ApiGatewayTest::test_region_pinned_urls_are_defined
Stack Traces | 0.632s run time
#x1B[1m#x1B[.../hybridcloud/apigateway/test_apigateway.py#x1B[0m:72: in test_region_pinned_urls_are_defined
    assert (
#x1B[1m#x1B[31mE   AssertionError: REGION_PINNED_URL_NAMES contains sentry-api-0-grouping-configs, but no route is registered with that name#x1B[0m
#x1B[1m#x1B[31mE   assert None#x1B[0m

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@ceorourke ceorourke force-pushed the ceorourke/rm-old-grouping-configs-endpoint branch from 8b59c06 to f6468f0 Compare December 3, 2025 19:08
@ceorourke ceorourke marked this pull request as ready for review December 3, 2025 19:09
@ceorourke ceorourke requested review from a team as code owners December 3, 2025 19:09
@ceorourke ceorourke requested a review from a team December 3, 2025 19:09
@ceorourke ceorourke force-pushed the ceorourke/rm-old-grouping-configs-endpoint branch from 8b79345 to d1912c6 Compare December 3, 2025 20:09
@ceorourke ceorourke merged commit b02aed5 into master Dec 3, 2025
67 checks passed
@ceorourke ceorourke deleted the ceorourke/rm-old-grouping-configs-endpoint branch December 3, 2025 20:41
@github-actions github-actions bot locked and limited conversation to collaborators Dec 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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