search

radar-parlamentar / radar

Estamos de mudança para o GitLab: https://gitlab.com/radar-parlamentar/radar.
http://radarparlamentar.polignu.org
84 stars 44 forks source link

Logs no Docker - container django #445

Closed leonardofl closed 6 years ago

leonardofl commented 6 years ago

A saída (console) do container django é a saída do uwsgi, não da aplicação.

Quando a aplicação (código Python) faz um log, um print ou dá uma exceção, precisamos ver isso em algum log.

Acho que a ideia é gravar td isso no /var/log/adar/radar.log.

Localmente, daria pra ver fácil com docker exec radar_django_1 tail -f /var/log/radar/radar.log. Em produção acho q tb... seria só logar na VM de produção e aí dar esse comando.

Pra isso funcionar, talvez só falte criar a pasta /var/log/radar.

leonardofl commented 6 years ago

Talvez essa issue inclua a #438 (ou talvez a torne desnecessária).

leonardofl commented 6 years ago

Correção: oq a aplicação escreve com print vai pra saída padrão, e assim aparece no docker logs -f radar_django_1. Mas de fato, não oq vai pro log.

leonardofl commented 6 years ago

Com os commits a57f2eb1d60cbb035061b573dcd9263aac91b52b e 19ffa79ddf125f5613ce264c5f979f033902f868 temos visível no radar.log tudo que é logado com o logger e as exceções não tratadas também.

O que ficou faltando seria ver no log o que é impresso com print. Mas talvez não seja necessário. Talvez seja razoável assumir que tudo que seja impresso no servidor deva ser feito com o logger, não com o print. Por outro lado, fazer isso talvez não seja tão difícil só mexendo no settings. O que vc acha @diraol ?

diraol commented 6 years ago

100% favorável a assumir que o logger deve ser usado!

leonardofl commented 6 years ago

Então essa issue tá feita! :D

leonardofl commented 6 years ago

Como não dei o "fix" vou fechar por aqui mesmo já.