Project

General

Profile

Développement #101573

Ignorer les sauts automatiques dont la condition est de manière évidente toujours fausse

Added by Benjamin Dauvergne 8 days ago. Updated 4 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
03 February 2025
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Dans wcs.wf.jump._apply_timeouts() la première étape et d'accumuler les statuts qui ont des actions de saut automatique avec expiration, dans workflows_with_timeout().

Dans le cas où ce saut automatique a une condition qui est de manière évidente ("statiquement" évaluable à faux) toujours fausse, on pourrait ignorer complètement ce saut.

En production actuellement je peux mesurer les choses suivantes (via un script reprenant le code de _apply_timeouts mais sans exécuter le workflow et exécuté toutes les 20 minutes en production, ici il y a 4 exécution donc une période de d'1h20).

Tenant Slug Status ID Conditions Min Delay Cron runs Formdata Count Jump Cond. Ok Temps total Temps select_iterator() Tempps check_condition()
formulaires.demarches.essonne.fr calcul-du-tarif-de-la-res…laire-au-college-2024-2025 wf-3
false
345600 4 55082 190988 (47747 p. cron run) 0 (0 p. cron run) 162.792 (40.698 p. cron run) 70.198 (17.549 p. cron run) 92.595 (23.149 p. cron run)
formulaires.simplici.grandchambery.fr souscrire-un-abonnement-au-service-des-eaux wf-85
False
1728000 4 11575 16712 (4178 p. cron run) 0 (0 p. cron run) 16.670 (4.168 p. cron run) 7.508 (1.877 p. cron run) 9.162 (2.291 p. cron run)
formulaires.mes-demarches.marseille.fr demande-d-intervention wf-23
False
20736000 4 26827 12856 (3214 p. cron run) 0 (0 p. cron run) 12.310 (3.078 p. cron run) 6.418 (1.604 p. cron run) 5.892 (1.473 p. cron run)
formulaires.demarches.rouen.fr eval-pedt-20192024-ecole wf-6
true == false
4800 4 951 3804 (951 p. cron run) 0 (0 p. cron run) 2.705 (0.676 p. cron run) 1.184 (0.296 p. cron run) 1.521 (0.380 p. cron run)
formulaires.demarches.essonne.fr calcul-du-tarif-de-la-res…-college-2023-2024-copie-3 wf-3
false
345600 4 586 1992 (498 p. cron run) 0 (0 p. cron run) 1.723 (0.431 p. cron run) 0.757 (0.189 p. cron run) 0.966 (0.242 p. cron run)
formulaires.demarches.rouen.fr changement-badge-secteur-pietionnier wf-2
true == false
4800 4 388 1464 (366 p. cron run) 0 (0 p. cron run) 0.817 (0.204 p. cron run) 0.256 (0.064 p. cron run) 0.560 (0.140 p. cron run)
demarches.strasbourg.eu ops-concerts-scolaires wf-3
false
5184000 4 794 828 (207 p. cron run) 0 (0 p. cron run) 0.458 (0.114 p. cron run) 0.113 (0.028 p. cron run) 0.344 (0.086 p. cron run)
formulaires.demarches.rouen.fr benevole wf-3
true == false
4800 4 184 368 (92 p. cron run) 0 (0 p. cron run) 0.223 (0.056 p. cron run) 0.051 (0.013 p. cron run) 0.172 (0.043 p. cron run)
demarches.toodego.com suivi-dossier-ph wf-5
False
2419200 4 252151 348 (87 p. cron run) 0 (0 p. cron run) 0.350 (0.088 p. cron run) 0.104 (0.026 p. cron run) 0.247 (0.062 p. cron run)
formulaires.demarches.rouen.fr eval-pedt-20192024-creche wf-6
true == false
4800 4 71 284 (71 p. cron run) 0 (0 p. cron run) 0.206 (0.051 p. cron run) 0.099 (0.025 p. cron run) 0.106 (0.027 p. cron run)
formulaires.demarches.rouen.fr speed-dating-benevolat wf-5
true == false
4800 4 48 188 (47 p. cron run) 0 (0 p. cron run) 0.090 (0.022 p. cron run) 0.017 (0.004 p. cron run) 0.073 (0.018 p. cron run)
formulaires.demarches.rouen.fr demande-de-vignette-pour-…residents-de-l-ile-lacroix wf-2
true == false
4800 4 47 136 (34 p. cron run) 0 (0 p. cron run) 0.074 (0.018 p. cron run) 0.022 (0.005 p. cron run) 0.052 (0.013 p. cron run)
formulaires.demarches.rouen.fr changement-badge-secteur-pietionnier wf-0
true == false
4800 4 388 88 (22 p. cron run) 0 (0 p. cron run) 0.055 (0.014 p. cron run) 0.021 (0.005 p. cron run) 0.034 (0.008 p. cron run)
formulaires.demarches.essonne.fr adherer-a-la-charte-des-j…jardins-naturels-sensibles wf-2
false
4800 4 119 36 (9 p. cron run) 0 (0 p. cron run) 0.065 (0.016 p. cron run) 0.045 (0.011 p. cron run) 0.020 (0.005 p. cron run)
formulaires.demarches.rouen.fr benevole wf-0
true == false
4800 4 184 36 (9 p. cron run) 0 (0 p. cron run) 0.022 (0.005 p. cron run) 0.009 (0.002 p. cron run) 0.013 (0.003 p. cron run)
formulaires.demarches.lenord.fr fiches-iodas-avec-prise-de-rdv wf-10
false
10368000 4 153355 20 (5 p. cron run) 0 (0 p. cron run) 0.025 (0.006 p. cron run) 0.011 (0.003 p. cron run) 0.014 (0.003 p. cron run)
demarches-roanne.icitoyen.fr je-demande-une-subvention…-a-la-ville-de-roanne-2023 wf-9
false
6912000 4 144 16 (4 p. cron run) 0 (0 p. cron run) 0.027 (0.007 p. cron run) 0.019 (0.005 p. cron run) 0.008 (0.002 p. cron run)
formulaires.demarches.rouen.fr candidature-sainte-croix-des-pelletiers-2024 wf-3
true == false
4800 4 4 12 (3 p. cron run) 0 (0 p. cron run) 0.012 (0.003 p. cron run) 0.006 (0.002 p. cron run) 0.006 (0.001 p. cron run)
demarches-vaulx-en-velin.toodego.com voirie-et-signalisation wf-47
false
86400 4 669 4 (1 p. cron run) 0 (0 p. cron run) 0.018 (0.005 p. cron run) 0.010 (0.003 p. cron run) 0.008 (0.002 p. cron run)
demarches-vaulx-en-velin.toodego.com dechets-et-proprete wf-47
false
86400 4 1254 4 (1 p. cron run) 0 (0 p. cron run) 0.013 (0.003 p. cron run) 0.010 (0.003 p. cron run) 0.003 (0.001 p. cron run)
formulaires.mesdemarches.nicecotedazur.org je-soutiens-les-sinistres…s-de-l-archipel-de-mayotte wf-12
false
4800 4 37 4 (1 p. cron run) 0 (0 p. cron run) 0.015 (0.004 p. cron run) 0.008 (0.002 p. cron run) 0.007 (0.002 p. cron run)
formulaires.demarches.rouen.fr candidature-sainte-croix-des-pelletiers-2024 wf-0
true == false
4800 4 4 4 (1 p. cron run) 0 (0 p. cron run) 0.006 (0.001 p. cron run) 0.004 (0.001 p. cron run) 0.002 (0.000 p. cron run)
formulaires.demarches.rouen.fr speed-dating-benevolat wf-0
true == false
4800 4 48 4 (1 p. cron run) 0 (0 p. cron run) 0.007 (0.002 p. cron run) 0.005 (0.001 p. cron run) 0.002 (0.000 p. cron run)
formulaires.demarches.rouen.fr demande-de-vignette-pour-…residents-de-l-ile-lacroix wf-0
true == false
4800 4 47 4 (1 p. cron run) 0 (0 p. cron run) 0.007 (0.002 p. cron run) 0.005 (0.001 p. cron run) 0.002 (0.000 p. cron run)

Éviter le .select_iterator() et la série de .check_condition() serait bénéfique ici.

Associated revisions

Revision 766ec15f (diff)
Added by Benjamin Dauvergne 4 days ago

misc: skip jump item with timeout and always false condition (#101573)

History

#1

Updated by Robot Gitea 8 days ago

  • Status changed from Nouveau to En cours

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#2

Updated by Robot Gitea 8 days ago

  • Status changed from En cours to Solution proposée
#3

Updated by Robot Gitea 6 days ago

  • Status changed from Solution proposée to En cours

Yann Weber (yweber) a relu et demandé des modifications sur une pull request sur Gitea concernant cette demande :

#4

Updated by Robot Gitea 5 days ago

  • Status changed from En cours to Solution proposée

Benjamin Dauvergne (bdauvergne) a demandé une relecture de Yann Weber (yweber) sur une pull request sur Gitea concernant cette demande :

#5

Updated by Robot Gitea 5 days ago

  • Status changed from Solution proposée to Solution validée

Yann Weber (yweber) a approuvé une pull request sur Gitea concernant cette demande :

#6

Updated by Robot Gitea 4 days ago

  • Status changed from Solution validée to Résolu (à déployer)

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

#7

Updated by Transition automatique 4 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF