diff --git a/.github/workflows/codeql.yml b/.github/workflows/codeql.yml index 928d8d534..aa350911d 100644 --- a/.github/workflows/codeql.yml +++ b/.github/workflows/codeql.yml @@ -37,7 +37,14 @@ jobs: steps: - name: Checkout repository uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1 - + - name: print repo + run: | + if [ pull_request = 'pull_request' ]; then + CI_REPO="igchor/unified-memory-framework" + else + CI_REPO="$GITHUB_REPOSITORY" + fi + echo "BUILDING $CI_REPO" - name: Initialize CodeQL uses: github/codeql-action/init@b7bf0a3ed3ecfa44160715d7c442788f65f0f923 # v3.23.2 with: diff --git a/.github/workflows/gpu.yml b/.github/workflows/gpu.yml index e2ce84f2c..6eafa899f 100644 --- a/.github/workflows/gpu.yml +++ b/.github/workflows/gpu.yml @@ -12,6 +12,8 @@ jobs: # TODO: add support for Windows ubuntu-build: name: Build - Ubuntu + # run only on upstream; forks will not have the HW + if: github.repository == 'oneapi-src/unified-memory-framework' strategy: matrix: diff --git a/test/memspaces/memspace_host_all.cpp b/test/memspaces/memspace_host_all.cpp index f16223cea..2e199d73f 100644 --- a/test/memspaces/memspace_host_all.cpp +++ b/test/memspaces/memspace_host_all.cpp @@ -157,9 +157,14 @@ TEST_F(memspaceHostAllProviderTest, memoryPolicyOOM) { UT_ASSERTeq(umf_ret, UMF_RESULT_SUCCESS); } + // TODO: we want to enable this check only when tests are running under QEMU. + // Otherwise it might sporadically fail on a real system where other processes + // occupied all memory from a aparticular NUMA node. +#if 0 // Confirm that all the NUMA nodes bound to 'HOST ALL' memspace were exhausted. for (auto &id : nodeIds) { auto it = std::find(allocNodeIds.begin(), allocNodeIds.end(), id); UT_ASSERT(it != allocNodeIds.end()); } +#endif }