Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

grandpa: maintain invariants when evaluating aggregated voting rules #8186

Merged
4 commits merged into from
Feb 28, 2021

Conversation

andresilva
Copy link
Contributor

@andresilva andresilva commented Feb 23, 2021

Voting rules can only restrict the result in the range [base, current_target], e.g. if they return some block lower than base then the result should be discard. Implementations of voting rules should keep this invariant but now it is enforced when evaluating multiple voting rules.

@andresilva andresilva added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Feb 23, 2021
@andresilva
Copy link
Contributor Author

bot merge

@ghost
Copy link

ghost commented Feb 28, 2021

Checks failed; merge aborted.

@andresilva
Copy link
Contributor Author

bot merge force

@ghost
Copy link

ghost commented Feb 28, 2021

Trying merge.

@ghost
Copy link

ghost commented Feb 28, 2021

Merge failed: "At least 2 approving reviews are required by reviewers with write access."

@andresilva andresilva requested a review from octol February 28, 2021 16:27
@andresilva
Copy link
Contributor Author

andresilva commented Feb 28, 2021

cargo-deny failure is unrelated.

@andresilva
Copy link
Contributor Author

bot merge force

@ghost
Copy link

ghost commented Feb 28, 2021

Trying merge.

@ghost ghost merged commit e84d2ae into master Feb 28, 2021
@ghost ghost deleted the andre/voting-rule-invariants branch February 28, 2021 16:53
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants