Skip to content

feat: stacks and demo supports OCI helmCharts#440

Merged
NickLarsenNZ merged 7 commits intomainfrom
feat/stack-demo-helm-chart-from-oci
Apr 23, 2026
Merged

feat: stacks and demo supports OCI helmCharts#440
NickLarsenNZ merged 7 commits intomainfrom
feat/stack-demo-helm-chart-from-oci

Conversation

@NickLarsenNZ
Copy link
Copy Markdown
Member

@NickLarsenNZ NickLarsenNZ commented Apr 22, 2026

Fixes #417

Note

This was bugging me enough when trying to validate the observability stack (which before this requires a bunch of manual steps).

You can now refer to Helm Charts stored in OCI.

Eg: in a stack/demo yaml:

- helmChart: release-file.yaml

release-file.yaml:

releaseName: nifi-operator
name: nifi-operator
repo:
  name: nifi-operator
  url: oci://oci.stackable.tech/sdp-charts # 👈 yeah boi
version: 0.0.0-dev
options: {}

Here's an example:

image

I also tested to make sure existing stacks still work (they do):

cargo stackablectl stack install logging

Note: Local isn't yet supported, but it is a fallback for when the URL doesn't match oci:// or http(s)://
Note: Only non-OCI remote charts need the helm::add_repo() call.
We emit an error if a Local (non oci:// or http(s)://) value is given.
This seems trivial now in this set of changes, but is intended to catch future regressions
@NickLarsenNZ NickLarsenNZ moved this to Development: In Progress in Stackable Engineering Apr 22, 2026
@NickLarsenNZ NickLarsenNZ moved this from Development: In Progress to Development: Waiting for Review in Stackable Engineering Apr 22, 2026
Copy link
Copy Markdown
Member

@Techassi Techassi left a comment

Choose a reason for hiding this comment

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

LGTM, left a few small comments.

Comment thread rust/stackable-cockpit/src/helm.rs
Comment thread rust/stackable-cockpit/src/helm.rs Outdated
Comment thread rust/stackable-cockpit/src/platform/manifests.rs Outdated
Comment thread rust/stackable-cockpit/src/platform/manifests.rs Outdated
@Techassi Techassi moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Apr 23, 2026
@NickLarsenNZ NickLarsenNZ requested a review from Techassi April 23, 2026 06:58
@NickLarsenNZ NickLarsenNZ enabled auto-merge April 23, 2026 07:02
@NickLarsenNZ NickLarsenNZ moved this from Development: In Review to Development: Done in Stackable Engineering Apr 23, 2026
@NickLarsenNZ NickLarsenNZ added this pull request to the merge queue Apr 23, 2026
Merged via the queue into main with commit 3ab83b6 Apr 23, 2026
21 checks passed
@NickLarsenNZ NickLarsenNZ deleted the feat/stack-demo-helm-chart-from-oci branch April 23, 2026 07:08
@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering May 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

Allow stacks/demos to deploy helmChart from OCI registry

3 participants