Project

General

Profile

Development #22682

nettoyer les fichiers orphelins et les objets oauth tempfile

Added by Benjamin Dauvergne over 1 year ago. Updated over 1 year ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
Start date:
21 Mar 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

La commande devra être lancée par un cron tous les soirs.

0001-add-a-fargo-cleanup-command-22682.patch View (11.3 KB) Benjamin Dauvergne, 23 Mar 2018 05:02 PM

0002-add-a-fargo-cleanup-command-22682.patch View (12.4 KB) Benjamin Dauvergne, 26 Mar 2018 03:37 PM

0001-oauth2-use-auto_now_add-for-creation_date-fields-226.patch View (2.32 KB) Benjamin Dauvergne, 26 Mar 2018 03:37 PM

0002-add-a-fargo-cleanup-command-22682.patch View (12.4 KB) Benjamin Dauvergne, 27 Mar 2018 11:20 AM

0001-oauth2-use-auto_now_add-for-creation_date-fields-226.patch View (2.32 KB) Benjamin Dauvergne, 27 Mar 2018 11:20 AM

Associated revisions

Revision 23124dbe (diff)
Added by Benjamin Dauvergne over 1 year ago

oauth2: use auto_now_add for creation_date fields (#22682)

Revision 7089a0fa (diff)
Added by Benjamin Dauvergne over 1 year ago

add a fargo-cleanup command (#22682)

Revision 2db8756f (diff)
Added by Benjamin Dauvergne over 1 year ago

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.

Revision 7846a6ab (diff)
Added by Benjamin Dauvergne over 1 year ago

dj11: replace deprecated symbols (#22682)

Revision 79fb2f8d (diff)
Added by Benjamin Dauvergne over 1 year ago

migrations: add default date in migration 0015 (#22682)

Revision 52929b1d (diff)
Added by Benjamin Dauvergne over 1 year ago

migrations: add default date in migration 0015 (#22682)

Revision 4ecdbada (diff)
Added by Benjamin Dauvergne over 1 year ago

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.

History

#1 Updated by Benjamin Dauvergne over 1 year ago

  • Subject changed from nettoyer les fichiers orphelins et les obets oauth tempfile to nettoyer les fichiers orphelins et les objets oauth tempfile

#2 Updated by Benjamin Dauvergne over 1 year ago

#3 Updated by Thomas Noël over 1 year ago

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
  • 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é)

#4 Updated by Benjamin Dauvergne over 1 year ago

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.

#6 Updated by Thomas Noël over 1 year ago

"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).

#8 Updated by Benjamin Dauvergne over 1 year ago

Je ne pense pas que ce soit la peine de spécifier ce qu'on nettoie, c'est forcément des trucs temporaires.

#9 Updated by Thomas Noël over 1 year ago

Ack

#10 Updated by Frédéric Péters over 1 year ago

  • Status changed from Nouveau to 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à.

#11 Updated by Benjamin Dauvergne over 1 year ago

Ça me va.

#12 Updated by Frédéric Péters over 1 year ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF