Skip to content

config: set time_iota_ms to timeout_commit in test genesis#5386

Merged
melekes merged 4 commits intomasterfrom
anton/light-rpctest-4489
Sep 23, 2020
Merged

config: set time_iota_ms to timeout_commit in test genesis#5386
melekes merged 4 commits intomasterfrom
anton/light-rpctest-4489

Conversation

@melekes
Copy link
Contributor

@melekes melekes commented Sep 23, 2020

also, document consensus parameters.
https://forum.cosmos.network/t/consensus-timeouts-explained/1421

Closes #4489

@codecov
Copy link

codecov bot commented Sep 23, 2020

Codecov Report

Merging #5386 into master will decrease coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5386      +/-   ##
==========================================
- Coverage   61.48%   61.27%   -0.22%     
==========================================
  Files         259      259              
  Lines       23385    23385              
==========================================
- Hits        14379    14328      -51     
- Misses       7549     7600      +51     
  Partials     1457     1457              
Impacted Files Coverage Δ
config/config.go 79.23% <ø> (ø)
config/toml.go 60.97% <ø> (ø)
blockchain/v0/pool.go 74.53% <0.00%> (-4.06%) ⬇️
consensus/reactor.go 74.33% <0.00%> (-3.53%) ⬇️
privval/signer_endpoint.go 78.78% <0.00%> (-3.04%) ⬇️
p2p/switch.go 63.66% <0.00%> (-2.18%) ⬇️
statesync/syncer.go 78.48% <0.00%> (-0.85%) ⬇️
p2p/pex/pex_reactor.go 79.46% <0.00%> (-0.30%) ⬇️
consensus/state.go 68.13% <0.00%> (-0.19%) ⬇️
crypto/sr25519/pubkey.go 52.17% <0.00%> (+8.69%) ⬆️

Copy link
Contributor

@erikgrinaker erikgrinaker left a comment

Choose a reason for hiding this comment

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

Thanks for documenting! ❤️

@melekes melekes merged commit 4b99502 into master Sep 23, 2020
@melekes melekes deleted the anton/light-rpctest-4489 branch September 23, 2020 08:24
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.

light: rpctest.Tendermint instance's clock drifts +7min

2 participants