[NEW PATCH] BZ#734712 - Set vdsm:kvm ownership to master dir. (via gerrit-bot)

Dan Kenigsberg danken at redhat.com
Sun Sep 11 10:17:58 UTC 2011


New patch submitted by Dan Kenigsberg (danken at redhat.com)

You can review this change at: http://gerrit.usersys.redhat.com/917

commit 60553fcbde0f3957bb8e89b80fe0886d8bb19822
Author: Eduardo Warszawski <ewarszaw at redhat.com>
Date:   Thu Sep 8 15:15:54 2011 +0300

    BZ#734712 - Set vdsm:kvm ownership to master dir.
    
    Change-Id: I9dac976b3ce0698e08fb4633480ae8edfd9c9b4e

diff --git a/vdsm/storage/blockSD.py b/vdsm/storage/blockSD.py
index 54bc161..3a1975f 100644
--- a/vdsm/storage/blockSD.py
+++ b/vdsm/storage/blockSD.py
@@ -818,7 +818,7 @@ class BlockStorageDomain(sd.StorageDomain):
         if rc != 0:
             raise se.BlockStorageDomainMasterMountError(masterfsdev, rc, out)
 
-        cmd = [constants.EXT_CHOWN, lvm.USER_GROUP, masterDir]
+        cmd = [constants.EXT_CHOWN, "%s:%s" % (constants.METADATA_USER, constants.METADATA_GROUP), masterDir]
         (rc, out, err) = misc.execCmd(cmd)
         if rc != 0:
             self.log.error("failed to chown %s", masterDir)




More information about the vdsm-patches mailing list