From 505191e85084a4d62bf4ce67fe90b56ed4542f7f Mon Sep 17 00:00:00 2001 From: pierre-yves Date: Tue, 7 Sep 2021 15:47:15 +0200 Subject: [PATCH] =?UTF-8?q?fix=20charts=20README.md=20to=20give=20addition?= =?UTF-8?q?al=20detail=20on=20prometheus=20metrics=20=E2=80=A6=20(#7601)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * fix charts README.md to give additional detail on prometheus metrics configuration * charts readme.md remove blank line --- charts/ingress-nginx/README.md | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/charts/ingress-nginx/README.md b/charts/ingress-nginx/README.md index 22e66fb85..fecbbcde2 100644 --- a/charts/ingress-nginx/README.md +++ b/charts/ingress-nginx/README.md @@ -84,7 +84,8 @@ else it would make it impossible to evacuate a node. See [gh issue #7127](https: The Nginx ingress controller can export Prometheus metrics, by setting `controller.metrics.enabled` to `true`. -You can add Prometheus annotations to the metrics service using `controller.metrics.service.annotations`. Alternatively, if you use the Prometheus Operator, you can enable ServiceMonitor creation using `controller.metrics.serviceMonitor.enabled`. +You can add Prometheus annotations to the metrics service using `controller.metrics.service.annotations`. +Alternatively, if you use the Prometheus Operator, you can enable ServiceMonitor creation using `controller.metrics.serviceMonitor.enabled`. And set `controller.metrics.serviceMonitor.additionalLabels.release="prometheus"`. "release=prometheus" should match the label configured in the prometheus servicemonitor ( see `kubectl get servicemonitor prometheus-kube-prom-prometheus -oyaml -n prometheus`) ### ingress-nginx nginx\_status page/stats server