envoyproxy/envoy

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

Open

#24.246 aperta il 29 nov 2022

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Esplora le attuali impostazioni di concorrenza nel framework di test di integrazione, quindi aggiungi un'opzione per configurare la concorrenza e imposta il valore predefinito su 1 o 2.
Tech stack
cpp
Dominio
backendtesting
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Envoy integration test framework
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40