Skip to content

feat(github-enterprise): Route installation_repositories to control silo#112245

Merged
wedamija merged 1 commit intomasterfrom
danf/ghe-routing
Apr 6, 2026
Merged

feat(github-enterprise): Route installation_repositories to control silo#112245
wedamija merged 1 commit intomasterfrom
danf/ghe-routing

Conversation

@wedamija
Copy link
Copy Markdown
Member

@wedamija wedamija commented Apr 3, 2026

Remove the should_route_to_control_silo override in GithubEnterpriseRequestParser so it inherits the parent's routing, which sends installation_repositories events to control silo.

We do this to make sure the webhooks for GHE will also route to the right place

@wedamija wedamija requested a review from a team April 3, 2026 23:26
@wedamija wedamija requested a review from a team as a code owner April 3, 2026 23:26
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 3, 2026
Comment thread src/sentry/middleware/integrations/parsers/github_enterprise.py
@wedamija wedamija force-pushed the danf/repo-sync-audit-logs branch from 7af940e to c7591b2 Compare April 6, 2026 17:55
@wedamija wedamija requested review from a team as code owners April 6, 2026 17:55
Remove the should_route_to_control_silo override in GithubEnterpriseRequestParser so it inherits the parent's routing, which sends installation_repositories events to control silo.

We do this to make sure the webhooks for GHE will also route to the right place
@wedamija wedamija changed the base branch from danf/repo-sync-audit-logs to master April 6, 2026 17:56
@wedamija wedamija merged commit 95827eb into master Apr 6, 2026
59 checks passed
@wedamija wedamija deleted the danf/ghe-routing branch April 6, 2026 21:33
george-sentry pushed a commit that referenced this pull request Apr 9, 2026
…ilo (#112245)

Remove the should_route_to_control_silo override in GithubEnterpriseRequestParser so it inherits the parent's routing, which sends installation_repositories events to control silo.

We do this to make sure the webhooks for GHE will also route to the right place

<!-- Describe your PR here. -->
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.

2 participants