| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
Change-Id: Ie8fdd046d111a4a46abe0e162985e833323bfd7d
Signed-off-by: Bala.FA <barumuga@redhat.com>
|