rhtyd commented on a change in pull request #4966:
URL: https://github.com/apache/cloudstack/pull/4966#discussion_r708863143



##########
File path: 
engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java
##########
@@ -1243,15 +1216,11 @@ public void orchestrateStart(final String vmUuid, final 
Map<VirtualMachineProfil
                                 throw new ConcurrentOperationException("Failed 
to deploy VM"+ vm.getUuid());
                             }
 
-                            // Update GPU device capacity
                             final GPUDeviceTO gpuDevice = 
startAnswer.getVirtualMachine().getGpuDevice();
                             if (gpuDevice != null) {
                                 _resourceMgr.updateGPUDetails(destHostId, 
gpuDevice.getGroupDetails());
                             }
 
-                            // Remove the information on whether it was a 
deploy vm request.The deployvm=true information

Review comment:
       Some comments may be kept




-- 
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.

To unsubscribe, e-mail: [email protected]

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


Reply via email to