Skip to content

Commit

Permalink
rotate voting rights for EC committee (#10164)
Browse files Browse the repository at this point in the history
closes: #10134
refs: #10134

## Description
This PR implements the following:

- Core eval script to replace the committee.
- Builder script for setting up the core eval.
- Bootstrap test to verify the functionality of the implementation.

### Security Considerations
The Economic Committee plays a crucial role in managing the Inter-Protocol. The current committee doesn't easily support a smooth transition when members must retire or be replaced. An incomplete transition to a new committee could severely hamper the chain operations.

Accidentally providing access to the wrong parties (or failing to remove parties whose terms have expired) could make it hard to ensure that the right committee members have control. Other chains have been sunk or badly damaged when the stakeholders couldn't restrict some people from making changes.

### Testing Considerations
Bootstrap tests have been added in this PR to run in CI for testing.

### Documentation Considerations
Specific documentation concerns belong in #10138. It’s important to ensure the proposal is clearly documented to avoid confusion, especially with the old committee still being around.

### Upgrade Considerations:
There is minimal risk of regression because the core-eval materials are introduced in new files. The changes introduced particularly the new builder script, can be seamlessly included in any upcoming release.

### Scaling Considerations
The core eval script’s performance depends on the number of Economic Committee (EC) members. Since the committee is limited to a maximum of 12 members, the script’s complexity is treated as constant because it won’t have to handle more than 12 people.

Also, the script is executed approximately once a year, and all stakers receive a 3-day notice before it runs, making the scaling impact negligible.
  • Loading branch information
mergify[bot] authored Oct 8, 2024
2 parents feae722 + ca27d2f commit fdb7004
Show file tree
Hide file tree
Showing 5 changed files with 961 additions and 64 deletions.
Loading

0 comments on commit fdb7004

Please sign in to comment.