envoyproxy/envoy

admin: making a streaming version of /clusters

Open

#32.055 aperta il 25 gen 2024

Vedi su GitHub
 (10 commenti) (1 reazione) (1 assegnatario)C++ (5373 fork)batch import
area/adminarea/perfenhancementhelp wanted

Metriche repository

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

Descrizione

/clusters can produce a remarkably huge response. This will need to be fully buffered as an in-memory continuous string. For large configs, this will be slow and may exert memory pressure as well.

See also #31755 as well as prior work streaming admin stats in https://github.com/envoyproxy/envoy/pull/19693. This work is not trivial and has risks. See also the difficult attempt to make /stats/prometheus stream as well: https://github.com/envoyproxy/envoy/pull/24998 which needed to be rolled back due to complexities in the Prom format that made streaming difficult.

See also #32054 which is a similar request for /config_dump

Guida contributor

Direzione di ricerca
Esamina l'implementazione corrente dell'endpoint /clusters e comprendi come costruisce la risposta completa. Rivedi il lavoro precedente sullo streaming nella PR #19693 e nota le difficoltà incontrate nella PR #24998. Identifica le strutture dati e i pattern di serializzazione che devono essere modificati per supportare lo streaming.
Tech stack
cpp
Dominio
backendinfrastructurenetworking
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Oltre 1 settimana
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 internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
20