[NEW PATCH] BZ#734136 - Stricter partition detection (via gerrit-bot)

Saggi Mizrahi smizrahi at redhat.com
Wed Aug 31 06:53:29 UTC 2011


New patch submitted by Saggi Mizrahi (smizrahi at redhat.com)

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

commit 2a214e581371dbb892e9fc599b8148ef704b7640
Author: Saggi Mizrahi <smizrahi at redhat.com>
Date:   Tue Aug 30 11:41:34 2011 +0300

    BZ#734136 - Stricter partition detection
    
    Change-Id: Ibc242c7a3810efbc89083d0f0ea6fb193a63e331

diff --git a/vdsm/storage/devicemapper.py b/vdsm/storage/devicemapper.py
index 08cd533..89d1135 100644
--- a/vdsm/storage/devicemapper.py
+++ b/vdsm/storage/devicemapper.py
@@ -99,8 +99,18 @@ def isBlockDevice(devName):
         return False
 
 def isPartitioned(devName):
-    devName = resolveDevName(devName)
-    return (len(getHolders(devName)) > 0)
+    dmName = resolveDevName(devName)
+    #Sometimes partitions are not managed by device mapper
+    possiblePartPath = os.path.join("/sys/block/", devName, devName + "1")
+    if os.path.exists(possiblePartPath):
+            return True
+
+    mpathName = getDevName(dmName)
+    for holder in getHolders(dmName):
+        if getDevName(holder).startswith(mpathName):
+            return True
+
+    return False
 
 def getAllSlaves():
     deps = {}




More information about the vdsm-patches mailing list