envoyproxy/envoy

tracing: Set span.kind tag for opencensus tracer when creating client/server spans

Open

#12.538 geöffnet am 7. Aug. 2020

Auf GitHub ansehen
 (11 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/tracingenhancementhelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

Currently OpenTracing tracers and the zipkin tracer derives the span.kind (or cs/cr/ss/sr annotations) from the tracing config "operation_name" - however opencensus tracer doesn't have this tag set.

The proxy needs to explicitly set this tag when creating the span. This is similiar to #3435 which was resolved in 42a9cae38dd3bf5264fb83f64e310b0d76ea4257 (but this time just for opencensus)

Contributor Guide

Research-Richtung
Untersuchen Sie den Fix für OpenTracing im Commit 42a9cae38dd3bf5264fb83f64e310b0d76ea4257 und wenden Sie einen ähnlichen Ansatz auf den OpenCensus Tracer an. Finden Sie die Stellen, an denen im OpenCensus Tracer Spans erstellt werden, und fügen Sie Logik hinzu, um span.kind basierend auf dem Operationsnamen (Client/Server) zu setzen.
Tech Stack
cpp
Domain
networkingbackend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++Envoy conceptsTracing concepts
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60