Update metadata.yaml for some python collectors (#15513)

* add more info to metadata.yaml for ceph, haproxy and tor

* changes

* add dovecot metdata

* add metdata for hpssa

* add icecast metadata

* add rethinkdbs metadata

* add am2320 metadata

* address review comments

* Update collectors/python.d.plugin/ceph/metadata.yaml

* Update collectors/python.d.plugin/am2320/metadata.yaml

* Apply suggestions from code review

* Update collectors/python.d.plugin/ceph/metadata.yaml

---------

Co-authored-by: Andrew Maguire <andrewm4894@gmail.com>
Co-authored-by: Fotis Voutsas <fotis@netdata.cloud>

1 files changed, 71 insertions, 16 deletions

diff --git a/collectors/python.d.plugin/am2320/metadata.yaml b/collectors/python.d.plugin/am2320/metadata.yaml
index 25fa19f18a..440b3a2123 100644
--- a/collectors/python.d.plugin/am2320/metadata.yaml
+++ b/collectors/python.d.plugin/am2320/metadata.yaml
@@ -3,7 +3,7 @@ meta:
   module_name: am2320
   monitored_instance:
     name: AM2320
-    link: ''
+    link: 'https://learn.adafruit.com/adafruit-am2320-temperature-humidity-i2c-sensor/overview'
     categories:
     - data-collection.hardware-devices-and-sensors
     icon_filename: 'microchip.svg'
@@ -12,43 +12,98 @@ meta:
       list: []
   info_provided_to_referring_integrations:
     description: ''
-  keywords: []
+  keywords:
+    - temperature
+    - am2320
+    - sensor
+    - humidity
   most_popular: false
 overview:
   data_collection:
-    metrics_description: 'Monitor AM2320 metrics with Netdata for optimal temperature and humidity sensor performance. Improve your sensor performance with comprehensive dashboards and anomaly detection.'
-    method_description: ''
+    metrics_description: 'This collector monitors AM2320 sensor metrics about temperature and humidity.'
+    method_description: 'It retrieves temperature and humidity values by contacting an AM2320 sensor over i2c.'
   supported_platforms:
     include: []
     exclude: []
-  multi_instance: true
+  multi_instance: false
   additional_permissions:
     description: ''
   default_behavior:
     auto_detection:
-      description: ''
+      description: 'Assuming prerequisites are met, the collector will try to connect to the sensor via i2c'
     limits:
       description: ''
     performance_impact:
       description: ''
 setup:
   prerequisites:
-    list: []
+    list:
+      - title: 'Sensor connection to a Raspberry Pi'
+        description: |
+          Connect the am2320 to the Raspberry Pi I2C pins
+
+          Raspberry Pi 3B/4 Pins:
+
+          - Board 3.3V (pin 1) to sensor VIN (pin 1)
+          - Board SDA (pin 3) to sensor SDA (pin 2)
+          - Board GND (pin 6) to sensor GND (pin 3)
+          - Board SCL (pin 5) to sensor SCL (pin 4)
+
+          You may also need to add two I2C pullup resistors if your board does not already have them. The Raspberry Pi does have internal pullup resistors but it doesn't hurt to add them anyway. You can use 2.2K - 10K but we will just use 10K. The resistors go from VDD to SCL and SDA each.
+      - title: 'Software requirements'
+        description: |
+          Install the Adafruit Circuit Python AM2320 library:
+
+          `sudo pip3 install adafruit-circuitpython-am2320`
   configuration:
     file:
-      name: ''
-      description: ''
+      name: python.d/am2320.conf
     options:
-      description: ''
+      description: |
+        There are 2 sections:
+
+        * Global variables
+        * One or more JOBS that can define multiple different instances to monitor.
+
+        The following options can be defined globally: priority, penalty, autodetection_retry, update_every, but can also be defined per JOB to override the global values.
+
+        Additionally, the following collapsed table contains all the options that can be configured inside a JOB definition.
+
+        Every configuration JOB starts with a `job_name` value which will appear in the dashboard, unless a `name` parameter is specified.
       folding:
-        title: ''
+        title: "Config options"
         enabled: true
-      list: []
+      list:
+        - name: update_every
+          description: Sets the default data collection frequency.
+          default_value: 5
+          required: false
+        - name: priority
+          description: Controls the order of charts at the netdata dashboard.
+          default_value: 60000
+          required: false
+        - name: autodetection_retry
+          description: Sets the job re-check interval in seconds.
+          default_value: 0
+          required: false
+        - name: penalty
+          description: Indicates whether to apply penalty to update_every in case of failures.
+          default_value: yes
+          required: false
+        - name: name
+          description: Job name. This value will overwrite the `job_name` value. JOBS with the same name are mutually exclusive. Only one of them will be allowed running at any time. This allows autodetection to try several alternatives and pick the one that works.
+          default_value: ''
+          required: false
     examples:
       folding:
-        enabled: true
-        title: ''
-      list: []
+        enabled: false
+        title: "Config"
+      list:
+        - name: Local sensor
+          description: A basic JOB configuration
+          config: |
+            local_sensor:
+              name: 'Local AM2320'
 troubleshooting:
   problems:
     list: []
@@ -61,7 +116,7 @@ metrics:
   availability: []
   scopes:
   - name: global
-    description: ""
+    description: "These metrics refer to the entire monitored application."
     labels: []
     metrics:
     - name: am2320.temperature