Skip to content

DVO-1536: Cleanup some logs#11

Open
KA-ROM wants to merge 8 commits intomainfrom
DVO-1536/remove_logs
Open

DVO-1536: Cleanup some logs#11
KA-ROM wants to merge 8 commits intomainfrom
DVO-1536/remove_logs

Conversation

@KA-ROM
Copy link
Copy Markdown

@KA-ROM KA-ROM commented May 6, 2026

Summary

Reduce log volume on high-frequency paths already covered by metrics. Keep detailed error logs for investigation.

What

  • Mirror success log removed: redundant with MirrorSuccessCount metric. Error logs kept with request body detail
    for investigation.
  • Websocket "Proxied message" log removed: fired on every flashblock message to every client, highest volume log
    in the codebase. Chaos events now log independently instead of piggybacking on this line.
  • Websocket write failure logs added: write failures now include message type, size, and connection addresses for
    debugging.
  • Proxy success log gated on enriched triage data: methods without decoded txs or deadlines are skipped in logs as metrics coves the info they expose (mainly affects eth_sendBundle proxy success logs)
  • Redundant DecodeEthRawTransaction call fixed: was decoding the same tx twice in prepareLogFields error path.

AI generated:

  Top log messages by volume:
  ┌────────────────────────────┬────────────────────┬────────────────┐
  │          Message           │       Proxy        │ Count (of 500) │
  ├────────────────────────────┼────────────────────┼────────────────┤
  │ Mirrored the request       │ bproxy-rpc         │ 156            │
  ├────────────────────────────┼────────────────────┼────────────────┤
  │ Proxied the request        │ bproxy-rpc         │ 114            │
  ├────────────────────────────┼────────────────────┼────────────────┤
  │ Proxied message            │ bproxy-flashblocks │ 100            │
  ├────────────────────────────┼────────────────────┼────────────────┤
  │ Mirrored the request       │ bproxy-authrpc     │ 64             │
  ├────────────────────────────┼────────────────────┼────────────────┤
  │ Proxied the request        │ bproxy-authrpc     │ 46             │
  ├────────────────────────────┼────────────────────┼────────────────┤
  │ Ignoring FCUv3 w/o payload │ bproxy-authrpc     │ 10             │
  ├────────────────────────────┼────────────────────┼────────────────┤
  │ Faked the request          │ bproxy-authrpc     │ 10             │
  └────────────────────────────┴────────────────────┴────────────────┘

  "Proxied the request" breakdown by method:

  ┌────────────────────────────────────────┬────────────────┬───────┐
  │                 Method                 │ Count (of 500) │   %   │
  ├────────────────────────────────────────┼────────────────┼───────┤
  │ eth_sendBundle                         │ 323            │ 64.6% │
  ├────────────────────────────────────────┼────────────────┼───────┤
  │ engine_newPayloadV4                    │ 106            │ 21.2% │
  ├────────────────────────────────────────┼────────────────┼───────┤
  │ eth_sendRawTransaction                 │ 31             │ 6.2%  │
  ├────────────────────────────────────────┼────────────────┼───────┤
  │ engine_forkchoiceUpdatedV3             │ 18             │ 3.6%  │
  ├────────────────────────────────────────┼────────────────┼───────┤
  │ engine_forkchoiceUpdatedV3_withPayload │ 16             │ 3.2%  │
  ├────────────────────────────────────────┼────────────────┼───────┤
  │ engine_getPayloadV4                    │ 6              │ 1.2%  │
  └────────────────────────────────────────┴────────────────┴───────┘

The changes in this PR target the top 3 producers.

@KA-ROM KA-ROM requested a review from 0x416e746f6e May 6, 2026 14:06
@KA-ROM
Copy link
Copy Markdown
Author

KA-ROM commented May 7, 2026

@claude review

EDIT: :(

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.

1 participant