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

Fix Possible bug: Vote import failed after aggression is enabled #6690

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Nov 28, 2024

  1. Fix Possible bug: Vote import failed after aggression is enabled

    After finality started lagging on kusama around 025-11-25 15:55:40
    validators started seeing ocassionally this log, when importing votes
    covering more than one assignment.
    ```
    Possible bug: Vote import failed
    ```
    
    That happens because the assumption that assignments from the same
    validator would have the same required routing doesn't hold after you
    enabled aggression, so you might end up receiving the first assignment
    then you modify the routing for it in `enable_aggression` then your
    receive the second assignment and the vote covering both assignments, so
    the rouing for the first and second assingment wouldn't match and we
    would fail to import the vote.
    
    From the logs I've seen, I don't think this is the reason the network
    didn't fully recover until the failsafe kicked it, because the votes
    had been already imported in approval-voting before this error.
    
    Signed-off-by: Alexandru Gheorghe <[email protected]>
    alexggh committed Nov 28, 2024
    Configuration menu
    Copy the full SHA
    fd11842 View commit details
    Browse the repository at this point in the history