summaryrefslogtreecommitdiffstats
path: root/glusternagios/glustercli.py
Commit message (Collapse)AuthorAgeFilesLines
* nagios-plugins: CLI to parse heal infoHEADmasterSahina Bose2016-03-241-22/+34
| | | | | | | | | | | | | | Added CLI to parse heal info and provide json output changed parsing of heal info to match the enhanced output patch - http://review.gluster.org/13231 Renamed VolumeSplitBrainStatus enum to generic VolumeHealInfoStatus BUG: 1267586 Change-Id: I948d00d0198894a9a670e72b311ab86c83af4b23 Signed-off-by: Sahina Bose <sabose@redhat.com> Reviewed-on: http://review.gluster.org/12260 Reviewed-by: Ramesh N <rnachimu@redhat.com>
* 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>
* gluster-nagios: geo rep status pluginSahina Bose2015-08-271-63/+58
| | | | | | | | | | | 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-161-2/+8
| | | | | | | | | | | | | 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-251-4/+27
| | | | | | | | | | | | | | 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-231-17/+31
| | | | | | | | | | | | 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>
* 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-061-4/+12
| | | | | | | | | | | | | | | | | | | 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-281-34/+68
| | | | | | | | | | | | | 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>
* gluster-nagios-common: PEP8 fixes.n darshan2014-05-051-6/+6
| | | | | | | | | | 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-051-11/+32
| | | | | | | | | | | | | | | | 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>
* glustercli: Added geo-rep statusSahina Bose2014-04-291-0/+67
| | | | | | | | | | | | | | | | | | 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>
* glustercli: Added method for split-brain statusSahina Bose2014-04-281-0/+50
| | | | | | | Added method to check "volume heal <vol-name> info" Change-Id: I367eee5c63e3256f8834ea3845ed7ada6b073001 Signed-off-by: Sahina Bose <sabose@redhat.com>
* enable volumeInfo to provide host uuidTimothy Asir2014-04-281-1/+1
| | | | | Change-Id: If1f9d22c59200f707c5a86d0f89628a2d8380d55 Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
* glustercli: Changed return code when quota not enabledSahina Bose2014-04-281-0/+3
| | | | | | | | | 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>
* common: Added volume quota statusSahina Bose2014-04-281-0/+30
| | | | | | | | 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-common: Added gluster cli moduleShubhendu Tripathi2014-04-281-0/+469
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>