diff options
author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2013-03-26 10:26:10 -0400 |
---|---|---|
committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2013-03-26 10:26:10 -0400 |
commit | f22ad439539c85ab32daf04bdfa88fca6adaf6da (patch) | |
tree | e6a1c6127f594592a05b7a2b94d6c14dee7d325b /src/lib/Bcfg2/Server/Plugins/Metadata.py | |
parent | d92f67d2f8148afd191c292903e3fb6db9abddbc (diff) | |
download | bcfg2-f22ad439539c85ab32daf04bdfa88fca6adaf6da.tar.gz bcfg2-f22ad439539c85ab32daf04bdfa88fca6adaf6da.tar.bz2 bcfg2-f22ad439539c85ab32daf04bdfa88fca6adaf6da.zip |
bcfg2-lint: only check clients.xml things if clients.xml is in use
Diffstat (limited to 'src/lib/Bcfg2/Server/Plugins/Metadata.py')
-rw-r--r-- | src/lib/Bcfg2/Server/Plugins/Metadata.py | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/src/lib/Bcfg2/Server/Plugins/Metadata.py b/src/lib/Bcfg2/Server/Plugins/Metadata.py index b2c68c1a7..41d65a7bd 100644 --- a/src/lib/Bcfg2/Server/Plugins/Metadata.py +++ b/src/lib/Bcfg2/Server/Plugins/Metadata.py @@ -1498,6 +1498,9 @@ class MetadataLint(Bcfg2.Server.Lint.ServerPlugin): def deprecated_options(self): """ check for the location='floating' option, which has been deprecated in favor of floating='true' """ + if not hasattr(self.metadata, "clients_xml"): + # using metadata database + return clientdata = self.metadata.clients_xml.xdata for el in clientdata.xpath("//Client"): loc = el.get("location") @@ -1523,6 +1526,9 @@ class MetadataLint(Bcfg2.Server.Lint.ServerPlugin): def bogus_profiles(self): """ check for clients that have profiles that are either not flagged as public groups in groups.xml, or don't exist """ + if not hasattr(self.metadata, "clients_xml"): + # using metadata database + return for client in self.metadata.clients_xml.xdata.findall('.//Client'): profile = client.get("profile") if profile not in self.metadata.groups: @@ -1563,6 +1569,9 @@ class MetadataLint(Bcfg2.Server.Lint.ServerPlugin): def duplicate_clients(self): """ check for clients that are defined twice. """ + if not hasattr(self.metadata, "clients_xml"): + # using metadata database + return self.duplicate_entries( self.metadata.clients_xml.xdata.xpath("//Client"), "client") |