Merge pull request #25 from sysfivecom/master

Bump version and fixup flow for externalTrafficPolicy

Author: funkypenguin

charts/docker-mailserver/Chart.yaml

@@ -1,8 +1,8 @@
 apiVersion: v1
-appVersion: "6.3.0"
+appVersion: "7.0.0"
 description: A fullstack but simple mailserver (smtp, imap, antispam, antivirus, ssl...) using Docker.
 name: docker-mailserver
-version: 0.2.6
+version: 0.2.7
 sources:
 - https://github.com/funkypenguin/helm-docker-mailserver
 maintainers:

charts/docker-mailserver/templates/service.yaml

@@ -25,7 +25,7 @@ spec:
   ## rather than the legacy one.
   {{- if eq .Values.service.type "LoadBalancer" }}
   externalTrafficPolicy: Local
-  {{ else }}
+  {{- else if eq .Values.service.type "NodePort" }}
   externalTrafficPolicy: Cluster
   {{ end }}
   selector:

charts/docker-mailserver/values.yaml

@@ -3,7 +3,7 @@ image:
   # image.name is the name of the container image to use. Refer to https://hub.docker.com/r/tvial/docker-mailserver
   name: "tvial/docker-mailserver"
   # image.tag is the tag of the container image to use. Refer to https://hub.docker.com/r/tvial/docker-mailserver
-  tag: "release-v6.1.0"
+  tag: "release-v7.0.0"
   pullPolicy: "IfNotPresent"
 
 initContainer: