Feature：add trace features #3451
Comments
|
This isnt on the roadmap at the moment but could be discussed if there is enough interest or even a PR that can kick off the conversation. |
|
@milosgajdos Do you mean to submit a PR first, and then discuss based on the PR? |
|
@milosgajdos I hope you can take the time to reply to this question, thanks. |
|
Can these failed problems be reproduced locally? How can I solve these problems? |
|
issue： #3475 |
|
Test Env: Otel: version: "2"
services:
# Jaeger
jaeger-all-in-one:
image: jaegertracing/all-in-one:latest
ports:
- "16686:16686"
- "14268"
- "14250"
# Collector
otel-collector:
image: ${OTELCOL_IMG}
command: ["--config=/etc/otel-collector-config.yaml", "${OTELCOL_ARGS}"]
volumes:
- ./otel-collector-config.yaml:/etc/otel-collector-config.yaml
ports:
- "1888:1888" # pprof extension
- "8888:8888" # Prometheus metrics exposed by the collector
- "8889:8889" # Prometheus exporter metrics
- "13133:13133" # health_check extension
- "53989:4317" # OTLP gRPC receiver
- "55670:55679" # zpages extension
depends_on:
- jaeger-all-in-onedistribution: // config
otel:
exporter_name: otlp
exporter_endpoint: demo_otel-collector_1:4317
// run command
$ docker run -p 5000:5000 -d --name registry registry
$ docker network connect demo_default registryEffect：
|
gotgelf
added a commit
to gotgelf/distribution
that referenced
this issue
Dec 7, 2023
This commit integrates the opentelemetry support functionality originally proposed in PR distribution#3701 by @lengrongfu into the current master branch. Due to the age and fragility of the original PR, a direct rebase proved challenging. Therefore, the changes have been moved and adjusted to align with the latest codebase. Key adjustments include: - Aligning the tracing code with the latest architectural changes in the main branch. - Resolving conflicts. All credit for the initial development of the tracing feature goes to @lengrongfu. This commit aims to build upon that foundation and bring this valuable feature into the current version of the project. Closes distribution#3451
gotgelf
added a commit
to gotgelf/distribution
that referenced
this issue
Dec 7, 2023
This commit integrates the opentelemetry support functionality originally proposed in PR distribution#3701 by @lengrongfu into the current master branch. Due to the age and fragility of the original PR, a direct rebase proved challenging. Therefore, the changes have been moved and adjusted to align with the latest codebase. Key adjustments include: - Aligning the tracing code with the latest architectural changes in the main branch. - Resolving conflicts. All credit for the initial development of the tracing feature goes to @lengrongfu. This commit aims to build upon that foundation and bring this valuable feature into the current version of the project. Closes distribution#3451 Signed-off-by: gotgelf <gotgelf@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I hope to observe my mirror distribution process through opentracing, so I hope to add the Trace function. Do you have this requirement? Can you introduce it?
The text was updated successfully, but these errors were encountered: