Make skip-ci label report required statuses as pass (#323)

* Refs #25442. Make `skip-ci` label report required statuses as pass.

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

* Refs #25442. Fix condition in Archive Test Results.

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

* Refs #25442. Fix condition for add-label.
Also improve readability.

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

---------

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>

Author: MiguelCompany

.github/workflows/mac-ci.yml

@@ -56,4 +56,4 @@ jobs:
       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'))) }}
+      run-tests: ${{ (inputs.run-tests == true) || ((github.event_name == 'pull_request') && (!contains(github.event.pull_request.labels.*.name, 'no-test')) && (!contains(github.event.pull_request.labels.*.name, 'skip-ci'))) }}

.github/workflows/reusable-ci.yml

@@ -62,7 +62,6 @@ defaults:
 jobs:
   fastcdr_test:
     runs-on: ${{ inputs.os-version }}
-    if: ${{ inputs.run-build == true }}
     strategy:
       fail-fast: false
       matrix:
@@ -82,44 +81,51 @@ jobs:
         repo: eProsima/Fast-CDR
 
     - name: Sync eProsima/Fast-CDR repository
+      if: ${{ inputs.run-build == true }}
       uses: eProsima/eProsima-CI/external/checkout@v0
       with:
         path: src/fastcdr
         ref: ${{ inputs.fastcdr-branch }}
 
     - name: Install Fix Python version
+      if: ${{ inputs.run-build == true }}
       uses: eProsima/eProsima-CI/external/setup-python@v0
       with:
         python-version: '3.11'
 
     - name: Get minimum supported version of CMake
+      if: ${{ inputs.run-build == true }}
       uses: eProsima/eProsima-CI/external/get-cmake@v0
       with:
         cmakeVersion: '3.22.6'
 
     - name: Install Colcon dependencies
+      if: ${{ inputs.run-build == true }}
       uses: eProsima/eProsima-CI/multiplatform/install_colcon@v0
 
     - name: Setup CCache
+      if: ${{ inputs.run-build == true && inputs.use-ccache == true }}
       uses: eProsima/eProsima-CI/external/setup-ccache-action@v0
-      if: ${{ inputs.use-ccache == true }}
       with:
         api_token: ${{ secrets.GITHUB_TOKEN }}
 
     - name: Install Python dependencies
+      if: ${{ inputs.run-build == true }}
       uses: eProsima/eProsima-CI/multiplatform/install_python_packages@v0
       with:
         packages: vcstool
         upgrade: false
 
     - name: Fetch Fast DDS CI dependencies
+      if: ${{ inputs.run-build == true }}
       uses: eProsima/eProsima-CI/multiplatform/vcs_import@v0
       with:
         vcs_repos_file: ${{ github.workspace }}/src/fastcdr/.github/workflows/config/test.repos
         destination_workspace: src
         skip_existing: 'true'
 
     - name: Colcon build
+      if: ${{ inputs.run-build == true }}
       uses: eProsima/eProsima-CI/multiplatform/colcon_build@v0
       with:
         colcon_meta_file: ${{ github.workspace }}/src/fastcdr/.github/workflows/config/build.meta
@@ -132,7 +138,7 @@ jobs:
 
     - name: Colcon test
       id: test_fastcdr
-      if: ${{ inputs.run-tests == true }}
+      if: ${{ inputs.run-build == true && inputs.run-tests == true }}
       uses: eProsima/eProsima-CI/multiplatform/colcon_test@v0
       with:
         colcon_meta_file: ${{ github.workspace }}/src/fastcdr/.github/workflows/config/build.meta ${{ github.workspace }}/src/fastcdr/.github/workflows/config/test.meta
@@ -144,7 +150,7 @@ jobs:
 
     - name: Fast CDR Test summary
       uses: eProsima/eProsima-CI/multiplatform/junit_summary@v0
-      if: ${{ !cancelled() && inputs.run-tests == true }}
+      if: ${{ !cancelled() && inputs.run-build == true && inputs.run-tests == true }}
       with:
         junit_reports_dir: ${{ steps.test_fastcdr.outputs.ctest_results_path }}
         print_summary: 'True'
@@ -153,7 +159,7 @@ jobs:
         show_skipped: 'False'
 
     - name: Archive Test Results
-      if: always()
+      if: ${{ !cancelled() && inputs.run-build == true && inputs.run-tests == true }}
       uses: eProsima/eProsima-CI/external/upload-artifact@v0
       with:
         name: test-results-${{ inputs.label }}${{ matrix.strict-bool == true && '-strict-bool' || '' }}

.github/workflows/ubuntu-ci.yml

@@ -60,7 +60,16 @@ jobs:
       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'))) }}
+      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')) &&
+                       (!contains(github.event.pull_request.labels.*.name, 'skip-ci'))
+                     ) }}
       use-ccache: ${{ inputs.use-ccache || false }}
-      add-label: ${{ (github.event_name == 'pull_request') && (github.event.pull_request.head.repo.full_name == github.repository) && true || false }}
+      add-label: ${{ (github.event_name == 'pull_request') &&
+                     (github.event.pull_request.head.repo.full_name == github.repository) &&
+                     (!contains(github.event.pull_request.labels.*.name, 'skip-ci')) &&
+                     true || false }}

.github/workflows/windows-ci.yml

@@ -68,4 +68,4 @@ jobs:
       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'))) }}
+      run-tests: ${{ (inputs.run-tests == true) || ((github.event_name == 'pull_request') && (!contains(github.event.pull_request.labels.*.name, 'no-test')) && (!contains(github.event.pull_request.labels.*.name, 'skip-ci'))) }}