Development #54497
log de debug des actions de workflow
0%
Description
Dans #54487#note-7 je note :
meilleure solution technique il y aurait sans doute à intégrer de base dans le statut un enregistrement "trace" des actions effectuées (TraceEvolutionPart, invisible, liste de (timestamp, référence à l'action)), pour ne pas avoir action par action à ouvrir et regarder la condition et se demander "ok au moment où l'évaluation a eu lieu, ça aurait donné quoi ?".
Fichiers
Demandes liées
Révisions associées
backoffice: add tracing information to inspect page (#54497)
Historique
Mis à jour par Frédéric Péters il y a presque 3 ans
- Lié à Development #38621: ne pas considérer l'info "erreur appel webservice" présente mais cachée dans l'historique, pour l'affichage "groupé" des statuts ajouté
Mis à jour par Frédéric Péters il y a presque 3 ans
- Fichier actions-tracing.png actions-tracing.png ajouté
Mis à jour par Frédéric Péters il y a presque 3 ans
- Fichier 0002-backoffice-add-tracing-information-to-inspect-page-5.patch 0002-backoffice-add-tracing-information-to-inspect-page-5.patch ajouté
- Fichier 0001-workflows-add-actions-tracing-54497.patch 0001-workflows-add-actions-tracing-54497.patch ajouté
- Fichier tracing.png tracing.png ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
0001 pour ajouter le tracing, en gros paramètre event passé à perform_items() et celui-ci en enregistre les nfos dans un objet ActionsTracingEvolutionPart qui se trouve ajouté à l'historique.
0002 pour exploiter ça pour afficher une nouvelle section dans l'inspect d'une demande; ça donne la capture attachée; ça ne va pas chercher le détail du workflow pour afficher des lignes détaillées (ex: "courriel vers destinataire"), ça prend juste le libellé de l'action + un lien sur base de l'id de l'action si on veut voir les détails.
Mis à jour par Thomas Noël il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Petite typo dans 0002 dans les label des événements (:wcs/workflows.py) "(by APIà" au lieu de "(by API)"
Un peu peur de l'affichage qui risque d'être très très long (en taille) sur certaines demandes (celles qui testent un statut sur une appli tierce chaque heure pendant 2 semaines), mais allons-y ainsi, on verra à l'usage.
Mini détail, 0001 : peut-être que le décorateur mark_empty et son usage (le "continue" qui évite d'exécuter l'action) pourrait être dans un premier mini patch séparé. Pendant que j'y suis j'aurai bien aimé que ça s'appelle "noop_mark" juste parce que j'aime bien le mot noop.
Voilà bon bref ça roule (modulo la typo "by API").
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
J'ai renommé avec noop, mais je l'ai laissé dans le même commit,
commit edaf3946f26c33eb9eeaec62d8314f7e0a6ceb3c Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Jun 1 22:25:07 2021 +0200 backoffice: add tracing information to inspect page (#54497) commit 622a2cb5b35d1ef6a5bbe911dc57432dd995df43 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Jun 1 21:17:12 2021 +0200 workflows: add actions tracing (#54497)
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
workflows: add actions tracing (#54497)