open-telemetry / opentelemetry-rust-contrib

OpenTelemetry Contrib Packages for Rust
Apache License 2.0
35 stars 34 forks source link

Update to otel-rust v0.23 #69

Closed lalitb closed 4 months ago

lalitb commented 4 months ago

Fixes # Design discussion issue (if applicable) #

Changes

Please provide a brief description of the changes here.

Merge requirement checklist

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 16.66667% with 10 lines in your changes are missing coverage. Please review.

Project coverage is 50.7%. Comparing base (56a6f45) to head (4a01b5a). Report is 3 commits behind head on main.

Files Patch % Lines
...elemetry-contrib/src/trace/exporter/jaeger_json.rs 0.0% 5 Missing :warning:
opentelemetry-etw-logs/src/logs/exporter.rs 0.0% 2 Missing :warning:
...pentelemetry-user-events-logs/src/logs/exporter.rs 0.0% 2 Missing :warning:
...ser-events-logs/src/logs/reentrant_logprocessor.rs 0.0% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #69 +/- ## ======================================= - Coverage 52.3% 50.7% -1.6% ======================================= Files 38 20 -18 Lines 4967 1937 -3030 ======================================= - Hits 2598 983 -1615 + Misses 2369 954 -1415 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

SimenB commented 4 months ago

Will this be published? 🙏

hdost commented 4 months ago

@lalitb do you want to take care of it or do you want me to?

lalitb commented 4 months ago

@hdost I and @cijothomas will handle the opentelemetry-user-events-* and opentelemetry-etw-* crates. If you could help with rest, that would be great. Otherwise, I can try to find some time next week to work on them. I’m not very familiar with how to validate the dynatrace, stackdriver, and datadog crates before publishing, so will need to look into that as well.