From 7849c7d8e39010a6a77e738a5110aea3adbd8c37 Mon Sep 17 00:00:00 2001 From: Kevin Carter Date: Fri, 14 Oct 2016 16:45:52 -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: I0e6b5fc5b10e379485cec5c58acca4b315d4a6c4 Closes-Bug: #1633438 Signed-off-by: Kevin Carter --- tasks/neutron_install.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tasks/neutron_install.yml b/tasks/neutron_install.yml index fe375eed..4bab84a4 100644 --- a/tasks/neutron_install.yml +++ b/tasks/neutron_install.yml @@ -87,7 +87,7 @@ url: "{{ neutron_venv_download_url }}" dest: "/var/cache/{{ neutron_venv_download_url | basename }}" force: yes - ignore_errors: true + failed_when: false register: get_venv when: - not neutron_developer_mode | bool