Commit graph

12 commits

Author SHA1 Message Date
mh
c1c3b11245 make trocla an optional dependency 2013-01-02 18:48:49 +01:00
mh
33241a38d2 linting 2013-01-02 17:13:45 +01:00
mh
dd6c24844b Merge remote-tracking branch 'shared/master'
Conflicts:
	files/config/my.cnf.Debian
	manifests/server/base.pp
	manifests/server/munin/default.pp
	manifests/server/nagios.pp
2013-01-02 17:03:18 +01:00
mh
956edf58cc take hiera out of the params 2012-06-14 14:53:44 -03:00
Gabriel Filion
64bbe64b96 Use the right parameter to nagios::service::mysql
check_hostname is not defined. We need to use check_host instead
2012-02-23 04:52:55 -05:00
mh
4ac7369515 migrate everything to hiera/trocla and use the new scoping style 2012-02-10 14:35:19 +01:00
Micah Anderson
fa67257056 . remove the check_mysql_health script from the mysql module, instead it belongs in the nagios module
. change the default nagios::service::mysql check to use the check_mysql_health 'connection-time' check mode, which is identical to the original check, with some additional information

. stop using nagios::plugin::deploy because this doesn't work when more than one node attempts to realize this class

. stop exporting the nagios_command because this doesn't work when more than one node attempts to realize this class

. remove the check_health define, instead this be how it was before, as the previous nagios::service::mysql define
2010-10-18 20:52:42 -04:00
Micah Anderson
f52a66ea0e change the default $check_hostname variable to be the localhost interface. This seems to make more sense than using $fqdn 2010-10-18 15:50:26 -04:00
Micah Anderson
6031165200 simplify the nagios check_mysql_health plugin process:
1. use the new plugin deploy feature in nagios (nagios::plugin::deploy)
2. remove unnecessary classes and inheritance - this plugin seems reasonable to install by default, and in fact it could be argued that the other 'check_mysql' plugin that still remains can be removed, as its functionality is vastly overshadowed by this one
3. add the 'repl_client_priv' mysql grant privs to the nagios user. these are needed for the check_mysql_health plugin slave replication modes. According to http://dev.mysql.com/doc/refman/5.0/en/privileges-provided.html#priv_replication-client - The REPLICATION CLIENT privilege enables the use of SHOW MASTER STATUS and SHOW SLAVE STATUS. These privileges are not too much to provide to the nagios user, as they are only informational
4. setup the define "check_health" so it can be used easily
2010-10-17 18:23:42 -04:00
Micah Anderson
784bd6271f add a more advanced mysql health check plugin:
* create a mysql::server::nagios::base class with the common parts needed for the basic plugin, and the health plugin
 * make mysql::server:nagios inherit mysql::server:nagios::base
 * create a new class mysql::server::nagios::check_health inheriting ::base

the nagios module has also received a new define to setup the different nagios::service pieces for the different health check modes that might be desired

its assumed you would setup the different health check modes in site-mysql/init.pp as different hosts will require different modes and/or parameters, for example:

class site-mysql::server {
  include mysql::server::nagios::check_health

  nagios::service::mysql_health { [ 'connection-time', 'uptime', 'threads-connected', 'threadcache-hitrate' ]:
    require => Mysql_grant[$nagios_mysql_user],
  }

  case $hostname {
    "eider": {
      nagios::service::mysql_health { [ 'slave-io-running', 'slave-sql-running', 'slave-lag' ]:
    require => Mysql_grant[$nagios_mysql_user],
      }
    }
  }
}
2010-09-27 13:18:51 -04:00
Jerome Charaoui
bd305f66b2 adjust nagios mysql user to allow tcp connections 2009-12-22 11:41:54 -05:00
Jerome Charaoui
b18a7ab8df add nagios class to create mysql user and grant for nagios, with option to skip nagios service registration if check is done through check_ssh or check_nrpe for example 2009-12-11 15:25:20 -05:00