Skip to content

[WIP] [p2p] Peer behaviour implementation (#3539)#3549

Closed
brapse wants to merge 2 commits intotendermint:developfrom
brapse:peer-behaviour-implementation
Closed

[WIP] [p2p] Peer behaviour implementation (#3539)#3549
brapse wants to merge 2 commits intotendermint:developfrom
brapse:peer-behaviour-implementation

Conversation

@brapse
Copy link
Contributor

@brapse brapse commented Apr 12, 2019

Follow up implementation from ADR-037 (#3539)

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

@brapse brapse requested review from ebuchman, melekes and xla as code owners April 12, 2019 14:20
@codecov-io
Copy link

Codecov Report

Merging #3549 into develop will decrease coverage by 0.09%.
The diff coverage is 60.86%.

@@            Coverage Diff             @@
##           develop    #3549     +/-   ##
==========================================
- Coverage    64.16%   64.07%   -0.1%     
==========================================
  Files          213      214      +1     
  Lines        17332    17387     +55     
==========================================
+ Hits         11121    11140     +19     
- Misses        5289     5322     +33     
- Partials       922      925      +3
Impacted Files Coverage Δ
p2p/peer_behaviour.go 60.86% <60.86%> (ø)
p2p/pex/errors.go 11.11% <0%> (-11.12%) ⬇️
p2p/node_info.go 70.51% <0%> (-5.88%) ⬇️
libs/events/events.go 93.2% <0%> (-4.86%) ⬇️
consensus/ticker.go 91.66% <0%> (-4.17%) ⬇️
privval/signer_remote.go 80% <0%> (-2%) ⬇️
consensus/wal.go 65.35% <0%> (-1.1%) ⬇️
consensus/reactor.go 70.83% <0%> (-0.71%) ⬇️
p2p/pex/addrbook.go 67% <0%> (-0.5%) ⬇️
... and 7 more

@brapse brapse closed this Apr 12, 2019
@brapse brapse deleted the peer-behaviour-implementation branch April 12, 2019 14:28
@brapse brapse restored the peer-behaviour-implementation branch April 12, 2019 14:29
@brapse brapse deleted the peer-behaviour-implementation branch April 12, 2019 14:31
@brapse brapse restored the peer-behaviour-implementation branch April 12, 2019 14:32
iKapitonau pushed a commit to scrtlabs/tendermint that referenced this pull request Sep 30, 2024
…htClientAttackEvidence` types (backport tendermint#3543) (tendermint#3549)

Closes tendermint#3528 

### Changes
Added missing JSON tags to `DuplicateVoteEvidence` and
`LightClientAttackEvidence` types.

---

#### PR checklist

- [x] Tests written/updated
- [x] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
~- [ ] Updated relevant documentation (`docs/` or `spec/`) and code
comments~
- [x] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec
<hr>This is an automatic backport of pull request tendermint#3543 done by
[Mergify](https://mergify.com).

---------

Co-authored-by: Alessandro Sforzin <alesforz@gmail.com>
Co-authored-by: Alessandro <alessandro@informal.systems>
iKapitonau pushed a commit to scrtlabs/tendermint that referenced this pull request Sep 30, 2024
…dermint#3705)

Given the discussion in tendermint#3528, we are reverting the changes made by
tendermint#3543 in `v0.38.x` (backported with tendermint#3549).
Additionally, we are removing the changes made in tendermint#3543 from the
CHANGELOG.

---

#### PR checklist

~- [ ] Tests written/updated~
~- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)~
~- [] Updated relevant documentation (`docs/` or `spec/`) and code
comments~
- [x] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec
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.

2 participants