ci: improve TSan job reliability

- Use RelWithDebInfo instead of Debug (2-3x faster under TSan)
- Set halt_on_error=0 so TSan reports races without killing the process
- Add history_size=4 for better race reports
- Run ctest with -j1 to avoid parallel test interference under TSan

Author: bburda

.github/workflows/quality.yml

@@ -350,7 +350,7 @@ jobs:
         run: |
           source /opt/ros/jazzy/setup.bash
           colcon build --symlink-install \
-            --cmake-args -DCMAKE_BUILD_TYPE=Debug -DSANITIZER=tsan \
+            --cmake-args -DCMAKE_BUILD_TYPE=RelWithDebInfo -DSANITIZER=tsan \
             --event-handlers console_direct+
           ccache -s
 
@@ -362,14 +362,14 @@ jobs:
       - name: Run unit tests with TSan
         timeout-minutes: 15
         run: |
-          export TSAN_OPTIONS="halt_on_error=1:suppressions=$(pwd)/tsan_suppressions.txt"
+          export TSAN_OPTIONS="halt_on_error=0:history_size=4:suppressions=$(pwd)/tsan_suppressions.txt"
           source /opt/ros/jazzy/setup.bash
           source install/setup.bash
           failed=0
           for pkg_dir in build/ros2_medkit_*/; do
             pkg=$(basename "$pkg_dir")
             echo "::group::Testing $pkg"
-            (cd "$pkg_dir" && ctest -LE "linter|integration" --output-on-failure) || failed=1
+            (cd "$pkg_dir" && ctest -j1 -LE "linter|integration" --output-on-failure) || failed=1
             echo "::endgroup::"
           done
           exit $failed