Repository navigation
fix(audit): accept NVPL as the NVHPC BLAS on aarch64 - #34
Open
robTheBuildr wants to merge 1 commit into
Open
robTheBuildr wants to merge 1 commit into
robTheBuildr wants to merge 1 commit into
Conversation
The aarch64 NVIDIA HPC SDK ships NVIDIA Performance Libraries in place of OpenBLAS. compilers/lib/libblas.so and liblapack.so link to NVPL, and only the OpenBLAS headers and licence remain, so a complete aarch64 release reported "missing components openblas". Require NVPL on aarch64 and OpenBLAS on other architectures. The collector tests now pin the architecture.
robTheBuildr
requested review from
JordanNanos,
Prathmesh234,
functionstackx and
samharshe
as code owners
October 8, 2026 16:40
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #32.
Changes
libnvpl*.so*) in place oflibopenblas.so*. The aarch64 SDK linkscompilers/lib/libblas.soandliblapack.soto NVPL and ships only the OpenBLAS headers and licence.Tests
tests/audit/test_nvhpc_validation.py: the collector tests set the architecture through aunamestub. New tests:nvplas missing;openblasas missing.test_security_cli.py::MinimumRefreshTests::test_read_only_install_says_so_and_keeps_the_table). It also fails on unchangedmasterin the same environment.mastercollector reportsCOMPONENTS_MISSING=openblas. This branch reportsCOMPONENTS_OK=trueandCOMPONENTS_MISSING=none.git diff --checkpassed.Limits
I checked the NVHPC 26.5 aarch64 layout only.
Note
Low Risk
Audit validation logic only; aarch64 nodes stop false-failing on missing OpenBLAS when NVPL is present, with no change to x86_64 behavior.
Overview
The NVHPC component completeness check in
host-check.shno longer always requires OpenBLAS. On aarch64 it expects NVPL (libnvpl*.so*) because that SDK layout uses NVIDIA Performance Libraries for BLAS/LAPACK; on other architectures it still requireslibopenblas.so*. Missing components continue to surface asWORKER_NVHPC_COMPONENTS_MISSING(e.g.nvplvsopenblas).Collector tests build arch-specific SDK trees (
Linux_x86_64/Linux_aarch64), stubuname -mso results are host-independent, and add cases for aarch64 with NVPL (pass), aarch64 without NVPL, and x86_64 without OpenBLAS.Reviewed by Cursor Bugbot for commit 058c80f. Bugbot is set up for automated code reviews on this repo. Configure here.