Skip to content

Commit e760de7

Browse files
authored
jaeger: use ops node label constraint (#823)
1 parent 4f28351 commit e760de7

File tree

5 files changed

+9
-30
lines changed

5 files changed

+9
-30
lines changed

services/jaeger/Makefile

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -24,16 +24,14 @@ up-letsencrypt-dns: .init .env ${TEMP_COMPOSE}-letsencrypt-dns
2424
@docker stack deploy --with-registry-auth --prune --compose-file ${TEMP_COMPOSE}-letsencrypt-dns ${STACK_NAME}
2525

2626
.PHONY: up-dalco ## Deploys jaeger stack for Dalco Cluster
27-
up-dalco: .init .env ${TEMP_COMPOSE}-dalco
28-
@docker stack deploy --with-registry-auth --prune --compose-file ${TEMP_COMPOSE}-dalco ${STACK_NAME}
27+
up-dalco: up
2928

3029
.PHONY: up-aws ## Deploys jaeger stack for aws
3130
up-aws: .init .env ${TEMP_COMPOSE}-aws ## Deploys jaeger stack in aws
3231
@docker stack deploy --with-registry-auth --prune --compose-file ${TEMP_COMPOSE}-aws ${STACK_NAME}
3332

3433
.PHONY: up-master ## Deploys jaeger stack for master Cluster
35-
up-master: .init .env ${TEMP_COMPOSE}-master
36-
@docker stack deploy --with-registry-auth --prune --compose-file ${TEMP_COMPOSE}-master ${STACK_NAME}
34+
up-master: up
3735

3836
.PHONY: up-public ## Deploys jaeger stack for public acess deploy
3937
up-public: up-dalco
@@ -55,14 +53,6 @@ ${TEMP_COMPOSE}-letsencrypt-http: docker-compose.yml docker-compose.letsencrypt.
5553
${TEMP_COMPOSE}-letsencrypt-dns: docker-compose.yml docker-compose.letsencrypt.dns.yml
5654
@${REPO_BASE_DIR}/scripts/docker-stack-config.bash $< docker-compose.letsencrypt.dns.yml > $@
5755

58-
.PHONY: ${TEMP_COMPOSE}-dalco
59-
${TEMP_COMPOSE}-dalco: docker-compose.yml docker-compose.dalco.yml
60-
@${REPO_BASE_DIR}/scripts/docker-stack-config.bash $< docker-compose.dalco.yml > $@
61-
62-
.PHONY: ${TEMP_COMPOSE}-master
63-
${TEMP_COMPOSE}-master: docker-compose.yml docker-compose.master.yml
64-
@${REPO_BASE_DIR}/scripts/docker-stack-config.bash $< docker-compose.master.yml > $@
65-
6656
.PHONY: ${TEMP_COMPOSE}-aws
6757
${TEMP_COMPOSE}-aws: docker-compose.yml docker-compose.aws.yml
6858
@${REPO_BASE_DIR}/scripts/docker-stack-config.bash $< docker-compose.aws.yml > $@

services/jaeger/docker-compose.aws.yml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,3 @@ services:
33
jaeger:
44
dns: # Add this always for AWS, otherwise we get "No such image: " for docker services
55
8.8.8.8
6-
deploy:
7-
placement:
8-
constraints:
9-
- node.labels.jaeger==true

services/jaeger/docker-compose.dalco.yml

Lines changed: 0 additions & 7 deletions
This file was deleted.

services/jaeger/docker-compose.master.yml

Lines changed: 0 additions & 7 deletions
This file was deleted.

services/jaeger/docker-compose.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@ services:
1414
environment:
1515
MEMORY_MAX_TRACES: 10000
1616
deploy:
17+
placement:
18+
constraints:
19+
- node.labels.ops==true
1720
labels:
1821
- traefik.enable=true
1922
- traefik.docker.network=${PUBLIC_NETWORK}
@@ -39,6 +42,10 @@ services:
3942
hostname: "{{.Node.Hostname}}-{{.Task.Slot}}"
4043
command:
4144
- "--config=/etc/otel/config.yaml"
45+
deploy:
46+
placement:
47+
constraints:
48+
- node.labels.ops==true
4249
ports:
4350
- "4318:4318" # OTLP HTTP receiver
4451
networks:

0 commit comments

Comments
 (0)