helm-charts icon indicating copy to clipboard operation
helm-charts copied to clipboard

[loki] Standard labels

Open blezoray opened this issue 2 years ago • 2 comments

Hello,

I use the loki helm chart and compare to other loki charts (loki-simple-scalable for example), the loki helm chart doesn't define the standard helm labels, specified here: https://helm.sh/docs/chart_best_practices/labels/

Here is the code of your labels. https://github.com/grafana/helm-charts/blob/24d178939d7f51c137a2a77b0ee5f00f343ef538/charts/loki/templates/_helpers.tpl#L78

Could you update the labels to be inline with this recommendation ?

Rgds.

blezoray avatar Jun 08 '22 15:06 blezoray

Something like this:

{{/*
Common labels
*/}}
{{- define "loki.labels" -}}
app.kubernetes.io/name: {{ template "loki.name" . }}
helm.sh/chart: {{ include "loki.chart" . }}
app.kubernetes.io/managed-by: {{ .Release.Service }}
app.kubernetes.io/instance: {{ .Release.Name }}
app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
{{- end }}

blezoray avatar Jun 08 '22 15:06 blezoray

And it doesn't allow you to add a custom label. Can we have any values config to be able to add a custom label if we want?

betorvs avatar Jun 29 '22 16:06 betorvs