Support #39382
Ne pas logger un fichier de métadonné invalide comme une erreur
Début:
29 janvier 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Non
Planning:
Non
Historique
Mis à jour par Thomas Noël il y a environ 4 ans
- Statut changé de Nouveau à Information nécessaire
- Assigné à changé de Thomas Noël à Benjamin Dauvergne
En fait on loggue en erreur l'absence de métadonnées (quand elles sont téléchargées), mais on warning quand un fichier est absent... C'est un peu le bronx.
111 warning = logger.warning 112 if 'METADATA' not in idp: 113 # if we have no metadata in cache, we must emit errors 114 warning = logger.error
Est-ce qu'on passe en warning partout tout le temps ? Des soucis sur les métadonnées ça me semble quand même un problème (surtout quand elles sont sur un fichier local) et je trouve ça normal de recevoir une alerte de "haut niveau" (error). Non ?
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Ouais j'avais oublié ça ; c'est justement prévu pour ne pas nous pourrir d'erreur tant qu'on a un truc valable en cache, je ne sais plus si le cache expire ou pas. Le but c'est de cacher l'erreur si ça dure 24h pas 1 semaine.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Information nécessaire à Rejeté
Je ne comprends pas ce qui est voulu ici :
- on log en erreur si les métadonnées doivent être téléchargées mais ne sont pas dispo en cache ou que le cache est trop vieux (24h)
- on log en erreur si un fichier local de métadonnée n'est pas ouvrable (open foire) ou qu'il est mal formé (on n'y trouve pas les noeuds de base et l'entityID)