Bump version and also change the toggle based on feedback

Author: furstblumier

charts/docker-mailserver/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 appVersion: "14.0.0"
 description: A fullstack but simple mailserver (smtp, imap, antispam, antivirus, ssl...) using Docker.
 name: docker-mailserver
-version: 4.1.2
+version: 4.1.3
 sources:
 - https://github.com/docker-mailserver/docker-mailserver-helm
 maintainers:

charts/docker-mailserver/templates/service.yaml

@@ -26,7 +26,7 @@ metadata:
 spec:
   ## If a load balancer is being used, ensure that the newer type of LB that passes along IP information is used
   ## rather than the legacy one.
-  externalTrafficPolicy: {{ default "Local" .Values.service.externalTrafficPolicy }}
+  externalTrafficPolicy: {{ .Values.service.externalTrafficPolicy }}
   selector:
     app.kubernetes.io/name: {{ template "dockermailserver.fullname" . }}
     release: "{{ .Release.Name }}"

charts/docker-mailserver/values.yaml

@@ -301,8 +301,9 @@ service:
   ## Manually overwrite the externalTrafficPolicy. One of:
   ## - Local
   ## - Cluster
-  ## Set it to "Local" when used with type "LoadBalancer", unless you have a good reason not to.
-  # externalTrafficPolicy: Local
+  ## Set it to "Local" when used with type "LoadBalancer" and set it to "Cluster" when used with type "ClusterIP",
+  ## unless you have a good reason not to.
+  externalTrafficPolicy: "Local"
   ## If there is a port associated with a given service, expose it here.
   # port:
   ## If there is a particular IP that should be used for the service, specify it here.