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

Refactor so client state watches L1 and trigger CL genesis only after horizon block reached #141

Closed
delbonis opened this issue Jul 15, 2024 · 0 comments · Fixed by #163
Closed
Assignees
Labels
consensus Consensus-critical, high-sensitivity required refactor Reworking code to improve structure or performance

Comments

@delbonis
Copy link
Contributor

delbonis commented Jul 15, 2024

The genesis block isn't going to be hardcoded in production, we'll set a horizon L1 block and switch to it as the oldest block we can fetch from and using operator nominations from before then to initialize the genesis state. This means that the client state has to take inputs from these blocks and do things with them before we actually initialize the chainstate genesis stuff. There should be a DoGenesis sync action that the CSM can produce and the worker carries out when we've decided that genesis is happening. This splits apart some of the genesis init work so that we can start running the L1 reader and run the CSM worker without fully starting the chain sync stuff (it would still be there, but just be idle waiting for a real state to start working off of).

Related #123
This should close #123

@delbonis delbonis self-assigned this Jul 15, 2024
@delbonis delbonis added consensus Consensus-critical, high-sensitivity required refactor Reworking code to improve structure or performance labels Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consensus Consensus-critical, high-sensitivity required refactor Reworking code to improve structure or performance
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant