search

scieloorg / PC-Programs

SciELO PC Programs
7 stars 24 forks source link

Ajustar o logging da aplicação #3221

Closed patymori closed 4 years ago

patymori commented 4 years ago

O que esse PR faz?

Este PR se propõe a ajustar a utilização do logging pela aplicação. São usadas as configurações definidas no módulo logging_config e o nível do log é redefinido conforme argumento informado nos comandos em --loglevel. Também foram substituídos prints em tela por log de mensagens.

Onde a revisão poderia começar?

É recomendada que a revisão seja feita por commits.

Como este poderia ser testado manualmente?

  1. Execute os comandos scieloxpm, scielojournals, scielo2pubmed, scieloxc, scieloxcserver e xml_transform com diferentes níveis de log no argumento --loglevel
  2. Verifique os arquivos de log prodtools.log, prodtools.err e exporter.log e o console e o log deve seguir a configuração do --loglevel
  3. Verifique os pontos do código que prints foram substituídos pelo log

Algum cenário de contexto que queira dar?

.

Screenshots

N/A

Quais são tickets relevantes?

3203

Referências

https://docs.python.org/3/howto/logging-cookbook.html https://docs.python.org/3/library/logging.html https://docs.python.org/3/library/logging.config.html

patymori commented 4 years ago

@patymori tá com conflitos, e acho que há uns prints que deveriam ser logger.debug na pasta validations.

OK, @robertatakenaka. Somente uma observação sobre os logs e a configuração padrão dos comandos: quando não informado, o logging fica com nível WARNING e, portanto, os comandos de log apontados para mudança de debug para info não seriam exibidos na tela. Acha que essa configuração deveria ser alterada?

robertatakenaka commented 4 years ago

Minha intenção é que os info seja impressos também na tela. Senão, teria que ser mantido print

patymori commented 4 years ago

@gustavofonseca e @robertatakenaka,

Simplifiquei as configurações e agora há logs em tela, com nível a definir na execução dos comandos, e em 1 arquivo único de log. Ainda faltou configurar por módulo mas não consegui fazer funcionar.

Por último, alterei o módulo de validação para utilizar o módulo logging.

patymori commented 4 years ago

@gustavofonseca, só avisando que estou fazendo o merge deste e, caso veja necessidade, podemos abrir issues para as melhorias sugeridas.