Code

ipmi plugin: Improve error messages when removing a sensor.
authorFlorian Forster <octo@noris.net>
Fri, 4 Apr 2008 09:02:38 +0000 (11:02 +0200)
committerFlorian Forster <octo@noris.net>
Fri, 4 Apr 2008 09:03:58 +0000 (11:03 +0200)
Don't tell the user to file a bug report, when a sensor is removed because IPMI
reported an error. Tell him to file a bug report, though, if collectd doesn't
know how to handle the value.

src/ipmi.c

index bba4599e8911a499338a0b2099125d7dc6389123..516dd7ffafc49991f9958a052a9a3ac9041e50a6 100644 (file)
@@ -114,8 +114,7 @@ static void sensor_read_handler (ipmi_sensor_t *sensor,
   if (err != 0)
   {
     INFO ("ipmi plugin: sensor_read_handler: Removing sensor %s, "
-        "because it failed with status %#x. If you need this sensor, "
-        "please file a bug report.",
+        "because it failed with status %#x.",
         sensor_name_ptr, err);
     sensor_list_remove (sensor);
     return;
@@ -124,7 +123,8 @@ static void sensor_read_handler (ipmi_sensor_t *sensor,
   if (value_present != IPMI_BOTH_VALUES_PRESENT)
   {
     INFO ("ipmi plugin: sensor_read_handler: Removing sensor %s, "
-        "because it provides %s.",
+        "because it provides %s. If you need this sensor, "
+        "please file a bug report.",
         sensor_name_ptr,
         (value_present == IPMI_RAW_VALUE_PRESENT)
         ? "only the raw value"
@@ -160,7 +160,8 @@ static void sensor_read_handler (ipmi_sensor_t *sensor,
         
         sensor_type_str = ipmi_sensor_get_sensor_type_string (sensor);
         INFO ("ipmi plugin: sensor_read_handler: Removing sensor %s, "
-            "because I don't know how to handle its type (%#x, %s).",
+            "because I don't know how to handle its type (%#x, %s). "
+            "If you need this sensor, please file a bug report.",
             sensor_name_ptr, sensor_type, sensor_type_str);
         sensor_list_remove (sensor);
         return;