From dced9723ffd3c349c1ac7af7d28ff2ae6db3e922 Mon Sep 17 00:00:00 2001 From: frank Date: Thu, 29 Aug 2013 10:29:17 -0700 Subject: [PATCH] [Upgrade from 3.0.6 to 4.2][vmware] After upgrade the system vms fail to come up because "Secondary storage mount point" is pointing to a wrong location when secondary storage is mounted as read-only, changing permission of files on it will fail. But we should still stick to current mount point instread of returning a wrong mount point /mnt/sec (cherry picked from commit db2d9703c16aac0aa0ebcadaa75022981fbfe6cc) Signed-off-by: animesh --- .../com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java | 1 - 1 file changed, 1 deletion(-) diff --git a/plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java b/plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java index b5e75354554..91d030b29c3 100755 --- a/plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java +++ b/plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java @@ -767,7 +767,6 @@ public class VmwareManagerImpl extends ManagerBase implements VmwareManager, Vmw result = script.execute(); if (result != null) { s_logger.warn("Unable to set permissions for " + mountPoint + " due to " + result); - return null; } return mountPoint; }