Skip to content
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

Holland monitor fails if root's my.cnf is changed #83

Open
rillip3 opened this issue Jul 10, 2015 · 2 comments
Open

Holland monitor fails if root's my.cnf is changed #83

rillip3 opened this issue Jul 10, 2015 · 2 comments

Comments

@rillip3
Copy link

rillip3 commented Jul 10, 2015

By default, /root/.my.cnf is used as the credentials for the mysql ping that Holland monitoring performs. This means if a user changes the root password, monitoring fails, even though Holland may still run without issue.

In discussing how this might best be fixed, the two ideas where to simplify the ping check (and not pass the --defaults-file at all, just the Holland user/pass) or to remove the check and use https://github.com/racker/rackspace-monitoring-agent-plugins-contrib/blob/master/mysql_ping.py to monitor MySQL in real time instead of trying to do everything in the one plugin.

@rphillips
Copy link
Contributor

@mckraken thoughts? I reverted #84 for now. Let's get some consensus before we merge it in.

@rillip3
Copy link
Author

rillip3 commented Jul 13, 2015

I had already posted my thoughts on #84 but I can repost them back here if we want to continue the discussion here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants