diff options
author | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2012-09-14 06:36:27 -0700 |
---|---|---|
committer | Chris St. Pierre <chris.a.st.pierre@gmail.com> | 2012-09-14 06:36:27 -0700 |
commit | 62cf189b1caf948cd01220edf59e991c7290e54e (patch) | |
tree | 834b30d9e0d4f2211eaeafb321b40aff8542dac1 /src/lib/Bcfg2/Client/Tools | |
parent | 91167b2c01e5a0abafd0c111fa3546601e1f5cf0 (diff) | |
parent | e20b039306c0bac3286ab76a267593d57a1b1e3f (diff) | |
download | bcfg2-62cf189b1caf948cd01220edf59e991c7290e54e.tar.gz bcfg2-62cf189b1caf948cd01220edf59e991c7290e54e.tar.bz2 bcfg2-62cf189b1caf948cd01220edf59e991c7290e54e.zip |
Merge pull request #48 from zultron/master
Chkconfig driver: two fixes and RFE
Diffstat (limited to 'src/lib/Bcfg2/Client/Tools')
-rw-r--r-- | src/lib/Bcfg2/Client/Tools/Chkconfig.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/lib/Bcfg2/Client/Tools/Chkconfig.py b/src/lib/Bcfg2/Client/Tools/Chkconfig.py index ad9e58350..b62fb9ef8 100644 --- a/src/lib/Bcfg2/Client/Tools/Chkconfig.py +++ b/src/lib/Bcfg2/Client/Tools/Chkconfig.py @@ -70,7 +70,7 @@ class Chkconfig(Bcfg2.Client.Tools.SvcTool): self.logger.info("Installing Service %s" % (entry.get('name'))) rv = True if entry.get('status') == 'off': - rv &= self.cmd.run(rcmd + " --level 0123456" % + rv &= self.cmd.run((rcmd + " --level 0123456") % (entry.get('name'), entry.get('status')))[0] == 0 if entry.get("current_status") == "on": @@ -79,7 +79,7 @@ class Chkconfig(Bcfg2.Client.Tools.SvcTool): rv &= self.cmd.run(rcmd % (entry.get('name'), entry.get('status')))[0] == 0 if entry.get("current_status") == "off": - rv &= self.start_service(entry) + rv &= (self.start_service(entry) == 0) return rv def FindExtra(self): |