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

Refactor gov+PoS joint logic #2821

Closed
wants to merge 25 commits into from

Conversation

brentstone
Copy link
Collaborator

@brentstone brentstone commented Mar 6, 2024

Describe your changes

Several refactors are performed that related to the interplay between governance and PoS. Closes #2796. Closes #1125.

  • Restrict voting in governance to prohibit jailed and inactive validators
  • Refactor finding all validators to which an account has delegations (instead of iteration over bonds, store this data)

TODO:

  • test for validator targets

Indicate on which release or other PRs this topic is based on

based on #2877, which is based on v0.32.0

Checklist before merging to draft

  • I have added a changelog
  • Git history is in acceptable state

@brentstone brentstone force-pushed the brent+fraccaman/active-validator-can-vote branch from b0998e7 to d3a5846 Compare March 6, 2024 11:13
@Fraccaman Fraccaman self-requested a review March 6, 2024 11:48
@Fraccaman
Copy link
Member

some e2e tests are not passing

@brentstone brentstone force-pushed the brent+fraccaman/active-validator-can-vote branch from d3a5846 to a8b9f30 Compare March 6, 2024 12:31
@brentstone brentstone force-pushed the brent+fraccaman/active-validator-can-vote branch from a8b9f30 to cce1c9a Compare March 15, 2024 22:05
@brentstone brentstone force-pushed the brent+fraccaman/active-validator-can-vote branch from cce1c9a to 3235577 Compare March 19, 2024 16:34
@brentstone brentstone force-pushed the brent+fraccaman/active-validator-can-vote branch from 3235577 to 2cc9709 Compare March 19, 2024 16:39
@brentstone
Copy link
Collaborator Author

@brentstone brentstone closed this Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants