Skip to content

gh-158294: Avoid relying on SystemTap probe handler ordering - #158346

Merged
hugovk merged 1 commit into
python:mainfrom
stratakis:handler_merge
Sep 29, 2026
Merged

hugovk merged 1 commit into
python:mainfrom
stratakis:handler_merge

Conversation

@stratakis

@stratakis stratakis commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Merge the tracing state and filter output in a single probe handler as SystemTap does not guarantee the execution order of handlers sharing a probe point.

Merge the tracing state and filter output in a single probe handler as
SystemTap does not guarantee the execution order of handlers sharing a
probe point.
@hugovk hugovk added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Sep 28, 2026
@bedevere-bot

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @hugovk for commit c519ba7 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F158346%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Sep 28, 2026

@hugovk hugovk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@hugovk
hugovk merged commit ec49d01 into python:main Sep 29, 2026
183 of 186 checks passed
@hugovk

hugovk commented Sep 29, 2026

Copy link
Copy Markdown
Member

Backport needed at least for 3.15 where this is failing (for example, https://buildbot.python.org/#/builders/2016/builds/413).

@hugovk hugovk added the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Sep 29, 2026
@miss-islington-app

Copy link
Copy Markdown

Thanks @stratakis for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.15.
🐍🍒⛏🤖

@bedevere-app

bedevere-app Bot commented Sep 29, 2026

Copy link
Copy Markdown

GH-158383 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Sep 29, 2026
hugovk pushed a commit that referenced this pull request Sep 29, 2026
…H-158346) (#158383)

Co-authored-by: stratakis <14812606+stratakis@users.noreply.github.com>
@vstinner

Copy link
Copy Markdown
Member

Thanks for the quick fix @stratakis!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants