envoyproxy/envoy

Why envoy behaves differently when envoy adds an error tag to trace according to HTTP and GRPC status?

Aperta

#15.883 aperta il 8 apr 2021

Vedi su GitHubGuida contributor
 (2 commenti) (2 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/tracingenhancementhelp wanted

Metriche repository

Star
 (27.997 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Description: Envoy adds an error tag to trace when HTTP status is 5xx (NOT when HTTP status is 4xx). On the other hand, envoy adds an error tag to trace when GRPC status is not "OK".

I think envoy should not adds an error tag to trace when GRPC status is such as "NOT_FOUND" or "PERMISSION_DENIED" because they are equivalent to HTTP status 4xx.

Is there any intention for this difference?

Relevant Links: https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/observability/tracing#what-data-each-trace-contains

Guida contributor

Direzione di ricerca
Esamina l'implementazione del tracing di Envoy per comprendere la logica alla base dell'aggiunta di tag di errore basati su HTTP 5xx e stati gRPC non OK. Confronta con le convenzioni OpenTelemetry e suggerisci una modifica per allineare il tagging degli errori gRPC con la distinzione HTTP 4xx/5xx.
Tech stack
cpp
Dominio
backendobservability
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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++tracing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70