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.

Sign up for GitHub

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

Jump to bottom

IF: Modify producer schedule promotion under hotstuff #1631

Closed
Tracked by #1508
heifner opened this issue Sep 13, 2023 · 0 comments · Fixed by #1640
Closed
Tracked by #1508

IF: Modify producer schedule promotion under hotstuff #1631

heifner opened this issue Sep 13, 2023 · 0 comments · Fixed by #1640
Assignees
@heifner
Labels
👍 lgtm OCI Work exclusive to OCI team

Comments

@heifner
Copy link
Member

heifner commented Sep 13, 2023 •
edited
Loading

Producer schedule promotion is driven by dpos_irreversible_blocknum. Once dpos_irreversible_blocknum is no longer updated, producer schedule needs to be updated under a new scheme.

Proposed: Update proposed schedule on the next().next() round after proposal. This would happen outside any changes in LIB.

Expect a follow on issue to add an integration test.
@heifner heifner mentioned this issue Sep 13, 2023
Closed
@heifner heifner mentioned this issue Sep 13, 2023
Closed
@arhag arhag added 👍 lgtm and removed triage labels Sep 13, 2023
@heifner heifner self-assigned this Sep 14, 2023
@heifner heifner added the OCI Work exclusive to OCI team label Sep 14, 2023
heifner added a commit that referenced this issue Sep 14, 2023
@heifner
GH-1523 GH-1631 Transition from dpos to hotstuff
bf3b284 
Producer schedule changes under hotstuff in next,next producer round.
heifner added a commit that referenced this issue Sep 14, 2023
@heifner
GH-1523 GH-1631 Transition from dpos to hotstuff
718e17d 
Add comment to schedule_lib_num. Do not want to rename as to not break clients that expect schedule_lib_num name.
@arhag arhag linked a pull request Nov 8, 2023 that will close this issue
IF: Activate hotstuff consensus #1640
Merged
@heifner heifner mentioned this issue Nov 17, 2023
Merged
@heifner heifner closed this as completed Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@heifner heifner

Labels
👍 lgtm OCI Work exclusive to OCI team
Projects
Team Backlog
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

IF: Activate hotstuff consensus AntelopeIO/leap
3 participants
@heifner @arhag @enf-ci-bot