Skip to content

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

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

[WIP] 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

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

Copy link
Copy Markdown
Contributor

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
@KA-ROM KA-ROM marked this pull request as draft June 4, 2026 08:15
@KA-ROM KA-ROM changed the title L2 Builder Uni: Trim redundant fields from rsyslog JSON envelope [WIP] L2 Builder Uni: Trim redundant fields from rsyslog JSON envelope Jun 4, 2026
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