From: Otto <otto.moerbeek@open-xchange.com>
Date: Wed, 10 Nov 2021 10:24:30 +0000 (+0100)
Subject: Enable tsan build plus tests for rec GH actions
X-Git-Tag: dnsdist-1.7.0-beta1~7^2~3
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=719e5764291e292d0b31f619cb0f69cac4294285;p=thirdparty%2Fpdns.git

Enable tsan build plus tests for rec GH actions
---

diff --git a/.github/workflows/build-and-test-all.yml b/.github/workflows/build-and-test-all.yml
index a374dbe764..d9fd887d50 100644
--- a/.github/workflows/build-and-test-all.yml
+++ b/.github/workflows/build-and-test-all.yml
@@ -52,7 +52,7 @@ jobs:
     runs-on: ubuntu-20.04
     strategy:
       matrix:
-        sanitizers: [ubsan+asan]
+        sanitizers: [ubsan+asan, tsan]
     env:
       UBSAN_OPTIONS: "print_stacktrace=1:halt_on_error=1:suppressions=${{ github.workspace }}/build-scripts/UBSan.supp"
       ASAN_OPTIONS: detect_leaks=0
@@ -277,7 +277,7 @@ jobs:
     runs-on: ubuntu-20.04
     strategy:
       matrix:
-        sanitizers: [ubsan+asan]
+        sanitizers: [ubsan+asan, tsan]
     env:
       UBSAN_OPTIONS: "print_stacktrace=1:halt_on_error=1:suppressions=${{ github.workspace }}/build-scripts/UBSan.supp"
       ASAN_OPTIONS: detect_leaks=0
@@ -303,7 +303,7 @@ jobs:
     runs-on: ubuntu-20.04
     strategy:
       matrix:
-        sanitizers: [ubsan+asan]
+        sanitizers: [ubsan+asan, tsan]
     env:
       UBSAN_OPTIONS: 'print_stacktrace=1:halt_on_error=1:suppressions=/home/runner/work/pdns/pdns/build-scripts/UBSan.supp'
       ASAN_OPTIONS: detect_leaks=0
@@ -330,7 +330,7 @@ jobs:
     runs-on: ubuntu-20.04
     strategy:
       matrix:
-        sanitizers: [ubsan+asan]
+        sanitizers: [ubsan+asan, tsan]
         threads: [1, 2, 3, 4, 8, 16]
         mthreads: [2048, 4096]
         shards: [1, 2, 1028]
diff --git a/pdns/recursordist/recursor-tsan.supp b/pdns/recursordist/recursor-tsan.supp
new file mode 100644
index 0000000000..9c7633d1ea
--- /dev/null
+++ b/pdns/recursordist/recursor-tsan.supp
@@ -0,0 +1,2 @@
+# We don't care about numberOfDistributedQueries
+race:doStats