Remove 'ignore_errors: true' in favor of 'failed_when: false'
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: I34f49f829a61e25393fcec6b047aa28858c8eeb6
Closes-Bug: #1633438
Signed-off-by: Kevin Carter <ffb54fded2f22052678e45d78b58d223654eb6fa@rackspace.com>

openstack/openstack-ansible-rabbitmq_server,openstack/openstack-ansible-rabbitmq_server