Skip to content

Add Logs info to OTel docs#1002

Merged
ProgrammerAL merged 7 commits intomainfrom
ar/logs-to-otel
Feb 3, 2026
Merged

Add Logs info to OTel docs#1002
ProgrammerAL merged 7 commits intomainfrom
ar/logs-to-otel

Conversation

@ProgrammerAL
Copy link
Copy Markdown
Contributor

@ProgrammerAL ProgrammerAL commented Jan 30, 2026

  • Updated /identityserver/diagnostics/otel page to detail how logs can be exported to OTel traces
    • Added the OpenTelemetry Signals header towards the top to group signals together (traces, logs, metrics)
    • Switch Traces and Logs categories
    • Added some images to show logs in a trace in Aspire
  • Updated /identityserver/diagnostics/logging page to include a blurb about logging OTel info in the otel page

Fixes #973

@ProgrammerAL ProgrammerAL self-assigned this Jan 30, 2026
@ProgrammerAL ProgrammerAL added the documentation Improvements or additions to documentation label Jan 30, 2026
Copy link
Copy Markdown
Member

@maartenba maartenba left a comment

Choose a reason for hiding this comment

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

Small change suggested to code sample.

Does this also work with Serilog? (given how https://github.com/orgs/DuendeSoftware/discussions/451#discussioncomment-15453156 is hooking things up directly)

Comment thread src/content/docs/identityserver/diagnostics/otel.md Outdated
Co-authored-by: Maarten Balliauw <maarten.balliauw@duendesoftware.com>
@ProgrammerAL
Copy link
Copy Markdown
Contributor Author

@maartenba The code sample in that GitHub Issue is specific to Serilog. The code sample in this PR is generic to ASP.NET Core for anyone using ILogger. I can include that Serilog specific sample, but if we do that should we also include samples for other logging frameworks?

@maartenba
Copy link
Copy Markdown
Member

We have Serilog in many other places (it's the default in our templates) so makes sense to add imo. Also feel free to cross-reference to (and from) https://docs.duendesoftware.com/general/logging/#setup-for-serilog

@ProgrammerAL
Copy link
Copy Markdown
Contributor Author

@maartenba Added a blurb at the bottom of src/content/docs/identityserver/diagnostics/otel.md for logging to OTel using the Serilog specific API. Let me know what you think.

Comment thread src/content/docs/identityserver/diagnostics/otel.md Outdated
Copy link
Copy Markdown
Member

@maartenba maartenba left a comment

Choose a reason for hiding this comment

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

Perfect! Merge at will. Thanks @ProgrammerAL !

@ProgrammerAL ProgrammerAL merged commit 5d9c7ff into main Feb 3, 2026
7 checks passed
@ProgrammerAL ProgrammerAL deleted the ar/logs-to-otel branch February 3, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add logging to OpenTelemetry with Serilog

2 participants