Repository navigation
[v3-3-test] Don't force full CI tests for non-test workflow and prek-only changes (#68802) - #68999
Merged
Merged
Conversation
…only changes (#68802) ENVIRONMENT_FILES forced the full test matrix on any .github/workflows change and on any scripts/ci/prek change. Those over-match: non-test workflows (security scans, doc publishing, notifications, backporting, stale/calendar bots) cannot affect test outcomes, and prek hooks drive static checks rather than the test matrix. Exclude non-test workflows from ENVIRONMENT_FILES (test/CI-config workflows like ci-amd, run-unit-tests, k8s-tests still force full), and move scripts/ci/prek to a new PREK_FILES group that triggers only the CI image build (so mypy-scripts and image-based static checks still run for a prek-only change) without forcing the full matrix. Static checks still run on every PR regardless; this only stops the unnecessary full test matrix for changes that cannot affect tests. (cherry picked from commit a4c269a) Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
1 task done
Lee-W
marked this pull request as ready for review
June 26, 2026 00:56
Lee-W
requested review from
amoghrajesh,
ashb,
bugraoz93,
choo121600,
ephraimbuddy,
gopidesupavan,
jason810496,
jedcunningham,
jscheffl,
potiuk and
vatsrahul1001
as code owners
June 26, 2026 00:56
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ENVIRONMENT_FILES forced the full test matrix on any .github/workflows
change and on any scripts/ci/prek change. Those over-match: non-test
workflows (security scans, doc publishing, notifications, backporting,
stale/calendar bots) cannot affect test outcomes, and prek hooks drive
static checks rather than the test matrix.
Exclude non-test workflows from ENVIRONMENT_FILES (test/CI-config
workflows like ci-amd, run-unit-tests, k8s-tests still force full), and
move scripts/ci/prek to a new PREK_FILES group that triggers only the CI
image build (so mypy-scripts and image-based static checks still run for
a prek-only change) without forcing the full matrix.
Static checks still run on every PR regardless; this only stops the
unnecessary full test matrix for changes that cannot affect tests.
(cherry picked from commit a4c269a)
Co-authored-by: Jarek Potiuk jarek@potiuk.com