| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
|
|
| |
Renaming self-heal status plugin to split-brain
status
Change-Id: Icdb934904eda9c47ccdc4ba41a9204048918f0a9
Bug-Url: https://bugzilla.redhat.com/1221872
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/10908
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rename the default NFS service in gluster-host to
Gluster NFS.
Bug-Url: https://bugzilla.redhat.com/1127657
Change-Id: I33e484357ec81255d98e4c4a42d24bef85e86d25
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/10919
Reviewed-by: darshan n <dnarayan@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently Nagios server address entered by the user during
auto config was not being verified. This patch helps to verify
the address entered by the user.
If IP address is given as the nagios server address then it
checks the pattern and verifies that it is mapped to one of the
non loopback device in the host
If user enters fqdn name, then it tries to resolve it,
also it verifies that resolved IP address maps to one of
the non loopback device in the host.
Bug-Url: https://bugzilla.redhat.com/1127657
Change-Id: I88d67cc6d8fa05f2934922fbc0d8e757b1d73e43
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7740
Reviewed-by: darshan n <dnarayan@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Replaced disk utilization graph legend name from Brick Usage
to Usage.
Change-Id: Id022d014530108766fcb50ed0a60d6f4c76c93f3
Bug-Url: bugzilla.redhat.com/1167771
Signed-off-by: ndarshan <dnarayan@redhat.com>
Reviewed-on: http://review.gluster.org/9754
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Reviewed-by: Bala FA <barumuga@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch modifies the graph template for brick utilization
and disk utilization to handle the changes made by check_disk
_and_inode plugin revamp. Now separate graphs for data, inode,
thinpool and thinpool-metadata utilization are shown.
Change-Id: Iad7da8ab3fede89bb528ceadfd817646c018795e
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1167771
Signed-off-by: ndarshan <dnarayan@redhat.com>
Reviewed-on: http://review.gluster.org/9394
Reviewed-by: Bala FA <barumuga@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Provide a command to perform auto-discovery.
Change-Id: I26aa48532f4e8c9f305c36294f48cddf5785b491
Bug-URL: https://bugzilla.redhat.com/1164682
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/9514
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Bala FA <barumuga@redhat.com>
Reviewed-by: Sahina Bose <sabose@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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
By default perfdata is created in single rrd file which causes
failure of fetching DS in upgrade scenario. Because earlier
brick utilization had only one DS. Now with thin LV monitoring
included in brick utilization, It will be having two DS. The
graph templates wont be able to read them in single rrd file.
So in case of upgrade we run the script rrd_convert as a pre
step to migrate the old data to new rrd files.and this patch
also sets pnp4nagios rrd configuration to multiple rrd file for
each DS.
Change-Id: Iadb50250b2e93d73b61e25be90a0770dae0780f3
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1143995
Signed-off-by: ndarshan <dnarayan@redhat.com>
Reviewed-on: http://review.gluster.org/9188
Reviewed-by: Sahina Bose <sabose@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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Modified brick utilization graph template to give
an additional graph to show the actual brick utilization
when the brick is made up of thin lv.
Change-Id: I0ecc2de9d0b6ca72b7a4ed87c342f242fb035598
Bug-Url:https://bugzilla.redhat.com/show_bug.cgi?id=1143995
Signed-off-by: ndarshan <dnarayan@redhat.com>
Reviewed-on: http://review.gluster.org/9116
Reviewed-by: Nishanth Thomas <nishusemail@gmail.com>
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Reviewed-by: Sahina Bose <sabose@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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
There was no php template available for cluster utilization and it was
using the the default template and causing the improper display
Change-Id: I21f458a678d12807d275cc536e56aa14eae22a80
Bug-url: https://bugzilla.redhat.com/show_bug.cgi?id=1119233
Signed-off-by: Nishanth Thomas <nthomas@redhat.com>
Reviewed-on: http://review.gluster.org/9002
Tested-by: Nishanth Thomas <nishusemail@gmail.com>
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Unregistering the temp_node1 instead of deleting it
while configuring nagios .
new field 'register' will be added to temp_node1 and its
services with value 0, so that it won't be considered by
nagios.
If both files, temp_node1.cfg and temp_node1.cfg.sample
are present during upgrade then we will remove the file
temp_node1.cfg.sample and add the 'register 0'
flag to definitions in temp_node1.cfg file.
Bug-Url: https://bugzilla.redhat.com/1162446
Bug-Url: https://bugzilla.redhat.com/1162443
Bug-Url: https://bugzilla.redhat.com/1115387
Change-Id: I3c76d6c6a9a9e411315859e07238a7902ddd2380
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/9123
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit 11f96adbb1f8bff8cf7b385da10200a5627d980a.
Bug-Url:https://bugzilla.redhat.com/1162446
Bug-Url:https://bugzilla.redhat.com/1162443
Change-Id: I831bb0c84721a0c3181a2d58421520a6fb788621
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/9122
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Consider the 'UNKNOWN' status of volumes while calculating the cluster
status. Following will be the cluster state and state information.
Cluster State State Information
UP "OK : None of the Volumes in the cluster are in Critical State"
UP "OK : No Volumes present in the cluster"
UP "WARNING : Some Volumes in the cluster are in Critical State"
UP "WARNING : Some Volumes in the cluster are in Unknown State"
UP "WARNING : Some Volumes in the cluster are in Warning State"
UP "WARNING : All Volumes in the cluster are in Warning State"
DOWN "CRITICAL: All Volumes in the cluster are in Critical State"
DOWN "CRITICAL: All Volumes in the cluster are in Unknown State"
Bug-url: https://bugzilla.redhat.com/1128007
Change-Id: I06fb1697cb4919420ab6d6ea54e9d9ee96820d2a
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/9053
Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Installing temp_node1.cfg conditionally during first installation
only. It won't be re-installed during upgrade.
Change-Id: I26cc945d5b8b650d8457b7232d9dc0f3c5eac105
Bug-Url: https://bugzilla.redhat.com/1115387
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/8303
Reviewed-by: Sahina Bose <sabose@redhat.com>
Tested-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Modified the error message for the case when there
is already a host with same name existing.
Change-Id: I2e6b5ef80baa0a45d94f069980ac81b1fbd8c058
Bug-Url: https://bugzilla.redhat.com/1112201
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/8658
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Auto Config tries to use the output of 'hostname' command as
a host_name for node in nagios. But if the hostnames are not unique,
then it uses IP Address as node_name name in nagios.
If after removal of a node, hostnames becomes unique then
auto config renames all the host configuration in nagios to their
hostnames instead of ip address. It creates multiples issues like
removing all the nodes, loss of time series data and unnecessary
confusion to the user.
Fixing this issue by retaining the perviously configured host
names always by going through the host uuid. So if duplication
comes after a node addition then only that node will be
configured with ip address as host_name but all other existing
hosts will not be touched.
Similary, even though there is no duplication after the removal
of a node with duplicate name, auto config will retain the
previously configured hostnames (which are ip addresss
becasue there was a hostname duplication earlier) instead of
renaming.
Change-Id: I0abef47e2e495a591dd3901d2b1766a04c59a701
Bug-Url: https://bugzilla.redhat.com/1139228
Bug-Url: https://bugzilla.redhat.com/1139505
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/8699
Reviewed-by: darshan n <dnarayan@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>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Verify the Nagios configuration before restarting the nagios
server. It helps in accidently stoping the nagios server because of
config error.
Change-Id: I8c7cb5663bf42d73812577e9b162434cb2ecfdfe
Bug-Url: https://bugzilla.redhat.com/1138943
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/8131
Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
The directive for check attempts should be max_check_attempts
Change-Id: Id7cb814a55dbde714aa9cf570150e5d0158763e4
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1128154
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/8482
Reviewed-by: Ramesh N <rnachimu@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Passive check results that result in state changes
need to be treated as HARD states, so that notifications
can be send out immediately
Change-Id: I747b0f1301ee9321adb559e02a47e30fd4e4a346
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1128154
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/8467
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Fixing typo 'NPRE' in server_utils.execNRPECommand()
Bug-Url: https://bugzilla.redhat.com/1108569
Change-Id: I9b29780661353eb0b82ef24a390c2d2091f7ea4e
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/8130
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
| |
Adding %config(noreplace) flag for config files so that config
files will not be over written during RPM upgarde.
Change-Id: Iff5ff8f8df589745e9d69bc22f8bf10674d1f49f
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/8220
Reviewed-by: Bala FA <barumuga@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
By default perfdata are created in single rrd file which causes
failure of fetching DS for newly configured disks.
This patch fixes this issue by setting pnp4nagios rrd
configuration to multiple.
Change-Id: I05ca0561e9b926a3ee0c655699fa30544c20e2ad
Signed-off-by: Timothy Asir Jeyasingh <tjeyasin@redhat.com>
Reviewed-on: http://review.gluster.org/8169
Tested-by: Timothy Asir <tim.gluster@gmail.com>
Reviewed-by: Sahina Bose <sabose@redhat.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>
|
|
|
|
|
|
|
|
| |
Change-Id: Ibb9fcdcb7f5a2c951011883f54ec14c209049555
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/8144
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Tested-by: Kanagaraj M <kmayilsa@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When a host is in disconnected status, we will not communicate
with the host and will not be able to get the hostname. So
we don't need to consider those disconnected hosts in finding
duplicate hosts.
Change-Id: Ie3fbef03635d46fd9d95e5b1f4df7435e800c55d
Bug-Url: https://bugzilla.redhat.com/1109723
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/8074
Reviewed-by: Sahina Bose <sabose@redhat.com>
Tested-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
By default perfdata are created in single rrd file which causes
failure of fetching DS for newly configured network interfaces.
This patch fixes this by setting pnp4nagios rrd configuration to
multiple rrd file for each DS of network interfaces.
Change-Id: I3e6fa0e4336fb866ceb37a5e3165a515d42bfae6
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1087837
Signed-off-by: Bala.FA <barumuga@redhat.com>
Reviewed-on: http://review.gluster.org/8059
Reviewed-by: Sahina Bose <sabose@redhat.com>
Tested-by: Sahina Bose <sabose@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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adding chkconfig for nagios and nsca services. By default nagios
and nsca is disabled in all the run levels. So we have to enable
them by running 'chkconfig <service> on' during installation.
Change-Id: I8527abca24ec286f85c425db9cbeb216047eaa97
Bug-Url: https://bugzilla.redhat.com/1080449
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/8012
Reviewed-by: Bala FA <barumuga@redhat.com>
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Timothy Asir <tim.gluster@gmail.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Auto config uses a user configured host to periodically sync the
gluster configuration. If the host used for sync itself is
removed from the cluster, then auto config will detect this
as all other hosts are getting removed from cluster.
Fixing this issue by not removing all the hosts except sync
host in auto mode. User has to run the auto config
manually with differnt host to remove sync host.
Change-Id: I18fa51b4ed9674264ea8b2d2f615f7680a59c04e
Bug-Url: https://bugzilla.redhat.com/1107998
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/8024
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
| |
Change-Id: Ibb601ace97c34a4bd91f2f9658d305a68a7aacaa
Bug-Url: https://bugzilla.redhat.com/1085291
Signed-off-by: Timothy Asir <tjeyasin@redhat.com>
Reviewed-on: http://review.gluster.org/7895
Reviewed-by: Sahina Bose <sabose@redhat.com>
Tested-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>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch replaces the configfile node1.cfg(gluster-host)
by temp_node1.cfg which is of type linux-server having
service PING. And this dummy config file is renamed to
temp_node1.cfg.sample during auto-discovery as it is not
needed after we configure actual hosts. Also this patch
sets the config variable allow_empty_hostgroup_assignment
to 1, which is needed for nagios to allow declaration of
empty host-group with services.
Change-Id: I03cc54975f5ff5857f8d82b66110e049e4195009
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1091236
Signed-off-by: ndarshan <dnarayan@redhat.com>
Reviewed-on: http://review.gluster.org/7952
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
| |
Added requires for mod_ssl. The same is required for SSL
enabling for nagios and pnp4nagios.
Change-Id: Ib391ebb7008396ea34e045cc1cc4041305470a14
Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-on: http://review.gluster.org/7935
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Enabled to execute check_nrpe for a given host to check
its status.
Change-Id: I938b78fcbf52cd46f4f493e2c2b8b927614834eb
Bug-URL: https://bugzilla.redhat.com/show_bug.cgi?id=1102506
Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-on: http://review.gluster.org/7923
Reviewed-by: Ramesh N <rnachimu@redhat.com>
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Timothy Asir <tim.gluster@gmail.com>
|
|
|
|
|
|
|
|
|
|
| |
Enabled redirecting nagios and pnp4nagios urls to https
Change-Id: I3d1e62438c7265aca6cd930f37fd4aac08d83f76
Signed-off-by: Shubhendu Tripathi <shtripat@redhat.com>
Reviewed-on: http://review.gluster.org/7906
Reviewed-by: Bala FA <barumuga@redhat.com>
Reviewed-by: Timothy Asir <tim.gluster@gmail.com>
|
|
|
|
|
|
|
|
|
|
| |
Corrected the message returned by plugin when all volumes in
cluster are stopped
Change-Id: Ie5e0fbbb10fa7d2787a82e1caa7ca86e8c7010a3
Bug-Url: https://bugzilla.redhat.com/1096730
Signed-off-by: Sahina Bose <sabose@redhat.com>
Reviewed-on: http://review.gluster.org/7780
|
|
|
|
|
|
|
|
|
|
|
|
| |
Mocking pynag using dicts in test cases
Change-Id: I7ddedb640910c8dca0b79d17efe5e6eeefc9ee6f
Bug-Url: https://bugzilla.redhat.com/1099093
Bug-Url: https://bugzilla.redhat.com/1099328
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7921
Reviewed-by: Kanagaraj M <kmayilsa@redhat.com>
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
| |
Refactoring the code to use constants for repeatedly used
strings and add some test case.
Change-Id: I7c88231a51ceea3b1cc94223018ecc704b657512
Bug-Url: https://bugzilla.redhat.com/1099093
Bug-Url: https://bugzilla.redhat.com/1099328
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7883
Reviewed-by: Sahina Bose <sabose@redhat.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently Auto-config won't update any service configurations
in nagios. It always adds or deletes services configurations but
no update. With the following bugs we have the need to update few
values in service configuration. Hence adding the support to update
specific fields in service config.
All the fields in the list config_generator.SERVICE_FIELDS_TO_FORCE_SYNC
will be force updated during auto configuration. With this patch only
'_VOL_NAME' and 'notes' field will be updated.
Change-Id: I6003ade375d2f0d6e7dee97ef05f7a9af4ce6ffd
Bug-Url: https://bugzilla.redhat.com/1099093
Bug-Url: https://bugzilla.redhat.com/1099328
Signed-off-by: Ramesh Nachimuthu <rnachimu@redhat.com>
Reviewed-on: http://review.gluster.org/7868
Reviewed-by: Sahina Bose <sabose@redhat.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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|