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]>
eProsima · Jul 11, 2024 · 203725d · 203725d
1 parent 0aa63eb
commit 203725d
Show file tree

Hide file tree

Showing 14 changed files with 600 additions and 140 deletions.
diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
@@ -17,7 +17,7 @@
     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 1.1.x 1.0.x -->
+<!-- @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) -->
@@ -36,7 +36,7 @@
 - [ ] 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 ABI compatible. <!-- Bug fixes should be ABI compatible if possible so a backport to previous affected releases can be made. -->
+- [ ] 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.

diff --git a/.github/workflows/config/colcon.meta → .github/workflows/config/build.meta b/.github/workflows/config/colcon.meta → .github/workflows/config/build.meta
@@ -1,6 +1,10 @@
 names:
+    fastcdr:
+        cmake-args:
+            - "-DBUILD_TESTING=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'))) }}
diff --git a/.github/workflows/nightly-mac-ci.yml b/.github/workflows/nightly-mac-ci.yml
@@ -0,0 +1,57 @@
+name: Fast CDR Mac CI (nightly)
+
+on:
+  workflow_dispatch:
+  schedule:
+    - cron: '0 1 * * *'
+
+jobs:
+  nightly-mac-ci-master:
+    uses: eProsima/Fast-CDR/.github/workflows/reusable-ci.yml@master
+    with:
+      os-version: 'macos-13'
+      label: 'nightly-mac-ci-master'
+      fastcdr-branch: 'master'
+      run-build: true
+      run-tests: true
+      use-ccache: false
+
+  nightly-mac-ci-2_2_x:
+    uses: eProsima/Fast-CDR/.github/workflows/[email protected]
+    with:
+      os-version: 'macos-13'
+      label: 'nightly-mac-ci-2.2.x'
+      fastcdr-branch: '2.2.x'
+      run-build: true
+      run-tests: true
+      use-ccache: false
+
+  nightly-mac-ci-2_1_x:
+    uses: eProsima/Fast-CDR/.github/workflows/[email protected]
+    with:
+      os-version: 'macos-13'
+      label: 'nightly-mac-ci-2.1.x'
+      fastcdr-branch: '2.1.x'
+      run-build: true
+      run-tests: true
+      use-ccache: false
+
+  nightly-mac-ci-1_1_x:
+    uses: eProsima/Fast-CDR/.github/workflows/[email protected]
+    with:
+      os-version: 'macos-13'
+      label: 'nightly-mac-ci-1.1.x'
+      fastcdr-branch: '1.1.x'
+      run-build: true
+      run-tests: true
+      use-ccache: false
+
+  nightly-mac-ci-1_0_x:
+    uses: eProsima/Fast-CDR/.github/workflows/[email protected]
+    with:
+      os-version: 'mac-13'
+      label: 'nightly-mac-ci-1.0.x'
+      fastcdr-branch: '1.0.x'
+      run-build: true
+      run-tests: true
+      use-ccache: false
diff --git a/.github/workflows/nightly-ubuntu-ci.yml b/.github/workflows/nightly-ubuntu-ci.yml
@@ -0,0 +1,57 @@
+name: Fast CDR Ubuntu CI (nightly)
+
+on:
+  workflow_dispatch:
+  schedule:
+    - cron: '0 1 * * *'
+
+jobs:
+  nightly-ubuntu-ci-master:
+    uses: eProsima/Fast-CDR/.github/workflows/reusable-ci.yml@master
+    with:
+      os-version: 'ubuntu-22.04'
+      label: 'nightly-ubuntu-ci-master'
+      fastcdr-branch: 'master'
+      run-build: true
+      run-tests: true
+      use-ccache: false
+
+  nightly-ubuntu-ci-2_2_x:
+    uses: eProsima/Fast-CDR/.github/workflows/[email protected]
+    with:
+      os-version: 'ubuntu-22.04'
+      label: 'nightly-ubuntu-ci-2.2.x'
+      fastcdr-branch: '2.2.x'
+      run-build: true
+      run-tests: true
+      use-ccache: false
+
+  nightly-ubuntu-ci-2_1_x:
+    uses: eProsima/Fast-CDR/.github/workflows/[email protected]
+    with:
+      os-version: 'ubuntu-22.04'
+      label: 'nightly-ubuntu-ci-2.1.x'
+      fastcdr-branch: '2.1.x'
+      run-build: true
+      run-tests: true
+      use-ccache: false
+
+  nightly-ubuntu-ci-1_1_x:
+    uses: eProsima/Fast-CDR/.github/workflows/[email protected]
+    with:
+      os-version: 'ubuntu-22.04'
+      label: 'nightly-ubuntu-ci-1.1.x'
+      fastcdr-branch: '1.1.x'
+      run-build: true
+      run-tests: true
+      use-ccache: false
+
+  nightly-ubuntu-ci-1_0_x:
+    uses: eProsima/Fast-CDR/.github/workflows/[email protected]
+    with:
+      os-version: 'ubuntu-20.04'
+      label: 'nightly-ubuntu-ci-1.0.x'
+      fastcdr-branch: '1.0.x'
+      run-build: true
+      run-tests: true
+      use-ccache: false