This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch v3-2-test
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/v3-2-test by this push:
new e83c1d79cd8 [v3-2-test] Simplify ci_image_build and add test coverage
for remote logging e2e (#64697) (#64698)
e83c1d79cd8 is described below
commit e83c1d79cd856c339bfe763c79d0ccc41e858e54
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Sat Apr 4 15:18:50 2026 +0200
[v3-2-test] Simplify ci_image_build and add test coverage for remote
logging e2e (#64697) (#64698)
Refactor ci_image_build to delegate to prod_image_build instead of
duplicating conditions. Add ci-image-build assertions to all 5 remote
logging e2e test cases to verify CI image is built when remote logging
files change.
(cherry picked from commit 3cf4f51c2f09532d5ffa6c028694f7073fe90ffc)
Co-authored-by: Jarek Potiuk <[email protected]>
---
dev/breeze/src/airflow_breeze/utils/selective_checks.py | 4 +---
dev/breeze/tests/test_selective_checks.py | 5 +++++
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/dev/breeze/src/airflow_breeze/utils/selective_checks.py
b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
index 9c40ee982ee..5167505275a 100644
--- a/dev/breeze/src/airflow_breeze/utils/selective_checks.py
+++ b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
@@ -1065,12 +1065,10 @@ class SelectiveChecks:
self.run_unit_tests
or self.docs_build
or self.run_kubernetes_tests
- or self.run_task_sdk_integration_tests
- or self.run_airflow_ctl_integration_tests
- or self.run_helm_tests
or self.run_ui_tests
or self.pyproject_toml_changed
or self.any_provider_yaml_or_pyproject_toml_changed
+ or self.prod_image_build
)
@cached_property
diff --git a/dev/breeze/tests/test_selective_checks.py
b/dev/breeze/tests/test_selective_checks.py
index 9d674bd761d..6135a9f7739 100644
--- a/dev/breeze/tests/test_selective_checks.py
+++ b/dev/breeze/tests/test_selective_checks.py
@@ -1256,6 +1256,7 @@ def assert_outputs_are_printed(expected_outputs:
dict[str, str], stderr: str):
{
"run-remote-logging-s3-e2e-tests": "true",
"run-remote-logging-elasticsearch-e2e-tests": "false",
+ "ci-image-build": "true",
"prod-image-build": "true",
},
id="S3 remote logging changes enable only S3 e2e",
@@ -1267,6 +1268,7 @@ def assert_outputs_are_printed(expected_outputs:
dict[str, str], stderr: str):
{
"run-remote-logging-s3-e2e-tests": "false",
"run-remote-logging-elasticsearch-e2e-tests": "true",
+ "ci-image-build": "true",
"prod-image-build": "true",
},
id="Elasticsearch remote logging changes enable only
Elasticsearch e2e",
@@ -1278,6 +1280,7 @@ def assert_outputs_are_printed(expected_outputs:
dict[str, str], stderr: str):
{
"run-remote-logging-s3-e2e-tests": "false",
"run-remote-logging-elasticsearch-e2e-tests": "true",
+ "ci-image-build": "true",
"prod-image-build": "true",
},
id="Elasticsearch helper changes enable Elasticsearch e2e",
@@ -1289,6 +1292,7 @@ def assert_outputs_are_printed(expected_outputs:
dict[str, str], stderr: str):
{
"run-remote-logging-s3-e2e-tests": "true",
"run-remote-logging-elasticsearch-e2e-tests": "true",
+ "ci-image-build": "true",
"prod-image-build": "true",
},
id="Shared remote logging changes enable both remote logging
e2e jobs",
@@ -1300,6 +1304,7 @@ def assert_outputs_are_printed(expected_outputs:
dict[str, str], stderr: str):
{
"run-remote-logging-s3-e2e-tests": "true",
"run-remote-logging-elasticsearch-e2e-tests": "true",
+ "ci-image-build": "true",
"prod-image-build": "true",
},
id="Shared logging library changes enable both remote logging
e2e jobs",
