Probably the most dangerous part of the cleanup, replace call to the
shell module that don't seems to make use of shell features with the
command module.
Comparison with litteral True and False ( `var == False` ⇔ `not var` or
`var is boolean and var` or ` var is sameas False` should there be a
need to check if it is really a boolean that is False and not only
something evaluationg to False)
Name all tasks.
It would seems that the task creating the `fail2ban_ignore_ips` fact for
the whitelist exist in both main.yml and ip_whitelist.yml, one of them
might be superfluous.
Remove the useless call for `cat` as `grep` take a file as it's second
argument that tell it to search for the pattern in a specific file
instead of whatever is passed to it through stdin.
Name both task defining the position of the config directory for
self-documentation purposes (and please ansible-lint defaults).
Change role description to it's description in the readme instead of the
default description asking for the role to be described.
Don't compare to empty string, an empty string is already false while a
non-empty string is true.
Mathieu Trossevin
