Evaluate `SpanStatus` against the sentry.status convention
メンテナーはふだん 3 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 25/100
- issue の種類
- 機能追加
- 明瞭さ
- 説明が足りない
- 活発さ
- 活発
- 技術スタック
- rust
調査の方向性
Start by reading the SpanStatus documentation linked in the issue, then compare it with the span protocol specification and Relay’s v1 and v2 status handling in the referenced files. The issue asks for a design decision across transaction serialization, future span-first support, and integrations such as Actix, Tower, and OpenTelemetry. Done means resolving those questions and documenting the chosen compatibility approach.
索引モデルが issue の本文から書いたものです。
説明
The sentry.status convention allows only "ok" and "error", and marks more specific errors set by older SDKs as deprecated. SpanStatus still uses the older set of 17 statuses (not_found, internal_error, ...) and has no error variant. Evaluate how SpanStatus should relate to the convention. This was split out of #1399, which only touches the sentry-tracing field.
Context from Relay and the conventions:
- The two-value status belongs to the span-first (v2) protocol. Relay's
SpanV2Status(ok,error, plus a catch-all variant) was added in getsentry/relay#4717 (2025-05-14) as a subset of the OTel statuses. A review thread there removedunsetfrom that set. - Relay still parses the transaction/v1
statuswith the legacy enum inrelay-base-schema/src/spans.rs. That enum accepts the 17 statuses plus the aliasessuccess,failureandunknown_error, but rejects"error". Its comment says it's stored as au8in Snuba/ClickHouse. - Relay converts v1 spans to v2 by mapping
oktookand every other status toerror(relay-spans/src/v1_to_v2.rs, added in getsentry/relay#5139). - The convention was added in getsentry/sentry-conventions#453 (2026-06-26). It describes
sentry.statusas set during ingestion: v2 spans can only beokorerror, while v1 spans have the longer list. - The span-first SDK spec sets the span
statusto either"ok"or"error", andsetStatusaccepts'ok' | 'error'(develop docs).
Open questions:
- Should
SpanStatuschange while the SDK sends transactions, given that Relay's transaction parser rejects"error"? - How should this tie in with span-first support in the Rust SDK?
- How should the existing variants and the integrations that set them (Actix, Tower, OpenTelemetry) be handled?
References #1398
via @szokeasaurusrex.
--
- 主要言語
- Rust
- スター
- 754
- フォーク
- 192
- 平均マージ
- 2日 23時間
- マージ済み PR（30日）
- 20
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
getsentry/sentry-rust のほかの issue
-
breaking change
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
getsentry/sentry-rust#1387 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
breaking change Rust
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
getsentry/sentry-rust#1386 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
breaking change Rust Task
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
getsentry/sentry-rust#1345 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
Pin Rust toolchainオープンRust Task
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
getsentry/sentry-rust#933 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
Support setting span status from tracing span fields対応中かも @sentry-junior が 2 日前に担当しました。 オープンFeature Rust Spans
難易度 4/5 3〜5日 初心者へのやさしさ 58/100
getsentry/sentry-rust#1398 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
getsentry/sentry-rust の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
NSPG13/agent-bounties#1749 ·
メンテナーはふだん 1 日以内に返信
-
area: web bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
kinqsradio/orbyn-engineering#227 ·
メンテナーはふだん 1 日以内に返信
-
agentic-workflows automation repo-assist
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
shanselman/winget-tui#483 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
lightonai/next-plaid#205 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
apache/datafusion-iceberg#100 ·
メンテナーはふだん 1 日以内に返信