Reworking internal result flags and making sure include_vars hides vault data

Fixes #10194

1 files changed, 2 insertions, 6 deletions

diff --git a/lib/ansible/plugins/callback/skippy.py b/lib/ansible/plugins/callback/skippy.py
index a128d2ca1f..06ebc078c9 100644
--- a/lib/ansible/plugins/callback/skippy.py
+++ b/lib/ansible/plugins/callback/skippy.py
@@ -63,12 +63,8 @@ class CallbackModule(CallbackBase):
             msg = "ok: [%s]" % result._host.get_name()
             color = 'green'
 
-        if (self._display.verbosity > 0 or 'verbose_always' in result._result) and result._task.action not in ('setup', 'include'):
-            indent = None
-            if 'verbose_always' in result._result:
-                indent = 4
-                del result._result['verbose_always']
-            msg += " => %s" % self._dump_results(result._result, indent=indent)
+        if (self._display.verbosity > 0 or '_ansible_verbose_always' in result._result) and result._task.action not in ('setup', 'include'):
+            msg += " => %s" % self._dump_results(result._result)
 
         self._display.display(msg, color=color)
         self._handle_warnings(result._result)