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

[1.0 -> main] Use other_branch_latest_time for liveness #650

Merged
merged 7 commits into from
Aug 27, 2024

Conversation

heifner
Copy link
Member

@heifner heifner commented Aug 27, 2024

Use other_branch_latest_time block timestamp instead of a boolean votes_forked_since_latest_strong_vote for tracking votes on other fork branches. See #621 for complete details.

Fixes currently failing test on #619.

Merges release/1.0 into main including #627

Resolves #621 (Tests coming in separate PR #636)

@heifner heifner requested review from greg7mdp, arhag and linh2931 August 27, 2024 12:10
@heifner heifner added the OCI Work exclusive to OCI team label Aug 27, 2024
@heifner heifner merged commit 2f13275 into main Aug 27, 2024
36 checks passed
@heifner heifner deleted the GH-621-liveness-main branch August 27, 2024 12:49
@ericpassmore
Copy link
Contributor

Note:start
group: STABILITY
category: INTERNALS
summary: Update finalizer logic to resolve weak vote, masking issues.
Note:end

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
Development

Successfully merging this pull request may close these issues.

Update fsi and decide_vote to solve weak masking issue without regression in liveness
4 participants