storm helm chart and external zookeeper

[alessio1983lx]

Hello,

I cannot figure out how to deploy the storm cluster with an external zookeeper…

Trying something like this:

helm install storm gresearch/storm
--set zookeeper.enabled=false
--set zookeeper.servers[0]=zookeeper.default.svc.cluster.local

And that’s the output:

Error: template: storm/templates/nimbus-statefulset.yaml:103:12: executing "storm/templates/nimbus-statefulset.yaml" at <include "common.storage.class" (dict "persistence" .Values.persistence "global" .Values.global)>: error calling include: template: no template "common.storage.class" associated with template "gotpl"

Trying to modify the default values of the chart's yaml and i achieve the same results. Where am i wrong?

My zookeeper is in the same kubernetes cluster (Rancher) deployed via bitnami helm chart and kafka can reach it. maybe the fact that usually "onpremise" zookeeper uses a quorum can be a problem?

Can you provide an example of how to set zookeeper container as disabled and external (but in same kubernetes cluster) zookeeper connection?

Regards

[danielskowronski]

It looks like common.storage.class is defined in https://github.com/bitnami/charts/blob/main/bitnami/common/templates/_storage.tpl

That library is normally loaded as dependency to Zookeeper loaded here:

charts/storm/Chart.yaml

Line 21 in 3d49021

- name: zookeeper

The problem seems to be that this dependency is loaded based upon zookeeper.enabled flag, so the code always relies on the Bitnami common library, which is only loaded when there's zookeeper loaded.

It looks like the easy fix would be to add new dependency to that shared code in the same way as the actual Zookeeper chart does it:

dependencies:
- name: common
  repository: oci://registry-1.docker.io/bitnamicharts
  tags:
  - bitnami-common
  version: 2.x.x

[danielskowronski]

Issue confirmed to be introduced with chart version 1.0.10, 1.0.9 works fine. It looks like feature was added without a regression test for zookeeper.enabled=false + persistence.enabled=true pair.

Related diff: 86cc090