diff options
author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2012-09-26 09:48:58 -0400 |
---|---|---|
committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2012-09-26 10:05:22 -0400 |
commit | aeddf10f2e129c70b182e91b727ccc8028e3fd45 (patch) | |
tree | 2213bf1a27912b75a7b160a0b2494acee052be19 /src/lib/Bcfg2/Client/Tools/Encap.py | |
parent | be0e43af56fbd56dbb3f8d0266503e7b820f2c11 (diff) | |
download | bcfg2-aeddf10f2e129c70b182e91b727ccc8028e3fd45.tar.gz bcfg2-aeddf10f2e129c70b182e91b727ccc8028e3fd45.tar.bz2 bcfg2-aeddf10f2e129c70b182e91b727ccc8028e3fd45.zip |
made full pylint checks the default, expanded pylint coverage to lots of tools
Diffstat (limited to 'src/lib/Bcfg2/Client/Tools/Encap.py')
-rw-r--r-- | src/lib/Bcfg2/Client/Tools/Encap.py | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/src/lib/Bcfg2/Client/Tools/Encap.py b/src/lib/Bcfg2/Client/Tools/Encap.py index fa09c3ec7..b5057786f 100644 --- a/src/lib/Bcfg2/Client/Tools/Encap.py +++ b/src/lib/Bcfg2/Client/Tools/Encap.py @@ -4,6 +4,7 @@ import glob import re import Bcfg2.Client.Tools + class Encap(Bcfg2.Client.Tools.PkgTool): """Support for Encap packages.""" name = 'Encap' @@ -14,9 +15,6 @@ class Encap(Bcfg2.Client.Tools.PkgTool): pkgtool = ("/usr/local/bin/epkg -l -f -q %s", ("%s", ["url"])) splitter = re.compile('.*/(?P<name>[\w-]+)\-(?P<version>[\w\.+-]+)') -# If you define self.pkgtool and self.pkgname it will [use] the Pkgtool.Install -# method will do the installation stuff for you - def RefreshPackages(self): """Try to find encap packages.""" self.installed = {} @@ -26,24 +24,24 @@ class Encap(Bcfg2.Client.Tools.PkgTool): self.installed[match.group('name')] = match.group('version') else: print("Failed to split name %s" % pkg) - self.logger.debug("Encap.py: RefreshPackages: self.installed.keys() are:") + self.logger.debug("Encap: RefreshPackages: self.installed.keys() are:") self.logger.debug("%s" % list(self.installed.keys())) def VerifyPackage(self, entry, _): """Verify Package status for entry.""" if not entry.get('version'): - self.logger.info("Insufficient information of Package %s; cannot Verify" % entry.get('name')) + self.logger.info("Insufficient information of Package %s; " + "cannot Verify" % entry.get('name')) return False cmdrc = self.cmd.run("/usr/local/bin/epkg -q -S -k %s-%s >/dev/null" % (entry.get('name'), entry.get('version')))[0] if cmdrc != 0: - self.logger.debug("Package %s version incorrect" % entry.get('name')) + self.logger.debug("Package %s version incorrect" % + entry.get('name')) else: return True return False - # Can use the FindExtraPackages method from Bcfg2.Client.Tools.PkgTool - def RemovePackages(self, packages): """Deal with extra configuration detected.""" names = " ".join([pkg.get('name') for pkg in packages]) |