Skip to content

solutions-engineering: tune lab bottleneck signals#10

Merged
adityamaru merged 2 commits into
mainfrom
se-migration-lab-tune-bottlenecks
May 16, 2026
Merged

solutions-engineering: tune lab bottleneck signals#10
adityamaru merged 2 commits into
mainfrom
se-migration-lab-tune-bottlenecks

Conversation

@adityamaru
Copy link
Copy Markdown
Contributor

@adityamaru adityamaru commented May 16, 2026

Summary

  • Make the fraud benchmark start from a 4-core runner label and increase its fixed CPU workload so it is the obvious first performance bottleneck.
  • Run both Docker platform builds on an x64 runner label so the arm64 target exercises cross-platform build behavior.
  • Make the Dockerfile intentionally less cache-friendly and add native build work so Docker build optimization has meaningful surface area.

Test plan

  • Ran npm run lint in solutions-engineering/migration-lab/app.
  • Ran DOCKER_BUILD_TASKS=4 DOCKER_BUILD_ITERATIONS=500000 DOCKER_BUILD_WORKERS=2 node scripts/docker-build-workload.js.
  • Ran PROFILE_CPU_TASKS=4 PROFILE_CPU_ITERATIONS=500000 PROFILE_CPU_WORKERS=2 npm run profile:cpu.

Made with Cursor


View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews

View in Codesmith
Need help on this PR? Tag @codesmith with what you need.

  • Let Codesmith autofix CI failures and bot reviews (Staging)

adityamaru and others added 2 commits May 16, 2026 17:44
Make the CPU benchmark the obvious starting bottleneck, run both Docker platform builds on an x64 runner, and make the Dockerfile intentionally cache-unfriendly with native build work.

Co-authored-by: Cursor <cursoragent@cursor.com>
Add stable system dependency work after the app copy so the bonus Dockerfile optimization has a clearer layer-ordering target.

Co-authored-by: Cursor <cursoragent@cursor.com>
@adityamaru adityamaru merged commit 42462a8 into main May 16, 2026
1 of 27 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