Skip to content

diag: surface stdout+stderr on build_package failure - #411

Merged
serhat andic (serhatandic) merged 3 commits into
developfrom
ihalatci/citus-package-diag
Aug 5, 2026
Merged

diag: surface stdout+stderr on build_package failure#411
serhat andic (serhatandic) merged 3 commits into
developfrom
ihalatci/citus-package-diag

Conversation

@ihalatci

Copy link
Copy Markdown
Contributor

Packaging build failures were masked by stderr-only docker pull preamble output. This now prints both stdout and stderr, including the return code, so the real container error is visible. No behavior change on success.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dropped a nit comment.

Comment thread packaging_automation/citus_package.py Outdated
@ihalatci
ibrahim halatci (ihalatci) marked this pull request as ready for review July 20, 2026 10:50
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 21828aca-24f9-441b-8e36-981b7666b5c0
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 4ab26727-8d1b-4dcd-8173-bc32a5ebd3ac
@serhatandic
serhat andic (serhatandic) merged commit 281c57f into develop Aug 5, 2026
43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants