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 1fc551b146 Fail image build in case dependencies are not regenerated
(#39847)
1fc551b146 is described below
commit 1fc551b146d59feb8a5f2af1887cd0a69656d7ea
Author: Jarek Potiuk <[email protected]>
AuthorDate: Sun May 26 18:02:35 2024 +0200
Fail image build in case dependencies are not regenerated (#39847)
When dependencies are not regenerated before pushing the PR, the
CI image build should fail rather than retry with regenerated
dependencies.
---
dev/breeze/src/airflow_breeze/commands/ci_image_commands.py | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/dev/breeze/src/airflow_breeze/commands/ci_image_commands.py
b/dev/breeze/src/airflow_breeze/commands/ci_image_commands.py
index 3dd8f5c179..cc6274775f 100644
--- a/dev/breeze/src/airflow_breeze/commands/ci_image_commands.py
+++ b/dev/breeze/src/airflow_breeze/commands/ci_image_commands.py
@@ -788,7 +788,7 @@ def run_build_ci_image(
)
else:
env = get_docker_build_env(ci_image_params)
- subprocess.run(
+ process = subprocess.run(
[
sys.executable,
os.fspath(
@@ -801,6 +801,8 @@ def run_build_ci_image(
],
check=False,
)
+ if process.returncode != 0:
+ sys.exit(process.returncode)
get_console(output=output).print(f"\n[info]Building CI Image for
{param_description}\n")
build_command_result = run_command(
prepare_docker_build_command(
