Development #45001
ne pas envoyer de traceback par mail lorsqu'on est dans check_status
0%
Description
Quand il y a un pépin profond dans check_status, on reçoit une trace (par exemple lors du check_status de base_adresse si api.adresse.gouv.fr n'est pas dispo, y'a une trace requests, cf 45000)
Idéalement on ne devrait pas envoyer de traces, car check_status fait son travail d'annonce de service down/up à travers des self.logger.error/warning nettement plus explicites.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-base-do-not-log-errors-in-check-status-45001.patch 0001-base-do-not-log-errors-in-check-status-45001.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Youpla
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-base-do-not-log-errors-in-check-status-45001.patch 0001-base-do-not-log-errors-in-check-status-45001.patch ajouté
Mon patch bloquait aussi la création des objets ResourceLog, revoilà un patch un plus fin qui va toucher au handler qui envoie des mails plutôt qu'au logger.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-base-do-not-log-errors-in-check-status-45001.patch 0001-base-do-not-log-errors-in-check-status-45001.patch ajouté
(cosmétique)
Mis à jour par Thomas Noël il y a plus de 2 ans
Plutôt que modifier le setLevel ce n'était pas possible d'ajouter un filter qui coupe tout ? Ou bien tu préfères qu'on reçoive quand même les erreurs CRITICAL ?
Mis à jour par Valentin Deniaud il y a plus de 2 ans
Thomas Noël a écrit :
Plutôt que modifier le setLevel ce n'était pas possible d'ajouter un filter qui coupe tout ? Ou bien tu préfères qu'on reçoive quand même les erreurs CRITICAL ?
Comme tu veux, mon raisonnement ça a effectivement été de chercher comment couper le logging, j'ai trouvé https://bugs.python.org/issue36318, qui dit en substance « il y a un attribut logger.disable
mais il est pas documenté parce que que la bonne pratique c'est de toujours logger les trucs hyper graves, donc désactivation = setLevel le plus haut » et je me suis dit que c'était OK pour notre cas d'usage.
Mis à jour par Thomas Noël il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Parfait !
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 3b762f92de1660a8b81dd1a59e029293094104d6 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Jul 21 15:48:41 2021 +0200 base: do not log errors in check status (#45001)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
base: do not log errors in check status (#45001)