-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Rust Getting Started Docs Error - the trait bound opentelemetry_stdout::SpanExporter: opentelemetry_sdk::export::trace::SpanExporter
is not satisfied
#5745
Comments
@open-telemetry/rust-approvers please take a look |
The versions are mismatched it seems. The current doc shows stdout newer version but keeps older version for api and sdk. Didn't realize that https://github.com/open-telemetry/opentelemetry.io/pull/5633/files affects the getting started docs, unfortunately:( @TownCube updating all crates to newest would be the right fix. |
can you point me to the lines in the documentation where the versions need to be kept in sync? |
https://opentelemetry.io/docs/languages/rust/getting-started/#instrumentation ^This. If all are on 0.27.0, things would compile. (Rust is preparing RC and 1.0 stable in a month or so, and we can come back to edit all docs here.) |
opentelemetry = "{{% version-from-registry otel-rust %}}"
opentelemetry_sdk = "{{% version-from-registry otel-rust-sdk %}}"
opentelemetry-stdout = { version = "{{% version-from-registry exporter-rust-stdout %}}", features = ["trace"] } they all pick the version from the registry, but currently they are not auto updated via this script: let me take a look if this is fixable via automation, such that they will be up-to-date as new releases are made |
@cijothomas since #5760 is not possible until the v1.0 release we would need to track updates manually. |
What needs to be changed?
Update Rust Getting Started example so it compiles.
What is the name + path of the page that needs changed? https://opentelemetry.io/docs/languages/rust/getting-started/
Additional context: The error I get after copy/pasting the getting started example:
Bumping the versions seems to resolve it but didn't want to blindly raise a PR to update without understanding the ramifications in case this break is a sign of something bigger not working correctly.
The text was updated successfully, but these errors were encountered: