From 2769d83e0b648d1d623b4a1648973d6c32945b13 Mon Sep 17 00:00:00 2001
From: Narayan Desai <desai@mcs.anl.gov>
Date: Wed, 7 Dec 2005 00:12:31 +0000
Subject: Fix error reporting for parse failures Fix comment printing for
 etc/metadata.xml Add a better error message for client communication
 bootstrapping failures

git-svn-id: https://svn.mcs.anl.gov/repos/bcfg/trunk/bcfg2@1615 ce84e21b-d406-0410-9b95-82705330c041
---
 src/lib/Server/Metadata.py | 4 +++-
 src/lib/Server/Plugin.py   | 2 +-
 src/sbin/bcfg2             | 7 ++++++-
 3 files changed, 10 insertions(+), 3 deletions(-)

(limited to 'src')

diff --git a/src/lib/Server/Metadata.py b/src/lib/Server/Metadata.py
index 957179751..47bbb3ecb 100644
--- a/src/lib/Server/Metadata.py
+++ b/src/lib/Server/Metadata.py
@@ -1,7 +1,7 @@
 '''This file stores persistent metadata for the BCFG Configuration Repository'''
 __revision__ = '$Revision$'
 
-from lxml.etree import XML, SubElement, Element
+from lxml.etree import XML, SubElement, Element, _Comment, tostring
 from syslog import syslog, LOG_ERR, LOG_INFO
 
 from Bcfg2.Server.Plugin import SingleXMLFileBacked
@@ -106,6 +106,8 @@ class MetadataStore(SingleXMLFileBacked):
 
     def pretty_print(self, element, level=0):
         '''Produce a pretty-printed text representation of element'''
+        if isinstance(element, _Comment):
+            return (level * " ") + tostring(element)
         if element.text:
             fmt = "%s<%%s %%s>%%s</%%s>" % (level*" ")
             data = (element.tag, (" ".join(["%s='%s'" % (key, element.attrib[key]) for key in element.attrib])),
diff --git a/src/lib/Server/Plugin.py b/src/lib/Server/Plugin.py
index 7139d21ab..262d6092d 100644
--- a/src/lib/Server/Plugin.py
+++ b/src/lib/Server/Plugin.py
@@ -194,7 +194,7 @@ class ScopedXMLFile(SingleXMLFileBacked):
             xdata = XML(self.data)
         except XMLSyntaxError, msg:
             syslog(LOG_ERR, "Failed to parse %s"%(self.name))
-            syslog(LOG_ERR, msg)
+            # need to add in lxml error messages, once they are supported
             return
         self.store = {}
         for entry in [ent for ent in  xdata.getchildren() if not isinstance(ent, _Comment)]:
diff --git a/src/sbin/bcfg2 b/src/sbin/bcfg2
index a2d0634f5..7a7eb7f77 100644
--- a/src/sbin/bcfg2
+++ b/src/sbin/bcfg2
@@ -97,7 +97,12 @@ if __name__ == '__main__':
 
         cf = ConfigParser()
         cf.read('/etc/bcfg2.conf')
-        location = cf.get("components", "bcfg2")
+        try:
+            location = cf.get("components", "bcfg2")
+        except (NoSectionError, NoOptionError):
+            print "Can't find bcfg2 server location to connect to"
+            print "Check /etc/bcfg2.conf"
+            raise SystemExit, 1
         proxy = ServerProxy(location)
         user = 'root'
         retries = 0
-- 
cgit v1.2.3-1-g7c22