Projet

Général

Profil

Bug #72914

base adresse, ne pas avoir de trace remontée vers sentry quand ils sont down

Ajouté par Frédéric Péters il y a plus d'un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
03 janvier 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Avec l'autocomplétion ça fait des requêtes qui ne sont pas les mêmes et se trouvent multipliées dans sentry (cf capture) et quand c'est un peu longtemps down ça peut faire vraiment beaucoup.


Fichiers

Révisions associées

Révision f914b854 (diff)
Ajouté par Benjamin Dauvergne il y a environ un an

utils: do not log requests errors when the resource is down (#72914)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ un an

  • Assigné à mis à Benjamin Dauvergne
#2

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Nouveau à En cours

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#3

Mis à jour par Benjamin Dauvergne il y a environ un an

J'ai choisir d'implémenter ça comme un comportement général, quand un connecteur est marqué down, ne pas journaliser les erreurs de requests au niveau erreur (on a par ailleurs déjà une configuration pour des délais pour remonter un connecteur en erreur comme une erreur).

#4

Mis à jour par Benjamin Dauvergne il y a environ un an

Yapuka faire des tests.

#5

Mis à jour par Benjamin Dauvergne il y a environ un an

Et avec une petite pensée pour Thomas j'ajoute aussi la désactivation du retry si la resource est down, pas la peine de martyriser le pauvre animal.

#6

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de En cours à Solution proposée
#7

Mis à jour par Benjamin Dauvergne il y a environ un an

On ne peut pas tester le comportement pour les ConnectionTimeout/ReadError ça n'est pas supporté par responses.

#8

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Solution proposée à Solution validée

Serghei Mihai (smihai) a approuvé une pull request sur Gitea concernant cette demande :

#9

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Solution validée à Résolu (à déployer)

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

#10

Mis à jour par Transition automatique il y a environ un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#11

Mis à jour par Transition automatique il y a 11 mois

Automatic expiration

Formats disponibles : Atom PDF