Fix #417 Add serviceMonitor template for the telegraf chart

.github/workflows/helm-charts-test.yaml

@@ -59,9 +59,15 @@ jobs:
           INFLUXDB_ENTERPRISE_LICENSE_KEY: "${{ secrets.INFLUXDB_ENTERPRISE_LICENSE_KEY }}"
         if: steps.list-changed.outputs.changed == 'true'
 
+      - name: Create custom resources
+        run: |
+          kubectl create -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/main/bundle.yaml
+        if: steps.list-changed.outputs.changed == 'true'
+
       - name: Run chart-testing (install)
         id: install
         run: |
+          set -o pipefail;
           ct install --namespace=default 2>&1 | tee $RUNNER_TEMP/$GITHUB_RUN_NUMBER.txt
 # When https://github.com/helm/chart-testing/issues/212 is fixed, this can be used to set the license key instead of using env from secret
 #        run: ct install --namespace=default --helm-extra-args="--set license.key=${INFLUXDB_ENTERPRISE_LICENSE_KEY}"

.github/workflows/patterns.txt

@@ -1 +1,2 @@
 Error loading config file
+Release not loaded

charts/telegraf/Chart.yaml

@@ -1,6 +1,6 @@
 apiVersion: v2
 name: telegraf
-version: 1.8.23
+version: 1.8.24
 appVersion: 1.24.3
 deprecated: false
 description: Telegraf is an agent written in Go for collecting, processing, aggregating, and writing metrics.

charts/telegraf/ci/prometheus-monitor-values.yaml

@@ -0,0 +1,8 @@
+config:
+  outputs:
+    - prometheus_client:
+        listen: ":9273"
+serviceMonitor:
+  create: true
+  labels:
+    team: losers

charts/telegraf/templates/_helpers.tpl

@@ -706,6 +706,17 @@ Create the name of the service account to use
 {{- end -}}
 {{- end -}}
 
+{{/*
+Create the name of the service monitor to use
+*/}}
+{{- define "telegraf.serviceMonitorName" -}}
+{{- if .Values.serviceMonitor.create -}}
+    {{ default (include "telegraf.fullname" .) .Values.serviceMonitor.name }}
+{{- else -}}
+    {{ default "default" .Values.serviceMonitor.name }}
+{{- end -}}
+{{- end -}}
+
 {{/*
 Get health configuration
 */}}

charts/telegraf/templates/servicemonitor.yaml

@@ -0,0 +1,33 @@
+{{- if .Values.serviceMonitor.create }}
+apiVersion: monitoring.coreos.com/v1
+kind: ServiceMonitor
+metadata:
+  name: {{ template "telegraf.serviceMonitorName" . }}
+  namespace: {{ .Values.serviceMonitor.namespaceOverride | default .Release.Namespace }}
+  labels:
+    {{- include "telegraf.labels" . | nindent 4 }}
+    {{- if .Values.serviceMonitor.labels }}
+      {{- toYaml .Values.serviceMonitor.labels | nindent 4 }}
+    {{- end }}    
+  {{- if .Values.serviceMonitor.annotations }}
+  annotations:
+    {{- toYaml .Values.serviceMonitor.annotations | nindent 4 }}
+  {{- end }}
+spec:
+  selector:
+    matchLabels:
+    {{- include "telegraf.labels" . | nindent 6 }}
+  namespaceSelector:
+    matchNames:
+    - {{ .Release.Namespace }}
+  endpoints:
+  {{- range $objectKey, $objectValue := .Values.config.outputs }}
+  {{- range $key, $value := . -}}
+    {{- $tp := typeOf $value -}}
+    {{- if eq $key "prometheus_client" }}
+    - port: {{ trimPrefix ":" $value.listen }}
+      path: {{ $value.path | default "/metrics" }}
+    {{- end }}
+  {{- end -}}
+  {{- end }}
+{{- end }}

charts/telegraf/values.yaml

@@ -109,6 +109,24 @@ serviceAccount:
   name:
   # Annotations for the ServiceAccount
   annotations: {}
+
+# To make use of the serviceMonitorm, please add a prometheus_client entry to config.outputs:
+    # - prometheus_client:
+    #     listen: ":9273"
+    #     path: /metrics
+serviceMonitor:
+  # Specifies whether a ServiceMonitor should be created
+  create: false
+  # The name of the ServiceMonitor to use.
+  # If not set and create is true, a name is generated using the fullname template
+  name:
+  # Extra Labels for the ServiceMonitor
+  labels: {}
+  # Annotations for the ServiceMonitor
+  annotations: {}
+  # To install the serviceMonitor in a different namespace than the release set its name here
+  # For example if your prometheus is <v0.19.0 the serviceMonitor must be installed in the prometheus namespace
+  namespaceOverride:
 ## Exposed telegraf configuration
 ## For full list of possible values see `/docs/all-config-values.yaml` and `/docs/all-config-values.toml`
 ## ref: https://docs.influxdata.com/telegraf/v1.1/administration/configuration/