Skip to content

feat(stack): bump self-managed stack chart pins - #2393

Open
balajinvda wants to merge 7 commits into
mainfrom
chore/stack-pin-bumps
Open

balajinvda wants to merge 7 commits into
mainfrom
chore/stack-pin-bumps

Conversation

@balajinvda

@balajinvda balajinvda commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Opened by .github/workflows/stack-pin-bump.yml when deploy/helm/api-keys-colocated/v1.9.1 was published.

The released tag carries the version, so this is a direct pin update rather than a lookup of the newest published chart.

Release notes: https://github.com/NVIDIA/nvcf/releases/tag/deploy/helm/api-keys-colocated/v1.9.1

If this pull request sits unmerged, later chart releases add their bumps to the same branch, so merging it applies all of them.

Github commit:
feat(stack): bump self-managed stack chart pins

@balajinvda
balajinvda requested a review from a team as a code owner October 8, 2026 18:21
@balajinvda
balajinvda requested a review from along-2017 October 8, 2026 18:21
@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: NVIDIA/nvcf/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Enterprise
  • Run ID: ef12b45b-31c5-4fef-97fa-018a50f39d5e

📥 Commits

Reviewing files that changed from the base of the PR and between 11b7882 and 0823f3e.


📒 Files selected for processing (1)
  • deploy/stacks/self-managed/helmfile.d/01-dependencies.yaml.gotmpl

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



📝 Walkthrough

Walkthrough

The self-managed Helmfiles update chart versions for SIS, API, the LLM request router, event ledger, and NATS.

Changes

Self-managed chart version update

Layer / File(s) Summary
Update release chart versions
deploy/stacks/self-managed/helmfile.d/02-core.yaml.gotmpl, deploy/stacks/self-managed/helmfile.d/01-dependencies.yaml.gotmpl
The SIS chart version changes from 2.6.0 to 2.6.1. The API chart version changes from 1.28.0 to 1.28.1. The LLM request router chart version changes from 1.15.0 to 1.15.2. The event ledger chart version changes from 0.5.0 to 0.6.0. The NATS chart version changes from 0.8.4 to 0.8.5.

Priority: ⬇️ Low

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

Change: Other


Merge Risk: ⚪ Minimal · up to 0823f

The stack will deploy newer chart versions. Available evidence identifies no concrete deployment failure or user-facing regression; NATS 0.8.5 is outside the documented QA-qualified set, but that discrepancy alone does not establish a blocker.

Pre-merge checks | Passed 4 | Failed 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check Warning The title uses valid Conventional Commits syntax and accurately describes the chart pin updates, but feat does not match the primary nature of this dependency-version maintenance change. Change the type to a maintenance-oriented type, such as chore(stack): bump self-managed stack chart pins. Rename the title only if the repository conventions require a different type for dependency pin updates.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.


  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR


  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@github-actions
github-actions Bot force-pushed the chore/stack-pin-bumps branch 4 times, most recently from 0823f3e to 90c5b0f Compare October 9, 2026 21:58
Opened by the stack pin bump workflow on release of deploy/helm/event-ledger/v0.6.0.
Opened by the stack pin bump workflow on release of deploy/helm/cloud-functions/v1.28.1.
Opened by the stack pin bump workflow on release of deploy/helm/icms/v2.6.1.
Opened by the stack pin bump workflow on release of deploy/helm/llm-request-router/v1.15.2.
Opened by the stack pin bump workflow on release of deploy/helm/nats/v0.8.5.
Opened by the stack pin bump workflow on release of deploy/helm/openbao/v0.33.1.
Opened by the stack pin bump workflow on release of deploy/helm/api-keys-colocated/v1.9.1.
@github-actions
github-actions Bot force-pushed the chore/stack-pin-bumps branch from 90c5b0f to 95f5013 Compare October 9, 2026 22:49
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.

1 participant