Skip to content

Commit 03813bd

Browse files
docs: specify the jaeger all-in-one docker image version (#374)
# Rationale for this change The latest Jaeger all-in-one Docker image, version `1.63.0`, has a bug. After running benchmarks you cannot view the output from the UI - jaegertracing/jaeger#6200. This PR updates the documents referencing the Jaeger all-in-one image version to point to the latest working version - `1.62.0`. # What changes are included in this PR? - Docs are updated to specify the latest working Jaeger all-in-one Docker image version - `1.62.0`. # Are these changes tested? Yes
2 parents 2b31823 + 51e1270 commit 03813bd

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

crates/proof-of-sql/benches/README.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ To run benchmarks with Jaeger, you need to do the following
66

77
1. Spin up Jaeger service on port 6831 to receive the benchmarks trace data, and provides Jaeger UI on port 16686.
88
```bash
9-
docker run --rm -d --name jaeger -p 6831:6831/udp -p 16686:16686 jaegertracing/all-in-one:latest
9+
docker run --rm -d --name jaeger -p 6831:6831/udp -p 16686:16686 jaegertracing/all-in-one:1.62.0
1010
```
1111
2. Run a benchmark.
1212
```bash

crates/proof-of-sql/benches/jaeger_benches.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
//! Benchmarking/Tracing using Jaeger.
22
//! To run, execute the following commands:
33
//! ```bash
4-
//! docker run --rm -d --name jaeger -p 6831:6831/udp -p 16686:16686 jaegertracing/all-in-one:latest
4+
//! docker run --rm -d --name jaeger -p 6831:6831/udp -p 16686:16686 jaegertracing/all-in-one:1.62.0
55
//! cargo bench -p proof-of-sql --bench jaeger_benches InnerProductProof
66
//! cargo bench -p proof-of-sql --bench jaeger_benches Dory
77
//! cargo bench -p proof-of-sql --bench jaeger_benches DynamicDory

0 commit comments

Comments
 (0)