Commit c80a3ee3 authored by Adrian Sutton's avatar Adrian Sutton

op-node: Reduce weight and cap for p2p sync error responses.

Previously these were capped to prevent peers being banned on this measure alone, but they could be downscored enough to ignore gossip from them. While p2p alt sync is not widely enabled we need to keep peers that don't support it and so significantly reduce this weighting.
parent d8d41bcb
...@@ -38,11 +38,11 @@ func LightApplicationScoreParams(cfg *rollup.Config) ApplicationScoreParams { ...@@ -38,11 +38,11 @@ func LightApplicationScoreParams(cfg *rollup.Config) ApplicationScoreParams {
ValidResponseWeight: 0.5, ValidResponseWeight: 0.5,
ValidResponseDecay: ScoreDecay(tenEpochs, slot), ValidResponseDecay: ScoreDecay(tenEpochs, slot),
// Takes 20 error responses to reach the default ban threshold of -100 // Takes 10 error responses to reach the default gossip threshold of -10
// But at most we track 10. These errors include not supporting p2p sync // But at most we track 9. These errors include not supporting p2p sync
// so we don't (yet) want to ban a peer based on this measure alone. // so we don't (yet) want to ignore gossip from a peer based on this measure alone.
ErrorResponseCap: 10, ErrorResponseCap: 9,
ErrorResponseWeight: -5, ErrorResponseWeight: -1,
ErrorResponseDecay: ScoreDecay(tenEpochs, slot), ErrorResponseDecay: ScoreDecay(tenEpochs, slot),
// Takes 5 rejected payloads to reach the default ban threshold of -100 // Takes 5 rejected payloads to reach the default ban threshold of -100
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment