entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 14 Releases Activity

logger en dehors des scopes atomic() (#31204) #203

Merged
bdauvergne merged 1 commits from wip/31204--es-requetes-HTTP-dans-des-blocs into main 2023-10-19 10:10:59 +02:00
Conversation 2 Commits 1 Files Changed 5 +288 -1
bdauvergne commented 2023-04-14 10:38:31 +02:00
Owner
Copy Link
No description provided.
bdauvergne force-pushed wip/31204--es-requetes-HTTP-dans-des-blocs from f7c6d55463 to fda39c7c31 2023-04-14 10:38:48 +02:00 Compare
bdauvergne force-pushed wip/31204--es-requetes-HTTP-dans-des-blocs from fda39c7c31 to 708cbca102 2023-04-14 10:40:34 +02:00 Compare
bdauvergne force-pushed wip/31204--es-requetes-HTTP-dans-des-blocs from 708cbca102 to 9623d989a9 2023-04-14 11:40:02 +02:00 Compare
bdauvergne force-pushed wip/31204--es-requetes-HTTP-dans-des-blocs from 9623d989a9 to 810c13e07e 2023-04-14 12:00:31 +02:00 Compare
bdauvergne changed title from WIP: logger en dehors des scopes atomic() (#31204) to logger en dehors des scopes atomic() (#31204) 2023-04-14 12:07:31 +02:00
bdauvergne force-pushed wip/31204--es-requetes-HTTP-dans-des-blocs from 1b788c0d6b to e38114f24d 2023-04-14 13:09:24 +02:00 Compare
bdauvergne force-pushed wip/31204--es-requetes-HTTP-dans-des-blocs from e38114f24d to 50928a419c 2023-04-20 13:34:24 +02:00 Compare
bdauvergne force-pushed wip/31204--es-requetes-HTTP-dans-des-blocs from 50928a419c to 6f8adfe569 2023-04-20 14:30:55 +02:00 Compare
bdauvergne force-pushed wip/31204--es-requetes-HTTP-dans-des-blocs from 6f8adfe569 to d2959f47d3 2023-10-16 12:28:40 +02:00 Compare
nroche reviewed 2023-10-17 11:10:11 +02:00
tests/test_proxylogger.py Outdated
@ -391,0 +425,4 @@
pass
assert qs.count() == 0
# with run_later_scope logs of level at least WARNING are kept
nroche commented 2023-10-17 11:10:11 +02:00
Owner
Copy Link

Pourtant la qs contient 2 logs : warning et info.
(comme je ne comprend pas le commentaire, j'ai testé en envoyant debug, info, warning et error que j'ai tous récupérés)

Pourtant la qs contient 2 logs : warning et info. (comme je ne comprend pas le commentaire, j'ai testé en envoyant debug, info, warning et error que j'ai tous récupérés)
bdauvergne commented 2023-10-17 12:19:35 +02:00
Author
Owner
Copy Link

Je vais retirer le "at least WARNING" c'est vieux je ne sais pas si ça date d'une version précédente du patch ou si ça veut dire autre chose.

Je vais retirer le "at least WARNING" c'est vieux je ne sais pas si ça date d'une version précédente du patch ou si ça veut dire autre chose.
bdauvergne commented 2023-10-17 12:30:13 +02:00
Author
Owner
Copy Link

Voilà j'ai changé le commentaire en quelque chose de plus explicite décrivant le comportement attendu.

Voilà j'ai changé le commentaire en quelque chose de plus explicite décrivant le comportement attendu.
nroche marked this conversation as resolved
nroche reviewed 2023-10-17 11:26:54 +02:00
passerelle/utils/defer.py Outdated
@ -0,0 +41,4 @@
for func, args, kwargs in run_later:
func(*args, **kwargs)
else:
yield
nroche commented 2023-10-17 11:26:54 +02:00
Owner
Copy Link

J'ai du mal à comprendre comment on passe ici.
Est-ce que tu pourrais ajouter un tests pour avoir la couverture à 100%, il manque juste cette ligne.

J'ai du mal à comprendre comment on passe ici. Est-ce que tu pourrais ajouter un tests pour avoir la couverture à 100%, il manque juste cette ligne.
bdauvergne commented 2023-10-17 12:20:48 +02:00
Author
Owner
Copy Link

D'ac, ça arrive si on a deux context manager run_later_scope() imbriqués, seul le premier a un effet et c'est lui qui lancera les commandes.

D'ac, ça arrive si on a deux context manager run_later_scope() imbriqués, seul le premier a un effet et c'est lui qui lancera les commandes.
bdauvergne commented 2023-10-17 12:32:32 +02:00
Author
Owner
Copy Link

Ajout de test_run_later_nested.

Ajout de test_run_later_nested.
nroche commented 2023-10-17 19:33:23 +02:00
Owner
Copy Link

Code testé à 100%, c'est la classe.
https://jenkins.entrouvert.org/job/gitea/job/passerelle/job/wip%252F31204--es-requetes-HTTP-dans-des-blocs/12/cobertura/utils/defer_py/

Code testé à 100%, c'est la classe. https://jenkins.entrouvert.org/job/gitea/job/passerelle/job/wip%252F31204--es-requetes-HTTP-dans-des-blocs/12/cobertura/utils/defer_py/
nroche marked this conversation as resolved
nroche reviewed 2023-10-17 11:33:08 +02:00
passerelle/utils/defer.py Outdated
@ -0,0 +32,4 @@
try:
run_later_context.get()
except LookupError:
run_later = []
nroche commented 2023-10-17 11:33:08 +02:00
Owner
Copy Link

Ci-dessous on a une fonction nommée pareil.
Peut-être nommer la variable "run_later_list" comme c'est fait plus bas dans la fonction en question.

Ci-dessous on a une fonction nommée pareil. Peut-être nommer la variable "run_later_list" comme c'est fait plus bas dans la fonction en question.
bdauvergne commented 2023-10-17 12:19:58 +02:00
Author
Owner
Copy Link

Ok.

Ok.
bdauvergne commented 2023-10-17 12:33:34 +02:00
Author
Owner
Copy Link

J'ai renommé tout en "callbacks" c'est plus court et plus uniforme et évite de surcharger le terme "run_later".

J'ai renommé tout en "callbacks" c'est plus court et plus uniforme et évite de surcharger le terme "run_later".
nroche marked this conversation as resolved
nroche requested changes 2023-10-17 11:51:12 +02:00
nroche left a comment
Owner
Copy Link

Désolé Benjamin, je suis pas super à l'aise pour relire :
j'ai posé des questions/commentaires un peu con, bien que dans l'ensemble tout m'a l'air bon.

Ma compréhension :
Ce patch a 2 points d'entrées, dans les commandes et dans le middleware,
pour initialiser un contexte qui permet de différer la création de l'objet ResourceLog.

Désolé Benjamin, je suis pas super à l'aise pour relire : j'ai posé des questions/commentaires un peu con, bien que dans l'ensemble tout m'a l'air bon. Ma compréhension : Ce patch a 2 points d'entrées, dans les commandes et dans le middleware, pour initialiser un contexte qui permet de différer la création de l'objet ResourceLog.
bdauvergne force-pushed wip/31204--es-requetes-HTTP-dans-des-blocs from d2959f47d3 to 0a51bb5a93 2023-10-17 12:29:40 +02:00 Compare
bdauvergne force-pushed wip/31204--es-requetes-HTTP-dans-des-blocs from 0a51bb5a93 to df56402517 2023-10-17 12:32:47 +02:00 Compare
bdauvergne commented 2023-10-17 12:36:36 +02:00
Author
Owner
Copy Link

Ce patch a 2 points d'entrées, dans les commandes et dans le middleware,
pour initialiser un contexte qui permet de différer la création de l'objet ResourceLog.

Oui.

> Ce patch a 2 points d'entrées, dans les commandes et dans le middleware, > pour initialiser un contexte qui permet de différer la création de l'objet ResourceLog. Oui.
bdauvergne requested review from nroche 2023-10-17 12:45:18 +02:00
nroche commented 2023-10-17 19:35:41 +02:00
Owner
Copy Link

C'est tout bon pour moi.
Si quelqu'un veut ajouter une relecture, faites, sinon je valide.

C'est tout bon pour moi. Si quelqu'un veut ajouter une relecture, faites, sinon je valide.
nroche approved these changes 2023-10-18 14:40:40 +02:00
bdauvergne force-pushed wip/31204--es-requetes-HTTP-dans-des-blocs from df56402517 to f63e250e0d 2023-10-19 09:59:12 +02:00 Compare
bdauvergne merged commit f63e250e0d into main 2023-10-19 10:10:59 +02:00
bdauvergne deleted branch wip/31204--es-requetes-HTTP-dans-des-blocs 2023-10-19 10:10:59 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
nroche
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/passerelle#203
No description provided.
Delete Branch "wip/31204--es-requetes-HTTP-dans-des-blocs"

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?