Skip to content

chore(insights): remove starfish-view feature flag#113435

Merged
gggritso merged 2 commits intomasterfrom
chore/remove-starfish-view-flag
Apr 20, 2026
Merged

chore(insights): remove starfish-view feature flag#113435
gggritso merged 2 commits intomasterfrom
chore/remove-starfish-view-flag

Conversation

@gggritso
Copy link
Copy Markdown
Member

@gggritso gggritso commented Apr 20, 2026

Removes the starfish-view feature flag. The flag was declared in temporary.py but was not actually gating any real behavior — it appeared in batch_has() feature name lists in organization_events and organization_events_stats endpoints but the returned value was never consumed downstream.

Cleanup after the feature is fully released (effectively always-on).

  • Removed the declaration from src/sentry/features/temporary.py
  • Removed the listings in the two endpoint get_features methods
  • Removed the setUp() entries in the span indexed and span metrics test files

Companion PR (flagpole config removal): https://github.com/getsentry/sentry-options-automator/pull/7359

The starfish-view flag was declared but not actually gating any real
behavior — it appeared in batch_has() feature name lists for
organization_events and organization_events_stats endpoints but the
result was never consumed. Remove the declaration, the listings in
those endpoints, and the corresponding setUp() entries in the span
indexed and span metrics tests.
@gggritso gggritso requested review from a team as code owners April 20, 2026 15:23
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 20, 2026
@gggritso gggritso merged commit 24f3418 into master Apr 20, 2026
77 checks passed
@gggritso gggritso deleted the chore/remove-starfish-view-flag branch April 20, 2026 20:59
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