[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


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/2a0e3718
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/2a0e3718
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/2a0e3718

Branch: refs/heads/ldapplugin
Commit: 2a0e371881dcb892bb1b206dccc7ce3e1f68a1ba
Parents: a19eb5b
Author: frank <[email protected]>
Authored: Thu Aug 29 10:29:17 2013 -0700
Committer: frank <[email protected]>
Committed: Thu Aug 29 10:32:13 2013 -0700

----------------------------------------------------------------------
 .../src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java  | 1 -
 1 file changed, 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/2a0e3718/plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java
----------------------------------------------------------------------
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 11c836b..6ecca28 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
@@ -758,7 +758,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;
     }

Reply via email to