From 09cefaf3e9912d8e99d4f9a131b50cc409593de8 Mon Sep 17 00:00:00 2001
From: anthony <anthony@cloud.com>
Date: Mon, 11 Oct 2010 12:04:56 -0700
Subject: [PATCH]     bug 6500: add back the special handle for secodary
 storage host, it is in the same path of handling xenserver host, it is easy
 to break that, so I add some comments there

    status 6500: resolved fixed
---
 server/src/com/cloud/agent/manager/AgentManagerImpl.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/server/src/com/cloud/agent/manager/AgentManagerImpl.java b/server/src/com/cloud/agent/manager/AgentManagerImpl.java
index 7e94163a6b5..dedca6a8e46 100755
--- a/server/src/com/cloud/agent/manager/AgentManagerImpl.java
+++ b/server/src/com/cloud/agent/manager/AgentManagerImpl.java
@@ -938,8 +938,10 @@ public class AgentManagerImpl implements AgentManager, HandlerFactory {
         ReadyCommand ready = new ReadyCommand(dcId);
         Answer answer = easySend(hostId, ready);
         if (answer == null) {
+            // this is tricky part for secondary storage
+            // make it as disconnected, wait for secondary storage VM to be up
+            // return the attache instead of null, even it is disconnectede
             handleDisconnect(attache, Event.AgentDisconnected, false);
-            return null;
         }
         
         _hostDao.updateStatus(host, Event.Ready, _nodeId);