Fix and refactor Windows Github CI (#4356)

* Fix and refactor Windows Github CI (#4356) * Refs #20318: Fix passing of RUNTIME_LIST to test discovery Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Refs #20318: Refactor windows workflow Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Refs #20318: Compile warnings as errors Signed-off-by: EduPonz <eduardoponz@eprosima.com> Refs #20318: Only use warnings as errors for Fast DDS Signed-off-by: EduPonz <eduardoponz@eprosima.com> Refs #20318: Do not override existing CXX_FLAGS Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Refs #20318: Use thirdparty for Asio and TinyXML2 Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Refs #20318: Add cancel-in-progress to sanitizer jobs Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Refs #20318: Apply Ricardo's suggestions Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Refs #20318. Avoid build error on default_macros_test Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #20318: Adjust working dir for test report Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Refs #20318: Uncrustify Signed-off-by: EduPonz <eduardoponz@eprosima.com> --------- Signed-off-by: EduPonz <eduardoponz@eprosima.com> Signed-off-by: Miguel Company <miguelcompany@eprosima.com> Co-authored-by: Miguel Company <miguelcompany@eprosima.com> (cherry picked from commit 7adc833) Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Fix conflicts Signed-off-by: Miguel Company <miguelcompany@eprosima.com> Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Test report action does not run when no-test label is set (#4366) * Refs #20400: Test report action does not run when no-test label is set Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Refs #20400: Add GTest cmake options to nightly.meta Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Refs #20400: Use test step conclusion to decide whether to run test report step Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Refs #20400: Apply Miguel's suggestion Signed-off-by: EduPonz <eduardoponz@eprosima.com> --------- Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Refs #20318. Add GTEST_INDIVIDUAL=ON Signed-off-by: Miguel Company <miguelcompany@eprosima.com> Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Refs #20318: Add test summary step Signed-off-by: EduPonz <eduardoponz@eprosima.com> * Use tagged version for test_reporter action Signed-off-by: Miguel Company <miguelcompany@eprosima.com> * Refs #20318: Enforce unique name for test artifacts Signed-off-by: EduPonz <eduardoponz@eprosima.com> --------- Signed-off-by: EduPonz <eduardoponz@eprosima.com> Signed-off-by: Miguel Company <miguelcompany@eprosima.com> Co-authored-by: Eduardo Ponz Segrelles <eduardoponz@eprosima.com> Co-authored-by: Miguel Company <miguelcompany@eprosima.com>
eProsima · Feb 14, 2024 · 5b98004 · 5b98004
1 parent 366a9c8
commit 5b98004
Show file tree

Hide file tree

Showing 6 changed files with 121 additions and 638 deletions.
diff --git a/.github/workflows/config/default_ci.meta b/.github/workflows/config/default_ci.meta
@@ -1,25 +1,34 @@
-{
-    "names":
-    {
-        "fastrtps" :
-        {
-            "cmake-args": [
-                "-DEPROSIMA_BUILD=ON",
-                "-DINTERNAL_DEBUG=ON",
-                "-DGTEST_INDIVIDUAL=ON",
-                "-DFASTRTPS_API_TESTS=OFF",
-                "-DFASTDDS_PIM_API_TESTS=ON",
-                "-DPERFORMANCE_TESTS=ON",
-                "-DNO_TLS=ON",
-                "-DSECURITY=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"]
-        }
-    }
-}
+names:
+    fastrtps:
+        cmake-args:
+            - "-DBUILD_DOCUMENTATION=OFF"
+            - "-DBUILD_SHARED_LIBS=ON"
+            - "-DCOMPILE_EXAMPLES=ON"
+            - "-DEPROSIMA_BUILD_TESTS=ON"
+            - "-DEPROSIMA_EXTRA_CMAKE_CXX_FLAGS=/MP /WX"
+            - "-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"
+            - "-DSECURITY=ON"
+            - "-DSHM_TRANSPORT_DEFAULT=ON"
+            - "-DSTRICT_REALTIME=OFF"
+            - "-DSYSTEM_TESTS=ON"
+            - "-DTHIRDPARTY_Asio=FORCE"
+            - "-DTHIRDPARTY_fastcdr=OFF"
+            - "-DTHIRDPARTY_TinyXML2=FORCE"
+            - "-DTHIRDPARTY_UPDATE=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/nightly.meta b/.github/workflows/config/nightly.meta
@@ -1,24 +1,33 @@
-{
-    "names":
-    {
-        "fastrtps" :
-        {
-            "cmake-args": [
-                "-DEPROSIMA_BUILD=ON",
-                "-DINTERNAL_DEBUG=ON",
-                "-DGTEST_INDIVIDUAL=ON",
-                "-DFASTRTPS_API_TESTS=OFF",
-                "-DFASTDDS_PIM_API_TESTS=ON",
-                "-DPERFORMANCE_TESTS=ON",
-                "-DNO_TLS=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"]
-        }
-    }
-}
+names:
+    fastrtps:
+        cmake-args:
+            - "-DBUILD_DOCUMENTATION=OFF"
+            - "-DBUILD_SHARED_LIBS=ON"
+            - "-DCOMPILE_EXAMPLES=ON"
+            - "-DEPROSIMA_BUILD_TESTS=ON"
+            - "-DEPROSIMA_EXTRA_CMAKE_CXX_FLAGS=/MP /WX"
+            - "-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"
+            - "-DTHIRDPARTY_Asio=FORCE"
+            - "-DTHIRDPARTY_fastcdr=OFF"
+            - "-DTHIRDPARTY_TinyXML2=FORCE"
+            - "-DTHIRDPARTY_UPDATE=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"