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

chore(ci): update internal testnet genesis and seed to support committee voting #2008

Merged
merged 3 commits into from
Aug 19, 2024

Conversation

DracoLi
Copy link
Contributor

@DracoLi DracoLi commented Aug 16, 2024

Description

  • Adds committee MsgVote to evm eip712_allowed_msgs to allow for committee voting.
  • Updates internal testnet seed script to submit a committee proposal for testing purposes.
  • Changes gov voting period to 7 days to help with testing active proposals.

Copy link
Member

@nddeluca nddeluca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice heredoc usage and good to merge, none of the comments are blocking

.github/scripts/seed-internal-testnet.sh Outdated Show resolved Hide resolved
@@ -2172,7 +2172,7 @@
"max_deposit_period": "172800s"
},
"voting_params": {
"voting_period": "600s"
"voting_period": "604800s"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this being updated on purpose? it's more useful for our internal testnets to use short voting periods so that we can submit & pass test proposals (waiting 10min vs waiting 1 week)

Copy link
Contributor Author

@DracoLi DracoLi Aug 16, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, this was requested so the proposal is up longer for testing on webapp. I suggest we increase it for now and can revert it back later. I'd also like to keep a shorter time period for faster proposals.

@DracoLi DracoLi merged commit b8e6e58 into master Aug 19, 2024
12 checks passed
@DracoLi DracoLi deleted the dl/internal-testnet-hard branch August 19, 2024 16:23
pirtleshell pushed a commit that referenced this pull request Oct 25, 2024
…tee voting (#2008)

* chore(ci): update internal testnet genesis and seed to support committee voting

* chore(ci): update gov proposal voting period to 7 days

* chore: use auto gas calculation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants