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

Adds stake-tracker pallet #1654

Closed
wants to merge 4 commits into from
Closed

Adds stake-tracker pallet #1654

wants to merge 4 commits into from

Conversation

gpestana
Copy link
Contributor

@gpestana gpestana commented Sep 20, 2023

This PR adds the stake-tracker pallet without integrating it with the staking and bags lists pallets.

The stake-tracker pallet implements the OnStakingUpdate trait to listen to staking events and multiplexes those events to one or multiple types (e.g. pallets). Initially, the stake tracker pallet is used as a degree of indirection to maintain the target and voter semi-sorted lists up to date.

The final goal is to achieve a design where the targets and voters lists are updated at each relevant staking event through the stake-tracker pallet. The list reads in staking are performed directly through the SortedListProvider.

stake-tracker

To finalize

Related and step towards #443

@gpestana gpestana added the T2-pallets This PR/Issue is related to a particular pallet. label Sep 20, 2023
@gpestana gpestana requested review from kianenigma, Ank4n and a team September 20, 2023 20:55
@gpestana gpestana self-assigned this Sep 20, 2023
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-clippy
Logs: https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3965985

@gpestana
Copy link
Contributor Author

Closing in lieu of #1933, where the stake tracker pallet is introduced AND integrated within staking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T2-pallets This PR/Issue is related to a particular pallet.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants