Skip to content
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

Configure sentry logging integration for ERROR logs only #559

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

mvandenburgh
Copy link
Member

Setting this to WARNING causes to many extraneous logs to be reported as issues in Sentry. rio_tiler in particular is very noisy, and it's difficult to calibrate things because we have no control over the source data its processing.

Setting this to WARNING causes to many extraneous logs to be reported
as issues in Sentry. `rio_tiler` in particular is very noisy, and it's
difficult to calibrate things because we have no control over the source
data its processing.
@mvandenburgh mvandenburgh merged commit 040022d into main Nov 25, 2024
8 checks passed
@mvandenburgh mvandenburgh deleted the update-sentry-logging branch November 25, 2024 17:03
@zachmullen
Copy link
Contributor

Some warnings are helpful to see, so I think a better solution would be to suppress this specific warning at the python level using warnings.filterwarning('ignore', ...) with some combination of message and module.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants