Skip to content

Commit 91a5f0e

Browse files
server: honor global setting system.vm.default.hypervisor as first option when deploy VRs (#6160)
1 parent 19a7774 commit 91a5f0e

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

server/src/main/java/com/cloud/network/router/NetworkHelperImpl.java

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -567,20 +567,18 @@ protected String getNoHypervisorsErrMsgDetails() {
567567
protected List<HypervisorType> getHypervisors(final RouterDeploymentDefinition routerDeploymentDefinition) throws InsufficientServerCapacityException {
568568
final DeployDestination dest = routerDeploymentDefinition.getDest();
569569
List<HypervisorType> hypervisors = new ArrayList<HypervisorType>();
570+
final HypervisorType defaults = _resourceMgr.getDefaultHypervisor(dest.getDataCenter().getId());
571+
if (defaults != HypervisorType.None) {
572+
hypervisors.add(defaults);
573+
}
570574
if (dest.getCluster() != null) {
571575
if (dest.getCluster().getHypervisorType() == HypervisorType.Ovm) {
572576
hypervisors.add(getClusterToStartDomainRouterForOvm(dest.getCluster().getPodId()));
573577
} else {
574578
hypervisors.add(dest.getCluster().getHypervisorType());
575579
}
576-
} else {
577-
final HypervisorType defaults = _resourceMgr.getDefaultHypervisor(dest.getDataCenter().getId());
578-
if (defaults != HypervisorType.None) {
579-
hypervisors.add(defaults);
580-
} else {
581-
// if there is no default hypervisor, get it from the cluster
582-
hypervisors = _resourceMgr.getSupportedHypervisorTypes(dest.getDataCenter().getId(), true, routerDeploymentDefinition.getPlan().getPodId());
583-
}
580+
} else if (defaults == HypervisorType.None) {
581+
hypervisors = _resourceMgr.getSupportedHypervisorTypes(dest.getDataCenter().getId(), true, routerDeploymentDefinition.getPlan().getPodId());
584582
}
585583

586584
filterSupportedHypervisors(hypervisors);

0 commit comments

Comments
 (0)