From e6f92bc3a937d8ca9593f0c973ff491779b24c93 Mon Sep 17 00:00:00 2001 From: Brian Coca Date: Tue, 31 Jan 2017 13:32:51 -0500 Subject: [PATCH] reworded conditional skipping to not imply error fixes #19068 --- lib/ansible/executor/task_executor.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/ansible/executor/task_executor.py b/lib/ansible/executor/task_executor.py index 7c710e6543b..2488fa74719 100644 --- a/lib/ansible/executor/task_executor.py +++ b/lib/ansible/executor/task_executor.py @@ -407,8 +407,8 @@ class TaskExecutor: # variable not being present which would otherwise cause validation to fail try: if not self._task.evaluate_conditional(templar, variables): - display.debug("when evaluation failed, skipping this task") - return dict(changed=False, skipped=True, skip_reason='Conditional check failed', _ansible_no_log=self._play_context.no_log) + display.debug("when evaluation is False, skipping this task") + return dict(changed=False, skipped=True, skip_reason='Conditional result was False', _ansible_no_log=self._play_context.no_log) except AnsibleError: # loop error takes precedence if self._loop_eval_error is not None: