8000 test_signal: test_stress_modifying_handlers() failed on s390x SLES 3.x · Issue #110647 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

test_signal: test_stress_modifying_handlers() failed on s390x SLES 3.x #110647

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
Closed
vstinner opened this issue Oct 10, 2023 · 1 comment
Closed

test_signal: test_stress_modifying_handlers() failed on s390x SLES 3.x #110647

vstinner opened this issue Oct 10, 2023 · 1 comment
Labels
tests Tests in the Lib/test dir

Comments

@vstinner
Copy link
Member
vstinner commented Oct 10, 2023 •
edited by bedevere-app bot
Loading

See also:

s390x SLES 3.x:

== CPU count: 4
...
0:03:27 load avg: 5.22 [280/467/1] test_signal failed (1 failure) (1 min 10 sec) -- running (1): test.test_multiprocessing_spawn.test_processes (38.4 sec)
(...)
FAIL: test_stress_modifying_handlers (test.test_signal.StressTest.test_stress_modifying_handlers)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/dje/cpython-buildarea/3.x.edelsohn-sles-z/build/Lib/test/test_signal.py", line 1374, in test_stress_modifying_handlers
    self.assertGreater(num_received_signals, 0)
AssertionError: 0 not greater than 0

Test pass when re-run in verbose mode.

build: https://buildbot.python.org/all/#/builders/540/builds/6801

Linked PRs
@vstinner vstinner added the tests Tests in the Lib/test dir label Oct 10, 2023
@vstinner
Copy link
Member Author

Change to make the error more likely:

diff --git a/Lib/test/test_signal.py b/Lib/test/test_signal.py
index 2a1a1ee22f..d7017c3281 100644
--- a/Lib/test/test_signal.py
+++ b/Lib/test/test_signal.py
@@ -1339,8 +1339,8 @@ def set_interrupts():
                 num_sent_signals += 1
 
         def cycle_handlers():
-            while num_sent_signals < 100:
-                for i in range(20000):
+            while num_sent_signals < 10:
+                for i in range(2000):
                     # Cycle between a Python-defined and a non-Python handler
                     for handler in [custom_handler, signal.SIG_IGN]:
                         signal.signal(signum, handler)

vstinner added a commit to vstinner/cpython that referenced this issue Oct 10, 2023
@vstinner
pythongh-110647: Fix signal test_stress_modifying_handlers()
f539618 
* cycle_handlers() now waits until at least one signal was received.
* num_received_signals can be equal to num_sent_signals.
vstinner added a commit to vstinner/cpython that referenced this issue Oct 10, 2023
@vstinner
pythongh-110647: Fix signal test_stress_modifying_handlers()
38db847 
* cycle_handlers() now waits until at least one signal was received.
* num_received_signals can be equal to num_sent_signals.
vstinner added a commit to vstinner/cpython that referenced this issue Oct 10, 2023
@vstinner
pythongh-110647: Fix signal test_stress_modifying_handlers()
ee14a38 
* cycle_handlers() now waits until at least one signal is received.
* num_received_signals can be equal to num_sent_signals.
@bedevere-app bedevere-app bot mentioned this issue Oct 10, 2023
Merged
vstinner added a commit that referenced this issue Oct 10, 2023
@vstinner
gh-110647: Fix signal test_stress_modifying_handlers() (#110650)
e07c37c 
* cycle_handlers() now waits until at least one signal is received.
* num_received_signals can be equal to num_sent_signals.
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Oct 10, 2023
@vstinner @miss-islington
pythongh-110647: Fix signal test_stress_modifying_handlers() (pythonG…
71cc381 
…H-110650)

* cycle_handlers() now waits until at least one signal is received.
* num_received_signals can be equal to num_sent_signals.
(cherry picked from commit e07c37c)

Co-authored-by: Victor Stinner <vstinner@python.org>
This was referenced Oct 10, 2023
Merged
Merged
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Oct 11, 2023
@vstinner @miss-islington
pythongh-110647: Fix signal test_stress_modifying_handlers() (pythonG…
0cec86e 
…H-110650)

* cycle_handlers() now waits until at least one signal is received.
* num_received_signals can be equal to num_sent_signals.
(cherry picked from commit e07c37c)

Co-authored-by: Victor Stinner <vstinner@python.org>
vstinner added a commit that referenced this issue Oct 11, 2023
@miss-islington @vstinner
[3.11] gh-110647: Fix signal test_stress_modifying_handlers() (GH-110650
46347d3 
) (#110659)

gh-110647: Fix signal test_stress_modifying_handlers() (GH-110650)

* cycle_handlers() now waits until at least one signal is received.
* num_received_signals can be equal to num_sent_signals.
(cherry picked from commit e07c37c)

Co-authored-by: Victor Stinner <vstinner@python.org>
vstinner added a commit that referenced this issue Oct 11, 2023
@miss-islington @vstinner
[3.12] gh-110647: Fix signal test_stress_modifying_handlers() (GH-110650
dcd47e5 
) (#110658)

gh-110647: Fix signal test_stress_modifying_handlers() (GH-110650)

* cycle_handlers() now waits until at least one signal is received.
* num_received_signals can be equal to num_sent_signals.
(cherry picked from commit e07c37c)

Co-authored-by: Victor Stinner <vstinner@python.org>
Glyphack pushed a commit to Glyphack/cpython that referenced this issue Sep 2, 2024
@vstinner @Glyphack
pythongh-110647: Fix signal test_stress_modifying_handlers() (python#…
4896d1c 
…110650)

* cycle_handlers() now waits until at least one signal is received.
* num_received_signals can be equal to num_sent_signals.
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
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vstinner
0