From 14df603af349e90f240ed4429bf1638ab26cc877 Mon Sep 17 00:00:00 2001 From: Kevin Carter Date: Fri, 14 Oct 2016 16:45:43 -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: Ib906fa3a67a8d70174da10608921a1a213dfd739 Closes-Bug: #1633438 Signed-off-by: Kevin Carter --- tasks/install.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tasks/install.yml b/tasks/install.yml index 671a8fd..69185b8 100644 --- a/tasks/install.yml +++ b/tasks/install.yml @@ -85,7 +85,7 @@ url: "{{ magnum_venv_download_url }}" dest: "/var/cache/{{ magnum_venv_download_url | basename }}" force: yes - ignore_errors: true + failed_when: false register: get_venv when: - not magnum_developer_mode | bool