Skip to content

Commit

Permalink
Refactor Github CI sanitizer related jobs (#4662) (#4677)
Browse files Browse the repository at this point in the history
* Refactor Github CI sanitizer related jobs (#4662)

* Refs #20542: Add Fast DDS Sanitizer worflow with support for asan fastdds

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20542: Add asan_discovery_server_test job

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20542: Add tsan_fastdds_test job

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20542: Remove colcon_meta input as max inputs for workflow_dispath is 10

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20542: Create a reusable workflow and a CI one for PRs or manual

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20542: Add nightly sanitizer CI for all supported branches

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20542: Change extension to .yml

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20542: Remove old workflows

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20542: Run sanitizers CI on PR review request

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20542: Exclude xfail tests in discovery server test run

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20542: Install all python packages needed for asan_fastdds_test reporting

Signed-off-by: eduponz <eduardoponz@eprosima.com>

* Refs #20542: Apply Jesús' suggestions

Signed-off-by: eduponz <eduardoponz@eprosima.com>

---------

Signed-off-by: eduponz <eduardoponz@eprosima.com>
(cherry picked from commit ac0e737)

# Conflicts:
#	.github/workflows/address-sanitizers.yaml
#	.github/workflows/config/asan_colcon.meta
#	.github/workflows/thread-sanitizer.yaml

* Refs #20703: Fix conflicts

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20703: Remove sanitizer nightlies

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20703: Remove old sanitizer jobs

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20703: set default discovery server branch to v1.2.1

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* #20703: Address review changes

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

---------

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
Co-authored-by: Eduardo Ponz Segrelles <eduardoponz@eprosima.com>
Co-authored-by: Mario Dominguez <mariodominguez@eprosima.com>
  • Loading branch information
3 people authored Apr 17, 2024
1 parent 8032e65 commit 26231b4
Show file tree
Hide file tree
Showing 5 changed files with 541 additions and 456 deletions.
245 changes: 0 additions & 245 deletions .github/workflows/address-sanitizers.yaml

This file was deleted.

8 changes: 0 additions & 8 deletions .github/workflows/config/asan_colcon.meta
Original file line number Diff line number Diff line change
@@ -1,21 +1,13 @@
names:
fastrtps:
cmake-args:
- "-DCMAKE_BUILD_TYPE=Debug"
- "-DEPROSIMA_BUILD_TESTS=ON"
- "-DGTEST_INDIVIDUAL=ON"
- "-DRTPS_API_TESTS=ON"
- "-DFASTRTPS_API_TESTS=ON"
- "-DFASTDDS_PIM_API_TESTS=ON"
- "-DPERFORMANCE_TESTS=ON"
- "-DNO_TLS=OFF"
- "-DSECURITY=ON"
- "-DFASTDDS_STATISTICS=ON"
- "-DSANITIZER=Address"
- "-DCMAKE_CXX_FLAGS='-Werror'"
discovery-server:
cmake-args:
- "-DCMAKE_BUILD_TYPE=Debug"
- "-DSANITIZER=Address"
googletest-distribution:
cmake-args:
Expand Down
Loading

0 comments on commit 26231b4

Please sign in to comment.