Skip to content

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

Open
balajinvda wants to merge 4 commits into
mainfrom
chore/chart-version-bumps
Open

fix(charts): bump chart versions for released services#1340
balajinvda wants to merge 4 commits 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.206 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.206

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 to 3.3.0-dev.206 for deployment packaging.

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: b10c159a-9a04-447a-a7cf-d999a54dab5f

📥 Commits

Reviewing files that changed from the base of the PR and between 25d3e86 and fea92ba.

📒 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; 10 remain after this review.


📝 Walkthrough

Walkthrough

The Helm chart appVersion changes from 3.3.0-dev.205 to 3.3.0-dev.206.

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.206 instead of 3.3.0-dev.205.

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

Merge Risk: ⚪ Minimal · up to fea92

This updates the released service chart version in a single metadata file and does not directly move the self-managed stack; 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 follows Conventional Commits syntax, but fix does not accurately describe the change. The pull request only bumps the Helm chart appVersion from 3.3.0-dev.205 to 3.3.0-dev.206 after … Change the title to a maintenance type, such as chore(charts): bump chart versions for released services.
✅ 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.
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.)

Full details: Title check

Explanation

The title follows Conventional Commits syntax, but fix does not accurately describe the change. The pull request only bumps the Helm chart appVersion from 3.3.0-dev.205 to 3.3.0-dev.206 after a release.

  • 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.

Opened by the chart version bump workflow on release of src/compute-plane-services/nvca/v3.3.0-dev.204.
Opened by the chart version bump workflow on release of src/compute-plane-services/nvca/v3.3.0-dev.205.
Opened by the chart version bump workflow on release of src/compute-plane-services/nvca/v3.3.0-dev.206.
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