Skip to content

perf(ticket-082): measure docker volume sizes in parallel batches - #112

Merged
ifuri-validator-agent[bot] merged 1 commit into
mainfrom
ticket/082-docker-volumes-parallel-sizes
Oct 13, 2026
Merged

ifuri-validator-agent[bot] merged 1 commit into
mainfrom
ticket/082-docker-volumes-parallel-sizes

Conversation

@tom-sapletta-com

Copy link
Copy Markdown
Contributor

fixos cleanup --docker-volumes --list took ~60 s on a host with 379 unused volumes, almost all in one sequential sudo -n du.

  • Sizes measured in 8 parallel round-robin batches (ThreadPoolExecutor).
  • Readable mountpoints use plain du; unreadable ones go straight to sudo -n du, so a partial size of an unreadable directory is never recorded.

Same host, same 379 volumes: 47.5 s → 13.0 s, identical 37.4 GB total.

Test: batching into _SIZE_WORKERS calls with complete results. Full suite 883 passed / 5 skipped; ruff clean; governance GOV-PASS.

🤖 Generated with Claude Code

…cket-082]

Co-Authored-By: Claude <noreply@anthropic.com>

@ifuri-validator-agent ifuri-validator-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Validator approval after policy checks for exact head c623bd016133388f6dd0d890b722a053a351ade1.

Ticket: ticket-082
Correlation ID: local-semcod-fixos-pr-112-ticket-082
Model: openai/cursor-auto
Reviewed diff chunks: 1
Advisory LLM verdict: APPROVE
Advisory summary: Reviewed all 1 diff chunk(s). The implementation parallelizes volume-size collection with bounded concurrency, routes top-level unreadable mountpoints through noninteractive sudo, and safely treats command errors or timeouts as unknown sizes. All required and observational checks passed, and no blocking correctness or security defects are evident in the visible diff.
Advisory findings: none
The LLM output above is advisory and was not used as the approval trust root.
Semantic review prerequisite: not_required; policy 676cb4516bbfed2a000e40b9b1b6e4a430ecc761ec546aeb53d721a1905cfdd7.

Actual PR impact radar

Exact range: f92edb837c3bf354a6e89a0da7b58f463e484397...c623bd016133388f6dd0d890b722a053a351ade1
Change digest: 690141b3949c1e4cdb079d46865a0e42421769a5cfb567b16c742ae7c809c347
Score: 48/100 (M), estimated 52 min, split recommended: true
Affected services/components: repository-wide/unclassified

Machine-readable radar JSONL and SVG
{"actual_change":{"additions":162,"base_sha":"f92edb837c3bf354a6e89a0da7b58f463e484397","binary_files":0,"categories":{"code":1,"configuration":1,"docs":1,"tests":1},"change_digest":"690141b3949c1e4cdb079d46865a0e42421769a5cfb567b16c742ae7c809c347","comparison":"f92edb837c3bf354a6e89a0da7b58f463e484397...c623bd016133388f6dd0d890b722a053a351ade1","deletions":13,"file_count":4,"files":["fixos/cli/_cleanup_docker_volumes.py","project/ticket-082/README.md","project/ticket-082/intent.json","tests/unit/test_cleanup_docker_volumes.py"],"head_sha":"c623bd016133388f6dd0d890b722a053a351ade1","service_count":0,"services":[]},"assessment_mode":"observed-pr","axes":{"coupling":4,"delivery":2,"scope":2,"uncertainty":3,"validation":1},"complexity":"M","confidence":0.9,"diagnostics":["RADAR-ACCEPTANCE-MISSING","RADAR-BUDGET-EXCEEDED"],"estimate":{"budget_minutes":30,"minutes":52,"within_budget":false},"impact":{"components":["claude.com","fixos","project","tests"],"files":["claude.com/claude-code","fixos/cli/_cleanup_docker_volumes.py","project/ticket-082/README.md","project/ticket-082/intent.json","tests/unit/test_cleanup_docker_volumes.py"],"public_interfaces":[],"runtime_dependencies":0},"schema":"subactor.ticket-radar/v1","score":48,"split":{"parts":[{"estimated_minutes":11,"name":"Implement claude.com","scope":["claude.com"]},{"estimated_minutes":11,"name":"Implement fixos","scope":["fixos"]},{"estimated_minutes":11,"name":"Implement project","scope":["project"]},{"estimated_minutes":11,"name":"Implement tests","scope":["tests"]},{"estimated_minutes":15,"name":"Validate and project to trackers","scope":["tests","planfile","github/gitlab/jira projections"]}],"reason":"estimated_minutes_exceed_budget","recommended":true},"standards":[{"id":"wellmanifest/dsl","revision":"6c60fc4e0dd1f1bb74f46a7745e28019908d1203","version":"0.1.0-dev"},{"id":"wellmanifest/ticket-lifecycle","revision":"5bf581907a87b46a13a73e6c033d3abe4d9a306f","version":"0.1.0-dev"},{"id":"wellmanifest/git-lifecycle","revision":"7d77d4b7af57e69bc75c3a0290b3a4805c5c4438","version":"0.2.0-dev"},{"id":"wellmanifest/logs","revision":"48c284ef7a069055c0bcb6b900147ce5e65f8b43","version":"0.3.0"}],"ticket_ref":"ticket-082"}
<svg xmlns="http://www.w3.org/2000/svg" width="128" height="128" viewBox="0 0 128 128" role="img"><title>ticket-082: perf(ticket-082): measure docker volume sizes in parallel batches</title><rect width="128" height="128" rx="12" fill="#f8fafc"/><g stroke-width="1"><polygon points="64,55 72,61 69,71 59,71 56,61" fill="none" stroke="#d7dde5"/><polygon points="64,47 80,59 74,78 54,78 48,59" fill="none" stroke="#d7dde5"/><polygon points="64,38 89,56 79,85 49,85 39,56" fill="none" stroke="#d7dde5"/><polygon points="64,30 97,53 84,92 44,92 31,53" fill="none" stroke="#d7dde5"/><polygon points="64,21 105,51 89,99 39,99 23,51" fill="none" stroke="#d7dde5"/><line x1="64" y1="64" x2="64" y2="21" stroke="#aab4c0"/><line x1="64" y1="64" x2="105" y2="51" stroke="#aab4c0"/><line x1="64" y1="64" x2="89" y2="99" stroke="#aab4c0"/><line x1="64" y1="64" x2="39" y2="99" stroke="#aab4c0"/><line x1="64" y1="64" x2="23" y2="51" stroke="#aab4c0"/></g><polygon points="64,47 97,53 79,85 59,71 48,59" fill="#fb923c" fill-opacity="0.45" stroke="#c2410c" stroke-width="2"/><circle cx="64" cy="64" r="3" fill="#c2410c"/><g font-family="sans-serif" font-size="7" fill="#334155"><text x="64" y="11" text-anchor="middle">SCO</text><text x="114" y="48" text-anchor="middle">COU</text><text x="95" y="107" text-anchor="middle">UNC</text><text x="33" y="107" text-anchor="middle">VAL</text><text x="14" y="48" text-anchor="middle">DEL</text></g><text x="64" y="124" text-anchor="middle" font-family="sans-serif" font-size="8" fill="#0f172a">M · 52m</text></svg>
Merge will be attempted after this approval when explicitly authorized. ## Decision record (recomputable)
DECISION D-082-4176
TICKET ticket-082
HEAD_SHA c623bd016133388f6dd0d890b722a053a351ade1
CORRELATION_ID local-semcod-fixos-pr-112-ticket-082
ACTOR agent:ifuri-validator-agent[bot]
APPLIED_RULE P-CORE-015
INPUT author_login = "tom-sapletta-com"
INPUT observed_checks = ["test-summary=PASS","test (3.12)=PASS","test (3.10)=PASS","test (3.11)=PASS","test-fedora=PASS","test-alpine=PASS","test-arch=PASS","test-ubuntu=PASS","governance / enforce=PASS","test-debian=PASS","governance / remote lifecycle=PASS","onedev/local-verify=PASS"]
INPUT required_checks = ["onedev/local-verify","test (3.10)","test (3.11)","test (3.12)","test-summary"]
INPUT required_checks_source = "protected registry (env/request)"
INPUT reviewer_login = "ifuri-validator-agent[bot]"
INPUT semantic_review_assessment = {"schema":"subactor.validator/semantic-review-assessment/v1","subject":{"repository":"semcod/fixos","pull_request":112,"head_sha":"c623bd016133388f6dd0d890b722a053a351ade1","base_sha":"f92edb837c3bf354a6e89a0da7b58f463e484397","diff_sha256":"cc0cc1562d5083becc548553a1535a864049a05800af42ce777c5cc7b2f43232"},"policy":{"policy_schema":"subactor.validator/semantic-review-policy/v1","policy_version":1,"policy_sha256":"676cb4516bbfed2a000e40b9b1b6e4a430ecc761ec546aeb53d721a1905cfdd7","required":false,"critical_paths":[],"observed_paths":["fixos/cli/_cleanup_docker_volumes.py","project/ticket-082/README.md","project/ticket-082/intent.json","tests/unit/test_cleanup_docker_volumes.py"]},"grounding":"full-diff-not-per-finding-proof","execution_authority":false,"status":"not_required","reason":null,"review_sha256":null,"unresolved":[]}
INPUT superseded_checks = []
INPUT ticket_radar_receipt = {"schema":"subactor.ticket-radar/v1","base_sha":"f92edb837c3bf354a6e89a0da7b58f463e484397","head_sha":"c623bd016133388f6dd0d890b722a053a351ade1","change_digest":"690141b3949c1e4cdb079d46865a0e42421769a5cfb567b16c742ae7c809c347","score":48,"complexity":"M","estimated_minutes":52,"split_recommended":true,"services":[],"authority":"ADVISORY","promotion":"FORBIDDEN"}
VERDICT APPROVE AUTHORITY DETERMINISTIC
REJECTED REQUEST_CHANGES BECAUSE NO_UNSAFE_CHANGE_REASON_FOUND
ADVISORY llm_verdict = "APPROVE" MODEL "openai/cursor-auto"
ASSERT VERDICT_AUTHORITY != "ADVISORY"

@ifuri-validator-agent
ifuri-validator-agent Bot merged commit e50be7b into main Oct 13, 2026
12 checks passed
@ifuri-validator-agent
ifuri-validator-agent Bot deleted the ticket/082-docker-volumes-parallel-sizes branch October 13, 2026 21:00
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