-
Notifications
You must be signed in to change notification settings - Fork 15
[crashtracking] Add sig_info and os_info as objects in errors payload #1350
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
[crashtracking] Add sig_info and os_info as objects in errors payload #1350
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
cd3b00b to
670f703
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1350 +/- ##
==========================================
+ Coverage 70.96% 71.15% +0.19%
==========================================
Files 390 390
Lines 62588 62570 -18
==========================================
+ Hits 44416 44523 +107
+ Misses 18172 18047 -125
🚀 New features to boost your workflow:
|
BenchmarksComparisonBenchmark execution time: 2025-11-18 22:05:48 Comparing candidate commit 2425179 in PR branch Found 0 performance improvements and 13 performance regressions! Performance is the same for 42 metrics, 2 unstable metrics. scenario:benching deserializing traces from msgpack to their internal representation
scenario:credit_card/is_card_number/ 378282246310005
scenario:credit_card/is_card_number/378282246310005
scenario:credit_card/is_card_number/37828224631000521389798
scenario:credit_card/is_card_number_no_luhn/ 378282246310005
scenario:credit_card/is_card_number_no_luhn/378282246310005
scenario:credit_card/is_card_number_no_luhn/37828224631000521389798
CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
Group 13
Group 14
Group 15
Group 16
Group 17
BaselineOmitted due to size. |
Artifact Size Benchmark Reportaarch64-alpine-linux-musl
aarch64-apple-darwin
aarch64-unknown-linux-gnu
libdatadog-x64-windows
libdatadog-x86-windows
x86_64-alpine-linux-musl
x86_64-apple-darwin
x86_64-unknown-linux-gnu
|
| experimental: crash_info.experimental.clone(), | ||
| }, | ||
| trace_id: None, | ||
| os_info: ::os_info::get().into(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ive seen telemetry crash reports with os info as the default unknown os_info. This makes sure we at least recalculate it (should be on same machine). I will investigate why original telemetry message sends unknown OS info sometimes...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
I saw this for some php crash report.
Maybe the crate needs an update.
| experimental: crash_info.experimental.clone(), | ||
| }, | ||
| trace_id: None, | ||
| os_info: ::os_info::get().into(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
I saw this for some php crash report.
Maybe the crate needs an update.
| @@ -1,9 +1,9 @@ | |||
| // Copyright 2024-Present Datadog, Inc. https://www.datadoghq.com/ | |||
| // SPDX-License-Identifier: Apache-2.0 | |||
|
|
|||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we have tests for this ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, bin_tests/tests/crashtracker_bin_test.rs:1797
670f703 to
8b86bc7
Compare
a297079 to
b7bc511
Compare
b7bc511 to
2425179
Compare
| std::env::remove_var("_DD_ERRORS_INTAKE_ENABLED"); | ||
| } | ||
|
|
||
| #[cfg_attr(miri, ignore)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Miri doesnt like creating os_info
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|

What does this PR do?
Parity with telemetry intake + errortracking needs OS info
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
How to test the change?