Skip to content

fix(charts): bump chart versions for released services - #1340

Open
balajinvda wants to merge 1 commit into
mainfrom
chore/chart-version-bumps
Open

fix(charts): bump chart versions for released services#1340
balajinvda wants to merge 1 commit into
mainfrom
chore/chart-version-bumps

Conversation

@balajinvda

@balajinvda balajinvda commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Opened by .github/workflows/chart-version-bump.yml when src/compute-plane-services/nvca/v3.3.0-dev.203 was published.

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

Merging this does not move the self-managed stack. A chart version reaches the stack only once the chart itself is released, and publishing that chart release is what triggers stack-pin-bump.yml.

Release notes: https://github.com/NVIDIA/nvcf/releases/tag/src/compute-plane-services/nvca/v3.3.0-dev.203

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

Github commit:
fix(charts): bump chart versions for released services

Summary by CodeRabbit

  • Chores
    • Updated the application version reported by the Helm chart to 3.3.0-dev.203.

Opened by the chart version bump workflow on release of src/compute-plane-services/nvca/v3.3.0-dev.203.
@balajinvda
balajinvda requested a review from a team as a code owner August 29, 2026 18:52
@balajinvda
balajinvda requested a review from mikeyrcamp August 29, 2026 18:52
@coderabbitai

coderabbitai Bot commented Aug 29, 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: 2b587fdf-1cb3-47fe-a9d8-dbc9d6a153bd

📥 Commits

Reviewing files that changed from the base of the PR and between 9b2dd53 and a3e1b40.

📒 Files selected for processing (1)
  • deploy/helm/nvca-operator/nvca-operator/Chart.yaml

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


📝 Walkthrough

Walkthrough

The Helm chart appVersion changes from 3.0.4 to 3.3.0-dev.203.

Changes

Helm chart version update

Layer / File(s) Summary
Update chart application version
deploy/helm/nvca-operator/nvca-operator/Chart.yaml
The chart references application version 3.3.0-dev.203 instead of 3.0.4.

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

Merge Risk: ⚪ Minimal · up to a3e1b

This PR only updates the released service version in a Helm chart; no actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: mikeyrcamp

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title describes the chart version bump, but the fix type does not match the maintenance-only change to Chart.yaml. Use a maintenance type, such as chore(charts): bump chart versions for released services. The fix type should be reserved for a customer-facing bug fix.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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.
Full details: Docstring Coverage

Explanation

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 files. (1 skipped: 1 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/chart-version-bumps

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

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