| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Macro "$HOSTGROUPNAME$" is used in brick status event handler. But
this macro gives only one of the host group to which the host belongs
to. But in gluster monitoring, all gluster hosts will have two
host groups ('gluster-host' and a host group with cluster name).
So using the macro "$HOSTGROUPNAMES$" to pass all the host groups to
event handler and internally getting the currect cluster name.
Change-Id: I61713ecabff52bcd7f585e9f678426370b9b24d4
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7761
Reviewed-by: Nishanth Thomas <nishusemail@gmail.com>
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Timothy Asir <tim.gluster@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Create QUOTA and GEO Replication status services for volume only when
respective services are enabled in gluster.
Bug-Url: https://bugzilla.redhat.com/1099731
Change-Id: I9813a64f7d2573aaf3b23534643662e198b5457d
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7838
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently auto-config removes all the custom services and hosts
defined by user. So with this issue, user can't add any extra
services on top of services generated by auto-config.
Adding a custom filed '__GENERATED_BY_AUTOCONFIG' as part of
all the services added by auto-config. Only services with the
above custom field will be deleted by auto-config when
respective volume/brick/ is removed in gluster.
Change-Id: I85d3d971d3b99f52ff603044f7f974746e247737
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7750
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
User may have a host configuration defined already for the host being
discovered by auto config. So checking if the host already exist as part
of another cluster before auto configuring.
Same issue may happen when user gives a differnt cluster name in
autoconfig after running for the first time.
Change-Id: Ia3dbf8e4895cfe4d24e99eda49313b44dbd25aef
Bug-Url: https://bugzilla.redhat.com/1085320
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7755
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Change-Id: I7e81e22e73dc4cf336ad335f69735d54b9e9714a
Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
Reviewed-on: http://review.gluster.org/7807
Tested-by: Timothy Asir <tim.gluster@gmail.com>
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Reviewed-by: Timothy Asir <tim.gluster@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Passing the timeout parameter to writeDelta() in auto mode also.
Change-Id: Ia18ae225e5b6e239b4d4b09357d51844d5a868f5
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7817
Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Timothy Asir <tim.gluster@gmail.com>
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Disabled the gluster host service check_command service
check_remote_host
Change-Id: If71ed9d42ef65a9a9b36668fed7b449745cc5df1
Bug-URL: https://bugzilla.redhat.com/show_bug.cgi?id=1098468
Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-on: http://review.gluster.org/7797
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Timothy Asir <tim.gluster@gmail.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>
|
|
|
|
|
|
|
|
| |
Change-Id: If55d9a0f36fbb16136fe46dbbdc31b2f52228a7b
Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
Reviewed-on: http://review.gluster.org/7691
Tested-by: Timothy Asir <tim.gluster@gmail.com>
Reviewed-by: Timothy Asir <tim.gluster@gmail.com>
|
|
|
|
|
|
|
|
| |
Change-Id: I2f3b18c730ffd3ab142763433e0c75f9214eef1e
Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
Reviewed-on: http://review.gluster.org/7685
Tested-by: Timothy Asir <tim.gluster@gmail.com>
Reviewed-by: Timothy Asir <tim.gluster@gmail.com>
|
|
|
|
|
|
|
|
|
|
| |
Modified to open NSCA port while installation
Bug-URL: https://bugzilla.redhat.com/show_bug.cgi?id=1097193
Change-Id: I52edd1cfde7d61945ae4bdc4f6339fe246713959
Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-on: http://review.gluster.org/7758
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
BZ 1089641, changed the mk-livestatus query so that
it picks the right services
BZ 1089670, introduced a check to set status to OK if
no volumes present in the cluster
Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1089670
Bug:Url:https://bugzilla.redhat.com/show_bug.cgi?id=1089641
Change-Id: I93aed685c75ff3455e1129c0d8f9e485b6c5f2b0
Signed-off-by: Nishanth Thomas <nthomas@redhat.com>
Reviewed-on: http://review.gluster.org/7708
Reviewed-by: Sahina Bose <sabose@redhat.com>
Tested-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Volume Utilization plugin didn't have a php template written and
hence the graph was not getting displayed properly
Change-Id: I6726a6de5370d271467be03e9e8500421cf66e39
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1095427
Signed-off-by: Nishanth Thomas <nthomas@redhat.com>
Reviewed-on: http://review.gluster.org/7724
Tested-by: Nishanth Thomas <nishusemail@gmail.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently auto-config removes all the hosts with disconnected
status in 'peer status' from nagios configuration. But that
is not correct as a host can goto disconnected status for
many differnt reasons and we should not remove the configuration.
Similarly disconnected hosts need not be added to nagios
configuration during auto discovery.
Fixing this by ingnoring all the disconnected hosts in delta
calculation.
Note: Host name can't be used to identify disconnected hosts
as we may not be able to communicate the disconnected host
to get the host name. Hence Host UUID in peer status is
used to intentify the host.
Bug-Url: https://bugzilla.redhat.com/1091170
Change-Id: I3fe730d2545f8cbc8224c576712b09bb4c16e712
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7677
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
The self heal service is only added for a replicate
volume type. Corrected the comparison to do so.
Change-Id: I78c783bbe9b2f17dc1d5cfa7d86d9a69690be3cd
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/7711
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Tested-by: Ramesh N <rnachimu@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Modified graph templates for memory and swap to handle the
unit change in respective plugins .
https://bugzilla.redhat.com/show_bug.cgi?id=1080004
Signed-off-by: ndarshan <dnarayan@redhat.com>
Change-Id: Id11dee8992b7a4ba93df0fb107fa73e29035088e
Reviewed-on: http://review.gluster.org/7641
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Changed the serice name to SMB
Change-Id: I717b538d0ebff04c288dc19396e42151f9e7bec1
Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-on: http://review.gluster.org/7704
Reviewed-by: Timothy Asir <tim.gluster@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Adding checks for gluster process and bricks.
Change-Id: Ie688233db2a5da0d1a26d5576507559e7e8f5d2d
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7697
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Disabled the event handler for gluster hosts
Change-Id: Ib81245add29126761185dec110015ecb73449a3a
Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-on: http://review.gluster.org/7664
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Reviewed-by: Timothy Asir <tim.gluster@gmail.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>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added a quorum service to auto-config generator.
Renamed "Volume Status Quota" to "Volume Quota"
service
Change-Id: Id83eefeee63bf7c34373196f4cfa2b326e1cb71d
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/7661
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Ramesh N <rnachimu@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Handling the exceptions and failure from NRPE commands.
Change-Id: I36cba2d6adf8484ba134ad3f7aec77437ba07857
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7659
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Change-Id: Ic2e285e758d0a15612f3e14d4f0dbd34e28aaac5
Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
Reviewed-on: http://review.gluster.org/7650
Tested-by: Timothy Asir <tim.gluster@gmail.com>
Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Sending a cusotm email notification whenever nagios configurations
are changed by auto-config service.
note: By default, nagios mail notification command 'notify-service-by-email'
doesn't include the comments macro. But in auto config, all the changes
are sent as comment to the custom notificiation command. Hence nagios
command 'notify-service-by-email' should be changed to include
the macro '$NOTIFICATIONCOMMENT$' as part of mail template.
Change-Id: Ie5dd23578e08fbc757c46a884a79923ff3c403b2
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7626
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Removed ovirt contact from host and service
as sending notifications to ovirt is not ready yet.
Change-Id: Ic62b5b62c1a32e3cdc068ad67ec60b781b1effde
Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-on: http://review.gluster.org/7627
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>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
NRPE doesn't support transfering large junk of data as a result.
Hence we have to discover the volume details one by one. First fetch
the volume list using 'discover_volume_list' NRPE command then fetch the
brick details of volume one by one using 'discover_volume_info' NRPE
command
Change-Id: I2cd3c2309ffa2f1bf5c271e06dd03ade75bf5d92
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7631
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>
|
|
|
|
|
|
|
|
|
| |
fixing a pep8 error in discovery.py
Change-Id: I2cbacea466e0d178636c00d7212d107fd82f2719
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7628
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
There were some key errors when there is no service in the hosts.
It fixed by using dict.get().
Change-Id: Ic479f5335240c772a520e4ee0e7b41ce85f905e5
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7594
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Addning an NRPE command to configure all the newly added
nodes to used passive check results using NSCA. It is done using
the NRPE command 'configure_gluster_node'.
Change-Id: I75d0214c8c52afdedd13dac3f38d52abc4ca5a93
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7593
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change the brick service description to
"Brick Status - <brick-path>". Currently it is
"Brick Status - <hostaddress>:<brick-path>". Same case for
Brick utilization service also.
Change-Id: I3924f15e3b8b86826c0019c5ab52bd371a970cf1
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7597
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
| |
Change-Id: I8b662cb36921d62f14eb9af7660585ffc30f94d1
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Enhace the auto configuration in nagios to preserve the user
changes. pynag apis are used to read/write/update the nagios config
files.
Change-Id: I6820928fb2b0baff5d6e65c811a48b5ff718f9e3
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
|
|
|
|
|
|
|
| |
Added BuildRequires for required packages.
Change-Id: Ie3c1fbc27b93c582a3caf0a61a07897c7892880c
Signed-off-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>
|
|
|
|
|
|
|
| |
Using host UUIDs to bricks in host
Change-Id: Ie5467b05391f709e292644f00495b41af8213b42
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
|
|
|
|
|
|
|
|
| |
Discover the host name configured for the gluster node and
use the same as host_name in nagios host configuration.
Change-Id: Ib9eb8b3f3a1a03d1be28fa2faba44c2fc81fa0cf
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Renamed the file to snmpmanagers.conf, as .cfg files
nagios specific files.
Changed the permission to nagios:nagios, as trapgenerater
fails otherwise.
Change-Id: I2fbc4404c7177ee3bcaeab3151cbcba8e062772a
Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Adding an event handler for brick status services. It will
reschedule the volume status service to determine the current
status of volume after any brick state change
Change-Id: I94fa018e5aed15be53f45506f1a38efd698eca79
Signed-off-by: Ramesh Nachimuthu <rnachimu@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>
|
|
|
|
|
| |
Change-Id: Iba14a8af63741a5a5014baccaa5b37aad97c7657
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Currently there is no way to identify the volume from brick
services. Hence adding volume name as custom variable and note in
brick services
Change-Id: If1ef000196dca2239f713fe41d033d8a4161d6c0
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
|
|
|
|
|
|
|
|
| |
Added a dependency on nsca . required by
passive check plugins
Change-Id: If9c5b50adcbe8803d36bce3bc5f9bf0e20c07256
Signed-off-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
These plugins will generate SNMP traps when there is a host
or service state change in Nagios. Plugged into the Nagios
Notification mechanism and uses netsnmp utils to generate
SNMP traps
Addressed the review comments
Change-Id: I42c4d1968a48bc80e767f6fbc24d1637a92d21b0
Signed-off-by: Nishanth Thomas <nthomas@redhat.com>
|
|
|
|
|
|
|
|
| |
Volume Quota status service was not appended to the service list.
Fixed it now.
Change-Id: Ia327deefc855619848a7d9ba0e41c155e2c15aa0
Signed-off-by: Kanagaraj M <kmayilsa@redhat.com>
|