chore(er): max frame setting (#11851)

We add the setting option to specifiy the maximum number of frames to
capture while collecting exception replay data. By default this is
infinite.

## Checklist
- [x] PR author has checked that all the criteria below are met
- The PR description includes an overview of the change
- The PR description articulates the motivation for the change
- The change includes tests OR the PR description describes a testing
strategy
- The PR description notes risks associated with the change, if any
- Newly-added code is easy to change
- The change follows the [library release note
guidelines](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [ ] Reviewer has checked that all the criteria below are met 
- Title is accurate
- All changes are related to the pull request's stated goal
- Avoids breaking
[API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
changes
- Testing strategy adequately addresses listed risks
- Newly-added code is easy to change
- Release note makes sense to a user of the library
- If necessary, author has acknowledged and discussed the performance
implications of this PR as reported in the benchmarks PR comment
- Backport labels are set in a manner that is consistent with the
[release branch maintenance
policy](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

Author: P403n1x87

ddtrace/debugging/_exception/replay.py

@@ -21,6 +21,7 @@
 from ddtrace.internal.rate_limiter import BudgetRateLimiterWithJitter as RateLimiter
 from ddtrace.internal.rate_limiter import RateLimitExceeded
 from ddtrace.internal.utils.time import HourGlass
+from ddtrace.settings.exception_replay import config
 
 
 log = get_logger(__name__)
@@ -225,15 +226,17 @@ def on_span_exception(
 
         seq = count(1)  # 1-based sequence number
 
-        while chain:
+        frames_captured = 0
+
+        while chain and frames_captured <= config.max_frames:
             exc, _tb = chain.pop()  # LIFO: reverse the chain
 
             if _tb is None or _tb.tb_frame is None:
                 # If we don't have a traceback there isn't much we can do
                 continue
 
             # DEV: We go from the handler up to the root exception
-            while _tb:
+            while _tb and frames_captured <= config.max_frames:
                 frame = _tb.tb_frame
                 code = frame.f_code
                 seq_nr = next(seq)
@@ -263,6 +266,9 @@ def on_span_exception(
                         # Memoize
                         frame.f_locals[SNAPSHOT_KEY] = snapshot_id = snapshot.uuid
 
+                        # Count
+                        frames_captured += 1
+
                     # Add correlation tags on the span
                     span.set_tag_str(FRAME_SNAPSHOT_ID_TAG % seq_nr, snapshot_id)
                     span.set_tag_str(FRAME_FUNCTION_TAG % seq_nr, code.co_name)

ddtrace/settings/exception_replay.py

@@ -14,6 +14,13 @@ class ExceptionReplayConfig(En):
         help="Enable automatic capturing of exception debugging information",
         deprecations=[("debugging.enabled", None, "3.0")],
     )
+    max_frames = En.v(
+        int,
+        "replay.capture_max_frames",
+        default=8,
+        help_type="int",
+        help="The maximum number of frames to capture for each exception",
+    )
 
 
 config = ExceptionReplayConfig()

tests/telemetry/test_writer.py

@@ -354,6 +354,7 @@ def test_app_started_event_configuration_override(test_agent_session, run_python
         {"name": "DD_DYNAMIC_INSTRUMENTATION_UPLOAD_FLUSH_INTERVAL", "origin": "default", "value": 1.0},
         {"name": "DD_DYNAMIC_INSTRUMENTATION_UPLOAD_TIMEOUT", "origin": "default", "value": 30},
         {"name": "DD_ENV", "origin": "default", "value": None},
+        {"name": "DD_EXCEPTION_REPLAY_CAPTURE_MAX_FRAMES", "origin": "default", "value": 8},
         {"name": "DD_EXCEPTION_REPLAY_ENABLED", "origin": "env_var", "value": True},
         {"name": "DD_EXPERIMENTAL_APPSEC_STANDALONE_ENABLED", "origin": "default", "value": False},
         {"name": "DD_HTTP_CLIENT_TAG_QUERY_STRING", "origin": "default", "value": None},