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

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

Merged
merged 28 commits into from
Mar 16, 2024
Merged

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

merged 28 commits into from
Mar 16, 2024

Conversation

corona10
Copy link
Member

@corona10 corona10 commented Mar 15, 2024 •
edited by bedevere-app bot
Loading

@bedevere-app bedevere-app bot mentioned this pull request Mar 15, 2024
Closed
@pitrou pitrou mentioned this pull request Mar 15, 2024
Merged
pitrou
pitrou reviewed Mar 15, 2024
View reviewed changes
.github/workflows/build.yml Outdated Show resolved Hide resolved
pitrou
pitrou reviewed Mar 15, 2024
View reviewed changes
.github/workflows/build.yml Outdated Show resolved Hide resolved
@pitrou
Copy link
Member

pitrou commented Mar 15, 2024

Ah, yeah, "ThreadSanitizer: unexpected memory mapping" is due to a recent kernel update.
You need this workaround:
https://github.com/apache/arrow/blob/00a48217e93bea1e84f10dbfdf3c0c93dfe1ea3d/dev/tasks/docker-tests/github.linux.yml#L38-L39

@corona10 corona10 marked this pull request as ready for review March 15, 2024 18:50
@corona10 corona10 requested a review from colesbury March 15, 2024 18:50
@corona10
Copy link
Member Author

In another PR we should add more tests to --tsan, because there are currently very few of them.

Yeah, yeah, I agree; this is just the initial setup; we need to do more tests for better coverage.

@corona10 corona10 requested a review from vsajip as a code owner March 16, 2024 01:36
@corona10 corona10 requested review from colesbury and pitrou March 16, 2024 02:36
pitrou
pitrou approved these changes Mar 16, 2024
View reviewed changes
Copy link
Member

@pitrou pitrou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great addition! Thank you @corona10 !

@pitrou pitrou changed the title gh-112536: Setting up TSAN test at Github Action gh-112536: Add TSAN builds on Github Actions Mar 16, 2024
@pitrou pitrou merged commit 20578a1 into python:main Mar 16, 2024
35 of 36 checks passed
@corona10 corona10 deleted the gh-112536-ci branch March 16, 2024 10:15
@itamaro
Copy link
Contributor

itamaro commented Mar 16, 2024

Are there buildbots for this configuration too?

@pitrou
Copy link
Member

pitrou commented Mar 16, 2024 •
edited
Loading

No, just GHA AFAIK.

@bedevere-app
Copy link

bedevere-app bot commented Mar 18, 2024

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

pitrou pushed a commit to pitrou/cpython that referenced this pull request Mar 18, 2024
@corona10 @pitrou
[3.12] pythongh-112536: Add TSAN builds on Github Actions (pythonGH-1…
0be64cb 
…16872)

(cherry picked from commit 20578a1)

Co-authored-by: Donghee Na <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Mar 18, 2024

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

pitrou added a commit that referenced this pull request Mar 18, 2024
@pitrou @corona10
[3.12] gh-112536: Add TSAN build on Github Actions (GH-116872)
25243b1 
(cherry picked from commit 20578a1)

Co-authored-by: Donghee Na <[email protected]>
vstinner pushed a commit to vstinner/cpython that referenced this pull request Mar 20, 2024
@corona10 @vstinner
pythongh-112536: Add TSAN builds on Github Actions (python#116872)
ba70b85
adorilson pushed a commit to adorilson/cpython that referenced this pull request Mar 25, 2024
@corona10 @adorilson
pythongh-112536: Add TSAN builds on Github Actions (python#116872)
2234f3e
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
@corona10 @diegorusso
pythongh-112536: Add TSAN builds on Github Actions (python#116872)
5ed7bd3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pitrou pitrou pitrou approved these changes

@ezio-melotti ezio-melotti Awaiting requested review from ezio-melotti ezio-melotti is a code owner

@hugovk hugovk Awaiting requested review from hugovk hugovk is a code owner

@vsajip vsajip Awaiting requested review from vsajip vsajip is a code owner

@colesbury colesbury Awaiting requested review from colesbury

Assignees
No one assigned
Labels
skip news topic-free-threading
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@corona10 @pitrou @hugovk @itamaro @colesbury