You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by po...@apache.org on 2024/04/18 10:14:50 UTC
(airflow) branch main updated: Switch committer builds to public runners (#39106)
This is an automated email from the ASF dual-hosted git repository.
potiuk 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 ab9444e0c2 Switch committer builds to public runners (#39106)
ab9444e0c2 is described below
commit ab9444e0c26df1f2acfb83bb47bb64acefec437a
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Thu Apr 18 12:14:43 2024 +0200
Switch committer builds to public runners (#39106)
Following the discussion on devlist, we are switching committer buids to
public runners.
Discussion:
* https://lists.apache.org/thread/f2kxdjm12nf2rh0tqxmv3rj49sjd6c3w
---
.../src/airflow_breeze/utils/selective_checks.py | 5 +----
dev/breeze/tests/test_selective_checks.py | 20 ++++++++++----------
2 files changed, 11 insertions(+), 14 deletions(-)
diff --git a/dev/breeze/src/airflow_breeze/utils/selective_checks.py b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
index ee12f2881c..6b42d91346 100644
--- a/dev/breeze/src/airflow_breeze/utils/selective_checks.py
+++ b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
@@ -1090,10 +1090,7 @@ class SelectiveChecks:
if USE_SELF_HOSTED_RUNNERS_LABEL in self._pr_labels:
# Forced self-hosted runners
return RUNS_ON_SELF_HOSTED_RUNNER
- if actor in COMMITTERS:
- return RUNS_ON_SELF_HOSTED_RUNNER
- else:
- return RUNS_ON_PUBLIC_RUNNER
+ return RUNS_ON_PUBLIC_RUNNER
return RUNS_ON_PUBLIC_RUNNER
@cached_property
diff --git a/dev/breeze/tests/test_selective_checks.py b/dev/breeze/tests/test_selective_checks.py
index 099478ea9c..e2ee1f0a6b 100644
--- a/dev/breeze/tests/test_selective_checks.py
+++ b/dev/breeze/tests/test_selective_checks.py
@@ -1700,13 +1700,13 @@ def test_helm_tests_trigger_ci_build(files: tuple[str, ...], expected_outputs: d
"apache/airflow",
(),
dict(),
- '["self-hosted", "Linux", "X64"]',
- "true",
- "true",
- "true",
+ '["ubuntu-22.04"]',
+ "false",
"false",
"true",
"false",
+ "false",
+ "false",
False,
id="Pull request committer",
),
@@ -1780,13 +1780,13 @@ def test_helm_tests_trigger_ci_build(files: tuple[str, ...], expected_outputs: d
"apache/airflow",
[],
dict(),
- '["self-hosted", "Linux", "X64"]',
- "true",
- "true",
- "true",
+ '["ubuntu-22.04"]',
+ "false",
"false",
"true",
"false",
+ "false",
+ "false",
False,
id="Pull request target committer",
),
@@ -2065,7 +2065,7 @@ def test_mypy_matches(
("README.md",),
{
"is-committer-build": "true",
- "runs-on-as-json-default": '["self-hosted", "Linux", "X64"]',
+ "runs-on-as-json-default": '["ubuntu-22.04"]',
},
"potiuk",
(),
@@ -2075,7 +2075,7 @@ def test_mypy_matches(
("README.md",),
{
"is-committer-build": "false",
- "runs-on-as-json-default": '["self-hosted", "Linux", "X64"]',
+ "runs-on-as-json-default": '["ubuntu-22.04"]',
},
"potiuk",
("non committer build",),