-
Notifications
You must be signed in to change notification settings - Fork 347
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
added all-in-one deployment and configmap for jaeger-v2 #606
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Mehul <[email protected]>
Signed-off-by: Mehul <[email protected]>
charts/jaeger-v2/Chart.yaml
Outdated
- opentracing | ||
- tracing | ||
- instrumentation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- opentracing | |
- tracing | |
- instrumentation | |
- opentelemetry | |
- tracing |
helm install jaeger-v2 ./ -f my-config-values.yaml | ||
``` | ||
|
||
This command installs the Jaeger chart and uses `my-config-values.yaml` for overriding any configuration specified in the default `values.yaml`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what does "overriding" mean? Does it completely ignore the default file, or performs some sort of merge?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's say you have specified
jaeger_storage:
backends:
some_store:
memory:
max_traces: 90000
another_store:
memory:
max_traces: 90000
but didn't specified
remote_sampling:
adaptive:
sampling_store: some_store
initial_sampling_probability: 0.1
so it will use default value of 0.1 from vaules.yaml
charts/jaeger-v2/values.yaml
Outdated
adaptive: | ||
sampling_store: some_store | ||
initial_sampling_probability: 0.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we want adaptive sampling to be the default.
Signed-off-by: Mehul <[email protected]>
Signed-off-by: Mehul <[email protected]>
Signed-off-by: Mehul <[email protected]>
Signed-off-by: Mehul <[email protected]>
Jaeger v2 test
What this PR does
Which issue this PR fixes
is a part of jaegertracing/jaeger#5767
Checklist
[jaeger]
or[jaeger-operator]
)