Skip to content

test/integration: descendantsOf reads each parent once per scan - #699

Merged
vyskocilm merged 2 commits into
mainfrom
test/parked-signal-scan
Oct 25, 2026
Merged

vyskocilm merged 2 commits into
mainfrom
test/parked-signal-scan

Conversation

@vyskocilm

Copy link
Copy Markdown
Contributor

TestASignalWhileThePayloadIsParkedNeverReleasesIt/net failed precondition on desktop host, 6/6 runs, origin/main 648950c too:

  • PRECONDITION: no bwrap ever appeared under snug or the parked sandbox pid 1 never appeared, after 30s.
  • Cause: descendantsOf reread /proc/<pid>/status at every hop. Measured 18.5ms/scan (135 pids of this uid). Stage's bwrap lived ~30ms in sampled @net run; poll missed window.
  • Fix: one ppid read per pid, walk over map. 7.4ms/scan. 3/3 runs pass; make integration-signals green.

Test-only. CI was passing (fewer processes, faster scan).

🤖 Generated with Claude Code

TestASignalWhileThePayloadIsParkedNeverReleasesIt/net failed its
precondition on a desktop host 6 of 6 runs, on origin/main too: the
per-hop ppid reads made one scan 18.5ms (135 pids of this uid), and the
stage's bwrap lived about 30ms in a sampled @net run. One read per pid,
walk over the map: 7.4ms a scan, 3 of 3 runs pass.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@vyskocilm
vyskocilm merged commit 424e8a9 into main Oct 25, 2026
5 checks passed
@vyskocilm
vyskocilm deleted the test/parked-signal-scan branch October 25, 2026 20:10
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