diff --git a/engine/storage/integration-test/test/org/apache/cloudstack/storage/test/DirectAgentManagerSimpleImpl.java b/engine/storage/integration-test/test/org/apache/cloudstack/storage/test/DirectAgentManagerSimpleImpl.java index 220a7b01f6f..6941ff8cfdc 100644 --- a/engine/storage/integration-test/test/org/apache/cloudstack/storage/test/DirectAgentManagerSimpleImpl.java +++ b/engine/storage/integration-test/test/org/apache/cloudstack/storage/test/DirectAgentManagerSimpleImpl.java @@ -175,18 +175,6 @@ public class DirectAgentManagerSimpleImpl extends ManagerBase implements AgentMa return null; } - @Override - public void sendToSecStorage(HostVO ssHost, Command cmd, Listener listener) throws AgentUnavailableException { - // TODO Auto-generated method stub - - } - - @Override - public Answer sendToSecStorage(HostVO ssHost, Command cmd) { - // TODO Auto-generated method stub - return null; - } - @Override public boolean tapLoadingAgents(Long hostId, TapAgentsAction action) { diff --git a/server/src/com/cloud/agent/AgentManager.java b/server/src/com/cloud/agent/AgentManager.java index e4d55979191..02dd10d810f 100755 --- a/server/src/com/cloud/agent/AgentManager.java +++ b/server/src/com/cloud/agent/AgentManager.java @@ -137,10 +137,6 @@ public interface AgentManager extends Manager { Answer sendTo(Long dcId, HypervisorType type, Command cmd); - void sendToSecStorage(HostVO ssHost, Command cmd, Listener listener) throws AgentUnavailableException; - - Answer sendToSecStorage(HostVO ssHost, Command cmd); - /* working as a lock while agent is being loaded */ public boolean tapLoadingAgents(Long hostId, TapAgentsAction action); diff --git a/server/src/com/cloud/agent/manager/AgentManagerImpl.java b/server/src/com/cloud/agent/manager/AgentManagerImpl.java index a32e0103d75..fb857a707c8 100755 --- a/server/src/com/cloud/agent/manager/AgentManagerImpl.java +++ b/server/src/com/cloud/agent/manager/AgentManagerImpl.java @@ -377,32 +377,6 @@ public class AgentManagerImpl extends ManagerBase implements AgentManager, Handl } - @Override - public Answer sendToSecStorage(HostVO ssHost, Command cmd) { - if( ssHost.getType() == Host.Type.LocalSecondaryStorage ) { - return easySend(ssHost.getId(), cmd); - } else if ( ssHost.getType() == Host.Type.SecondaryStorage) { - return sendToSSVM(ssHost.getDataCenterId(), cmd); - } else { - String msg = "do not support Secondary Storage type " + ssHost.getType(); - s_logger.warn(msg); - return new Answer(cmd, false, msg); - } - } - - @Override - public void sendToSecStorage(HostVO ssHost, Command cmd, Listener listener) throws AgentUnavailableException { - if( ssHost.getType() == Host.Type.LocalSecondaryStorage ) { - send(ssHost.getId(), new Commands(cmd), listener); - } else if ( ssHost.getType() == Host.Type.SecondaryStorage) { - sendToSSVM(ssHost.getDataCenterId(), cmd, listener); - } else { - String err = "do not support Secondary Storage type " + ssHost.getType(); - s_logger.warn(err); - throw new CloudRuntimeException(err); - } - } - private void sendToSSVM(final long dcId, final Command cmd, final Listener listener) throws AgentUnavailableException { List ssAHosts = _ssvmMgr.listUpAndConnectingSecondaryStorageVmHost(dcId); diff --git a/server/test/com/cloud/agent/MockAgentManagerImpl.java b/server/test/com/cloud/agent/MockAgentManagerImpl.java index 3ebb5495114..01f4e9cb071 100755 --- a/server/test/com/cloud/agent/MockAgentManagerImpl.java +++ b/server/test/com/cloud/agent/MockAgentManagerImpl.java @@ -131,16 +131,6 @@ public class MockAgentManagerImpl extends ManagerBase implements AgentManager { return null; } - @Override - public void sendToSecStorage(HostVO ssHost, Command cmd, Listener listener) { - } - - @Override - public Answer sendToSecStorage(HostVO ssHost, Command cmd) { - // TODO Auto-generated method stub - return null; - } - @Override public boolean tapLoadingAgents(Long hostId, TapAgentsAction action) {