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.

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

GlobalOpenTelemetry instance recreated in Reactive Messaging channels #2539

Closed
ozangunalp opened this issue Mar 15, 2024 · 2 comments · Fixed by #2540
Closed

GlobalOpenTelemetry instance recreated in Reactive Messaging channels #2539

ozangunalp opened this issue Mar 15, 2024 · 2 comments · Fixed by #2540
Assignees
@ozangunalp

Comments

@ozangunalp
Copy link
Collaborator

quarkusio/quarkus#39449

It can be injected with CDI with a failover to getOrCreate through GlobalOpenTelemetry.get()
@ozangunalp ozangunalp self-assigned this Mar 15, 2024
@ozangunalp
Copy link
Collaborator Author

@kabir do you remember more than I do about our discussion ?

ozangunalp added a commit to ozangunalp/smallrye-reactive-messaging that referenced this issue Mar 15, 2024
@ozangunalp
Retreive OpenTelemetry instance through CDI injection instead of rely…
e80d2f9 
…ing on GlobalOpenTelemetry.get

Still falls back to "GlobalOpenTelemetry.get"

Fixes smallrye#2539
ozangunalp added a commit to ozangunalp/smallrye-reactive-messaging that referenced this issue Mar 15, 2024
@ozangunalp
Retreive OpenTelemetry instance through CDI injection instead of rely…
dfa880e 
…ing on GlobalOpenTelemetry.get

Still falls back to "GlobalOpenTelemetry.get"

Fixes smallrye#2539
ozangunalp added a commit to ozangunalp/smallrye-reactive-messaging that referenced this issue Mar 15, 2024
@ozangunalp
Retreive OpenTelemetry instance through CDI injection instead of rely…
30493b4 
…ing on GlobalOpenTelemetry.get

Still falls back to "GlobalOpenTelemetry.get"

Fixes smallrye#2539
@ozangunalp ozangunalp mentioned this issue Mar 15, 2024
Merged
@kabir
Copy link
Contributor

kabir commented Mar 15, 2024

@ozangunalp No, sorry....

ozangunalp added a commit that referenced this issue Jul 25, 2024
@ozangunalp
Retreive OpenTelemetry instance through CDI injection instead of rely…
2cb3951 
…ing on GlobalOpenTelemetry.get

Still falls back to "GlobalOpenTelemetry.get"

Fixes #2539

(cherry picked from commit 30493b4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ozangunalp ozangunalp

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@kabir @ozangunalp