From c9a74c86b2f67e342a51559310451eedfb752d18 Mon Sep 17 00:00:00 2001
From: Calin Don <calin.don@gmail.com>
Date: Thu, 10 Aug 2017 08:46:51 +0300
Subject: [PATCH] Allow setting externalTrafficPolicy for nginx-ingress
 controller service (#1542)

* Allow setting externalTrafficPolicy for nginx-ingress controller service

* Update Chart.yaml
---
 stable/nginx-ingress/Chart.yaml                        | 2 +-
 stable/nginx-ingress/templates/controller-service.yaml | 3 +++
 stable/nginx-ingress/values.yaml                       | 5 +++++
 3 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/stable/nginx-ingress/Chart.yaml b/stable/nginx-ingress/Chart.yaml
index 02f8681ae4..47da67c145 100755
--- a/stable/nginx-ingress/Chart.yaml
+++ b/stable/nginx-ingress/Chart.yaml
@@ -1,5 +1,5 @@
 name: nginx-ingress
-version: 0.7.1
+version: 0.7.2
 appVersion: 0.9.0-beta.7
 description: An nginx Ingress controller that uses ConfigMap to store the nginx configuration.
 icon: https://upload.wikimedia.org/wikipedia/commons/thumb/c/c5/Nginx_logo.svg/500px-Nginx_logo.svg.png
diff --git a/stable/nginx-ingress/templates/controller-service.yaml b/stable/nginx-ingress/templates/controller-service.yaml
index 25b4fb86f3..08c523dc50 100644
--- a/stable/nginx-ingress/templates/controller-service.yaml
+++ b/stable/nginx-ingress/templates/controller-service.yaml
@@ -24,6 +24,9 @@ spec:
 {{- if .Values.controller.service.loadBalancerSourceRanges }}
   loadBalancerSourceRanges:
 {{ toYaml .Values.controller.service.loadBalancerSourceRanges | indent 4 }}
+{{- end }}
+{{- if .Values.controller.service.externalTrafficPolicy }}
+  externalTrafficPolicy: "{{ .Values.controller.service.externalTrafficPolicy }}"
 {{- end }}
   ports:
     - name: http
diff --git a/stable/nginx-ingress/values.yaml b/stable/nginx-ingress/values.yaml
index e1945743bc..4a2038c137 100644
--- a/stable/nginx-ingress/values.yaml
+++ b/stable/nginx-ingress/values.yaml
@@ -77,6 +77,11 @@ controller:
     loadBalancerIP: ""
     loadBalancerSourceRanges: []
 
+    ## Set external traffic policy to: "Local" to preserve source IP on
+    ## providers supporting it
+    ## Ref: https://kubernetes.io/docs/tutorials/services/source-ip/#source-ip-for-services-with-typeloadbalancer
+    externalTrafficPolicy: ""
+
     targetPorts:
       http: 80
       https: 443