Skip to content

feat(charts): bump chart versions for released services - #1952

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

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

Conversation

@balajinvda

@balajinvda balajinvda commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Opened by .github/workflows/chart-version-bump.yml when src/control-plane-services/function-autoscaler/v1.22.0 was published.

The released tag identifies the version and source tree, 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/control-plane-services/function-autoscaler/v1.22.0

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:
feat(charts): bump chart versions for released services

Summary by CodeRabbit

  • Updates
    • Updated the LLM API Gateway to version 0.14.3.
    • Updated the Function Autoscaler to version 1.22.0.
    • Updated the HTTP Invocation Service to version 0.13.0.
    • Updated deployment configurations to use the corresponding container images, ensuring installed components match their declared application versions.

@balajinvda
balajinvda requested a review from a team as a code owner September 17, 2026 16:49
@coderabbitai

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown

Review Change StackReview 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: 4bd9786c-a857-483b-886f-7ba4a440a9ce

📥 Commits

Reviewing files that changed from the base of the PR and between 162f6f2 and 2ec74d2.

📒 Files selected for processing (5)
  • deploy/helm/function-autoscaler/Chart.yaml
  • deploy/helm/http-invocation/nvcf-invocation-service/Chart.yaml
  • deploy/helm/http-invocation/nvcf-invocation-service/values.yaml
  • deploy/helm/llm-api-gateway/llm-api-gateway/Chart.yaml
  • deploy/helm/llm-api-gateway/llm-api-gateway/values.yaml

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


📝 Walkthrough

Walkthrough

The Helm chart versions are updated for the function autoscaler, invocation service, and LLM API gateway. The invocation service and LLM API gateway image tags are updated to match their chart versions.

Changes

Helm version updates

Layer / File(s) Summary
Update function autoscaler version
deploy/helm/function-autoscaler/Chart.yaml
The chart appVersion now uses 1.22.0.
Align invocation service versions
deploy/helm/http-invocation/nvcf-invocation-service/Chart.yaml, deploy/helm/http-invocation/nvcf-invocation-service/values.yaml
The chart appVersion and default container image tag now use 0.13.0.
Align LLM API gateway versions
deploy/helm/llm-api-gateway/llm-api-gateway/Chart.yaml, deploy/helm/llm-api-gateway/llm-api-gateway/values.yaml
The chart appVersion and configured container image tag now use 0.14.3.

Priority: ⬇️ Low

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

Change: Other

Merge Risk: ⚪ Minimal · up to 2ec74

The PR contains aligned Helm version and image-tag updates with no identified behavior or deployment risk.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title follows the Conventional Commits syntax and includes the required scope. However, the changes only update Helm chart and image versions, so feat does not accurately describe the primary ch… Change the title to chore(charts): bump chart versions for released services or another accurate non-customer-impact type.
✅ 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: Title check

Explanation

The title follows the Conventional Commits syntax and includes the required scope. However, the changes only update Helm chart and image versions, so feat does not accurately describe the primary change.

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

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

…0.14.3

Opened by the chart version bump workflow on release of src/invocation-plane-services/llm-api-gateway/v0.14.3.
…v0.13.0

Opened by the chart version bump workflow on release of src/invocation-plane-services/http-invocation/v0.13.0.
@github-actions
github-actions Bot force-pushed the chore/chart-version-bumps branch from 162f6f2 to 9edd52a Compare September 17, 2026 19:36
@balajinvda balajinvda changed the title fix(charts): bump chart versions for released services feat(charts): bump chart versions for released services Sep 17, 2026
…/v1.22.0

Opened by the chart version bump workflow on release of src/control-plane-services/function-autoscaler/v1.22.0.
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