Développement #101573
Ignorer les sauts automatiques dont la condition est de manière évidente toujours fausse
0%
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
History
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 :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2142
- Titre : WIP: misc: skip jump item with timeout and always false condition (#101573)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/2142/files
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 :
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 :
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 :
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 :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/2142
- Titre : misc: skip jump item with timeout and always false condition (#101573)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/2142/files
Updated by Transition automatique 4 days ago
- Status changed from Résolu (à déployer) to Solution déployée
misc: skip jump item with timeout and always false condition (#101573)