diff options
author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2013-09-26 13:49:27 -0400 |
---|---|---|
committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2013-09-26 13:49:27 -0400 |
commit | 0bf89e137796811478575455c6bcbb008bec71f8 (patch) | |
tree | f53a5b9a1720644292aef44c02bb442b37e566ab /src/lib/Bcfg2/Server | |
parent | 35b53c77c4b7edad7cf84146abf5722ea5323eba (diff) | |
download | bcfg2-0bf89e137796811478575455c6bcbb008bec71f8.tar.gz bcfg2-0bf89e137796811478575455c6bcbb008bec71f8.tar.bz2 bcfg2-0bf89e137796811478575455c6bcbb008bec71f8.zip |
Metadata: better logging when updating XML data fails
Diffstat (limited to 'src/lib/Bcfg2/Server')
-rw-r--r-- | src/lib/Bcfg2/Server/Plugins/Metadata.py | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/src/lib/Bcfg2/Server/Plugins/Metadata.py b/src/lib/Bcfg2/Server/Plugins/Metadata.py index 04ba79c55..d47fd644b 100644 --- a/src/lib/Bcfg2/Server/Plugins/Metadata.py +++ b/src/lib/Bcfg2/Server/Plugins/Metadata.py @@ -677,14 +677,15 @@ class Metadata(Bcfg2.Server.Plugin.Metadata, """ Generic method to modify XML data (group, client, etc.) """ node = self._search_xdata(tag, name, config.xdata, alias=alias) if node is None: - self.logger.error("%s \"%s\" does not exist" % (tag, name)) - raise Bcfg2.Server.Plugin.MetadataConsistencyError + msg = "%s \"%s\" does not exist" % (tag, name) + self.logger.error(msg) + raise Bcfg2.Server.Plugin.MetadataConsistencyError(msg) xdict = config.find_xml_for_xpath('.//%s[@name="%s"]' % (tag, node.get('name'))) if not xdict: - self.logger.error("Unexpected error finding %s \"%s\"" % - (tag, name)) - raise Bcfg2.Server.Plugin.MetadataConsistencyError + msg = 'Unexpected error finding %s "%s"' % (tag, name) + self.logger.error(msg) + raise Bcfg2.Server.Plugin.MetadataConsistencyError(msg) for key, val in list(attribs.items()): xdict['xquery'][0].set(key, val) config.write_xml(xdict['filename'], xdict['xmltree']) |