Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

cleanup(storage): remove deprecated OpenTelemetry rpc.message.type and rpc.message.id attributes

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#16,544 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
76/100
Issue-Typ
Refactoring
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
cpp
Bereich
backend

Rechercherichtung

Prüfe die fünf aufgeführten Dateien unter google/cloud/storage/internal/async/ und beginne mit den Tracing-Implementierungen und ihren Tests. Suche nach Verwendungen von rpc.message.type und rpc.message.id und entferne oder bereinige sie, da der Issue besagt, dass die Attribute veraltet sind und es keinen Ersatz gibt. Führe die Tracing-Tests für Reader und Writer aus, um das aktualisierte Verhalten zu überprüfen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

api: storage type: cleanup

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())
  • Unlike other /*sc::...=*/ string literals across the codebase, we intentionally did not migrate these to opentelemetry::semconv::rpc::kRpcMessageType / kRpcMessageId because doing so triggers -Wdeprecated-declarations compiler warnings when building against opentelemetry-cpp >= 1.26.0.

Affected Files

  • google/cloud/storage/internal/async/object_descriptor_reader_tracing.cc
  • google/cloud/storage/internal/async/reader_connection_tracing.cc
  • google/cloud/storage/internal/async/reader_connection_tracing_test.cc
  • google/cloud/storage/internal/async/writer_connection_tracing.cc
  • google/cloud/storage/internal/async/writer_connection_tracing_test.cc
Vorherrschende Sprache
C++
Sterne
659
Forks
471
Ø Merge
1 T. 10 Std.
Gemergte PRs (30 T.)
100

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus googleapis/google-cloud-cpp

Alle Issues in googleapis/google-cloud-cpp

Ähnliche Issues

Weitere Issues zu C++

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.