Skip to content

Comments

feat(install): add Helm chart for Operator installation#1175

Open
Prasunamadasu wants to merge 2 commits intocryostatio:mainfrom
Prasunamadasu:feature/add-helm-chart
Open

feat(install): add Helm chart for Operator installation#1175
Prasunamadasu wants to merge 2 commits intocryostatio:mainfrom
Prasunamadasu:feature/add-helm-chart

Conversation

@Prasunamadasu
Copy link

As discussed in issue #1140, this PR introduces the initial version of the custom Helm chart for deploying the Cryostat operator.

@andrewazores
Copy link
Member

Hi @Prasunamadasu , sorry for such a delayed response here - I was away on vacation, and since getting back we've been busy with wrapping up Cryostat 4.1 and preparing for the release. I'll circle back to this in the near future.

@andrewazores
Copy link
Member

This looks very cool, and thanks for also including the GitHub Actions YAML definition.

A few general thoughts:

  • this will need to be adjusted to deploy latest/snapshot/dev builds of the various operand container images rather than 3.0, and in release branches it'll get updated to deploy builds from the appropriate branches/tags (same as how the Operator and the mainline Helm chart do)
  • it'd be nice if some of these Helm templates could be generated from Operator manifest bases, or vice-versa. For example, the Helm template for the Cryostat CRD could probably be copied from or at least partially generated from the Operator CRD base, which would save some maintenance effort and reduce the risk of missing changes/updates.

```bash
git clone https://github.com/cryostatio/cryostat-operator.git
cd cryostat-operator/helm/charts/cryostat-operator
helm install cryostat-operator . --create-namespace -n cryostat-operator
Copy link
Member

Choose a reason for hiding this comment

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

This produces a certmanager-related complaint:

$ helm install cryostat-operator . --create-namespace -n cryostat-operator
Error: INSTALLATION FAILED: template: cryostat-operator/templates/secret.yaml:21:20: executing "cryostat-operator/templates/secret.yaml" at <.Values.certManager.issuer.kind>: nil pointer evaluating interface {}.kind

values.yaml doesn't seem to have certManager.issuer for any default values, so either this should be included in the values.yaml or the installation instructions should describe the prerequisite setup steps.

@andrewazores andrewazores added the feat New feature or request label Dec 4, 2025
@andrewazores andrewazores changed the title feat(): Add cryostat-helm-chart feat(install): add Helm chart for Operator installation Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants