envoyproxy/envoy

test framework: set default concurrency, and make concurrency an explicit option in integration test framework

Open

#24.246 aberto em 29 de nov. de 2022

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

Description

Title: test framework: set default concurrency, and make concurrency an explicit option in integration test framework

Description: Envoy by default starts a worker for every hardware thread, https://blog.envoyproxy.io/envoy-threading-model-a8d44b922310

for tests run on a beefy machine, this usually means either 64/72 or even 128 workers are started in an integration test.

I am not sure so many workers are desired, which may even cause noise in test. I think we should make it an explicit option to let each integration test to pick how many workers are started, and set the default to 1 or 2 if multiple workers are not desired for the test.

Guia do colaborador

Direção de pesquisa
Investigue as configurações atuais de concorrência no framework de testes de integração, depois adicione uma opção para configurar a concorrência e defina o padrão para 1 ou 2.
Pilha de tecnologia
cpp
Domain
backendtesting
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Envoy integration test framework
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40