cleanup(storage): remove deprecated OpenTelemetry rpc.message.type and rpc.message.id attributes
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 76/100
Direzione di ricerca
Esamina i cinque file elencati sotto google/cloud/storage/internal/async/, iniziando dalle implementazioni di tracing e dai relativi test. Cerca gli utilizzi di rpc.message.type e rpc.message.id e rimuovili o ripuliscili, perché l’issue dice che gli attributi sono obsoleti e non hanno sostituti. Esegui i test di tracing di reader e writer per verificare il comportamento aggiornato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Summary
Several async Cloud Storage tracing implementations currently record span events with the hardcoded string literals /*sc::kRpcMessageType=*/"rpc.message.type" and /*sc::kRpcMessageId=*/"rpc.message.id".
In OpenTelemetry Semantic Conventions v1.40.0 (introduced in opentelemetry-cpp v1.26.0 in <opentelemetry/semconv/incubating/rpc_attributes.h>), opentelemetry::semconv::rpc::kRpcMessageType and opentelemetry::semconv::rpc::kRpcMessageId were marked OPENTELEMETRY_DEPRECATED with:
"reason": "obsoleted""note": "Deprecated, no replacement at this time."
Because these semantic convention attributes are obsolete upstream with no replacement, we should remove or clean up our usage of "rpc.message.type" and "rpc.message.id" in google/cloud/storage/internal/async/.
Background
- These attributes were originally added/updated as string literals in:
- #14373 (
chore(otel): use newer messaging semantic conventions) - #15078 (
feat(storage): Add bidiread and bidi appendable write to the async client.) - #16118 (
fix(storage): Add telemetry tracing support for async stream Close())
- #14373 (
- Unlike other
/*sc::...=*/string literals across the codebase, we intentionally did not migrate these toopentelemetry::semconv::rpc::kRpcMessageType/kRpcMessageIdbecause doing so triggers-Wdeprecated-declarationscompiler warnings when building againstopentelemetry-cpp >= 1.26.0.
Affected Files
google/cloud/storage/internal/async/object_descriptor_reader_tracing.ccgoogle/cloud/storage/internal/async/reader_connection_tracing.ccgoogle/cloud/storage/internal/async/reader_connection_tracing_test.ccgoogle/cloud/storage/internal/async/writer_connection_tracing.ccgoogle/cloud/storage/internal/async/writer_connection_tracing_test.cc
- Lingua principale
- C++
- Stelle
- 659
- Fork
- 471
- Merge medio
- 1g 10h
- PR unite (30g)
- 100
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di googleapis/google-cloud-cpp
-
api: storage priority: p2 type: bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
googleapis/google-cloud-cpp#16529 ·
I maintainer di solito rispondono entro 1 giorno
-
type: feature request
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
googleapis/google-cloud-cpp#16193 ·
I maintainer di solito rispondono entro 1 giorno
-
api: storage type: cleanup
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
googleapis/google-cloud-cpp#16545 ·
I maintainer di solito rispondono entro 1 giorno
-
api: storage type: cleanup
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
googleapis/google-cloud-cpp#16404 ·
I maintainer di solito rispondono entro 1 giorno
-
cpp: generator type: cleanup
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
googleapis/google-cloud-cpp#16394 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di googleapis/google-cloud-cpp
Issue simili
-
c++ good first issue M2: audio plumbing no hardware needed tests
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
GageLawton/noise-cancellation#23 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
objectionary/eo-graphs#85 ·
-
bug derived types format I/O medium priority semantics
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Kindly pin CUDA 12.X as required toolkit for stable release v0.3.3 compilation in CUDA systemsAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 68/100
facebookresearch/habitat-sim#2645 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno