Skip to content

feat(ci): selective testing escape hatch#109136

Merged
joshuarli merged 1 commit intomasterfrom
ci-add-escape-hatch
Feb 23, 2026
Merged

feat(ci): selective testing escape hatch#109136
joshuarli merged 1 commit intomasterfrom
ci-add-escape-hatch

Conversation

@joshuarli
Copy link
Member

this adds the escape hatch for selective testing - apply the label Trigger: Override Selective Testing and backend-selective.yml will re-run, this time running the full suite!

@joshuarli joshuarli requested review from a team as code owners February 23, 2026 21:04
@joshuarli joshuarli requested a review from rbro112 February 23, 2026 21:05
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Feb 23, 2026
@getsentry getsentry deleted a comment from github-actions bot Feb 23, 2026
@joshuarli
Copy link
Member Author

joshuarli commented Feb 23, 2026

Umm... i'm going to delete the label syncer, it's been practically untouched in 4 years and wants to now delete a bunch of labels that we haven't defined in it: https://github.com/getsentry/sentry/actions/runs/22324782722/job/64592270442?pr=109136

#109140

Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

id: calculate-shards
env:
SELECTED_TESTS_FILE: ${{ needs.prepare-selective-tests.outputs.has-selected-tests == 'true' && '.artifacts/selected-tests.txt' || '' }}
SELECTED_TESTS_FILE: "${{ !contains(github.event.pull_request.labels.*.name, 'Trigger: Override Selective Testing') && needs.prepare-selective-tests.outputs.has-selected-tests == 'true' && '.artifacts/selected-tests.txt' || '' }}"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Override label check may not apply

Medium Severity

The override behavior hinges on contains(github.event.pull_request.labels.*.name, 'Trigger: Override Selective Testing'). If the pull_request.labels list in the labeled event payload doesn’t yet include the newly applied label (or is truncated), SELECTED_TESTS_FILE stays set and the run won’t switch to the full suite despite the workflow being triggered by the override label.

Additional Locations (1)

Fix in Cursor Fix in Web

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this label already exists

@joshuarli joshuarli merged commit bdebae8 into master Feb 23, 2026
48 checks passed
@joshuarli joshuarli deleted the ci-add-escape-hatch branch February 23, 2026 21:53
mchen-sentry pushed a commit that referenced this pull request Feb 24, 2026
this adds the escape hatch for selective testing - apply the label
`Trigger: Override Selective Testing` and backend-selective.yml will
re-run, this time running the full suite!
wedamija pushed a commit that referenced this pull request Feb 24, 2026
this adds the escape hatch for selective testing - apply the label
`Trigger: Override Selective Testing` and backend-selective.yml will
re-run, this time running the full suite!
@github-actions github-actions bot locked and limited conversation to collaborators Mar 11, 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