vishesh92 commented on code in PR #8085:
URL: https://github.com/apache/cloudstack/pull/8085#discussion_r1357313019
##########
engine/orchestration/src/main/java/com/cloud/vm/VirtualMachineManagerImpl.java:
##########
@@ -1443,6 +1443,12 @@ public void orchestrateStart(final String vmUuid, final
Map<VirtualMachineProfil
}
if (canRetry) {
try {
+ // Setting pod id to null will result in migration of
Volumes across pods
+ // We set it to null only if migration of volumes
across cluster is enabled
+ // Or volumes are still in allocated state for that VM
(in case of failure during deployment)
+ if
(MIGRATE_VM_ACROSS_CLUSTERS.valueIn(vm.getDataCenterId()) ||
checkForNonAllocatedVolumes(vm.getId())) {
+ vm.setPodIdToDeployIn(null);
+ }
Review Comment:
Let me update the comment in detail about why we are doing this here.
--
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]
