Projet

Général

Profil

Development #84610

lever l'alerte /live trop long uniquement si c'est le traitement des champs dynamiques qui prend du temps

Ajouté par Frédéric Péters il y a 5 mois. Mis à jour il y a 5 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
12 décembre 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Exemple ici,

     timings = {'relative_start': 5.2298994064331055, 'visibility': '0.0253', 'item-options-52': '0.0026', 'field-86': '0.0000', 'field-83': '0.0000', 'field-85': '0.0000', 'field-11': '0.0000', 'field-8': '0.0000', 'field-5': '0.0000', 'field-6': '0.0000', 'field-7': '0.0000', 'field-9': '0.0000', 'field-195': '0.0000', 'field-196': '0.0000', 'field-cdc657af-7bef-402d-94ad-e1452e954cc5': '0.0000', 'field-12': '0.0000', 'field-17': '0.0000', 'field-19': '0.0000', 'field-13': '0.0000', 'field-288': '0.0000', 'field-250': '0.0000', 'field-52': '0.0000', 'field-217': '0.0000', 'field-198': '0.0000', 'total': 5.257875919342041}

Le traitement des champs dynamiques commence alors qu'il y a déjà 5 secondes écoulées. C'est signe d'un truc à creuser mais pas au niveau des champs dynamiques et l'alerte devrait seulement y apparaitre si cette partie prend du temps.

Révisions associées

Révision 358ac61f (diff)
Ajouté par Frédéric Péters il y a 5 mois

misc: only consider /live processing in live timing alert (#84610)

Historique

#1

Mis à jour par Robot Gitea il y a 5 mois

  • Statut changé de Nouveau à En cours

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#3

Mis à jour par Robot Gitea il y a 5 mois

  • Statut changé de En cours à Solution proposée
#4

Mis à jour par Thomas Noël il y a 5 mois

Mais l'appel à /live a effectivement pris plus de 5 secondes, et donc côté usager le ressenti est que "ça ne marche pas". Non ?

#5

Mis à jour par Frédéric Péters il y a 5 mois

Oui mais l'alerte à l'admin fonctionnel devrait servir à pointer un problème autour de champs dynamiques, ce qui n'est pas le cas ici. (c'est un problème de dev à creuser pourquoi ça prend 5 secondes pour arriver jusque là, pas un problème à remonter dans l'interface où l'admin ne pourra rien faire, selon moi).

#6

Mis à jour par Robot Gitea il y a 5 mois

  • Statut changé de Solution proposée à Solution validée

Thomas NOËL (tnoel) a approuvé une pull request sur Gitea concernant cette demande :

#7

Mis à jour par Robot Gitea il y a 5 mois

  • Statut changé de Solution validée à Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#8

Mis à jour par Transition automatique il y a 5 mois

  • Statut changé de Résolu (à déployer) à Solution déployée
#9

Mis à jour par Transition automatique il y a 2 mois

Automatic expiration

Formats disponibles : Atom PDF