DEVX-163: Fix _run_push to check stdout for HTTP 500 (docker sends errors to stdout) #309

Merged
kireto merged 1 commits from DEVX-163-fix-push-500-stdout into master 2026-08-26 14:10:00 +00:00
Member

Follow-up fix for DEVX-163: use stderr=STDOUT to merge all docker push output into stdout. Closes DEVX-163

Follow-up fix for DEVX-163: use stderr=STDOUT to merge all docker push output into stdout. Closes DEVX-163
emil approved these changes 2026-08-26 14:00:58 +00:00
Dismissed
emil left a comment
Owner

Auto-approved: all CI checks passed (validate job).

Auto-approved: all CI checks passed (validate job).
emil approved these changes 2026-08-26 14:06:31 +00:00
Dismissed
emil left a comment
Owner

Auto-approved: all CI checks passed (validate job).

Auto-approved: all CI checks passed (validate job).
emil added 1 commit 2026-08-26 14:08:43 +00:00
fix: use stderr=STDOUT to capture all docker push output in one stream
CI / validate (pull_request) Successful in 1m1s
CI / auto-merge (pull_request) Successful in 13s
fae23a6d53
Docker push output behavior varies with TTY detection. Using
stderr=subprocess.STDOUT merges both streams into stdout, ensuring
the 500 error is always captured. Previous approach with separate
capture_output=True didn't reliably catch the error.

# Implements: REQ-1
emil force-pushed DEVX-163-fix-push-500-stdout from 11d930ea2a to fae23a6d53 2026-08-26 14:08:43 +00:00 Compare
emil approved these changes 2026-08-26 14:09:56 +00:00
emil left a comment
Owner

Auto-approved: all CI checks passed (validate job).

Auto-approved: all CI checks passed (validate job).
kireto merged commit 149e8846b8 into master 2026-08-26 14:10:00 +00:00
kireto deleted branch DEVX-163-fix-push-500-stdout 2026-08-26 14:10:00 +00:00
Sign in to join this conversation.