-
Notifications
You must be signed in to change notification settings - Fork 292
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
[Resources.OperatingSystem] Implement additional osdetector attributes #1983
[Resources.OperatingSystem] Implement additional osdetector attributes #1983
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1983 +/- ##
==========================================
+ Coverage 73.91% 80.00% +6.08%
==========================================
Files 267 3 -264
Lines 9615 130 -9485
==========================================
- Hits 7107 104 -7003
+ Misses 2508 26 -2482
Flags with carried forward coverage won't be shown. Click here to find out more.
|
The Docker image doesn’t include an OS build ID, which is causing test errors. We need to find a way to track or include build IDs to resolve this issue. |
b5cb64e
to
d0d2cb5
Compare
…lomchenko/opentelemetry-dotnet-contrib into add-additional-os-attributes
…lomchenko/opentelemetry-dotnet-contrib into add-additional-os-attributes
…ack in resource detector/remove non-existent build_id for Ubuntu
Fixes #1957
Changes
Implement additional Operating System Detector attributes
https://github.com/open-telemetry/semantic-conventions/blob/main/docs/resource/os.md
The data detected by our detector comes from the same sources as those used by the opentelemetry-go detector.
Merge requirement checklist
CHANGELOG.md
files updated for non-trivial changes