[do not merge] Raise CI errors in test.sh#3574
[do not merge] Raise CI errors in test.sh#3574ethanglaser wants to merge 2 commits intouxlfoundation:mainfrom
Conversation
327ff45 to
449bc49
Compare
david-cortes-intel
left a comment
There was a problem hiding this comment.
LGTM, but don't know what to do about the errors. Will defer to @Alexandr-Solovev .
|
@theComputeKid FYI the Arm-hosted examples are all failing with illegal instruction. We aren't sure when the regression occurred because the examples were enabled with |
CI Failure Triage ReportRun: 23363249154 | Date: 2026-03-20 Failed Jobs: Please include labels on your pull request
AnalysisCI Failure Analysis1. Failure Classification: PR-specific2. Root Cause: Missing PR labels requirementThe failure is caused by a repository policy that requires pull requests to have labels applied. The CI job specifically checks for labels and fails if none are present (or only "RFC" label exists). 3. EvidenceFrom the error log: LABELS=`gh api -H "Accept: application/vnd.github+json" /repos/uxlfoundation/oneDAL/issues/3574/labels | jq '[.[].name]'`
echo $LABELS
# Output: []
if [[ $LABELS == "[]" ]] || [[ $LABELS == "[\"RFC\"]" ]]; then
echo "::error::No label set on the pull request"
exit 1
fiThe API call returns an empty array 4. Relevant Code Changes: NoneThe code changes in the diff (removing 5. RecommendationImmediate action: The PR author should add appropriate labels to PR #3574. Common labels might include:
Note: The actual code changes in this PR appear valid - they remove error suppression ( Similar failures in recent runsNo similar failures found in recent runs. Generated by CI Triage Bot |
|
I haven't had time to look into this in detail but at first glance it does strike me as odd that you are building with oneDAL/.github/workflows/ci-aarch64.yml Line 101 in 449bc49 and then running on a c6g machine, which is Neoverse N1 and therefore does not support SVE. Unless you explicitly have code to gracefully handle runtime dispatch it would be expected to encounter |
|
|
||
| - name: make daal | ||
| run: | | ||
| .ci/scripts/build.sh --compiler ${{ matrix.os.cxx }} --optimizations sve --target daal --backend-config ref --tbb-dir $TBBROOT --blas-dir ./__deps/openblas_aarch64 |
There was a problem hiding this comment.
| .ci/scripts/build.sh --compiler ${{ matrix.os.cxx }} --optimizations none --target daal --backend-config ref --tbb-dir $TBBROOT --blas-dir ./__deps/openblas_aarch64 |
Description
Checklist:
Completeness and readability
Testing
Performance