Merge pull request #756 from sfromm/issue755

Issue755 - fixes for service module

library/service

@@ -132,14 +132,14 @@ def main():
     module = AnsibleModule(
         argument_spec = dict(
             name = dict(required=True),
-            state = dict(required=True, choices=['running', 'started', 'stopped', 'restarted', 'reloaded']),
-            enable = dict(choices=BOOLEANS)
+            state = dict(choices=['running', 'started', 'stopped', 'restarted', 'reloaded']),
+            enabled = dict(choices=BOOLEANS)
         )
     )
 
     name = module.params['name']
     state = module.params['state']
-    enable = module.boolean(module.params.get('enable', None))
+    enable = module.boolean(module.params.get('enabled', None))
 
     # ===========================================
     # find binaries locations on minion
@@ -156,8 +156,10 @@ def main():
     err = ''
     out = ''
     
-    if enable:
+    if module.params['enabled']:
         rc_enable, out_enable, err_enable = _do_enable(name, enable)
+        if rc == 0:
+            changed = True
         rc += rc_enable
         out += out_enable
         err += err_enable
@@ -201,6 +203,10 @@ def main():
         module.fail_json(msg=err)
 
     result = {"changed": changed}
+    if module.params['enabled']:
+        result['enabled'] = module.params['enabled']
+    if state:
+        result['state'] = state
     rc, stdout, stderr = _run("%s %s status" % (SERVICE, name))
     module.exit_json(**result);