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

docker image sets log level to warn by default #3370

Closed
richvdh opened this issue Jun 8, 2018 · 3 comments
Closed

docker image sets log level to warn by default #3370

richvdh opened this issue Jun 8, 2018 · 3 comments
Assignees
Labels
A-Docker Docker images, or making it easier to run Synapse in a container. z-minor (Deprecated Label) z-p2 (Deprecated Label)

Comments

@richvdh
Copy link
Member

richvdh commented Jun 8, 2018

... apparently? that's insane; it ought to be at least INFO

@neilisfragile neilisfragile added z-p2 (Deprecated Label) z-minor (Deprecated Label) labels Jun 8, 2018
@michaelkaye
Copy link
Contributor

Is there something specific we should be storing from running a synapse that means we should not default to only showing WARNING? I'm of a general opinion that not logging if we don't need to is great - there's a certain amount of config and setup needed for users to make docker not just store logs in a ever-growing stdout file (the default json-file driver).

@richvdh
Copy link
Member Author

richvdh commented Jun 8, 2018

I'd expect us to at least log the requests. It's essentially impossible to debug anything without any logs.

@richvdh richvdh added the A-Docker Docker images, or making it easier to run Synapse in a container. label Jun 19, 2019
@hawkowl
Copy link
Contributor

hawkowl commented Jun 25, 2019

Fixed in #5547

@hawkowl hawkowl closed this as completed Jun 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Docker Docker images, or making it easier to run Synapse in a container. z-minor (Deprecated Label) z-p2 (Deprecated Label)
Projects
None yet
Development

No branches or pull requests

4 participants