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 3cf4f51c2f0 Simplify ci_image_build and add test coverage for remote
logging e2e (#64697)
3cf4f51c2f0 is described below
commit 3cf4f51c2f09532d5ffa6c028694f7073fe90ffc
Author: Jarek Potiuk <[email protected]>
AuthorDate: Fri Apr 3 23:26:43 2026 +0200
Simplify ci_image_build and add test coverage for remote logging e2e
(#64697)
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.
---
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 2cfbd1c4fc9..99e74fcfa50 100644
--- a/dev/breeze/src/airflow_breeze/utils/selective_checks.py
+++ b/dev/breeze/src/airflow_breeze/utils/selective_checks.py
@@ -1081,12 +1081,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 f9b20225132..5d6ff376872 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",
