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

CLOUDSTACk-9002: VM deployment is successful even when dhcp entry command fails - Fixed

This commit is contained in:
Kshitij Kansal 2015-10-28 11:49:10 +05:30
parent 535ab51b9a
commit 301ea330ce
2 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
engine/orchestration/src/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
View File

@ -744,7 +744,7 @@ public class NetworkOrchestrator extends ManagerBase implements NetworkOrchestra
Pair<NicProfile, Integer> vmNicPair = allocateNic(requested, config, isDefaultNic, deviceId, vm);
NicProfile vmNic = null;
if(vmNicPair != null) {
if (vmNicPair != null) {
vmNic = vmNicPair.first();
if (vmNic == null) {
continue;
@ -1244,12 +1244,16 @@ public class NetworkOrchestrator extends ManagerBase implements NetworkOrchestra
return false;
}
}
sp.addDhcpEntry(network, profile, vmProfile, dest, context);
if(!sp.addDhcpEntry(network, profile, vmProfile, dest, context)) {
return false;
}
}
if (_networkModel.areServicesSupportedInNetwork(network.getId(), Service.UserData)
&& _networkModel.isProviderSupportServiceInNetwork(network.getId(), Service.UserData, element.getProvider()) && element instanceof UserDataServiceProvider) {
UserDataServiceProvider sp = (UserDataServiceProvider)element;
sp.addPasswordAndUserdata(network, profile, vmProfile, dest, context);
if(!sp.addPasswordAndUserdata(network, profile, vmProfile, dest, context)){
return false;
}
}
}
return true;

3
server/src/org/apache/cloudstack/network/topology/BasicNetworkTopology.java
View File

@ -384,8 +384,7 @@ public class BasicNetworkTopology implements NetworkTopology {
}
try {
ruleApplier.accept(getVisitor(), router);
result = ruleApplier.accept(getVisitor(), router);
connectedRouters.add(router);
} catch (final AgentUnavailableException e) {
s_logger.warn(msg + router.getInstanceName(), e);