datastax / pulsar-helm-chart

Apache Pulsar Helm chart
Apache License 2.0
47 stars 37 forks source link

fix parse error unclosed action #281

Closed anxuanzi closed 1 year ago

anxuanzi commented 1 year ago

Fix parse error at (pulsar/templates/broker-sts/broker-sts-transactions-metadata.yaml:19): unclosed action

anxuanzi commented 1 year ago
fullnameOverride: pulsar
dnsName: pulsar.example.com
kubernetesClusterDomain: cluster.local
dnsConfig:
  options:
    - name: ndots
      value: '4'
persistence: true
enableWaitContainers: true
default_storage:
  existingStorageClassName: default
  reclaimPolicy: Retain
image:
  brokerSts:
    # If not using tiered storage, you can use the smaller pulsar image for the broker
    repository: datastax/lunastreaming-all
    pullPolicy: IfNotPresent
    tag: 2.10_3.2
  function:
    repository: datastax/lunastreaming-all
    pullPolicy: IfNotPresent
    tag: 2.10_3.2
  zookeeper:
    repository: datastax/lunastreaming
    pullPolicy: IfNotPresent
    tag: 2.10_3.2
  bookkeeper:
    repository: datastax/lunastreaming
    pullPolicy: IfNotPresent
    tag: 2.10_3.2
  proxy:
    repository: datastax/lunastreaming-all
    pullPolicy: IfNotPresent
    tag: 2.10_3.2
  bastion:
    repository: datastax/lunastreaming
    pullPolicy: IfNotPresent
    tag: 2.10_3.2
  pulsarBeam:
    repository: kesque/pulsar-beam
    pullPolicy: IfNotPresent
    tag: 1.0.0
  burnell:
    repository: datastax/burnell
    pullPolicy: Always
    tag: 1.0.3
  burnellLogCollector:
    repository: datastax/burnell
    pullPolicy: IfNotPresent
    tag: logcollector_latest
  pulsarSQL:
    repository: datastax/lunastreaming-all
    tag: 2.10_3.2
    pullPolicy: IfNotPresent
  tardigrade:
    repository: storjlabs/gateway
    pullPolicy: IfNotPresent
    tag: 981f92a-v1.20.0-go1.17.5
enableAntiAffinity: false
enableTls: true
tls:
  zookeeper:
    enabled: true
    createCertificates: true
    enableHostnameVerification: true
    tlsSecretName: "pulsar-zookeeper-tls"
    configureKeystoreWithPasswordFile: true
    certSpec:
      privateKey:
        algorithm: "ECDSA"
        size: 256
  bookkeeper:
    enabled: true
    createCertificates: true
    tlsSecretName: "pulsar-bookkeeper-tls"
    certSpec:
      privateKey:
        algorithm: "ECDSA"
        size: 256
  function:
    enableTlsWithBroker: true
    createCertificates: true
    tlsSecretName: "pulsar-function-tls"
    enableHostnameVerification: true
    certSpec:
      privateKey:
        algorithm: "ECDSA"
        size: 256
  websocket:
    enableTlsWithBroker: true
    enableHostnameVerification: true
    certSpec:
      privateKey:
        algorithm: "ECDSA"
        size: 256
  proxy:
    enableTlsWithBroker: true
    enableHostnameVerification: true
    createCertificates: true
    tlsSecretName: "pulsar-proxy-tls"
    certSpec:
      privateKey:
        algorithm: "ECDSA"
        size: 256
  broker:
    createCertificates: true
    tlsSecretName: "pulsar-broker-tls"
    certSpec:
      privateKey:
        algorithm: "ECDSA"
        size: 256
  autoRecovery:
    enableHostnameVerification: true
    createCertificates: true
    tlsSecretName: "pulsar-autorecovery-tls"
    certSpec:
      privateKey:
        algorithm: "ECDSA"
        size: 256
  pulsarAdminConsole:
    enableTlsWithBroker: true
    createCertificates: true
    tlsSecretName: "pulsar-adminconsole-tls"
    certSpec:
      privateKey:
        algorithm: "ECDSA"
        size: 256
  ssCaCert:
    tlsSecretName: "pulsar-ss-ca"
    certSpec:
      privateKey:
        algorithm: "ECDSA"
        size: 256
enableTokenAuth: true
restartOnConfigMapChange:
  enabled: true
extra:
  broker: false
  brokerSts: true
  proxy: true
  function: true
  burnellLogCollector: false
  burnell: true
  pulsarHeartbeat: true
  pulsarAdminConsole: true
cert-manager:
  enabled: true
createCertificates:
  selfSignedPerComponent:
    enabled: true
zookeeper:
  replicaCount: 1
  resources:
    requests:
      memory: 300Mi
      cpu: 0.3
  configData:
    PULSAR_MEM: '-Xms300m -Xmx300m -Djute.maxbuffer=10485760 -XX:+ExitOnOutOfMemoryError'
bookkeeper:
  replicaCount: 1
  resources:
    requests:
      memory: 512Mi
      cpu: 0.3
  configData:
    BOOKIE_MEM: '-Xms312m -Xmx312m -XX:MaxDirectMemorySize=200m -XX:+ExitOnOutOfMemoryError'
brokerSts:
  component: broker
  replicaCount: 1
  service:
    headless: true
  ledger:
    defaultEnsembleSize: 1
    defaultAckQuorum: 1
    defaultWriteQuorum: 1
  resources:
    requests:
      memory: 600Mi
      cpu: 0.3
  configData:
    PULSAR_MEM: '-Xms400m -Xmx400m -XX:MaxDirectMemorySize=200m -XX:+ExitOnOutOfMemoryError'
autoRecovery:
  enableProvisionContainer: true
  resources:
    requests:
      memory: 300Mi
      cpu: 0.3
function:
  replicaCount: 1
  functionReplicaCount: 1
  resources:
    requests:
      memory: 512Mi
      cpu: 0.3
  configData:
    PULSAR_MEM: '-Xms312m -Xmx312m -XX:MaxDirectMemorySize=200m -XX:+ExitOnOutOfMemoryError'
proxy:
  replicaCount: 1
  resources:
    requests:
      memory: 512Mi
      cpu: 0.3
  wsResources:
    requests:
      memory: 512Mi
      cpu: 0.3
  configData:
    PULSAR_MEM: '-Xms400m -Xmx400m -XX:MaxDirectMemorySize=112m'
  autoPortAssign:
    enablePlainTextWithTLS: true
  service:
    autoPortAssign:
      enabled: true
grafanaDashboards:
  enabled: false
pulsarAdminConsole:
  replicaCount: 1
  authMode: k8s
  createUserSecret:
    enabled: true
    user: admin
    password: e9JYtk83*4#PM8
kube-prometheus-stack:
  enabled: false
  prometheusOperator:
    enabled: false
  grafana:
    enabled: false

this is the configuration I was using.

versions

Client Version: v1.24.8
Kustomize Version: v4.5.4
Server Version: v1.24.8

helm: version.BuildInfo{Version:"v3.9.0", GitCommit:"7ceeda6c585217a19a1131663d8cd1f7d641b2a7", GitTreeState:"clean", GoVersion:"go1.17.5"}
michaeljmarshall commented 1 year ago

@anxuanzi - which version of the helm chart are you using? I am using master, which is essentially 3.2.0. I wasn't able to reproduce the error with those values. I tried with a more recent version of helm and that didn't cause any errors either.

$ helm version
version.BuildInfo{Version:"v3.11.1", GitCommit:"293b50c65d4d56187cd4e2f390f0ada46b4c4737", GitTreeState:"clean", GoVersion:"go1.19.5"}

Does it fail for you with a more recent version of helm? Is there documentation that suggests this formatting is wrong? I ask because we use it elsewhere in the template files, so it'd be a larger change to make a complete fix.

anxuanzi commented 1 year ago

@michaeljmarshall I'm currently using helm Version:"v3.9.0"

michaeljmarshall commented 1 year ago

@anxuanzi - which version of the datastax pulsar helm chart are you using?

anxuanzi commented 1 year ago

@michaeljmarshall

pulsar chart version: 3.2.0 [2.8.0]

btw, I'm also using KubeSphere as GUI for Kubernetes

here is the full error message I got from the GUI:

Error: parse error at (pulsar/templates/broker-sts/broker-sts-transactions-metadata.yaml:19): unclosed action