Skip to content

Commit

Permalink
:common: fix GHA
Browse files Browse the repository at this point in the history
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
  • Loading branch information
grom72 committed Mar 20, 2024
1 parent b5b7907 commit 6fc252d
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 2 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/pmem_test_matrix.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,12 +38,11 @@ jobs:
- name: Test prepare
uses: ./.github/actions/pmem_test_prepare
with:
valgrind: ${{ fromJSON(inputs.valgrind) }}
valgrind: ${{ inputs.valgrind }}

- name: Test run
uses: ./.github/actions/pmem_test_run
with:
test_script: ${{ matrix.test_script }}
build: ${{ matrix.build }}
force_enable: ${{ matrix.force_enable }}
valgrind: ${{ fromJSON(inputs.valgrind) }}
10 changes: 10 additions & 0 deletions .github/workflows/pmem_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ jobs:
force_enable: '["pmemcheck", "memcheck"]'
valgrind: 1


# Test the default build with force-enabled Valgrind tooling for thread error
# detection.
Thread:
Expand All @@ -38,6 +39,7 @@ jobs:
# 9h = 7h20m (the longest workflow execution time) + ~20% leeway.
timeout_minutes: 540


# Static builds are tested in this limited scope only.
static:
name: Static
Expand Down Expand Up @@ -108,3 +110,11 @@ jobs:
uses: ./.github/actions/pmem_test_run
with:
build: nondebug # only the production build is considered

# Test the default build with the basic test suite.
Basic_w_Valgrind:
uses: ./.github/workflows/pmem_test_matrix.yml
with:
# Exclude all Valgrind tests but keep Valgrind tools built-in.
force_enable: '["none"]'
valgrind: 1

0 comments on commit 6fc252d

Please sign in to comment.