[logstash] add headless service for statefulset

logstash/examples/6.x/test/goss.yaml

@@ -9,10 +9,10 @@ http:
     status: 200
     timeout: 2000
     body:
-      - '"host" : "helm-logstash-six-logstash-0"'
+      - '"host" : "helm-logstash-six-logstash-0.helm-logstash-six-logstash-headless.helm-charts-testing.svc.cluster.local"'
       - '"version" : "6.8.10"'
       - '"http_address" : "0.0.0.0:9600"'
-      - '"name" : "helm-logstash-six-logstash-0"'
+      - '"name" : "helm-logstash-six-logstash-0.helm-logstash-six-logstash-headless.helm-charts-testing.svc.cluster.local"'
 
 file:
   /usr/share/logstash/config/logstash.yml:

logstash/templates/service-headless.yaml

@@ -0,0 +1,23 @@
+---
+kind: Service
+apiVersion: v1
+metadata:
+  name: "{{ template "logstash.fullname" . }}-headless"
+  labels:
+    app: "{{ template "logstash.fullname" . }}"
+    chart: "{{ .Chart.Name }}"
+    heritage: {{ .Release.Service | quote }}
+    release: {{ .Release.Name | quote }}
+{{- if .Values.labels }}
+{{ toYaml .Values.labels | indent 4 }}
+{{- end }}
+spec:
+  clusterIP: None
+  selector:
+    app: "{{ template "logstash.fullname" . }}"
+  ports:
+  - name: http
+    port: {{ .Values.httpPort }}
+{{- if .Values.extraPorts }}
+{{- toYaml .Values.extraPorts | nindent 2 }}
+{{- end }}

logstash/templates/statefulset.yaml

@@ -12,9 +12,7 @@ metadata:
     {{ $key }}: {{ $value | quote }}
     {{- end }}
 spec:
-  {{- if .Values.service }}
-  serviceName: {{ template "logstash.fullname" . }}
-  {{- end }}
+  serviceName: {{ template "logstash.fullname" . }}-headless
   selector:
     matchLabels:
       app: "{{ template "logstash.fullname" . }}"
@@ -151,7 +149,7 @@ spec:
         - name: http
           containerPort: {{ .Values.httpPort }}
       {{- if .Values.extraPorts }}
-        {{- toYaml .Values.extraPorts | nindent 8 }}  
+        {{- toYaml .Values.extraPorts | nindent 8 }}
       {{- end }}
         resources:
 {{ toYaml .Values.resources | indent 10 }}

logstash/tests/logstash_test.py

@@ -78,8 +78,9 @@ def test_defaults():
     )
 
     # Service
-    assert "serviceName" not in r["statefulset"][name]["spec"]
-    assert "service" not in r
+    assert r["statefulset"][name]["spec"]["serviceName"] == name + "-headless"
+    assert name + "-headless" in r["service"]
+    assert r["service"][name + "-headless"]["spec"]["ports"][0]["port"] == 9600
 
     # Other
     assert r["statefulset"][name]["spec"]["template"]["spec"]["securityContext"] == {