| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added a plugin to monitor heal info for a volume
Corrected pep8 errors
Changed default monitoring interval to 1 min for
heal info monitoring
Bug-Url: https://bugzilla.redhat.com/1267586
BUG: 1267586
Change-Id: I1c51d4d0f2129a7bb19f44b63382fcfc7384dc40
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/12262
Reviewed-by: Ramesh N <rnachimu@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Fixing the quorum service plugin flipping to
OK state when the servers are down - due to
wrong query of services
BUG: 1284874
Change-Id: I5b6201f430778b92bd30c95fe3c37f4d89240eb8
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/12735
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds logic to check_vol_server plugin to determine
the status of disperse and distributed dispersed volume. With
this patch volume status of dispersed and distributed volume
are correctly determined.
Change-Id: I4c0ee4a63c00201127fe50be424c9f9d3fa5300d
Bug-Url: https://bugzilla.redhat.com/1235651
Signed-off-by: Darshan N <dnarayan@redhat.com>
Reviewed-on: http://review.gluster.org/11503
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
| |
Change-Id: I920ed33f0e257dd741d1c6df39b46efebe9abd02
Signed-off-by: Darshan N <dnarayan@redhat.com>
Reviewed-on: http://review.gluster.org/11836
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added a server side plugin to change status from Pending
to OK/UNKNOWN depending on if volumes have server side
quorum turned on
Renamed service to ensure service definition gets
updated on upgrades.
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1106421
Change-Id: I1f1ca1dcf56872342ec8e25396a89163d22aedeb
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/8023
Reviewed-by: darshan n <dnarayan@redhat.com>
Reviewed-by: Ramesh N <rnachimu@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Wrongly used str.contains in python. Fixed this
Change-Id: Idea139329dc430294fc016b30db2578335bb3666
Bug-Url: https://bugzilla.redhat.com/1109752
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/9226
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Reviewed-by: kasturi narra <kasturi.narra@gmail.com>
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When plugin returns unknown status due to
transaction in progress, the plugin should try
again after some time to avoid propogating this
error to Nagios
Change-Id: I83fa7c3ec7382d1263e36995662df83b5ae39aa6
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/9192
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Bala FA <barumuga@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
service
During the creation of eligible list of hosts, need to consider the status
of glusterd service also. this is included. Also added some additional checks
to handle the errors
https://bugzilla.redhat.com/show_bug.cgi?id=1136205
https://bugzilla.redhat.com/show_bug.cgi?id=1109843
Signed-off-by: Nishanth Thomas <nthomas@redhat.com>
Change-Id: Ic757ecc0df5ef3c261048a961af59715ca889a26
Reviewed-on: http://review.gluster.org/9066
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Tested-by: Nishanth Thomas <nishusemail@gmail.com>
Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When the inital request fails, executeRandomHost is supposed to
iterate over the list of avialable hosts and resend the request
until get a successful response. This part of code was falty,
where the request was send without getting the proper
ipaddress/hostname
Change-Id: I1f1c7a5bccba3bd601c9e3241083784f15a2c4b6
Bug-url: https://bugzilla.redhat.com/show_bug.cgi?id=1109843
Signed-off-by: Nishanth Thomas <nthomas@redhat.com>
Reviewed-on: http://review.gluster.org/8101
Reviewed-by: Sahina Bose <sabose@redhat.com>
Tested-by: Nishanth Thomas <nishusemail@gmail.com>
|
|
|
|
|
|
|
|
| |
Change-Id: Ia0f570dfb74d6c41484e509f704df12d3b96db17
Signed-off-by: Timothy Asir Jeyasingh <tjeyasin@redhat.com>
Reviewed-on: http://review.gluster.org/8161
Tested-by: Timothy Asir <tim.gluster@gmail.com>
Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When a node in the cluster is down, nrpe request sent to the
node gets timed out resulting a failure in executeRandomHost.
This is fixed by selecting only those nodes which are UP
Change-Id: I5b22dcb30c0e2a006a549dc642b16db278c9c0f1
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1109025
Signed-off-by: Nishanth Thomas <nthomas@redhat.com>
Reviewed-on: http://review.gluster.org/8061
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Tested-by: Nishanth Thomas <nishusemail@gmail.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added the volume type in the plugin output
The logic of determining the volume status changed based on the
volume type.Added the volume type and the list of bricks down
in the plugin output
Change-Id: Ib8d3111bdcc04264ec8bb6383fcb4fad97a17bab
Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1096159
Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1096169
Signed-off-by: Nishanth Thomas <nthomas@redhat.com>
Reviewed-on: http://review.gluster.org/7874
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
Tested-by: Nishanth Thomas <nishusemail@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The exit status from NRPE plugin was wrapped with
WEXITSTATUS which was earlier used when running commands
directly.
With the subprocess implementation, this is not required
as it always returns 0 for successful execution even when
nrpe plugin returns 1/2.
Change-Id: I3cda8d4aa5b4f726cb5c7faf05c7f2536115eafe
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1101211
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/7905
|
|
|
|
|
|
|
|
|
|
| |
Passing the required arguments to the functions
instead of passing 'arg'.
Change-Id: Ib6cb654bc448c871a24dc636e6829ccf7523c2c8
Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-on: http://review.gluster.org/7756
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Introduced a utility method to return a NRPE base command
with timeout set externally.
Currently if a plugin internally executes NRPE to get the
details from the node, there is no mechanism that timeout
could be set to more than 10 sec. This method provides the
NRPE command with timeout (if passed).
This is required for some of the NRPE calls where gluster commands
get executed and they might take more time.
All the plugins, which execute a NRPE within, can provide
optional command line argument for timeout, and same can be used
for forming the proper NRPE call with timeout value.
Change-Id: Id97624df743664a320a585acc4a85cfcf64d0a07
Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-on: http://review.gluster.org/7682
Reviewed-by: Sahina Bose <sabose@redhat.com>
Tested-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
As the service name has changed, fixed the livestatus
query for Quota service
Change-Id: I4967429f357786aad5f6402293054a930371a52e
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/7673
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
service descriptions:
- Gluster Management
- NFS
- Self-Heal
- Quota
- Brick - <brick_dir>
Change-Id: I3ece94ef61fcb3fd264e848508362375842dcc6f
Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-on: http://review.gluster.org/7671
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently the brick description will have only brick path.
Changed the code to query based on volume name,cluster name.
Volume status will be decided based on state of the bricks in nagios.
Change-Id: If1e4f85f5643cf2b747ae8837b905f0d5b8d9b22
Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-on: http://review.gluster.org/7611
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Added server side plugin option for geo-rep
status
Change-Id: I6f604862ec7d4813298a670b7ed2cc0b448faf23
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/7592
|
|
|
|
|
|
|
|
|
|
|
| |
Changed the nrpe vol_status commands to pass arguments
for the different status - self-heal, quota etc
Change-Id: Ib26d445a582139fd9d5a07d2b9646a3517843548
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/7638
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
|
|
|
|
|
|
|
|
| |
Server side plugin to check the volume
self heal status
Change-Id: Ia1ee3fb653913360edb1b54b42cce3ed105bdb79
Signed-off-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
| |
Fix the aggregation issue when all bricks in CRITICAL state.
Change-Id: I8fba79fe77cb876780d4d00cd2992daee758b580
Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
When a volume is in Up state, Volume status will be decided
based on the status of the bricks.
CRITICAL - If all bricks are in CRITICAL state
WARNING - If some bricks are in CRITICAL state
OK - If all bricks are in OK state
Change-Id: Id90f8a5b1e0aaaa1de488b4356efbdd45fa43b7b
Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Fixed the error when there's no current
status for volume quota status plugin
Also corrected the livestatus query
Change-Id: Icb7fd0a363588c27c6237a6af397f07b5cfa6bdc
Signed-off-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Added a way to query for quota status to the
check_vol_server plugin
This will poll for quota status only if the
service status for quota returns non-OK status
Change-Id: I6260042ef54308a1f4f7b6d482a2b58a4682d36c
Signed-off-by: Sahina Bose <sabose@redhat.com>
|
|
Modified the existing volume utilization server plugin to
include status information as well.
Refactored the plugin with an argument -o|--option
to get the specific volume status.
Change-Id: I2b6f8612e32390fb20917098a255de1ee6b3a170
Signed-off-by: ndarshan <dnarayan@redhat.com>
Signed-off-by: Sahina Bose <sabose@redhat.com>
|