Skip to content

fix(compute-plane): bump nvca to 3.2.6 - #980

Merged
along-2017 merged 1 commit into
mainfrom
fix/compute-plane/bump-nvca-to-3.2.6
Aug 19, 2026
Merged

fix(compute-plane): bump nvca to 3.2.6#980
along-2017 merged 1 commit into
mainfrom
fix/compute-plane/bump-nvca-to-3.2.6

Conversation

@along-2017

@along-2017 along-2017 commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

TL;DR

Bump the compute-plane stack's nvca-operator and NVCA pins from 3.2.0 to 3.2.6 and regenerate the golden testdata. 3.2.6 carries the Helm LLM transport trust backport that self-managed clusters need when addons.llm.pki is enabled.

Additional Details (optional for docs, build, test, refactor, ci, chore, style, and revert PRs)

Regenerating the goldens also normalizes one line in agent-config-merge-cm.yaml (the merged config block scalar header). That drift predates this change: a fresh render of the unmodified stack produces the same line, so the committed golden was stale and this PR brings it back in sync with what make generate-golden produces today.

For the Reviewer

The functional change is two values in deploy/stacks/nvcf-compute-plane/environments/base.yaml. Everything else is regenerated golden testdata from make generate-golden.

For QA (optional for docs, build, test, refactor, ci, chore, style, and revert PRs)

  • make test-local passes in deploy/stacks/nvcf-compute-plane: the fresh render matches the regenerated goldens and the nvca-entrypoints checks pass.
  • Verified live on a self-managed k3d cluster built from main with addons.llm.pki enabled and nvca 3.2.6: the cluster registers, a container LLM function deploys to ACTIVE, and chatinvocations pass over the PKI-secured QUIC path.

Issues

Relates to #52

Checklist

  • I am familiar with the Contributing Guidelines.
  • I have signed off my commits for Developer Certificate of Origin (DCO) compliance.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

The stack pinned nvca-operator and NVCA at 3.2.0 while the v3.2 patch
line is at 3.2.6. 3.2.6 carries the Helm LLM transport trust backport,
which self-managed clusters need when addons.llm.pki is enabled:
without it, Helm-based LLM workloads cannot verify the PKI-issued
stargate QUIC certificate against the cluster trust bundle.

Golden testdata regenerated with make generate-golden. Besides the
version bumps, regeneration normalizes one line in
agent-config-merge-cm.yaml (the merged config block scalar). That
drift predates this change: a fresh render of the unmodified stack
produces the same line, so the committed golden was stale.

Verified on a self-managed k3d cluster from main with addons.llm.pki
enabled and nvca 3.2.6: a container LLM function deploys to ACTIVE and
serves invocations over the TLS QUIC path. make test-local passes.

Signed-off-by: along <along@nvidia.com>
@along-2017
along-2017 requested a review from a team as a code owner August 19, 2026 01:06
@along-2017
along-2017 requested a review from vrv3814 August 19, 2026 01:06
@coderabbitai

coderabbitai Bot commented Aug 19, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 0fe3f12d-e38e-446d-ba82-e8509eaf19cf

📥 Commits

Reviewing files that changed from the base of the PR and between 2a08528 and c6580ee.

⛔ Files ignored due to path filters (4)
  • deploy/stacks/nvcf-compute-plane/testdata/golden/local/02-nvca.yaml-nvca-operator/helm-nvca-operator/templates/agent-config-merge-cm.yaml is excluded by !**/testdata/**
  • deploy/stacks/nvcf-compute-plane/testdata/golden/local/02-nvca.yaml-nvca-operator/helm-nvca-operator/templates/deployment.yaml is excluded by !**/testdata/**
  • deploy/stacks/nvcf-compute-plane/testdata/golden/local/02-nvca.yaml-nvca-operator/helm-nvca-operator/templates/pre-delete-cleanup-job.yaml is excluded by !**/testdata/**
  • deploy/stacks/nvcf-compute-plane/testdata/golden/local/02-nvca.yaml-nvca-operator/helm-nvca-operator/templates/self-managed-nvcfbackend-cm.yaml is excluded by !**/testdata/**
📒 Files selected for processing (1)
  • deploy/stacks/nvcf-compute-plane/environments/base.yaml

Included review availability: Your plan provides up to 12 included reviews per hour; 10 remain after this review.


📝 Walkthrough

Walkthrough

The base environment updates the NVCA operator image tag and self-managed NVCA version from 3.2.0 to 3.2.6.

Changes

NVCA version update

Layer / File(s) Summary
Update NVCA version declarations
deploy/stacks/nvcf-compute-plane/environments/base.yaml
The NVCA operator image tag and self-managed NVCA version now use 3.2.6.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to c6580

This change updates the compute-plane NVCA versions and matching generated outputs without introducing an actionable merge-blocking risk; it is ready for normal checks and review.

Suggested reviewers: vrv3814

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title follows Conventional Commits and accurately describes the NVCA version update for the compute-plane configuration.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/compute-plane/bump-nvca-to-3.2.6

Comment @coderabbitai help to get the list of available commands.

@along-2017
along-2017 requested a review from mikeyrcamp August 19, 2026 01:08
@along-2017
along-2017 added this pull request to the merge queue Aug 19, 2026
Merged via the queue into main with commit 404c1fe Aug 19, 2026
16 checks passed
@along-2017
along-2017 deleted the fix/compute-plane/bump-nvca-to-3.2.6 branch August 19, 2026 01:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants