etan39
/
cloudstack
mirror of https://github.com/apache/cloudstack.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

hypervisor type for deployvmCmd is optional, only deploying blankvm 

needs it.
This commit is contained in:
edison 2010-10-05 09:51:18 -07:00
parent f7f2b5666a
commit 1cd370fc36
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
server/src/com/cloud/api/commands/DeployVMCmd.java
View File

@ -59,7 +59,7 @@ public class DeployVMCmd extends BaseCmd {
s_properties.add(new Pair<Enum, Boolean>(BaseCmd.Properties.DOMAIN_ID, Boolean.FALSE));
s_properties.add(new Pair<Enum, Boolean>(BaseCmd.Properties.NETWORK_GROUP_LIST, Boolean.FALSE));
s_properties.add(new Pair<Enum, Boolean>(BaseCmd.Properties.SIZE, Boolean.FALSE));
s_properties.add(new Pair<Enum, Boolean>(BaseCmd.Properties.HYPERVISOR, Boolean.TRUE));
s_properties.add(new Pair<Enum, Boolean>(BaseCmd.Properties.HYPERVISOR, Boolean.FALSE));
}
@Override
@ -110,7 +110,10 @@ public class DeployVMCmd extends BaseCmd {
if (template.getFormat().equals(ImageFormat.ISO)) {
/*TODO:Hack here*/
template.setHypervisorType(HypervisorType.XenServer);
if (hyperType == HypervisorType.None) {
throw new ServerApiException(BaseCmd.VM_INVALID_PARAM_ERROR, "Need hypervisorType to deploy blankvm");
}
template.setHypervisorType(hyperType);
}
if (diskOfferingId != null) {