Elasticsearch: do not include heritage selector (#437)

According to https://github.com/helm/charts/blob/master/REVIEW_GUIDELINES.md, service selector should not contain heritage. It would break (future) migration from helm 2 to helm 3 (during the rolling update only, there could be a downtime when only old nodes are ready but new services is there).
elastic · Jun 11, 2020 · a8e8752 · a8e8752
1 parent 4eceebc
commit a8e8752
Showing 1 changed file with 0 additions and 1 deletion.
diff --git a/elasticsearch/templates/service.yaml b/elasticsearch/templates/service.yaml
@@ -20,7 +20,6 @@ metadata:
 spec:
   type: {{ .Values.service.type }}
   selector:
-    heritage: {{ .Release.Service | quote }}
     release: {{ .Release.Name | quote }}
     chart: "{{ .Chart.Name }}"
     app: "{{ template "elasticsearch.uname" . }}"