Skip to content

gh-112536: Add TSAN builds on Github Actions #116872

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

Merged
pitrou merged 28 commits into from
Mar 16, 2024
Merged

gh-112536: Add TSAN builds on Github Actions #116872

Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
34cd372
gh-112536: Setting up TSAN test at Github Action
corona10 Mar 15, 2024
8f789df
Apply TSAN to the default build too
corona10 Mar 15, 2024
432c3a2
Update
corona10 Mar 15, 2024
7c04959
Use gcc 11 to resolve compiler issue
corona10 Mar 15, 2024
4d271cb
Remove test_logging from the TSAN test for the temporary
corona10 Mar 15, 2024
853ea8c
Test --with-pydebug
corona10 Mar 15, 2024
43135d4
Use Ubuntu 22.04
corona10 Mar 15, 2024
7304fca
Use --disable-ipv6 sigh..
corona10 Mar 15, 2024
37d048d
revert --with-pydebug
corona10 Mar 15, 2024
b05c9f7
Use clang
corona10 Mar 15, 2024
7e373ee
Reduce ASLR to avoid TSAN crashing
corona10 Mar 15, 2024
920d8b4
Rebirth test_logging
corona10 Mar 15, 2024
73cbf74
Add --with-pydebug
corona10 Mar 15, 2024
0ff8212
Update Tools/tsan/supressions.txt
corona10 Mar 15, 2024
c6dfb25
Add ccache
corona10 Mar 15, 2024
e2e90ee
Address code review
corona10 Mar 16, 2024
2d29262
Don't use sem_clockwait with TSAN
corona10 Mar 16, 2024
e2265a5
Skip tests from test_threading and test_concurrent_futures
corona10 Mar 16, 2024
414c0e9
Revert Tools/tsan/supressions.txt changes
corona10 Mar 16, 2024
71dc4aa
Add skip_if_tsan
corona10 Mar 16, 2024
f399bd2
nit
corona10 Mar 16, 2024
ec743ff
Use reusable workflow
corona10 Mar 16, 2024
e99087f
nit
corona10 Mar 16, 2024
8bfee01
Rename to .github/workflows/reusable-tsan.yml
corona10 Mar 16, 2024
8577792
fix
corona10 Mar 16, 2024
f9c6e56
fix
corona10 Mar 16, 2024
d9afff0
Fix
corona10 Mar 16, 2024
e88086f
nit
corona10 Mar 16, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
nit
  • Loading branch information
@corona10
corona10 committed Mar 16, 2024
commit f399bd2724e20154599df225b22bfc760c976ca9
13 changes: 6 additions & 7 deletions Lib/test/test_threading.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,9 @@ def skip_unless_reliable_fork(test):
return test


def skip_for_tsan(test):
if support.check_sanitizer(thread=True):
return test.skip("TSAN doesn't support threads after fork")
return test
skip_if_tsan_fork = unittest.skipIf(
support.check_sanitizer(thread=True),
"TSAN doesn't support threads after fork")


def requires_subinterpreters(meth):
Expand Down Expand Up @@ -640,7 +639,7 @@ def test_daemon_param(self):
self.assertTrue(t.daemon)

@skip_unless_reliable_fork
@skip_for_tsan
@skip_if_tsan_fork
def test_dummy_thread_after_fork(self):
# Issue #14308: a dummy thread in the active list doesn't mess up
# the after-fork mechanism.
Expand Down Expand Up @@ -710,7 +709,7 @@ def f():

@skip_unless_reliable_fork
@unittest.skipUnless(hasattr(os, 'waitpid'), "test needs os.waitpid()")
@skip_for_tsan
@skip_if_tsan_fork
def test_main_thread_after_fork(self):
code = """if 1:
import os, threading
Expand Down Expand Up @@ -1279,7 +1278,7 @@ def test_2_join_in_forked_process(self):
self._run_and_join(script)

@skip_unless_reliable_fork
@skip_for_tsan
@skip_if_tsan_fork
def test_3_join_in_forked_from_thread(self):
# Like the test above, but fork() was called from a worker thread
# In the forked process, the main Thread object must be marked as stopped.
Expand Down