Skip to content

Commit

Permalink
Add jaeger remote sampler to SDK configuration (#1791)
Browse files Browse the repository at this point in the history
* Add jaeger remote sampler to SDK configuration

Signed-off-by: Pavol Loffay <[email protected]>

* Use =

Signed-off-by: Pavol Loffay <[email protected]>

* add title back

Signed-off-by: Pavol Loffay <[email protected]>

Co-authored-by: Yuri Shkuro <[email protected]>
  • Loading branch information
pavolloffay and yurishkuro authored Jul 28, 2021
1 parent 83098f0 commit 30c3881
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,8 @@ release.

### Traces

- Adding SDK configuration for Jaeger remote sampler ([#1791](https://github.com/open-telemetry/opentelemetry-specification/pull/1791))

### Metrics

### Logs
Expand Down Expand Up @@ -42,6 +44,7 @@ release.
([#1738](https://github.com/open-telemetry/opentelemetry-specification/pull/1738))
- Clarify meaning of the Certificate File option.
([#1803](https://github.com/open-telemetry/opentelemetry-specification/pull/1803))
- Adding environment variables for event and link attribute limits. ([#1751](https://github.com/open-telemetry/opentelemetry-specification/pull/1751))

### Metrics

Expand Down
6 changes: 6 additions & 0 deletions specification/sdk-environment-variables.md
Original file line number Diff line number Diff line change
Expand Up @@ -56,10 +56,16 @@ Known values for `OTEL_TRACES_SAMPLER` are:
- `"parentbased_always_on"`: `ParentBased(root=AlwaysOnSampler)`
- `"parentbased_always_off"`: `ParentBased(root=AlwaysOffSampler)`
- `"parentbased_traceidratio"`: `ParentBased(root=TraceIdRatioBased)`
- `"jaeger_remote"`: `JaegerRemoteSampler`

Depending on the value of `OTEL_TRACES_SAMPLER`, `OTEL_TRACES_SAMPLER_ARG` may be set as follows:

- For `traceidratio` and `parentbased_traceidratio` samplers: Sampling probability, a number in the [0..1] range, e.g. "0.25". Default is 1.0 if unset.
- For `jaeger_remote`: The value is a comma separated list:
- `endpoint`: the endpoint in form of `host:port` of gRPC server that serves the sampling strategy for the service ([sampling.proto](https://github.com/jaegertracing/jaeger-idl/blob/master/proto/api_v2/sampling.proto)).
- `pollingIntervalMs`: in milliseconds indicating how often the sampler will poll the backend for updates to sampling strategy.
- `initialSamplingRate`: in the [0..1] range, which is used as the sampling probability when the backend cannot be reached to retrieve a sampling strategy. This value stops having an effect once a sampling strategy is retrieved successfully, as the remote strategy will be used until a new update is retrieved.
- Example: `endpoint=localhost:14250,pollingIntervalMs=5000,initialSamplingRate=0.25`

## Batch Span Processor

Expand Down

0 comments on commit 30c3881

Please sign in to comment.