-
Notifications
You must be signed in to change notification settings - Fork 38.7k
[doc] add documentation about reducing traffic #6987
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
Conversation
|
Release notes for maxuploadtarget is handled over #6958. |
|
Looks good. Concept ACK |
doc/reducetraffic.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: No CAPSLOCK
e63226e to
29ede38
Compare
|
Fixed @MarcoFalke's nits. The bold test part is okay IMO. |
doc/reducetraffic.md
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- bitcoin's
- handful
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Force-Push fixed.
29ede38 to
de75f50
Compare
de75f50 to
42bae94
Compare
|
ACK |
|
utACK |
|
ACK. (Nit: I don't like the use of the passive voice; for example "Reducing the maximum connected nodes to a miniumum could be desirable" would be clearer as "Set -maxconnections=1 for minimal bandwidth usage" ... but I think this is good enough to pull as-is). |
Introduce -maxuploadtarget Cherry-picked from the following upstream PRs: - bitcoin/bitcoin#6622 - bitcoin/bitcoin#6987 - bitcoin/bitcoin#6958 - bitcoin/bitcoin#6984 - bitcoin/bitcoin#6999 Part of #2074.
We have seen node operators shutting down nodes because of traffic exceeding. Reducing traffic is possible with different measures and therefore very likely a "communication problem". This documentation might help those who like to or need to reduce traffic.
Thanks @fanquake for the overhaul!