apache/openwhisk

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

Open

#4.798 aberto em 12 de jan. de 2020

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)Scala (1.177 forks)batch import
good first issue

Métricas do repositório

Stars
 (6.777 stars)
Métricas de merge de PR
 (Mesclagem média 17h 31m) (3 fundiu PRs em 30d)

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)

Guia do colaborador

Direção de pesquisa
A mensagem de erro é gerada no código do controlador, provavelmente no arquivo que trata dos limites de concorrência (por exemplo, LoadBalancer ou ActivationThrottle). O problema sugere alterar o campo 'allowed' para refletir o limite actionInvokesConcurrent definido pelo usuário ou o total permitido entre os controladores. Uma correção deve atualizar a string da mensagem de erro para esclarecer que a contagem permitida é por controlador, ou calcular o total permitido. A variável 'allowed' está atualmente definida como o limite por controlador; uma abordagem melhor pode ser exibir tanto o limite por controlador quanto o limite total. Como não há PRs ou comentários vinculados, é necessária a contribuição do mantenedor para confirmar a abordagem desejada.
Pilha de tecnologia
scala
Domain
backendcloud
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ScalaGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80