search

pythian / opsviz

25 stars 24 forks source link

Adds dependencies for clients subscribing to mysql #38

Closed dtest closed 9 years ago

dtest commented 9 years ago

This adds the 'mysql2' and 'inifile' dependencies, in addition to a /opt/sensu/.my.cnf file to avoid passing user/password in plaintext.

I don't create the sensu user here, as this requires access to a privileged user, which is outside the scope of the client installation.

alexlovelltroy commented 9 years ago

lgtm @taylorludwig what do you think?

taylorludwig commented 9 years ago

It looks great to me as well, nice job. I like the concept with the conf file, we could do something similar in other places to get the passwords out of the sensu command.

dtest commented 9 years ago

Thanks! But I can't take credit for the conf file :)