Skip to content

chore(github-enterprise): Remove fully-GA github.com source flag checks#116385

Merged
tnt-sentry merged 1 commit into
masterfrom
tnt-sentry/remove-ghe-github-com-flag
May 28, 2026
Merged

chore(github-enterprise): Remove fully-GA github.com source flag checks#116385
tnt-sentry merged 1 commit into
masterfrom
tnt-sentry/remove-ghe-github-com-flag

Conversation

@tnt-sentry
Copy link
Copy Markdown
Contributor

Remove the features.has("organizations:github-enterprise-github-com-source", ...) guards from the two GHE setup pipelines (form-based InstallationConfigView and API-driven GHEInstallationConfigApiStep), along with the tests that exercised the rejection path.

The flag is at 100% rollout (segment GA, no conditions) in flagpole, so every org already passes the check — removing it makes that explicit and lets us proceed with the standard flagpole removal sequence.

This is step 1 of three. Follow-ups:

  1. Remove the feature.organizations:github-enterprise-github-com-source block from sentry-options-automator/options/default/flagpole.yaml.
  2. Remove the manager.add(...) registration from src/sentry/features/temporary.py.

Each waits for the previous to deploy.

The `organizations:github-enterprise-github-com-source` flag is at 100%
rollout (segment `GA`, no conditions) in flagpole. Drop the `features.has`
guards in both the form-based `InstallationConfigView` and the API-driven
`GHEInstallationConfigApiStep`, and the tests that exercised the rejection
path. github.com installs now succeed for every organization.

This is step 1 of the standard flagpole removal order. Follow-up PRs will
remove the flag config from sentry-options-automator and the registration
from `temporary.py`.
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 28, 2026
@tnt-sentry tnt-sentry marked this pull request as ready for review May 28, 2026 15:44
@tnt-sentry tnt-sentry requested a review from a team as a code owner May 28, 2026 15:44
@tnt-sentry tnt-sentry merged commit 324798a into master May 28, 2026
65 checks passed
@tnt-sentry tnt-sentry deleted the tnt-sentry/remove-ghe-github-com-flag branch May 28, 2026 16:00
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