Skip to content

Commit ec38a39

Browse files
kumaraditya303miss-islington
authored andcommitted
gh-153201: Make TSan CI mandatory (GH-153491)
(cherry picked from commit d071438) Co-authored-by: Kumar Aditya <kumaraditya@python.org>
1 parent cb912e1 commit ec38a39

1 file changed

Lines changed: 1 addition & 13 deletions

File tree

.github/workflows/build.yml

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -601,6 +601,7 @@ jobs:
601601
- Thread
602602
free-threading:
603603
- false
604+
- true
604605
sanitizer:
605606
- TSan
606607
include:
@@ -612,17 +613,6 @@ jobs:
612613
sanitizer: ${{ matrix.sanitizer }}
613614
free-threading: ${{ matrix.free-threading }}
614615

615-
# XXX: Temporarily allow this job to fail to not block PRs.
616-
build-san-free-threading:
617-
# ${{ '' } is a hack to nest jobs under the same sidebar category.
618-
name: Sanitizers${{ '' }} # zizmor: ignore[obfuscation]
619-
needs: build-context
620-
if: needs.build-context.outputs.run-ubuntu == 'true'
621-
uses: ./.github/workflows/reusable-san.yml
622-
with:
623-
sanitizer: TSan
624-
free-threading: true
625-
626616
cross-build-linux:
627617
name: Cross build Linux
628618
runs-on: ubuntu-latest
@@ -726,7 +716,6 @@ jobs:
726716
- test-hypothesis
727717
- build-asan
728718
- build-san
729-
- build-san-free-threading
730719
- cross-build-linux
731720
- cifuzz
732721
if: always()
@@ -738,7 +727,6 @@ jobs:
738727
allowed-failures: >-
739728
build-android,
740729
build-emscripten,
741-
build-san-free-threading,
742730
build-windows-msi,
743731
build-ubuntu-ssltests,
744732
test-hypothesis,

0 commit comments

Comments
 (0)