Skip to content

[reorg fix] Clarify auto multi-line before/after example in Overview - #38844

Merged
rhy988 merged 14 commits into
masterfrom
reorg-fix/pr-38816
Aug 7, 2026
Merged

[reorg fix] Clarify auto multi-line before/after example in Overview#38844
rhy988 merged 14 commits into
masterfrom
reorg-fix/pr-38816

Conversation

@jhgilbert

@jhgilbert jhgilbert commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

🤖 Auto-generated fix for #38816.

@rhy988 — the docs repo reorg created merge conflicts in your PR #38816. This is an auto-generated replacement with the file paths fixed; please use it instead of the original.

This PR replays the commits from #38816 with file paths translated to the post-reorg hugo/ layout. The original commits are preserved — same messages and authorship.

The original PR (#38816) will be closed in favor of this one.

Next steps:

  1. Verify that this PR looks correct in the browser.
  2. Remove the WORK IN PROGRESS label from this PR.
  3. Wait for the standard docs team approval before merging. Optionally, you can check the 'ready for merge' checkbox below if you would like the docs team to merge it for you.
  • Ready for merge

Original PR description:

Follow-up to #37889, addressing @Brian-Floersch's review feedback.

What does this PR do? What is the motivation?

Two fixes to the example in the Overview:

  • The "before" and "after" blocks were identical, so the rendered page didn't actually show what aggregation does. The lines are now annotated to show the log count changing from 5 to 2.
  • The text said the stack trace is aggregated into "a single log." It's two — the trailing INFO line begins with a datetime, so it starts its own log.

Merge readiness

  • Ready for merge

For Datadog employees:

  • ⚠️ Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). If you've already created your PR with an incorrect branch name, please rename your branch and open a fresh PR.
  • 🤖 New: Comment with /review to run an automated check that catches common issues before a Documentation team member reviews your PR.

AI assistance

Used Claude Code to revise the example and draft this description.

Additional notes

Opened as a new PR since #37889 is already merged. Happy to swap the annotated text blocks for a Logs Explorer screenshot if that reads better.

The Overview example rendered identically before and after aggregation, so
the effect of the feature was not visible, and the surrounding text stated
the stack trace was aggregated into "a single log" when the example
actually produces two.

- Annotate each line in the "before" block to show the Agent emits one log
  per line when detection is disabled.
- Bracket the aggregated lines in the "after" block and label the two
  resulting logs, so the grouping is legible in rendered output.
- Explain that only lines beginning with a datetime start a new log, which
  is why the trailing INFO line remains its own log.
- Use `text` fences and add blank lines around them, matching the
  convention used in the custom samples section.

Co-Authored-By: Claude <noreply@anthropic.com>
@jhgilbert
jhgilbert requested a review from a team as a code owner August 6, 2026 13:56
@jhgilbert jhgilbert added WORK IN PROGRESS No review needed, it's a wip ;) astro-reorg-auto-pr Needs manual conflict resolution after replatforming reorg labels Aug 6, 2026
@datadog-official

This comment has been minimized.

@rhy988
rhy988 requested a review from a team August 6, 2026 14:49
@rhy988
rhy988 requested review from a team and removed request for a team August 6, 2026 16:19
Updated examples to illustrate the effect of Auto multi-line detection on log formatting.
```
With Auto multi-line detection enabled, the stack trace is aggregated into a single log:

For example, an application writes the following five lines. Without Auto multi-line detection, the Agent sends each line as its own log, splitting the exception away from the message that introduced it:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@rhy988 Is there a reason Auto is capitalized in this doc? It makes it look like a proper noun or product name, and I'm not sure if that's intentional

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not intentional — good catch. There were a few more instances too, fixed them all.

rhy988 added 2 commits August 6, 2026 15:06
Corrected a minor grammatical error in the explanation of auto multi-line detection.
Corrected capitalization and formatting inconsistencies in the documentation regarding auto multi-line detection.

@evazorro evazorro left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One more edit!

Comment thread hugo/content/en/agent/logs/auto_multiline_detection.md Outdated
@rhy988
rhy988 requested a review from evazorro August 7, 2026 15:22

@evazorro evazorro left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @rhy988!

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

rhy988 added 4 commits August 7, 2026 16:10
Updated example log output to reflect auto multi-line detection.
Clarified how auto multi-line detection aggregates logs.
Clarified the default behavior of auto multi-line detection regarding log entries.
@rhy988
rhy988 merged commit 3c35abc into master Aug 7, 2026
16 of 17 checks passed
@rhy988
rhy988 deleted the reorg-fix/pr-38816 branch August 7, 2026 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astro-reorg-auto-pr Needs manual conflict resolution after replatforming reorg WORK IN PROGRESS No review needed, it's a wip ;)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants