This is an automated email from the ASF dual-hosted git repository. potiuk pushed a commit to branch test-failing-composite-action in repository https://gitbox.apache.org/repos/asf/airflow.git
commit 346bdb040eeb4c42156c0db7cc80bc83bc5464fd Author: Jarek Potiuk <[email protected]> AuthorDate: Mon Oct 30 21:31:12 2023 +0100 Testing --- Dockerfile.ci | 2 +- .../src/airflow_breeze/commands/ci_image_commands.py | 2 +- dev/breeze/src/airflow_breeze/utils/parallel.py | 18 ++++++++---------- 3 files changed, 10 insertions(+), 12 deletions(-) diff --git a/Dockerfile.ci b/Dockerfile.ci index 48301ba232..a22d3fdee5 100644 --- a/Dockerfile.ci +++ b/Dockerfile.ci @@ -16,7 +16,7 @@ # # WARNING: THIS DOCKERFILE IS NOT INTENDED FOR PRODUCTION USE OR DEPLOYMENT. # -ARG PYTHON_BASE_IMAGE="python:3.8-slim-bullseye" +ARGxxx PYTHON_BASE_IMAGE="python:3.8-slim-bullseye" ############################################################################################## # This is the script image where we keep all inlined bash scripts needed in other segments 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 f5ab0b2d0f..bafbbd2e77 100644 --- a/dev/breeze/src/airflow_breeze/commands/ci_image_commands.py +++ b/dev/breeze/src/airflow_breeze/commands/ci_image_commands.py @@ -152,6 +152,7 @@ def run_build_in_parallel( ) for index, image_params in enumerate(image_params_list) ] + sys.exit(1) check_async_run_results( results=results, success="All images built correctly", @@ -258,7 +259,6 @@ def build( **kwargs: dict[str, Any], ): """Build CI image. Include building multiple images for all python versions.""" - def run_build(ci_image_params: BuildCiParams) -> None: return_code, info = run_build_ci_image( ci_image_params=ci_image_params, diff --git a/dev/breeze/src/airflow_breeze/utils/parallel.py b/dev/breeze/src/airflow_breeze/utils/parallel.py index ea4ef06030..80415cc0e4 100644 --- a/dev/breeze/src/airflow_breeze/utils/parallel.py +++ b/dev/breeze/src/airflow_breeze/utils/parallel.py @@ -440,16 +440,14 @@ def check_async_run_results( get_console().print(f"\n[info]Summary: {outputs[i].escaped_title:<30}:\n") if print_lines: print(line) - try: - if errors: - get_console().print("\n[error]There were errors when running some tasks. Quitting.[/]\n") - sys.exit(1) - else: - get_console().print(f"\n[success]{success}[/]\n") - finally: - if not skip_cleanup: - for output in outputs: - Path(output.file_name).unlink(missing_ok=True) + if not skip_cleanup: + for output in outputs: + Path(output.file_name).unlink(missing_ok=True) + if errors: + get_console().print("\n[error]There were errors when running some tasks. Quitting.[/]\n") + sys.exit(1) + else: + get_console().print(f"\n[success]{success}[/]\n") @contextmanager
