[prometheus-snmp-exporter] default serviceMonitor to Release Namespace

charts/prometheus-snmp-exporter/Chart.yaml

@@ -1,7 +1,7 @@
 apiVersion: v1
 description: Prometheus SNMP Exporter
 name: prometheus-snmp-exporter
-version: 1.8.2
+version: 2.0.0
 appVersion: v0.21.0
 home: https://github.com/prometheus/snmp_exporter
 sources:

charts/prometheus-snmp-exporter/README.md

@@ -27,6 +27,22 @@ _See [configuration](#configuration) below._
 
 _See [helm install](https://helm.sh/docs/helm/helm_install/) for command documentation._
 
+## Upgrading Chart
+
+```console
+helm upgrade [RELEASE_NAME] prometheus-community/prometheus-snmp-exporter
+```
+
+_See [helm upgrade](https://helm.sh/docs/helm/helm_upgrade/) for command documentation._
+
+### Upgrading an existing Release to a new major version
+
+A major chart version change (like v1.2.3 -> v2.0.0) indicates that there is an incompatible breaking change needing manual actions.
+
+### From 1.x to 2.x
+
+This version upgrades changes the `serviceMonitor.namespace` value from `monitoring` to the namespace the release is deployed to.
+
 ## Uninstall Chart
 
 ```console

charts/prometheus-snmp-exporter/templates/servicemonitor.yaml

@@ -5,9 +5,7 @@ apiVersion: monitoring.coreos.com/v1
 kind: ServiceMonitor
 metadata:
   name: {{ template "prometheus-snmp-exporter.fullname" $  }}-{{ .name }}
-  {{- if $.Values.serviceMonitor.namespace }}
-  namespace: {{ $.Values.serviceMonitor.namespace }}
-  {{- end }}
+  namespace: {{ $.Values.serviceMonitor.namespace | default $.Release.Namespace }}
   labels:
     {{- include "prometheus-snmp-exporter.labels" $ | indent 4 }}
     {{- range $key, $value := .labels | default $.Values.serviceMonitor.selector }}

charts/prometheus-snmp-exporter/values.yaml

@@ -146,7 +146,8 @@ configmapReload:
 # A service monitor will be created per each item in serviceMonitor.params[]
 serviceMonitor:
   enabled: false
-  namespace: monitoring
+  # Default value is the namespace the release is deployed to
+  # namespace: monitoring
 
   path: /snmp