getsentry / sentry-python

The official Python SDK for Sentry.io
https://sentry.io/for/python/
MIT License
1.93k stars 509 forks source link

Sample on root span level #3767

Closed sentrivana closed 1 week ago

sentrivana commented 1 week ago

Closes https://github.com/getsentry/sentry-python/issues/3766

codecov[bot] commented 1 week ago

:x: 1594 Tests Failed:

Tests completed Failed Passed Skipped
19899 1594 18305 4412
View the top 1 failed tests by shortest run time > > ```python > tests.test_utils test_datetime_from_isoformat[2021-01-01T00:00:00.000000-00:00-expected_output4] > ``` > >
Stack Traces | 0.001s run time > > > > > ```python > > tests/test_utils.py:109: in test_datetime_from_isoformat > > assert datetime_from_isoformat(input_str) == expected_output, input_str > > E NameError: name 'datetime_from_isoformat' is not defined > > ``` > >

View the full list of 2 :snowflake: flaky tests > > ```python > tests.test_utils test_datetime_from_isoformat[2021-01-01T00:00:00.000000-0000-expected_output6] > ``` > > **Flake rate in main:** 43.86% (Passed 32 times, Failed 25 times) >
Stack Traces | 0.001s run time > > > > > ```python > > tests/test_utils.py:109: in test_datetime_from_isoformat > > assert datetime_from_isoformat(input_str) == expected_output, input_str > > E NameError: name 'datetime_from_isoformat' is not defined > > ``` > >
tests.test_scope test_with_isolation_scope_data

Flake rate in main: 99.09% (Passed 2 times, Failed 218 times)

Stack Traces | 0.001s run time > > ```python > tests/test_scope.py:387: in test_with_isolation_scope_data > assert scope._tags == {"before_isolation_scope": 1} > E AssertionError: assert {} == {'before_isolation_scope': 1} > E > E Right contains 1 more item: > E {'before_isolation_scope': 1} > E > E Full diff: > E + {} > E - { > E - 'before_isolation_scope': 1, > E - } > ```

To view more test analytics, go to the Test Analytics Dashboard Got feedback? Let us know on Github

sentrivana commented 1 week ago

@sl0thentr0py Removed parent_sampled and updated the tests a bit more, please recheck