Updated Branches: refs/heads/4.2-forward eaea8f8c5 -> db2d9703c
[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/db2d9703 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/db2d9703 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/db2d9703 Branch: refs/heads/4.2-forward Commit: db2d9703c16aac0aa0ebcadaa75022981fbfe6cc Parents: eaea8f8 Author: frank <[email protected]> Authored: Thu Aug 29 10:29:17 2013 -0700 Committer: frank <[email protected]> Committed: Thu Aug 29 10:29:17 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/db2d9703/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 0c4bd14..a1671c9 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; }
