Bug #89213
création inutile de fichiers dans afterjobs/
0%
Description
Les moments où on fait des check quand on charge des objets, genre sur un block :
if check_deprecated:
# check for deprecated elements
job = DeprecationsScan()
try:
job.check_deprecated_elements_in_object(blockdef)
except DeprecatedElementsDetected as e:
raise BlockdefImportError(str(e))
Le job = DeprecationsScan()
- créé un objet de type AfterJob (par héritage)
- puis ça fait job.check_deprecated_elements_in_object
- qui appelle self.check_objects
- qui appelle self.increment_count()
- qui appelle self.store()
- et boum sur le disque
Sur des sites avec beaucoup de block, wscall, data_source, on se retrouve avec un répertoire afterjobs/ bourrés de choses vide, et ça pose beaucoup de problèmes ailleurs (par exemple sur un changement de workflow qui cherche à savoir si un AfterJob n'est pas déjà en cours).
Il faut:- ne pas faire de store() sur ces AfterJob qui s'en sont pas
- ne faire ce scan dès qu'on charge ces objets (et uniquement quand on les importe)
Demandes liées
Révisions associées
misc: add test to be sure depreciations are not checked on load (#89213)
depreciations: don't check depreciations on snapshot load (#89213)
depreciations: check them only during UI and API imports (#89213)
depreciations: don't store job on scan on import (#89213)
misc: add test to be sure depreciations are not checked on load (#89213)
depreciations: don't check depreciations on snapshot load (#89213)
depreciations: check them only during UI and API imports (#89213)
Historique
Mis à jour par Robot Gitea il y a 25 jours
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Lauréline Guérin
Lauréline Guérin (lguerin) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/1370
- Titre : depreciations: don't store job on scan on import (#89213)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/1370/files
Mis à jour par Emmanuel Cazenave il y a 25 jours
- Lié à Development #72093: Dépréciations - interdire l'import d'un objet avec une expression python ajouté
Mis à jour par Emmanuel Cazenave il y a 25 jours
C'est le ticket lié qui introduit le bug, je pense.
Mis à jour par Robot Gitea il y a 25 jours
- Statut changé de Solution proposée à En cours
- Assigné à changé de Lauréline Guérin à Emmanuel Cazenave
Emmanuel Cazenave (ecazenave) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/1372
- Titre : WIP: depreciations: check them only during UI and API imports (#89213)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/1372/files
Mis à jour par Robot Gitea il y a 25 jours
- Statut changé de Solution proposée à Solution validée
Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 25 jours
- Statut changé de Solution validée à Résolu (à déployer)
Thomas NOËL (tnoel) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/1370
- Titre : depreciations: don't store job on scan on import (#89213)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/1370/files
Mis à jour par Thomas Noël il y a 25 jours
- Statut changé de Résolu (à déployer) à Solution proposée
Ok pour le PR de Lauréline pour éviter les fichiers sur les "faux AfterJobs". Reste ensuite à valider la PR d'Emmanuel pour ne pas faire de check des dépréciations en dehors des imports de nouveaux objets.
Ensuite, un hotfix sur v11.13 avec le tout.
Mis à jour par Robot Gitea il y a 25 jours
- Statut changé de Solution proposée à Solution validée
Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 25 jours
- Statut changé de Solution validée à Résolu (à déployer)
Thomas NOËL (tnoel) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/1372
- Titre : depreciations: check them only during UI and API imports (#89213)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/1372/files
Mis à jour par Transition automatique il y a 24 jours
- Statut changé de Résolu (à déployer) à Solution déployée
depreciations: don't store job on scan on import (#89213)