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

have issue with check_http #283

Open
helmi1987 opened this issue Apr 5, 2022 · 9 comments
Open

have issue with check_http #283

helmi1987 opened this issue Apr 5, 2022 · 9 comments

Comments

@helmi1987
Copy link

Hello jjethwa

very cool docker image. is it possible to update the nagios plugins?
in the docker is the version from check_http with the version number "check_http v2.2 (monitoring-plugins 2.2)"

this version have not the command "--continue-after-certificate" inside and so we must make 2 checks for ssl and response.

in the newer version is it inside.

very much thanks from my site for your great work

greetings helmi1987

@helmi1987
Copy link
Author

https://nagios-plugins.org/doc/man/check_http.html
version 2.3.3 or newer

@jjethwa
Copy link
Owner

jjethwa commented Apr 5, 2022

Hi @helmi1987

Thanks so much, really appreciate it 😃

The image pulls the check packages from the standard repo:

icinga2/Dockerfile

Lines 73 to 77 in c0dfb83

monitoring-plugins \
nagios-nrpe-plugin \
nagios-plugins-contrib \
nagios-snmp-plugins \
libmonitoring-plugin-perl \

It looks like those are out of date. I can't find anything besides a source code download here: https://nagios-plugins.org/downloads/

I think it might be best to request an update to Debian package so everyone downstream like us can get it?

In the interim, you can include the updated checks via a volume. What do you think?

@jjethwa
Copy link
Owner

jjethwa commented Apr 5, 2022

HI @helmi1987

Did a bit more research on this and Icinga2 recommends using the checks from Monitoring Plugins: https://www.monitoring-plugins.org/ (reference: https://icinga.com/docs/icinga-2/latest/doc/02-installation/01-Debian/ )

Their check_http plugin does not have the functionality you are looking for unfortunately: https://www.monitoring-plugins.org/doc/man/check_http.html

I think requesting the functionality from them will be the way forward as we'll automatically get it when their package is updated: https://github.com/monitoring-plugins/monitoring-plugins/issues/new

@helmi1987
Copy link
Author

Hi @jjethwa

Thanks for all informations.
i have open an issue by monitoring-plugins.
(monitoring-plugins/monitoring-plugins#1761)

@jjethwa
Copy link
Owner

jjethwa commented Apr 6, 2022

Thanks so much, @helmi1987 I just subscribed to see the updates 😄

@helmi1987
Copy link
Author

Helo @jjethwa

can you make a dev docker container to check the new funtionality?

greetings @helmi1987

@jjethwa
Copy link
Owner

jjethwa commented Apr 12, 2022

Hi @helmi1987

Do you know if they released the updated package? I saw the commits, but could not tell about the package.

@helmi1987
Copy link
Author

helmi1987 commented Apr 12, 2022

Helo @jjethwa

in the news release was it not included.
i have make a test ( compile it insite the docker ;-))

apt update
apt upgrade
apt install git automake autoconf gettext gcc

#make new directory
git clone https://github.com/monitoring-plugins/monitoring-plugins.git
cd monitoring-plugins
./tools/setup
./configure -prefix=/user/local --with-cgiurl=/usr/lib/nagios/custom-plugins/test/plugins --with-openssl=/usr/bin/openssl 
make
make install
make install-root

after this steps was the plugin work.

i have ask when it will include in the monitoring-plugin package. i think we must wait for the next release

@jjethwa
Copy link
Owner

jjethwa commented Apr 12, 2022

That's great news, @helmi1987 Once they release the new package, I just need to trigger a new image build and we'll be good to go. Thanks so much for following up 👍

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