[go: up one dir, main page]

Baker: fix consensus operations branch filtering

What

Under aggregate_attestations feature flag:

  • Fix the branch filters that was set incorrectly in !17208 (merged)
  • Add filtering in the consensus monitoring: since operations not branched correctly are not eligible to be added in a block, they must not count in the quorum monitoring

Why

Currently the branch filtering is incorrect for reproposals

How

Adding a grandparent field in the type block_info made this quiet handy

Manually testing the MR

The test is in !17578 (merged) you can run it locally as I'm still struggling to make it run in the CI without flakyness.

Checklist

  • Document the interface of any function added or modified (see the coding guidelines)
  • Document any change to the user interface, including configuration parameters (see node configuration)
  • Provide automatic testing (see the testing guide).
  • For new features and bug fixes, add an item in the appropriate changelog (docs/protocols/alpha.rst for the protocol and the environment, CHANGES.rst at the root of the repository for everything else).
  • Select suitable reviewers using the Reviewers field below.
  • Select as Assignee the next person who should take action on that MR
Edited by Adam

Merge request reports

Loading