Skip to content

feat(cells) Allow staff users to create orgs in hidden cells#116503

Open
markstory wants to merge 2 commits into
masterfrom
prodeng-1457-staff-org-create
Open

feat(cells) Allow staff users to create orgs in hidden cells#116503
markstory wants to merge 2 commits into
masterfrom
prodeng-1457-staff-org-create

Conversation

@markstory
Copy link
Copy Markdown
Member

Staff users should be allowed to create orgs in hidden locale/cells. We'll use this when we're testing new cells

Fixes INFRENG-325

Staff users should be allowed to create orgs in hidden locale/cells.
We'll use this when we're testing new cells

Fixes INFRENG-325
@markstory markstory requested a review from a team as a code owner May 29, 2026 18:30
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 29, 2026

INFRENG-325

PRODENG-1457

@markstory markstory requested a review from a team May 29, 2026 18:30
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 29, 2026
Copy link
Copy Markdown
Member

@strongs strongs left a comment

Choose a reason for hiding this comment

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

nice one

Comment thread src/sentry/core/endpoints/organization_index.py Outdated
Comment thread src/sentry/core/endpoints/organization_index.py Outdated
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

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 d85af2f. Configure here.

Comment thread src/sentry/core/endpoints/organization_index.py
locality = get_locality_by_name(value)
except CellResolutionError:
raise serializers.ValidationError(f"Unknown data storage location {value!r}.")
if "request" in self.context and is_active_staff(self.context["request"]):
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we still want to restrict this to MULTI_TENANT cells only? I can't think of a use case where this wouldn't be true.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants