Add ability to template ServiceMonitor labels and add metrics port to Service

Author: jfcmartins

weaviate/Chart.yaml

@@ -7,7 +7,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 16.8.0
+version: 16.9.0
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to

weaviate/templates/weaviateService.yaml

@@ -15,8 +15,13 @@ spec:
   ports:
     {{- range .Values.service.ports }}
     - {{ toYaml . | indent 6 | trim }}
-      targetPort: 8080
     {{- end }}
+{{ if eq .Values.serviceMonitor.enabled true -}}
+    - name: metrics
+      protocol: TCP
+      port: {{ .Values.env.PROMETHEUS_MONITORING_PORT }}
+      targetPort: {{ .Values.env.PROMETHEUS_MONITORING_PORT }}
+{{ end }}
 {{ if eq .Values.service.type "ClusterIP" -}}
   {{ if .Values.service.clusterIP }}
   clusterIP: {{ .Values.service.clusterIP }}

weaviate/templates/weaviateServiceMonitor.yaml

@@ -8,6 +8,9 @@ metadata:
     app: weaviate
     app.kubernetes.io/name: weaviate
     app.kubernetes.io/managed-by: helm
+{{- if .Values.serviceMonitor.additionalLabels }}
+    {{- toYaml .Values.serviceMonitor.additionalLabels | nindent 4 }}
+{{- end }}
 spec:
   jobLabel: weaviate
   selector:

weaviate/values.yaml

@@ -106,7 +106,7 @@ service:
     - name: http
       protocol: TCP
       port: 80
-      # Target port is going to be the same for every port
+      targetPort: 8080
   type: LoadBalancer
   loadBalancerSourceRanges: []
   # optionally set cluster IP if you want to set a static IP
@@ -137,6 +137,7 @@ serviceMonitor:
   enabled: false
   interval: 30s
   scrapeTimeout: 10s
+  additionalLabels: {}
 
 # Adjust liveness, readiness and startup probes configuration
 startupProbe:
@@ -229,6 +230,7 @@ env:
   # Expose metrics on port 2112 for Prometheus to scrape
   PROMETHEUS_MONITORING_ENABLED: false
   PROMETHEUS_MONITORING_GROUP: false
+  PROMETHEUS_MONITORING_PORT: 2112
 
   # Set a MEM limit for the Weaviate Pod so it can help you both increase GC-related 
   # performance as well as avoid GC-related out-of-memory (“OOM”) situations