Skip to content

Commit 28dfaaa

Browse files
authored
Fix TSAN CI to use gcc-12 (#116)
* Fix TSAN CI to use gcc-12 Signed-off-by: Raul Sanchez-Mateos <raul@eprosima.com> * Remove warning note in tsan job Signed-off-by: Raul Sanchez-Mateos <raul@eprosima.com> --------- Signed-off-by: Raul Sanchez-Mateos <raul@eprosima.com>
1 parent 4f16869 commit 28dfaaa

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

.github/workflows/reusable-workflow.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -153,9 +153,6 @@ jobs:
153153
# TSAN
154154

155155
tsan:
156-
# NOTE: there is a known issue in TSAN 20.04 with std::condition_variable::wait_for
157-
# https://github.com/google/sanitizers/issues/1259
158-
# Until this is fixed, we use 20.04 for TSAN work
159156
runs-on: ubuntu-22.04
160157
steps:
161158

@@ -176,6 +173,11 @@ jobs:
176173

177174
- name: Compile and run tests
178175
uses: eProsima/eProsima-CI/multiplatform/colcon_build_test@v0
176+
env:
177+
# GCC 11.3 (Ubuntu Jammy default) produces several false positives regarding timed synchronization protocols
178+
# These issues were fixed in GCC 12 so we upgrade to that version.
179+
CC: gcc-12
180+
CXX: g++-12
179181
with:
180182
packages_names: ${{ env.cpp_packages_names }}
181183
cmake_args: -DBUILD_TESTS=ON -DTSAN_BUILD=ON

0 commit comments

Comments
 (0)