Updated with recommendations from review and latest OpenCost release

Signed-off-by: Matt Ray <[email protected]>

.github/CODEOWNERS

@@ -31,6 +31,7 @@
 /charts/prometheus-nats-exporter/ @caarlos0 @okgolove
 /charts/prometheus-nginx-exporter/ @nlamirault @zeritti
 /charts/prometheus-node-exporter/ @gianrubio @zanhsieh @zeritti
+/charts/prometheus-opencost-exporter/ @mattray
 /charts/prometheus-operator-admission-webhook/ @zeritti
 /charts/prometheus-operator-crds/ @dacamposol @desaintmartin @jkroepke @QuentinBisson
 /charts/prometheus-pgbouncer-exporter/ @stewartshea @zeritti

charts/prometheus-opencost-exporter/Chart.yaml

@@ -1,7 +1,7 @@
-appVersion: 1.106.2
+appVersion: 1.106.3
 version: 0.1.0
 description: Prometheus OpenCost Exporter
-home: https://github.com/prometheus-community/opencost_exporter
+home: https://github.com/opencost/opencost-helm-chart
 name: prometheus-opencost-exporter
 sources:
 - https://github.com/prometheus-community/opencost_exporter

charts/prometheus-opencost-exporter/templates/_helpers.tpl

@@ -35,20 +35,20 @@ Create chart name and version as used by the chart label.
 Common labels
 */}}
 {{- define "prometheus-opencost-exporter.labels" -}}
-chart: {{ include "prometheus-opencost-exporter.chart" . }}
+helm.sh/chart: {{ include "prometheus-opencost-exporter.chart" . }}
 {{ include "prometheus-opencost-exporter.selectorLabels" . }}
-heritage: {{ .Release.Service }}
-{{- if .Values.commonLabels}}
-{{ toYaml .Values.commonLabels }}
+{{- if .Chart.AppVersion }}
+app.kubernetes.io/version: {{ .Chart.AppVersion | quote }}
 {{- end }}
+app.kubernetes.io/managed-by: {{ .Release.Service }}
 {{- end }}
 
 {{/*
 Selector labels
 */}}
 {{- define "prometheus-opencost-exporter.selectorLabels" -}}
-app: {{ include "prometheus-opencost-exporter.name" . }}
-release: {{ .Release.Name }}
+app.kubernetes.io/name: {{ include "prometheus-opencost-exporter.name" . }}
+app.kubernetes.io/instance: {{ .Release.Name }}
 {{- end }}
 
 {{/*