dlmarion commented on pull request #2259:
URL: https://github.com/apache/accumulo/pull/2259#issuecomment-912790866
Commit d873448 includes changes from testing locally. It includes:
1. bug fixes and fixes for build issues
2. Removing tracer from scripts
3. Addition of dependencies to export traces to Jaeger*
*It's intended that these changes will be backed out as I had to include all
of the transitive dependencies in the poms and component.xml for everything to
run. There does appear to be dependency convergence issues between our jars and
the dependencies of OpenTelemetry. With this commit I had to build the code
with the `-Denforcer.skip` property to get past the dependency convergence
checks.
With this commit I was able to build the binary tarball using `mvn clean
package -DskipTests -Denforcer.skip` and visualize traces in
[Jaeger](https://www.jaegertracing.io/) by doing the following:
1. Use the Jaeger
[All-In-One](https://www.jaegertracing.io/docs/1.25/getting-started/#all-in-one)
Docker image to start Jaeger.
```
docker run -d --name jaeger -e COLLECTOR_ZIPKIN_HOST_PORT=:9411 -p
5775:5775/udp -p 6831:6831/udp -p 6832:6832/udp -p 5778:5778 -p 16686:16686 -p
14268:14268 -p 14250:14250 -p 9411:9411 jaegertracing/all-in-one:1.25
```
2. Configure the OpenTracing implementation to export to Jaeger by setting
the following environment variable
```
export OTEL_TRACES_EXPORTER=jaeger
```
3. Then, in the same terminal where step 2 was performed, start Accumulo.
I'm thinking that we might want to move the tracer module into it's own
repository.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
