Add support to rrdui from reading configs and rrds from subdirectories.

2 files changed, 35 insertions, 15 deletions

diff --git a/doc/httpd-sample.conf b/doc/httpd-sample.conf
new file mode 100644
index 0000000..c028cf6
--- /dev/null
+++ b/doc/httpd-sample.conf
@@ -0,0 +1,10 @@
+Alias /rrdui/ /home/sean/Projects/rrdui/html/
+ScriptAlias /rrduiback /home/sean/Projects/rrdui/tools/rrdui-cgi.py
+
+<Location "/rrduiback">
+    SetEnv CONFDIR "/opt/rrd/etc/rrdbot"
+    SetEnv WORKDIR "/opt/rrd/var"
+</Location>
+
+
+
diff --git a/tools/rrdui-cgi.py b/tools/rrdui-cgi.py
index fc58608..7390547 100755
--- a/tools/rrdui-cgi.py
+++ b/tools/rrdui-cgi.py
@@ -5,8 +5,6 @@ import sets, cgi, shlex
 import ConfigParser
 import rrdtool
 
-from rrdui import *
-
 # TODO: Temporary
 if not os.environ.has_key("CONFDIR") or not os.environ.has_key("WORKDIR"):
     raise "not setup properly. CONFDIR and WORKDIR env variables must be set"
@@ -14,6 +12,10 @@ if not os.environ.has_key("CONFDIR") or not os.environ.has_key("WORKDIR"):
 CONFDIR = os.environ["CONFDIR"]
 WORKDIR = os.environ["WORKDIR"]
 
+# We always have this as our current directory.
+# It helps when constructing graphs and the like
+os.chdir(WORKDIR)
+
 class GraphDef:
     filename = None
     filedata = None
@@ -24,6 +26,7 @@ class GraphDef:
     commands = ""
     category = ""
     name = ""
+    valid = False
 
     __config = None
 
@@ -32,6 +35,7 @@ class GraphDef:
         self.filedata = "%s/%s.rrd" % (WORKDIR, name)
         self.category = "All"
         self.name = name
+        self.valid = False
 
         cfg = self.__config = ConfigParser.RawConfigParser()
         cfg.read(self.filename)
@@ -43,15 +47,17 @@ class GraphDef:
             self.category = cfg.get("general", "category")
 
         # Loading graph stuff
-        if cfg.has_option("graph", "width"):
-            self.width = int(cfg.get("graph", "width"))
-        if cfg.has_option("graph", "height"):
-            self.height = int(cfg.get("graph", "height"))
-        if cfg.has_option("graph", "options"):
-            self.options = cfg.get("graph", "options")
-        if not cfg.has_option("graph", "commands"):
-            raise "Missing commands attribute in: %s" % self.filename
-        self.commands = cfg.get("graph", "commands")
+        if cfg.has_section("graph"):
+            if not cfg.has_option("graph", "commands"):
+                raise "Missing commands attribute in: %s" % self.filename
+            self.commands = cfg.get("graph", "commands")
+            if cfg.has_option("graph", "width"):
+                self.width = int(cfg.get("graph", "width"))
+            if cfg.has_option("graph", "height"):
+                self.height = int(cfg.get("graph", "height"))
+            if cfg.has_option("graph", "options"):
+                self.options = cfg.get("graph", "options")
+            self.valid = True
 
 
     def getCreateInfo(self):
@@ -92,14 +98,16 @@ class GraphDef:
         return (fields, interval)
 
 
-def loadGraphs():
+def loadGraphs(path = ""):
 
     # List files and add appropriate paths
     graphs = []
-    for file in os.listdir(CONFDIR):
-        if os.path.isdir(file):
+    for file in os.listdir(CONFDIR + "/" + path):
+        if os.path.isdir(CONFDIR + "/" + path + file):
+            if file != "." and file != "..":
+                graphs.extend(loadGraphs(path + file + "/"))
             continue
-        graphs.append(GraphDef(file))
+        graphs.append(GraphDef(path + file))
     return graphs
 
 
@@ -108,6 +116,8 @@ def listGraphs():
     graphs = loadGraphs()
     categories = {}
     for item in graphs:
+        if not item.valid:
+            continue
         if not categories.has_key(item.category):
             categories[item.category] = []
         categories[item.category].append(item)