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

Mutiny & Opentelemetry #911

Closed
ferencbeutel4711 opened this issue May 9, 2022 · 1 comment · Fixed by #1219 · May be fixed by jponge/smallrye-mutiny#69, geoand/smallrye-mutiny#36 or hpehl/smallrye-mutiny#49
Closed

Mutiny & Opentelemetry #911

ferencbeutel4711 opened this issue May 9, 2022 · 1 comment · Fixed by #1219 · May be fixed by jponge/smallrye-mutiny#69, geoand/smallrye-mutiny#36 or hpehl/smallrye-mutiny#49

Comments

@ferencbeutel4711
Copy link

ferencbeutel4711 commented May 9, 2022

Hello there,

we are currently trying to implement opentelemetry using the quarkus opentelemetry package. We are using Mutiny in a lot of places in our application. Now we are facing the following issue:
Every time we are using Uni/Multi in our Code, the tracing information is wrong. There are 2 main issues:

  • The recorded duration takes only the synchronous part of the method into account
  • methods called inside of the Uni/Multi, for example transformation or invoke blocks, are not correlated to the parent span

We think these issues are there due to the threading nature of Mutiny and that the opentelemetry context is lost during those thread switches. Do you have any idea on compatibility? I know there is a smallrye-opentelemetry package here on github, however it is labeled as a POC so not sure what the current situation is

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