Skip to content

Commit

Permalink
change error to reason
Browse files Browse the repository at this point in the history
  • Loading branch information
lalitb committed Oct 28, 2024
1 parent c6e358d commit 08ef266
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions opentelemetry-sdk/src/trace/sampler/jaeger_remote/sampler.rs
Original file line number Diff line number Diff line change
Expand Up @@ -206,8 +206,8 @@ impl JaegerRemoteSampler {
Err(err_msg) => {
otel_debug!(

Check warning on line 207 in opentelemetry-sdk/src/trace/sampler/jaeger_remote/sampler.rs

View check run for this annotation

Codecov / codecov/patch

opentelemetry-sdk/src/trace/sampler/jaeger_remote/sampler.rs#L206-L207

Added lines #L206 - L207 were not covered by tests
name: "JaegerRemoteSampler.UpdateStrategy.RequestFailed",
reason = "Failed to fetch new sampling strategy",
error = format!("{}", err_msg),
message = "Failed to fetch new sampling strategy",
reason = format!("{}", err_msg),

Check warning on line 210 in opentelemetry-sdk/src/trace/sampler/jaeger_remote/sampler.rs

View check run for this annotation

Codecov / codecov/patch

opentelemetry-sdk/src/trace/sampler/jaeger_remote/sampler.rs#L210

Added line #L210 was not covered by tests
);
}
};
Expand Down

0 comments on commit 08ef266

Please sign in to comment.