Skip to content

Commit b22da9e

Browse files
authored
Jaeger: rolling config update (#1020)
Ensure jaeger is deployed automatically on config content changes Related Issue(s): * #984
1 parent 40d246e commit b22da9e

File tree

4 files changed

+13
-4
lines changed

4 files changed

+13
-4
lines changed

services/jaeger/.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
docker-compose.yml

services/jaeger/Makefile

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,9 @@ up-local: up
4141

4242
# Helpers -------------------------------------------------
4343

44+
docker-compose.yml: docker-compose.yml.j2 .venv .env
45+
@$(call jinja, $<, .env, $@)
46+
4447
.PHONY: ${TEMP_COMPOSE}
4548
${TEMP_COMPOSE}: docker-compose.yml
4649
@${REPO_BASE_DIR}/scripts/docker-stack-config.bash $< > $@

services/jaeger/docker-compose.yml renamed to services/jaeger/docker-compose.yml.j2

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
version: "3.7"
2-
configs:
3-
opentelemetry-collector-config:
4-
file: ./opentelemetry-collector-config.yaml
2+
53
services:
64
jaeger:
75
image: jaegertracing/all-in-one:1.60.0
@@ -39,7 +37,7 @@ services:
3937
configs:
4038
- source: opentelemetry-collector-config
4139
target: /etc/otel/config.yaml
42-
hostname: "{{.Node.Hostname}}-{{.Task.Slot}}"
40+
hostname: "{% raw %}{{.Node.Hostname}}-{{.Task.Slot}}{% endraw %}"
4341
command:
4442
- "--config=/etc/otel/config.yaml"
4543
deploy:
@@ -57,6 +55,12 @@ services:
5755
TRACING_OPENTELEMETRY_COLLECTOR_SAMPLING_PERCENTAGE: ${TRACING_OPENTELEMETRY_COLLECTOR_SAMPLING_PERCENTAGE}
5856
TRACING_OPENTELEMETRY_COLLECTOR_EXPORTER_ENDPOINT: ${TRACING_OPENTELEMETRY_COLLECTOR_EXPORTER_ENDPOINT}
5957
TRACING_OPENTELEMETRY_COLLECTOR_SERVICE_TELEMETRY_LOG_LEVEL: ${TRACING_OPENTELEMETRY_COLLECTOR_SERVICE_TELEMETRY_LOG_LEVEL}
58+
59+
configs:
60+
opentelemetry-collector-config:
61+
name: ${STACK_NAME}_opentelemetry-collector-config_{{ "./opentelemetry-collector-config.yaml" | sha256file | substring(0,10) }}
62+
file: ./opentelemetry-collector-config.yaml
63+
6064
networks:
6165
public:
6266
external: true

services/jaeger/template.env

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
STACK_NAME=${STACK_NAME}
12
MONITORING_DOMAIN=${MONITORING_DOMAIN}
23
PUBLIC_NETWORK=${PUBLIC_NETWORK}
34
MONITORED_NETWORK=${MONITORED_NETWORK}

0 commit comments

Comments
 (0)