HeinleinSupport / check_mk_extensions

check_mk Plugins
https://www.heinlein-consulting.de/
GNU General Public License v2.0
232 stars 107 forks source link

Webinject enhancements #49

Closed robertoschwald closed 4 years ago

robertoschwald commented 5 years ago

I developed some enhancements for Webinject checks based on your extension.

https://github.com/robertoschwald/check_mk_plugins/tree/master/webinject/src

Enhancements:

Feel free to move it into your plugin. Just leave a note when you've done, so I can remove it from my repo. Thanks.

gurubert commented 5 years ago

check_webinject is already part of the official CheckMK package.

For the rest please create a pull request.

robertoschwald commented 5 years ago

Which Check_MK version includes check_webinject? I searched all files in the 1.5.0p15 docker image, but didn't find it.

gurubert commented 5 years ago
OMD[heinlein]:~$ ./lib/nagios/plugins/check_webinject --version
WebInject version 1.88
OMD[heinlein]:~$ cmk --version
This is Check_MK version 1.5.0p16 CEE
robertoschwald commented 5 years ago

Ah, ok, thanks. One version newer..

robertoschwald commented 5 years ago

Please note that 1.5.0p15 does include check_webinject, but does not work in the Docker Container, as the Check_MK Docker image is missing Error.pm. The version I provide for now conditionally loads the Error module. Need to check if it is still the case with the newer versions.

OMD[demo]:~$ lib/nagios/plugins/check_webinject --version Can't locate Error.pm in @INC (you may need to install the Error module) (@INC contains: /omd/sites/slave_bmw_ger/local/lib/perl5/lib/perl5 /omd/sites/slave_bmw_ger/lib/perl5/lib/perl5/x86_64-linux-gnu-thread-multi /omd/sites/slave_bmw_ger/lib/perl5/lib/perl5 /etc/perl /usr/local/lib/x86_64-linux-gnu/perl/5.24.1 /usr/local/share/perl/5.24.1 /usr/lib/x86_64-linux-gnu/perl5/5.24 /usr/share/perl5 /usr/lib/x86_64-linux-gnu/perl/5.24 /usr/share/perl/5.24 /usr/local/lib/site_perl /usr/lib/x86_64-linux-gnu/perl-base) at lib/nagios/plugins/check_webinject line 34. BEGIN failed--compilation aborted at lib/nagios/plugins/check_webinject line 34.

gurubert commented 5 years ago

Please report this as a bug against the checkmk Docker image.

symentis-info commented 4 years ago

Just for completeness: Error.pm bug is fixed in Docker version 1.6.0p5.cee