From 0b2218d1d9c8305160591fc815a0f51b744f2ca1 Mon Sep 17 00:00:00 2001
From: Alexandre Garcia <ag94441@gmail.com>
Date: Mon, 12 Aug 2019 09:57:56 -0400
Subject: [PATCH] Only include clusterIp on vault service if set in values
 (#12)

---
 templates/server-service.yaml | 2 ++
 values.yaml                   | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/templates/server-service.yaml b/templates/server-service.yaml
index a70ab4e..3b37d8f 100644
--- a/templates/server-service.yaml
+++ b/templates/server-service.yaml
@@ -16,7 +16,9 @@ metadata:
     # https://github.com/kubernetes/kubernetes/issues/58662
     service.alpha.kubernetes.io/tolerate-unready-endpoints: "true"
 spec:
+  {{- if .Values.server.service.clusterIP }}
   clusterIP: {{ .Values.server.service.clusterIP }}
+  {{- end }}
   # We want the servers to become available even if they're not ready
   # since this DNS is also used for join operations.
   publishNotReadyAddresses: true
diff --git a/values.yaml b/values.yaml
index e4772a8..c363a11 100644
--- a/values.yaml
+++ b/values.yaml
@@ -81,7 +81,7 @@ server:
     # Kubernetes will create a "headless" service.  Headless services can be 
     # used to communicate with pods directly through DNS instead of a round robin
     # load balancer.
-    clusterIP: ""
+    # clusterIP: None
 
   # This configures the Vault Statefulset to create a PVC for data 
   # storage when using the file backend.