path: root/tests/
diff options
authorRamesh Nachimuthu <>2014-05-05 16:14:50 +0530
committerSahina Bose <>2014-05-12 02:58:51 -0700
commitfdc74de01faf52114578b64481c44d58abc06405 (patch)
tree19b93c999377ce19083e9e5a97049d0fa99322ce /tests/
parent9605a7b9f60369f72ad5f7f142c37802edf9db3f (diff)
autoconf: don't remove disconnected hosts from nagios
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: Change-Id: I3fe730d2545f8cbc8224c576712b09bb4c16e712 Signed-off-by: Ramesh Nachimuthu <> Reviewed-on: Reviewed-by: Sahina Bose <>
Diffstat (limited to 'tests/')
1 files changed, 5 insertions, 0 deletions
diff --git a/tests/ b/tests/
index 9d6d1ab..d54ddfe 100644
--- a/tests/
+++ b/tests/
@@ -17,6 +17,7 @@
from plugins import config_generator
+from glusternagios.glustercli import HostStatus
from testrunner import PluginsTestCase as TestCaseBase
@@ -116,10 +117,14 @@ class TestGlusterNagiosConfManager(TestCaseBase):
cluster = {'name': 'Test-Cluster', 'hosts': [], 'volumes': []}
cluster['hosts'].append({'hostip': '',
'hostname': 'host-1',
+ 'uuid': '0000-1111',
+ 'status': HostStatus.CONNECTED,
'bricks': self.createBricks(1, "Volume1",
cluster['hosts'].append({'hostip': '',
'hostname': 'host-2',
+ 'status': HostStatus.CONNECTED,
+ 'uuid': '0000-1112',
'bricks': self.createBricks(2, "Volume1",
cluster['volumes'].append({'name': 'Volume1', "type": "Replicate"})