From 8bcf07e6f0ccaa4590d233d8e747b0d9a79111a3 Mon Sep 17 00:00:00 2001
From: Chiradeep Vittal <chiradeep@cloud.com>
Date: Fri, 28 Jan 2011 17:26:35 -0800
Subject: [PATCH] bug 6844: don't chmod, respect permissions set by NFS server
 admin

---
 .../storage/resource/NfsSecondaryStorageResource.java  | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/core/src/com/cloud/storage/resource/NfsSecondaryStorageResource.java b/core/src/com/cloud/storage/resource/NfsSecondaryStorageResource.java
index ece7b57f8c5..b823a323059 100755
--- a/core/src/com/cloud/storage/resource/NfsSecondaryStorageResource.java
+++ b/core/src/com/cloud/storage/resource/NfsSecondaryStorageResource.java
@@ -588,15 +588,7 @@ public class NfsSecondaryStorageResource extends ServerResourceBase implements S
             return null;
         }
         
-        // Change permissions for the mountpoint
-        script = new Script(!_inSystemVM, "chmod", _timeout, s_logger);
-        script.add("777", mountPoint);
-        script.add("-R");
-        result = script.execute();
-        if (result != null) {
-            s_logger.warn("Unable to set permissions for " + mountPoint + " due to " + result);
-            return null;
-        }
+        
         
         // XXX: Adding the check for creation of snapshots dir here. Might have to move it somewhere more logical later.
         if (!checkForSnapshotsDir(mountPoint)) {