Skip to content

ref(cells): Introduce new cell config keys#111749

Merged
lynnagara merged 2 commits intomasterfrom
update-config-to-cell-naming
Mar 27, 2026
Merged

ref(cells): Introduce new cell config keys#111749
lynnagara merged 2 commits intomasterfrom
update-config-to-cell-naming

Conversation

@lynnagara
Copy link
Copy Markdown
Member

introduce config that isn't used yet.
once this config is set in prod, we can flip usages in the codebase

introduce config that isn't used yet.
once this config is set in prod, we can flip usages in the codebase
@lynnagara lynnagara requested a review from a team March 27, 2026 20:02
@lynnagara lynnagara requested a review from a team as a code owner March 27, 2026 20:02
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Backend Test Failures

Failures on e94407e in this run:

tests/sentry/management/commands/test_generate_controlsilo_urls.py::TestGenerateControlsiloUrls::test_no_missing_urlslog
tests/sentry/management/commands/test_generate_controlsilo_urls.py:70: in test_no_missing_urls
    assert line in current_state, msg
E   AssertionError: 
E                 New control silo URL patterns detected!
E     
E                 The pattern:   new RegExp('^api/0/organizations/[^/]+/pipeline/[^/]+/$'),
E     
E                 Does not exist in the current pattern inventory. You should regenerate
E                 the pattern inventory with:
E     
E                 cd ../getsentry
E                 getsentry django generate_controlsilo_urls --format=js --output=../sentry/static/app/data/controlsiloUrlPatterns.ts
E     
E                 This command needs to be run in a getsentry environment
E                 in order to not lose patterns that are important for sentry.io
E                 
E   assert "  new RegExp('^api/0/organizations/[^/]+/pipeline/[^/]+/$')," in "// This is generated code.\n// To update it run `getsentry django generate_controlsilo_urls --format=js --output=/pat...gExp('^extensions/discord/link-identity/[^/]+/$'),\n  new RegExp('^extensions/discord/unlink-identity/[^/]+/$'),\n];\n"

@lynnagara lynnagara merged commit f2883c3 into master Mar 27, 2026
105 of 106 checks passed
@lynnagara lynnagara deleted the update-config-to-cell-naming branch March 27, 2026 21:41
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2026
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants