Merge pull request newrelic#18793 from paologallinaharbur/fix/discovery

fix(discovery): fix docs related to discovery
SIAM-T · Oct 21, 2024 · c09cbd2 · c09cbd2
2 parents baef4f2 + 329a668
commit c09cbd2
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/...ost-integrations/host-integrations-list/monitor-services-running-amazon-ecs.mdx b/...ost-integrations/host-integrations-list/monitor-services-running-amazon-ecs.mdx
@@ -284,7 +284,7 @@ Here's a detailed example of doing the above procedure for NGINX:
    integrations:
      - name: nri-nginx
        env:
-         STATUS_URL: http://${'${discovery.ip}'}:/status
+         STATUS_URL: http://${discovery.ip}:${discovery.port}/status
          REMOTE_MONITORING: true
          METRICS: 1
    ```
@@ -311,7 +311,7 @@ Here's a detailed example of doing the above procedure for NGINX:
           integrations:
             - name: nri-nginx
               env:
-                STATUS_URL: http://${'${discovery.ip}'}:/status
+                STATUS_URL: http://${discovery.ip}:${discovery.port}/status
                 REMOTE_MONITORING: true
                 METRICS: 1
           path: /etc/newrelic-infra/integrations.d/nginx-config.yml

diff --git a/...integrations-sdk/troubleshooting/not-seeing-infrastructure-integration-data.mdx b/...integrations-sdk/troubleshooting/not-seeing-infrastructure-integration-data.mdx
@@ -145,7 +145,7 @@ To troubleshoot and resolve the problem:
        </DNT>
 
        ```
-       nginx-server-metrics: {\"name\":\"com.newrelic.nginx\",\"protocol_version\":\"1\",\"integration_version\":\"0.5.0\",\"metrics\":
+       nri-nginx: {\"name\":\"com.newrelic.nginx\",\"protocol_version\":\"1\",\"integration_version\":\"0.5.0\",\"metrics\":
        [{\"event_type\":\"NginxSample\",\"net.connectionsAcceptedPerSecond\":0.10344827586206896,\"net.connectionsActive\":1,
        \"net.connectionsDroppedPerSecond\":0,\"net.connectionsReading\":0,\"net.connectionsWaiting\":0,\"net.connectionsWriting\":1,
        \"net.requestsPerSecond\":0.10344827586206896,\"software.edition\":\"open source\",\"software.version\":\"1.4.6 (Ubuntu)\"}],\"inventory\":{},\"events\":[]}

diff --git a/...e/host-integrations/installation/container-auto-discovery-host-integrations.mdx b/...e/host-integrations/installation/container-auto-discovery-host-integrations.mdx
@@ -44,13 +44,12 @@ These examples (for Docker-only environments and for Kubernetes) show how to con
           image: /nginx/
           label.env: production
     integrations:
-      - name: nginx-server-metrics
-        command: metrics
+      - name: nri-nginx
         env:
-    # use discovery.private.ip if container doesn't have attached public ip address
+          # use discovery.private.ip if container doesn't have attached public ip address
           STATUS_URL: http://${discovery.ip}:${discovery.port}/status
-          STATUS_MODULE: discover
           REMOTE_MONITORING: true
+          METRICS: 1
         labels:
           env: ${discovery.label.env}
           role: ${discovery.label.role}
@@ -94,6 +93,7 @@ These examples (for Docker-only environments and for Kubernetes) show how to con
               env:
                 # Use the discovered IP as the host address
                 STATUS_URL: http://${discovery.ip}/server-status?auto
+                REMOTE_MONITORING: true
                 METRICS: 1
       ```
   </Collapser>