Merge pull request #296 from lachie83/feat-chart-enable-hostPort

feat(charts): make hostPort and service type configurable

Author: Matthew Fisher

charts/router/templates/router-deployment.yaml

@@ -49,13 +49,21 @@ spec:
               fieldPath: metadata.namespace
         ports:
         - containerPort: 8080
+{{- if .Values.host_port.enabled }}
           hostPort: 80
+{{- end }}
         - containerPort: 6443
+{{- if .Values.host_port.enabled }}
           hostPort: 443
+{{- end }}
         - containerPort: 2222
+{{- if .Values.host_port.enabled }}
           hostPort: 2222
+{{- end }}
         - containerPort: 9090
+{{- if .Values.host_port.enabled }}
           hostPort: 9090
+{{- end }}
         livenessProbe:
           httpGet:
             path: /healthz

charts/router/templates/router-service.yaml

@@ -11,7 +11,7 @@ metadata:
   labels:
     heritage: deis
 spec:
-  type: LoadBalancer
+  type: {{ default "LoadBalancer" .Values.service_type}}
   selector:
     app: deis-router
   ports:

charts/router/values.yaml

@@ -15,3 +15,10 @@ deployment_annotations:
 # which need to be applied can be specified as key-value pairs under "service_annotations"
 service_annotations:
   #<example-key>: <example-value>
+
+# Enable to pin router pod hostPort when using minikube or vagrant
+host_port:
+  enabled: false
+
+# Service type default to LoadBalancer 
+# service_type: LoadBalancer