| Age | Commit message (Collapse) | Author |
|---|
|
Currently the nagios::apache class manages some files for debian. This
means that when using the main class for setting up nagios with apache
(which is the default!) lacks those files and the setup doesn't work.
Split out management of those files into another class to avoid making
the code in the main class too encumbered and make the main class
include that (instead of nagios::apache). nagios::apache will include
the main class anyway.
Adjust README file so that all cases of http management are done through
the "nagios" class.
|
|
remove non-ascii char in comment
puppet 3.x refuses to run when there are non-ASCII chars in comments
(wth!). so replace that char with an ASCII representation of it.
See merge request !11
|
|
puppet 3.x refuses to run when there are non-ASCII chars in comments
(wth!). so replace that char with an ASCII representation of it.
|
|
require package before config files
In the current state of the module, almost 100% of the time nagios
doesn't install correctly since ordering of the resources is not
enforced.
This is a major issue since the module is currently not usable in its current state because of the problem it creates.
See merge request !5
|
|
In the current state of the module, almost 100% of the time nagios
doesn't install correctly since ordering of the resources is not
enforced.
|
|
stop defining an alias by default for hosts
Since the "alias" attribute is special for puppet in that it needs to be
a string that's unique throughout all of the resources on a given host's
catalogue, we shouldn't always define an alias unless users really want
them.
This alias might (and did for some ppl) create a conflict when using a
class that is named the same as $::hostname.
We've just hit hit limitation on a Koumbit node. The alias attribute is legitimate in terms of nagios host attributes, but the name of the attribute clashes with the special-purpose attribute for puppet. So one might argue that the native puppet types are flawed, and they would be correct! but getting those types fixed would be a pain in the ass since they haven't been maintained for quite some time.
See merge request !10
|
|
added templates/nrpe/nrpe_command.erb
commit:7fb80fbe uses nagios::nrpe::command { "check_load": ...}, but the
template for it is missing.
amended by LeLutin: use @ prefix on variables in template
This is a followup of: https://labs.riseup.net/code/issues/4100
See merge request !8
|
|
Plugin path
Avoid using lib64 in the path for x86_64 architecture when using debian. this path is not available anymore starting with debian wheezy.
Then lint the file to make it easier to read / maintain.
This is a followup of https://labs.riseup.net/code/issues/4237
See merge request !7
|
|
Ordering in example
The usage example in the readme has a couple of small and bigger problems. This branch makes the example more accurate.
See merge request !6
|
|
Only use the private directory on CentOS
Debian and ubuntu have their resource.cfg file placed in /etc/nagios3/
and when it's absent it can cause issues with the package (because the
package is badly done).
This is a followup to issue https://labs.riseup.net/code/issues/4240
See merge request !4
|
|
Since the "alias" attribute is special for puppet in that it needs to be
a string that's unique throughout all of the resources on a given host's
catalogue, we shouldn't always define an alias unless users really want
them.
This alias might (and did for some ppl) create a conflict when using a
class that is named the same as $::hostname.
|
|
Unquote booleans
Some variables and if blocks use quoted booleans when we should be using the native boolean type.
Also, when working on this, I saw an issue with a mistyped variable in nagios::service so I fixed it.
This is a followup of: https://labs.riseup.net/code/issues/4093
See merge request !9
|
|
This invalidates the point of that if block since it will probably never
be true that some undefined (read mis-typed) variable is set to
'absent'.
|
|
|
|
commit:7fb80fbe uses nagios::nrpe::command { "check_load": ...}, but the
template for it is missing.
amended by LeLutin: use @ prefix on variables in template
|
|
This should make it easier to read the code.
|
|
|
|
The example doesn't set any ordering on the classes and this leads to
errors at first install since all of the configuration files that the
nagios::defaults class installs need the nagios package to be installed
first in order for the config dirs to exist.
|
|
Example uses weird spacing in the beginning of the lines.
Also the example usage of nagios::command is broken: it lacks the
closing curly brace.
|
|
Debian and ubuntu have their resource.cfg file placed in /etc/nagios3/
and when it's absent it can cause issues with the package (because the
package is badly done).
|
|
|
|
defined by the nagios package
amended by LeLutin: added missing : after default keyword and fixed
style a bit.
|
|
Remove unused 'mountpoints' fact. Closes #1
See merge request !3
|
|
|
|
|
|
Add missing DirectoryIndex index.php to the pnp4nagios apache config snippet
Without this you get a 404, and have to explicitly specify index.php to load it
See merge request !2
|
|
|
|
Don't screw up ENV["PATH"] for other platforms
Minor bugfix to prevent overriding ENV["PATH"] for non-Linux platforms.
See merge request !1
|
|
|
|
|
|
|
|
debian/ubuntu commands.
|
|
|
|
the basic path for nagiosstats is wrong/hardcoded.
that binary comes in with the nagios3 package in Debian. set the default
value to the right path under Debian and make it possible to override
the path with an environment variable.
Signed-off-by: Gabriel Filion <lelutin@gmail.com>
|
|
|
|
that old baggage around
|
|
add_pnp4nagios
Conflicts:
README.pnp4nagios
manifests/defaults/pnp4nagios.pp
manifests/pnp4nagios.pp
manifests/pnp4nagios/popup.pp
|
|
remove line that says that there are no packages (there are now)
|
|
This reverts commit 2f8d046607ad0bd5555ef812880cd97431b24977.
This was never used in any resource
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
