-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor task names for clarity and consistency across Icinga2 role #364
base: main
Are you sure you want to change the base?
Conversation
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL.
|
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL.
|
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Guillaume FAUVEL.
|
@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.
|
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. |
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 :
__icinga2
to reduce its exposure to extern variable or any unwanted superseedI carefully bear attention to not touch the business logic being executed.
Also, this would be the 1st stone to build automated documentation for
antsibull
andansible-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.