Merge #252

252: Support dynamic serviceMonitor Namespace r=brunoocasali a=felipevacar

In the default values the serviceMonitor.Namespace is available but this is not taking into account

# Pull Request

## Related issue
Fixes #<issue_number>

## What does this PR do?
- ...

## PR checklist
Please check if your PR fulfills the following requirements:
- [ ] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
- [ ] Have you read the contributing guidelines?
- [ ] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!


Co-authored-by: felipevacar <[email protected]>
Co-authored-by: Bruno Casali <[email protected]>

Author: 3 people

charts/meilisearch/templates/serviceMonitor.yaml

@@ -3,6 +3,7 @@ apiVersion: monitoring.coreos.com/v1
 kind: ServiceMonitor
 metadata:
   name: {{ include "meilisearch.fullname" . }}
+  namespace: {{ default .Release.Namespace .Values.serviceMonitor.namespace }}
   labels:
     {{- include "meilisearch.labels" . | nindent 4 }}
     {{- with .Values.serviceMonitor.additionalLabels }}

charts/meilisearch/values.yaml

@@ -185,6 +185,8 @@ command: []
 serviceMonitor:
   # -- Enable ServiceMonitor to configure scraping
   enabled: false
+  # -- Set Namespace
+  namespace: null
   # -- Set of labels to transfer from the Kubernetes Service onto the target
   additionalLabels: {}
   # -- Set scraping frequency