search

docker-mailserver / docker-mailserver-helm

Kubernetes Helm chart for docker-mailserver
https://github.com/docker-mailserver/docker-mailserver/
MIT License
107 stars 67 forks source link

Fix load balancer traffic policy #18

Closed bklang closed 4 years ago

bklang commented 4 years ago

This makes two changes: First, it upgrades the syntax to use newer version of the external traffic policy. This is based on testing on my Kubernetes 1.15 cluster, and was discovered from this reference Second, it makes the non-load-balancer case switch to Cluster policy, which is necessary to allow the cluster to route NodePort traffic without a Kubernetes load balancer

Also, fixed tiny whitespace nit

funkypenguin commented 4 years ago

Great fix, thanks @bklang !