Skip to content

Commit 388457c

Browse files
committed
Fix names of coverage data files
This patch fixes the regression introduced by the commits: 9d23a57 and 2a7a229. The name of the coverage data file in the `upload-artifact` step must be exactly the same like `$COVERAGE_FILE_NAME` in the `Check coverage` step where this file is generated. Signed-off-by: Lukasz Dorau <[email protected]>
1 parent 66b161a commit 388457c

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

.github/workflows/reusable_gpu.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,5 +129,5 @@ jobs:
129129
- uses: actions/upload-artifact@65462800fd760344b1a7b4382951275a0abb4808 # v4.3.3
130130
if: ${{ matrix.build_type == 'Debug' && matrix.os == 'Ubuntu' }}
131131
with:
132-
name: ${{env.COVERAGE_NAME}}-${{matrix.os}}-${{matrix.build_type}}-shared-${{matrix.shared_library}}
132+
name: ${{env.COVERAGE_NAME}}-shared-${{matrix.shared_library}}
133133
path: ${{env.COVERAGE_DIR}}

.github/workflows/reusable_multi_numa.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ jobs:
6969
--gtest_filter="-*checkModeLocal/*:*checkModePreferredEmptyNodeset/*:testNuma.checkModeInterleave"
7070
7171
- name: Check coverage
72-
if: matrix.os == 'ubuntu-22.04'
72+
if: ${{ matrix.build_type == 'Debug' && matrix.os == 'ubuntu-22.04' }}
7373
working-directory: ${{env.BUILD_DIR}}
7474
run: |
7575
export COVERAGE_FILE_NAME=${{env.COVERAGE_NAME}}-${{matrix.os}}-shared-${{matrix.shared_library}}
@@ -79,7 +79,7 @@ jobs:
7979
mv ./$COVERAGE_FILE_NAME ${{env.COVERAGE_DIR}}
8080
8181
- uses: actions/upload-artifact@65462800fd760344b1a7b4382951275a0abb4808 # v4.3.3
82-
if: matrix.os == 'ubuntu-22.04'
82+
if: ${{ matrix.build_type == 'Debug' && matrix.os == 'ubuntu-22.04' }}
8383
with:
84-
name: ${{env.COVERAGE_NAME}}-${{matrix.os}}-${{matrix.build_type}}-shared-${{matrix.shared_library}}
84+
name: ${{env.COVERAGE_NAME}}-${{matrix.os}}-shared-${{matrix.shared_library}}
8585
path: ${{env.COVERAGE_DIR}}

0 commit comments

Comments
 (0)