Repository navigation
fix: use GRID v19 LTS for A10 - #9709
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 69f7a2db-19b6-4f3c-adde-54733265ab39
Windows Unit Test Results 3 files 18 suites 1m 10s ⏱️ Results for commit ba8e715. |
|
There was a problem hiding this comment.
Copilot review overview
🔵 Needs a closer look
The R580 artifact and routing appear consistent, but this production GPU driver transition warrants human review and real-hardware validation.
Review effort: Balanced
Findings: None
What changed in this PR
Pins A10 GRID workloads to patched R580 LTS while retaining R595 for RTX PRO 6000 BSE.
Changes:
- Updates and Renovate-pins the A10 GRID image to R580.
- Adds exact multi-GPU driver-version validation.
- Expands SKU routing tests and CI coverage.
| File | Description |
|---|---|
README.md |
Documents GRID branch separation. |
parts/common/components.json |
Pins the R580 GRID image. |
.github/renovate.json |
Restricts GRID updates by branch. |
pkg/agent/gpu_driver_selection_test.go |
Tests 30 SKU mappings. |
pkg/agent/datamodel/gpu_components_test.go |
Verifies R580/R595 branches. |
e2e/scenario/validators.go |
Adds exact-version validation. |
e2e/scenario/validators_gpu_driver_test.go |
Exercises generated validation scripts. |
e2e/scenario/scenario.go |
Applies validation to GRID scenarios. |
.github/workflows/go-test.yml |
Runs validator tests in CI. |
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
GPU E2E verification: passedAgentBaker GPU E2E build 184427740 succeeded on PR merge commit Results: 16 passed, 1 flaky (passed on retry), 177 skipped, 0 failed.
The versions were verified in the scenario The sole flaky case was Cleanup: all 19 explicitly logged E2E VMSS targets are absent, verified through read-only Azure queries. No resources were manually deleted. Relevant Go tests, main-module lint, CUE/component schema, version consistency, DCGM compatibility, and workflow lint checks also passed. Broader VHD/general E2E gates are still pending at the time of this comment; this is not a claim that every PR gate is complete. This verifies GPU bootstrap/runtime behavior on the tested nodes. Isaac Sim renderer compatibility remains untested. |
What this PR does / why we need it:
Consume the patched GRID v19.6 LTS image published after Azure/aks-gpu#196, retaining the R580 option for A10 instead of moving A10 applications to R595.
aks/aks-gpu-gridto580.178.04-20261006181431.aks-gpu-grid-v20image,595.91.07-20260917204440.Which issue(s) this PR fixes:
Related to Azure/aks-gpu#195 and the merged producer PR Azure/aks-gpu#196.
This is the R580/LTS alternative to the A10 R595 image pins proposed in #9672 and #9661. Their A10 image-version changes should not be combined with this pin.
Published artifact
The producer merge is
432207c529aebbd0604751f3fc7955cfbd9c8f03. Its publication run completed the GRIDBuild and Pushjob successfully.The exact MCR manifest for
580.178.04-20261006181431returns HTTP 200:sha256:0aa1ec2015e78f1063432b929aa1c78a20d8aee9f37cf1226ea68bda0b642ae0No placeholder or inferred timestamp is used.
Compatibility and scope
Azure's Linux GRID support matrix lists v19.6 and v20.2 for NVadsA10_v5, and only v20.2 for NCv6 and NCasT4_v3.
An available GRID option does not require switching a CUDA compute SKU to GRID. CUDA selection is unchanged. Azure Linux/ACL sysext installation paths are also unchanged by this Ubuntu image pin.
Isaac Sim 5.1's source pins Kit SDK 107.3.3; upstream explains that this version does not support 59x drivers. This PR retains a supported R580 option, but does not claim an exact Azure A10 Isaac Sim renderer run. The separate NCv6 MIG identifier issue still requires newer Kit.
Validation
nvidia-smi, and invalid expected versions. Tests execute the generated executable script with stub binaries; they do not access a real GPU.git diff --checkpass.Hardware GPU verification
GPU E2E build 184427740 succeeded on PR merge commit
1aa7e22b6c40ec89526ae458ed86041646d99edc, whose second parent is this PR'sba8e715eb63f679c8aa3042fd1108f0bcecba914.Final results: 16 passed, 1 flaky (passed on retry), 177 skipped, 0 failed.
580.178.04580.178.04595.91.07The versions above are observed in the scenario
cluster-provision.logartifacts, not inferred from the image tag. The exact-version validators also passed.The only flaky scenario was the unchanged T4 CUDA path,
Ubuntu2204_GPUNC: the first attempt recorded a WALinuxAgent goal-state/Wireserver HTTP timeout; attempt 2 passed. The A10 and RTX GRID scenarios did not need a retry.Read-only Azure verification found all 19 explicitly logged E2E VMSS targets absent after the run. No resources were manually deleted.
This proves the tested GPU bootstrap/runtime checks, not Isaac Sim renderer compatibility. Final GitHub rollup: 52 checks passed, 1 intentionally skipped, 0 failed, 0 pending. Both Linux and Windows VHD gates passed. The PR remains open and awaits review.