Bug #80491
rapport de dépréciations invalide, KeyError: 'source'
0%
Description
Exception:
type = '<class 'KeyError'>', value = ''source''
Stack trace (most recent call first):
File "/usr/lib/python3/dist-packages/wcs/backoffice/deprecations.py", line 46, in <listcomp>
44 with open(report_path) as fd:
45 report = json.load(fd)
> 46 report['report_lines'] = [x for x in report['report_lines'] if x['source'] == source]
47 report['report_lines'].sort(key=lambda x: x['category'])
48 return report
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a 8 mois
- Assigné à mis à Valentin Deniaud
Introduit par #79463, pas encore en prod.
Le patch casse tous les inspects au pire pendant deux heures (cette trace), puis le cron scan des dépréciations arrive et ça répare.
Sauf que sur l'instance réunion, il y a une erreur lors de la génération du rapport, donc ça reste sur l'ancien, et patatra.
Je crois aussi que j'avais eu la trace en local et pas cherché à comprendre.
Bref, éviter de crasher sur cette clé manquante me paraît bien.
Mis à jour par Robot Gitea il y a 8 mois
- Statut changé de Nouveau à En cours
Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/620
- Titre : WIP: backoffice: do not crash on deprecation file without source (#80491)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/620/files
Mis à jour par Robot Gitea il y a 8 mois
- Statut changé de Solution proposée à Solution validée
Serghei Mihai (smihai) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 8 mois
- Statut changé de Solution validée à Résolu (à déployer)
Valentin Deniaud (vdeniaud) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/620
- Titre : backoffice: do not crash on deprecation file without source (#80491)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/620/files
Mis à jour par Transition automatique il y a 8 mois
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Frédéric Péters il y a 8 mois
Sauf que sur l'instance réunion, il y a une erreur lors de la génération du rapport, donc ça reste sur l'ancien, et patatra.
Il aurait fallu s'intéresser à ça.
backoffice: do not crash on deprecation file without source (#80491)