[stable/redis] Fix ServiceMonitor to not leak.

stable/redis/Chart.yaml

@@ -1,5 +1,5 @@
 name: redis
-version: 5.0.0
+version: 5.1.0
 appVersion: 4.0.11
 description: Open source, advanced key-value store. It is often referred to as a data structure server since keys can contain strings, hashes, lists, sets and sorted sets.
 keywords:

stable/redis/README.md

@@ -121,7 +121,7 @@ The following table lists the configurable parameters of the Redis chart and the
 | `metrics.service.loadBalancerIP`           | loadBalancerIP if redis metrics service type is `LoadBalancer`                                                 | `nil`                                                |
 | `metrics.resources`                        | Exporter resource requests/limit                                                                               | Memory: `256Mi`, CPU: `100m`                         |
 | `metrics.serviceMonitor.enabled`           | if `true`, creates a Prometheus Operator ServiceMonitor (also requires `metrics.enabled` to be `true`)         | `false`                                              |
-| `metrics.serviceMonitor.namespace`         | Namespace which Prometheus is running in                                                                       | `monitoring`                                         |
+| `metrics.serviceMonitor.namespace`         | Optional namespace which Prometheus is running in                                                              | `nil`                                                |
 | `metrics.serviceMonitor.interval`          | How frequently to scrape metrics (use by default, falling back to Prometheus' default)                         |  `nil`                                               |
 | `metrics.serviceMonitor.selector`          | Default to kube-prometheus install (CoreOS recommended), but should be set according to Prometheus install     | `{ prometheus: kube-prometheus }`                    |
 | `persistence.existingClaim`                | Provide an existing PersistentVolumeClaim                                                                      | `nil`                                                |

stable/redis/templates/metrics-prometheus.yaml

@@ -23,6 +23,8 @@ spec:
   selector:
     matchLabels:
       app: {{ template "redis.name" . }}
+      release: {{ .Release.Name }}
   namespaceSelector:
-      any: true
+    matchNames:
+    - {{ .Release.Namespace }}
 {{- end -}}

stable/redis/values.yaml

@@ -339,7 +339,8 @@ metrics:
   # Enable this if you're using https://github.com/coreos/prometheus-operator
   serviceMonitor:
     enabled: false
-    namespace: monitoring
+    ## Specify a namespace if needed
+    # namespace: monitoring
     # fallback to the prometheus default unless specified
     # interval: 10s
     ## Defaults to what's used if you follow CoreOS [Prometheus Install Instructions](https://github.com/helm/charts/tree/master/stable/prometheus-operator#tldr)