fnproject / fn-helm

Helm Chart for Fn
Apache License 2.0
56 stars 24 forks source link

default to using empty Redis password #10

Closed derekschultz closed 6 years ago

derekschultz commented 6 years ago

By setting usePassword to false, the Redis Helm chart should allow an empty password (https://github.com/kubernetes/charts/blob/master/stable/redis/templates/deployment.yaml#L43-L52).

Prior to this, the fn-api pods were spitting out auth errors from Redis: level=error msg="error fetching queued calls" error="NOAUTH Authentication required."

In addition, sync functions worked while async didn't (because of Redis). This fixes that issue.