mirror of https://github.com/apache/cloudstack.git
CLOUDSTACK-6750: [OVS] With stretched network deploying vm in a ovs
disabled zone does not fail
Fix ensures that connectivity service provider is enabled in the zone on
to which nework is being streched.
(cherry picked from commit d8cbba1bc6)
This commit is contained in:
parent
99dd86e588
commit
6bd9d1241b
|
|
@ -2716,8 +2716,29 @@ public class UserVmManagerImpl extends ManagerBase implements UserVmManager, Vir
|
|||
boolean securityGroupEnabled = false;
|
||||
boolean vpcNetwork = false;
|
||||
for (NetworkVO network : networkList) {
|
||||
if ((network.getDataCenterId() != zone.getId()) && !network.isStrechedL2Network()) {
|
||||
throw new InvalidParameterValueException("Network id=" + network.getId() + " doesn't belong to zone " + zone.getId());
|
||||
if ((network.getDataCenterId() != zone.getId())) {
|
||||
if (!network.isStrechedL2Network()) {
|
||||
throw new InvalidParameterValueException("Network id=" + network.getId() +
|
||||
" doesn't belong to zone " + zone.getId());
|
||||
}
|
||||
|
||||
NetworkOffering ntwkOffering = _networkOfferingDao.findById(network.getNetworkOfferingId());
|
||||
Long physicalNetworkId = _networkModel.findPhysicalNetworkId(zone.getId(), ntwkOffering.getTags(),
|
||||
ntwkOffering.getTrafficType());
|
||||
if (physicalNetworkId == null) {
|
||||
s_logger.warn("Network id " + network.getId() + " could not be streched to the zone " + zone.getId()
|
||||
+ " as valid phyical network could not be found");
|
||||
throw new InvalidParameterValueException("Network in which is VM getting deployed could not be" +
|
||||
" streched to the zone.");
|
||||
}
|
||||
|
||||
String provider = _ntwkSrvcDao.getProviderForServiceInNetwork(network.getId(), Service.Connectivity);
|
||||
if (!_networkModel.isProviderEnabledInPhysicalNetwork(physicalNetworkId, provider)) {
|
||||
s_logger.warn("Network id " + network.getId() + " could not be streched to the zone " +zone.getId()
|
||||
+ " as Connectivity service provider is not enabled in the zone " + zone.getId());
|
||||
throw new InvalidParameterValueException("Network in which is VM getting deployed could not be" +
|
||||
" streched to the zone.");
|
||||
}
|
||||
}
|
||||
|
||||
//relax the check if the caller is admin account
|
||||
|
|
|
|||
Loading…
Reference in New Issue