add in the missing --warning and --critical options to check_mysql_health
fix check_mysql_health real_check_host variable
fix the --warning --critical by appending it as optional ending arguments in the command specifier
it might be that having ! when there are no arguments will cause nagios to be unhappy, so lets only put them in when we have arguments to pass
Update check_mysql_health plugin, changes:

. fixed a division by zero bug in index-usage (Thanks Wiltmut Gerdes)
. fixed a severe bug when loading dynamic extensions (Thanks Ralph Schneider)
. added mode table-fragmentation
. fixed a bug in table-lock-contention (thanks mayukmok00)
. mode sql can now have a non-numerical output which is compared to a string/regexp
. new parameter –dbthresholds
. new mode report can be used to output only the bad news (short,long,html)
. added modes threads-created, threads-running, threads-cached
. added connects-aborted, clients-aborted
Merge commit 'f2df62c9d17d481a3d616a4f2de9496638fadc0a'

Conflicts:
	manifests/defaults/commands.pp
Fix nagios check_smtp duplicate definition

On Debian, the nagios-plugins packages install
/etc/nagios-plugins/config/mail.cfg which contains the check_smtp/check_imap
commands, and these commands cannot be duplicated. The addition of these
commands in this module in c9c4f5bb895e98e31d84c751a7fcbdae9bf57008 and
1c4de48baf4ffcf5f39894a101a3c59479959b47 now cause Debian nagios servers to
fail to start with this message:

Warning: Duplicate definition found for command 'check_smtp' (config file '/etc/starting on line 9)

The versions of these check commands that were added in the above commits are
actually more full-featured than the ones that Debian ships, most notably
because they allow for the port to be passed as an option. In most cases their
names are the same, although the imap and pop ssl commands are different. This
leads to a somewhat unfortunate situation where the module defines duplicated
commands, which are better, but in some cases are differently named than the
ones shipped in the package. Ideally, those names would converge, and the
debian package commands would add the port options by default, and we'd case
out debian/ubuntu to not ship these in this module.

However, due to the added benefit of the options provided in the commands
shipped in this module, along with the benefits that come with the module
being able to automatically configure these, I've elected to simply remove
the debian package file (which is a config file, so it will be honored
by the packaging system).
add imap and pop service definitions
fix typo in imap/pop services
add check_git command to debian/ubuntu as well
fix check_health_mode parameter
add the potential for the --name to be set for arbitrary sql commands
fix formatting to be standard
add configuration variable to irc bot to enable IRC notice type messages which can be turned on by the puppet variable $nagios_nsa_usenotices
fix syntax error from previous commit
fix the usenotices template variable
Merge branch 'master', remote branch 'origin'
Ensure commands.cfg exists

The file commands.cfg was introduced at 61aab91919b2e6d9cb5d7afe3befe40de67ac132.
This change make sure the file exists so nagios doesn't stop running.
Debian doesn't need explicit check_smtp definition

This plugin is already defined at nagios-plugins, so removing it
for the same reason as e7f85f647e6f641bd85fbdfdb26982350f782370.
Debian doesn't need explicit check_smtp definition (2)

This plugin is already defined at nagios-plugins, so removing it
for the same reason as e7f85f647e6f641bd85fbdfdb26982350f782370.
This commit has more readable implementation than the previous.