Skip to content
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

Allow logging via OpenTelemetry #9916

Closed
knutwannheden opened this issue Jun 10, 2020 · 4 comments
Closed

Allow logging via OpenTelemetry #9916

knutwannheden opened this issue Jun 10, 2020 · 4 comments
Labels
area/tracing kind/enhancement New feature or request

Comments

@knutwannheden
Copy link
Contributor

Description
Allow the smallrye-opentracing to optionally register a logging handler which would add all logging events above a given threshold level to the currently active tracing span (if any).

The user would use dedicated config properties to enable this feature (disabled by default) and configure the threshold level to use to filter the events to log via OpenTracing.

@knutwannheden knutwannheden added the kind/enhancement New feature or request label Jun 10, 2020
@kenfinnigan
Copy link
Member

I don't believe we'd implement this with OpenTracing, but we could look to do it with OpenTelemetry when that's added to Quarkus

@kenfinnigan kenfinnigan changed the title Allow logging via OpenTracing Allow logging via OpenTelemetry May 10, 2021
@kenfinnigan
Copy link
Member

I changed the title to refer to OpenTelemetry, as that's the tracing approach we'd implement this idea with

@jtama
Copy link
Contributor

jtama commented Aug 3, 2022

Is it possible to help on this ?

For reference Log and Event Record Definition

@brunobat
Copy link
Contributor

@knutwannheden will close this issue because we plan to have support for OpenTelemetry Logging: #30712

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tracing kind/enhancement New feature or request
Projects
Status: Done
Development

No branches or pull requests

4 participants