Skip to content

fix(seer): add GitLab support to web vitals Seer eligibility check#117746

Open
billyvg wants to merge 2 commits into
masterfrom
billy/cw-1504-be-web-vitals-gitlab-providers
Open

fix(seer): add GitLab support to web vitals Seer eligibility check#117746
billyvg wants to merge 2 commits into
masterfrom
billy/cw-1504-be-web-vitals-gitlab-providers

Conversation

@billyvg

@billyvg billyvg commented Jun 15, 2026

Copy link
Copy Markdown
Member

Fixes CW-1504.

What

check_seer_setup_for_project in tasks/web_vitals_issue_detection.py was checking repos against the static SEER_SUPPORTED_SCM_PROVIDERS list, which only contains GitHub and GitHub Enterprise. GitLab repos were silently excluded even when the organizations:seer-gitlab-support flag is on.

Changes:

  • Swap SEER_SUPPORTED_SCM_PROVIDERS import for get_supported_scm_providers from seer_setup
  • Replace github_repos local variable with supported_repos
  • Update docstring that said "github code mapping"

Why

Brings this task's eligibility gate in line with the rest of the Seer stack, which already uses the feature-flag-aware helper. Companion FE PR: https://github.com/getsentry/sentry/compare/billy/cw-1504-fe-web-vitals-gitlab-providers


View Session in Sentry

@linear-code

linear-code Bot commented Jun 15, 2026

Copy link
Copy Markdown

CW-1504

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jun 15, 2026
@billyvg billyvg marked this pull request as ready for review June 15, 2026 21:52
@billyvg billyvg requested a review from a team as a code owner June 15, 2026 21:52
@billyvg

billyvg commented Jun 15, 2026

Copy link
Copy Markdown
Member Author

@edwardgou-sentry there's no specific reason this wouldn't work for gitlab right? gitlab autofix works and has code mappings

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