FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-109832: concurrent.futures test_deadlock restores sys.stderr by vstinner · Pull Request #109887 · python/cpython · GitHub

/ cpython Public

gh-109832: concurrent.futures test_deadlock restores sys.stderr - #109887

Merged
vstinner merged 1 commit into
python:mainfrom
vstinner:cf_restore_stderr
Sep 26, 2023
Merged

gh-109832: concurrent.futures test_deadlock restores sys.stderr#109887
vstinner merged 1 commit into
python:mainfrom
vstinner:cf_restore_stderr

Conversation

vstinner commented Sep 26, 2023
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

test_error_at_task_unpickle() and
test_error_during_result_unpickle_in_result_handler() now restore sys.stderr which is overriden by _raise_error_ignore_stderr().

test_error_at_task_unpickle() and
test_error_during_result_unpickle_in_result_handler() now restore
sys.stderr which is overriden by _raise_error_ignore_stderr().

Copy link
Copy Markdown
Member Author

Windows x64 failed:

ERROR: test_CTRL_BREAK_EVENT (test.test_os.Win32KillTests.test_CTRL_BREAK_EVENT)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "D:\a\cpython\cpython\Lib\test\test_os.py", line 2616, in test_CTRL_BREAK_EVENT
    self._kill_with_event(signal.CTRL_BREAK_EVENT, "CTRL_BREAK_EVENT")
  File "D:\a\cpython\cpython\Lib\test\test_os.py", line 2591, in _kill_with_event
    os.kill(proc.pid, signal.SIGINT)
PermissionError: [WinError 5] Access is denied

Test passed when re-run:

test_CTRL_BREAK_EVENT (test.test_os.Win32KillTests.test_CTRL_BREAK_EVENT) ... ok

Copy link
Copy Markdown
Member Author

Windows x64 failed: ERROR: test_CTRL_BREAK_EVENT (test.test_os.Win32KillTests.test_CTRL_BREAK_EVENT)

I created issue gh-109888 for this failure.

vstinner merged commit 2897142 into python:main Sep 26, 2023
vstinner deleted the cf_restore_stderr branch September 26, 2023 10:43

Copy link
Copy Markdown
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 26, 2023
…pythonGH-109887)

test_error_at_task_unpickle() and
test_error_during_result_unpickle_in_result_handler() now restore
sys.stderr which is overriden by _raise_error_ignore_stderr().
(cherry picked from commit 2897142)

Co-authored-by: Victor Stinner <vstinner@python.org>

bedevere-app Bot commented Sep 26, 2023

Copy link
Copy Markdown

GH-109892 is a backport of this pull request to the 3.12 branch.

bedevere-app Bot removed the needs backport to 3.12 only security fixes label Sep 26, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 26, 2023
…pythonGH-109887)

test_error_at_task_unpickle() and
test_error_during_result_unpickle_in_result_handler() now restore
sys.stderr which is overriden by _raise_error_ignore_stderr().
(cherry picked from commit 2897142)

Co-authored-by: Victor Stinner <vstinner@python.org>

bedevere-app Bot commented Sep 26, 2023

Copy link
Copy Markdown

GH-109893 is a backport of this pull request to the 3.11 branch.

bedevere-app Bot removed the needs backport to 3.11 only security fixes label Sep 26, 2023
vstinner added a commit that referenced this pull request Sep 26, 2023
GH-109887) (#109893)

gh-109832: concurrent.futures test_deadlock restores sys.stderr (GH-109887)

test_error_at_task_unpickle() and
test_error_during_result_unpickle_in_result_handler() now restore
sys.stderr which is overriden by _raise_error_ignore_stderr().
(cherry picked from commit 2897142)

Co-authored-by: Victor Stinner <vstinner@python.org>
csm10495 pushed a commit to csm10495/cpython that referenced this pull request Sep 28, 2023
…python#109887)

test_error_at_task_unpickle() and
test_error_during_result_unpickle_in_result_handler() now restore
sys.stderr which is overriden by _raise_error_ignore_stderr().
Yhg1s pushed a commit that referenced this pull request Oct 2, 2023
GH-109887) (#109892)

gh-109832: concurrent.futures test_deadlock restores sys.stderr (GH-109887)

test_error_at_task_unpickle() and
test_error_during_result_unpickle_in_result_handler() now restore
sys.stderr which is overriden by _raise_error_ignore_stderr().
(cherry picked from commit 2897142)

Co-authored-by: Victor Stinner <vstinner@python.org>
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
…python#109887)

test_error_at_task_unpickle() and
test_error_during_result_unpickle_in_result_handler() now restore
sys.stderr which is overriden by _raise_error_ignore_stderr().
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL