Développement #100460
openla conf de logging est trop différente de la prod
0%
Description
Dans devinst on fait au mieux pour avoir des settings proche de la prod mais il y a un raté pour le logging. On redéfini la variable LOGGING
Ca serait utile de se rapprocher des conditions de prod en modifiant le dict importé depuis la conf hobo
Updated by Robot Gitea about 1 year ago
- Status changed from Nouveau to Solution proposée
- Assignee set to Gael Pasgrimaud
Gael Pasgrimaud (gpasgrimaud) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/publik-devinst/pulls/65
- Titre : app-setup: remove LOGGING config. you can use syslog if needed (#100460)
- Modifications : https://git.entrouvert.org/entrouvert/publik-devinst/pulls/65/files
Updated by Robot Gitea about 1 year ago
- Status changed from Solution proposée to En cours
Gael Pasgrimaud (gpasgrimaud) a commencé à travailler sur une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/publik-devinst/pulls/65
- Titre : WIP: app-setup: remove LOGGING config. you can use syslog if needed (#100460)
- Modifications : https://git.entrouvert.org/entrouvert/publik-devinst/pulls/65/files
Updated by Robot Gitea about 1 year ago
- Status changed from En cours to Solution proposée
Updated by Robot Gitea about 1 year ago
- Status changed from Solution proposée to En cours
Gael Pasgrimaud (gpasgrimaud) a commencé à travailler sur une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/publik-devinst/pulls/65
- Titre : WIP: app-setup: remove LOGGING config. you can use syslog if needed (#100460)
- Modifications : https://git.entrouvert.org/entrouvert/publik-devinst/pulls/65/files
Updated by Robot Gitea about 1 year ago
- Status changed from En cours to Solution proposée
Updated by Gael Pasgrimaud about 1 year ago
En fait on a vu que ça servait à rien d'avoir une conf spécifique. Personne ne lit la console. Si ? Ca ne sert qu'à faire du pdb, pas à lire des logs.
Je n'ai mis qu'un logger pour django.server parce que c'est spécifique au server de dev django. Sans ça ça envoyait des mails à chaque réponse en 500
Updated by Frédéric Péters about 1 year ago
Personne ne lit la console
Perso je lis et j'apprécie avoir les logs dans la console, et que si un pdb est posé je puisse lire au-dessus les logs éventuellement émis. (mais je n'utilise déjà pas publik-devinst donc je n'ai pas d'objection à ce qui y serait commité).
Updated by Robot Gitea about 1 year ago
- Status changed from Solution proposée to Solution validée
Emmanuel Cazenave (ecazenave) a approuvé une pull request sur Gitea concernant cette demande :
Updated by Robot Gitea about 1 year ago
- Status changed from Solution validée to Résolu (à déployer)
Gael Pasgrimaud (gpasgrimaud) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/publik-devinst/pulls/65
- Titre : app-setup: remove LOGGING config. you can use syslog if needed (#100460)
- Modifications : https://git.entrouvert.org/entrouvert/publik-devinst/pulls/65/files
Updated by Gael Pasgrimaud about 1 year ago
- Status changed from Résolu (à déployer) to Nouveau
Frédéric Péters a écrit :
Personne ne lit la console
Perso je lis et j'apprécie avoir les logs dans la console, et que si un pdb est posé je puisse lire au-dessus les logs éventuellement émis. (mais je n'utilise déjà pas publik-devinst donc je n'ai pas d'objection à ce qui y serait commité).
Avec un peu de recule, c'est pas pire. J'ai un term ouvert avec journalctl -xfu supervisor.service qui affiche les logs de toutes les apps. C'est pas mal. On voit passer toutes les traces.
Par contre quand on lance un service à la main pour lancer un pdb, on a plus de log facilement. Ça, c'est pas top du tout. Il va falloir trouver un truc
Updated by Gael Pasgrimaud about 1 year ago
- Related to Développement #100897: Installer cysystemd added