Uh oh!
There was an error while loading. Please reload this page.
Update thread sanitizer CI to use a Python 3.15 image - #243
Conversation
kumaraditya303
left a comment
There was a problem hiding this comment.
I can't reproduce the TSan failures under 3.15. There have been a number of thread safety fixes to the standard library since 3.14, so in addition to fixing the issue we're hitting in CI, we can also completely remove the suppressions file as well.
Yes, in main branch all of them are fixed.
This LGTM
mattip
commented
Apr 21, 2026
I am a little uncomfortable using an alpha-release python as the ground truth for the TSAN builds. What does numpy do? |
ngoldbaum
commented
Apr 21, 2026
NumPy uses Python 3.14, although I personally think it'd probably be fine to update those builds.
Fair enough, let me see if I can add a suppression for the issue CI is currently hitting. |
ngoldbaum
commented
Apr 21, 2026
See #244. |
I can't reproduce the TSan failures under 3.15. There have been a number of thread safety fixes to the standard library since 3.14, so in addition to fixing the issue we're hitting in CI, we can also completely remove the suppressions file as well.
Also adds test timeouts per the advice here.
Ping @kumaraditya303 since he originally set all this up last year.
Fixes#242.