From 559178e148c94d50e91b1ba42db0992b3fac079b Mon Sep 17 00:00:00 2001
From: Rick Box <boxrick@gmail.com>
Date: Thu, 29 Nov 2018 01:19:41 +0000
Subject: [PATCH] Improving helm chart to allow for service serviceAnnotations:

I need the ability to define serviceAnnotations for the keel service.
This is a small fix to allow that functionality in the same way you can
do it for the pods themselves.
---
 chart/keel/Chart.yaml             | 2 +-
 chart/keel/templates/service.yaml | 4 ++++
 chart/keel/values.yaml            | 5 +++++
 3 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/chart/keel/Chart.yaml b/chart/keel/Chart.yaml
index 51d18f95..15e81c47 100644
--- a/chart/keel/Chart.yaml
+++ b/chart/keel/Chart.yaml
@@ -1,7 +1,7 @@
 apiVersion: v1
 name: keel
 description: Open source, tool for automating Kubernetes deployment updates. Keel is stateless, robust and lightweight.
-version: 0.7.4
+version: 0.7.5
 # Note that we use appVersion to get images tag, so make sure this is correct.
 appVersion: 0.12.0
 keywords:
diff --git a/chart/keel/templates/service.yaml b/chart/keel/templates/service.yaml
index e9056826..bc17c0df 100644
--- a/chart/keel/templates/service.yaml
+++ b/chart/keel/templates/service.yaml
@@ -9,6 +9,10 @@ metadata:
     chart: {{ template "keel.chart" . }}
     release: {{ .Release.Name }}
     heritage: {{ .Release.Service }}
+{{- with .Values.serviceAnnotations }}
+  annotations:
+{{ toYaml . | indent 4 }}
+{{- end }}
 spec:
   type: {{ .Values.service.type }}
   {{- if .Values.service.clusterIP }}
diff --git a/chart/keel/values.yaml b/chart/keel/values.yaml
index d329cae5..531e0cc2 100644
--- a/chart/keel/values.yaml
+++ b/chart/keel/values.yaml
@@ -143,6 +143,11 @@ createNamespaceResource: false
 
 podAnnotations: {}
 
+serviceAnnotations: {}
+# Useful for making the load balancer internal
+# serviceAnnotations:
+#    cloud.google.com/load-balancer-type: Internal
+
 aws:
   region: null