Merge pull request #316 from skvidal/devel

fix up service module to make it start processes that are listed to be
This commit is contained in:
Michael DeHaan 2012-05-06 16:43:20 -07:00
commit 9a54cda6f2

10
service
View file

@ -138,9 +138,9 @@ status = status_stdout + status_stderr
running = False running = False
if status_stdout.find("not running") != -1: if status_stdout.find("stopped") != -1 or rc == 3:
running = False running = False
elif status_stdout.find("running") != -1: elif status_stdout.find("running") != -1 or rc == 0:
running = True running = True
elif name == 'iptables' and status_stdout.find("ACCEPT") != -1: elif name == 'iptables' and status_stdout.find("ACCEPT") != -1:
# iptables status command output is lame # iptables status command output is lame
@ -165,7 +165,7 @@ if state or enable:
# =========================================== # ===========================================
# determine if we are going to change anything # determine if we are going to change anything
if not running and state == "started": if not running and state in ("started", "running"):
changed = True changed = True
elif running and state == "stopped": elif running and state == "stopped":
changed = True changed = True
@ -183,13 +183,13 @@ if state or enable:
elif state == 'restarted': elif state == 'restarted':
rc1, stdout1, stderr1 = _run("%s %s stop" % (SERVICE, name)) rc1, stdout1, stderr1 = _run("%s %s stop" % (SERVICE, name))
rc2, stdout2, stderr2 = _run("%s %s start" % (SERVICE, name)) rc2, stdout2, stderr2 = _run("%s %s start" % (SERVICE, name))
rc_state = rc and rc1 and rc2 rc_state = rc + rc1 + rc2
stdout = stdout1 + stdout2 stdout = stdout1 + stdout2
stderr = stderr1 + stderr2 stderr = stderr1 + stderr2
out += stdout out += stdout
err += stderr err += stderr
rc = rc and rc_state rc = rc + rc_state
if rc != 0: if rc != 0: