Refactor task names for clarity and consistency across Icinga2 role
Hi icinga team, I've been trying to troubleshoot a deployment issue (only on my side) and ended up looking at your collection. I found some improvments to be performed, so i took the freedom to give it a try. Here you find my proposal.
In a nutshell, i have :
- moved all modules call to fqdn
- prefixed internal variables with
__icinga2to reduce its exposure to extern variable or any unwanted superseed - implemented the argument_specs for role variable validation
- done some linting
- rearranged the defaults/vars where applicable.
I carefully bear attention to not touch the business logic being executed.
Also, this would be the 1st stone to build automated documentation for antsibull and ansible-doc. (cf #321 )
So far, i've done it only on the icinga2 role. If this suits you, i could do further on the remaining roles.
Let me know your thoughts.
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL. This is most likely caused by a git client misconfiguration; please make sure to:
- check if your git client is configured with an email to sign commits
git config --list | grep email - If not, set it up using
git config --global user.email [email protected] - Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL. This is most likely caused by a git client misconfiguration; please make sure to:
- check if your git client is configured with an email to sign commits
git config --list | grep email - If not, set it up using
git config --global user.email [email protected] - Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL. This is most likely caused by a git client misconfiguration; please make sure to:
- check if your git client is configured with an email to sign commits
git config --list | grep email - If not, set it up using
git config --global user.email [email protected] - Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails
@bobapple you can retrigger the cla-bot here as well
@cla-bot check
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL. This is most likely caused by a git client misconfiguration; please make sure to:
- check if your git client is configured with an email to sign commits
git config --list | grep email - If not, set it up using
git config --global user.email [email protected] - Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails
Seems like there's a misconfiguration in your git client. If you follow the steps from cla-bot you should be able to fix it.
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL. This is most likely caused by a git client misconfiguration; please make sure to:
- check if your git client is configured with an email to sign commits
git config --list | grep email - If not, set it up using
git config --global user.email [email protected] - Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL. This is most likely caused by a git client misconfiguration; please make sure to:
- check if your git client is configured with an email to sign commits
git config --list | grep email - If not, set it up using
git config --global user.email [email protected] - Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL, lutin_malin. This is most likely caused by a git client misconfiguration; please make sure to:
- check if your git client is configured with an email to sign commits
git config --list | grep email - If not, set it up using
git config --global user.email [email protected] - Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL, lutin_malin. This is most likely caused by a git client misconfiguration; please make sure to:
- check if your git client is configured with an email to sign commits
git config --list | grep email - If not, set it up using
git config --global user.email [email protected] - Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL, lutin_malin. This is most likely caused by a git client misconfiguration; please make sure to:
- check if your git client is configured with an email to sign commits
git config --list | grep email - If not, set it up using
git config --global user.email [email protected] - Make sure that the git commit email is configured in your GitHub account settings, see https://github.com/settings/emails