use a logging configuration file

According to Cédric [1], trytond should fail when logfile can't be opened [2].

[1] http://codereview.tryton.org/831002/diff/1/trytond/server.py#inline-preview-0-46-b
[2] http://hg.tryton.org/trytond/file/62f1b9747e9f/trytond/server.py#l43

assigned to @pilou_2

added trytond type::behavior + 1 deleted label

I think the best will be to completly drop this code and just allow to pass a logging configuration file to load.
See: https://docs.python.org/2/library/logging.config.html#
This post explain quite well how real usage could be: http://antonym.org/2005/03/a-real-python-logging-example.html

added 1 deleted label and removed 1 deleted label

What path would you like for the configuration file ? Should it be configurable ?

On 10 Jul 17:00, Pierre-Louis Bonicoli wrote:
> What path would you like for the configuration file ? Should it be configurable ?

It must be a cmd line option and not required.

Which default behavior do you want when this option is not set ? only error messages in stderr ?

Let say for now it stays as the current default.
But there is #3182 (closed)

Do you prefer use of logging.config.fileconfig (external file with config-parser format) or logging.config.dictconfig (external file with JSON format) [1] ? The second one is recommended in Python HowTo [2].

[1] https://docs.python.org/2/library/logging.config.html#configuration-functions
[2] https://docs.python.org/2/howto/logging.html#configuring-logging.

On 17 Jul 11:49, Pierre-Louis Bonicoli wrote:
> Do you prefer use of logging.config.fileconfig (external file with config-parser format) or logging.config.dictconfig (external file with JSON format) [1] ? The second one is recommended in Python HowTo [2].

Definitely the second one.

OK the order of the links lead me to misunderstand.
The JSON format is horrible and not design to be read or write by human, so it should be the configuration format.

New changeset 779c0101239d by Pierre-Louis Bonicoli in branch 'default':
Use a configuration file for logging
http://hg.tryton.org/trytond/rev/779c0101239d

added 1 deleted label and removed 1 deleted label

closed

mentioned in issue #4043 (closed)