Skip to content

Added documentation page for traces processor #1614

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Apr 28, 2025

Conversation

eschabell
Copy link
Contributor

@eschabell eschabell commented Apr 27, 2025

This is a new documentation page for the traces feature in Processors with the following additions:

  • new entry in Processors documentation page linking to the Traces doc page.
  • new Traces page in the Processors docs dir with features documented.
  • two new images added and used in the Traces page, from Eduardo's v4 webinar describing tracing features.

Resolves #1611

…age for traces.

Signed-off-by: Eric D. Schabell <eric@schabell.org>
Signed-off-by: Eric D. Schabell <eric@schabell.org>
Added documentation page for traces feature. Fixes issue fluent#1611.
@eschabell eschabell requested review from a team as code owners April 27, 2025 16:43
Copy link
Contributor

@patrick-stephens patrick-stephens left a comment

Choose a reason for hiding this comment

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

Minor typo but otherwise looks ok

Signed-off-by: Eric D. Schabell <eric@schabell.org>
@patrick-stephens patrick-stephens changed the title Added documentation page for traces feature. Fixes issue #1611. Added documentation page for traces processor Apr 28, 2025
@patrick-stephens patrick-stephens merged commit 6dd456e into fluent:master Apr 28, 2025
5 checks passed
@eschabell
Copy link
Contributor Author

@patrick-stephens Just noticed that the sidebar does not generate the entry for traces.

Screenshot 2025-04-28 at 17 07 33

And the page for traces is linked into the repo... not in the docs site... not sure how to fix that?

@patrick-stephens
Copy link
Contributor

patrick-stephens commented Apr 28, 2025

Ah you have to add the new page to the index, missed that but it's pretty common and we should really catch it with CI or be nicer to have a preview.

Check the recent PR that added the filters as processors one is probably the easiest way to see.

@eschabell
Copy link
Contributor Author

I can add it, I'll do a separate PR.

@eschabell
Copy link
Contributor Author

Added it here: #1615

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.

Missing tracing docs in the processors section
2 participants