getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
37.08k stars 3.98k forks source link

fix(post-process): remove some noisy logs #70075

Closed JoshFerge closed 2 weeks ago

JoshFerge commented 2 weeks ago

these log literally tens of millions of times per hour and are not critical. lets save some money. added a metric to replace one of them. link to logs: https://cloudlogging.app.goo.gl/MGVM1k6F874pbjtn7

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.83%. Comparing base (9a31bb0) to head (d647945). Report is 21 commits behind head on master.

:exclamation: Current head d647945 differs from pull request most recent head b852fe3. Consider uploading reports for the commit b852fe3 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #70075 +/- ## =========================================== + Coverage 58.39% 79.83% +21.44% =========================================== Files 6490 6502 +12 Lines 288802 289254 +452 Branches 49750 49811 +61 =========================================== + Hits 168641 230929 +62288 + Misses 119745 57914 -61831 + Partials 416 411 -5 ``` | [Files](https://app.codecov.io/gh/getsentry/sentry/pull/70075?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry) | Coverage Δ | | |---|---|---| | [src/sentry/tasks/post\_process.py](https://app.codecov.io/gh/getsentry/sentry/pull/70075?src=pr&el=tree&filepath=src%2Fsentry%2Ftasks%2Fpost_process.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry#diff-c3JjL3NlbnRyeS90YXNrcy9wb3N0X3Byb2Nlc3MucHk=) | `90.93% <100.00%> (+76.77%)` | :arrow_up: | ... and [1932 files with indirect coverage changes](https://app.codecov.io/gh/getsentry/sentry/pull/70075/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=getsentry)