Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Decrease log clutter from tcpserver/session #5033

Merged
merged 2 commits into from
Jan 3, 2020

Conversation

jiivan
Copy link
Contributor

@jiivan jiivan commented Jan 3, 2020

Logs are often spammed with non-essential info:

2020-01-03 11:59:27 INFO     golem.network.transport.session     Received disconnect message. reason=ProtocolVersion, address=35.158.160.231:40103 
2020-01-03 11:59:28 INFO     golem.network.transport.tcpserver   Connecting to peer. node='ImALumberJack'(839b6556..3adb9cba), adresses=['35.158.160.231:40103'] 
2020-01-03 11:59:29 INFO     golem.network.transport.tcpserver   Connecting to peer. node='SPLYGolem-002'(d5134615..2c67ab42), adresses=['45.73.63.50:40103'] 
2020-01-03 11:59:29 INFO     golem.network.transport.session     Received disconnect message. reason=ProtocolVersion, address=35.158.160.231:40103 
2020-01-03 11:59:30 INFO     golem.network.transport.tcpserver   Connecting to peer. node='ImALumberJack'(839b6556..3adb9cba), adresses=['35.158.160.231:40103'] 
2020-01-03 11:59:31 INFO     golem.network.transport.tcpserver   Connecting to peer. node='SPLYGolem-002'(d5134615..2c67ab42), adresses=['45.73.63.50:40103'] 
2020-01-03 11:59:31 INFO     golem.network.transport.session     Received disconnect message. reason=ProtocolVersion, address=35.158.160.231:40103 
2020-01-03 11:59:32 INFO     golem.network.transport.tcpserver   Connecting to peer. node='ImALumberJack'(839b6556..3adb9cba), adresses=['35.158.160.231:40103'] 

@jiivan jiivan requested review from etam and kmazurek January 3, 2020 13:07
@codecov
Copy link

codecov bot commented Jan 3, 2020

Codecov Report

Merging #5033 into b0.22 will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            b0.22    #5033      +/-   ##
==========================================
- Coverage   90.08%   90.07%   -0.01%     
==========================================
  Files         234      234              
  Lines       21965    21965              
==========================================
- Hits        19787    19786       -1     
- Misses       2178     2179       +1

Copy link
Contributor

@shadeofblue shadeofblue left a comment

Choose a reason for hiding this comment

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

looks okay

@shadeofblue shadeofblue merged commit 6b6ce0e into b0.22 Jan 3, 2020
@shadeofblue shadeofblue deleted the b0.22-network-logs-severity branch January 3, 2020 13:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants