Skip to content

Conversation

@unnawut
Copy link
Collaborator

@unnawut unnawut commented Feb 3, 2026

🗒️ Description

Following #320 that we had to limit the number of build workers. I think it was because the key manager is loading all keys even though most of them are not used. With parallel workers, this saturates the memory really fast.

This PR update the test key manager to lazy load individual keys only when they are needed.

Because -n auto was failing for all builds previously, so #320 set it to 2. If this PR sets it back to -n auto and the build passes then this solution fixes the original issue.

🔗 Related Issues or PRs

✅ Checklist

  • Ran tox checks to avoid unnecessary CI fails:
    uvx tox
  • Considered adding appropriate tests for the changes.
  • Considered updating the online docs in the ./docs/ directory.

@unnawut unnawut added bug Category: bug fix framework Scope: Changes to the testing framework labels Feb 3, 2026
@unnawut unnawut marked this pull request as ready for review February 3, 2026 04:46
Copy link
Collaborator

@anshalshukla anshalshukla left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Collaborator

@tcoratger tcoratger left a comment

Choose a reason for hiding this comment

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

Sounds good

@tcoratger tcoratger merged commit 13d0896 into leanEthereum:main Feb 3, 2026
10 checks passed
@unnawut unnawut deleted the parallelized-aggregation-test-vectors branch February 4, 2026 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Category: bug fix framework Scope: Changes to the testing framework

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants