[GitHub] [cloudstack] shwstppr commented on a change in pull request #5242: Add internal cs name to vm during the ingest

[GitBox <gi...@apache.org>]

shwstppr commented on a change in pull request #5242:
URL: https://github.com/apache/cloudstack/pull/5242#discussion_r686593157



##########
File path: vmware-base/src/main/java/com/cloud/hypervisor/vmware/mo/HostMO.java
##########
@@ -579,8 +579,9 @@ private void loadVmCache() throws Exception {
 
                     if (s_logger.isTraceEnabled())
                         s_logger.trace("put " + vmName + " into host cache");
-
-                    _vmCache.put(vmName, new VirtualMachineMO(_context, oc.getObj()));
+                    VirtualMachineMO virtualMachine = new VirtualMachineMO(_context, oc.getObj());
+                    virtualMaschine.setInternalCSName(vmName);
+                    _vmCache.put(vmName, virtualMaschine);

Review comment:
       @DK101010 wrt previous commit variable name not changed here. This is causing build failures




-- 
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: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org