-
Couldn't load subscription status.
- Fork 778
feat Jaeger v2 #4921
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
feat Jaeger v2 #4921
Conversation
Signed-off-by: MyMirelHub <[email protected]>
Signed-off-by: MyMirelHub <[email protected]>
|
/LGTM please approve as the current docs are broken |
...ontent/en/operations/observability/tracing/otel-collector/open-telemetry-collector-jaeger.md
Outdated
Show resolved
Hide resolved
Signed-off-by: MyMirelHub <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are more coming from me for changes needed here. Please create a new PR
| ```bash | ||
| docker run -d --name jaeger \ | ||
| -p 4317:4317 \ | ||
| docker run --rm --name jaeger \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest two updates here.
-
Add that
-p 9411:9411clashes with zipkin that may already be running and you should stop zipkin first with adocker stopcommand -
Add the
-dcommand so this runs detached in the background. Like this
docker run -d --rm --name jaeger \
-p 16686:16686 \
-p 4317:4317 \
-p 4318:4318 \
-p 5778:5778 \
-p 9411:9411 \
cr.jaegertracing.io/jaegertracing/jaeger:2.11.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll make a new PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:
In addition, please fill out the following to help reviewers understand this pull request:
Description
Updated the Jaeger tracing documentation to focus on Jaeger V2 with native OTLP support, using the OpenTelemetry Operator and in-memory storage, with a note on persistent storage backends. The changes replace outdated Jaeger V1 content
Issue reference
#4920