Skip to content

ref(sdk): allow to disable threading integration#113154

Merged
gi0baro merged 1 commit intomasterfrom
gi0baro/allow-disable-sdk-threading-integration
Apr 16, 2026
Merged

ref(sdk): allow to disable threading integration#113154
gi0baro merged 1 commit intomasterfrom
gi0baro/allow-disable-sdk-threading-integration

Conversation

@gi0baro
Copy link
Copy Markdown
Member

@gi0baro gi0baro commented Apr 16, 2026

SSIA

With ASGI experimentation, we suffer from a memory leak, which might be caused by getsentry/sentry-python#5928, given the usage of sync_to_async.

This setting allow us to verify the assumption.

@gi0baro gi0baro requested review from a team as code owners April 16, 2026 10:50
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 16, 2026
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 52504fad64208dfee15c0a3f9062479ed810a9a1. Configure here.

Comment thread src/sentry/utils/sdk.py
@gi0baro gi0baro force-pushed the gi0baro/allow-disable-sdk-threading-integration branch from 52504fa to 2a61328 Compare April 16, 2026 14:16
@gi0baro gi0baro merged commit 7752ab9 into master Apr 16, 2026
77 checks passed
@gi0baro gi0baro deleted the gi0baro/allow-disable-sdk-threading-integration branch April 16, 2026 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants