harvard-lil / perma

Indelible links
411 stars 70 forks source link

Scoop logging #3417

Closed rebeccacremona closed 9 months ago

rebeccacremona commented 9 months ago

We are seeing a couple varieties of error go by, capturing with Scoop.

This PR adds special handling for two categories of error, so that we can a) more easily count and track occurrences over time; b) get easier-to-read reports when those errors do happen (saving more verbose logging for anomalies); c) direct them to the 'warning' log instead of the 'error' log, to quiet the stream.

This PR also updates watchdog on the side... turns out this version of Werkzeug requires that tweak for our dev setup to work as expected.

codecov[bot] commented 9 months ago

Codecov Report

Attention: 8 lines in your changes are missing coverage. Please review.

Comparison is base (ee508f6) 68.89% compared to head (3077110) 68.82%. Report is 5 commits behind head on develop.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #3417 +/- ## =========================================== - Coverage 68.89% 68.82% -0.07% =========================================== Files 53 53 Lines 7297 7304 +7 =========================================== Hits 5027 5027 - Misses 2270 2277 +7 ``` | [Files](https://app.codecov.io/gh/harvard-lil/perma/pull/3417?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=harvard-lil) | Coverage Δ | | |---|---|---| | [perma\_web/perma/celery\_tasks.py](https://app.codecov.io/gh/harvard-lil/perma/pull/3417?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=harvard-lil#diff-cGVybWFfd2ViL3Blcm1hL2NlbGVyeV90YXNrcy5weQ==) | `47.46% <0.00%> (-0.26%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.