Development #22682
nettoyer les fichiers orphelins et les objets oauth tempfile
0%
Description
La commande devra être lancée par un cron tous les soirs.
Fichiers
Révisions associées
add a fargo-cleanup command (#22682)
fargo: prevent pushing the same file many times (#22682)
Bug was discovered because we added the creation_date non-null field as
content_hash is the primary key. When saving a Document with the same
content_hash, Django try to update the existing document by setting
creation_date to NULL which fails.
dj11: replace deprecated symbols (#22682)
migrations: add default date in migration 0015 (#22682)
migrations: add default date in migration 0015 (#22682)
fargo: add missing migration (#22682)
We set a default to create the new Document.creation_date field but it's
not part of the real models, this migration revert it.
Historique
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
- Sujet changé de nettoyer les fichiers orphelins et les obets oauth tempfile à nettoyer les fichiers orphelins et les objets oauth tempfile
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
- Fichier 0001-add-a-fargo-cleanup-command-22682.patch 0001-add-a-fargo-cleanup-command-22682.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 6 ans
- le created_date devrait sans doute plutôt utiliser auto_now_add que auto_now
- indentation sur base de TAB dans le tox.ini
- je suis pas sûr que related_name nécessite une "vraie" migration (pas de modif sql ?), on pourrait éventuellement la coller dans la migration précédente (mais je suis pas sûr de mon coup)
- max(settings.FARGO_CODE_LIFETIME, settings.FARGO_ACCESS_TOKEN_LIFETIME) : il faudrait poser un commentaire dans le settings.py qui dit que c'est la plus grande qui gagne ? (intuitivement j'aurai pensé à un "min" ici, s'agissant d'une durée de vie, mais j'ai pas bien pigé les conséquences de ce max, en vérité)
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
Thomas Noël a écrit :
Remarques de forme pendant que je cherche à comprendre le fond
- le created_date devrait sans doute plutôt utiliser auto_now_add que auto_now
Mais bien sûr, merci je corrige les deux autres qui ont le même problème dans un pre-patch.
- indentation sur base de TAB dans le tox.ini
Ok.
- je suis pas sûr que related_name nécessite une "vraie" migration (pas de modif sql ?), on pourrait éventuellement la coller dans la migration précédente (mais je suis pas sûr de mon coup)
Ok.
- max(settings.FARGO_CODE_LIFETIME, settings.FARGO_ACCESS_TOKEN_LIFETIME) : il faudrait poser un commentaire dans le settings.py qui dit que c'est la plus grande qui gagne ? (intuitivement j'aurai pensé à un "min" ici, s'agissant d'une durée de vie, mais j'ai pas bien pigé les conséquences de ce max, en vérité)
L'objet porte deux valeurs code et access_token, qui ont des durées de vie différentes, la durée vie du conteneur est forcément le max des deux, sinon on réduit de fait la durée de vie la plus longue à celle la plus courte.
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
- Fichier 0002-add-a-fargo-cleanup-command-22682.patch 0002-add-a-fargo-cleanup-command-22682.patch ajouté
- Fichier 0001-oauth2-use-auto_now_add-for-creation_date-fields-226.patch 0001-oauth2-use-auto_now_add-for-creation_date-fields-226.patch ajouté
Voili voilou.
Mis à jour par Thomas Noël il y a environ 6 ans
"Clean expired models of authentic2." : j'imagine que c'est un reste de copié collé, si tu peux mettre quelque chose de plus clair qui explique ce qui est nettoyé (sessions oauth, fichiers orphelins, ... liste à compléter si d'autres modèles ajoutent leur cleanup plus tard).
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
Je ne pense pas que ce soit la peine de spécifier ce qu'on nettoie, c'est forcément des trucs temporaires.
Mis à jour par Frédéric Péters il y a environ 6 ans
- Statut changé de Nouveau à Résolu (à déployer)
Et ça a été poussé et ça a foiré dans jenkins et il y a eu deux commits tapés derrière pour arranger les choses et voilà.
Mis à jour par Frédéric Péters il y a environ 6 ans
- Statut changé de Résolu (à déployer) à Fermé
oauth2: use auto_now_add for creation_date fields (#22682)