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

feat(node): integrating topdown cert into interpreters #1187

Open
wants to merge 23 commits into
base: refactor-syncer
Choose a base branch
from

Conversation

cryptoAtwill
Copy link
Contributor

@cryptoAtwill cryptoAtwill commented Oct 28, 2024

Following the previous PR, this PR integrates the new topdown flow (cert, reactor) into the interpreters.

This is the branch where all topdown changes are merged.

The key changes includes:

  • Added ECDSACertificate that contains validator signatures with topdown checkpoint as payload. This will be used by interpreters on chain as the final outcome of topdown checkpointing.
  • Integrated Prepare, Process and Exec: calls into topdown checkpoint proposal client to see if there are new proposals. If yes, it will attach the quorum cert and the actual topdown checkpoint. For Process, interpreter validates the cert against the checkpoint. For Exec, interpreter extracts the side effects from the checkpoint and executes the messages.
  • Due to the above change and integration, all the types in current fendermint App, Cmd needs to be updated. This is why this PR has around 65 files changes. Mostly are just type changes.
  • Updated configuration in app::run to make sure the setup is correct.
  • Updated ipld-resolver to use a more generic Vote payload instead of force VoteRecord. This allows more flexibility in the message to gossip.

@cryptoAtwill cryptoAtwill requested a review from a team as a code owner October 28, 2024 09:05
@cryptoAtwill cryptoAtwill force-pushed the integrate-observation branch from ea23982 to 00e14bd Compare October 28, 2024 09:26
Base automatically changed from integrate-observation to refactor-syncer November 28, 2024 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Backlog
Development

Successfully merging this pull request may close these issues.

1 participant