Lors de la migration des pjs, détruit les blobs qui n'existent pas dans l'espace de stockage

[LeSim]

No description provided.

[what-the-diff]

PR Summary

• Improved Robustness for pjs_migration_job.rb
  In order to improve the stability of our system, a safety measure has been introduced in the pjs_migration_job.rb file. This improvement addresses the case when the system encounters a Fog::OpenStack::Storage::NotFound error. The error handling procedure now involves getting rid of a specific temporary data unit, referred as blob, which potentially may have caused the problem, allowing the system to continue operating without interruption. This change means fewer disruptions and a smoother experience for our users.

[colinux]

Si le blob a encore un attachment, ça va provoquer ActiveRecord::InvalidForeignKey: ActiveRecord::InvalidForeignKey

A la place :
blob.attachments.destroy_all va enqueue les PurgeJob qui supprimera le blob, et sera NOOP en essayant de le supprimer sur le stockage vu qu'il n'existe pas

Par contre ça veut dire que théoriquement des dossiers deviendront invalides s'ils manquent une PJ obligatoire. En théorie ça pourrait même échouer si ça joue les validations sur le modèle attaché qui valident la présence d'un attachment

Je sais pas quel est le moins pire.

[LeSim]

Bien vu. Je tente le blob.attachments.destroy_all dans un premier temps.

On aura après une meilleure vue de combien de dossier sont invalides. et dans ce cas, je passe que je passerai avec un
blob.attachements.delete_all blob.delete

Qu'est ce que tu en penses ?

[colinux]

OK ça me paraît un bon plan, étant donné que les dossiers concernés devraient être assez anciens

[sentry-io]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ **NoMethodError: undefined method attachment_reflections' for nil:NilClass (NoMethodError)** PjsMigrationJob` View Issue

_{Did you find this useful? React with a 👍 or 👎}