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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Done

Development

Successfully merging this pull request may close these issues.

Allow stacks/demos to deploy helmChart from OCI registry

2 participants