Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Corrige des logs en double sur l'environnement de dév (#6159) #6174

Merged
merged 3 commits into from
Sep 21, 2021

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Sep 18, 2021

Fix #6159.

Tous les logs ne sont pas en double, seulement ceux qui ne sont pas dans le périmètre du logger "django.server". Le souci vient de logs qui remontent jusqu'à la racine (ce qui n'est pas le cas de ceux "django.server") et sont donc traités par un logger configuré explicitement, et ensuite encore une fois par le logger root.

Différentes solutions sont envisageables :

  1. Indiquer aux loggers existants de ne pas propager les enregistrements qu'ils voient passer (propagate = False);
  2. Configurer le logger root pour qu'il ne fasse rien, au lieu de dupliquer des logs ;
  3. Configurer les loggers existants pour qu'ils ne fassent rien eux-mêmes et confier le boulot au logger root .

J'ai choisi l'option 3, parce que c'est pour moi la plus propre. C'est une solution à laquelle il est fait allusion dans la documentation. La solution 1 bloque la remontée dans la hiérarchie, c'est peu naturel de le faire systématiquement. La solution 2 a un petit risque qui est de cacher des logs par inadvertance.

Contrôle qualité

  • Lancer le serveur make run-back
  • Modifier un fichier Python au pif et constater que le message de "watch/reload" de Django n'est plus en double ;
  • Parfois quand on va sur la page d'accueil, il y a un warning "favicon not found" qui ne devrait plus être en double non plus ;
  • On peut aussi volontairement introduire des erreurs dans le code pour logger des stack traces (qui n'apparaissent plus en double) ;
  • Constater qu'on conserve bien tous les logs quand même.

@Arnaud-D Arnaud-D added C-Back Concerne le back-end Django C-DevelopmentEnv Amélioration de l'environnement de dev labels Sep 18, 2021
Copy link
Member

@Situphen Situphen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci beaucoup ! Ça me portait sur le système depuis longtemps :D

@Situphen Situphen enabled auto-merge (rebase) September 21, 2021 17:26
@Situphen Situphen merged commit ecfa438 into zestedesavoir:dev Sep 21, 2021
@Arnaud-D Arnaud-D deleted the arrete_de_voir_double branch September 21, 2021 19:57
@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.621% when pulling 7d9d7f1 on Arnaud-D:arrete_de_voir_double into 11bbf8e on zestedesavoir:dev.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django C-DevelopmentEnv Amélioration de l'environnement de dev
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Doublon pour les logs en couleur dans l'env de dev
3 participants