shahar1 opened a new pull request, #68053: URL: https://github.com/apache/airflow/pull/68053
On push to release branches like v3-2-test, the full test matrix was unconditionally running because `_should_run_all_tests_and_versions()` treated all PUSH events identically — forcing `full-tests-needed=true` regardless of branch. Root cause: the method returned `True` for every push event without checking whether the push was to a release branch or main. Since `main` has no push trigger in `ci-amd.yml`, this only affects release branches in practice. **Fix:** add a branch guard — push to a release branch (any branch ≠ `main`) returns `False` from `_should_run_all_tests_and_versions()`, letting selective checks decide which jobs to run based on changed files. Canaries (`SCHEDULE`) and manual triggers (`WORKFLOW_DISPATCH`) are unaffected and still run the full matrix. ## Test plan New parametrized test `test_push_to_release_branch_does_not_force_full_tests` covers three cases: - Push to `v3-2-test` → `all-versions=false` ✓ - Schedule (canary) on `main` → `all-versions=true` ✓ - Workflow dispatch on `main` → `all-versions=true` ✓ --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Haiku 4.5 Generated-by: Claude Haiku 4.5 following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
