search

Icinga / ansible-playbooks

Ansible Playbook for Icinga 2
GNU General Public License v2.0
225 stars 122 forks source link

Enhancing the upstream role for our needs. #26

Closed peterjenkins1 closed 6 years ago

aflatto commented 7 years ago

Thanks for your contribution. Merging this change pending testing .

aflatto commented 7 years ago

The Director is not a component that should be "pushed" to every install, I believe that IcingaWeb2 modules install should become a separate playbook. Please rebase your changes and separate the Web2 modules to a new playbook.

peterjenkins1 commented 7 years ago

Thanks for your feedback, I agree that it could be a lot cleaner and separated by function. This was really only a prototype implementation. I'm no longer working with Icinga2 so I won't have time to make further improvements. I suggest you close this PR and maybe some day someone might use this code as a starting point for a better set of roles.