You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by sh...@apache.org on 2026/06/06 07:05:00 UTC
(airflow) branch main updated: Revert "Fix CI/CD running full matrix on push to release branches (v3-X-test) (#68057)" (#68120)
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 <60...@users.noreply.github.com>
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))
-
-
-@pytest.mark.parametrize(
- ("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",
- ),
- ],
-)
-@patch.dict("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),
- )