Skip to content

L2 Builder Uni: Trim redundant fields from rsyslog JSON envelope#146

Open
KA-ROM wants to merge 1 commit into
mainfrom
DVO-1539/l2_builder_uni_trim_log_envelope
Open

L2 Builder Uni: Trim redundant fields from rsyslog JSON envelope#146
KA-ROM wants to merge 1 commit into
mainfrom
DVO-1539/l2_builder_uni_trim_log_envelope

Conversation

@KA-ROM
Copy link
Copy Markdown
Contributor

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

Summary

Removes 3 redundant fields from rsyslog JSON envelope templates (sysloghost, severity, facility). All live mainnet builder logs carry these duplications:

  • Hostname (2x): jsonPayload.sysloghost duplicates labels.compute.googleapis.com/resource_name
  • Severity (2x): jsonPayload.severity always reports info regardless of actual app log level. Misleading and duplicated by jsonPayload.message.level
  • Facility: always "daemon"

AI generated:

  ┌─────────────┬───────────────┬──────────────┬─────────┐
  │   Source    │ Avg line size │ Removed/line │ Savings │
  ├─────────────┼───────────────┼──────────────┼─────────┤
  │ bproxy      │ 718 bytes     │ 90 bytes     │ 12.5%   │
  ├─────────────┼───────────────┼──────────────┼─────────┤
  │ op-rbuilder │ 462 bytes     │ 90 bytes     │ 19.5%   │
  └─────────────┴───────────────┴──────────────┴─────────┘

 ~90 bytes removed is constant. Percentage depends on how fat the line is: short op-rbuilder lines 
 save more proportionally, longer bproxy lines save less.
 

Base automatically changed from trunk/l2-builder-uni to main May 25, 2026 11:45
@0x416e746f6e 0x416e746f6e requested a review from a team as a code owner May 25, 2026 11:45
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