mirror of
https://github.com/apache/cloudstack.git
synced 2025-11-02 11:52:28 +01:00
Some minor fixes in simulator code
This commit is contained in:
parent
f275eb1ff9
commit
e85d1f911c
@ -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);
|
||||
}
|
||||
|
||||
@ -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
|
||||
|
||||
@ -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;
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user