mosaicstack/stack
16
0
Fork 1
Code Issues 335 Pull Requests 15 Actions Packages Projects Releases 15 Wiki Activity

fix(tools/git/pr-ci-wait): stdin collision in python3 here-doc caused wrapper to always return unknown #513

Closed
jason.woltje wants to merge 0 commits from fix/pr-ci-wait-stdin-collision into main
pull from: fix/pr-ci-wait-stdin-collision
merge into: :main
Conversation 1 Commits - Files Changed -
jason.woltje commented 2026-05-13 23:31:26 +00:00
Owner
Copy Link
Copy Source

See commit a4cbd4b for full diagnosis. Heredoc-bound stdin consumed Python program text; json.load(sys.stdin) then saw EOF and bailed to unknown. Fix: capture stdin to local var before heredoc; pass via PR_CI_STATUS_JSON env. Same fix applied to both extract_state_from_status_json and print_status_summary.

See commit a4cbd4b for full diagnosis. Heredoc-bound stdin consumed Python program text; json.load(sys.stdin) then saw EOF and bailed to unknown. Fix: capture stdin to local var before heredoc; pass via PR_CI_STATUS_JSON env. Same fix applied to both extract_state_from_status_json and print_status_summary.
jason.woltje force-pushed fix/pr-ci-wait-stdin-collision from a4cbd4be51 to 59b611ba8a 2026-06-11 18:20:12 +00:00 Compare
jason.woltje commented 2026-06-11 18:21:00 +00:00
Author
Owner
Copy Link
Copy Source

Closing as superseded by #524 (rollup, merged 2026-05-26 as 821e19d). Verified during orchestration rebase: the pr-ci-wait.sh stdin-collision fix is already present on current main in both extract_state_from_status_json() and print_status_summary(); rebasing this branch drops its commit as already-upstream (net-zero diff vs main). The underlying bug is fixed in main — no code action needed. — orchestrator (mos)

Closing as **superseded by #524** (rollup, merged 2026-05-26 as 821e19d). Verified during orchestration rebase: the pr-ci-wait.sh stdin-collision fix is already present on current main in both extract_state_from_status_json() and print_status_summary(); rebasing this branch drops its commit as already-upstream (net-zero diff vs main). The underlying bug is fixed in main — no code action needed. — orchestrator (mos)
jason.woltje closed this pull request 2026-06-11 18:21:00 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Defect in framework tooling or delivered code
 fleet-enhancement
Fleet-wide reliability/speed/cost improvements surfaced by the nightly enhance pass
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
code-be-01 (Mosaic fleet seat code-be-01) code-be-02 (Mosaic fleet seat code-be-02) code-infra-01 (Mosaic fleet seat code-infra-01) fargo fred gate-merge-01 (Mosaic fleet seat gate-merge-01) happy jason.woltje (Jason Woltje) merge-gate ops-01 (Mosaic fleet seat ops-01) ops-02 (Mosaic fleet seat ops-02) ops-03 (Mosaic fleet seat ops-03) ops-ci-01 (Mosaic fleet seat ops-ci-01) ops-deploy-01 (Mosaic fleet seat ops-deploy-01) pepper rev-code-01 rev-code-02 rev-security-01 rev-security-02 rev-security-03 (Mosaic fleet seat rev-security-03) sanity scooby (Scooby) scrappy shaggy tiny velma veronica (Mosaic fleet seat veronica) vision woodpecker
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mosaicstack/stack#513