Actions
Review #570
closedPlease review Fail2ban role
Start date:
2017-06-25
Branch:
Description
ssh://vcs-git.duckcorp.org/srv/vcs/git/duckcorp/ansible-role-fail2ban
Actions
Added by Marc Dequènes over 8 years ago. Updated about 8 years ago.
Description
ssh://vcs-git.duckcorp.org/srv/vcs/git/duckcorp/ansible-role-fail2ban
Some task files (_install.yml) are prefixed with an underscore, why ?
Content of _install.yml could be moved to a block in main.yml.
In tasks/detect_config.yml, an item matching distribution name and distribution major version could be added to with_first_found loop.
In tasks/_detect_daemon.yml, a block could be used for the last two tasks: both tasks are using the same notify entry.
Add ansible_managed at the beginning of templates/common.conf.
Add a comment at the beginning of files located in files/jail.d, something like:
managed by ansible
I'm using underscores to mark the task file as not being an entrypoint. It's the name I use for files part of the public API when using include_role + tasks_from. There is such kind of convention in languages where there is no private scope for method for example.
I chose readability over factorization, but implemented all other changes.
Thanks for the review.