apache/openwhisk

"Too many concurrent requests in flight" error message is misleading

Open

#4 798 ouverte le 12 janv. 2020

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)Scala (1 177 forks)batch import
good first issue

Métriques du dépôt

Stars
 (6 777 stars)
Métriques de merge PR
 (Merge moyen 17h 31m) (3 PRs mergées en 30 j)

Description

@ali-raza-tariq commented on Fri Jan 10 2020

The existing implementation shows the following error message when inflight limit is exceeded

<Response [429]> b'{\n  "code": "BWm0N3Qemn4uMAcZWY0JCy8PjTAQ191A",\n  "error": "Too many concurrent requests in flight (count: 240, allowed: 240)."\n}'

Few suggestions:

The message is misleading because the allowed should show the existing actionInvokesConcurrent limit set by the user (in this case 1000- which got diluted to 1200-20% increase, because of more than 1 controller in deployment)

So either it should mention allowed per controller, or simply show the total allowed over all the controllers (in this case, 5 controllers (5*240=1200)

Also, It think it would be better to change the count to something more meaningful like currently running for easy debugging. (good first issue label)

Guide contributeur

Direction de recherche
Le message d'erreur est généré dans le code du contrôleur, probablement dans le fichier qui gère les limites de concurrence (par exemple LoadBalancer ou ActivationThrottle). Le problème suggère de modifier le champ 'allowed' pour refléter la limite actionInvokesConcurrent définie par l'utilisateur ou le total autorisé entre les contrôleurs. Un correctif doit mettre à jour la chaîne du message d'erreur pour clarifier que le nombre autorisé est par contrôleur, ou calculer le total autorisé. La variable 'allowed' est actuellement définie sur la limite par contrôleur ; une meilleure approche pourrait être d'afficher à la fois les limites par contrôleur et les limites totales. Comme il n'y a pas de PR ou de commentaires liés, l'avis du mainteneur est nécessaire pour confirmer l'approche souhaitée.
Stack technique
scala
Domaine
backendcloud
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ScalaGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80