feat(issues): Add parameterization for traceparent (#93721)

These are currently parameterized as something like the following which
is unhelpful for grouping (redacted example):
`<int>-<md5>-0a0000000000abcd<int>`.

The regex here is intentionally strict and will only support the current
version (`00`) to help ensure this is conservative in matching.

The option for this is done outside of our normal path since we need the
`traceparent` parameterization to happen before `int`/`md5` in the order
of operations. Once this is rolled out, a follow-up PR will clean up
quite a bit of our experiment logic here.

See also: https://www.w3.org/TR/trace-context/#traceparent-header

Author: mrduncan

src/sentry/grouping/parameterization.py

@@ -94,6 +94,9 @@ def compiled_pattern(self) -> re.Pattern[str]:
             )
         """,
     ),
+    ParameterizationRegex(
+        name="traceparent", raw_pattern=r"""\b00-[0-9a-f]{32}-[0-9a-f]{16}-0[01]\b"""
+    ),
     ParameterizationRegex(
         name="uuid",
         raw_pattern=r"""\b[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}\b""",

src/sentry/grouping/strategies/message.py

@@ -37,6 +37,24 @@
     "bool",
 )
 
+REGEX_PATTERN_KEYS_WITH_TRACEPARENT = (
+    "email",
+    "url",
+    "hostname",
+    "ip",
+    "traceparent",
+    "uuid",
+    "sha1",
+    "md5",
+    "date",
+    "duration",
+    "hex",
+    "float",
+    "int",
+    "quoted_str",
+    "bool",
+)
+
 EXPERIMENT_PROJECTS = [  # Active internal Sentry projects
     1,
     11276,
@@ -71,11 +89,7 @@ def normalize_message_for_grouping(message: str, event: Event, share_analytics:
     if trimmed != message:
         trimmed += "..."
 
-    parameterizer = Parameterizer(
-        regex_pattern_keys=REGEX_PATTERN_KEYS, experiments=(UniqueIdExperiment,)
-    )
-
-    def _shoudl_run_experiment(experiment_name: str) -> bool:
+    def _should_run_experiment(experiment_name: str) -> bool:
         return bool(
             not is_self_hosted()
             and event.project_id
@@ -87,7 +101,16 @@ def _shoudl_run_experiment(experiment_name: str) -> bool:
             )
         )
 
-    normalized = parameterizer.parameterize_all(trimmed, _shoudl_run_experiment)
+    parameterizer = Parameterizer(
+        regex_pattern_keys=REGEX_PATTERN_KEYS, experiments=(UniqueIdExperiment,)
+    )
+    if _should_run_experiment("traceparent"):
+        parameterizer = Parameterizer(
+            regex_pattern_keys=REGEX_PATTERN_KEYS_WITH_TRACEPARENT,
+            experiments=(UniqueIdExperiment,),
+        )
+
+    normalized = parameterizer.parameterize_all(trimmed, _should_run_experiment)
 
     for experiment in parameterizer.get_successful_experiments():
         if share_analytics and experiment.counter < 100:

src/sentry/options/defaults.py

@@ -2624,6 +2624,11 @@
     default=0.0,
     flags=FLAG_ADMIN_MODIFIABLE | FLAG_AUTOMATOR_MODIFIABLE | FLAG_RATE,
 )
+register(
+    "grouping.experiments.parameterization.traceparent",
+    default=0.0,
+    flags=FLAG_ADMIN_MODIFIABLE | FLAG_AUTOMATOR_MODIFIABLE | FLAG_RATE,
+)
 
 # TODO: For now, only a small number of projects are going through a grouping config transition at
 # any given time, so we're sampling at 100% in order to be able to get good signal. Once we've fully

tests/sentry/grouping/test_parameterization.py

@@ -7,27 +7,13 @@
     Parameterizer,
     UniqueIdExperiment,
 )
+from sentry.grouping.strategies.message import REGEX_PATTERN_KEYS_WITH_TRACEPARENT
 
 
 @pytest.fixture
 def parameterizer():
     return Parameterizer(
-        regex_pattern_keys=(
-            "email",
-            "url",
-            "hostname",
-            "ip",
-            "uuid",
-            "sha1",
-            "md5",
-            "date",
-            "duration",
-            "hex",
-            "float",
-            "int",
-            "quoted_str",
-            "bool",
-        ),
+        regex_pattern_keys=REGEX_PATTERN_KEYS_WITH_TRACEPARENT,
         experiments=(UniqueIdExperiment,),
     )
 
@@ -130,6 +116,11 @@ def parameterizer():
         ("hex", """blah 0x9af8c3b had a problem""", """blah <hex> had a problem"""),
         ("float", """blah 0.23 had a problem""", """blah <float> had a problem"""),
         ("int", """blah 23 had a problem""", """blah <int> had a problem"""),
+        (
+            "traceparent",
+            """traceparent: 00-0af7651916cd43dd8448eb211c80319c-b7ad6b7169203331-01""",
+            """traceparent: <traceparent>""",
+        ),
         ("quoted str", """blah b="1" had a problem""", """blah b=<quoted_str> had a problem"""),
         ("bool", """blah a=true had a problem""", """blah a=<bool> had a problem"""),
         (