Project

General

Profile

Bug #80491

rapport de dépréciations invalide, KeyError: 'source'

Added by Valentin Deniaud 3 months ago. Updated 3 months ago.

Status:
Fermé
Priority:
Normal
Assignee:
Valentin Deniaud
Target version:
-
Start date:
22 August 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

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

Associated revisions

Revision 97096536 (diff)
Added by Valentin Deniaud 3 months ago

backoffice: do not crash on deprecation file without source (#80491)

History

#2

Updated by Valentin Deniaud 3 months ago

  • Assignee set to 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.

#3

Updated by Robot Gitea 3 months ago

  • Status changed from Nouveau to En cours

Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :

#4

Updated by Robot Gitea 3 months ago

  • Status changed from En cours to Solution proposée
#5

Updated by Robot Gitea 3 months ago

  • Status changed from Solution proposée to Solution validée

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

#6

Updated by Robot Gitea 3 months ago

  • Status changed from Solution validée to Résolu (à déployer)

Valentin Deniaud (vdeniaud) a mergé une pull request sur Gitea concernant cette demande :

#7

Updated by Transition automatique 3 months ago

  • Status changed from Résolu (à déployer) to Solution déployée
#8

Updated by Frédéric Péters 3 months ago

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.

#9

Updated by Transition automatique about 1 month ago

Automatic expiration

Also available in: Atom PDF