Skip to content

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
Signed-off-by: zzzk1 <[email protected]>
  • Loading branch information
zzzk1 committed Jan 13, 2025
1 parent 80c02ab commit 379e71e
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 9 deletions.
10 changes: 5 additions & 5 deletions docker-compose/monitor/Makefile
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Copyright (c) 2024 The Jaeger Authors.
# SPDX-License-Identifier: Apache-2.0

BINARY ?= jaeger # Default value uses v1 binary
BINARY ?= jaeger # Default value uses v2 binary

.PHONY: build
build: clean-jaeger
Expand All @@ -22,10 +22,10 @@ dev: export JAEGER_IMAGE_TAG = dev
dev:
docker compose up $(DOCKER_COMPOSE_ARGS)

.PHONY: dev-v2
dev-v2: export JAEGER_IMAGE_TAG = dev
dev-v2: export BINARY = all-in-one
dev-v2: build
.PHONY: dev-v1
dev-v1: export JAEGER_IMAGE_TAG = dev
dev-v1: export BINARY = all-in-one
dev-v1: build
docker compose -f docker-compose-v1.yml up $(DOCKER_COMPOSE_ARGS)

.PHONY: clean-jaeger
Expand Down
2 changes: 1 addition & 1 deletion examples/hotrod/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ As of Jaeger v1.42.0 this application was upgraded to use the OpenTelemetry SDK

### Run everything via `docker compose`

* Download `docker-compose-v2.yml` from https://github.com/jaegertracing/jaeger/blob/main/examples/hotrod/docker-compose-v2.yml
* Download `docker-compose.yml` from https://github.com/jaegertracing/jaeger/blob/main/examples/hotrod/docker-compose.yml
* Optional: find the latest Jaeger version (see https://www.jaegertracing.io/download/) and pass it via environment variable `JAEGER_VERSION`. Otherwise `docker compose` will use the `latest` tag, which is fine for the first time you download the images, but once they are in your local registry the `latest` tag is never updated and you may be running stale (and possibly incompatible) verions of Jaeger and the HotROD app.
* Run Jaeger backend and HotROD demo, e.g.:
* `JAEGER_VERSION=1.52 docker compose -f path-to-yml-file up`
Expand Down
File renamed without changes.
4 changes: 2 additions & 2 deletions scripts/build/build-hotrod-image.sh
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ print_help() {
exit 1
}

docker_compose_file="./examples/hotrod/docker-compose-v2.yml"
docker_compose_file="./examples/hotrod/docker-compose.yml"
platforms="$(make echo-linux-platforms)"
current_platform="$(go env GOOS)/$(go env GOARCH)"
jaeger_version="v2"
Expand Down Expand Up @@ -58,7 +58,7 @@ case "$jaeger_version" in
binary="all-in-one"
;;
v2)
docker_compose_file="./examples/hotrod/docker-compose-v2.yml"
docker_compose_file="./examples/hotrod/docker-compose.yml"
binary="jaeger"
;;
*)
Expand Down
2 changes: 1 addition & 1 deletion scripts/e2e/spm.sh
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ main() {
if [ "$BINARY" == "jaeger" ]; then
(cd docker-compose/monitor && make build BINARY="$BINARY" && make dev DOCKER_COMPOSE_ARGS="-d")
else
(cd docker-compose/monitor && make build BINARY="$BINARY" && make dev-v2 DOCKER_COMPOSE_ARGS="-d")
(cd docker-compose/monitor && make build BINARY="$BINARY" && make dev-v1 DOCKER_COMPOSE_ARGS="-d")
fi
wait_for_services
check_spm
Expand Down

0 comments on commit 379e71e

Please sign in to comment.