Skip to content

fix(ess-helm): align app version to ESS 0.4.13 - #791

Merged
nvaghela-oss merged 4 commits into
mainfrom
fix/ess-helm-appversion-0.4.13
Aug 12, 2026
Merged

fix(ess-helm): align app version to ESS 0.4.13#791
nvaghela-oss merged 4 commits into
mainfrom
fix/ess-helm-appversion-0.4.13

Conversation

@nvaghela-oss

@nvaghela-oss nvaghela-oss commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

TL;DR

Bump the ESS Helm chart's app version from the stale upstream v0.58.5 to 0.4.13, the current ESS service release from this monorepo, so a default install deploys the matching image.

Additional Details

  • ess-api/Chart.yaml appVersion: v0.58.5 -> 0.4.13.
  • ess-api/values.yaml ess.image.tag: v0.58.5 -> 0.4.13 (kept in sync; the tag default is the chart appVersion).
  • No template changes. helm lint passes.
  • fix(ess-helm) is scoped to deploy/helm/encrypted-secret-store, so release automation cuts a chart patch under the new prefix (deploy/helm/encrypted-secret-store/v1.7.2) via legacy_tag_prefix. The chart version field stays 0.0.0 (autoversioned by the pipeline); only appVersion changes here.

For the Reviewer

appVersion (deployed ESS image) is independent of the chart version (package release line). This aligns the former to the ESS service released from this repo.

For QA

No QA needed. helm lint and helm template render with the CI values file. Verify the release helper resolves ess-helm to 1.7.2 under deploy/helm/encrypted-secret-store/.

Issues

Relates to #747

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.

Summary by CodeRabbit

  • Chores
    • Updated the ESS API deployment to version 0.4.13.
    • Aligned chart metadata and container image versions for consistent deployments.
    • Configured the local deployment to use the NVIDIA container registry and ESS image repository.

The chart still shipped the upstream image version v0.58.5, which does not
correspond to the ESS service released from this monorepo. Point the chart at
the current ESS release image so a default install deploys the matching build.

- Chart.yaml appVersion: v0.58.5 -> 0.4.13
- values.yaml ess.image.tag: v0.58.5 -> 0.4.13 (kept in sync with appVersion)
@nvaghela-oss
nvaghela-oss requested a review from a team as a code owner August 12, 2026 19:23
@coderabbitai

coderabbitai Bot commented Aug 12, 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: 81287477-846b-42b3-b92d-5e273d5dfa27

📥 Commits

Reviewing files that changed from the base of the PR and between 05a21a7 and a212942.

📒 Files selected for processing (1)
  • deploy/helm/encrypted-secret-store/values.local.yaml

📝 Walkthrough

Walkthrough

The ESS API Helm chart reports version 0.4.13, deploys the 0.4.13 image tag, and sets the local ESS image registry and repository.

Changes

ESS API version update

Layer / File(s) Summary
Align chart and image versions
deploy/helm/encrypted-secret-store/ess-api/Chart.yaml, deploy/helm/encrypted-secret-store/ess-api/values.yaml, deploy/helm/encrypted-secret-store/values.local.yaml
The chart appVersion and ESS API image tag change from v0.58.5 to 0.4.13. The local ESS image uses nvcr.io/0651155215864979/ncp-dev/nvcf-ess.

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

Suggested reviewers: gsharma-nv, kristinapathak

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title follows Conventional Commits format and accurately describes the ESS Helm version alignment fix.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ess-helm-appversion-0.4.13

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

@balajinvda balajinvda left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Populate values.local.yaml with the dev image coordinates so a local
`make install` renders a pullable ESS image without extra overrides.
@nvaghela-oss
nvaghela-oss enabled auto-merge August 12, 2026 20:45
@nvaghela-oss
nvaghela-oss disabled auto-merge August 12, 2026 20:55
@nvaghela-oss
nvaghela-oss enabled auto-merge August 12, 2026 20:57
@nvaghela-oss
nvaghela-oss added this pull request to the merge queue Aug 12, 2026
Merged via the queue into main with commit e0c2e06 Aug 12, 2026
18 of 19 checks passed
@nvaghela-oss
nvaghela-oss deleted the fix/ess-helm-appversion-0.4.13 branch August 12, 2026 21:35
@balajinvda

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version helm-nvcf-ess-api-v1.7.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants