Skip to content

feat(sandbox): await process termination - #414

Merged
ankrgyl merged 10 commits into
exoharness:export-oauth-refreshfrom
braintrustdata:fix/await-process-termination
Nov 8, 2026
Merged

ankrgyl merged 10 commits into
exoharness:export-oauth-refreshfrom
braintrustdata:fix/await-process-termination

Conversation

@ankrgyl

@ankrgyl ankrgyl commented Nov 7, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Remember when local groups have no live members so later terminate calls
do not signal an ID observed to be retired. Probe immediately after
reaping the leader without changing normal wait or pipe ownership.

On Darwin, verify an EPERM group contains no live members before
accepting it as complete, preserving real permission errors. Reap the
child even when a duplicate kill fails.

Add bounded coverage for group retirement and unreaped macOS zombies.
Treat macOS EPERM as an existing group and retry until ESRCH within the
termination timeout. Keep kill attempts best-effort and reap the leader
before confirming group exit, removing the platform member scan.

Use a unit mutex to serialize termination and avoid panicking during
test cleanup. Exercise an unreaped zombie and its subsequent reap.
@ankrgyl
ankrgyl merged commit de7547f into exoharness:export-oauth-refresh Nov 8, 2026
8 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.

1 participant