finer grained failed/skipped on async for actions

fixes #23729

(cherry picked from commit 3965689328)

lib/ansible/plugins/action/__init__.py

@@ -90,17 +90,17 @@ class ActionBase(with_metaclass(ABCMeta, object)):
         * Module parameters.  These are stored in self._task.args
         """
 
-        result = {'skipped': True}
+        result = {}
 
         if self._task.async and not self._supports_async:
             result['msg'] = 'async is not supported for this task.'
+            result['failed'] = True
         elif self._play_context.check_mode and not self._supports_check_mode:
             result['msg'] = 'check mode is not supported for this task.'
+            result['skipped'] = True
         elif self._task.async and self._play_context.check_mode:
             result['msg'] = 'check mode and async cannot be used on same task.'
-        else:
-            # we run!
-            del result['skipped']
+            result['failed'] = True
 
         return result
 

lib/ansible/plugins/action/add_host.py

@@ -46,7 +46,7 @@ class ActionModule(ActionBase):
 
         result = super(ActionModule, self).run(tmp, task_vars)
 
-        if result.get('skipped', False):
+        if result.get('skipped', False) or result.get('failed', False):
             return result
 
         # Parse out any hostname:port patterns

lib/ansible/plugins/action/assemble.py

@@ -84,7 +84,7 @@ class ActionModule(ActionBase):
 
         result = super(ActionModule, self).run(tmp, task_vars)
 
-        if result.get('skipped', False):
+        if result.get('skipped', False) or result.get('failed', False):
             return result
 
         if task_vars is None:

lib/ansible/plugins/action/copy.py

@@ -40,7 +40,7 @@ class ActionModule(ActionBase):
 
         result = super(ActionModule, self).run(tmp, task_vars)
 
-        if result.get('skipped'):
+        if result.get('skipped', False) or result.get('failed', False):
             return result
 
         source  = self._task.args.get('src', None)

lib/ansible/plugins/action/package.py

@@ -38,7 +38,7 @@ class ActionModule(ActionBase):
 
         result = super(ActionModule, self).run(tmp, task_vars)
 
-        if result.get('skipped', False):
+        if result.get('skipped', False) or result.get('failed', False):
             return result
 
         module = self._task.args.get('use', 'auto')

lib/ansible/plugins/action/service.py

@@ -37,7 +37,7 @@ class ActionModule(ActionBase):
 
         result = super(ActionModule, self).run(tmp, task_vars)
 
-        if result.get('skipped', False):
+        if result.get('skipped', False) or result.get('failed', False):
             return result
 
         module = self._task.args.get('use', 'auto').lower()