Refactor Fast DDS Ubuntu CI to include several tests (#4860)

* Refs #20822: Split current job into different jobs Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Add shapes demo build job Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Add discovery server build test job Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Add python build test job Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Add documentation build test job Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Remove previous documentation workflow Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Avoid building with tests if not running tests Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Try to fix documentation job Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Move conditions to reusable workflows Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Apply internal review suggestions Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Refactor meta files into build and test, and combine them in test steps Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Remove matrix from ubuntu CI Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Revert "Refs #20822: Move conditions to reusable workflows" This reverts commit d4eea0d. * Refs #20822: Apply rev suggestions Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Add missing discovery server requirements in the job Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Apply rev suggestions (2) Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Fixing windows workflow by merging metas Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Add step with alternative builds Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Use colcon test meta in all jobs, and fix xfail filter Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Remove redundant CMake args, and fix alternative builds Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Apply rev suggestions (3) Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Explicitly compare boolean inputs Signed-off-by: eduponz <eduardoponz@eprosima.com> * Refs #20822: Install tools on test steps to avoid losing symlink permissions Signed-off-by: eduponz <eduardoponz@eprosima.com> * Refs #20822: Update discovery_server meta file name Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Adjust ternary operators Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> * Refs #20822: Use CCache based on an input parameter Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> --------- Signed-off-by: JesusPoderoso <jesuspoderoso@eprosima.com> Signed-off-by: eduponz <eduardoponz@eprosima.com> Co-authored-by: eduponz <eduardoponz@eprosima.com> (cherry picked from commit 22338f3)
eProsima · Jun 18, 2024 · fc3c06a · fc3c06a
1 parent 15a93a0
commit fc3c06a
Show file tree

Hide file tree

Showing 15 changed files with 759 additions and 246 deletions.
diff --git a/.github/workflows/config/asan_colcon.meta → .github/workflows/config/asan.meta b/.github/workflows/config/asan_colcon.meta → .github/workflows/config/asan.meta
diff --git a/.github/workflows/config/discovery_server.meta b/.github/workflows/config/discovery_server.meta
@@ -0,0 +1,11 @@
+names:
+    fastrtps:
+        cmake-args:
+            - "-DCOMPILE_TOOLS=ON"
+            - "-DINSTALL_TOOLS=ON"
+    discovery-server:
+        ctest-args: [
+                "--repeat", "until-pass:3",
+                "--timeout", "300",
+                "--label-exclude", "xfail"
+        ]
diff --git a/.github/workflows/config/documentation.meta b/.github/workflows/config/documentation.meta
@@ -1,15 +1,11 @@
 names:
-    fastrtps:
-        cmake-args:
-            - "-DBUILD_SHARED_LIBS=ON"
-            - "-DSECURITY=ON"
-            - "-DTHIRDPARTY_Asio=FORCE"
-            - "-DTHIRDPARTY_TinyXML2=FORCE"
-            - "-DTHIRDPARTY_UPDATE=OFF"
     fastdds-docs:
         cmake-args:
             - "-DBUILD_DOCUMENTATION=ON"
             - "-DCOMPILE_TESTS=ON"
+        ctest-args: [
+                "--timeout", "500"
+        ]
     googletest-distribution:
         cmake-args:
             - "-Dgtest_force_shared_crt=ON"

diff --git a/.github/workflows/config/ci.meta → .github/workflows/config/fastdds_build.meta b/.github/workflows/config/ci.meta → .github/workflows/config/fastdds_build.meta
@@ -2,27 +2,12 @@ names:
     fastrtps:
         cmake-args:
             - "-DBUILD_DOCUMENTATION=OFF"
-            - "-DBUILD_SHARED_LIBS=ON"
             - "-DCOMPILE_EXAMPLES=ON"
-            - "-DEPROSIMA_BUILD_TESTS=ON"
             - "-DFASTDDS_ENFORCE_LOG_INFO=ON"
-            - "-DFASTDDS_PIM_API_TESTS=ON"
-            - "-DFASTDDS_STATISTICS=ON"
-            - "-DFASTRTPS_API_TESTS=OFF"
-            - "-DGTEST_INDIVIDUAL=ON"
             - "-DINSTALL_EXAMPLES=ON"
-            - "-DINSTALL_TOOLS=ON"
             - "-DINTERNAL_DEBUG=ON"
             - "-DNO_TLS=OFF"
-            - "-DPERFORMANCE_TESTS=ON"
-            - "-DPROFILING_TESTS=OFF"
             - "-DSHM_TRANSPORT_DEFAULT=ON"
             - "-DSTRICT_REALTIME=OFF"
-            - "-DSYSTEM_TESTS=ON"
             - "-DMEMORYCHECK_COMMAND_OPTIONS=-q --tool=memcheck --leak-check=yes --show-reachable=yes --num-callers=50 --log-fd=2 --error-exitcode=1"
             - "-DMEMORYCHECK_SUPPRESSIONS_FILE=../../src/fastrtps/valgrind.supp"
-    googletest-distribution:
-        cmake-args:
-            - "-Dgtest_force_shared_crt=ON"
-            - "-DBUILD_SHARED_LIBS=ON"
-            - "-DBUILD_GMOCK=ON"
diff --git a/.github/workflows/config/fastdds_test.meta b/.github/workflows/config/fastdds_test.meta
@@ -0,0 +1,21 @@
+names:
+    fastrtps:
+        cmake-args:
+            - "-DEPROSIMA_BUILD_TESTS=ON"
+            - "-DFASTDDS_PIM_API_TESTS=ON"
+            - "-DFASTRTPS_API_TESTS=OFF"
+            - "-DGTEST_INDIVIDUAL=ON"
+            - "-DPERFORMANCE_TESTS=ON"
+            - "-DPROFILING_TESTS=OFF"
+            - "-DCOMPILE_TOOLS=ON"
+            - "-DSYSTEM_TESTS=ON"
+        ctest-args: [
+                "--repeat", "until-pass:3",
+                "--timeout", "300",
+                "--label-exclude", "xfail"
+        ]
+    googletest-distribution:
+        cmake-args:
+            - "-Dgtest_force_shared_crt=ON"
+            - "-DBUILD_SHARED_LIBS=ON"
+            - "-DBUILD_GMOCK=ON"
diff --git a/.github/workflows/config/ci.repos → .github/workflows/config/fastdds_test.repos b/.github/workflows/config/ci.repos → .github/workflows/config/fastdds_test.repos
diff --git a/.github/workflows/config/python_test.meta b/.github/workflows/config/python_test.meta
@@ -0,0 +1,15 @@
+names:
+    fastdds_python:
+        cmake-args:
+            - "-DBUILD_TESTING=ON"
+            - "-DBUILD_DOCUMENTATION=ON"
+        ctest-args: [
+                "--repeat", "until-pass:3",
+                "--timeout", "300",
+                "--label-exclude", "xfail"
+        ]
+    googletest-distribution:
+        cmake-args:
+            - "-Dgtest_force_shared_crt=ON"
+            - "-DBUILD_SHARED_LIBS=ON"
+            - "-DBUILD_GMOCK=ON"
diff --git a/.github/workflows/config/tsan_colcon.meta → .github/workflows/config/tsan.meta b/.github/workflows/config/tsan_colcon.meta → .github/workflows/config/tsan.meta
diff --git a/.github/workflows/documentation-tests.yaml b/.github/workflows/documentation-tests.yaml
diff --git a/.github/workflows/reusable-mac-ci.yml b/.github/workflows/reusable-mac-ci.yml
@@ -37,7 +37,6 @@ jobs:
     # so we'll use that one for now, as clang 15 is the supported compiler for Fast DDS in macOS.
     # (see https://github.com/eProsima/Fast-DDS/blob/master/PLATFORM_SUPPORT.md#compilers)
     runs-on: macos-13
-    if: ${{ !contains(github.event.pull_request.labels.*.name, 'skip-ci') }}
     strategy:
       fail-fast: false
       matrix:
@@ -111,7 +110,7 @@ jobs:
       - name: Fetch Fast DDS CI dependencies
         uses: eProsima/eProsima-CI/multiplatform/vcs_import@v0
         with:
-          vcs_repos_file: ${{ github.workspace }}/src/fastrtps/.github/workflows/config/ci.repos
+          vcs_repos_file: ${{ github.workspace }}/src/fastrtps/.github/workflows/config/fastdds_test.repos
           destination_workspace: src
           skip_existing: 'true'
 
@@ -123,7 +122,7 @@ jobs:
         continue-on-error: false
         uses: eProsima/eProsima-CI/multiplatform/colcon_build@v0
         with:
-          colcon_meta_file: ${{ github.workspace }}/src/fastrtps/.github/workflows/config/ci.meta
+          colcon_meta_file: ${{ github.workspace }}/src/fastrtps/.github/workflows/config/fastdds_build.meta ${{ github.workspace }}/src/fastrtps/.github/workflows/config/fastdds_test.meta
           colcon_build_args: ${{ inputs.colcon-args }}
           cmake_args: ${{ inputs.cmake-args }}
           cmake_args_default: -DCMAKE_CXX_FLAGS="-Werror -Wall -Wno-enum-constexpr-conversion"
@@ -135,10 +134,10 @@ jobs:
         if: ${{ !contains(github.event.pull_request.labels.*.name, 'no-test') }}
         uses: eProsima/eProsima-CI/multiplatform/colcon_test@v0
         with:
+          colcon_meta_file: ${{ github.workspace }}/src/fastrtps/.github/workflows/config/fastdds_test.meta
           colcon_test_args: ${{ inputs.colcon-args }}
           colcon_test_args_default: --event-handlers=console_direct+
           ctest_args: ${{ inputs.ctest-args }}
-          ctest_args_default: --repeat until-pass:3 --timeout 300 --label-exclude "xfail"
           packages_names: fastrtps
           workspace: ${{ github.workspace }}
           test_report_artifact: ${{ format('test_report_{0}_{1}_{2}', inputs.label, github.job, join(matrix.*, '_')) }}