From 4bda10087ce97b16cafb79ff34b0c298d1bad903 Mon Sep 17 00:00:00 2001
From: dave <dave@davepedu.com>
Date: Mon, 15 May 2017 17:49:29 -0700
Subject: [PATCH] diskspace: discover available filesystems

---
 pymonitor/monitors/diskspace.py | 33 ++++++++++++++++++++++++++++-----
 1 file changed, 28 insertions(+), 5 deletions(-)

diff --git a/pymonitor/monitors/diskspace.py b/pymonitor/monitors/diskspace.py
index 4845594..f8ec0db 100644
--- a/pymonitor/monitors/diskspace.py
+++ b/pymonitor/monitors/diskspace.py
@@ -1,9 +1,30 @@
 from os import statvfs
+import logging
 
 
-def diskspace(filesystems=[]):
-    for fs in filesystems:
-        stats = statvfs(fs)
+def diskspace(filesystems=[], discover=True):
+    """
+    Emit disk space usage statistics for the passed filesystems.
+    :param filesystems: list of mountpoints to gather stats for
+    :param discover: automatically find non-temporary filesystems to gather statistics for. Duplicates from the
+                     filesystems param will be ignored.
+    """
+    filesystems = [f.rstrip("/") for f in filesystems]
+    if discover:
+        with open("/proc/mounts") as f:
+            for line in f.readlines():
+                device, mountpoint, fstype, options, _, _ = line.split(" ")
+                # filter out some mountpoints we probably don't care about space on
+                if any([mountpoint.startswith(prefix) for prefix in ["/sys", "/proc", "/dev", "/run"]]):
+                    continue
+                filesystems.append(mountpoint)
+
+    for fs in set(filesystems):
+        try:
+            stats = statvfs(fs)
+        except FileNotFoundError:
+            logging.warning("filesystem not found: %s", fs)
+            continue
 
         info = {
             "fs": fs,
@@ -71,5 +92,7 @@ mapping = {
 
 
 if __name__ == '__main__':
-    for item in diskspace(filesystems=["/", "/dev"]):
-        print(item)
+    import sys
+    from pprint import pprint
+    for item in diskspace(filesystems=sys.argv[1:]):
+        pprint(item)