Collectd: 5.7.1 df не захватывает данные '/'

Привет @jblaine!

спасибо, что сообщили об этом! Не могли бы вы также предоставить свои mtab ? Я помню, что раньше плагин df был сбит с толку из-за того, что rootfs монтировался поверх фактического (например, xfs) монтирования.

С наилучшими пожеланиями,
—Окто

Для справки: # 1402

rootfs / rootfs rw 0 0
...
/dev/mapper/vg_root-root / xfs rw,relatime,attr2,inode64,noquota 0 0

Я вижу именно это и потратил немного времени на то, чтобы возиться с настройками плагина df, и я могу заставить его работать _только_, если я укажу MountPoint "/" - если я укажу что-нибудь еще, я получу / boot, а не /. Это на CentOS 7

@jblaine @ChrisHeerschap имел ту же проблему в RHEL 7.4 - попробуйте FSType "rootfs"
это сработало для нас сразу

@ xneo64 Я попробовал это и подтвердил, что это работает - не подумал попробовать, спасибо! Также работает без проблем в 6.x, поэтому добавляйте его повсеместно. Спасибо!

Я думаю, что эта проблема возникает из-за того, что определено, что /dev/mapper/vg_root-root / xfs rw,relatime,attr2,inode64,noquota 0 0 дублируется с rootfs / rootfs rw 0 0 (оба собирают '/').
Следовательно, '/' не собирается в условиях, которые определяют только строку /dev/mapper/vg_root-root / xfs ~ , например, здесь: FSType "xfs" или Device "/dev/mapper/vg_root-root" .

Я хочу исправить эту ошибку. # 2564 - это один план, или есть также план по удалению "rootfs device (или filetype)" из цели дублирования. Что лучше?

Правильное решение здесь - не пропускать файловую систему, которая является дубликатом файловой системы, которая была проигнорирована. Это означает, что если запись в списке файловой системы для «/» игнорируется из-за того, что она является rootfs (а не ext4 или xfs), то позже, когда встречается код «/» для ext4 / xfs / etc, тогда эта запись не t игнорируется. Это работает и с другими комбинациями. Игнорируемая запись в списке монтирования не должна препятствовать записи, которая не игнорируется.

diff --git a/src/df.c b/src/df.c
index dd90f24..e7d2019 100644
--- a/src/df.c
+++ b/src/df.c
@@ -173,12 +180,18 @@ static int df_read(void) {
     char const *dev =
         (mnt_ptr->spec_device != NULL) ? mnt_ptr->spec_device : mnt_ptr->device;

-    if (ignorelist_match(il_device, dev))
+    if (ignorelist_match(il_device, dev)) {
+      mnt_ptr->ignored = 1;
       continue;
-    if (ignorelist_match(il_mountpoint, mnt_ptr->dir))
+    }
+    if (ignorelist_match(il_mountpoint, mnt_ptr->dir)) {
+      mnt_ptr->ignored = 1;
       continue;
-    if (ignorelist_match(il_fstype, mnt_ptr->type))
+    }
+    if (ignorelist_match(il_fstype, mnt_ptr->type)) {
+      mnt_ptr->ignored = 1;
       continue;
+    }

     /* search for duplicates *in front of* the current mnt_ptr. */
     for (dup_ptr = mnt_list; dup_ptr != NULL; dup_ptr = dup_ptr->next) {
@@ -188,6 +201,8 @@ static int df_read(void) {
         dup_ptr = NULL;
         break;
       }
+      if (dup_ptr->ignored == 1)
+        continue;

       /* Duplicate found: leave non-NULL dup_ptr. */
       if (by_device && (mnt_ptr->spec_device != NULL) &&
diff --git a/src/utils_mount.h b/src/utils_mount.h
index 0ad7d02..fd62c09 100644
--- a/src/utils_mount.h
+++ b/src/utils_mount.h
@@ -82,6 +82,7 @@ struct _cu_mount_t {
   char *device;      /* "none" or "proc" or "/dev/hda1" */
   char *type;        /* "sysfs" or "ext3" */
   char *options;     /* "rw,noatime,commit=600,quota,grpquota" */
+  int  ignored;
   cu_mount_t *next;
 };