Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Fixing tracetest configuration #65

Merged
merged 3 commits into from
Sep 19, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
83 changes: 65 additions & 18 deletions docker-compose.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Copyright The OpenTelemetry Authors
# SPDX-License-Identifier: Apache-2.0

version: '3.9'
version: "3.9"
x-default-logging: &logging
driver: "json-file"
options:
Expand Down Expand Up @@ -176,7 +176,7 @@
environment:
- CURRENCY_SERVICE_PORT
- OTEL_EXPORTER_OTLP_ENDPOINT
- OTEL_RESOURCE_ATTRIBUTES=${OTEL_RESOURCE_ATTRIBUTES},service.name=currencyservice # The C++ SDK does not support OTEL_SERVICE_NAME
- OTEL_RESOURCE_ATTRIBUTES=${OTEL_RESOURCE_ATTRIBUTES},service.name=currencyservice # The C++ SDK does not support OTEL_SERVICE_NAME

Check warning on line 179 in docker-compose.yml

View workflow job for this annotation

GitHub Actions / yamllint

179:91 [comments] too few spaces before comment
depends_on:
otelcol:
condition: service_started
Expand Down Expand Up @@ -223,8 +223,8 @@
memory: 175M
restart: unless-stopped
ports:
- "${FEATURE_FLAG_SERVICE_PORT}:${FEATURE_FLAG_SERVICE_PORT}" # Feature Flag Service UI
- "${FEATURE_FLAG_GRPC_SERVICE_PORT}" # Feature Flag Service gRPC API
- "${FEATURE_FLAG_SERVICE_PORT}:${FEATURE_FLAG_SERVICE_PORT}" # Feature Flag Service UI

Check warning on line 226 in docker-compose.yml

View workflow job for this annotation

GitHub Actions / yamllint

226:69 [comments] too few spaces before comment
- "${FEATURE_FLAG_GRPC_SERVICE_PORT}" # Feature Flag Service gRPC API

Check warning on line 227 in docker-compose.yml

View workflow job for this annotation

GitHub Actions / yamllint

227:45 [comments] too few spaces before comment
environment:
- FEATURE_FLAG_SERVICE_PORT
- FEATURE_FLAG_GRPC_SERVICE_PORT
Expand All @@ -233,7 +233,15 @@
- OTEL_SERVICE_NAME=featureflagservice
- DATABASE_URL=ecto://ffs:ffs@ffs_postgres:5432/ffs
healthcheck:
test: ["CMD", "curl", "-H", "baggage: synthetic_request=true", "-f", "http://localhost:${FEATURE_FLAG_SERVICE_PORT}"]
test:
[
"CMD",
"curl",
"-H",
"baggage: synthetic_request=true",
"-f",
"http://localhost:${FEATURE_FLAG_SERVICE_PORT}",
]
depends_on:
ffs_postgres:
condition: service_healthy
Expand Down Expand Up @@ -490,7 +498,7 @@
deploy:
resources:
limits:
memory: 500M # This is high to enable supporting the recommendationCache feature flag use case
memory: 500M # This is high to enable supporting the recommendationCache feature flag use case

Check warning on line 501 in docker-compose.yml

View workflow job for this annotation

GitHub Actions / yamllint

501:24 [comments] too few spaces before comment
restart: unless-stopped
ports:
- "${RECOMMENDATION_SERVICE_PORT}"
Expand Down Expand Up @@ -640,7 +648,6 @@
- "${REDIS_PORT}"
logging: *logging


# ********************
# Telemetry Components
# ********************
Expand Down Expand Up @@ -671,8 +678,8 @@
memory: 300M
restart: unless-stopped
ports:
- "${JAEGER_SERVICE_PORT}" # Jaeger UI
- "4317" # OTLP gRPC default port
- "${JAEGER_SERVICE_PORT}" # Jaeger UI

Check warning on line 681 in docker-compose.yml

View workflow job for this annotation

GitHub Actions / yamllint

681:34 [comments] too few spaces before comment
- "4317" # OTLP gRPC default port

Check warning on line 682 in docker-compose.yml

View workflow job for this annotation

GitHub Actions / yamllint

682:16 [comments] too few spaces before comment
- "14269:14269"
- "14268:14268"
- "14267:14267"
Expand All @@ -692,6 +699,33 @@

logging: *logging

jaeger-query:
image: jaegertracing/jaeger-query:latest
container_name: jaeger-query
command:
- "--metrics-backend=prometheus"
- "--es.server-urls=https://opensearch-node1:9200"
- "--es.tls.enabled=true"
deploy:
resources:
limits:
memory: 300M
restart: unless-stopped
ports:
- 16685
environment:
- COLLECTOR_OTLP_ENABLED=true
- METRICS_STORAGE_TYPE=prometheus
- SPAN_STORAGE_TYPE=opensearch
- ES_TAGS_AS_FIELDS_ALL=true
- ES_USERNAME=admin
- ES_PASSWORD=admin
- ES_TLS_SKIP_HOST_VERIFY=true
depends_on:
- opensearch-node1
- opensearch-node2
logging: *logging

jaeger-agent:
image: jaegertracing/jaeger-agent:latest
container_name: jaeger-agent
Expand Down Expand Up @@ -733,19 +767,25 @@
limits:
memory: 125M
restart: unless-stopped
command: [ "--config=/etc/otelcol-config.yml", "--config=/etc/otelcol-observability.yml", "--config=/etc/otelcol-config-extras.yml" ]
command:
[
"--config=/etc/otelcol-config.yml",
"--config=/etc/otelcol-observability.yml",
"--config=/etc/otelcol-config-extras.yml",
]
volumes:
- ./src/otelcollector/otelcol-config.yml:/etc/otelcol-config.yml
- ./src/otelcollector/otelcol-observability.yml:/etc/otelcol-observability.yml
- ./src/otelcollector/otelcol-config-extras.yml:/etc/otelcol-config-extras.yml
ports:
- "4317" # OTLP over gRPC receiver
- "4318:4318" # OTLP over HTTP receiver
- "13133:13133" # health check port
- "9464" # Prometheus exporter
- "8888" # metrics endpoint
- "4317" # OTLP over gRPC receiver

Check warning on line 781 in docker-compose.yml

View workflow job for this annotation

GitHub Actions / yamllint

781:16 [comments] too few spaces before comment
- "4318:4318" # OTLP over HTTP receiver

Check warning on line 782 in docker-compose.yml

View workflow job for this annotation

GitHub Actions / yamllint

782:21 [comments] too few spaces before comment
- "13133:13133" # health check port

Check warning on line 783 in docker-compose.yml

View workflow job for this annotation

GitHub Actions / yamllint

783:23 [comments] too few spaces before comment
- "9464" # Prometheus exporter

Check warning on line 784 in docker-compose.yml

View workflow job for this annotation

GitHub Actions / yamllint

784:16 [comments] too few spaces before comment
- "8888" # metrics endpoint
depends_on:
- jaeger-agent
- jaeger-query
- data-prepper
logging: *logging

Expand Down Expand Up @@ -793,7 +833,14 @@
volumes:
- opensearch-data1:/usr/share/opensearch/data # Creates volume called opensearch-data1 and mounts it to the container
healthcheck:
test: ["CMD", "curl", "-f", "https://opensearch-node1:9200/_cluster/health?wait_for_status=yellow", "-ku admin:admin"]
test:
[
"CMD",
"curl",
"-f",
"https://opensearch-node1:9200/_cluster/health?wait_for_status=yellow",
"-ku admin:admin",
]
interval: 5s
timeout: 25s
retries: 4
Expand Down Expand Up @@ -837,7 +884,7 @@
- opensearch-node2
- prometheus

# Observability OSD Integrations
# Observability OSD Integrations
integrations:
container_name: opensearch-integrations
build:
Expand Down Expand Up @@ -976,7 +1023,7 @@
otelcol:
condition: service_started
healthcheck:
test: [ "CMD", "wget", "--spider", "localhost:11633" ]
test: ["CMD", "wget", "--spider", "localhost:11633"]
interval: 1s
timeout: 3s
retries: 60
Expand Down
2 changes: 1 addition & 1 deletion test/tracetesting/tracetest-provision.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,6 @@ spec:
name: Jaeger
type: jaeger
jaeger:
endpoint: jaeger:16685
endpoint: jaeger-query:16685
tls:
insecure: true
Loading