Project

General

Profile

Bug #89213

création inutile de fichiers dans afterjobs/

Added by Thomas Noël about 1 month ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Haut
Target version:
-
Start date:
05 April 2024
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

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)

Related issues

Related to w.c.s. - Development #72093: Dépréciations - interdire l'import d'un objet avec une expression pythonSolution déployée06 December 2022

Actions

Associated revisions

Revision 39fed220 (diff)
Added by Lauréline Guérin about 1 month ago

depreciations: don't store job on scan on import (#89213)

Revision ff5299b7 (diff)
Added by Lauréline Guérin about 1 month ago

misc: add test to be sure depreciations are not checked on load (#89213)

Revision e8cd2aa8 (diff)
Added by Lauréline Guérin about 1 month ago

depreciations: don't check depreciations on snapshot load (#89213)

Revision 9ff89e41 (diff)
Added by Emmanuel Cazenave about 1 month ago

depreciations: check them only during UI and API imports (#89213)

Revision 9010c84c (diff)
Added by Lauréline Guérin about 1 month ago

depreciations: don't store job on scan on import (#89213)

Revision 4d4d6577 (diff)
Added by Lauréline Guérin about 1 month ago

misc: add test to be sure depreciations are not checked on load (#89213)

Revision 8c3c7ed7 (diff)
Added by Lauréline Guérin about 1 month ago

depreciations: don't check depreciations on snapshot load (#89213)

Revision 6c76a80f (diff)
Added by Emmanuel Cazenave about 1 month ago

depreciations: check them only during UI and API imports (#89213)

History

#1

Updated by Robot Gitea about 1 month ago

  • Status changed from Nouveau to Solution proposée
  • Assignee set to Lauréline Guérin (retour le 23/05)

Lauréline Guérin (lguerin) a ouvert une pull request sur Gitea concernant cette demande :

#2

Updated by Emmanuel Cazenave about 1 month ago

  • Related to Development #72093: Dépréciations - interdire l'import d'un objet avec une expression python added
#3

Updated by Emmanuel Cazenave about 1 month ago

C'est le ticket lié qui introduit le bug, je pense.

#4

Updated by Robot Gitea about 1 month ago

  • Status changed from Solution proposée to En cours
  • Assignee changed from Lauréline Guérin (retour le 23/05) to Emmanuel Cazenave

Emmanuel Cazenave (ecazenave) a ouvert une pull request sur Gitea concernant cette demande :

#6

Updated by Robot Gitea about 1 month ago

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

Updated by Robot Gitea about 1 month ago

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

Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :

#8

Updated by Robot Gitea about 1 month ago

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

Thomas NOËL (tnoel) a mergé une pull request sur Gitea concernant cette demande :

#9

Updated by Thomas Noël about 1 month ago

  • Status changed from Résolu (à déployer) to 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.

#10

Updated by Robot Gitea about 1 month ago

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

Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :

#11

Updated by Robot Gitea about 1 month ago

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

Thomas NOËL (tnoel) a mergé une pull request sur Gitea concernant cette demande :

#12

Updated by Transition automatique about 1 month ago

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

Also available in: Atom PDF