| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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 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>
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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._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>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
Fixed test case failures related to command execution in
test_storage.py
Change-Id: I056b03d4a41ce884e8ed03bd446ea189d556cd7d
Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
|
|
|
|
|
|
|
| |
Added method to check "volume heal <vol-name> info"
Change-Id: I367eee5c63e3256f8834ea3845ed7ada6b073001
Signed-off-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
| |
Open tne snmpmanagers.conf file in READ-ONLY mode.
Change-Id: Ifbb487ef5da6e09b1ff0a09cfe2efb0ab10d6ab3
Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
|
|
|
|
|
| |
Change-Id: If1f9d22c59200f707c5a86d0f89628a2d8380d55
Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
Also added the the commands for sending SNMP traps
Addressed the review comments
Change-Id: I5612aaa241eb05d606a98d604be4f70486a141b0
Signed-off-by: Nishanth Thomas <nthomas@redhat.com>
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
| |
Change-Id: If4a253b347ea60ce18c43cd88bd946e9b62e491f
Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
|
|
|
|
|
|
|
| |
Fixed the test failure because of gluster command
Change-Id: Ica6cd01c17e6bc9a9951bfe2c488eb6a359af447
Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
|
|
|
|
|
|
|
|
| |
Added test cases test_glustercli and test_storage to the
make file
Change-Id: Ia75c7abbad565ee505ccb150b1806246ba81eedb
Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
| |
Change-Id: Ie8fdd046d111a4a46abe0e162985e833323bfd7d
Signed-off-by: Bala.FA <barumuga@redhat.com>
|
|
|