Skip to content

[do not merge] Raise CI errors in test.sh #3574

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
ethanglaser wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

[do not merge] Raise CI errors in test.sh #3574

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
449bc49
Remove || true from tests
ethanglaser Mar 20, 2026
56b6733
Merge branch 'main' into dev/eglaser-raise-ci-errors
Alexandr-Solovev Mar 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .github/workflows/ci-aarch64.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -108,14 +108,14 @@ jobs:
run: |
echo "::warning::This step is designed to always pass, please look for more details"
# timeout added due to hanging enable_thread_pinning example, and should be removed when fixed
timeout 12m .ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface daal/cpp --build-system cmake --backend ref || true
timeout 12m .ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface daal/cpp --build-system cmake --backend ref
env:
CMAKE_GENERATOR: Ninja

- name: oneapi/cpp examples
run: |
echo "::warning::This step is designed to always pass, please look for more details"
.ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface oneapi/cpp --build-system cmake --backend ref || true
.ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface oneapi/cpp --build-system cmake --backend ref
env:
CMAKE_GENERATOR: Ninja

Expand Down Expand Up @@ -185,15 +185,15 @@ jobs:
run: |
echo "::warning::This step is designed to always pass, please check logs for more details"
# timeout added due to hanging enable_thread_pinning example, and should be removed when fixed
timeout 12m .ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface daal/cpp --build-system cmake --backend ref --rng-backend openrng || true
timeout 12m .ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface daal/cpp --build-system cmake --backend ref --rng-backend openrng
env:
CMAKE_GENERATOR: Ninja

- name: oneapi/cpp examples
run: |
echo "::warning::This step is designed to always pass, please check logs for more details"
# excludes examples from EXCLUDE_LIST present in examples/daal/cpp/CMakeLists.txt
.ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface oneapi/cpp --build-system cmake --backend ref --rng-backend openrng || true
.ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface oneapi/cpp --build-system cmake --backend ref --rng-backend openrng
env:
CMAKE_GENERATOR: Ninja

Expand Down
6 changes: 3 additions & 3 deletions dev/bazel/tests/mkl_linkage_test.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,11 +56,11 @@ fi
# Static lib must NOT contain MKL archive object files
# (libmkl_core.a, libmkl_intel_ilp64.a, libmkl_tbb_thread.a objects)
AR_CONTENTS=$(ar t "$STATIC_LIB") || { echo "ERROR: ar failed on $STATIC_LIB"; exit 1; }
MKL_ARCHIVE_OBJS=$(echo "$AR_CONTENTS" | grep -c '^_mkl_\|^mkl_blas\|^mkl_vml\|^libmkl' || true)
MKL_ARCHIVE_OBJS=$(echo "$AR_CONTENTS" | grep -c '^_mkl_\|^mkl_blas\|^mkl_vml\|^libmkl)
check "Static lib: no MKL archive objects merged in" "$MKL_ARCHIVE_OBJS"

# Static lib must NOT have defined MKL symbols (only undefined refs are ok)
MKL_DEFINED=$(nm "$STATIC_LIB" 2>/dev/null | grep -c ' [Tt] mkl_' || true)
MKL_DEFINED=$(nm "$STATIC_LIB" 2>/dev/null | grep -c ' [Tt] mkl_')
check "Static lib: no defined MKL symbols (U refs are ok)" "$MKL_DEFINED"

echo ""
Expand All @@ -74,7 +74,7 @@ fi

# Dynamic lib must NOT export MKL symbols (they must be hidden).
# Use --defined-only to exclude undefined (U) imports — only defined exports matter.
MKL_EXPORTED=$(nm -D --defined-only "$DYNAMIC_LIB" 2>/dev/null | grep -c ' mkl_' || true)
MKL_EXPORTED=$(nm -D --defined-only "$DYNAMIC_LIB" 2>/dev/null | grep -c ' mkl_')
check "Dynamic lib: no exported MKL symbols (symbols hidden via --exclude-libs)" "$MKL_EXPORTED"

echo ""
Expand Down
Loading