datastax / pulsar-helm-chart

Apache Pulsar Helm chart
Apache License 2.0
46 stars 38 forks source link

Pulsar heartbeat init container logic #221

Closed ddieruf closed 2 years ago

ddieruf commented 2 years ago

The heartbeat deployment has an initContainer that waits for bookkeeper to be ready. The logic to wait is a loop that curls the broker on a certain port. If enableTls is true then the loop uses the broker statefulset and attempts an https connection on port 8443. What seems odd to me, is the broker statefulset only opens ports 8080 and 6650.

Should the broker statefulset also open an additional https port?

    ports:
        {{- if .Values.enableTls }}
        - name: https
          containerPort: 8443
        {{- end }}
        - name: http
          containerPort: 8080
        - name: pulsar
          containerPort: 6650

Note: adding this new port opening didn't solve my issue of the heartbeat's initContainer completing successfully but this seems like a needed thing.

ddieruf commented 2 years ago

I see what's happening. Nevermind.