vishesh92 commented on code in PR #14108:
URL: https://github.com/apache/cloudstack/pull/14108#discussion_r3988289267
##########
api/src/main/java/org/apache/cloudstack/affinity/AffinityProcessorBase.java:
##########
@@ -41,6 +43,23 @@ public void process(VirtualMachineProfile vm, DeploymentPlan
plan, ExcludeList a
}
+
+ /**
+ * Indexes placements supplied by the caller. Callers such as DRS build a
plan of several moves
+ * in memory and persist it only at the end, so during planning the
database still shows the old
+ * host for every VM the plan has already moved.
+ */
+ protected Map<Long, VirtualMachine> getVmIdVmMap(List<VirtualMachine>
vmList) {
Review Comment:
`HostAffinityProcessor.java` also has the same method. Please remove that.
--
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]
