From e85d1f911c4f5b0f4eea32cbec7f42d9f01d630f Mon Sep 17 00:00:00 2001 From: Koushik Das Date: Wed, 9 Jul 2014 12:27:47 +0530 Subject: [PATCH] Some minor fixes in simulator code --- .../src/com/cloud/agent/manager/MockStorageManagerImpl.java | 6 +++--- .../src/com/cloud/resource/SimulatorDiscoverer.java | 5 ----- .../com/cloud/resource/SimulatorSecondaryDiscoverer.java | 3 --- 3 files changed, 3 insertions(+), 11 deletions(-) diff --git a/plugins/hypervisors/simulator/src/com/cloud/agent/manager/MockStorageManagerImpl.java b/plugins/hypervisors/simulator/src/com/cloud/agent/manager/MockStorageManagerImpl.java index 8e1aaed8470..89b79506cdf 100644 --- a/plugins/hypervisors/simulator/src/com/cloud/agent/manager/MockStorageManagerImpl.java +++ b/plugins/hypervisors/simulator/src/com/cloud/agent/manager/MockStorageManagerImpl.java @@ -289,11 +289,11 @@ public class MockStorageManagerImpl extends ManagerBase implements MockStorageMa vm = _mockVMDao.findByVmName(vmName); txn.commit(); if (vm == null) { - return new Answer(cmd, false, "can't vm :" + vmName); + return new Answer(cmd, false, "can't find vm :" + vmName); } } catch (Exception ex) { txn.rollback(); - throw new CloudRuntimeException("Error when attaching iso to vm " + vm.getName(), ex); + throw new CloudRuntimeException("Error when attaching iso to vm " + vmName, ex); } finally { txn.close(); txn = TransactionLegacy.open(TransactionLegacy.CLOUD_DB); @@ -514,8 +514,8 @@ public class MockStorageManagerImpl extends ManagerBase implements MockStorageMa if (cmd.getVmName() != null) { MockVm vm = _mockVMDao.findByVmName(cmd.getVmName()); - vm.setState(State.Expunging); if (vm != null) { + vm.setState(State.Expunging); MockVMVO vmVo = _mockVMDao.createForUpdate(vm.getId()); _mockVMDao.update(vm.getId(), vmVo); } diff --git a/plugins/hypervisors/simulator/src/com/cloud/resource/SimulatorDiscoverer.java b/plugins/hypervisors/simulator/src/com/cloud/resource/SimulatorDiscoverer.java index d1ebfb557fd..7c31a9af754 100644 --- a/plugins/hypervisors/simulator/src/com/cloud/resource/SimulatorDiscoverer.java +++ b/plugins/hypervisors/simulator/src/com/cloud/resource/SimulatorDiscoverer.java @@ -41,7 +41,6 @@ import com.cloud.agent.api.StartupRoutingCommand; import com.cloud.agent.manager.MockAgentManager; import com.cloud.agent.manager.MockStorageManager; import com.cloud.dc.ClusterVO; -import com.cloud.dc.dao.ClusterDao; import com.cloud.exception.ConnectionException; import com.cloud.exception.DiscoveryException; import com.cloud.host.Host; @@ -65,15 +64,11 @@ public class SimulatorDiscoverer extends DiscovererBase implements Discoverer, L @Inject VMTemplateZoneDao _vmTemplateZoneDao; @Inject - ClusterDao _clusterDao; - @Inject AgentManager _agentMgr = null; @Inject MockAgentManager _mockAgentMgr = null; @Inject MockStorageManager _mockStorageMgr = null; - @Inject - ResourceManager _resourceMgr; /** * Finds ServerResources of an in-process simulator diff --git a/plugins/hypervisors/simulator/src/com/cloud/resource/SimulatorSecondaryDiscoverer.java b/plugins/hypervisors/simulator/src/com/cloud/resource/SimulatorSecondaryDiscoverer.java index 715e9f37f61..11bf3466105 100644 --- a/plugins/hypervisors/simulator/src/com/cloud/resource/SimulatorSecondaryDiscoverer.java +++ b/plugins/hypervisors/simulator/src/com/cloud/resource/SimulatorSecondaryDiscoverer.java @@ -31,7 +31,6 @@ import org.apache.cloudstack.storage.resource.SecondaryStorageDiscoverer; import org.apache.cloudstack.storage.resource.SecondaryStorageResource; import org.apache.log4j.Logger; -import com.cloud.agent.AgentManager; import com.cloud.agent.Listener; import com.cloud.agent.api.AgentControlAnswer; import com.cloud.agent.api.AgentControlCommand; @@ -52,8 +51,6 @@ public class SimulatorSecondaryDiscoverer extends SecondaryStorageDiscoverer imp @Inject MockStorageManager _mockStorageMgr = null; @Inject - AgentManager _agentMgr; - @Inject SnapshotDao _snapshotDao; @Inject ImageStoreDao imageStoreDao;