summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* nagios: georep status fixing error when no sessionsSahina Bose2015-11-051-0/+2
| | | | | | | | | | Fixing error returned when geo-rep sessions are deleted Change-Id: If263ddf11dd486b32aae1e979f754b191ad082b6 Bug-Url:https://bugzilla.redhat.com/1261004 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/12441 Reviewed-by: Ramesh N <rnachimu@redhat.com>
* nagios-common: Fixing pep8 errorsv1.1.0Sahina Bose2015-09-281-3/+6
| | | | | | | | | Fixing pep8 errors on older version of pep8 Change-Id: I23253b9fcb830a2c71542f058ae50cc46009fd88 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/12243 Reviewed-by: darshan n <dnarayan@redhat.com>
* gluster-nagios: geo rep status pluginSahina Bose2015-08-272-134/+203
| | | | | | | | | | | Changed the georep status plugin to reflect the changes in georep status output Change-Id: I7f8f7c07d3da552283758a221c819744c616777b Bug-Url: https://bugzilla.redhat.com/1240959 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/11983 Reviewed-by: darshan n <dnarayan@redhat.com>
* nagios-common: Added disperse-vol related details to glustercli utility.Darshan N2015-08-162-2/+16
| | | | | | | | | | | | | This patch adds disperse volume related details like disperse count redundancy count to glustercli utility. This is needed to monitor volume status of disperse volume. It also fixes few pep8 issues. Change-Id: I38b4a0fd0a3ffb550be8bc91863b70fe8e70d883 Bug-Url: https://bugzilla.redhat.com/1235651 Signed-off-by: Darshan N <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/11502 Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* PEP8: Fixed pep8 issues in glusternagios/glustercli.pydarshan n2015-08-161-4/+4
| | | | | | | Change-Id: I52b568bc450fcdf546466db29c49ad6a8a2eda67 Signed-off-by: darshan n <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/9592 Reviewed-by: Sahina Bose <sabose@redhat.com>
* nagios-common: Split brain status function addedSahina Bose2015-03-252-6/+29
| | | | | | | | | | | | | | Modified the existing function to return the "volume heal <vol-name> info split-brain" output to be used by nagios plugins. The function that uses "volume heal <vol-name> info" command has been renamed Change-Id: I5a05930ae76be04535c74cffd5f535bfb6a6d97a Bug-Url: https://bugzilla.redhat.com/1081900 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/9782 Reviewed-by: darshan n <dnarayan@redhat.com>
* nagios-common: Corrected geo-rep status in case of replicate volumesv1.0.0Sahina Bose2014-09-261-5/+8
| | | | | | | | | | | | | | | The logic to determine faulty should be if count of passive+faulty > (brickcount/replicacount) Separated logic for replicate volumes to fix the comparison Change-Id: Ia7797cb091dc14e551ddbe2e540d532930e41be3 Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1109727 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/8443 Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* nagios-commons: Added a new exception classSahina Bose2014-09-232-18/+32
| | | | | | | | | | | | Added different exception in case of transaction lock errors, so that the plugins can return correct state based on the exception messages Change-Id: Ibbe5dd17b4829765505451f5a6554d2ade4b2980 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/8442 Reviewed-by: Bala FA <barumuga@redhat.com>
* fix pep8 errorsv0.1.1Timothy Asir Jeyasingh2014-06-266-72/+72
| | | | | | | | Change-Id: I3121e6b4044e2f0831fe3c862fb12e8bc27cfd3c Signed-off-by: Timothy Asir Jeyasingh <tjeyasin@redhat.com> Reviewed-on: http://review.gluster.org/8160 Tested-by: Timothy Asir <tim.gluster@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* nagios-common: Added noarch directive to buildSahina Bose2014-06-211-0/+2
| | | | | | | | Change-Id: Iea01bf9158e2cba200541cbc2426521f08a08ea8 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/8143 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Tested-by: Kanagaraj M <kmayilsa@redhat.com>
* nagios-common: Geo-rep status fixed check for replicacountSahina Bose2014-06-121-2/+2
| | | | | | | | | | | An error with comparison of replicacount returned from dict fixed with this patch Change-Id: I554dee99af5abc8cb88c337aa05ae41d3017224a Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1101223 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/8051 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* nagios-common: Georep cli session wise parsing correctedSahina Bose2014-06-062-14/+26
| | | | | | | | | | | | | | | | | | | Geo-rep status cli command does not uniquely identify sessions per node line output. It prefixes the slave node with slave volume name. And when the slave volume has multiple nodes, the session status cannot be aggregated based on this. This patch strips out the slave node ip to uniquely identify the session. This will not work when a volume has geo-rep sessions created for multiple slave volumes - all with the same name Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1101223 Change-Id: Ie6e12c1b3af4752c5ef14f0c7fe8db579013d19e Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7988 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* nagios-common: geo-rep-corrected the comparison check for faultySahina Bose2014-05-291-4/+4
| | | | | | | | | | | | Geo rep faulty status would be if any 1 node is faulty in distributed type Also corrected the doc text Change-Id: Ie926e8c44ddff986d616369f1fcab7d0f3956f01 Bug-Url: https://bugzilla.redhat.com/1101218 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7922 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
* nagios-common: Enhanced geo rep status parsingSahina Bose2014-05-282-55/+145
| | | | | | | | | | | | | Enhanced the gluster cli - geo rep status- parsing to consider other node states like stopped and to combine it with volume topology. Change-Id: I21876339f82a0137c41f7ec7ccc0672815590e88 Bug-Url: https://bugzilla.redhat.com/1101218 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7888 Reviewed-by: Ramesh N <rnachimu@redhat.com> Reviewed-by: Aravinda VK <avishwan@redhat.com>
* glustercli: fix for local ip address issue in peerStatusRamesh Nachimuthu2014-05-061-3/+6
| | | | | | | | | | | | | | | | | glustercli._getIpAddresses() throws an exception when 'ovirtmgmt' bridge is present. This is used by peerStatus() to get the local ip address of the node and peer status is failing because of the exception. Current approach is anyway not fool proof as the node can have multiple network devices and we may not get the right ip address. So adding 'localhost' as the address for the current host in peerStatus() API. Change-Id: Ib804b2109a2578f180b6bf54904a28f19bbdbb7a Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com> Reviewed-on: http://review.gluster.org/7688 Reviewed-by: Sahina Bose <sabose@redhat.com>
* git-nagios-common : fixed the unittest errorNishanth Thomas2014-05-051-5/+4
| | | | | | | | | | | | Fixed the issues with UT due to refactoring of getDisksForBrick function Change-Id: Iae2737913d35d117f8160e3f985d9f07d9f9e333 Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Reviewed-on: http://review.gluster.org/7675 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Tested-by: Nishanth Thomas <nishusemail@gmail.com> Reviewed-by: Bala FA <barumuga@redhat.com>
* gluster-nagios-common: check disk status along with brick statusNishanth Thomas2014-05-051-1/+33
| | | | | | | | | | | | Modifed the storage utils to add the utility functions check the disk status Change-Id: If4eac75451071456876a858e2bc4d2d262a9e451 Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Reviewed-on: http://review.gluster.org/7635 Reviewed-by: Kanagaraj M <kmayilsa@redhat.com> Tested-by: Nishanth Thomas <nishusemail@gmail.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-common: PEP8 fixes.n darshan2014-05-052-8/+8
| | | | | | | | | | fixed pep8 issues in glusternagios/glustercli.py tests/test_glustercli.py Change-Id: Iea9d12543f854dca322f774f3fc255d0716107ff Signed-off-by: n darshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/7670 Reviewed-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-common: Enhanced module for volumeQuotaStatus.n darshan2014-05-052-15/+44
| | | | | | | | | | | | | | | | Enhanced the module volumeQuotaStatus to return the list of directories crossing the hard and soft limit along with quota status.Now this module returns a dictionary of following format: {status: OK|DISABLED|SOFT_LIMIT_EXCEEDED|HARD_LIMIT_EXCEEDED, hard_ex_dirs: ['dir1','dir2'.....], soft_ex_dirs: ['dir3','dir4'.....]} Change-Id: I4e8ebd622604d3d6bd28a078c73bad834d06da5d Signed-off-by: n darshan <dnarayan@redhat.com> Reviewed-on: http://review.gluster.org/7667 Reviewed-by: Sahina Bose <sabose@redhat.com>
* spec: fix dependencies issues in spec fileKanagaraj M2014-05-021-0/+4
| | | | | | | | | Adding BuildRequires for the required packages. Change-Id: I69a25f9fd6ec70d7bce778daedb8a1ac3131f454 Signed-off-by: Kanagaraj M <kmayilsa@redhat.com> Reviewed-on: http://review.gluster.org/7629 Reviewed-by: Sahina Bose <sabose@redhat.com>
* glustercli: Added geo-rep statusSahina Bose2014-04-292-0/+163
| | | | | | | | | | | | | | | | | | Added method to parse gluster cli output of "gluster volume geo-rep <vol-name> status" and provide output in the format {volumename : {'status': STATUS, 'detail': message string}} Temporarily parsing string output till cli xml output is available for geo-rep (https://bugzilla.redhat.com/show_bug.cgi?id=1090910) Change-Id: Ie12cfcd8bb0d3bf0b3d9c13567e40d9014c47f59 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/7590 Reviewed-by: Aravinda VK <avishwan@redhat.com> Reviewed-by: Bala FA <barumuga@redhat.com>
* gluster-nagios-common: test_storage.py fixesShubhendu Tripathi2014-04-281-3/+21
| | | | | | | | Fixed test case failures related to command execution in test_storage.py Change-Id: I056b03d4a41ce884e8ed03bd446ea189d556cd7d Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
* glustercli: Added method for split-brain statusSahina Bose2014-04-282-0/+100
| | | | | | | Added method to check "volume heal <vol-name> info" Change-Id: I367eee5c63e3256f8834ea3845ed7ada6b073001 Signed-off-by: Sahina Bose <sabose@redhat.com>
* snmp: open snmp conf file in READ modeKanagaraj M2014-04-281-1/+1
| | | | | | | Open tne snmpmanagers.conf file in READ-ONLY mode. Change-Id: Ifbb487ef5da6e09b1ff0a09cfe2efb0ab10d6ab3 Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
* enable volumeInfo to provide host uuidTimothy Asir2014-04-282-1/+7
| | | | | Change-Id: If1f9d22c59200f707c5a86d0f89628a2d8380d55 Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
* glustercli: Changed return code when quota not enabledSahina Bose2014-04-282-3/+5
| | | | | | | | | When quota is not enabled the gluster cli command fails and was giving the wrong status. Fixed this. Change-Id: Ibf9709f7ccd17565473d5a258b323123f6904b82 Signed-off-by: Sahina Bose <sabose@redhat.com>
* SNMP Notifications:utility funtion to read SNMP managersNishanth Thomas2014-04-281-0/+14
| | | | | | | | Also added the the commands for sending SNMP traps Addressed the review comments Change-Id: I5612aaa241eb05d606a98d604be4f70486a141b0 Signed-off-by: Nishanth Thomas <nthomas@redhat.com>
* common: Added volume quota statusSahina Bose2014-04-282-0/+73
| | | | | | | | Added volume quota status to glustercli Change-Id: Iefe0766d652dab7adb5d2514410434fb5c190631 Signed-off-by: Sahina Bose <sabose@redhat.com> Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
* gluster-nagios-comon: Minor pep8 fixShubhendu Tripathi2014-04-281-1/+2
| | | | | Change-Id: If4a253b347ea60ce18c43cd88bd946e9b62e491f Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
* gluster-nagios-common: Fixed test failure for glusterShubhendu Tripathi2014-04-281-2/+9
| | | | | | | Fixed the test failure because of gluster command Change-Id: Ica6cd01c17e6bc9a9951bfe2c488eb6a359af447 Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
* gluster-nagios-common: Added test cases to make fileShubhendu Tripathi2014-04-282-5/+6
| | | | | | | | Added test cases test_glustercli and test_storage to the make file Change-Id: Ia75c7abbad565ee505ccb150b1806246ba81eedb Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
* gluster-nagios-common: Corrected command definitionsShubhendu Tripathi2014-04-281-0/+2
| | | | | | | | | | Corrected the command definitions which were removed accidently Change-Id: I1e069d178d48534b72245b025b63b5e2616bd4bd Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-on: https://code.engineering.redhat.com/gerrit/22992 Reviewed-by: Kanagaraj Mayilsamy <kmayilsa@redhat.com>
* gluster-nagios-common: Added storage utils moduleShubhendu Tripathi2014-04-284-11/+645
| | | | | | | | | | Introduced storage device utils module Change-Id: I738b46b0750deaf91b896a8edab07b5f1a692ddb Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-on: https://code.engineering.redhat.com/gerrit/22254 Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Sahina Bose <sabose@redhat.com>
* gluster-nagios-common: Added gluster cli moduleShubhendu Tripathi2014-04-284-0/+1571
| | | | | | | | | | | | | | Introduced gluster cli module to add all the gluster related get methods Change-Id: I440ae89ac3f93f961024a6e78870154f57b7dfbd Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-on: https://code.engineering.redhat.com/gerrit/22253 Reviewed-by: Darshan Narayana Murthy <dnarayan@redhat.com> Reviewed-by: Timothy Asir Jeyasingh <tjeyasin@redhat.com> Reviewed-by: Balamurugan Arumugam <barumuga@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com> Tested-by: Sahina Bose <sabose@redhat.com>
* Autoconf:moved parseXML fun to utilsNishanth Thomas2014-04-281-2/+12
| | | | | | | | | | Added few command definitions Change-Id: I90a584868f684896da49d5101ceb512a82e231dc Signed-off-by: Nishanth Thomas <nthomas@redhat.com> Reviewed-on: https://code.engineering.redhat.com/gerrit/22548 Reviewed-by: Kanagaraj Mayilsamy <kmayilsa@redhat.com> Tested-by: Ramesh Nachimuthu <rnachimu@redhat.com>
* gluster-nagios-common: Added cmd path for pvsShubhendu Tripathi2014-04-281-0/+8
| | | | | | | | | | Added command path entry for pvs command Change-Id: I933f97fd688f67131a1109e248af4bd58d79dc81 Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-on: https://code.engineering.redhat.com/gerrit/21687 Reviewed-by: Balamurugan Arumugam <barumuga@redhat.com> Reviewed-by: Sahina Bose <sabose@redhat.com>
* utils: Added utils method to convert sizeSahina Bose2014-04-282-0/+35
| | | | | | | | | | | | The method converts a value from a given unit to another. Example usage: convertSize (value, 'KB', 'MB) Change-Id: If8e004076493006234be81e656eabde7d3bf9a6a Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: https://cuckoo.blr.redhat.com:8443/32 Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com> Reviewed-by: Nishanth Thomas <nishusemail@gmail.com>
* utils: add xml2dict()Bala.FA2014-04-284-0/+105
| | | | | | | | The method coverts xml object as elementTree to dictionary. Change-Id: Ia7eca3eb52a79979d7d1ee9d4ce1c842cc86e8ed Signed-off-by: Bala.FA <barumuga@redhat.com> Reviewed-on: https://cuckoo.blr.redhat.com:8443/22
* constants: add host and service plugin status codesBala.FA2014-04-281-0/+26
| | | | | | | | Change-Id: Ideb9975c73ebc7c7339fa4aac354a974f1e3921e Signed-off-by: Bala.FA <barumuga@redhat.com> Reviewed-on: https://cuckoo.blr.redhat.com:8443/17 Reviewed-by: Sahina Bose <sabose@redhat.com> Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
* rfc: refactor to make more usable and friendlyBala.FA2014-04-282-64/+182
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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: I6ca6f4ad70ea3a902c3415a12a12c3e46e79adef Signed-off-by: Bala.FA <barumuga@redhat.com> Reviewed-on: https://cuckoo.blr.redhat.com:8443/12
* Initial commitv0.1.0Bala.FA2014-04-2825-0/+2177
| | | | | Change-Id: Ie8fdd046d111a4a46abe0e162985e833323bfd7d Signed-off-by: Bala.FA <barumuga@redhat.com>
* Initial empty repositoryHumble Devassy Chirammal2014-04-220-0/+0