Skip to content

ref(span-buffer): Disable multiprocessing by default#87268

Merged
untitaker merged 6 commits into
masterfrom
ref/span-buffer-disable-multiprocessing
Mar 18, 2025
Merged

ref(span-buffer): Disable multiprocessing by default#87268
untitaker merged 6 commits into
masterfrom
ref/span-buffer-disable-multiprocessing

Conversation

@untitaker

@untitaker untitaker commented Mar 18, 2025

Copy link
Copy Markdown
Member

We have an issue with multiprocessing where this particular consumer
hangs on startup. When we dump stacks, we see it's deep in the
multiprocessing stdlib functions, on some IPC read() call.

It's not clear what causes this right now. The issue is not
reproducible in development, and has been happening inconsistently in
the tf-sandbox

We have an issue with multiprocessing where this particula consumer
hangs on startup. When we dump stacks, we see it's deep in the
multiprocessing stdlib functions, on some IPC read() call.

It's not clear what causes this right now. The issue is not
reproducible in development, and has been happening inconsistently in
the tf-sandbox
@untitaker untitaker requested a review from a team March 18, 2025 10:46
@untitaker untitaker requested review from a team as code owners March 18, 2025 10:46
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 18, 2025
@codecov

codecov Bot commented Mar 18, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 78.57143% with 3 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/spans/consumers/process/factory.py 66.66% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #87268      +/-   ##
==========================================
- Coverage   87.73%   87.73%   -0.01%     
==========================================
  Files        9863     9863              
  Lines      558345   558332      -13     
  Branches    22030    22024       -6     
==========================================
- Hits       489888   489875      -13     
  Misses      68050    68050              
  Partials      407      407              

@untitaker untitaker disabled auto-merge March 18, 2025 11:41
@untitaker untitaker requested a review from a team as a code owner March 18, 2025 12:08
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 18, 2025
@github-actions

Copy link
Copy Markdown
Contributor

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

@untitaker untitaker merged commit 340a65b into master Mar 18, 2025
@untitaker untitaker deleted the ref/span-buffer-disable-multiprocessing branch March 18, 2025 17:26
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants