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

Tracing: support jaeger-agent tracer #15429

Open
cocotyty opened this issue Mar 11, 2021 · 3 comments
Open

Tracing: support jaeger-agent tracer #15429

cocotyty opened this issue Mar 11, 2021 · 3 comments
Labels
area/tracing enhancement Feature requests. Not bugs or questions. help wanted Needs help!

Comments

@cocotyty
Copy link

cocotyty commented Mar 11, 2021

Title: One line description
When we used envoy and jaeger together, we found that envoy does not support sending data directly to jaeger-agent.
Description:

Describe the the desired behavior, what scenario it enables and how it
would be used.

[optional Relevant Links:]

Any extra documentation required to understand the issue.
https://github.com/jaegertracing/jaeger-client-cpp
https://www.jaegertracing.io/docs/1.22/architecture/

@cocotyty cocotyty added enhancement Feature requests. Not bugs or questions. triage Issue requires triage labels Mar 11, 2021
@dio dio added area/tracing help wanted Needs help! and removed triage Issue requires triage labels Mar 12, 2021
@suniltheta
Copy link
Contributor

suniltheta commented Dec 30, 2021

Could this issue be same as #16401 which is closed due to staleness. May be we should revisit this problem.

I also notice that if we are using latest image from https://hub.docker.com/r/jaegertracing/all-in-one then the zipkin traces sent to this jaeger collector doesn't show up on jaeger UI.

The last working version seem to be jaegertracing/all-in-one:1.21

Checking if jaegertracing/jaeger-operator#1441 is the root cause of the problem.

cc: @darox

@suniltheta
Copy link
Contributor

So setting COLLECTOR_ZIPKIN_HOST_PORT instead of COLLECTOR_ZIPKIN_HTTP_PORT fixed the issue for me.
A deprecated variable COLLECTOR_ZIPKIN_HTTP_PORT was removed from v1.22
ref: https://github.com/jaegertracing/jaeger/releases/tag/v1.22.0

@suniltheta
Copy link
Contributor

Hi @cocotyty is this still an issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tracing enhancement Feature requests. Not bugs or questions. help wanted Needs help!
Projects
None yet
Development

No branches or pull requests

3 participants