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

OpenTracing Jaeger: Probabilistic sampler didn't work #21002

Closed
loicmathieu opened this issue Oct 26, 2021 · 12 comments · Fixed by #22813
Closed

OpenTracing Jaeger: Probabilistic sampler didn't work #21002

loicmathieu opened this issue Oct 26, 2021 · 12 comments · Fixed by #22813
Labels
area/jaeger area/smallrye area/tracing env/windows Impacts Windows machines kind/bug Something isn't working
Milestone

Comments

@loicmathieu
Copy link
Contributor

Describe the bug

When using a probabilistic sample (with 0.5 as sampling rate) no traces are generated.
When using a constant sampler (with 1 as sampling rate) traces are correctly generated.

Expected behavior

We can use a probabilistic sampler in Quarkus.

Actual behavior

No response

How to Reproduce?

Steps to reproduce:

  1. Download the sample project (configured with a probabilistic sampler, see
    https://www.jaegertracing.io/docs/1.22/sampling/#client-sampling-configuration): jaeger-probabilistic-sampler.zip
  2. Start Jaeger: docker run -p 5775:5775/udp -p 6831:6831/udp -p 6832:6832/udp -p 5778:5778 -p 16686:16686 -p 14268:14268 jaegertracing/all-in-one:latest
  3. Start the applicaiton via mvn quarkus:dev
  4. Call the endpoint a few times: curl localhost:8080/hello
  5. Go to the Jager UI (should be on port 16686): nothing shows up!

If you switch to a constant sampler, traces will shows up, you can verify this by updating the configuration with

quarkus.jaeger.sampler-type=const
quarkus.jaeger.sampler-param=1

Output of uname -a or ver

Microsoft Windows [version 10.0.19042.1237]

Output of java -version

openjdk version "17" 2021-09-14

GraalVM version (if different from Java)

No response

Quarkus version or git rev

2.3.1.Final

Build tool (ie. output of mvnw --version or gradlew --version)

Apache Maven 3.6.3

Additional information

No response

@loicmathieu loicmathieu added the kind/bug Something isn't working label Oct 26, 2021
@quarkus-bot
Copy link

quarkus-bot bot commented Oct 26, 2021

/cc @Ladicek, @radcortez

@radcortez
Copy link
Member

Hum, I've followed the repro steps and it seems to work fine for me:

Screenshot 2021-11-05 at 17 37 16

Can you confirm?

@radcortez
Copy link
Member

The only difference that I have is the OS. Let me see if I can dig up a Windows VM to try that out.

@loicmathieu
Copy link
Contributor Author

I'll ask on Monday one of my colleagues that have a Mac to test and see if it works.

@radcortez
Copy link
Member

I was able to try this on my Windows VM and it also works... wondering if it may be a difference in the jaeger docker image we are getting? I would expect it would be the same for both of us.

@radcortez
Copy link
Member

Any update on this?

@loicmathieu
Copy link
Contributor Author

Just tried it on my Windows laptop using Quarkus 2.6.1 and I can reproduce the issue.

@loicmathieu
Copy link
Contributor Author

Just tried also on my Linux laptop and I can also reproduce the issue.

@loicmathieu
Copy link
Contributor Author

Using tcpdump I can clearly see that when using probabilistic sampler no UDB packet are sent to the port 6831 but when using constant sampler I can see the packets.

@radcortez
Copy link
Member

Ok, I'll have another look.

@loicmathieu
Copy link
Contributor Author

Thanks

@radcortez
Copy link
Member

We were able to determine that this is caused by writing the probabilistic value in format, that the Jaeger configuration would not read property due to the default Locale.

@quarkus-bot quarkus-bot bot added this to the 2.7 - main milestone Jan 11, 2022
@gsmet gsmet modified the milestones: 2.7 - main, 2.6.3.Final Jan 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/jaeger area/smallrye area/tracing env/windows Impacts Windows machines kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants