Skip to content

Ensure correct execution order for user-provided delegates#325

Merged
stevejgordon merged 4 commits into
mainfrom
311-bug-custom-processors
Nov 24, 2025
Merged

Ensure correct execution order for user-provided delegates#325
stevejgordon merged 4 commits into
mainfrom
311-bug-custom-processors

Conversation

@stevejgordon
Copy link
Copy Markdown
Contributor

@stevejgordon stevejgordon commented Nov 11, 2025

The primary change here is to ensure that we register EDOT defaults before user-provided configuration delegates and add the OTLP exporter after them. Introduces BuilderContext and BuilderOptions to support this.

Other changes

  • Upgrade OpenTelemetry packages in sample apps to version 1.13.x across project files.
  • Enhance logging with new DeferredLogger and update FileLogger for better log handling.
  • Improve documentation and configuration patterns in builder extension files.

Closes #311
Closes #319

The primary change here is to ensure that we register EDOT defaults before
user-provided configuration delegates and add the OTLP exporter after them.
Introduces `BuilderContext` and `BuilderOptions` to support this.

Other changes
- Upgrade OpenTelemetry packages in sample apps to version 1.13.x across project files.
- Enhance logging with new `DeferredLogger` and update `FileLogger` for better log handling.
- Improve documentation and configuration patterns in builder extension files.
@stevejgordon stevejgordon marked this pull request as ready for review November 23, 2025 06:59
@stevejgordon stevejgordon requested a review from a team as a code owner November 23, 2025 06:59
Comment thread src/Elastic.OpenTelemetry.Core/Diagnostics/DeferredLogger.cs Outdated
Comment thread src/Elastic.OpenTelemetry.Core/Diagnostics/FileLogger.cs
Comment thread src/Elastic.OpenTelemetry.Core/SignalBuilder.cs Outdated
@stevejgordon stevejgordon force-pushed the 311-bug-custom-processors branch from 2cf6e89 to c910b03 Compare November 24, 2025 11:46
Copy link
Copy Markdown

@JeremyBessonElastic JeremyBessonElastic left a comment

Choose a reason for hiding this comment

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

LGTM

@stevejgordon stevejgordon merged commit 88708d7 into main Nov 24, 2025
9 checks passed
@stevejgordon stevejgordon deleted the 311-bug-custom-processors branch November 24, 2025 13:03
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.

Update XML documentation comments for registration methods Custom processors run after OTLP exporter

2 participants