Skip to content

fix(self-managed): bump ess-api chart pin to 1.7.2 - #828

Open
nvaghela-oss wants to merge 1 commit into
mainfrom
fix/self-managed-ess-api-1.7.2
Open

fix(self-managed): bump ess-api chart pin to 1.7.2#828
nvaghela-oss wants to merge 1 commit into
mainfrom
fix/self-managed-ess-api-1.7.2

Conversation

@nvaghela-oss

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

Copy link
Copy Markdown
Contributor

TL;DR

Bump the ess-api chart version pinned by the self-managed stack from 1.7.0 to 1.7.2 so a stack install pulls the current ESS API chart.

Why

The self-managed Helmfile pins each service chart by version. The ess-api release was still on 1.7.0. The ESS chart line has moved forward (appVersion aligned to the ESS 0.4.13 service image), so the stack should consume the newer chart.

What changed

  • deploy/stacks/self-managed/helmfile.d/02-core.yaml.gotmpl: ess-api release version: 1.7.0 -> 1.7.2. No other releases touched.

Customer Release Notes

Self-managed stack now deploys ESS API chart 1.7.2.

Plan Summary

Single chart-version pin change. No template or values changes. Rendered topology is unchanged apart from the ess-api chart version.

Usage

Not applicable (picked up automatically by make -C deploy/stacks/self-managed install).

Testing

helm lint/helm template for the ess-api chart pass locally. Full stack render/install requires NGC credentials and a k3d cluster (not run here). The deploy/stacks/self-managed OCI pull of helm-nvcf-ess-api:1.7.2 requires that chart version to be published in the registry.

Notes

Depends on ESS chart 1.7.2 being published to the OCI registry. That release is produced by the ess-helm release automation once #791 (which bumps the chart appVersion) merges and tags deploy/helm/encrypted-secret-store/v1.7.2. Do not merge/deploy this stack bump before 1.7.2 is available in the registry, or the install will fail to pull the chart.

References

None

Related Pull Requests

Dependencies

None

Issues

Relates to #747

Summary by CodeRabbit

  • Chores
    • Updated the self-managed deployment to use ESS API version 1.7.2.

Pick up the latest ESS API chart release in the self-managed stack. The
ess-api release was pinned to 1.7.0; move it to 1.7.2 so a stack install
deploys the current chart (which aligns the ESS app image via appVersion).
@nvaghela-oss
nvaghela-oss requested a review from a team as a code owner August 13, 2026 17:57
@coderabbitai

coderabbitai Bot commented Aug 13, 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: 9ef38542-1e28-498b-94da-7432b84f1c1d

📥 Commits

Reviewing files that changed from the base of the PR and between 6384071 and 8591bc7.

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

📝 Walkthrough

Walkthrough

The self-managed core Helmfile updates the ess-api release from version 1.7.0 to 1.7.2.

Changes

ESS API release

Layer / File(s) Summary
Update ess-api release version
deploy/stacks/self-managed/helmfile.d/02-core.yaml.gotmpl
The ess-api Helm release version changes from 1.7.0 to 1.7.2.

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

Mergeability Score: 🔵 Low · up to 8591b

This change updates the ESS API chart pin from 1.7.0 to 1.7.2. It is low risk, but installations can fail if chart 1.7.2 is not yet available in the configured OCI source, so that dependency requires explicit owner awareness before deployment.

Possibly related PRs

Suggested labels: released

Suggested reviewers: shelleyshen-0

🚥 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 uses valid Conventional Commits syntax and accurately describes the ESS API chart pin update.
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/self-managed-ess-api-1.7.2

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

@nvaghela-oss nvaghela-oss self-assigned this Aug 13, 2026

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

👍

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.

2 participants