Skip to content

Commit

Permalink
Fix broken v2 config (#5747)
Browse files Browse the repository at this point in the history
Unfortunately, we don't seem to have CI validation for this config and
it was broken since the recent config refactoring.

---------

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Yuri Shkuro <github@ysh.us>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Yuri Shkuro <github@ysh.us>
  • Loading branch information
dependabot[bot] and yurishkuro authored Jul 15, 2024
1 parent 2151acf commit 8d2aab6
Showing 1 changed file with 11 additions and 9 deletions.
20 changes: 11 additions & 9 deletions cmd/jaeger/config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,23 +14,25 @@ extensions:
# endpoint: 0.0.0.0:55679

jaeger_query:
trace_storage: memstore
trace_storage_archive: memstore_archive
trace_storage: some_store
trace_storage_archive: another_store
ui_config: ./cmd/jaeger/config-ui.json

jaeger_storage:
memory:
memstore:
max_traces: 100000
memstore_archive:
max_traces: 100000
backends:
some_store:
memory:
max_traces: 100000
another_store:
memory:
max_traces: 100000

remote_sampling:
# You can either use file or adaptive sampling strategy in remote_sampling
# file:
# path: ./cmd/jaeger/sampling-strategies.json
adaptive:
sampling_store: memstore
sampling_store: some_store
initial_sampling_probability: 0.1
http:
grpc:
Expand Down Expand Up @@ -58,4 +60,4 @@ processors:

exporters:
jaeger_storage_exporter:
trace_storage: memstore
trace_storage: some_store

0 comments on commit 8d2aab6

Please sign in to comment.