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

[5.0 -> main] Signal accepted_block after it is marked valid #1778

Merged
merged 6 commits into from
Oct 16, 2023

Conversation

heifner
Copy link
Member

@heifner heifner commented Oct 16, 2023

Backport of #1386 hence the small diff.

As indicated in #1275, produced blocks were signaled to plugins after being marked valid and placed in the forkdb but signaled before being marked valid and placed in the forkdb for validated blocks. This PR makes sure that the block is marked valid before being signaled for both produced blocks and validated blocks. Also refactors location of updating head for applied blocks, consolidating it in commit_block.

This also fixes #1694 as head is updated consistently before accepted_block is signaled which is what state_history_plugin SHiP is using for reporting head in get_status_result_v0 messages.

Merges release/5.0 into main including #1769 & #1777

Resolves #1275
Resolves #1694

@heifner heifner requested review from greg7mdp and linh2931 October 16, 2023 15:35
@heifner heifner added the OCI Work exclusive to OCI team label Oct 16, 2023
@heifner heifner merged commit 0544e2c into main Oct 16, 2023
21 checks passed
@heifner heifner deleted the GH-1694-ship-head-main branch October 16, 2023 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCI Work exclusive to OCI team
Projects
None yet
3 participants