Skip to content

Commit 2f0cb3a

Browse files
authored
fix(eap): Update attribute key name provided in DeleteTraceItemsRequest RPCs (#103867)
Resolves [SNUBA-9Y9](https://sentry.sentry.io/issues/7052540005?project=300688). Resolves [SENTRY-5DCB](https://sentry.sentry.io/issues/7049009648?project=1). Testing locally confirms that the request is accepted by Snuba after correcting the name on the `AttributeKey`.
1 parent 90df829 commit 2f0cb3a

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/sentry/eventstream/eap.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ def _create_group_id_filter(group_ids: list[int]) -> TraceItemFilter:
5959
comparison_filter=ComparisonFilter(
6060
key=AttributeKey(
6161
type=AttributeKey.TYPE_INT,
62-
name="sentry.group_id",
62+
name="group_id",
6363
),
6464
op=ComparisonFilter.OP_IN,
6565
value=AttributeValue(val_int_array=IntArray(values=group_ids)),

tests/sentry/eventstream/test_eap.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ def test_deletion_with_error_dataset(self, mock_rpc):
3838
assert len(request.filters) == 1
3939
assert request.filters[0].item_type == TraceItemType.TRACE_ITEM_TYPE_OCCURRENCE
4040
assert request.filters[0].filter.HasField("comparison_filter")
41-
assert request.filters[0].filter.comparison_filter.key.name == "sentry.group_id"
41+
assert request.filters[0].filter.comparison_filter.key.name == "group_id"
4242
assert (
4343
list(request.filters[0].filter.comparison_filter.value.val_int_array.values)
4444
== self.group_ids
@@ -60,7 +60,7 @@ def test_multiple_group_ids(self, mock_rpc):
6060
request = mock_rpc.call_args[0][0]
6161
group_filter = request.filters[0].filter
6262
assert group_filter.HasField("comparison_filter")
63-
assert group_filter.comparison_filter.key.name == "sentry.group_id"
63+
assert group_filter.comparison_filter.key.name == "group_id"
6464
assert list(group_filter.comparison_filter.value.val_int_array.values) == many_group_ids
6565

6666
@patch("sentry.eventstream.eap.snuba_rpc.delete_trace_items_rpc")

0 commit comments

Comments
 (0)