Update values file and added templates to both services

The helm lint tests were failing because the httpPortName and transportPortName key value were missing from the values.yaml file. Also update both services to include the port name template.
elastic · Sep 18, 2019 · eb334d8 · eb334d8
1 parent 9e0ddbb
commit eb334d8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/elasticsearch/templates/service.yaml b/elasticsearch/templates/service.yaml
@@ -46,7 +46,7 @@ spec:
   selector:
     app: "{{ template "uname" . }}"
   ports:
-  - name: http
+  - name: {{ .Values.httpPortName | default "http" }}
     port: {{ .Values.httpPort }}
-  - name: transport
+  - name: {{ .Values.transportPortName | default "transport" }}
     port: {{ .Values.transportPort }}
diff --git a/elasticsearch/values.yaml b/elasticsearch/values.yaml
@@ -132,6 +132,8 @@ service:
   type: ClusterIP
   nodePort:
   annotations: {}
+  httpPortName: http
+  transportPort: transport
 
 updateStrategy: RollingUpdate