-
Notifications
You must be signed in to change notification settings - Fork 10
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
Add a metric for the raw report size #574
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #574 +/- ##
=======================================
Coverage 97.49% 97.49%
=======================================
Files 414 414
Lines 34982 34986 +4
=======================================
+ Hits 34105 34109 +4
Misses 877 877
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #574 +/- ##
=======================================
Coverage 97.49% 97.49%
=======================================
Files 414 414
Lines 34982 34986 +4
=======================================
+ Hits 34105 34109 +4
Misses 877 877
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #574 +/- ##
=======================================
Coverage 97.49% 97.49%
=======================================
Files 414 414
Lines 34982 34986 +4
=======================================
+ Hits 34105 34109 +4
Misses 877 877
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #574 +/- ##
=======================================
Coverage 97.54% 97.54%
=======================================
Files 449 449
Lines 36188 36192 +4
=======================================
+ Hits 35299 35303 +4
Misses 889 889
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
lgtm! would like to see buckets for larger report sizes, but also once this is deployed i imagine we'll immediately query it in grafana and see if there's even an issue. it's easy to change the bucket sizes later
for processor in processors: | ||
if processor.matches_content(parsed_report, first_line, name): | ||
sentry_metrics.incr( | ||
"services.report.report_processor.parser", | ||
tags={"type": type(processor).__name__}, | ||
) | ||
RAW_REPORT_SIZE.labels(processor=processor.name).observe(report.size) |
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.
checked the callsite for this function to double-check that report
here is indeed a single raw coverage report embedded inside of a larger Codecov upload
worker/services/report/raw_upload_processor.py
Lines 159 to 161 in a07ddef
report = process_report( | |
report=report_file, report_builder=report_builder_to_use | |
) |
It might be nice to have a histogram with the raw size in bytes for various report formats
d6da843
to
0fdf25c
Compare
It might be nice to have a histogram with the raw size in bytes for various report formats