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

Add staking pallets via a runtime upgrade. #111

Open
PopcornPaws opened this issue Feb 28, 2023 · 0 comments
Open

Add staking pallets via a runtime upgrade. #111

PopcornPaws opened this issue Feb 28, 2023 · 0 comments
Assignees
Labels
research requires research before implementation

Comments

@PopcornPaws
Copy link
Contributor

Description

It seems that because we don't have pallet-staking or similar in our runtime, the grandpa validators can actually finalize blocks without any funds. They are only voting on finalized blocks, but not staking anything that could be slashed. Staking, slashing, etc are managed by pallet-stakingwhich works as an optional plugin under grandpa if I understand correctly.

However, according to the docs there are still some extrinsics in pallet-grandpa that are not free and require funds to call thereof. These extrinsics are used to report equivocations (double votes) of misbehaving nodes and punish (slash) them. Thus, we should still send some funds to all validators just to be sure.

Solution

@PopcornPaws PopcornPaws self-assigned this Feb 28, 2023
@PopcornPaws PopcornPaws added research requires research before implementation and removed research labels Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
research requires research before implementation
Projects
None yet
Development

No branches or pull requests

1 participant