-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Add OpenTelemetry tracing spans for RESTEasy Reactive clients #18543
Conversation
kenfinnigan
commented
Jul 8, 2021
- Fixes Distributed tracing for RestEasy Reactive #17800
- Remove "hidden" opentelemetry-rest-client extension as it's no longer needed
- Prevent the VertxTracingAdaptor from creating a CLIENT span when REST Client has done it. Check for "traceparent" header in request already
- Fix RESTEasy Reactive client url template path by removing double slash
@kenfinnigan I'm really happy to see this :) Could you add some test for Rest Client Reactive? |
In the integration tests for RR? |
I was thinking somewhere in the integration tests in the Quarkus project. |
Sorry, that's what I meant. The integration tests for RR in Q |
- Fixes quarkusio#17800 - Remove "hidden" opentelemetry-rest-client extension as it's no longer needed - Prevent the VertxTracingAdaptor from creating a CLIENT span when REST Client has done it. Check for "traceparent" header in request already - Fix RESTEasy Reactive client url template path by removing double slash
207e0aa
to
f3329f9
Compare
@michalszynkiewicz test added |
that was fast. I don't know much about opentelemetry but the parts I understand look good to me. |
@ebullient could you have a look at this one before we wrap 2.1 CR1? It looks like an interesting addition. Thanks! |