Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Optimize UX loading +100 keystores on vc #4179

Closed
3 tasks done
dapplion opened this issue Jun 21, 2022 · 4 comments · Fixed by #5043 or #5357
Closed
3 tasks done

Optimize UX loading +100 keystores on vc #4179

dapplion opened this issue Jun 21, 2022 · 4 comments · Fixed by #5043 or #5357
Assignees
Labels
prio-high Resolve issues as soon as possible. scope-ux Issues for CLI UX or general consumer UX.
Milestone

Comments

@dapplion
Copy link
Contributor

dapplion commented Jun 21, 2022

Describe the bug

After fixing #4166 now it takes a long time to load secure scrypt keystores.

Note that this UX improvement only affects whales and institutional players:

  • Home stakers are likely to have < 10 keys
  • Testnets + Kurtosis are run with either insecure scrypt keystores or mnemonics

Next steps:

  • There's a bug currently where memory is not released after decrypting the keystores. Must investigate and ensure memory is released
  • Instead of decrypting 100 keystores at once, decrypting 5-10 is fine, which provides a significant speed up. A good number would be to decrypt thread count at once
  • Implement similar strategy to Prysm and Lighthouse where private keys are stored in a single keystore such that subsequent start-ups are fast
@dapplion dapplion added prio-medium Resolve this some time soon (tm). scope-ux Issues for CLI UX or general consumer UX. labels Jun 21, 2022
@dapplion dapplion added prio-high Resolve issues as soon as possible. and removed prio-medium Resolve this some time soon (tm). labels Jan 12, 2023
@nazarhussain nazarhussain self-assigned this Jan 19, 2023
@philknows
Copy link
Member

@wemeetagain @dapplion Do we still want to keep this issue open to address some of the ideal next steps listed in the original post? Unless I'm mistaken and these have actually been completed?

@wemeetagain
Copy link
Member

We can reopen, but the last item is low priority, imo. The big win was @nazarhussain 's recent PR #5043

Last item is:

Instead of decrypting 100 keystores at once, decrypting 5-10 is fine, which provides a significant speed up. A good number would be to decrypt thread count at once

@wemeetagain wemeetagain reopened this Feb 16, 2023
@nflaig
Copy link
Member

nflaig commented Feb 16, 2023

A good number would be to decrypt thread count at once

in node v19.4.0 a new API was added to properly determine this, os.availableParallelism(), v20 will be active LTS in 8 months (2023-10-24) but initial release is already in April (2023-04-18).

@philknows philknows added prio-medium Resolve this some time soon (tm). and removed prio-high Resolve issues as soon as possible. labels Feb 16, 2023
@philknows
Copy link
Member

This was brought up at the March 7, 2022 standup to prioritize the remaining item so institutional type node operators would be able to consider Lodestar in their setup. This is in reference to the tweet made by node operator RockLogic as a blocker to allow them to consider our validator client for their setup: https://twitter.com/Stefa2kEth/status/1628407293325807623?s=20

cc: @dapplion

@philknows philknows added prio-high Resolve issues as soon as possible. and removed prio-medium Resolve this some time soon (tm). labels Mar 7, 2023
@philknows philknows added this to the v1.7.0 milestone Mar 8, 2023
@philknows philknows modified the milestones: v1.7.0, v1.8.0 Mar 25, 2023
maschad added a commit to maschad/lodestar that referenced this issue Mar 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
prio-high Resolve issues as soon as possible. scope-ux Issues for CLI UX or general consumer UX.
Projects
None yet
5 participants