Skip to content

ref: Move contextvars_context out of integrations #4271

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 18 commits into from
Apr 10, 2025

Conversation

sentrivana
Copy link
Contributor

Moving stuff out of integrations/opentelemetry/ step by step since there is no OpenTelemetry integration anymore -- it's part of the core SDK.

This moves sentry_sdk/integrations/opentelemetry/contextvars_context.py -> sentry_sdk/opentelemetry/contextvars_context.py

Ref #3853

Copy link

codecov bot commented Apr 10, 2025

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
20480 1 20479 4416
View the top 1 failed test(s) by shortest run time
tests.test_basics test_classmethod_tracing
Stack Traces | 0.085s run time
tests/test_basics.py:811: in test_classmethod_tracing
    assert fake_start_child.call_count == 1
E   AssertionError: assert 3 == 1
E    +  where 3 = <MagicMock name='mock.start_child' id='139943123060096'>.call_count

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

@sentrivana sentrivana marked this pull request as ready for review April 10, 2025 10:36
@sentrivana sentrivana requested a review from a team as a code owner April 10, 2025 10:36
@sentrivana sentrivana requested review from a team and removed request for a team April 10, 2025 12:57
Base automatically changed from ivana/potel/move-propagator to potel-base April 10, 2025 14:20
@sentrivana sentrivana merged commit dfad14f into potel-base Apr 10, 2025
123 of 124 checks passed
@sentrivana sentrivana deleted the ivana/potel/move-context-sync branch April 10, 2025 14:27
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants