summaryrefslogtreecommitdiffstats
path: root/glusternagios/glustercli.py
Commit message (Collapse)AuthorAgeFilesLines
* 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>
generated by cgit (git 2.34.1) at 2024-04-19 06:58:49 +0000