Improve CI workflows in master (#216)

* Refs #21286: Add Ubuntu CI Signed-off-by: JesusPoderoso <[email protected]> * Refs #21286: Add Windows CI Signed-off-by: JesusPoderoso <[email protected]> * Refs #21286: Add MacOS CI Signed-off-by: JesusPoderoso <[email protected]> * Refs #21286: Remove previous job Signed-off-by: JesusPoderoso <[email protected]> * Refs #21286: Update PR template Signed-off-by: JesusPoderoso <[email protected]> * Refs #21286: Add missing dependency Signed-off-by: JesusPoderoso <[email protected]> * Refs #21286: Fix colcon test warning Signed-off-by: JesusPoderoso <[email protected]> * Refs #21286: Add RELEASE_SUPPORT.md Signed-off-by: JesusPoderoso <[email protected]> * Refs #21286: Apply rev suggestions Signed-off-by: JesusPoderoso <[email protected]> * Refs #21286: Fix labels Signed-off-by: JesusPoderoso <[email protected]> * Refs #21286: Try to fix windows CI Signed-off-by: JesusPoderoso <[email protected]> * Refs #21286: Include GitHub CI badges in readme Signed-off-by: JesusPoderoso <[email protected]> --------- Signed-off-by: JesusPoderoso <[email protected]> (cherry picked from commit 203725d)
eProsima · Jul 11, 2024 · bd54fda · bd54fda
1 parent bdceab5
commit bd54fda
Show file tree

Hide file tree

Showing 11 changed files with 435 additions and 134 deletions.
diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
@@ -0,0 +1,50 @@
+<!-- Provide a general summary of your changes in the Title above -->
+<!-- It must be meaningful and coherent with the changes -->
+
+<!--
+    If this PR is still a Work in Progress [WIP], please open it as DRAFT.
+    Please consider if any label should be added to this PR.
+-->
+
+## Description
+<!--
+    Describe changes in detail.
+    This includes depicting the context, use case or current behavior and describe the proposed changes.
+    If several features/bug fixes are included with these changes, please consider opening separated pull requests.
+-->
+
+<!--
+    In case of bug fixes, please provide the list of supported branches where this fix should be also merged.
+    Please uncomment following line, adjusting the corresponding target branches for the backport.
+-->
+<!-- @Mergifyio backport 2.2.x 2.1.x 1.1.x 1.0.x -->
+
+<!-- If an issue is already opened, please uncomment next line with the corresponding issue number. -->
+<!-- Fixes #(issue) -->
+
+<!-- In case the changes are built over a previous pull request, please uncomment next line. -->
+<!-- This PR depends on #(PR) and must be merged after that one. -->
+
+## Contributor Checklist
+
+<!--
+    - If any of the elements of the following checklist is not applicable, substitute the checkbox [ ] by _N/A_:
+    - If any of the elements of the following checklist is not fulfilled on purpose, please provide a reason and substitute the checkbox [ ] with ❌: or __NO__:.
+-->
+
+- [ ] Commit messages follow the project guidelines. <!-- External contributors should sign the DCO. Fast DDS developers must also refer to the internal Redmine task. -->
+- [ ] The code follows the style guidelines of this project. <!-- Please refer to the [Quality Declaration](https://github.com/eProsima/Fast-CDR/blob/master/QUALITY.md#linters-and-static-analysis-4v) for more information. -->
+- [ ] Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally <!-- Blackbox tests checking the new functionality are required. Changes that add/modify public API must include unit tests covering all possible cases. In case that no tests are provided, please justify why. -->
+- [ ] Any new/modified methods have been properly documented using Doxygen. <!-- Even internal classes, and private methods and members should be documented, not only the public API. -->
+- [ ] Changes are backport compatible: they do **NOT** break ABI nor change library core behavior. <!-- Bug fixes should be ABI compatible if possible so a backport to previous affected releases can be made. -->
+- [ ] Changes are API compatible. <!-- Public API must not be broken within the same major release. -->
+- [ ] New feature has been added to the `versions.md` file (if applicable).
+- [ ] Applicable backports have been included in the description.
+
+## Reviewer Checklist
+
+- [ ] The PR has a milestone assigned.
+- [ ] The title and description correctly express the PR's purpose.
+- [ ] Check contributor checklist is correct.
+- [ ] Check CI results: changes do not issue any warning.
+- [ ] Check CI results: CI pass and failing tests are unrelated with the changes.
diff --git a/.github/workflows/config/build.meta b/.github/workflows/config/build.meta
@@ -0,0 +1,10 @@
+names:
+    fastcdr:
+        cmake-args:
+            - "-EPROSIMA_BUILD_TESTS=ON"
+    googletest-distribution:
+        cmake-args:
+            - "-Dgtest_force_shared_crt=ON"
+            - "-DBUILD_SHARED_LIBS=ON"
+            - "-DBUILD_GMOCK=ON"
+
diff --git a/.github/workflows/config/test.meta b/.github/workflows/config/test.meta
@@ -0,0 +1,7 @@
+names:
+    fastcdr:
+        ctest-args: [
+            "--repeat", "until-pass:3",
+            "--timeout", "300",
+            "--output-junit", "junit/junit.xml"
+        ]
diff --git a/.github/workflows/config/test.repos b/.github/workflows/config/test.repos
@@ -0,0 +1,5 @@
+repositories:
+    googletest-distribution:
+        type: git
+        url: https://github.com/google/googletest.git
+        version: release-1.11.0
diff --git a/.github/workflows/fastcdr-test.yml b/.github/workflows/fastcdr-test.yml
diff --git a/.github/workflows/mac-ci.yml b/.github/workflows/mac-ci.yml
@@ -0,0 +1,59 @@
+name: Fast CDR Mac CI
+
+on:
+  workflow_dispatch:
+    inputs:
+      os-version:
+        description: 'OS version to run the workflow'
+        required: false
+        default: 'macos-13'
+        type: string
+      colcon-args:
+        description: 'Extra arguments for colcon cli'
+        required: false
+        type: string
+      cmake-args:
+        description: 'Extra arguments for cmake cli'
+        required: false
+        type: string
+      ctest-args:
+        description: 'Extra arguments for ctest cli'
+        required: false
+        type: string
+      fastcdr-branch:
+        description: 'Branch or tag of Fast CDR repository'
+        type: string
+        required: true
+      run-tests:
+        description: 'Run test suite of Fast CDR'
+        required: false
+        type: boolean
+        default: true
+
+  pull_request:
+    types:
+      - review_requested
+    paths-ignore:
+      - '**.md'
+      - '**.txt'
+      - '!**/CMakeLists.txt'
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  mac-ci:
+    if: ${{ !(github.event_name == 'pull_request') || !contains(github.event.pull_request.labels.*.name, 'conflicts') }}
+    uses: ./.github/workflows/reusable-ci.yml
+    with:
+      # It would be desirable to have a matrix of macos OS for this job, but due to the issue opened in this ticket:
+      # https://github.com/orgs/community/discussions/128118 , it has been set as a single OS job.
+      os-version: ${{ inputs.os-version || 'macos-13' }}
+      label: ${{ format('mac-ci-{0}', inputs.fastcdr-branch || github.ref) }}
+      colcon-args: ${{ inputs.colcon-args }}
+      cmake-args: ${{ inputs.cmake-args }}
+      ctest-args: ${{ inputs.ctest-args }}
+      fastcdr-branch: ${{ inputs.fastcdr-branch || github.ref }}
+      run-build: ${{ !(github.event_name == 'pull_request') || !contains(github.event.pull_request.labels.*.name, 'skip-ci') }}
+      run-tests: ${{ (inputs.run-tests == true) || ((github.event_name == 'pull_request') && (!contains(github.event.pull_request.labels.*.name, 'no-test'))) }}