pulsar-helm-chart icon indicating copy to clipboard operation
pulsar-helm-chart copied to clipboard

PulsarSQL coordinator liveness probe unauthorized

Open hyperevo opened this issue 9 months ago • 0 comments

The PulsarSQL liveness probe fails causing a crashloopbackoff. It fails because because presto requires authentication but the livenessprobe does not use authentication as seen here: https://github.com/datastax/pulsar-helm-chart/blob/a6e2cfb6b76044f1d88678756f5aa41cae671701/helm-chart-sources/pulsar/templates/pulsarSql/deployment-coordinator.yaml#L152

The presto service doesnt by defualt even require a password. just a username. This problem is fixed by adding the following to the liveness probe in the above link. This adds the base64 encoded username of "admin":

          httpHeaders:
            - name: Authorization
              value: Basic YWRtaW46

I can make a PR to add this in but I wanted to ask about it here first. Does anyone object to this solution? Another solution could be to set up actual authentication and handle that correctly.

hyperevo avatar May 07 '24 16:05 hyperevo