collectd.conf(5): Add new config option.
authorPaul Sadauskas <psadauskas@gmail.com>
Thu, 30 Jul 2009 06:31:58 +0000 (08:31 +0200)
committerFlorian Forster <octo@leeloo.lan.home.verplant.org>
Thu, 30 Jul 2009 06:33:57 +0000 (08:33 +0200)
src/collectd.conf.pod
src/df.c

index 8a5f6fd..6483755 100644 (file)
@@ -870,6 +870,12 @@ match any one of the criteria are collected. By default only selected
 partitions are collected if a selection is made. If no selection is configured
 at all, B<all> partitions are selected.
 
+=item B<ReportByDevice> I<true>|I<false>
+
+Report using the device name rather than the mountpoint. i.e. with this I<false>,
+(the default), it will report a disk as "root", but with it I<true>, it will be
+"sda1" (or whichever).
+
 =back
 
 =head2 Plugin C<disk>
index dedf1bf..a9c1bdb 100644 (file)
--- a/src/df.c
+++ b/src/df.c
@@ -117,7 +117,7 @@ static int df_config (const char *key, const char *value)
                                || (strcasecmp (value, "Yes") == 0)
                                || (strcasecmp (value, "On") == 0))
                {
-                        by_device = false;
+                        by_device = true;
                }
                return (0);
        }
@@ -187,7 +187,12 @@ static int df_read (void)
                if (by_device) 
                {
                        // eg, /dev/hda1  -- strip off the "/dev/"
-                       strncpy (disk_name, mnt_ptr->device + 5, sizeof (disk_name));
+                       strncpy (disk_name, mnt_ptr->spec_device + 5, sizeof (disk_name));
+                       if (strlen(disk_name) < 1) 
+                       {
+                               DEBUG("df: no device name name for mountpoint %s, skipping", mnt_ptr->dir);
+                               continue;
+                       }
                } 
                else 
                {