From e7dd1eb0cbf0275a46ddd73391f846978df953eb Mon Sep 17 00:00:00 2001 From: Kevin Carter Date: Fri, 14 Oct 2016 16:46:14 -0500 Subject: [PATCH] Remove 'ignore_errors: true' in favor of 'failed_when: false' This change removes the use of 'ignore_errors: true' because it causes deployers to see red output and a stacktrace, which traditionally means something is broken, even when the failure is known to have a fall back option or be intentional. This conversion will provide a generally cleaner interface. It should be noted that the 'failed' filter will still function normally. Tasks with the 'failed_when: false' option will still be marked as 'failed' in any registered variable. This change simply makes the output look cleaner. Change-Id: Idb837672142c26d0198bf914abab900fcade7ccb Closes-Bug: #1633438 Signed-off-by: Kevin Carter --- tasks/swift_install.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tasks/swift_install.yml b/tasks/swift_install.yml index a652f24b..f55d6ab4 100644 --- a/tasks/swift_install.yml +++ b/tasks/swift_install.yml @@ -87,7 +87,7 @@ url: "{{ swift_venv_download_url }}" dest: "/var/cache/{{ swift_venv_download_url | basename }}" force: yes - ignore_errors: true + failed_when: false register: get_venv when: - not swift_pypy_enabled | bool