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

[EPIC] Provide pandora consensus engine #45

Open
blazejkrzak opened this issue Mar 3, 2021 · 5 comments · Fixed by #55 · May be fixed by #58
Open

[EPIC] Provide pandora consensus engine #45

blazejkrzak opened this issue Mar 3, 2021 · 5 comments · Fixed by #55 · May be fixed by #58
Assignees
Labels
enhancement New feature or request

Comments

@blazejkrzak
Copy link

blazejkrzak commented Mar 3, 2021

[MIGHT BE CANDIDATE FOR MILESTONE/EPIC]

@currentSlot @synchronizedEpochList
Given as a Vanguard I want to produce consensus block
  And I have notified about validators that will be signing current epoch
  And in current slot I should create the block

When Pandora prepares the execution block for signature
  And notifies about it

Then Vanguard signs the block
  And Vanguard notifies about it
  And Vanguard asynchronously push this block to consensus peers
  And Pandora asynchronously push this block to execution peers

Expected behaviour

Engine is allowed to prepare block data and push it to the vanguard.

How to implement

Provide engine named pandora under consensus package.

@blazejkrzak blazejkrzak added the enhancement New feature or request label Mar 3, 2021
@blazejkrzak
Copy link
Author

@blazejkrzak blazejkrzak self-assigned this Mar 5, 2021
@blazejkrzak
Copy link
Author

@blazejkrzak
Copy link
Author

Spike now has some serious discovery and implementation about flow. Lets discuss it @frozeman @patred20 @mxmar @atif-konasl

@blazejkrzak blazejkrzak changed the title Provide pandora consensus engine [EPIC] Provide pandora consensus engine Apr 27, 2021
@blazejkrzak blazejkrzak linked a pull request Apr 27, 2021 that will close this issue
@blazejkrzak blazejkrzak linked a pull request Apr 28, 2021 that will close this issue
@blazejkrzak
Copy link
Author

We have prototype that works ATM.
Several issues yet to discover.
Related merge request #58

@blazejkrzak blazejkrzak linked a pull request May 11, 2021 that will close this issue
@blazejkrzak
Copy link
Author

IMHO it requires refactor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
1 participant