fix externalTrafficPolicy

Author: furstblumier

charts/docker-mailserver/ci/ci-haproxy.yaml

@@ -1,2 +1,2 @@
 service:
-  type: ClusterIP
+  type: ClusterIP

charts/docker-mailserver/ci/ci-values.yaml

@@ -1,6 +1,5 @@
 service:
   type: ClusterIP
-  externalTrafficPolicy: Local
 
 configMaps:
   postfix-accounts.cf:

charts/docker-mailserver/templates/service.yaml

@@ -26,7 +26,9 @@ 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.
+  {{- if .Values.service.externalTrafficPolicy }}
   externalTrafficPolicy: {{ .Values.service.externalTrafficPolicy }}
+  {{- end }}
   selector:
     app.kubernetes.io/name: {{ template "dockermailserver.fullname" . }}
     release: "{{ .Release.Name }}"

charts/docker-mailserver/values.yaml

@@ -304,9 +304,9 @@ service:
   ## Manually overwrite the externalTrafficPolicy. One of:
   ## - Local
   ## - Cluster
-  ## Set it to "Local" when used with type "LoadBalancer" or "ClusterIP" and set it to "Cluster" when used with "NodePort",
+  ## Set it to "Local" when used with type "LoadBalancer" and set it to "Cluster" when used with "NodePort",
   ## unless you have a good reason not to.
-  externalTrafficPolicy: "Local"
+  # externalTrafficPolicy: "Cluster"
   ## 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.