envoyproxy/envoy

Add the filtering parameters of the cluster in the cluster admin interface

Open

#14.702 aperta il 14 gen 2021

Vedi su GitHub
 (1 commento) (0 reazioni) (1 assegnatario)C++ (5373 fork)batch import
area/adminhelp wanted

Metriche repository

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

Descrizione

Title: Add the filtering parameters of the cluster in the cluster admin interface

Description: Current we curl clusters interface(localhost:15000/clusters) will return all clusters' stats , the response will be too large when the cluster is large. I want to add a cluster_name_prefix param to get only the statistics of the cluster I care about.I don’t know if this requirement is a public requirement, if you don’t mind if I add this feature, I will propose a pr to realize it

The final result is as follows

curl localhost:15000/clusters?cluster_prefix=test_service
test_service_1::127.0.0.1:15000::cx_active::0
test_service_1::127.0.0.1:15000::cx_connect_fail::0
test_service_2::127.0.0.1:15000::cx_active::0
test_service_2::127.0.0.1:15000::cx_connect_fail::0

Guida contributor

Direzione di ricerca
Esamina il codice sorgente dell'admin handler di Envoy per l'endpoint /clusters, probabilmente in source/server/admin/clusters handler.h/cc. La funzionalità richiede l'aggiunta di un parametro di query 'cluster prefix' e il filtraggio delle statistiche del cluster prima della restituzione. Rivedi gli admin handler esistenti per il pattern di parsing dei parametri di query.
Tech stack
Nessuno
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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++HTTPEnvoy basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70