naparuba / check-linux-by-ssh

Shinken checks designed to be run without installing something on the distant linux (aside ssh keys of course!)
MIT License
43 stars 48 forks source link

commit broke check_ntp_sync_by_ssh on RHEL/CENTOS #17

Open Vye opened 10 years ago

Vye commented 10 years ago

https://github.com/naparuba/check-linux-by-ssh/commit/41543a8fe6e869cf53f7979fbe1ed1e0c904fd7b

The above commit breaks https://github.com/naparuba/check-linux-by-ssh/blob/master/check_ntp_sync_by_ssh.py out of the box for RHEL/CENTOS because /usr/sbin is not included in $PATH for shinken user by default.

What platform did you make this change for? I realize I can modify the check_command to include -n /usr/sbin/ntpq but out of the box operation is preferred.

naparuba commented 10 years ago

Can you give a test? It should works now :)

david-guenault commented 8 years ago

@Vye @naparuba Is it fixed and can we close this one ?