Skip to content
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

Some questions about trace/sampler/jaeger_remote #954

Open
yurishkuro opened this issue Jan 24, 2023 · 5 comments
Open

Some questions about trace/sampler/jaeger_remote #954

yurishkuro opened this issue Jan 24, 2023 · 5 comments
Assignees
Labels
A-trace Area: issues related to tracing documentation/examples Improvements or additions to documentation or examples help wanted Good for taking. Extra help will be provided by maintainers/approvers

Comments

@yurishkuro
Copy link
Member

yurishkuro commented Jan 24, 2023

cc @TommyCpp

@yurishkuro yurishkuro changed the title Some questions about opentelemetry-sdk/src/trace/sampler/jaeger_remote Some questions about trace/sampler/jaeger_remote Jan 24, 2023
@TommyCpp TommyCpp self-assigned this Jan 24, 2023
@TommyCpp
Copy link
Contributor

Sorry spent this weekend to chase some memory issue but I will try to write some more doc tomorrow.

Which endpoint the SDK is expected to query for sampling strategy?

The default endpoint is /sampling

Which servers was the SDK tested against? For comparison, Go SDK uses deprecated format not supported by OTel Collector

IIRC I was using jaeger all in one docker image as the backend. We have unit test to make sure we can deserialize the response from jaeger and an example for jaeger remote sampler. But need to add an integration test.

@Narenderbhcu
Copy link

Hi, Is there any update on this ?

@TommyCpp TommyCpp added documentation/examples Improvements or additions to documentation or examples A-trace Area: issues related to tracing labels Feb 21, 2023
@TommyCpp
Copy link
Contributor

Please find the example in https://github.com/open-telemetry/opentelemetry-rust/tree/main/examples/jaeger-remote-sampler.

It contains a docker-compose file where we point the rust remote sampler to the jaeger container which loaded the strategies files in the example.

We don't support env vars in jaeger remote sampler yet.

As for the configuration we currently use a builder pattern where the user provides the required parameters to start the builder and can provide optional parameters using functions. We provide documents including the default value for all builder methods.

@TommyCpp
Copy link
Contributor

Typically, we do not include READMEs for features that are part of the SDK. However, I can include a configuration list in the example's README if it would be helpful.

@yurishkuro
Copy link
Member Author

Typically, we do not include READMEs for features that are part of the SDK.

why not? I think a readme is the actual module is a lot more discoverable than a readme tucked away in examples/. Unless you have some other form of documentation (which I would still keep close to the module code).

@hdost hdost added this to the Tracing API And SDK Stable milestone Mar 2, 2023
@hdost hdost removed this from the Tracing API Stable milestone Oct 17, 2023
@hdost hdost added the help wanted Good for taking. Extra help will be provided by maintainers/approvers label Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-trace Area: issues related to tracing documentation/examples Improvements or additions to documentation or examples help wanted Good for taking. Extra help will be provided by maintainers/approvers
Projects
None yet
Development

No branches or pull requests

4 participants