Skip to content

Conversation

@ktoso
Copy link
Contributor

@ktoso ktoso commented Oct 8, 2025

The default tracer should be picked up at configuration creation time from the bootstrapped one, so we don't have to manually configure it in the simplest cases.

It is possible to disable it by explicitly setting nil on the tracing configuration, even if the global tracer was bootstrapped.

FYI @slashmo

The default tracer should be picked up at configuration creation time
from the bootstrapped one, so we don't have to manually configure it in
the simplest cases.

It is possible to disable it by explicitly setting `nil` on the tracing
configuration, even if the global tracer was bootstrapped.
@ktoso ktoso requested a review from glbrntt October 8, 2025 23:35
@ktoso ktoso added the 🔨 semver/patch No public API change. label Oct 8, 2025
@ktoso ktoso merged commit c2a3a2c into swift-server:main Oct 9, 2025
34 of 36 checks passed
@ktoso ktoso deleted the wip-forgot-to-default-to-global branch October 9, 2025 08:01
@ktoso
Copy link
Contributor Author

ktoso commented Oct 9, 2025

Thank you!

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

Labels

🔨 semver/patch No public API change.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants