-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
update params for x/gov and default config #1690
Conversation
test_cli failing locally for me |
* [gaiad] Default config updates: | ||
- `timeout_commit=5000` so blocks only made every 5s | ||
- `prof_listen_addr=localhost:6060` so profile server is on by default | ||
- `p2p.send_rate` and `p2p.recv_rate` increases 10x (~5MB/s) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have these changed? The before/after config values are both 5120000
. Or did we miss the changelog last time?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changelog entry is in v0.20 but I don't think it actually took effect, so it was just using the Tendermint defaults
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested ACK, not sure about one changelog comment but very minor.
Reverting just the governance changes: diff --git a/x/gov/keeper.go b/x/gov/keeper.go
index 4569743e..992147ab 100644
--- a/x/gov/keeper.go
+++ b/x/gov/keeper.go
@@ -129,9 +129,9 @@ func (keeper Keeper) activateVotingPeriod(ctx sdk.Context, proposal Proposal) {
// Procedures
var (
- defaultMinDeposit int64 = 100
- defaultMaxDepositPeriod int64 = 10000
- defaultVotingPeriod int64 = 10000
+ defaultMinDeposit int64 = 10
+ defaultMaxDepositPeriod int64 = 200
+ defaultVotingPeriod int64 = 200
)
|
Governance CLI test fixes
Codecov Report
@@ Coverage Diff @@
## develop #1690 +/- ##
===========================================
- Coverage 62.37% 60.64% -1.73%
===========================================
Files 120 119 -1
Lines 7117 6906 -211
===========================================
- Hits 4439 4188 -251
- Misses 2427 2479 +52
+ Partials 251 239 -12 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested (make && make test_cli
) ACK
Tagged. |
This is based on v0.21.1 ...