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>
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>
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>
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>
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>
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>
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>
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>
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>
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>