Icinga / ansible-playbooks

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

Added the Debian os Part for Web2 ui. #16

Closed tduemesnil closed 8 years ago

tduemesnil commented 8 years ago

Mainly a copy of the RedHat version which uses APT as Package Manager. Added a Modified Path for the SQL Schema files. Different Package for Debian

HTH Thomas

valentinogagliardi commented 8 years ago

Hello. Would you check out the conflicts and modify the PR accordingly? Thank you!

tduemesnil commented 8 years ago

Hello. I fixed the conflict in defaults and modified the var name to you new naming schema.

HTH

valentinogagliardi commented 8 years ago

Thank you, I'm going to test that and I'll merge the PR when done.

valentinogagliardi commented 8 years ago

I saw you placed login_user=root and login_pass={{ mysql_root_db_pass }} in icinga2-ansible-web2-ui/tasks/icinga2_web2_ui_Debian_install.yml. I think it'd better to omit that and let .my.cnf do the job as in the RH tasks. Starting on a fresh server, one could install MariaDB/Mysql, then install Icinga2 with UI and at the end runs mysql_secure_installation setting up the Mysql root password. Would you mind to remove login_user and login_pass? Apart from this the tasks works fine. Thank you very much.

tduemesnil commented 8 years ago

The mysql role (bennojoy.mysql) I use to setup the server immediately sets a root password. And creates a ~/.my.cnf file for the root user. Are the credentials use during the script ?

valentinogagliardi commented 8 years ago

When the .my.cnf is already in place, there is no need to pass the credential to the tasks, Ansible automatically use .my.cnf

tduemesnil commented 8 years ago

Always something to learn.