remove get_stacktrace_string_with_metrics

Author: lobsterkatie

src/sentry/grouping/ingest/seer.py

@@ -21,7 +21,7 @@
     ReferrerOptions,
     event_content_has_stacktrace,
     filter_null_from_string,
-    get_stacktrace_string_with_metrics,
+    get_stacktrace_string,
     has_too_many_contributing_frames,
     killswitch_enabled,
     record_did_call_seer_metric,
@@ -215,9 +215,7 @@ def _has_empty_stacktrace_string(event: Event, variants: dict[str, BaseVariant])
     if contributing_component is not None and hasattr(contributing_component, "frame_counts"):
         logger_extra["frame_counts"] = contributing_component.frame_counts
 
-    stacktrace_string = get_stacktrace_string_with_metrics(
-        get_grouping_info_from_variants(variants)
-    )
+    stacktrace_string = get_stacktrace_string(get_grouping_info_from_variants(variants))
     if not stacktrace_string:
         if stacktrace_string == "":
             record_did_call_seer_metric(call_made=False, blocker="empty-stacktrace-string")
@@ -243,7 +241,7 @@ def get_seer_similar_issues(
 
     stacktrace_string = event.data.get(
         "stacktrace_string",
-        get_stacktrace_string_with_metrics(get_grouping_info_from_variants(variants)),
+        get_stacktrace_string(get_grouping_info_from_variants(variants)),
     )
 
     if not stacktrace_string:

src/sentry/seer/similarity/utils.py

@@ -255,16 +255,6 @@ def is_base64_encoded_frame(frame_dict: Mapping[str, Any]) -> bool:
     return base64_encoded
 
 
-def get_stacktrace_string_with_metrics(data: dict[str, Any]) -> str | None:
-    stacktrace_string = None
-    try:
-        stacktrace_string = get_stacktrace_string(data)
-    except Exception:
-        logger.exception("Unexpected exception in stacktrace string formatting")
-
-    return stacktrace_string
-
-
 def event_content_has_stacktrace(event: GroupEvent | Event) -> bool:
     # If an event has no stacktrace, there's no data for Seer to analyze, so no point in making the
     # API call. If we ever start analyzing message-only events, we'll need to add `event.title in

src/sentry/tasks/embeddings_grouping/utils.py

@@ -34,7 +34,7 @@
     ReferrerOptions,
     event_content_has_stacktrace,
     filter_null_from_string,
-    get_stacktrace_string_with_metrics,
+    get_stacktrace_string,
     has_too_many_contributing_frames,
 )
 from sentry.snuba.dataset import Dataset
@@ -373,7 +373,7 @@ def get_events_from_nodestore(
 
             if not has_too_many_contributing_frames(event, variants, ReferrerOptions.BACKFILL):
                 grouping_info = get_grouping_info_from_variants(variants)
-                stacktrace_string = get_stacktrace_string_with_metrics(grouping_info)
+                stacktrace_string = get_stacktrace_string(grouping_info)
 
             if not stacktrace_string:
                 invalid_event_group_ids.append(group_id)

tests/sentry/seer/similarity/test_utils.py

@@ -15,7 +15,6 @@
     _is_snipped_context_line,
     filter_null_from_string,
     get_stacktrace_string,
-    get_stacktrace_string_with_metrics,
     has_too_many_contributing_frames,
 )
 from sentry.testutils.cases import TestCase
@@ -796,7 +795,7 @@ def test_replace_file_with_module(self):
         exception = copy.deepcopy(self.BASE_APP_DATA)
         # delete filename from the exception
         del exception["app"]["component"]["values"][0]["values"][0]["values"][0]["values"][1]
-        stacktrace_string = get_stacktrace_string_with_metrics(exception)
+        stacktrace_string = get_stacktrace_string(exception)
         assert (
             stacktrace_string
             == 'ZeroDivisionError: division by zero\n  File "__main__", function divide_by_zero\n    divide = 1/0'