Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Sentry noise: Sentry is attempting to send N pending error messages #14069

Open
DMRobertson opened this issue Oct 5, 2022 · 0 comments
Open

Sentry noise: Sentry is attempting to send N pending error messages #14069

DMRobertson opened this issue Oct 5, 2022 · 0 comments
Labels
A-Logging Synapse's logs (structured or otherwise). Not metrics. O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Tolerable Minor significance, cosmetic issues, low or no impact to users. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. Z-Sentry Issue was discovered by looking at Sentry reports on Matrix.org

Comments

@DMRobertson
Copy link
Contributor

E.g. https://sentry.tools.element.io/organizations/element/issues/608

2022-09-28 10:27:50,446 - twisted - 274 - INFO - sentinel - Main loop terminated.
2022-09-28 10:27:50,549 - twisted - 274 - ERROR - sentinel - Sentry is attempting to send 94 pending error messages
2022-09-28 10:27:50,555 - twisted - 274 - ERROR - sentinel - Waiting up to 2 seconds
2022-09-28 10:27:50,563 - twisted - 274 - ERROR - sentinel - Press Ctrl-C to quit

Presumably comes from https://github.com/getsentry/sentry-python/blob/4c09f3203d6d19789c6fa729a2e46557ad4ea913/sentry_sdk/integrations/atexit.py#L30-L33

Given that we redirect stderr -> twisted -> stdlib logging (for Reasons :TM:) this ends up as an ERROR; I think a warning makes more sense. There are some notes here.
@DMRobertson DMRobertson added A-Logging Synapse's logs (structured or otherwise). Not metrics. S-Tolerable Minor significance, cosmetic issues, low or no impact to users. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. O-Uncommon Most users are unlikely to come across this or unexpected workflow Z-Sentry Issue was discovered by looking at Sentry reports on Matrix.org labels Oct 5, 2022
@matrixbot matrixbot mentioned this issue Dec 21, 2023
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
A-Logging Synapse's logs (structured or otherwise). Not metrics. O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Tolerable Minor significance, cosmetic issues, low or no impact to users. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. Z-Sentry Issue was discovered by looking at Sentry reports on Matrix.org
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@DMRobertson