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/RcUpdate.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/RcUpdate.py')
-rw-r--r-- | src/lib/Bcfg2/Client/Tools/RcUpdate.py | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/lib/Bcfg2/Client/Tools/RcUpdate.py b/src/lib/Bcfg2/Client/Tools/RcUpdate.py index a53a45251..d5cef6e34 100644 --- a/src/lib/Bcfg2/Client/Tools/RcUpdate.py +++ b/src/lib/Bcfg2/Client/Tools/RcUpdate.py @@ -23,8 +23,8 @@ class RcUpdate(Bcfg2.Client.Tools.SvcTool): # check if service is enabled cmd = '/sbin/rc-update show default | grep %s' - rc = self.cmd.run(cmd % entry.get('name'))[0] - is_enabled = (rc == 0) + rv = self.cmd.run(cmd % entry.get('name'))[0] + is_enabled = (rv == 0) # check if init script exists try: @@ -36,8 +36,8 @@ class RcUpdate(Bcfg2.Client.Tools.SvcTool): # check if service is enabled cmd = '/etc/init.d/%s status | grep started' - rc = self.cmd.run(cmd % entry.attrib['name'])[0] - is_running = (rc == 0) + rv = self.cmd.run(cmd % entry.attrib['name'])[0] + is_running = (rv == 0) if entry.get('status') == 'on' and not (is_enabled and is_running): entry.set('current_status', 'off') @@ -60,16 +60,16 @@ class RcUpdate(Bcfg2.Client.Tools.SvcTool): self.start_service(entry) # make sure it's enabled cmd = '/sbin/rc-update add %s default' - rc = self.cmd.run(cmd % entry.get('name'))[0] - return (rc == 0) + rv = self.cmd.run(cmd % entry.get('name'))[0] + return (rv == 0) elif entry.get('status') == 'off': if entry.get('current_status') == 'on': self.stop_service(entry) # make sure it's disabled cmd = '/sbin/rc-update del %s default' - rc = self.cmd.run(cmd % entry.get('name'))[0] - return (rc == 0) + rv = self.cmd.run(cmd % entry.get('name'))[0] + return (rv == 0) return False |