From a6b482c5000c54a990e0a9413bc4e792240e4619 Mon Sep 17 00:00:00 2001
From: Antti Salminen <antti.salminen@gmail.com>
Date: Sat, 3 May 2014 10:36:22 +0300
Subject: [PATCH] Switch to mapper device name if pvs returns the wrong one.

---
 system/lvg | 20 ++++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/system/lvg b/system/lvg
index 906e13d6469..7f2b58a7c0b 100644
--- a/system/lvg
+++ b/system/lvg
@@ -67,7 +67,7 @@ EXAMPLES = '''
 # Create a volume group on top of /dev/sda1 with physical extent size = 32MB.
 - lvg:  vg=vg.services pvs=/dev/sda1 pesize=32
 
-# Create or resize a volume group on top of /dev/sdb1 and /dev/sdc5. 
+# Create or resize a volume group on top of /dev/sdb1 and /dev/sdc5.
 # If, for example, we already have VG vg.services on top of /dev/sdb1,
 # this VG will be extended by /dev/sdc5.  Or if vg.services was created on
 # top of /dev/sda5, we first extend it with /dev/sdb1 and /dev/sdc5,
@@ -89,10 +89,22 @@ def parse_vgs(data):
         })
     return vgs
 
-def parse_pvs(data):
+def find_mapper_device_name(module, dm_device):
+        dmsetup_cmd = module.get_bin_path('dmsetup', True)
+        mapper_prefix = '/dev/mapper/'
+        rc, dm_name, err = module.run_command("%s info -C --noheadings -o name %s" % (dmsetup_cmd, dm_device))
+        if rc != 0:
+            module.fail_json(msg="Failed executing dmsetup command.", rc=rc, err=err)
+        mapper_device = mapper_prefix + dm_name.rstrip()
+        return mapper_device
+
+def parse_pvs(module, data):
     pvs = []
+    dm_prefix = '/dev/dm-'
     for line in data.splitlines():
         parts = line.strip().split(';')
+        if parts[0].startswith(dm_prefix):
+            parts[0] = find_mapper_device_name(module, parts[0])
         pvs.append({
             'name': parts[0],
             'vg_name': parts[1],
@@ -125,7 +137,7 @@ def main():
         module.fail_json(msg="No physical volumes given.")
 
 
-    
+
     if state=='present':
         ### check given devices
         for test_dev in dev_list:
@@ -139,7 +151,7 @@ def main():
             module.fail_json(msg="Failed executing pvs command.",rc=rc, err=err)
 
         ### check pv for devices
-        pvs = parse_pvs(current_pvs)
+        pvs = parse_pvs(module, current_pvs)
         used_pvs = [ pv for pv in pvs if pv['name'] in dev_list and pv['vg_name'] and pv['vg_name'] != vg ]
         if used_pvs:
             module.fail_json(msg="Device %s is already in %s volume group."%(used_pvs[0]['name'],used_pvs[0]['vg_name']))