Skip to content

[doc] Document event log levels#658

Merged
wenjin272 merged 2 commits into
apache:mainfrom
weiqingy:541-docs
May 13, 2026
Merged

[doc] Document event log levels#658
wenjin272 merged 2 commits into
apache:mainfrom
weiqingy:541-docs

Conversation

@weiqingy
Copy link
Copy Markdown
Collaborator

Linked issue: #541

Purpose of change

Documentation follow-up to #609 — that PR shipped the per-event-type configurable event log levels, but the user-facing docs were not updated. Reviewer asked that docs be supplemented before closing #541.

This PR documents:

docs/content/docs/operations/configuration.md — five new rows in the Core Options table:

  • event-log.level — global default (OFF / STANDARD / VERBOSE)
  • event-log.type.<EVENT_TYPE>.level — per-event-type override
  • event-log.standard.max-string-length, event-log.standard.max-array-elements, event-log.standard.max-depthSTANDARD-level truncation knobs (set to 0 to disable individually)

docs/content/docs/operations/monitoring.md — new Event Log subsections:

  • JSON Format — top-level timestamp / logLevel / eventType layout, plus the full event object
  • Event Log Levels — semantics of OFF / STANDARD / VERBOSE, what is and is not truncated, and the truncation wrapper format (truncatedString / truncatedList / truncatedObject with omitted* counters), with a STANDARD-level truncated record example
  • Per-event-type log levels — reference table of built-in event type routing strings (_input_event, _chat_request_event, etc.), hierarchical (Log4j-style) resolution order for dotted user-defined types, config.yaml example, and -D job-submission override example
  • Compatibility NotesSTANDARD-by-default is a behavior change vs prior releases (use VERBOSE to restore); pre-feature log records still parse
  • Event and Action Metrics — new eventLogTruncatedEvents counter row

Tests

Docs-only change — no executable changes. Verified all referenced config keys, defaults, JSON field names, truncation wrapper shapes, and metric names against the code shipped in #609 (AgentConfigOptions, EventLogLevel, EventLogLevelResolver, FileEventLogger, EventLogRecordJsonSerializer, EventLogRecordJsonDeserializer, JsonTruncator, BuiltInMetrics).

API

No API change.

Documentation

  • doc-needed
  • doc-not-needed
  • doc-included

weiqingy added 2 commits May 11, 2026 21:00
Document the event log level feature shipped in apache#609 (issue apache#541):

- New core options: event-log.level, event-log.type.<EVENT_TYPE>.level,
  event-log.standard.max-string-length, event-log.standard.max-array-elements,
  event-log.standard.max-depth.
- Event Log section in monitoring.md now describes the JSON layout
  (including top-level eventType and logLevel), the three levels
  (OFF / STANDARD / VERBOSE), and the hierarchical per-event-type
  override resolution.
- Built-in event type strings table for quick reference.

Closes apache#541
…t notes

Additional coverage based on the DESIGN doc (docs/PR-541/541-DESIGN.md):

- Document the STANDARD-level truncation wrapper format
  (truncatedString / truncatedList / truncatedObject + omitted* counters)
  and note that consumers needing a stable schema should use VERBOSE.
- Add a STANDARD-level truncated record example.
- Note that fields like timestamp, logLevel, top-level eventType,
  and the event's id/type/scalar fields are never truncated.
- Explain that setting any threshold to 0 disables that truncation,
  and setting all three to 0 makes STANDARD behave like VERBOSE.
- Add eventLogTruncatedEvents to the Event and Action Metrics table.
- Add a -D job-submission override example showing how a single
  type-level override does not clobber the rest of config.yaml.
- Compatibility notes: STANDARD-by-default is a behavior change
  (use VERBOSE to restore), and pre-feature log records still parse.
@github-actions github-actions Bot added doc-included Your PR already contains the necessary documentation updates. fixVersion/0.3.0 The feature or bug should be implemented/fixed in the 0.3.0 version. priority/major Default priority of the PR or issue. labels May 12, 2026
Copy link
Copy Markdown
Collaborator

@wenjin272 wenjin272 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wenjin272 wenjin272 merged commit 3abbb20 into apache:main May 13, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-included Your PR already contains the necessary documentation updates. fixVersion/0.3.0 The feature or bug should be implemented/fixed in the 0.3.0 version. priority/major Default priority of the PR or issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants