Skip to content

W-19607301: OpenTelemetry traces, logs and direct export documentation. #3193

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.

Sign up for GitHub

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

Jump to bottom
Open
IvanAndresFritzler wants to merge 18 commits into
base: v4.10
Choose a base branch
from
Open

W-19607301: OpenTelemetry traces, logs and direct export documentation. #3193

IvanAndresFritzler wants to merge 18 commits into from

Conversation

@IvanAndresFritzler
Copy link
Contributor

@IvanAndresFritzler IvanAndresFritzler commented Nov 4, 2025

Writer's Quality Checklist

Before merging your PR, did you:

  • Run spell checker
  • Run link checker to check for broken xrefs
  • Check for orphan files
  • Perform a local build and do a final visual check of your content, including checking for:
    • Broken images
    • Dead links
    • Correct rendering of partials if they are used in your content
    • Formatting issues, such as:
      • Misnumbered ordered lists (steps) or incorrectly nested unordered lists
      • Messed up tables
      • Proper indentation
      • Correct header levels
  • Receive final review and signoff from:
    • Technical SME
    • Product Manager
    • Editor or peer reviewer
    • Reporter, if this content is in response to a reported issue (internal or external feedback)
  • If applicable, verify that the software actually got released

@IvanAndresFritzler IvanAndresFritzler added the DO NOT MERGE (YET) This is a work in progress. Do not merge nor review these changes yet label Nov 4, 2025
@IvanAndresFritzler IvanAndresFritzler marked this pull request as ready for review November 4, 2025 13:11
@IvanAndresFritzler IvanAndresFritzler requested review from a team and dmerlob as code owners November 4, 2025 13:11
jmbarbosa80
jmbarbosa80 reviewed Nov 4, 2025
View reviewed changes
jmbarbosa80
jmbarbosa80 reviewed Nov 4, 2025
View reviewed changes
jmbarbosa80
jmbarbosa80 reviewed Nov 4, 2025
View reviewed changes
IvanAndresFritzler and others added 2 commits November 4, 2025 10:48
@IvanAndresFritzler
Review
cb858a4
@IvanAndresFritzler @jmbarbosa80
Peer review
b826848 
Co-authored-by: Jorge Barbosa <59669286+jmbarbosa80@users.noreply.github.com>
@dmerlob dmerlob self-assigned this Nov 4, 2025
dmerlob
dmerlob reviewed Nov 10, 2025
View reviewed changes
dmerlob
dmerlob reviewed Nov 11, 2025
View reviewed changes
@IvanAndresFritzler @dmerlob
Update modules/ROOT/pages/otel-support.adoc
0595dc1 
Co-authored-by: Daniela Merlo <97902112+dmerlob@users.noreply.github.com>
dmerlob
dmerlob reviewed Nov 20, 2025
View reviewed changes
@IvanAndresFritzler @dmerlob
Apply suggestions from code review
89a6c0c 
Co-authored-by: Daniela Merlo <97902112+dmerlob@users.noreply.github.com>
dmerlob
dmerlob reviewed Nov 20, 2025
View reviewed changes
dmerlob
dmerlob previously approved these changes Nov 20, 2025
View reviewed changes
dmerlob
dmerlob reviewed Nov 20, 2025
View reviewed changes
modules/ROOT/pages/otel-support.adoc
----

[NOTE]
The `mule.put.trace.id.and.span.id.in.mdc` property won't be needed after GA because Mule runtime sets it automatically when traces are exported.
Copy link
Contributor

@dmerlob dmerlob Nov 20, 2025

Choose a reason for hiding this comment

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

This needs to be modified when GA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jmbarbosa80 jmbarbosa80 jmbarbosa80 left review comments

@dmerlob dmerlob dmerlob left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@dmerlob dmerlob

Labels

DO NOT MERGE (YET) This is a work in progress. Do not merge nor review these changes yet Mule-4.11

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@IvanAndresFritzler @jmbarbosa80 @dmerlob