Skip to content

fix: Enable OpenVINO model generation for SBSA#8883

Merged
mc-nv merged 1 commit into
mainfrom
mchornyi/TRI-1105/models-update
Jul 14, 2026
Merged

fix: Enable OpenVINO model generation for SBSA#8883
mc-nv merged 1 commit into
mainfrom
mchornyi/TRI-1105/models-update

Conversation

@mc-nv

@mc-nv mc-nv commented Jul 14, 2026

Copy link
Copy Markdown
Contributor
CI (internal): [#57889355](http://tritonserver.local/ci/pipelines/57889355)

What does the PR do?

Remove the [[ "aarch64" != $(uname -m) ]] guard around the OpenVINO model generation step in qa/common/gen_qa_model_repository. OpenVINO now supports aarch64 (SBSA) targets, so the QA model repository can generate OpenVINO test models on SBSA runners the same way it does on x86 — closing a gap that caused downstream CI failures (empty openvino/* directories → tar: Cowardly refusing to create an empty archive in the per-framework upload loop, e.g. GenModels-build--sbsa-dgx_spark-12.1 job 360691523).

Checklist

  • PR title reflects the change and is of format <commit_type>: <Title>
  • Changes are described in the pull request.
  • Related issues are referenced.
  • Populated github labels field
  • Added test plan and verified test passes.
  • Verified that the PR passes existing CI.
  • Verified copyright is correct on all changed files.
  • Added succinct git squash message before merging ref.
  • All template sections are filled out.
  • Optional: Additional screenshots for behavior/output changes with before/after.

Commit Type:

  • build
  • ci
  • docs
  • feat
  • fix
  • perf
  • refactor
  • revert
  • style
  • test

Related PRs:

(none — this change is contained in server)

Where should the reviewer start?

qa/common/gen_qa_model_repository:470-497 — the [[ "aarch64" != $(uname -m) ]] conditional wrapping the OpenVINO docker run, its error handling, and the cleanup of $OPENVINOSCRIPT is removed. Every line inside the removed conditional is unindented back to the parent scope; no other logic changes. Diff is +18/-20 on a single file.

Test plan:

  • Trigger a full GenModels-build matrix run and confirm OpenVINO models are produced on SBSA runners: GenModels-build--sbsa-a100-8.0, GenModels-build--sbsa-agx_thor-11.0, GenModels-build--sbsa-dgx_spark-12.1, GenModels-build--sbsa-h100-9.0, GenModels-build--sbsa-gb300-10.3, GenModels-build--sbsa-rubin-10.7, GenModels-build--sbsa-gb200-10.0.

  • Inspect SBSA build output (ls -d */openvino_*) to confirm openvino model directories now exist.

  • Confirm the downstream per-framework upload loop no longer errors on tar -cvf openvino.tar $(find …) on SBSA — the previous silent-skip / hard-fail path in job 360691523 must be gone.

  • Verify no regression on x86 jobs (GenModels-build--x86-a100-8.0, GenModels-build--x86-rtx50-12.0, GenModels-build--x86-rubin-10.7) — the OpenVINO generation path is unchanged for x86, just no longer gated.

  • CI Pipeline ID: 57889355

Caveats:

Assumes the base $UBUNTU_IMAGE used inside the OpenVINO generation step provides an OpenVINO runtime with aarch64 support. If any specific SBSA runner still resolves to an older OpenVINO that lacks ARM support, this will now surface as a real failure inside $OPENVINOSCRIPT rather than the previous silent skip — visible failure is preferable, but worth calling out in case one of the runner images needs a bump.

Background

The prior guard was written when OpenVINO had no aarch64 support and the check avoided a hard failure on SBSA runners. OpenVINO gained ARM support (2024+) and the Triton OpenVINO backend now publishes aarch64 artifacts, so the guard is no longer needed. Keeping it caused SBSA CI to attempt to publish an openvino.tar with zero contents, which GNU tar refuses with Cowardly refusing to create an empty archive, failing GenModels-build--sbsa-* end-to-end. Full analysis in TRI-1105.

Related Issues:

  • Resolves: TRI-1105

@mc-nv mc-nv self-assigned this Jul 14, 2026
@mc-nv
mc-nv requested review from mattwittwer, whoisj and yinggeh July 14, 2026 21:40
@mc-nv
mc-nv merged commit a510ae2 into main Jul 14, 2026
3 checks passed
@mc-nv
mc-nv deleted the mchornyi/TRI-1105/models-update branch July 14, 2026 22:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants