Skip to content

feat: support opensearch as a tracing backend #1087

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.

Sign up for GitHub

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

Merged
merged 11 commits into from
Aug 19, 2022
Merged

feat: support opensearch as a tracing backend #1087

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
f6744b7
get trace from opensearch
mathnogueira Aug 17, 2022
bfb5fab
make tracedb return tracetest's trace object instead of otel's format
mathnogueira Aug 17, 2022
866f38a
convert opensearch format into tracetest trace format
mathnogueira Aug 17, 2022
5cb0768
opensearch local setup
mathnogueira Aug 17, 2022
8945b7b
add opensearch documentation
mathnogueira Aug 17, 2022
e049c15
set retry delay to 2s
mathnogueira Aug 17, 2022
b271ef2
update minimal number of spans when pooling based on config
mathnogueira Aug 17, 2022
3eed1ee
use config.ApplicationExporter instead of checking for configuration
mathnogueira Aug 17, 2022
54679cf
remove network options from docker-compose
mathnogueira Aug 19, 2022
931cab6
rename NewTrace to New and make it receive an array
mathnogueira Aug 19, 2022
ff420ca
rename getSpanDuration to spanDuration
mathnogueira Aug 19, 2022
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
Prev Previous commit
Next Next commit
remove network options from docker-compose
  • Loading branch information
@mathnogueira
mathnogueira committed Aug 19, 2022
commit 54679cfbe08f9c8340ea2b2b549384333c08a62d
27 changes: 0 additions & 27 deletions docker-compose.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,6 @@ services:
condition: service_healthy
otel-collector:
condition: service_started
networks:
- local

postgres:
container_name: postgres
Expand All @@ -41,8 +39,6 @@ services:
interval: 1s
timeout: 5s
retries: 60
networks:
- local

otel-collector:
image: otel/opentelemetry-collector:0.54.0
Expand All @@ -61,8 +57,6 @@ services:
depends_on:
jaeger:
condition: service_healthy
networks:
- local


jaeger:
Expand All @@ -76,8 +70,6 @@ services:
interval: 1s
timeout: 3s
retries: 60
networks:
- local

cache:
image: redis:6
Expand All @@ -88,8 +80,6 @@ services:
interval: 1s
timeout: 3s
retries: 60
networks:
- local

queue:
image: rabbitmq:3.9
Expand All @@ -101,8 +91,6 @@ services:
interval: 1s
timeout: 5s
retries: 60
networks:
- local

demo-api:
image: kubeshop/demo-pokemon-api:latest
Expand Down Expand Up @@ -131,8 +119,6 @@ services:
condition: service_healthy
jaeger:
condition: service_healthy
networks:
- local

demo-worker:
image: kubeshop/demo-pokemon-api:latest
Expand All @@ -154,8 +140,6 @@ services:
condition: service_healthy
jaeger:
condition: service_healthy
networks:
- local

demo-rpc:
image: kubeshop/demo-pokemon-api:latest
Expand Down Expand Up @@ -184,8 +168,6 @@ services:
condition: service_healthy
jaeger:
condition: service_healthy
networks:
- local

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this really needed? it's usually easier to let defaults do their thing

data-prepper:
restart: unless-stopped
Expand All @@ -198,8 +180,6 @@ services:
- "21890:21890"
depends_on:
- "opensearch"
networks:
- local

opensearch:
image: opensearchproject/opensearch:latest
Expand All @@ -219,8 +199,6 @@ services:
ports:
- 9200:9200
- 9600:9600 # required for Performance Analyzer
networks:
- local

dashboards:
image: opensearchproject/opensearch-dashboards:latest
Expand All @@ -235,8 +213,3 @@ services:
OPENSEARCH_PASSWORD: admin
depends_on:
- opensearch
networks:
- local

networks:
local: