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

Reduce logging to Sentry #3856

Closed
agjohnson opened this issue Mar 26, 2018 · 3 comments · Fixed by #5118
Closed

Reduce logging to Sentry #3856

agjohnson opened this issue Mar 26, 2018 · 3 comments · Fixed by #5118
Labels
Accepted Accepted issue on our roadmap Improvement Minor improvement to code
Milestone

Comments

@agjohnson
Copy link
Contributor

We're error logging a number of things that shouldn't be error logs. This pollutes Sentry with a number of issues that aren't really our issue (build failures on pip/etc). We should clean up our logging here to reduce noise and get back to a reasonable number of logging events.

@humitos
Copy link
Member

humitos commented Oct 23, 2018

I'm linking some work that has done outside this issue.

Related PRs: #4779 #4761 #4760 #4730

Issues: #4791 #4790 #4590

This one, can be closed soon. In theory, after the following deploy, we will decrease the spam-logging into Sentry a lot.

@humitos
Copy link
Member

humitos commented Oct 30, 2018

Maybe this setting can be used to do not log known exceptions to Sentry on Celery tasks:

https://docs.sentry.io/clients/python/integrations/django/#additional-settings

@humitos
Copy link
Member

humitos commented Oct 31, 2018

@humitos humitos added Improvement Minor improvement to code Accepted Accepted issue on our roadmap labels Jan 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Accepted issue on our roadmap Improvement Minor improvement to code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants