Skip to content

gh-91048: Add ability to list all pending asyncio tasks in a process remotely - #132807

Merged
pablogsal merged 6 commits into
python:mainfrom
ambv:asyncio-inspect-all-tasks
Apr 23, 2025
Merged

gh-91048: Add ability to list all pending asyncio tasks in a process remotely#132807
pablogsal merged 6 commits into
python:mainfrom
ambv:asyncio-inspect-all-tasks

Conversation

@ambv

@ambvambv commented Apr 22, 2025•
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

@bedevere-appbedevere-appBot mentioned this pull request Apr 22, 2025
Closed
@ambv
ambv marked this pull request as ready for review April 23, 2025 10:40
@ambv

ambv commented Apr 23, 2025

Copy link
Copy Markdown
ContributorAuthor

!buildbot

pablogsal
pablogsal approved these changes Apr 23, 2025
View reviewed changes

@pablogsalpablogsal 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.

LGTM

left some comments

Comment threadModules/_testexternalinspection.c
Comment threadModules/_testexternalinspection.c
Comment threadModules/_testexternalinspection.c
Comment threadModules/_testexternalinspection.c
pablogsal
pablogsal reviewed Apr 23, 2025
View reviewed changes
Comment threadModules/_testexternalinspection.c
@pablogsal

pablogsal commented Apr 23, 2025

Copy link
Copy Markdown
Member

can we add a comment explaining the special case of thread ID 0 for interpreter-level tasks to avoid confusion

@ambv
ambvforce-pushed the asyncio-inspect-all-tasks branch from 7e01e05 to de92aabCompareApril 23, 2025 14:59
@ambvambv added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Apr 23, 2025
@bedevere-bot

bedevere-bot commented Apr 23, 2025

Copy link
Copy Markdown

🤖 New build scheduled with the buildbot fleet by @ambv for commit de92aab 🤖

Results will be shown at:

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

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

@bedevere-botbedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Apr 23, 2025
@ambv

ambv commented Apr 23, 2025

Copy link
Copy Markdown
ContributorAuthor

The FreeBSD buildbot's been broken since last week. I reported on #132561.

@pablogsal
pablogsal merged commit c9bc458 into python:mainApr 23, 2025
@ambv
ambv deleted the asyncio-inspect-all-tasks branch May 5, 2025 21:10
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pablogsalpablogsalpablogsal approved these changes
@1st11st1Awaiting requested review from 1st11st1 is a code owner
@asvetlovasvetlovAwaiting requested review from asvetlovasvetlov is a code owner
@kumaraditya303kumaraditya303Awaiting requested review from kumaraditya303kumaraditya303 is a code owner
@willingcwillingcAwaiting requested review from willingcwillingc is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ambv@pablogsal@bedevere-bot@1st1