Skip to content

ref(allocation-policies): Remove CrossOrgQueryAllocationPolicy - #8654

Merged
dezzeddi merged 2 commits into
masterfrom
dani/remove-dead-allocation-policies
Nov 3, 2026
Merged

dezzeddi merged 2 commits into
masterfrom
dani/remove-dead-allocation-policies

Conversation

@dezzeddi

@dezzeddi dezzeddi commented Nov 2, 2026 •

Copy link
Copy Markdown
Member

Removes CrossOrgQueryAllocationPolicy, which never limits a query.

  • It only acts on queries tagged cross_org_query, and none reach the storages it was configured on.
  • Its config is already gone (getsentry/sentry-options-automator#10432). An unknown policy name in config is skipped, so deploy order doesn't matter.

It only limits queries with tenant_ids.cross_org_query set or a referrer in
cross_org_referrer_limits. No config sets cross_org_referrer_limits, and
cross_org_query is only set by the outcomes routing queries, which read storages
the policy is not configured on, so it passes every query through.
@dezzeddi dezzeddi changed the title ref(allocation-policies): Remove CrossOrgQuery and DeleteConcurrentRateLimit policies ref(allocation-policies): Remove CrossOrgQueryAllocationPolicy Nov 2, 2026
@dezzeddi
dezzeddi force-pushed the dani/remove-dead-allocation-policies branch from b7f1166 to 7805511 Compare November 2, 2026 23:54
@dezzeddi
dezzeddi marked this pull request as ready for review November 3, 2026 19:51
@dezzeddi
dezzeddi requested a review from a team as a code owner November 3, 2026 19:51
@dezzeddi
dezzeddi merged commit b66d3ae into master Nov 3, 2026
73 checks passed
@dezzeddi
dezzeddi deleted the dani/remove-dead-allocation-policies branch November 3, 2026 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants