Skip to content

Conversation

@alexander-alderman-webb
Copy link
Contributor

@alexander-alderman-webb alexander-alderman-webb commented Nov 13, 2025

Description

Increase databag serialization depth and breadth limits to 100.

There are no grouping changes with the events generated by the scripts in https://github.com/getsentry/python-sdk-databag-limit-test.

See https://sentry-sdks.sentry.io/issues/?project=4510379916328960.

Issues

Closes #4987
Closes PY-1915

Reminders

@codecov
Copy link

codecov bot commented Nov 13, 2025

❌ 3 Tests Failed:

Tests completed Failed Passed Skipped
27880 3 27877 2222
View the top 3 failed test(s) by shortest run time
tests.integrations.pure_eval.test_pure_eval::test_include_local_variables_enabled[integrations1]
Stack Traces | 0.101s run time
.../integrations/pure_eval/test_pure_eval.py:74: in test_include_local_variables_enabled
    assert list(frame_vars.keys()) == expected_keys
E   AssertionError: assert ['namespace', 'namespace.d', 'namespace.d[1]', 's', 'a', 'p', 'o', 'i', 'u', 'y', 't', 'r', 'e', 'w', 'q', 'events'] == ['namespace', 'namespace.d', 'namespace.d[1]', 's', 'a', 'p', 'o', 'i', 'u', 'y', 't', 'r', 'e', 'w', 'q', '(q, w, e, r, t, y, u, i, o, p, a, s)', 'str((q, w, e, r, t, y, u, i, o, p, a, s))', 'events']
E     At index 15 diff: 'events' != '(q, w, e, r, t, y, u, i, o, p, a, s)'
E     Right contains 2 more items, first extra item: 'str((q, w, e, r, t, y, u, i, o, p, a, s))'
E     Full diff:
E       [
E        'namespace',
E        'namespace.d',
E        'namespace.d[1]',
E        's',
E        'a',
E        'p',
E        'o',
E        'i',
E        'u',
E        'y',
E        't',
E        'r',
E        'e',
E        'w',
E        'q',
E     -  '(q, w, e, r, t, y, u, i, o, p, a, s)',
E     -  'str((q, w, e, r, t, y, u, i, o, p, a, s))',
E        'events',
E       ]
tests.integrations.pure_eval.test_pure_eval::test_include_local_variables_enabled[integrations1]
Stack Traces | 0.105s run time
.../integrations/pure_eval/test_pure_eval.py:74: in test_include_local_variables_enabled
    assert list(frame_vars.keys()) == expected_keys
E   AssertionError: assert ['namespace', 'namespace.d', 'namespace.d[1]', 's', 'a', 'p', 'o', 'i', 'u', 'y', 't', 'r', 'e', 'w', 'q', 'events'] == ['namespace', 'namespace.d', 'namespace.d[1]', 's', 'a', 'p', 'o', 'i', 'u', 'y', 't', 'r', 'e', 'w', 'q', '(q, w, e, r, t, y, u, i, o, p, a, s)', 'str((q, w, e, r, t, y, u, i, o, p, a, s))', 'events']
E     
E     At index 15 diff: 'events' != '(q, w, e, r, t, y, u, i, o, p, a, s)'
E     Right contains 2 more items, first extra item: 'str((q, w, e, r, t, y, u, i, o, p, a, s))'
E     
E     Full diff:
E       [
E           'namespace',
E           'namespace.d',
E           'namespace.d[1]',
E           's',
E           'a',
E           'p',
E           'o',
E           'i',
E           'u',
E           'y',
E           't',
E           'r',
E           'e',
E           'w',
E           'q',
E     -     '(q, w, e, r, t, y, u, i, o, p, a, s)',
E     -     'str((q, w, e, r, t, y, u, i, o, p, a, s))',
E           'events',
E       ]
tests.integrations.pure_eval.test_pure_eval::test_include_local_variables_enabled[integrations1]
Stack Traces | 0.132s run time
.../integrations/pure_eval/test_pure_eval.py:74: in test_include_local_variables_enabled
    assert list(frame_vars.keys()) == expected_keys
E   AssertionError: assert ['namespace',\n 'namespace.d',\n 'namespace.d[1]',\n 's',\n 'a',\n 'p',\n 'o',\n 'i',\n 'u',\n 'y',\n 't',\n 'r',\n 'e',\n 'w',\n 'q',\n 'events'] == ['namespace',\n 'namespace.d',\n 'namespace.d[1]',\n 's',\n 'a',\n 'p',\n 'o',\n 'i',\n 'u',\n 'y',\n 't',\n 'r',\n 'e',\n 'w',\n 'q',\n '(q, w, e, r, t, y, u, i, o, p, a, s)',\n 'str((q, w, e, r, t, y, u, i, o, p, a, s))',\n 'events']
E     At index 15 diff: 'events' != '(q, w, e, r, t, y, u, i, o, p, a, s)'
E     Right contains 2 more items, first extra item: 'str((q, w, e, r, t, y, u, i, o, p, a, s))'
E     Full diff:
E       [
E        'namespace',
E        'namespace.d',
E        'namespace.d[1]',
E        's',
E        'a',
E        'p',
E        'o',
E        'i',
E        'u',
E        'y',
E        't',
E        'r',
E        'e',
E        'w',
E        'q',
E     -  '(q, w, e, r, t, y, u, i, o, p, a, s)',
E     -  'str((q, w, e, r, t, y, u, i, o, p, a, s))',
E        'events',
E       ]

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@linear
Copy link

linear bot commented Nov 17, 2025

@alexander-alderman-webb alexander-alderman-webb marked this pull request as ready for review November 17, 2025 14:16
@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner November 17, 2025 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Increase databag limits

3 participants