Skip to content

Fix cleanup scan timeouts under load - #10

Merged
ellmos-multi-bot merged 1 commit into
mainfrom
fix/scan-budget-20261022
Oct 22, 2026
Merged

ellmos-multi-bot merged 1 commit into
mainfrom
fix/scan-budget-20261022

Conversation

@lukisch

@lukisch lukisch commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

The second process snapshot could time out on a loaded Windows host because the eight-second work deadline also included the mandatory two-second observation gap. Reserve that gap separately so seven seconds of scan work can finish, while scans exceeding the work budget still abort before terminating anything.

Process identity, two-sample CPU checks, parent-death checks, retained-handle termination and audit-before-kill are unchanged.

Validation: the new observation-gap regression failed before the fix; 78 Windows pytest tests and Ruff pass after it. A separate Codex instance reviewed the complete diff at c51f7b7 and independently passed 13 safety regressions.

@github-actions

Copy link
Copy Markdown

Welcome! 👋 Thanks for your first pull request in this repository.

A maintainer will review it soon. Please make sure:

  • Your changes are tested
  • Documentation is updated if needed
  • The PR description explains what and why

Thanks for contributing!

@ellmos-multi-bot

Copy link
Copy Markdown
Collaborator

Unabhängiges MULTI-BOT-Review durch Codex (GPT-6, eigene Sitzung) am Head c51f7b722d08e21840dfcc37e3699ed03bedf364: keine blockierenden Befunde.

Den vollständigen Diff aller drei Dateien in einem eigenen isolierten Checkout gelesen und das angrenzende Verhalten von cycle(), snapshot() und safe_terminate() geprüft. Die Beobachtungspause wird zusätzlich zum achtsekündigen Arbeitsbudget reserviert. Überschrittene Scanfristen verhindern weiterhin die Aktion; Identität, CPU-Abgleich, Elternzustand, gehaltenes Handle und Audit-vor-Aktion bleiben erhalten.

Eigene Prüfung auf Windows/Python 3.12: python -X utf8 -m pytest -q -o addopts= . → 78 bestanden; python -X utf8 -m ruff check . → erfolgreich; git diff b1f983e756802daf0b60f797a47979511cc16c9c c51f7b722d08e21840dfcc37e3699ed03bedf364 --check → erfolgreich. Enthalten sind die neue Budget-Regression und die bestehenden Abbruch-/Identitätsprüfungen. Der Win32-Smoke-Test verwendet ausschließlich seinen eigenen Hilfsprozess.

Die vollständige aktuelle Check-Liste am genannten Head ist grün, einschließlich der Windows-Testläufe. Kein Lauf der Bereinigung gegen fremde Hostprozesse und keine produktive Laufzeitabnahme behauptet.

@ellmos-multi-bot
ellmos-multi-bot merged commit 9604e1a into main Oct 22, 2026
4 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.

2 participants