microsoft / onefuzz

A self-hosted Fuzzing-As-A-Service platform
MIT License
2.82k stars 199 forks source link

Emit `CrashReported` Metric to App insights. #3571

Closed nharper285 closed 9 months ago

nharper285 commented 9 months ago

Summary of the Pull Request

What is this about?

PR Checklist

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

codecov-commenter commented 9 months ago

Codecov Report

Merging #3571 (d4fd422) into main (4a67a48) will decrease coverage by 0.43%. The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #3571      +/-   ##
==========================================
- Coverage   39.99%   39.57%   -0.43%     
==========================================
  Files         303      275      -28     
  Lines       37679    32667    -5012     
  Branches     1728     1728              
==========================================
- Hits        15070    12928    -2142     
+ Misses      22287    19417    -2870     
  Partials      322      322              
Files Coverage Δ
src/agent/onefuzz-telemetry/src/lib.rs 17.32% <0.00%> (-0.06%) :arrow_down:
...gent/onefuzz-task/src/tasks/report/crash_report.rs 10.00% <0.00%> (-0.13%) :arrow_down:

... and 59 files with indirect coverage changes