Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

test_asyncio: test_unhandled_exceptions() fails on x86-64 macOS 3.x buildbot #111644

Closed
vstinner opened this issue Nov 2, 2023 · 5 comments
Closed

test_asyncio: test_unhandled_exceptions() fails on x86-64 macOS 3.x buildbot #111644

vstinner opened this issue Nov 2, 2023 · 5 comments
Labels
tests Tests in the Lib/test dir topic-asyncio

Comments

@vstinner
Copy link
Member

vstinner commented Nov 2, 2023 •
edited by bedevere-app bot
Loading

Apparently, the test is new and was added by PR #111601.

cc @kumaraditya303

Logs:

test_unhandled_exceptions (test.test_asyncio.test_streams.StreamTests.test_unhandled_exceptions) ... Warning -- threading_cleanup() failed to cleanup 1 threads (count: 1, dangling: 2)
Warning -- Dangling thread: <_MainThread(MainThread, started 4648830464)>
ok

build: https://buildbot.python.org/all/#/builders/366/builds/5796

Linked PRs
@github-project-automation github-project-automation bot moved this to Todo in asyncio Nov 2, 2023
@vstinner
Copy link
Member Author

vstinner commented Nov 2, 2023

Oh, test fails on tons of buildbots: #111601 (comment)

@vstinner vstinner mentioned this issue Nov 2, 2023
Merged
@AlexWaygood AlexWaygood added the tests Tests in the Lib/test dir label Nov 2, 2023
@cfbolz cfbolz mentioned this issue Nov 3, 2023
Merged
@kumaraditya303
Copy link
Contributor

I am not able to reproduce it on my system. Also I don't understand what Warning -- Dangling thread: <_MainThread(MainThread, started 4648830464)> means, how can main thread be considered dangling?

@vstinner
Copy link
Member Author

vstinner commented Nov 3, 2023

I am not able to reproduce it on my system. Also I don't understand what Warning -- Dangling thread: <_MainThread(MainThread, started 4648830464)> means,

It usually means that a test spawns a thread and don't fully wait until the thread completes.

vstinner added a commit to vstinner/cpython that referenced this issue Nov 3, 2023
@vstinner
pythongh-111644: Fix asyncio test_unhandled_exceptions()
a755131 
Fix test_unhandled_exceptions() of test_asyncio.test_streams: break
explicitly a reference cycle.
@bedevere-app bedevere-app bot mentioned this issue Nov 3, 2023
Merged
@vstinner
Copy link
Member Author

vstinner commented Nov 3, 2023

I can reproduce the issue with the command:

./python -m test test_asyncio.test_streams -m test_unhandled_exceptions -v -j10 --fail-env-changed -F

vstinner added a commit to vstinner/cpython that referenced this issue Nov 3, 2023
@vstinner
pythongh-111644: Enhance support threading_cleanup()
0367d2d 
Copy the list of dangling thread to provide a consistent output
between the warning and the "Dangling thread" listing.
@bedevere-app bedevere-app bot mentioned this issue Nov 3, 2023
Merged
vstinner added a commit to vstinner/cpython that referenced this issue Nov 3, 2023
@vstinner
pythongh-111644: Enhance support threading_cleanup()
20c4268 
Copy the list of dangling thread to provide a consistent output
between the warning and the "Dangling thread" listing.
vstinner added a commit to vstinner/cpython that referenced this issue Nov 3, 2023
@vstinner
pythongh-111644: Fix support threading_cleanup()
b1001dd 
Copy the list of dangling threads to make sure that the list of
"Dangling thread" is complete. Previously, the list was incomplete if
threads completed just before the list is displayed.

Changes:

* Reformat the warning to make it easier to understand.
* Use support.sleeping_retry().
* threading_cleanup() no longer copies threading._dangling: it only
  counts the number of dangling thread.
* Remove support.gc_support() call.
vstinner added a commit to vstinner/cpython that referenced this issue Nov 3, 2023
@vstinner
pythongh-111644: Fix support threading_cleanup()
1223857 
Copy the list of dangling threads to make sure that the list of
"Dangling thread" is complete. Previously, the list was incomplete if
threads completed just before the list is displayed.

Changes:

* Reformat the warning to make it easier to understand.
* Use support.sleeping_retry().
* threading_cleanup() no longer copies threading._dangling: it only
  counts the number of dangling thread.
* Remove support.gc_support() call.
vstinner added a commit that referenced this issue Nov 4, 2023
@vstinner
gh-111644: Fix support threading_cleanup() (#111714)
f62c7cc 
Copy the list of dangling threads to make sure that the list of
"Dangling thread" is complete. Previously, the list was incomplete if
threads completed just before the list was displayed.

Changes:

* Rewrite the warning to make it easier to understand.
* Use support.sleeping_retry().
* threading_cleanup() no longer copies threading._dangling,
  but only counts the number of dangling thread.
* Remove support.gc_support() call.
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Nov 4, 2023
@vstinner @miss-islington
pythongh-111644: Fix support threading_cleanup() (pythonGH-111714)
91a2fdc 
Copy the list of dangling threads to make sure that the list of
"Dangling thread" is complete. Previously, the list was incomplete if
threads completed just before the list was displayed.

Changes:

* Rewrite the warning to make it easier to understand.
* Use support.sleeping_retry().
* threading_cleanup() no longer copies threading._dangling,
  but only counts the number of dangling thread.
* Remove support.gc_support() call.
(cherry picked from commit f62c7cc)

Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-app bedevere-app bot mentioned this issue Nov 4, 2023
Merged
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Nov 4, 2023
@vstinner @miss-islington
pythongh-111644: Fix support threading_cleanup() (pythonGH-111714)
678ecde 
Copy the list of dangling threads to make sure that the list of
"Dangling thread" is complete. Previously, the list was incomplete if
threads completed just before the list was displayed.

Changes:

* Rewrite the warning to make it easier to understand.
* Use support.sleeping_retry().
* threading_cleanup() no longer copies threading._dangling,
  but only counts the number of dangling thread.
* Remove support.gc_support() call.
(cherry picked from commit f62c7cc)

Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-app bedevere-app bot mentioned this issue Nov 4, 2023
Merged
vstinner added a commit to vstinner/cpython that referenced this issue Nov 4, 2023
@vstinner
pythongh-111644: Fix asyncio test_unhandled_exceptions()
c855b53 
Fix test_unhandled_exceptions() of test_asyncio.test_streams: break
explicitly a reference cycle.

Fix also StreamTests.tearDown(): the loop must not be closed
explicitly, but using set_event_loop() which takes care of shutting
down the executor with executor.shutdown(wait=True).
BaseEventLoop.close() calls executor.shutdown(wait=False).
vstinner added a commit that referenced this issue Nov 4, 2023
@miss-islington @vstinner
[3.11] gh-111644: Fix support threading_cleanup() (GH-111714) (#111717)
89264a3 
gh-111644: Fix support threading_cleanup() (GH-111714)

Copy the list of dangling threads to make sure that the list of
"Dangling thread" is complete. Previously, the list was incomplete if
threads completed just before the list was displayed.

Changes:

* Rewrite the warning to make it easier to understand.
* Use support.sleeping_retry().
* threading_cleanup() no longer copies threading._dangling,
  but only counts the number of dangling thread.
* Remove support.gc_support() call.
(cherry picked from commit f62c7cc)

Co-authored-by: Victor Stinner <vstinner@python.org>
vstinner added a commit that referenced this issue Nov 4, 2023
@vstinner
gh-111644: Fix asyncio test_unhandled_exceptions() (#111713)
ac01e22 
Fix test_unhandled_exceptions() of test_asyncio.test_streams: break
explicitly a reference cycle.

Fix also StreamTests.tearDown(): the loop must not be closed
explicitly, but using set_event_loop() which takes care of shutting
down the executor with executor.shutdown(wait=True).
BaseEventLoop.close() calls executor.shutdown(wait=False).
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Nov 4, 2023
@vstinner @miss-islington
pythongh-111644: Fix asyncio test_unhandled_exceptions() (pythonGH-11…
50596df 
…1713)

Fix test_unhandled_exceptions() of test_asyncio.test_streams: break
explicitly a reference cycle.

Fix also StreamTests.tearDown(): the loop must not be closed
explicitly, but using set_event_loop() which takes care of shutting
down the executor with executor.shutdown(wait=True).
BaseEventLoop.close() calls executor.shutdown(wait=False).
(cherry picked from commit ac01e22)

Co-authored-by: Victor Stinner <vstinner@python.org>
@bedevere-app bedevere-app bot mentioned this issue Nov 4, 2023
Merged
vstinner added a commit to miss-islington/cpython that referenced this issue Nov 4, 2023
@vstinner
pythongh-111644: Fix asyncio test_unhandled_exceptions() (python#111713)
dced4b8 
Fix test_unhandled_exceptions() of test_asyncio.test_streams: break
explicitly a reference cycle.

Fix also StreamTests.tearDown(): the loop must not be closed
explicitly, but using set_event_loop() which takes care of shutting
down the executor with executor.shutdown(wait=True).
BaseEventLoop.close() calls executor.shutdown(wait=False).

(cherry picked from commit ac01e22)
vstinner added a commit that referenced this issue Nov 4, 2023
@miss-islington @vstinner
[3.12] gh-111644: Fix support threading_cleanup() (GH-111714) (#111716)
02f8f78 
gh-111644: Fix support threading_cleanup() (GH-111714)

Copy the list of dangling threads to make sure that the list of
"Dangling thread" is complete. Previously, the list was incomplete if
threads completed just before the list was displayed.

Changes:

* Rewrite the warning to make it easier to understand.
* Use support.sleeping_retry().
* threading_cleanup() no longer copies threading._dangling,
  but only counts the number of dangling thread.
* Remove support.gc_support() call.
(cherry picked from commit f62c7cc)

Co-authored-by: Victor Stinner <vstinner@python.org>
vstinner added a commit that referenced this issue Nov 4, 2023
@miss-islington @kumaraditya303 @vstinner
[3.11] [3.12] GH-110894: Call loop exception handler for exceptions i…
f7ffe4a 
…n client_connected_cb (GH-111601) (GH-111632) (#111634)

* [3.12] GH-110894: Call loop exception handler for exceptions in client_connected_cb (GH-111601) (GH-111632)
(cherry picked from commit 9aa8829)

Co-authored-by: Kumar Aditya <kumaraditya@python.org>
Call loop exception handler for exceptions in `client_connected_cb` of `asyncio.start_server` so that applications can handle it..
(cherry picked from commit 229f44d)

* gh-111644: Fix asyncio test_unhandled_exceptions() (#111713)

Fix test_unhandled_exceptions() of test_asyncio.test_streams: break
explicitly a reference cycle.

Fix also StreamTests.tearDown(): the loop must not be closed
explicitly, but using set_event_loop() which takes care of shutting
down the executor with executor.shutdown(wait=True).
BaseEventLoop.close() calls executor.shutdown(wait=False).

(cherry picked from commit ac01e22)

---------

Co-authored-by: Kumar Aditya <kumaraditya@python.org>
Co-authored-by: Victor Stinner <vstinner@python.org>
@kumaraditya303
Copy link
Contributor

Thanks @vstinner

vstinner added a commit that referenced this issue Nov 4, 2023
@miss-islington @vstinner
[3.12] gh-111644: Fix asyncio test_unhandled_exceptions() (GH-111713) (…
d15fac2 
…#111718)

gh-111644: Fix asyncio test_unhandled_exceptions() (GH-111713)

Fix test_unhandled_exceptions() of test_asyncio.test_streams: break
explicitly a reference cycle.

Fix also StreamTests.tearDown(): the loop must not be closed
explicitly, but using set_event_loop() which takes care of shutting
down the executor with executor.shutdown(wait=True).
BaseEventLoop.close() calls executor.shutdown(wait=False).
(cherry picked from commit ac01e22)

Co-authored-by: Victor Stinner <vstinner@python.org>
@github-project-automation github-project-automation bot moved this from Todo to Done in asyncio Nov 4, 2023
aisk pushed a commit to aisk/cpython that referenced this issue Feb 11, 2024
@vstinner @aisk
pythongh-111644: Fix support threading_cleanup() (python#111714)
d95cfbe 
Copy the list of dangling threads to make sure that the list of
"Dangling thread" is complete. Previously, the list was incomplete if
threads completed just before the list was displayed.

Changes:

* Rewrite the warning to make it easier to understand.
* Use support.sleeping_retry().
* threading_cleanup() no longer copies threading._dangling,
  but only counts the number of dangling thread.
* Remove support.gc_support() call.
aisk pushed a commit to aisk/cpython that referenced this issue Feb 11, 2024
@vstinner @aisk
pythongh-111644: Fix asyncio test_unhandled_exceptions() (python#111713)
265e089 
Fix test_unhandled_exceptions() of test_asyncio.test_streams: break
explicitly a reference cycle.

Fix also StreamTests.tearDown(): the loop must not be closed
explicitly, but using set_event_loop() which takes care of shutting
down the executor with executor.shutdown(wait=True).
BaseEventLoop.close() calls executor.shutdown(wait=False).
Glyphack pushed a commit to Glyphack/cpython that referenced this issue Sep 2, 2024
@vstinner @Glyphack
pythongh-111644: Fix support threading_cleanup() (python#111714)
9147bba 
Copy the list of dangling threads to make sure that the list of
"Dangling thread" is complete. Previously, the list was incomplete if
threads completed just before the list was displayed.

Changes:

* Rewrite the warning to make it easier to understand.
* Use support.sleeping_retry().
* threading_cleanup() no longer copies threading._dangling,
  but only counts the number of dangling thread.
* Remove support.gc_support() call.
Glyphack pushed a commit to Glyphack/cpython that referenced this issue Sep 2, 2024
@vstinner @Glyphack
pythongh-111644: Fix asyncio test_unhandled_exceptions() (python#111713)
7e3e96f 
Fix test_unhandled_exceptions() of test_asyncio.test_streams: break
explicitly a reference cycle.

Fix also StreamTests.tearDown(): the loop must not be closed
explicitly, but using set_event_loop() which takes care of shutting
down the executor with executor.shutdown(wait=True).
BaseEventLoop.close() calls executor.shutdown(wait=False).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tests Tests in the Lib/test dir topic-asyncio
Projects
asyncio
Status: Done
Milestone
No milestone
Development

No branches or pull requests
3 participants
@vstinner @kumaraditya303 @AlexWaygood