Info logging is default True without config setting #1860

Activity

1. deniszh commented on Mar 20, 2017

   

   deniszh

   on Mar 20, 2017

   Member

   More actions

   Hi @Kenterfie ,
   Could you please show an example of logs which you have? I do not see any enabled by default logging in master...
2. deniszh

   added

   question

   on Mar 20, 2017
3. deniszh commented on Mar 26, 2017

   

   deniszh

   on Mar 26, 2017

   Member

   More actions

   Hello @Kenterfie,
   Did you find source of excessive logging? As far as I see in the code by default logging should be disabled.
4. xneo64 commented on May 2, 2017

   

   xneo64

   on May 2, 2017

   More actions

   Same here.

   I think the culprit is

   graphite-web/webapp/graphite/logger.py

   Line 39 in 622830d

   True,

   
   I usually patch my graphite installations with the above set to False.

   As @Kenterfie said, this used to quickly fill up docker containers on busy systems, even with log rotation set to 1 day. Would be great to have the possibility to disable.
5. deniszh

   added

   enhancement

   and removed

   question

   on May 3, 2017
6. deniszh commented on May 3, 2017

   

   deniszh

   on May 3, 2017

   Member

   More actions

   OK, this should be addressed.
7. Quentin-M commented on Sep 11, 2018

   

   Quentin-M

   on Sep 11, 2018

   Contributor

   More actions

   Apps that forcefully log to disk are a major pain when using containers.

   👍React with 👍1Reacted by bob454522
8. stale commented on Apr 13, 2020

   

   stalebot

   on Apr 13, 2020

   More actions

   This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

   👎React with 👎2Reacted by Nicolas Boutet and bob454522
9. stale

   added

   stale

   on Apr 13, 2020
10. stale

    closed this as completedon Apr 20, 2020
11. henningw commented on Mar 25, 2021

    

    henningw

    on Mar 25, 2021

    More actions

    This issue is still present in default installation, e.g. from Debian 10 repository.

    👍React with 👍1Reacted by bob454522
12. deniszh

    reopened this on Mar 25, 2021
13. stale

    removed

    stale

    on Mar 25, 2021
14. deniszh

    added

    pinned

    on Mar 25, 2021
15. deniszh commented on Mar 25, 2021

    

    deniszh

    on Mar 25, 2021

    Member

    More actions

    Btw, you can set log file name to "-" to redirect logs to stdout
    E.g.

    LOG_FILE_INFO="-"
    LOG_FILE_EXCEPTION="-"
    LOG_FILE_CACHE="-"
    LOG_FILE_RENDERING="-"
    

    👍React with 👍1Reacted by Xellor
16. deniszh

    added a commit that references this issue on Apr 11, 2021

    Merge pull request #2691 from deniszh/DZ-info-log-amend

    ...

    2278019