summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* glusterpmd: Change to NFS service nameSahina Bose2015-06-221-1/+1
| | | | | | | | | | | | Changed the NFS service name to Gluster NFS to correspond with the service name change in Nagios Change-Id: Ib088ee920b87eadf13b0e88f3e90147d3aad0e07 Bug-Url: https://bugzilla.redhat.com/1232730 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/11349 Reviewed-by: Ramesh N <rnachimu@redhat.com>
* nagios-plugins: Return status for glusterd service checkSahina Bose2015-06-111-6/+7
| | | | | | | | | | | | Corrected the nagios plugin return status based on the service return status Change-Id: I14e89645a0b2eacc016c9c5abb21a0bd0b85c9b3 Bug-Url: https://bugzilla.redhat.com/1177129 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/11161 Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Timothy Asir
* nagios-plugins: Corrected typo in split-brain status pluginSahina Bose2015-06-111-1/+1
| | | | | | | | | | | Corrected a typo Change-Id: I06369cbce0b7f965fe4993bea63c19b06963d105 Bug-Url: https://bugzilla.redhat.com/1228533 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/11163 Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
* plugins: nrpe plugin to check volumes with server quorumSahina Bose2015-05-316-0/+139
| | | | | | | | | | | | Added a plugin to check if any volume in the cluster has the server side quorum turned on Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1106421 Change-Id: I897aedd737a622832548e7470e8f3cf9fb7dbc2c Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/8016 Reviewed-by: darshan n <dnarayan@redhat.com> Reviewed-by: Ramesh N <rnachimu@redhat.com>
* Enhance plugin to display smb statusTimothy Asir Jeyasingh2015-05-141-0/+7
| | | | | | | | | | | | | | Plugin will be changed to look at chkconfig status of SMB. If turned off, plugin will report SMB service status as "Not Configured". Change-Id: I227b1215eb12066cf6367ce41388a133332e2be2 Signed-off-by: Timothy Asir Jeyasingh <tjeyasin@redhat.com> Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1178765 Signed-off-by: Timothy Asir Jeyasingh <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/10423 Tested-by: Timothy Asir Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* fix glusterd service status issueTimothy Asir Jeyasingh2015-05-061-2/+5
| | | | | | | | | | | | | | | | This patch fixes glusterd service status issue in nagios when glusterd is hung on the node. Currently it checks the status thru pid and return the status as 'running' when it finds any pid for glusterd process. Change-Id: Ia5045cae7ef1f28d3f8bb558039f47bcbe5d101b Signed-off-by: Timothy Asir Jeyasingh <tjeyasin@redhat.com> Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1177129 Signed-off-by: Timothy Asir Jeyasingh <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/10246 Tested-by: Timothy Asir Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* Enhance error message when glusterd not respondTimothy Asir Jeyasingh2015-05-061-4/+4
| | | | | | | | | | | | | | Enhance the message to suggest to user that issues may be with glusterd. Change the message "temporary error" into "Glusterd cannot be queried." Change-Id: I11ca09e03bebe87a083ac8e6a6d281f0172e2899 Signed-off-by: Timothy Asir Jeyasingh <tjeyasin@redhat.com> Bug-url: https://bugzilla.redhat.com/show_bug.cgi?id=1166602 Reviewed-on: http://review.gluster.org/10421 Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Tested-by: Timothy Asir
* plugins: Deduct cached space from used space in memory utilizationndarshan2015-04-283-19/+18
| | | | | | | | | | | | | | This patch modifies memory utilization plugin to retrun memory used after deducting cached space from it. As this cached space can be got back by applications whenever needed. So we can consider cached space as free. Change-Id: I6e6b7d0fc8148065d806c735606e4605741534d4 Bug-Url: https://bugzilla.redhat.com/1204314 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/9977 Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* nagios-plugins: Enhanced nagios self-heal pluginSahina Bose2015-03-251-6/+6
| | | | | | | | | | | | | Enhanced nagios self-heal plugin to report split-brain entries if found. Earlier this output was not reliable and hence was not used. With glusterfs 3.7, this issue has been fixed Change-Id: Icb79516383245839d45f4c9570c08993844fc5bd Bug-Url: https://bugzilla.redhat.com/1081900 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/9783 Reviewed-by: Ramesh N <rnachimu@redhat.com>
* nagios-plugins: Pep8 fixes in tests/test_mounts.pyndarshan2015-02-191-2/+2
| | | | | | | Change-Id: Ia1e2039294466f7f297d32cb2a9e3d1644bea313 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/9704 Reviewed-by: Sahina Bose <sabose@redhat.com>
* nagios-plugins: Monitor selected mount points.ndarshan2015-02-191-1/+1
| | | | | | | | | | | | | Disk utilization plugin will monitor only the following mount points: / , /boot , /home , /var , /usr. This is done to avoid duplication of monitoring brick mount points. Change-Id: I1e14c5bb18384492a3110f86a23c1102e3e9a3ac Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1135983 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/9673 Reviewed-by: Sahina Bose <sabose@redhat.com>
* nagios-plugins: Unit test and pep8 fix for check_mounts pluginndarshan2015-02-192-6/+6
| | | | | | | Change-Id: I2caf3515768e918446bfa68498ccad315d4ca300 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/9698 Reviewed-by: Sahina Bose <sabose@redhat.com>
* nagios-plugins: Disk utilization plugin revamp.ndarshan2015-02-199-634/+672
| | | | | | | | | | | | | | | | | This patch is aimed at revamping the existing check_disk_and_inode plugin in such a way that the plugin behaviour is same for both disk utilization service and brick utilization service. It has just two options include and exclude to include or exclude the mountpoints to be monitoried. This plugin monitors the data, inode, thinpool, thinpool-metadata utilization irrespective of service using this plugin. Change-Id: I98b18d6f3b22915978f8c6381adc99729a4e1161 Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1167771 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/9389 Reviewed-by: Bala FA <barumuga@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* fix unit test error for network.pyv1.0.0Timothy Asir2015-02-031-3/+3
| | | | | | | | | Change-Id: I9eae0c1857ab0a0c7c47123aeef8c39696f014a0 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/9570 Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Tested-by: Timothy Asir <tim.gluster@gmail.com>
* plugins/network: fix interface up statusBala.FA2015-02-031-1/+2
| | | | | | | | | | | | | This patch identifies interface's up status by additionally using IFF_RUNNING flag Change-Id: I26bd356065e9167196d09973aa378b00b589fcf2 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1110804 Signed-off-by: Bala.FA <barumuga@redhat.com> Reviewed-on: http://review.gluster.org/8111 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Timothy Asir <tim.gluster@gmail.com>
* plugin::check_disk_and_inode: fix backward compatibility in perfdatandarshan2014-11-261-1/+2
| | | | | | | | | | | | | | changed the label for brick utilization perfdata from "Virtual" to the brick mount path. This is done to retain backward compatibility. Change-Id: I45c5058d254ecb4ee23a28e093bed3ebf378db2f Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1143995 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/9202 Reviewed-by: Bala FA <barumuga@redhat.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-addons: Add thin lv details to check_disk_and_inode plugin.ndarshan2014-11-182-23/+105
| | | | | | | | | | | | | | | | check_disk_and_inode plugin is modified to show the underlying thin pool details when invoked with -t option. This is used to get the brick utilization. If the brick is made up of thin lv then it returns the virtual and actual utilization detail else the plugin returns the normal output. Change-Id: I2a2853a1b6e3e91c0178c3ed1cbc34de88ce3da2 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1143995 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/9115 Reviewed-by: Sahina Bose <sabose@redhat.com> Reviewed-by: Bala FA <barumuga@redhat.com>
* Fix bug:1119273 - invalid ctdb status message on ctdb service stopTimothy Asir2014-11-171-0/+8
| | | | | | | | | | | | | | | | This patch check for if CTDB process is not running and ctdb chkconfig status is on: then it will return "CTDB is not running", CRITICAL level and if the CTDB process is not running and the chkconfig status is off: then it will return "CTDB not configured", Unknown Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1119273 Change-Id: I13c586ceefc86978acd154f9c641a9cc0ffb443b Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/9124 Reviewed-by: Ramesh N <rnachimu@redhat.com> Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-addons:discover_volumes to return proper error in case of ↵Nishanth Thomas2014-11-172-6/+16
| | | | | | | | | | | | | | | | | | exception discoverVolumes was not returning the error message in case of exception while retrieving the data from the backend. Due this the calling funtions always expect a list and causing inconsistent results https://bugzilla.redhat.com/show_bug.cgi?id=1136205 https://bugzilla.redhat.com/show_bug.cgi?id=1109843 Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Change-Id: I518a5f94fd5d414028970379e268e0aa8a408285 Reviewed-on: http://review.gluster.org/9065 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Tested-by: Nishanth Thomas <nishusemail@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* plugins: Modified the quorum messagesSahina Bose2014-11-122-3/+4
| | | | | | | | | | Modified the quorum messages to be friendlier Change-Id: I8d76a094cc455847a76ece5c1744b8399f4e7e58 Bug-Url: https://bugzilla.redhat.com/1109702 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/9101 Reviewed-by: Ramesh N <rnachimu@redhat.com>
* plugins: Change the message for quorum pluginSahina Bose2014-11-112-4/+3
| | | | | | | | | | | | The message for Cluster-quorum monitoring contained the volume names too...removed this as it was retained even post volume deletion Bug-Url: https://bugzilla.redhat.com/1109702 Change-Id: Ibcb4cfcdb10041e83976e8672623a0140777a539 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/9095 Reviewed-by: Ramesh N <rnachimu@redhat.com>
* nagios-addons : fixing volcap compile issueNishanth Thomas2014-11-111-1/+0
| | | | | | | | Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Change-Id: I3e98bdc1653787cd7dc4826eab0fd42dc3118ba0 Reviewed-on: http://review.gluster.org/9067 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Sahina Bose <sabose@redhat.com>
* plugins: Changed the syslog filter pluginSahina Bose2014-10-281-42/+13
| | | | | | | | | | | | There was an error in the rsyslog omprog plugin where the last message was being buffered. This caused issues with the nagios service status being set incorrectly Change-Id: Icb5a0370fb732ad6e72b35f4d00cc867d39e20d4 Bug-Url: https://bugzilla.redhat.com/1141171 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/8740
* Plugins: Log message change in glusterpmdndarshan2014-10-201-1/+3
| | | | | | | | | | | | | | Changed the log message in check_proc_status.py from "Hostname not configured" to "'hostname_in_nagios' is not configured in /etc/nagios/nagios_server.conf" to make it more meaningful. Change-Id: Iacd0c6165bba97e87e0ab6a855a7314e4ac57102 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1120832 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/8920 Reviewed-by: Sahina Bose <sabose@redhat.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* plugins: Return unknown status when trans locked messageSahina Bose2014-09-231-6/+18
| | | | | | | | | | | | When the gluster command fails due to locking by another transaction, return UNKNOWN status, so that the command can be tried again on another node. Change-Id: Ibae111c0a95ddd3594a7082d3c36ffe9b15c41fc Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1109752 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/8440 Reviewed-by: Bala FA <barumuga@redhat.com>
* nagios-addons: Using ctypes to call gfapiRamesh Nachimuthu2014-08-148-196/+187
| | | | | | | | | | | | Removing the volCap.c which was used to call libgfapi and calling libgfapi directly from python using ctypes. Change-Id: Ia78f212c769c34e5cb991e429623946a4256e2af Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/8090 Reviewed-by: Sahina Bose <sabose@redhat.com> Reviewed-by: Bala FA <barumuga@redhat.com>
* glusterpmd: do not start the service by defaultKanagaraj M2014-07-181-1/+0
| | | | | | | | | | | | | | | user needs to start this service before start monitoring $ service glusterpmd start if the user wants the service to be started in the next reboot then $ chkconfig --add glusterpmd Change-Id: I91953c29e32dfa63b4bb71618654498b9bbcfff4 Bug-Url: https://bugzilla.redhat.com/1116368 Signed-off-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-on: http://review.gluster.org/8329 Reviewed-by: Sahina Bose <sabose@redhat.com>
* nagios-plugins: Disabled push notification for quota alertsSahina Bose2014-07-161-1/+1
| | | | | | | | | | | | | | | Disable the quota push notification using NSCA as this only supports soft-limit alerts and overrides the existing hard limit alerts. The poll based NRPE status check for quota is sufficient for now. Change-Id: I8203f27fa66c47b1c725819816e326cd6ee13331 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1119717 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/8314 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
* Add option to show critical and warning disks in the statusTimothy Asir2014-07-102-10/+24
| | | | | | | | | | | | | | | | | | Currently the critical status message shows the sum of all usage which is actually confusing the user when there are multiple disks available and any/few disk status moved to critical state. This patch adds option to show critical and warning disk details in the status message whenever there is any change in disk status to critical or warning. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1116710 Change-Id: I7a44f669a4a554aff748c2758bcf6a104365aed8 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/8273 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* nagios-addons: return correct error mesage when glusterd is stoppedRamesh Nachimuthu2014-07-101-2/+2
| | | | | | | | | | | | Fixing issue in error message returned for gluster related process when 'glusterd' is stopped. Change-Id: I8dd4595c45357ce7192db72ff3e8f2365e6fbefa Bug-Url: https://bugzilla.redhat.com/1105568 Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/8125 Reviewed-by: Humble Devassy Chirammal <humble.devassy@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* ckaging: Avoid overwriting config files during upgrade.Ramesh Nachimuthu2014-07-021-1/+1
| | | | | | | | | | | Adding %config(noreplace) flag for config files so that config files will not be overwritten during RPM upgarde. Change-Id: I9e78e9a8b109018a9db65853b55943c8ce9454ca Bug-Url: https://bugzilla.redhat.com/1107997 Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/8221 Reviewed-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-addons: Removed iptables modification.ndarshan2014-07-011-11/+0
| | | | | | | | | | | | | Removed addition of nrpe port to iptables and restarting of iptables from spec file as Its better to leave this task to sys admin. Change-Id: I3e86c1af6d5aefd7fed99a00c02ce7d5534e83ce Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1110719 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/8212 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* Fix pep8 errorsv0.1.1Timothy Asir Jeyasingh2014-06-267-19/+19
| | | | | | | | Change-Id: I0d41725a02636228bc613b40c1b4d2d57a35e3e3 Signed-off-by: Timothy Asir Jeyasingh <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/8159 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
* fix brick utilization graph errorTimothy Asir Jeyasingh2014-06-251-1/+1
| | | | | | | | | | | | | | Remove unit type from disk utilization plugin perf data which is not allowed by pnp4nagios which causes failure in creating RRD file. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1112966 Change-Id: I508f724da5a7896e310699180f319c0195378ca4 Signed-off-by: Timothy Asir Jeyasingh <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/8170 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-addons: Volume utilization returns negative values when size ↵Nishanth Thomas2014-06-202-20/+14
| | | | | | | | | | | | | | | | in TB data overflow error in volcap.c. statvfs returns unsigned long which is boxed into int causing the error Change-Id: I85a1e0d584bf4d9e73b2bfebebea83cd9fbddf68 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1111558 Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Reviewed-on: http://review.gluster.org/8136 Tested-by: Nishanth Thomas <nishusemail@gmail.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Bala FA <barumuga@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* rpm: fix service restart on postBala.FA2014-06-201-4/+9
| | | | | | | | | | | | | Currently iptables and other services are restarted without checking current state. This patch fixes this issue by doing conditional restart. Change-Id: I6149af00aae7ba7acda9e0f5f6295b9f78363631 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1110719 Signed-off-by: Bala.FA <barumuga@redhat.com> Reviewed-on: http://review.gluster.org/8112 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Humble Devassy Chirammal <humble.devassy@gmail.com>
* Add status and condrestart options to glusterpmd serviceTimothy Asir2014-06-201-8/+23
| | | | | | | | | | | Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1110719 Change-Id: I28af8b335af7dd9cc0ea58b98923261f95fc98dc Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/8021 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Bala FA <barumuga@redhat.com>
* Remove python-psutil dependencyTimothy Asir2014-06-162-13/+25
| | | | | | | | | Change-Id: I6267242de480f4c7a572eeac7a74472bf7ab9226 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1103759 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/8037 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* glusterpmd: fix issue in CTDB statusRamesh Nachimuthu2014-06-111-1/+2
| | | | | | | | | | | Fixing issue in CTDB status. SMB and NFS status codes were not passed properly to the check_proc_util.getCtdbStatus() method. Bug-Url: https://bugzilla.redhat.com/1107977 Change-Id: I4bb7f61488e83fb1fbd6d0b4369b782ada37af7c Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/8033 Reviewed-by: Sahina Bose <sabose@redhat.com>
* build: fixing make file issue in volcapRamesh Nachimuthu2014-06-101-6/+4
| | | | | | | | | | | | Not distributing capacity.so in src rpms. Bug-Url: https://bugzilla.redhat.com/1104183 Change-Id: I9eca10f9aa1fea290026a7f3f3220ff96142e5d8 Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/8026 Reviewed-by: Bala FA <barumuga@redhat.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* Add status information to show disk usageTimothy Asir2014-06-104-125/+222
| | | | | | | | | | | | | | | | | | | | | | This will show the status like: for critical usage: CRITICAL: 4% used (4GB out of 100GB):mounts: (CRITICAL : <critical list> followed by WARNING if any followed by OK for disk status "WARNING": WARNING: 4% used (4GB out of 100GB):mounts: (WARNING if any followed by OK for disk status "OK": OK: 4% used (4GB out of 100GB):mounts:(<mounts>) Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1081495 Change-Id: I9dbda7a5d6ea992ba73acce2174e6d66f2e16066 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7936 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* plugins: Handle geo-rep plugin output for no session infoSahina Bose2014-06-061-0/+2
| | | | | | | | | | | When there are no active geo-replication sessions, the plugin is changed to return output accordingly Change-Id: Icbc4ec061cc7a1e948914e284531531b00e9ec4b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1104014 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/8000 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* plugins: fix pep8 error in check_disk_inodeKanagaraj M2014-06-051-2/+2
| | | | | | | | | | Fix pep8 issue. Change-Id: Ib4cb81e74f2fa05e68be71e6c33b58ab64713eac Bug-Url: https://bugzilla.redhat.com/1083849 Signed-off-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-on: http://review.gluster.org/7990 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
* plugins: ignore offline volumes in smb/quotad monitoringKanagaraj M2014-06-051-5/+9
| | | | | | | | | | Volumes which are not ONLINE will be ignored. Change-Id: I36eade96be36b39f436852b1a9570c27965d29a0 Bug-Url: https://bugzilla.redhat.com/1104490 Signed-off-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-on: http://review.gluster.org/7968 Reviewed-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-addons:fixed testcase failuresNishanth Thomas2014-06-052-3/+5
| | | | | | | | | | Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1096159 Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1096169 Change-Id: I41d320ab63ec47bbd9b8b413cc7dffed68470d82 Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Reviewed-on: http://review.gluster.org/7974 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Nishanth Thomas <nishusemail@gmail.com>
* Add sudo configuration for nrpe users for specific pathTimothy Asir2014-06-054-0/+20
| | | | | | | | Change-Id: Ib0a25a13423fa3020f55bfddc831e2b653d4c9ea Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7808 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-addons: Fixed /dev/tty issue during rebootShubhendu Tripathi2014-06-041-1/+1
| | | | | | | | | | | | | | While the RHS node is getting booted, the /dev/tty is not yet set so setting stdout_path to /dev/tty fails. As a result start of the check_proc_status daemon fails. Fixed to set stdout_path to /dev/null to resolve the same. Change-Id: I2d09d373fcf6d0b21bda3f35b4cf4ee83594fcb8 Bug-URL: https://bugzilla.redhat.com/show_bug.cgi?id=1103268 Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-on: http://review.gluster.org/7962 Reviewed-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-addons:volume status based on volume typeNishanth Thomas2014-06-042-3/+7
| | | | | | | | | | | | | | | Added the volume type in the plugin output The logic of determining the volume status changed based on the volume type. Added the volume type and the list of bricks down in the plugin output Change-Id: Ib999fc78059c725a29f4f7bc3b3328c34b6a6b33 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1096159 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1096169 Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Reviewed-on: http://review.gluster.org/7878 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-addons:Added more checks to provide details in plugin outputNishanth Thomas2014-06-041-4/+19
| | | | | | | | | | | | | | | | check_proc_status.py - when a volume is down, brick status was always showing up unless initiate a pre-scheduled check. Removed an unnecessary check to fix this issue check_vol_utilization.py - modified the volume utilization plugin so that the plugin ouput is meaninful when a volume is nonexistent or stopped Change-Id: I04b8d8fc5ae19b2ae9f4969a7239a9449f2acd0f Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1083451 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1090854 Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Reviewed-on: http://review.gluster.org/7959 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Sahina Bose <sabose@redhat.com>
* Send brick path not found error when a brick path not found.Timothy Asir2014-06-041-2/+14
| | | | | | | | | | | Change-Id: Ib8d40b862d0102d352af472c7862e79ac0e085d8 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1083849 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7930 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>