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

Start deposit tree sync only after beacon node is synced #2910

Open
pawanjay176 opened this issue Jan 13, 2022 · 0 comments
Open

Start deposit tree sync only after beacon node is synced #2910

pawanjay176 opened this issue Jan 13, 2022 · 0 comments
Labels
bellatrix Required to support the Bellatrix Upgrade

Comments

@pawanjay176
Copy link
Member

pawanjay176 commented Jan 13, 2022

Description

Before the merge, since the the EL and CL systems were separate, we could do an eth1 deposit tree sync concurrently with the beacon node sync assuming that the EL is already synced. This assumption is not true after the merge as the EL and CL together make up an ethereum node and come to sync together.

Hence, all the requests we make to sync the deposit cache before the main sync is completed will return errors. So we should only make requests to sync the deposit tree after main sync is completed.

This might be related to @ethDreamer 's work on syncing the deposit tree directly over p2p.

@pawanjay176 pawanjay176 added the bellatrix Required to support the Bellatrix Upgrade label May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bellatrix Required to support the Bellatrix Upgrade
Projects
None yet
Development

No branches or pull requests

1 participant