This is an automated email from the ASF dual-hosted git repository.
shahar1 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 0c5da3d3d37 Revert "Fix CI/CD running full matrix on push to release
branches (v3-X-test) (#68057)" (#68120)
0c5da3d3d37 is described below
commit 0c5da3d3d377126ccc413c4cdeadd61d0244754d
Author: Shahar Epstein <[email protected]>
AuthorDate: Sat Jun 6 10:04:51 2026 +0300
Revert "Fix CI/CD running full matrix on push to release branches
(v3-X-test) (#68057)" (#68120)
This reverts commit 834385195f525f841d250cc8e6a4e00c51e165b4.
---
.../src/airflow_breeze/utils/selective_checks.py | 4 --
dev/breeze/tests/test_selective_checks.py | 56 ----------------------
2 files changed, 60 deletions(-)
diff --git a/dev/breeze/src/airflow_breeze/utils/selective_checks.py
b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
index 6bf7b589b1a..53da70d9b0d 100644
--- a/dev/breeze/src/airflow_breeze/utils/selective_checks.py
+++ b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
@@ -656,10 +656,6 @@ class SelectiveChecks:
f"[warning]Only text non doc files changed in
{self._github_event}, skip full tests[/]"
)
return False
- # On push to release branches (v3-X-test, etc), only run selective
tests.
- # Canaries (SCHEDULE) and manual triggers (WORKFLOW_DISPATCH)
still run full matrix.
- if self._github_event == GithubEvents.PUSH and
self._default_branch != "main":
- return False
console_print(f"[warning]Running everything because event is
{self._github_event}[/]")
return True
if not self._commit_ref:
diff --git a/dev/breeze/tests/test_selective_checks.py
b/dev/breeze/tests/test_selective_checks.py
index 1df9e4463cb..29de650f108 100644
--- a/dev/breeze/tests/test_selective_checks.py
+++ b/dev/breeze/tests/test_selective_checks.py
@@ -3842,59 +3842,3 @@ def test_helm_test_kubernetes_versions(
default_branch="main",
)
assert_outputs_are_printed(expected_outputs, str(stderr))
-
-
[email protected](
- ("github_event", "default_branch", "expected_all_versions"),
- [
- pytest.param(
- GithubEvents.PUSH,
- "v3-2-test",
- "false",
- id="Push to release branch does not force all versions",
- ),
- pytest.param(
- GithubEvents.SCHEDULE,
- "main",
- "true",
- id="Schedule (canary) forces all versions",
- ),
- pytest.param(
- GithubEvents.WORKFLOW_DISPATCH,
- "main",
- "true",
- id="Workflow dispatch forces all versions",
- ),
- ],
-)
[email protected]("os.environ", {"GITHUB_TOKEN": "test_token"})
-@patch("requests.get")
-def test_push_to_release_branch_does_not_force_full_tests(
- mock_get, github_event, default_branch, expected_all_versions
-):
- """Test that push to release branches (v3-X-test) does not force full test
matrix,
- while canaries (SCHEDULE) and manual triggers still do."""
- # Mock GitHub API calls for runner_type property (used in PUSH/SCHEDULE
events)
- workflow_response = Mock()
- workflow_response.status_code = 200
- workflow_response.json.return_value = {"workflow_runs": [{"jobs_url":
"https://api.github.com/jobs/123"}]}
- jobs_response = Mock()
- jobs_response.status_code = 200
- jobs_response.json.return_value = {
- "jobs": [{"name": "Basic tests (ubuntu-22.04)", "labels":
["ubuntu-22.04"]}]
- }
- mock_get.side_effect = [workflow_response, jobs_response]
-
- stderr = SelectiveChecks(
- files=("airflow-core/src/airflow/models/dag.py",),
- commit_ref=NEUTRAL_COMMIT,
- github_event=github_event,
- pr_labels=(),
- default_branch=default_branch,
- )
- assert_outputs_are_printed(
- {
- "all-versions": expected_all_versions,
- },
- str(stderr),
- )
