apache/openwhisk

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

Aperta

#4798 aperta il 12 gen 2020

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Scala (1177 fork)batch import
good first issue

Metriche repository

Star
 (6777 stelle)
Metriche merge PR
 (Merge medio 17h 31m) (3 PR mergiate in 30 g)

Descrizione

@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)

Guida contributor

Direzione di ricerca
Il messaggio di errore viene generato nel codice del controller, probabilmente nel file che gestisce i limiti di concorrenza (ad esempio LoadBalancer o ActivationThrottle). Il problema suggerisce di modificare il campo 'allowed' per riflettere il limite actionInvokesConcurrent impostato dall'utente o il totale consentito tra i controller. Una correzione dovrebbe aggiornare la stringa del messaggio di errore per chiarire che il conteggio consentito è per controller, o calcolare il totale consentito. La variabile 'allowed' è attualmente impostata sul limite per controller; un approccio migliore potrebbe essere visualizzare sia i limiti per controller che quelli totali. Poiché non ci sono PR o commenti collegati, è necessario l'input del maintainer per confermare l'approccio desiderato.
Tech stack
scala
Dominio
backendcloud
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ScalaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80