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

Relegate some p2p messages #4725

Merged
merged 1 commit into from Feb 3, 2020
Merged

Relegate some p2p messages #4725

merged 1 commit into from Feb 3, 2020

Conversation

mcdee
Copy link
Contributor

@mcdee mcdee commented Feb 3, 2020

p2p is quite chatty when it has connection requests from peers it does not want to talk to. This patch changes a number of log entries from debug to trace, to reduce this.

(Note that when libp2p takes application-layer hints as to which connections to proceed with these could be returned to debug, as at that point they will point out real issues with the handshake process).

@codecov
Copy link

codecov bot commented Feb 3, 2020

Codecov Report

Merging #4725 into master will increase coverage by 0.28%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #4725      +/-   ##
==========================================
+ Coverage   43.38%   43.66%   +0.28%     
==========================================
  Files         231      204      -27     
  Lines       17484    15298    -2186     
==========================================
- Hits         7585     6680     -905     
+ Misses       8647     7471    -1176     
+ Partials     1252     1147     -105

@prylabs-bulldozer prylabs-bulldozer bot merged commit 00a6361 into prysmaticlabs:master Feb 3, 2020
@mcdee mcdee deleted the connlog branch February 5, 2020 09:23
cryptomental pushed a commit to cryptomental/prysm that referenced this pull request Feb 24, 2020
cryptomental pushed a commit to cryptomental/prysm that referenced this pull request Feb 28, 2020
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