summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* 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>
* gluster-nagios-addons:when volume is down brick also will be downNishanth Thomas2014-06-021-4/+6
| | | | | | | | | | | | | | modified the check_proc_status plugin to make the brick to DOWN when volume is DOWN Change-Id: Iafc8294d542753de23867baefeed3db6cc8d0663 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/7801 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Tested-by: Nishanth Thomas <nishusemail@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* resend gluster proc status to nagios on any status message changeTimothy Asir2014-05-291-25/+27
| | | | | | | | | Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1096620 Change-Id: I8b7b3f11426c2c6841165cb05740cd9bbfc24195 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7774 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* procstat: fix status information to show appropriate messages.Timothy Asir2014-05-291-6/+9
| | | | | | | | | | | | This patch fixes status information to show appropriate message when the process is already running. Change-Id: I82e8cb76bdbc3b2e6ee7f6e3f15fc3633a3792bc Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1089705 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7826 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* Add performance data to have unit type for brick and disk usage.Timothy Asir2014-05-291-2/+2
| | | | | | | | | Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1081375 Change-Id: I5f3f97275c6a4d8ad3a28c8d7add0a2ed3253283 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7912 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
* plugins: Geo-rep plugin status message enhancedSahina Bose2014-05-282-16/+40
| | | | | | | | | | | Added geo-rep session wise details to the plugin output. Change-Id: I2086a493c2befa4c1e25a1cc5523213f0f13c8dd Bug-Url: https://bugzilla.redhat.com/1101218 Bug-Url: https://bugzilla.redhat.com/1101223 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7896
* Fix pep8 line too long error in check_gluster_proc_status.py file.Timothy Asir2014-05-281-1/+2
| | | | | | | | Change-Id: I0cab5c2909240ec4c04bd302869bd4b0a62854f5 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7898 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* gluster-nagios-addons: Fix for sadf plugins going to unknown state.ndarshan2014-05-271-0/+3
| | | | | | | | | | | | | | | | This patch fixes the issue of sadf plugins going to unknown state giving xml error. This issue occurs during the day's transition due to sadf returning output without any records. This patch adds a job to collect data two times during the last minute of the day, because of which we get a record for time 00:00:01, which was not got earlier. Change-Id: Ibcecfe22215bdf4e41a5e9fed329a7a99320bedc Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1081953 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/7802 Reviewed-by: Bala FA <barumuga@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* plugins: Fixed repeated warn/crit words in disk util pluginSahina Bose2014-05-261-21/+22
| | | | | | | | | | The disk utilization plugin output had repetition of words - Warn/Crit. Fixed this Change-Id: Idf5d657fd373189a1eb40c1f0d1b45ba4dd49e7b Bug-Url: https://bugzilla.redhat.com/1084421 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7869
* brickstat: fix brick status message to have brick details.Timothy Asir2014-05-261-1/+1
| | | | | | | | | | | | | | | Add brick details to the status message when the brick status is unknown. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1097231 Change-Id: I4692f7fb99696f64da9d48139da6e2763899c02d Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7828 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Ramesh N <rnachimu@redhat.com> Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* autoConfig: Return quota and geo rep status in discover volumesRamesh Nachimuthu2014-05-231-0/+9
| | | | | | | | | | | | Returning quota and geo replication status as part of volume discovery. It can be used by the auto config in nagios to create or delete the quota/geo rep services based on their enablement in gluster Bug-Url: https://bugzilla.redhat.com/1099731 Change-Id: Iff9406204f4c55298021a6b262185396713b9b47 Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/7839 Reviewed-by: Sahina Bose <sabose@redhat.com>
* plugins: Fixing issue in SMB status pluginRamesh Nachimuthu2014-05-231-3/+3
| | | | | | | | | | | | SMB is enabled by default. So even if there is no option specified in volume options, we have to consider that SMB is enabled. Also if either SMB or CIFS is disabled, it means that SMB is disabled. Change-Id: Ie1aca2ac51308ba3ae33c767a217188e82570a33 Bug-Url: https://bugzilla.redhat.com/1099334 Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/7847 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* brickstat: fix brick status informationTimothy Asir2014-05-221-1/+1
| | | | | | | | | | | | | Status Information for brick status service will have a proper message like "ok: brick <brickpath> is up" when the brick is up. Bug-url: https://bugzilla.redhat.com/show_bug.cgi?id=1090864 Change-Id: Id03c70d0e4d3b79a7cfc12552047f6905ec74fc9 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7827 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Tested-by: Timothy Asir <tim.gluster@gmail.com>
* gluster-nagios-addons: Fix for improper display of error message.ndarshan2014-05-201-2/+2
| | | | | | | | | | | | Fixed the issue with error message which was displayed when exception was raised. Change-Id: I73400111c8031b269edc32c2e9d387369aeea445 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1094765 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/7793 Reviewed-by: Bala FA <barumuga@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* plugins: Corrected the omprog filter for quota logsSahina Bose2014-05-201-1/+1
| | | | | | | | | | | | The app-name for quota log is based on the brick path and not "bricks". Corrected the filter pattern to match the logs for quota Change-Id: I9619a1131ee54e60b22d36040038a3bbc5d6233d Bug-Url: https://bugzilla.redhat.com/1091998 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7806 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* gluster-nagois-addon: setting interval option for sadf plugins.ndarshan2014-05-201-4/+4
| | | | | | | | | | | | | | | | Setting interval option for memory,swap,network,cpy plugins to 2 minutes. This interval is used for checking if the data retrived from sadf command is old. Before data from sadf older than a minute was considred as stale and hence state was moved to unknown, now its relaxed to two minutes. Change-Id: I237d04ffba8d2dd128ae1c9b2248425460143b8c Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1081953 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/7754 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com> Reviewed-by: Bala FA <barumuga@redhat.com>
* nagios-addons: fix issue during upgrade of nagios-addonsRamesh Nachimuthu2014-05-201-12/+13
| | | | | | | | | | | | | | | | Updating gluster-nagios-addons removes few NRPE commands in the nrpe.cfg. Its because of the way how upgrade works. Normally upgrade works by installing the latest rpms first and then un-installing the older version. So during upgarde we should not delete any configuration. Only during un-installation all the NRPE commands should be deleted. Bug-Url: https://bugzilla.redhat.com/1098393 Change-Id: I4f9f256ad0f7aec5ef85dc42976a7da5fe9799cd Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/7795 Reviewed-by: Bala FA <barumuga@redhat.com>
* plugins: Corrected quota status messagesSahina Bose2014-05-202-2/+4
| | | | | | | | | | | | | Corrected the quota status message when hard limit is reached. Also removed the ";" at the end of the message. Change-Id: If10e07a69d838439d1bfa73eca68b6dc13cf83a8 Bug-Url: https://bugzilla.redhat.com/1098046 Bug-Url: https://bugzilla.redhat.com/1098427 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7798 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* network: show only interfaces having IP addressesBala.FA2014-05-166-270/+231
| | | | | | | | | | | | | | | | | | | | | | | | Network plugin for Nagios provides status and performance data of network interfaces. By default, it provides details of all the interfaces having IP addresses The plugin can be used to get status of specific interface: $ python network.py -i em1 get status except specific interface: $ python network.py -e em1 get status of all interfaces: $ python network.py -a Change-Id: If3cc2e4c00c866ce1a3c839a643d63b1e23ff8ea Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1085237 Signed-off-by: Bala.FA <barumuga@redhat.com> Reviewed-on: http://review.gluster.org/7588 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Sahina Bose <sabose@redhat.com>
* plugins: fix issues in CTDB monitoring pluginKanagaraj M2014-05-151-10/+31
| | | | | | | | | | | | | | | | | | | | | | | | When CTDB service is running, it can be in following states - Ok - Disconnected - Banned - Disabled - Unhealthy - Stopped - Inactive - PartiallyOnline - [And combinations of the above except Ok] Plugin Output: - OK if state is OK - WARNING if state is UNHEALTHY/DISABLED/PARTIALLYONLINE - CRITICAL if any other state Change-Id: I85bcf8cf863619ef9d0296fb93e77eea8a3dda6a Bug-Url: https://bugzilla.redhat.com/1097682 Signed-off-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-on: http://review.gluster.org/7776 Reviewed-by: Sahina Bose <sabose@redhat.com>
* fix CTDB status shows UNKNOWN instead of OKTimothy Asir2014-05-141-0/+2
| | | | | | | | | | | | when ctdb is configured and running, status column shows "UNKNOWN" but status information shows "Node Status :OK". Change-Id: Iaba78ddc276061652b9285139deffcbc17791043 Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1097682 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7763 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* plugins: Change service state to Warning for self-heal problemSahina Bose2014-05-141-1/+1
| | | | | | | | | | If unsynced entries are present, the service "Volume Self-Heal" changes to WARNING rather than CRITICAL Change-Id: Ib258d766dd63c69ef86083a8ea1faee0cbe5a8b6 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7760 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
* fixed pep8 error in check_disk_and_inode.pyTimothy Asir2014-05-131-8/+8
| | | | | | | | Change-Id: I4a36319fd2d5bf15adc46a2ff37732a83dc0bd13 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7752 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-addons : modified the plugin output for UNKNOWNNishanth Thomas2014-05-131-5/+6
| | | | | | | | | | | | Modified the plugin output when there is failure from ligfapi and not able to retrieve the data Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1095611 Change-Id: I3b48772b80882c67d4260c036090961a97347390 Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Reviewed-on: http://review.gluster.org/7712 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Nishanth Thomas <nishusemail@gmail.com>
* Enhance brick usage plugin config to return percentage wise usageTimothy Asir2014-05-121-1/+1
| | | | | | | | Change-Id: I46e79a20f1077e9ea3d893af6dde4c747fd493a6 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7692 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Sahina Bose <sabose@redhat.com>
* Add disk and inode total usage in the performance dataTimothy Asir2014-05-121-8/+10
| | | | | | | | Change-Id: Ifc2b679a4f4e6f7ab50ba091befacb4194790ccf Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7684 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-addons : Remove unused fileNishanth Thomas2014-05-122-129/+0
| | | | | | | | | | | | | Removed the unused file check_vol_status.py from the repo and Makefile Change-Id: Ie42e24223e5703b7ca024da685a3f0748b39f2cf Bug-URL: https://bugzilla.redhat.com/show_bug.cgi?id=1090937 Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Reviewed-on: http://review.gluster.org/7725 Tested-by: Nishanth Thomas <nishusemail@gmail.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* autoconf: return all hosts in discoverpeers commandRamesh Nachimuthu2014-05-121-6/+5
| | | | | | | | | | | | Return all the hosts including disconnected hosts with the status field in discoverpeers NRPE command Bug-Url: https://bugzilla.redhat.com/1091170 Change-Id: If3f8076ba6857940bdd7c8eb7a5b2d0715985733 Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/7679 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Sahina Bose <sabose@redhat.com>
* nagios-plugins: Fixed exit status and message for quota pluginSahina Bose2014-05-083-9/+13
| | | | | | | | | | | Changed the passive service check to return warning when soft limit exceeded Also corrected the message- if only soft limit exceeded, the message should not contain any entry for hard limit. Change-Id: Ice693997c769d9029825fcd4aafacc45d1577e23 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7706
* nagios-plugins: Fixed multiline output in geo-rep statusSahina Bose2014-05-081-2/+2
| | | | | | | | | | | Fixed the multiline output issue in geo-rep status plugin. The extra details are now separated by a line-break so that it does not get displayed in the services summary page Change-Id: I2ff6e099adf4969cc79eec8a80abcb7a4ae88ae4 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7707
* rhs-addons:Fix: Modified swap and memory plugin to show in GB.ndarshan2014-05-084-35/+49
| | | | | | | | | | | Modified swap and memory plugin to show message and performance data in GB instead of kB. https://bugzilla.redhat.com/show_bug.cgi?id=1080004 Signed-off-by: ndarshan <dnarayan@redhat.com> Change-Id: I06cfd4cd8e19ac8bc21c50ee0f37f15b4ae8cc2e Reviewed-on: http://review.gluster.org/7640 Reviewed-by: Sahina Bose <sabose@redhat.com>
* plugin: fix issue in smb monitoringKanagaraj M2014-05-083-5/+5
| | | | | | | | | | | - changed the service name to 'SMB' - changed the default value to 'disable' Change-Id: I3b7a3b789495278e4b1de0a78e500994739d305a Signed-off-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-on: http://review.gluster.org/7701 Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* nagios-addons: Add NRPE command for gluster processesRamesh Nachimuthu2014-05-075-172/+300
| | | | | | | | | | | | | | | | | | | | | Adding NRPE command for all process related services. We already have passive checks through NSCA for all these services and bricks. But passive checks are submitted only when there is a state change and Nagios may miss those check results if those services are not cofigured when we submit passive checks. So, we should have active checks running atleast for the first time to determine the initial status of the service Command 'check_proc_status' helps to check the status of all gluster related process Command 'check_brick_status' helps to check the status of bricks in the host. Change-Id: I1f442c0c1e54d606576bc0814044f2f149bca220 Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/7694 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* glusterpmd: fix brick service nameKanagaraj M2014-05-071-2/+2
| | | | | | | | | changed from "Brick Status - <dir>" to "Brick - <dir>" Change-Id: If71b54e7b5fad50d849fb90d20caec6a59353a6f Signed-off-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-on: http://review.gluster.org/7690 Reviewed-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-addons: check disk status along with brick statusNishanth Thomas2014-05-051-0/+15
| | | | | | | | | | | | | Modifed the check_proc_status plugin to check the disk status as well for the brick and accordingly report the status to brick service in nagios server. Change-Id: Ie9c59382e5926682254eff1c89e3a992deda6a38 Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Reviewed-on: http://review.gluster.org/7634 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Tested-by: Nishanth Thomas <nishusemail@gmail.com> Reviewed-by: Timothy Asir <tim.gluster@gmail.com>
* plugins: Enhanced volume quota status pluginSahina Bose2014-05-052-7/+36
| | | | | | | | | | | Quota status plugin enhanced to return soft-limit as well as hard-limit exceeded messages along with the directories that it's exceeded on. Change-Id: I77a8686e2742e2174c9f5ebdade8f017a1ffad4e Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7672 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* refactored service description for gluster servicesKanagaraj M2014-05-051-5/+5
| | | | | | | | | | | | | | | | service descriptions: - Gluster Management - NFS - Self-Heal - Quota - CIFS - Brick - <brick_dir> Change-Id: Icf31ff3c93c8763e4751ed7fdecce904cee59c14 Signed-off-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-on: http://review.gluster.org/7666 Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* Enhanced check proc and added CTDB service checkTimothy Asir2014-05-051-13/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | CTDB is node based and any gluster volume can be configured to be used by CTDB. Normally when CTDB starts it will start Samba automatically. However one can switch off SMB at any given time. But If SMB/NFS are not running in any node, then CTDB is also not required. This patch checks for CTDB process and will send an appropriate message to nagios based on its status using 'ctdb nodestatus' command. Currently there is no direct way to find whether CTDB is required to run on a given node. Its because, the CTDB volume name / CTDB volume mount path can have any name. It could be easy if gluster-cli comes with any tag/set command to know whether the given volume is for CTDB or CTDB enabled. Change-Id: Iccb98296ebd902838acc63805bbe20cd77cdcc61 Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7647 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* plugins: Fix pep8 error in nscautilsSahina Bose2014-05-051-1/+1
| | | | | | | | Fix pep8 error Change-Id: I9dc12c6a72b86ec8fadb9bd2447a4eb522a44afb Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7668
* plugins: Plugin to handle server quorumSahina Bose2014-05-054-13/+78
| | | | | | | | | | | | | | | | | | | Plugin to handle log messages related to server quorum Message is parsed for msgid as the msgid is not correctly populated now Fixed pep8 errors in nscautils and added tests Also changed the service name to remove "Status" -it now reads Volume Quota, for instance Change-Id: I339ed16b3806f853ae3555a998974c47157c95ad Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7648 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* plugins: Changed nsca method for syslog monitoringSahina Bose2014-05-043-5/+29
| | | | | | | | | | | | | Changed the syslog monitoring plugin to use subprocess to send the nsca command There was an issue with using the utils method when being called from rsyslog service Change-Id: Ia4dea2b0f942ce815f123ece1fba8fd1f18fc6be Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7639 Reviewed-by: Timothy Asir <tim.gluster@gmail.com>
* plugins: fix exitstatus in geo-rep status pluginKanagaraj M2014-05-041-1/+1
| | | | | | | | | | Changed the exit code from PluginStatus to PluginStatusCode Change-Id: I7a54184516e4c9456ec7c93d02f7f0c1e0224368 Signed-off-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-on: http://review.gluster.org/7657 Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
* Fix pyflakes import unused errorTimothy Asir2014-05-021-1/+0
| | | | | | | | | | Removed 'import socket' from check_proc_status.py Change-Id: Ib6a4247b7f73b06b569e067b6af94d5810ad7fde Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7649 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>