-
Notifications
You must be signed in to change notification settings - Fork 805
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
Change noisy frontend poll timeout log to debug level #5725
Change noisy frontend poll timeout log to debug level #5725
Conversation
3c91334
to
94f3c1a
Compare
Codecov Report
Additional details and impacted files
... and 8 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Pull Request Test Coverage Report for Build 018e10d5-6a1f-4cd2-88e3-6e48230dc0e7Details
💛 - Coveralls |
c71c892
to
2475169
Compare
2475169
to
e3ca207
Compare
What changed?
Poll requests with timeout less than 20s causes a lot of noise in the logs. In total we see about 80% of the frontend logs is this single line:
Context timeout is lower than critical value for long poll API.
There's not much value in keeping this as warning level so switching to debug level. Also adding domain name and task list tags to these logs
Why?
Reduce log noise.