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

tendermint unconditional peers #2855

Merged
merged 2 commits into from
Apr 20, 2020

Conversation

ptrus
Copy link
Member

@ptrus ptrus commented Apr 18, 2020

Expose new p2p config options which were added in tendermint 0.33

@ptrus ptrus force-pushed the ptrus/feature/tendermint-unconditional-peers branch 2 times, most recently from b4425ab to 228771d Compare April 18, 2020 07:31
@codecov
Copy link

codecov bot commented Apr 18, 2020

Codecov Report

Merging #2855 into master will increase coverage by 0.22%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2855      +/-   ##
==========================================
+ Coverage   67.33%   67.56%   +0.22%     
==========================================
  Files         344      344              
  Lines       33406    33411       +5     
==========================================
+ Hits        22495    22575      +80     
+ Misses       8007     7940      -67     
+ Partials     2904     2896       -8     
Impacted Files Coverage Δ
go/consensus/tendermint/tendermint.go 68.32% <100.00%> (+1.70%) ⬆️
...n/crypto/signature/signers/memory/memory_signer.go 71.42% <0.00%> (-4.77%) ⬇️
go/storage/metrics.go 81.52% <0.00%> (-3.27%) ⬇️
go/storage/mkvs/lookup.go 73.46% <0.00%> (-2.05%) ⬇️
go/storage/client/client.go 76.21% <0.00%> (-0.89%) ⬇️
go/worker/compute/merge/committee/node.go 75.37% <0.00%> (-0.75%) ⬇️
go/storage/database/database.go 77.27% <0.00%> (ø)
go/consensus/tendermint/apps/staking/state/gas.go 81.25% <0.00%> (ø)
go/worker/registration/worker.go 67.14% <0.00%> (+0.41%) ⬆️
...o/consensus/tendermint/apps/staking/state/state.go 57.35% <0.00%> (+0.84%) ⬆️
... and 15 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 33e86c0...b949606. Read the comment docs.

Copy link
Member

@kostko kostko left a comment

Choose a reason for hiding this comment

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

Minor comments, otherwise looks good, thanks!

.changelog/2855.feature.md Outdated Show resolved Hide resolved
.changelog/2855.feature.md Show resolved Hide resolved
go/consensus/tendermint/tendermint.go Outdated Show resolved Hide resolved
@ptrus ptrus force-pushed the ptrus/feature/tendermint-unconditional-peers branch from 228771d to 9380f6b Compare April 20, 2020 07:15
Tendermint 0.33 added the concept of unconditional P2P peers. This
adds support for setting the unconditional peers via
`tendermint.p2p.unconditional_peer_ids` configuration flag.

On sentry node, upstream nodes will automatically be set as
unconditional peers.
@ptrus ptrus force-pushed the ptrus/feature/tendermint-unconditional-peers branch from 9380f6b to 492f835 Compare April 20, 2020 07:15
@ptrus ptrus requested a review from kostko April 20, 2020 07:16
@ptrus ptrus force-pushed the ptrus/feature/tendermint-unconditional-peers branch 2 times, most recently from a3f9c74 to 24eb89f Compare April 20, 2020 07:17
go/consensus/tendermint/tendermint.go Outdated Show resolved Hide resolved
Tendermint 0.33 added support for setting maximum re-dial period when
dialing persistent peers. This adds support for setting the period via
`tendermint.p2p.persistent_peers_max_dial_period` flag.
@ptrus ptrus force-pushed the ptrus/feature/tendermint-unconditional-peers branch from 24eb89f to b949606 Compare April 20, 2020 07:31
@ptrus ptrus merged commit 2092f90 into master Apr 20, 2020
@ptrus ptrus deleted the ptrus/feature/tendermint-unconditional-peers branch April 20, 2020 07:52
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.

None yet

2 participants