summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* 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>
* Run check gluster process status as a daemonTimothy Asir2014-05-026-46/+203
| | | | | | | | | | | | Enhanced to send specific gluster related process status only when there is a change detected in a service status or for any critical status to avoid too many logs in the nagios service side. Change-Id: I26e389ae2d1ccba1b5ccadc45d202d3b5219c74a Signed-off-by: Timothy Asir <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/7602 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Timothy Asir <tim.gluster@gmail.com>
* vol-status: remove volume-info from statusKanagaraj M2014-05-022-20/+2
| | | | | | | | | | | When Volume is in OK state, volume info will not be sent along with status. Change-Id: I3b3644c45676c35f7b7dec048531a40b3de2fb8d Signed-off-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-on: http://review.gluster.org/7612 Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* autoconf:Fix for pep8 issueRamesh Nachimuthu2014-05-021-2/+2
| | | | | | | | | fixing a small pep8 issue Change-Id: I203ae51d78b846184dc873b18a6bfbf526f937e3 Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/7646 Reviewed-by: Sahina Bose <sabose@redhat.com>
* autoconf: discover volume list and info separatelyRamesh Nachimuthu2014-05-026-52/+186
| | | | | | | | | | | | | | | | | | | | NRPE doesn't support transfering large junk of data as a result. Hence we have to discover the volume details one by one. Added two NRPE commands 'discover_volume_list' and 'discover_volume_info'. 'discover_volume_list' returns the list of volume names with volume type. 'discover_volume_info' returns the bricks details of a given volume. Change-Id: I753be5e407fe14988f23ca77007b3a585537b360 Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/7630 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* plugins: Fixed vol_status nrpe command argumentsSahina Bose2014-05-011-4/+1
| | | | | | | | | | | Fixed the check_vol_status nrpe plugin to take arguments instead of defining multiple commands Change-Id: I8d12848e4bbb2394c018b4968baa8df69ce7793a Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7637 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Ramesh N <rnachimu@redhat.com>
* autoconf: fix for pep8 issueRamesh Nachimuthu2014-05-011-2/+2
| | | | | | | | | | fix for pep8 issue Change-Id: Ia402d2f152a5ecd169f43c759c5beb1c4038ff3b Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/7625 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* plugins: Added geo-rep status to volume_statusSahina Bose2014-05-013-1/+53
| | | | | | | | | | | Added option to get the geo rep status to check_volume_status plugin Change-Id: I32d2a0240e3888505c54fdce9a3fb731b9300e38 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7591 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* autoconf: Add NRPE command to configure NSCARamesh Nachimuthu2014-04-293-0/+60
| | | | | | | | | | Adding a new NRPE command 'configure_gluster_node' to configure gluster nodes to send passive check results using NSCA. Change-Id: I1993eefab112f9fb5ba7764593cebca9923b4aba Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/7595 Reviewed-by: Sahina Bose <sabose@redhat.com>
* brickstatus: Use only brick path in brick status serviceRamesh Nachimuthu2014-04-291-4/+5
| | | | | | | | | | | | | Change the brick status service descrption to "Brick Status - <brick-path>". Currently it is "Brick Status - <hostaddress>:<brick-path>". Change-Id: Ib6ca14447756ef96bff65f5732699ac5842eabce Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/7596 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* spec: fix dependencies issues in spec fileKanagaraj M2014-04-291-0/+7
| | | | | | | Adding BuildRequires for the required packages. Change-Id: Iacfe1e09610b4801548cc56f703ee12ba294ec24 Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
* plugins: Fixed pep8 error in check_volume_statusSahina Bose2014-04-291-0/+1
| | | | | Change-Id: I5464c916e1885c5f4109a5c1ae9c3abd9d040fab Signed-off-by: Sahina Bose <sabose@redhat.com>
* autoconf: adding sudo in nrpe commandsRamesh Nachimuthu2014-04-291-2/+2
| | | | | | | Adding sudo in nrpe commands to use clustercli. Change-Id: I3ad29d942def0969489c2ff7c17065546b0a1427 Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
* plugins: Added self heal status optionSahina Bose2014-04-292-6/+37
| | | | | | | | Added self-heal status option to the check_volume_status plugin Change-Id: Ia48f7baa3b6e4f0091be2116ab058ab48f1e5a29 Signed-off-by: Sahina Bose <sabose@redhat.com>
* autoconf: use host uuid to identity bricksRamesh Nachimuthu2014-04-292-19/+15
| | | | | | | | | | Currently host IP/Name is used to indetify the bricks in host. But it breaks when we use fqdn names in peer probe. So changing the auto conf to use Host UUIDs to identify the bricks Change-Id: Id28ac7a47a4ce5beb1fc8f91cdb136bca17f071f Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
* autoconf: use host name for host config - node sideRamesh Nachimuthu2014-04-291-0/+1
| | | | | | | | adding the NRPE command 'discoverhostparams' to find the host name to use it host configuration. Change-Id: Ic80ac487059566f62afad83a5880922f8b5eb4cd Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
* check-proc: Fix issues in SHD and NFSKanagaraj M2014-04-291-8/+16
| | | | | | | | - Skipped OFFLINE volumes for checking nfs option - Fixed a loop issue in SHD Change-Id: I76a07ceb6724c6d723aec135021501518341fab7 Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
* fix send_nsca packet errorTimothy Asir2014-04-291-4/+4
| | | | | Change-Id: Id1a490777b0765b7e2bbfcbdb3615970af82f015 Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
* Use hostname from the config for sending proc status thru nscaTimothy Asir2014-04-294-4/+16
| | | | | | | | | | fetch the hostname from nagios_server.conf file otherwise use the host-fqdn name for sending proc status thru nsca. And fix the check_proc_status plugin path in cron.d/gluster-proc.crontab file. Change-Id: Ibfc027742b58d87daa7e54aa553892d42c71e795 Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
* fix test case failure and pep8 error for check_proc testTimothy Asir2014-04-292-184/+184
| | | | | Change-Id: I3c965b95d85373f79ddebd5c7e38010fc7fef691 Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
* plugins: Added volume info as part of volume statusSahina Bose2014-04-292-2/+31
| | | | | | | | | The volume status also needs to look at the brick statuses Changed the plugin to return the volume info in case the volume is UP Change-Id: Iee0b2ad84c7a3c5d5ebdc7e0616fb6d987cfb25a Signed-off-by: Sahina Bose <sabose@redhat.com>
* Remove gluster-proc.crontab from AC_CONFIG_FILESTimothy Asir2014-04-291-1/+0
| | | | | Change-Id: I51ebd03f2bd8d3339a35ec28176785fd77afbc37 Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
* Add gluster process monitoring pluginTimothy Asir2014-04-298-0/+442
| | | | | | | | | This helps monitoring gluster related process such as glusterd, glusterfsd, gluster self heal process, nfs and also helps to monitor other running services Change-Id: I8c6f252270a847e7ff007c358125a9310525f201 Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
* autoconf: use glustercli to run volume info commandRamesh Nachimuthu2014-04-292-39/+17
| | | | | | | | Using the glustercli.volumeInfo() from common to get the volume info. It makes the discovery logic very simple Change-Id: Iacf319eb3b3b39805a483cbe43b02496802316a7 Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
* plugins: Fixed errors in syslog pluginSahina Bose2014-04-295-6/+4
| | | | | | | | Added a requires on nsca and fixed the errors on calling syslog plugin Change-Id: I310a6a8aa64042a290d45207f78cf1d1d2b3b09b Signed-off-by: Sahina Bose <sabose@redhat.com>
* spec: python-cpopen >= 1.3Kanagaraj M2014-04-291-1/+1
| | | | | | | Updated the spec file with required python-cpopen version. Change-Id: I156ecf072297538186f0d4c5851f24d9ea90ba88 Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
* plugins: Plugin to act on syslog messageSahina Bose2014-04-2912-35/+280
| | | | | | | | | | | | This plugin integrates with syslog messages and sends the output to Nagios server using NSCA Refactored code to separate out common NSCA related code to nscautils Change-Id: I6a874da7f2296310060742acb6b1cd6562f5d0f0 Signed-off-by: Sahina Bose <sabose@redhat.com>
* plugins: Enhanced volume status with quota statusSahina Bose2014-04-295-19/+56
| | | | | | | | | Added an optional parameter to query quota status Added command to nrpe.cfg Change-Id: I9f60ed1a98cb2ca59b799cf9c09e3621b7bd8c0c Signed-off-by: Sahina Bose <sabose@redhat.com>
* rhs-nagios-addons: Plugin for volume statusndarshan2014-04-293-0/+131
| | | | | | | | This patch adds a plugin for volume status, it makes use of gluster command "gluster volume info". Signed-off-by: ndarshan <dnarayan@redhat.com> Change-Id: I6984d52c998bf1b832654b99abc3e2843b54f5ac
* fix the host discovery issue.Ramesh Nachimuthu2014-04-291-1/+1
| | | | | | | | Fixing the issue in discoverying host when there are more than two hosts in the cluster. Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Change-Id: I1d70f7e848b189f346a9cff4994f5bb478ce2a92
* Added the plugins for discovering the gluster componentsNishanth Thomas2014-04-295-0/+170
| | | | | | | | | | Modified the code to address the review comments Change-Id: Iaf18745920ec1177622b5f8a667a2e77f89796da Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Reviewed-on: https://code.engineering.redhat.com/gerrit/22345 Tested-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-by: Kanagaraj Mayilsamy <kmayilsa@redhat.com>
* rhs-nagios-addon: Added nagios-plugins-ide_smart dependencyShubhendu Tripathi2014-04-291-0/+1
| | | | | | | | | | | Added the nagios-plugins-ide_smart dependency to be installed Change-Id: I7fa05fcac6a769a9cdc253f3fe55b2b4f8e72fd5 Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-on: https://code.engineering.redhat.com/gerrit/21956 Reviewed-by: Timothy Asir Jeyasingh <tjeyasin@redhat.com> Reviewed-by: Darshan Narayana Murthy <dnarayan@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* Add command entry for check_vol_utilization in the spec file.Timothy Asir2014-04-291-0/+1
| | | | | | | Change-Id: I726478c9a36674c24ef4dbecdea6f0f6a1f432f2 Signed-off-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-on: https://cuckoo.blr.redhat.com:8443/67 Reviewed-by: Sahina Bose <sabose@redhat.com>
* Add python-cpopen requires dependency in the spec file.Timothy Asir2014-04-291-0/+1
| | | | | | | Change-Id: I41d4f611c9e2cde12eae802476684c13b82f5c3f Signed-off-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-on: https://cuckoo.blr.redhat.com:8443/60 Reviewed-by: Sahina Bose <sabose@redhat.com>
* fix: fix for build issue with respect to capacity.soRamesh Nachimuthu2014-04-291-4/+4
| | | | | | | Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Change-Id: Ic680cee909eb41358f524256b9036790370cb53b Reviewed-on: https://cuckoo.blr.redhat.com:8443/55 Reviewed-by: Sahina Bose <sabose@redhat.com>
* plugins: Added the files for volume utilization and status plugins fixed few ↵Nishanth Thomas2014-04-299-0/+444
| | | | | | | | | | | | issues with code fixed the pep8 issues Change-Id: I2c2b32e7bbdc30a7b68acc1ce4e0468fd53a3050 Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Reviewed-on: https://cuckoo.blr.redhat.com:8443/42 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Sahina Bose <sabose@redhat.com>
* Enhanced check_disk_and_inode pluginTimothy Asir2014-04-293-121/+345
| | | | | | | | | | | Add option to include / exclude disk Add option to get disk usage without inode usage Add unit test Change-Id: Ifa8c13033269fea184d2dedc720dba4239770336 Signed-off-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-on: https://cuckoo.blr.redhat.com:8443/18 Reviewed-by: Sahina Bose <sabose@redhat.com>
* Plugin: Fix for wrong argument for network plugin in spec.ndarshan2014-04-291-1/+1
| | | | | | | | | | | Modified the argument for network plugin to be a string, in spec file. Change-Id: Ia4bb1c113f1117a9e06cf81e7bb78a11b7b4c048 Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: https://cuckoo.blr.redhat.com:8443/52 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Sahina Bose <sabose@redhat.com>
* plugins:Fix to handle sadf not accepting time range, test case additionndarshan2014-04-2920-440/+3590
| | | | | | | | | | | | | This patch handles the issue of sadf not accepting time range when used with -x (xml output) option(seen in version 9.0.4). Added unit-test for memory, cpu, swap, network plugins and refactored them. Change-Id: Ie7c2ecfbb38060f236a6faed606bce0aedd27d7a Signed-off-by: ndarshan <dnarayan@redhat.com> Reviewed-on: https://cuckoo.blr.redhat.com:8443/14 Reviewed-by: Bala FA <barumuga@redhat.com> Tested-by: Bala FA <barumuga@redhat.com>
* rfc: refactor to make more usable and friendlyBala.FA2014-04-291-57/+176
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Usage: rfc.sh [OPTION]... [ <REVIEWER> ]... Options: -a ask for Bug ID addition. This option is set by default for non 'master' branches. -b BRANCH use BRANCH to submitting patch. Default branch is 'master'. -d dry run. Show what command to run. -n do not fetch origin. -t TOPIC use TOPIC to submitting patch. -h display this help text and exit. By default, BUGID is used as topic in patch submission. If TOPIC and BUGID are used together, TOPIC gets used. Examples: # submit patch to master branch without reviewer $ rfc.sh # submit patch to master branch with reviewer $ rfc.sh charlie@example.com # submit patch to release-3.0 branch with topic "awesome feature" and # reviewers charlie@example.com alice@example.com $ rfc.sh -b release-3.0 -t "awesome feature" charlie@example.com alice@example.com Change-Id: I3519725c05456564be7fc21b7d761a21bdf773eb Signed-off-by: Bala.FA <barumuga@redhat.com> Reviewed-on: https://cuckoo.blr.redhat.com:8443/15
* plugins: remove server pluginsBala.FA2014-04-295-423/+0
| | | | | | | | These plugins are moved to nagios-server-addons repository Change-Id: Ie5daa7736e2acfbdf44e5c9c782d8c9b54fc413d Signed-off-by: Bala.FA <barumuga@redhat.com> Reviewed-on: https://cuckoo.blr.redhat.com:8443/7