From 9a74ee0751905b98889efd0a81406dbda3b0790f Mon Sep 17 00:00:00 2001 From: getsentry-bot <10587625+getsentry-bot@users.noreply.github.com> Date: Fri, 29 May 2026 17:33:25 +0000 Subject: [PATCH 1/3] ref: bump sentry-protos to 0.17.0 Co-Authored-By: phacops <336345+phacops@users.noreply.github.com> --- pyproject.toml | 2 +- rust_snuba/Cargo.lock | 6 +++--- rust_snuba/Cargo.toml | 2 +- uv.lock | 6 +++--- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/pyproject.toml b/pyproject.toml index d79276d5ff..bfde28662e 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -31,7 +31,7 @@ dependencies = [ "sentry-arroyo>=2.39.2", "sentry-conventions>=0.8.0", "sentry-kafka-schemas>=2.1.24", - "sentry-protos>=0.8.14", + "sentry-protos>=0.17.0", "sentry-redis-tools>=0.5.1", "sentry-relay>=0.9.25", "sentry-sdk>=2.35.0", diff --git a/rust_snuba/Cargo.lock b/rust_snuba/Cargo.lock index 1bdbacce26..09d7e7725b 100644 --- a/rust_snuba/Cargo.lock +++ b/rust_snuba/Cargo.lock @@ -1734,7 +1734,7 @@ dependencies = [ "httpdate", "itoa", "pin-project-lite", - "socket2 0.4.10", + "socket2 0.5.6", "tokio", "tower-service", "tracing", @@ -3912,9 +3912,9 @@ dependencies = [ [[package]] name = "sentry_protos" -version = "0.7.0" +version = "0.17.0" source = "registry+https://github.com/rust-lang/crates.io-index" -checksum = "dcca2ce7135d2721bd635ff5a55ce8f3b124f854a643a1f8dfc82410b04487df" +checksum = "7b888d60a004c4c8af45bbecce631d0114bdbfc5cedb16958bec851d45cf1467" dependencies = [ "prost", "prost-types", diff --git a/rust_snuba/Cargo.toml b/rust_snuba/Cargo.toml index 70e647016f..af95366cea 100644 --- a/rust_snuba/Cargo.toml +++ b/rust_snuba/Cargo.toml @@ -55,7 +55,7 @@ sentry = { version = "0.41.0", default-features = false, features = [ ] } sentry-kafka-schemas = "2.1.24" sentry-options = "1.0.11" -sentry_protos = "0.7.0" +sentry_protos = "0.17.0" sentry_arroyo = { version = "2.39.2", features = ["ssl"] } sentry_usage_accountant = { version = "0.1.2", features = ["kafka"] } seq-macro = "0.3" diff --git a/uv.lock b/uv.lock index f979f84c76..fc853cb432 100644 --- a/uv.lock +++ b/uv.lock @@ -935,7 +935,7 @@ wheels = [ [[package]] name = "sentry-protos" -version = "0.8.14" +version = "0.17.0" source = { registry = "https://pypi.devinfra.sentry.io/simple" } dependencies = [ { name = "grpc-stubs", marker = "sys_platform == 'darwin' or sys_platform == 'linux'" }, @@ -943,7 +943,7 @@ dependencies = [ { name = "protobuf", marker = "sys_platform == 'darwin' or sys_platform == 'linux'" }, ] wheels = [ - { url = "https://pypi.devinfra.sentry.io/wheels/sentry_protos-0.8.14-py3-none-any.whl", hash = "sha256:109eb990d27c193ce7878fa3a37f3cb9bc4c9a57ad8d454ed8a8e6f104f0c21a" }, + { url = "https://pypi.devinfra.sentry.io/wheels/sentry_protos-0.17.0-py3-none-any.whl", hash = "sha256:e5a7c320678a6204cfa6e8a8981952e8dd1080131de076c5cec8702a24f62d7d" }, ] [[package]] @@ -1141,7 +1141,7 @@ requires-dist = [ { name = "sentry-arroyo", specifier = ">=2.39.2" }, { name = "sentry-conventions", specifier = ">=0.8.0" }, { name = "sentry-kafka-schemas", specifier = ">=2.1.24" }, - { name = "sentry-protos", specifier = ">=0.8.14" }, + { name = "sentry-protos", specifier = ">=0.17.0" }, { name = "sentry-redis-tools", specifier = ">=0.5.1" }, { name = "sentry-relay", specifier = ">=0.9.25" }, { name = "sentry-sdk", specifier = ">=2.35.0" }, From 24ec1c03be2c2df697a73e73da088c08771ff999 Mon Sep 17 00:00:00 2001 From: Pierre Massat Date: Fri, 29 May 2026 12:15:48 -0700 Subject: [PATCH 2/3] fix(eap): Handle ProcessingError variant in TraceItemType match sentry_protos 0.17.0 added TraceItemType::ProcessingError. Map it to "processing_errors" for COGS tracking. Co-Authored-By: Claude Opus 4.7 (1M context) --- rust_snuba/src/processors/eap_items.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/rust_snuba/src/processors/eap_items.rs b/rust_snuba/src/processors/eap_items.rs index e2f5f09aa4..21d2bcbdf3 100644 --- a/rust_snuba/src/processors/eap_items.rs +++ b/rust_snuba/src/processors/eap_items.rs @@ -134,6 +134,7 @@ fn process_eap_item(msg: KafkaPayload, config: &ProcessorConfig) -> anyhow::Resu TraceItemType::Attachment => "attachments", TraceItemType::Preprod => "preprod", TraceItemType::UserSession => "sessions", + TraceItemType::ProcessingError => "processing_errors", TraceItemType::Unspecified => "null", } .to_string(); From acb1540ca6b9ef63d7787af92bca824ae9dae637 Mon Sep 17 00:00:00 2001 From: Pierre Massat Date: Fri, 29 May 2026 12:25:37 -0700 Subject: [PATCH 3/3] fix(eap): Map TRACE_ITEM_TYPE_PROCESSING_ERROR in querylog COGS dict Mirrors the Rust write-path mapping so query-path COGS attribution for the new ProcessingError trace item type lands in "processing_errors" instead of falling back to "null". Co-Authored-By: Claude Opus 4.7 (1M context) --- snuba/querylog/__init__.py | 1 + 1 file changed, 1 insertion(+) diff --git a/snuba/querylog/__init__.py b/snuba/querylog/__init__.py index ccc4ef3da6..fcef4cccf7 100644 --- a/snuba/querylog/__init__.py +++ b/snuba/querylog/__init__.py @@ -35,6 +35,7 @@ "TRACE_ITEM_TYPE_ATTACHMENT": "attachments", "TRACE_ITEM_TYPE_PREPROD": "preprod", "TRACE_ITEM_TYPE_USER_SESSION": "sessions", + "TRACE_ITEM_TYPE_PROCESSING_ERROR": "processing_errors", "TRACE_ITEM_TYPE_UNSPECIFIED": "null", }