From c8a1cbcc6db4c8c1144e9d66c67b3096edac32bf Mon Sep 17 00:00:00 2001 From: Ibrahim Halatci Date: Mon, 29 Jun 2026 13:48:01 +0300 Subject: [PATCH 1/2] diag: surface stdout+stderr on build_package failure Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- packaging_automation/citus_package.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/packaging_automation/citus_package.py b/packaging_automation/citus_package.py index 59fee1c9..b039abb9 100644 --- a/packaging_automation/citus_package.py +++ b/packaging_automation/citus_package.py @@ -367,7 +367,11 @@ def build_package( if output.stdout: print("Output:" + output.stdout) if output.returncode != 0: - raise ValueError(output.stderr) + raise ValueError( + f"Build failed (rc={output.returncode}).\n" + f"STDOUT:\n{output.stdout}\n" + f"STDERR:\n{output.stderr}" + ) if input_output_parameters.output_validation: validate_output( From 8cac569e8ae8e66e062a8eaba9122b78a05bc223 Mon Sep 17 00:00:00 2001 From: Ibrahim Halatci Date: Mon, 20 Jul 2026 14:05:24 +0300 Subject: [PATCH 2/2] diag: print build stdout only on success (addresses review nit) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 21828aca-24f9-441b-8e36-981b7666b5c0 --- packaging_automation/citus_package.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packaging_automation/citus_package.py b/packaging_automation/citus_package.py index b039abb9..d1eb1e31 100644 --- a/packaging_automation/citus_package.py +++ b/packaging_automation/citus_package.py @@ -364,14 +364,14 @@ def build_package( print(f"Executing docker command: {docker_command}") output = run_with_output(docker_command, text=True) - if output.stdout: - print("Output:" + output.stdout) if output.returncode != 0: raise ValueError( f"Build failed (rc={output.returncode}).\n" f"STDOUT:\n{output.stdout}\n" f"STDERR:\n{output.stderr}" ) + if output.stdout: + print("Output:" + output.stdout) if input_output_parameters.output_validation: validate_output(