Skip to content
This repository was archived by the owner on Jul 11, 2022. It is now read-only.

Commit cb14ed8

Browse files
pavolloffayyurishkuro
authored andcommitted
Use shared jaeger-compose in xdock (#91)
1 parent 3922f98 commit cb14ed8

File tree

4 files changed

+29
-22
lines changed

4 files changed

+29
-22
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,3 +56,4 @@ perf.svg
5656
*.swp
5757
.idea/
5858

59+
crossdock/jaeger-docker-compose.yml

crossdock/docker-compose.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,8 @@ services:
4040
image: jaegertracing/xdock-java
4141
ports:
4242
- "8080-8082"
43+
links:
44+
- "jaeger-agent:test_driver"
4345

4446
python:
4547
build:
@@ -48,14 +50,11 @@ services:
4850
ports:
4951
- "8080-8082"
5052

51-
cassandra:
52-
image: "cassandra:3.9"
53-
5453
test_driver:
5554
image: jaegertracing/test-driver
56-
links:
57-
- cassandra
5855
depends_on:
59-
- cassandra
56+
- jaeger-query
57+
- jaeger-collector
58+
- jaeger-agent
6059
ports:
6160
- "8080"

crossdock/rules.mk

Lines changed: 21 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
XDOCK_YAML=crossdock/docker-compose.yml
22
TRACETEST_THRIFT=idl/thrift/crossdock/tracetest.thrift
3+
JAEGER_COMPOSE_URL=https://raw.githubusercontent.com/jaegertracing/jaeger/master/docker-compose/jaeger-docker-compose.yml
4+
XDOCK_JAEGER_YAML=crossdock/jaeger-docker-compose.yml
35

46
.PHONY: clean-compile
57
clean-compile:
@@ -9,27 +11,27 @@ ${TRACETEST_THRIFT}:
911
git submodule update --init idl
1012

1113
.PHONY: docker
12-
docker: clean-compile
14+
docker: clean-compile crossdock-download-jaeger
1315
docker build -f crossdock/Dockerfile -t jaeger-client-python .
1416

1517
.PHONY: crossdock
16-
crossdock: ${TRACETEST_THRIFT}
17-
docker-compose -f $(XDOCK_YAML) kill python
18-
docker-compose -f $(XDOCK_YAML) rm -f python
19-
docker-compose -f $(XDOCK_YAML) build python
20-
docker-compose -f $(XDOCK_YAML) run crossdock
18+
crossdock: ${TRACETEST_THRIFT} crossdock-download-jaeger
19+
docker-compose -f $(XDOCK_YAML) -f $(XDOCK_JAEGER_YAML) kill python
20+
docker-compose -f $(XDOCK_YAML) -f $(XDOCK_JAEGER_YAML) rm -f python
21+
docker-compose -f $(XDOCK_YAML) -f $(XDOCK_JAEGER_YAML) build python
22+
docker-compose -f $(XDOCK_YAML) -f $(XDOCK_JAEGER_YAML) run crossdock
2123

2224
.PHONY: crossdock-fresh
23-
crossdock-fresh: ${TRACETEST_THRIFT}
24-
docker-compose -f $(XDOCK_YAML) kill
25-
docker-compose -f $(XDOCK_YAML) rm --force
26-
docker-compose -f $(XDOCK_YAML) pull
27-
docker-compose -f $(XDOCK_YAML) build
28-
docker-compose -f $(XDOCK_YAML) run crossdock
29-
30-
.PHONY: crossdock-logs
25+
crossdock-fresh: ${TRACETEST_THRIFT} crossdock-download-jaeger
26+
docker-compose -f $(XDOCK_YAML) -f $(XDOCK_JAEGER_YAML) kill
27+
docker-compose -f $(XDOCK_YAML) -f $(XDOCK_JAEGER_YAML) rm --force
28+
docker-compose -f $(XDOCK_YAML) -f $(XDOCK_JAEGER_YAML) pull
29+
docker-compose -f $(XDOCK_YAML) -f $(XDOCK_JAEGER_YAML) build
30+
docker-compose -f $(XDOCK_YAML) -f $(XDOCK_JAEGER_YAML) run crossdock
31+
32+
.PHONY: crossdock-logs crossdock-download-jaeger
3133
crossdock-logs:
32-
docker-compose -f $(XDOCK_YAML) logs
34+
docker-compose -f $(XDOCK_YAML) -f $(XDOCK_JAEGER_YAML) logs
3335

3436
.PHONY: install_docker_ci
3537
install_docker_ci:
@@ -39,3 +41,7 @@ install_docker_ci:
3941
chmod +x docker-compose
4042
sudo mv docker-compose /usr/local/bin
4143
docker-compose version
44+
45+
.PHONY: crossdock-download-jaeger
46+
crossdock-download-jaeger:
47+
curl -o $(XDOCK_JAEGER_YAML) $(JAEGER_COMPOSE_URL)

crossdock/server/endtoend.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414

1515
import tornado.web
1616
import json
17+
import os
1718

1819
from jaeger_client.local_agent_net import LocalAgentSender
1920
from jaeger_client.config import (
@@ -101,7 +102,7 @@ def tracers(self, tracers):
101102
@property
102103
def local_agent_sender(self):
103104
return LocalAgentSender(
104-
host='test_driver',
105+
host=os.getenv('AGENT_HOST', 'jaeger-agent'),
105106
sampling_port=DEFAULT_SAMPLING_PORT,
106107
reporting_port=DEFAULT_REPORTING_PORT,
107108
)

0 commit comments

Comments
 (0)