entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

wscalls: unify error reporting between wscall objects and actions (#13593) #601

Merged
fpeters merged 1 commits from wip/13593-wscall-errors into main 2023-11-24 16:19:17 +01:00
Conversation 0 Commits 1 Files Changed 4 +89 -40
fpeters commented 2023-08-13 21:04:39 +02:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/13593-wscall-errors from 2b17449679 to 4cd66937a8 2023-08-13 21:26:47 +02:00 Compare
fpeters changed title from WIP: wscalls: unify error reporting between wscall objects and actions (#14289) to wscalls: unify error reporting between wscall objects and actions (#14289) 2023-08-14 08:22:44 +02:00
pmarillonnet reviewed 2023-08-16 10:54:29 +02:00
wcs/wf/wscall.py Outdated
@ -550,20 +550,25 @@ class WebserviceCallStatusItem(WorkflowStatusItem):
self.notify_on_errors or self.record_on_errors or self.record_errors
):
if exception is None:
pmarillonnet commented 2023-08-16 10:54:29 +02:00
Owner
Copy Link

À lire le code de la nouvelle fonction record_wscall_error, je ne comprends pas pourquoi on ne va pas plus loin, par exemple en faisant en sorte que cette fonction puisse prendre une exception en argument par mot-clé.

Et que cette fonction gère elle-même les cas exception is None et inversement (plutôt que ce soit l’appelant qui gère ici cela et qui décide d’appeler record_wscall_error ou non).

En lisant le code le wcs/wf/wscall.py j’ai l’impression que les cas où ws.action_on_error() est appelée avec une exception relèvent quand même d’une erreur de l’appel WS en tant que tel, et donc qu’un recours à record_wscall_error() n’en serait pas un mésusage.

À lire le code de la nouvelle fonction `record_wscall_error`, je ne comprends pas pourquoi on ne va pas plus loin, par exemple en faisant en sorte que cette fonction puisse prendre une exception en argument par mot-clé. Et que cette fonction gère elle-même les cas `exception is None` et inversement (plutôt que ce soit l’appelant qui gère ici cela et qui décide d’appeler `record_wscall_error` ou non). En lisant le code le `wcs/wf/wscall.py` j’ai l’impression que les cas où `ws.action_on_error()` est appelée avec une exception relèvent quand même d’une erreur de l’appel WS en tant que tel, et donc qu’un recours à `record_wscall_error()` n’en serait pas un mésusage.
fpeters commented 2023-09-29 17:16:31 +02:00
Author
Owner
Copy Link

je ne comprends pas pourquoi on ne va pas plus loin

On y va à petits pas.

> je ne comprends pas pourquoi on ne va pas plus loin On y va à petits pas.
fpeters changed title from wscalls: unify error reporting between wscall objects and actions (#14289) to wscalls: unify error reporting between wscall objects and actions (#13593) 2023-09-04 12:06:01 +02:00
fpeters force-pushed wip/13593-wscall-errors from 4cd66937a8 to 3ef6e96733 2023-09-04 14:27:59 +02:00 Compare
fpeters force-pushed wip/13593-wscall-errors from 3ef6e96733 to de6f8ed1eb 2023-09-04 19:35:58 +02:00 Compare
fpeters force-pushed wip/13593-wscall-errors from de6f8ed1eb to fa82eaf51b 2023-09-29 16:40:50 +02:00 Compare
fpeters force-pushed wip/13593-wscall-errors from fa82eaf51b to 004dff3a8c 2023-10-07 19:07:52 +02:00 Compare
nroche approved these changes 2023-11-24 16:16:40 +01:00
fpeters merged commit 0e3dcdf2b2 into main 2023-11-24 16:19:17 +01:00
fpeters deleted branch wip/13593-wscall-errors 2023-11-24 16:19:17 +01: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
3 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/wcs#601
No description provided.
Delete Branch "wip/13593-wscall-errors"

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?