From c24f6f438ba95299f80e2c40a305e6f4ad698f97 Mon Sep 17 00:00:00 2001 From: Kavin Kankeshwar Date: Fri, 6 Sep 2013 22:37:17 -0700 Subject: [PATCH] Avoid ugly formatting due to a long if statement --- lib/ansible/playbook/__init__.py | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/lib/ansible/playbook/__init__.py b/lib/ansible/playbook/__init__.py index 0ad9a65c0ce..6700e4c3c97 100644 --- a/lib/ansible/playbook/__init__.py +++ b/lib/ansible/playbook/__init__.py @@ -573,11 +573,12 @@ class PlayBook(object): host_list = self._list_available_hosts(play.hosts) - # If threshold for max nodes failed is exceeded or if any_errors_fatal is true, bail out. - if ( - (task.any_errors_fatal and len(host_list) < hosts_count) or - ((hosts_count - len(host_list)) > int((play.max_fail_pct)/100.0 * hosts_count)) - ): + # Set max_fail_pct to 0, So if any hosts fails, bail out + if task.any_errors_fatal and len(host_list) < hosts_count: + play.max_fail_pct = 0 + + # If threshold for max nodes failed is exceeded , bail out. + if (hosts_count - len(host_list)) > int((play.max_fail_pct)/100.0 * hosts_count): host_list = None # if no hosts remain, drop out