Merge pull request #216 from sysdiglabs/staging

Release Y22W18
sysdiglabs · May 6, 2022 · 3de3b53 · 3de3b53
2 parents 96c4d69 + 55aece8
commit 3de3b53
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/resources/nginx-ingress/INSTALL.md b/resources/nginx-ingress/INSTALL.md
@@ -1,7 +1,7 @@
 # Installing the exporter
-Nginx ingress controller is already instrumented so you don't have to add any extra exporter. However, ensure that your nginx ingress controller is already exposing these metrics.
+Nginx ingress controller is already instrumented so you don't have to add any extra exporter. However, ensure that your nginx ingress controller is already exposing these metrics. Don't forget to include the `prometheus.io/port` and `prometheus.io/scrape` annotations.
 
-For example if you are deploying your nginx ingress controller with helm, make sure you have these values.
+For example if you are deploying your nginx ingress controller using the helm chart from Kubernetes, the values would be:
 
 ```yaml
 controller:
@@ -10,6 +10,7 @@ controller:
     enabled: true
   podAnnotations:
     prometheus.io/port: "10254"
+    prometheus.io/scrape: "true"
 ```
 
 # Sysdig Agent configuration