https://pinot.apache.org/ logo
a

ayush sharma

05/06/2021, 5:07 PM
Hi all, We want to assign storageClass to pinot zookeeper in the helm chart so that whenever a pinot-zookeeper pod gets created it automatically claims a PV. But I see, there is no direct way to assign using values.yaml We tried installing incubator/zookeeper using the below command
Copy code
helm -n my-pinot-kube install pinot-zookeeper incubator/zookeeper --set replicaCount=1
and disabled the zookeeper in helm/pinot/values.yaml by modifying zookeeper.enabled to false. But we see face an error from
_helpers.tpl
indicating not able to fetch the
zookeeper.url
and related to
configurationOverride
.
Copy code
Error: template: pinot/templates/server/statefulset.yml:63:27: executing "pinot/templates/server/statefulset.yml" at <include "zookeeper.url" .>: error calling include: template: pinot/templates/_helpers.tpl:79:33: executing "zookeeper.url" at <index .Values "configurationOverrides" "zookeeper.connect">: error calling index: index of nil pointer
Any help is appreciated !
x

Xiang Fu

05/06/2021, 5:45 PM
You can find it under zookeeper helmChart 😛 in short, here are the options under
zookeeper
section in
values.yaml
file:
Copy code
persistence:
  enabled: true
  ## zookeeper data Persistent Volume Storage Class
  ## If defined, storageClassName: <storageClass>
  ## If set to "-", storageClassName: "", which disables dynamic provisioning
  ## If undefined (the default) or set to null, no storageClassName spec is
  ##   set, choosing the default provisioner.  (gp2 on AWS, standard on
  ##   GKE, AWS & OpenStack)
  ##
  # storageClass: "-"
  accessMode: ReadWriteOnce
  size: 5Gi
if you already have zk, then disable zk should be good, I will check the
_helpers.tpl
script
can you try to define
zookeeper.url
in the
values.yml
file and that should provide the overrided zk url
a

ayush sharma

05/06/2021, 6:05 PM
Thank you @Xiang Fu for the help! I tried mentioning just the below values in values.yaml for zookeeper section
Copy code
zookeeper:
    enabled: false
    url: <ip>
    port: <pod-exposed-port>
When I install pinot using
helm -n my-pinot-kube install pinot .
it throws error:
Copy code
Error: template: pinot/templates/server/statefulset.yml:63:27: executing "pinot/templates/server/statefulset.yml" at <include "zookeeper.url" .>: error calling include: template: pinot/templates/_helpers.tpl:79:33: executing "zookeeper.url" at <index .Values "configurationOverrides" "zookeeper.connect">: error calling index: index of nil pointer
Line 79 in _helpers.tpl is
Copy code
{{- $zookeeperConnectOverride := index .Values "configurationOverrides" "zookeeper.connect" }}
x

Xiang Fu

05/06/2021, 6:09 PM
isee
can you add below into values.yml
Copy code
zookeeper.connect=<ip>:2181
Copy code
zookeeper:
    enabled: false
    url: <ip>
    port: <pod-exposed-port>
    connect: <ip>:<pod-exposed-port>
a

ayush sharma

05/06/2021, 6:14 PM
I tried :
Copy code
zookeeper:
    enabled: false
    url: <ip>
    port: <pod-exposed-port>
    connect: 192.168.49.2:30760
    zookeeper.connect: 192.168.49.2:30760
Still
Copy code
Error: template: pinot/templates/server/statefulset.yml:63:27: executing "pinot/templates/server/statefulset.yml" at <include "zookeeper.url" .>: error calling include: template: pinot/templates/_helpers.tpl:79:33: executing "zookeeper.url" at <index .Values "configurationOverrides" "zookeeper.connect">: error calling index: index of nil pointer
Does it have issue with index .Values "configurationOverrides" ?
x

Xiang Fu

05/06/2021, 8:27 PM
I feel so, let’s remove that line in _helpers.tpl
since we only need zk url
Fix this in the PR