DK101010 commented on a change in pull request #5148:
URL: https://github.com/apache/cloudstack/pull/5148#discussion_r657649570



##########
File path: 
engine/orchestration/src/main/java/org/apache/cloudstack/engine/cloud/entity/api/VMEntityManagerImpl.java
##########
@@ -253,6 +253,8 @@ public void deployVirtualMachine(String reservationId, 
VMEntityVO vmEntityVO, St
                     }
 
                     _itMgr.start(vm.getUuid(), params, plan, null);
+                }else{
+                    throw ex;

Review comment:
       @GabrielBrascher Maybe I'am wrong but I think is a little bit to much 
info, the vm is  not the problem in that case. Also throws the _itMgr.start 
method a couple exception, so that we can not say here when the vm the problem 
is. 




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to