entrouvert
/
hobo
16
0
Fork 0
Code Issues Pull Requests 16 Releases Activity

ne lever des erreurs au refresh du cache des templates que si ça dure depuis un bail (#86346) #106

Open
bdauvergne wants to merge 3 commits from wip/86346-Le-cache-de-RemoteTemplate-leve into main
pull from: wip/86346-Le-cache-de-RemoteTemplate-leve
merge into: entrouvert:main
Conversation 0 Commits 3 Files Changed 2 +257 -138
bdauvergne commented 2024-01-30 19:53:47 +01:00
Owner
Copy Link
No description provided.
bdauvergne added 2 commits 2024-01-30 19:53:48 +01:00
d9b6430bb4 context_processor: only trace if refresh is failing for 1 hour (#86346) 
Aslo capture network errors in the same error handling code.
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from 689dabaa76 to 079991b463 2024-01-30 23:08:53 +01:00 Compare
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from 079991b463 to f17952cc13 2024-01-30 23:09:50 +01:00 Compare
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from f17952cc13 to 0a7e5966bb 2024-01-30 23:40:09 +01:00 Compare
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from 0a7e5966bb to cae2ba9afa 2024-01-30 23:41:32 +01:00 Compare
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from cae2ba9afa to 9d9798a125 2024-01-30 23:44:18 +01:00 Compare
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from 9d9798a125 to ffcc1c16f3 2024-01-30 23:45:39 +01:00 Compare
bdauvergne changed title from WIP: ne lever des erreurs au refresh du cache des templates que si ça dure depuis un bail (#86346) to ne lever des erreurs au refresh du cache des templates que si ça dure depuis un bail (#86346) 2024-01-30 23:55:34 +01:00
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from ffcc1c16f3 to 1f533c08fa 2024-01-31 10:02:40 +01:00 Compare
tnoel reviewed 2024-02-02 10:09:25 +01:00
hobo/context_processors.py Outdated
@ -124,0 +122,4 @@
)
r.raise_for_status()
except requests.RequestException as e:
if last_update and datetime.datetime.now() - last_update < datetime.timedelta(hours=1):
tnoel commented 2024-02-02 10:09:25 +01:00
Owner
Copy Link

Juste pour m'assurer que je comprends : ce « if » nous permet tout de même de recevoir une alerte à la première panne (quand last_update n'existe pas encore), puis de les recevoir toutes après une heure, c'est bien l'idée ?

Juste pour m'assurer que je comprends : ce « if » nous permet tout de même de recevoir une alerte à la première panne (quand last_update n'existe pas encore), puis de les recevoir toutes après une heure, c'est bien l'idée ?
bdauvergne commented 2024-02-03 18:01:42 +01:00
Author
Owner
Copy Link

Oui, en cas d'erreur si le cache est vide c'est qu'on est dans le cas synchrone et il y aura encore l'exception + le log avec le statut de l'erreur. Ce qui maintenant que je j'y pense est un peu redondant, on pourrait aussi juste lancer l'exception si last_update est None et simplifier la logique, vu qu'on a on forcément in_thread=True si last_update!=None. Le but ici c'est de ne plus avoir d'erreur remontée par les mises à jour en tâche de fond (dans un thread) du cache, sauf si ça dure longtemps (à vrai dire sur des tenants peu touchés en test ça arrivera encore, idéalement il faudrait un compteur).

Oui, en cas d'erreur si le cache est vide c'est qu'on est dans le cas synchrone et il y aura encore l'exception + le log avec le statut de l'erreur. Ce qui maintenant que je j'y pense est un peu redondant, on pourrait aussi juste lancer l'exception si last_update est None et simplifier la logique, vu qu'on a on forcément in_thread=True si last_update!=None. Le but ici c'est de ne plus avoir d'erreur remontée par les mises à jour en tâche de fond (dans un thread) du cache, sauf si ça dure longtemps (à vrai dire sur des tenants peu touchés en test ça arrivera encore, idéalement il faudrait un compteur).
bdauvergne commented 2024-02-03 19:49:48 +01:00
Author
Owner
Copy Link

Donc dernière version, j'ai rendu explicite l'exception en cas d'absence de cache et d'erreur du GET, je suis passé à un dico parce que le tuple commençait à être long et j'ai ajouté l'histoire du compteur pour éviter les logs de tenants inutilisés.

Donc dernière version, j'ai rendu explicite l'exception en cas d'absence de cache et d'erreur du GET, je suis passé à un dico parce que le tuple commençait à être long et j'ai ajouté l'histoire du compteur pour éviter les logs de tenants inutilisés.
bdauvergne commented 2024-02-03 22:36:17 +01:00
Author
Owner
Copy Link

Aussi le dernier code prend en compte la migration des entrées de ache existantes vers le nouveau format (j'aurai pu m'en passer en modifiant les clés de cache aussi...). Chaque fois que j'écris un truc je pense à un autre... pfiou...

Aussi le dernier code prend en compte la migration des entrées de ache existantes vers le nouveau format (j'aurai pu m'en passer en modifiant les clés de cache aussi...). Chaque fois que j'écris un truc je pense à un autre... pfiou...
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from 1f533c08fa to 68f946cf8b 2024-02-03 19:47:51 +01:00 Compare
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from 68f946cf8b to 145f91402f 2024-02-03 19:50:53 +01:00 Compare
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from 145f91402f to 7d5fb7abee 2024-02-03 23:01:38 +01:00 Compare
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from 7d5fb7abee to bc67996b2a 2024-02-03 23:12:52 +01:00 Compare
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from bc67996b2a to b6c80721a6 2024-02-03 23:19:01 +01:00 Compare
bdauvergne force-pushed wip/86346-Le-cache-de-RemoteTemplate-leve from b6c80721a6 to 43c31aa2c4 2024-02-03 23:48:15 +01:00 Compare
bdauvergne requested review from tnoel 2024-02-21 10:21:45 +01:00
tnoel approved these changes 2024-04-26 15:01:57 +02:00
tnoel left a comment
Owner
Copy Link

Ouf. Tout ça pour ça :) Allons-y !

Ouf. Tout ça pour ça :) Allons-y !
All checks were successful
gitea/hobo/pipeline/head This commit looks good
Details
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b wip/86346-Le-cache-de-RemoteTemplate-leve main
git pull origin wip/86346-Le-cache-de-RemoteTemplate-leve

Step 2:

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff wip/86346-Le-cache-de-RemoteTemplate-leve
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/hobo#106
No description provided.
Delete Branch "wip/86346-Le-cache-de-RemoteTemplate-leve"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?