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

No logging during startup #10244

Closed
richvdh opened this issue Jun 23, 2021 · 0 comments · Fixed by #10407
Closed

No logging during startup #10244

richvdh opened this issue Jun 23, 2021 · 0 comments · Fixed by #10407
Assignees
@reivilibre
Labels
P3 (OBSOLETE: use S- labels.) Approved backlog: not yet scheduled, will accept patches S-Minor Blocks non-critical functionality, workarounds exist. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.

Comments

@richvdh
Copy link
Member

richvdh commented Jun 23, 2021

Because we write logs to a buffer, and there isn't much logging during startup, Synapse tends to be ominously silent during startup. Worse, if there is a problem, and you kill synapse, you have no way of telling where it got stuck.

It is suggested that we could extend MemoryHandler to flush its logs every few seconds to alleviate this problem.
@babolivier babolivier added P2 S-Minor Blocks non-critical functionality, workarounds exist. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. labels Jun 25, 2021
@callahad callahad added P3 (OBSOLETE: use S- labels.) Approved backlog: not yet scheduled, will accept patches and removed P2 labels Jul 8, 2021
@reivilibre reivilibre self-assigned this Jul 15, 2021
@reivilibre reivilibre mentioned this issue Jul 15, 2021
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@reivilibre reivilibre

Labels
P3 (OBSOLETE: use S- labels.) Approved backlog: not yet scheduled, will accept patches S-Minor Blocks non-critical functionality, workarounds exist. T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add a PeriodicallyFlushingMemoryHandler to prevent logging silence matrix-org/synapse
4 participants
@callahad @richvdh @babolivier @reivilibre