search

billimek / billimek-charts

DEPRECATED - new home is https://github.com/k8s-at-home/charts
Apache License 2.0
89 stars 52 forks source link

[sonarr] Fix liveness and readiness probes on basic auth deployment #257

Closed LEI closed 4 years ago

LEI commented 4 years ago

Once basic auth enabled, deployment fails because the probes wait for a successful status (2xx or 3xx):

HTTP probe failed with statuscode: 401

This changes httpGet to tcpSocket, making it identical to radarr and lidarr charts health checks.

billimek commented 4 years ago

Nice catch, thanks for the fix, @LEI. Would you be willing to do the same for the radarr chart as well in a different pull request?

LEI commented 4 years ago

You are welcome. It seems the radarr chart already uses tcpSocket.port: http, as well as lidarr.