Be more flexible about where the service binary lives for better cross platform support.
This commit is contained in:
parent
9569be8bdb
commit
d86dad76ba
1 changed files with 7 additions and 7 deletions
|
@ -33,7 +33,7 @@ args = open(argfile, 'r').read()
|
|||
items = shlex.split(args)
|
||||
|
||||
if not len(items):
|
||||
print "failed=True msg='the module requires arguments (-a)'"
|
||||
print json.dumps(dict(failed=True, msg='this module requires arguments (-a)'))
|
||||
sys.exit(1)
|
||||
|
||||
params = {}
|
||||
|
@ -46,13 +46,13 @@ state = params.get('state','unknown')
|
|||
|
||||
# running and started are the same
|
||||
if state not in [ 'running', 'started', 'stopped', 'restarted' ]:
|
||||
print "failed=True msg='invalid state'"
|
||||
print json.dumps(dict(failed=True, msg='invalid state'))
|
||||
sys.exit(1)
|
||||
|
||||
# ===========================================
|
||||
# get service status
|
||||
|
||||
status = os.popen("/sbin/service %s status" % name).read()
|
||||
status = os.popen("service %s status" % name).read()
|
||||
|
||||
# ===========================================
|
||||
# determine if we are going to change anything
|
||||
|
@ -84,12 +84,12 @@ def _run(cmd):
|
|||
rc = 0
|
||||
if changed:
|
||||
if state in ('started', 'running'):
|
||||
rc = _run("/sbin/service %s start" % name)
|
||||
rc = _run("service %s start" % name)
|
||||
elif state == 'stopped':
|
||||
rc = _run("/sbin/service %s stop" % name)
|
||||
rc = _run("service %s stop" % name)
|
||||
elif state == 'restarted':
|
||||
rc1 = _run("/sbin/service %s stop" % name)
|
||||
rc2 = _run("/sbin/service %s start" % name)
|
||||
rc1 = _run("service %s stop" % name)
|
||||
rc2 = _run("service %s start" % name)
|
||||
rc = rc1 and rc2
|
||||
|
||||
if rc != 0:
|
||||
|
|
Loading…
Reference in a new issue