Skip to content

Partial revert of #5806: drop check_harness_server_cleanup.sh + 817 (keep launcher fixes) - #5863

Closed
noahgift wants to merge 2 commits into
mainfrom
revert/5806-crux-harness-guard
Closed

noahgift wants to merge 2 commits into
mainfrom
revert/5806-crux-harness-guard

Conversation

@noahgift

@noahgift noahgift commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Operator 2026-10-21 13:20Z (verbatim, relayed): "Partial revert, not full: revert only scripts/check_harness_server_cleanup.sh + 817-harness-server-cleanup.cmd. Keep the 3 launcher zombie fixes from #5806. Open now, top of the queue. Re-land: guard rewritten in bash (bashrs) or Rust. NO python3 in any image, NO skip-if-missing."

Removes only the guard and its explicit command; the guard_server fixes in check_crux_ollama_in_lock.sh, check_crux_serve_code.sh and crux_inference_dogfood.sh stay. Fixes the main-red (python3 missing in the container, #5844 shard 5; apr-stub race, #5854). Supersedes #5860 (draft).

Not armed.

🤖 Generated with Claude Code

noahgift and others added 2 commits October 21, 2026 15:18
…RM/INT/KILL (#5806)"

Operator 2026-10-21 13:02Z: Python is banned; revert #5806 as ruled. The re-land is the guard rewritten in bash (bashrs-checked) or Rust, with no python3 dependency.
check_harness_server_cleanup.sh exits 'ENV - python3 missing' in the sovereign-ci container (#5844 shard 5) and its apr-stub row races on guard-tree (#5854).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Operator 2026-10-21 13:20Z: revert only check_harness_server_cleanup.sh and 817-harness-server-cleanup.cmd; keep the launcher guard_server fixes.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@noahgift noahgift changed the title Revert "fix(harness): crux launchers kill their fixture servers (#5806)" Partial revert of #5806: drop check_harness_server_cleanup.sh + 817 (keep launcher fixes) Oct 21, 2026
@noahgift
noahgift marked this pull request as draft October 21, 2026 13:21
@noahgift

Copy link
Copy Markdown
Member Author

patch-id proof (git patch-id --stable, diff vs merge-base b9cb67a):

@noahgift

Copy link
Copy Markdown
Member Author

Closing per operator 13:18Z (relayed): superseded by #5867 @844320dec2, same tree (stable patch-id c22027f073f2 == #58631a2a168, proven by wsweep and re-checked by me). Quorum and seat are on #5867: handoff/triage-quorum-5867-844320dec2.md.

@noahgift noahgift closed this Oct 21, 2026
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.

1 participant