envoyproxy/envoy

Envoy /stats/prometheus?filter works only with statsd metrics

Open

#10.927 aperta il 24 apr 2020

Vedi su GitHub
 (6 commenti) (1 reazione) (0 assegnatari)C++ (5373 fork)batch import
area/statshelp wanted

Metriche repository

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

Descrizione

curl  'localhost:9901/stats/prometheus?filter=runtime.load_error'
# TYPE envoy_runtime_load_error counter
envoy_runtime_load_error{} 0

I was expecting envoy_runtime_load_error to be used when filtering metrics on the prometheus stats endpoint, but filtering on it returns nothing. Only statsd runtime.load_error works.

This forces a weird mapping on the monitoring side. We have to maintain a map of statsd:prometheus metric names or re-implement the reverse transformation on the monitoring side.

I'm willing to work on this if it's considered a bug and not expected behaviour. What I think is happening is that the filtering is done before the statsd->prometheus metric transformation -- and should be moved after.

Guida contributor

Direzione di ricerca
Traccia il percorso del codice per l'endpoint /stats/prometheus, concentrandoti sulla logica di filtraggio e sulla trasformazione dei nomi delle metriche. Identifica dove il filtro viene applicato prima della conversione del nome della metrica da statsd a prometheus, quindi proponi di spostare il filtro dopo la conversione.
Tech stack
cpp
Dominio
networkingapi
Tipo issue
Bug
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 internals
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70