Skip to content

locator: enable locator to request subset of localities#114

Open
lynnagara wants to merge 4 commits intomainfrom
allow-locality-subset-in-locator
Open

locator: enable locator to request subset of localities#114
lynnagara wants to merge 4 commits intomainfrom
allow-locality-subset-in-locator

Conversation

@lynnagara
Copy link
Member

@lynnagara lynnagara commented Jan 14, 2026

when synapse is deployed in a locality it should only request the mappings for that locality not the others.

for example, there is no reason for the US synapse to be loading DE or single-tenant mappings.

this provides better data isolation as well as memory and network efficiency.

when synapse is deployed in a locality it should only request the
mappings for that locality not the others. for example, there is no
reason for the US synapse to be loading DE or single-tenant mappings.

this provides better data isolation as well as memory and network efficiency.
@lynnagara lynnagara changed the title locator: enable locator request subset of localities locator: enable locator to request subset of localities Jan 14, 2026
Copy link
Member

@markstory markstory left a comment

Choose a reason for hiding this comment

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

Makes sense to me. I'm assuming we'll also need to snake this through into ControlSilo?

markstory added a commit to getsentry/sentry that referenced this pull request Jan 30, 2026
Each synapse should only fetch mappings for the locale/region it is in.

Refs getsentry/synapse#114
markstory added a commit to getsentry/sentry that referenced this pull request Feb 3, 2026
Each synapse should only fetch mappings for the locale/region it is in.

Refs getsentry/synapse#114
@markstory
Copy link
Member

I've added a locale filter to the org-cell-mappings endpoint to do filtering by locale. Right now it is wired up to the region_name field of OrganizationMemberMapping, but once sentry understands locales + cells it can be reworked.

shellmayr pushed a commit to getsentry/sentry that referenced this pull request Feb 4, 2026
Each synapse should only fetch mappings for the locale/region it is in.

Refs getsentry/synapse#114
jaydgoss pushed a commit to getsentry/sentry that referenced this pull request Feb 12, 2026
Each synapse should only fetch mappings for the locale/region it is in.

Refs getsentry/synapse#114
@lynnagara lynnagara marked this pull request as ready for review February 17, 2026 23:04
@lynnagara lynnagara requested a review from a team as a code owner February 17, 2026 23:04
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