Use failed_when instead of ignore_errors

ignore_errors still displays red on the screen and it looks like a
failed task. Makes it difficult to grep logs and visually inspect
output. failed_when does not do this, instead it just looks as if it
were a success, which is what we're after. The return json still marks
the task as failed so we can use that data elsewhere if desired.
ansible-lint: Add patched fork to requirements.txt
ansible-lint: service module not shell, ssh reload
fix nbd mispelling in /etc/modules
Merge pull request #617 from blueboxgroup/modules-nbd-fixup-for-master

fix nbd mispelling in /etc/modules
Merge pull request #611 from blueboxgroup/cleanup-ignore-errors

Use failed_when instead of ignore_errors
Merge pull request #614 from blueboxgroup/ansible-lint-fixes

Ansible lint fixes
Clean up nova-data role

names, whitespace, logic changes, failed_when.