You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by fr...@apache.org on 2014/10/13 21:03:03 UTC

git commit: updated refs/heads/master to 49c01e2

Repository: cloudstack
Updated Branches:
  refs/heads/master e02c38246 -> 49c01e2ae


Don't do chmod -R when mounting secondary storage for vmware
It causes extreme long vm start time when managment server has slow connection to secondary storage


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

Branch: refs/heads/master
Commit: 49c01e2ae4eb164438f78f9686813497d7fd162f
Parents: e02c382
Author: Frank Zhang <fr...@citrix.com>
Authored: Mon Oct 13 12:05:22 2014 -0700
Committer: Frank Zhang <fr...@citrix.com>
Committed: Mon Oct 13 12:06:37 2014 -0700

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


http://git-wip-us.apache.org/repos/asf/cloudstack/blob/49c01e2a/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 3121bfb..4f24882 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
@@ -778,7 +778,7 @@ public class VmwareManagerImpl extends ManagerBase implements VmwareManager, Vmw
 
         // Change permissions for the mountpoint
         script = new Script(true, "chmod", _timeout, s_logger);
-        script.add("-R", "777", mountPoint);
+        script.add("777", mountPoint);
         result = script.execute();
         if (result != null) {
             s_logger.warn("Unable to set permissions for " + mountPoint + " due to " + result);