cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From du...@apache.org
Subject [34/50] [abbrv] git commit: updated refs/heads/ldapplugin to bdba0dd
Date Fri, 30 Aug 2013 23:28:27 GMT
[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 <frank.zhang@citrix.com>
Authored: Thu Aug 29 10:29:17 2013 -0700
Committer: frank <frank.zhang@citrix.com>
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;
     }


Mime
View raw message