diff options
| author | pcarrier <pcarrier@ef72aa8b-4018-0410-8976-d6e080ef94d8> | 2010-11-15 21:33:32 +0000 |
|---|---|---|
| committer | pcarrier <pcarrier@ef72aa8b-4018-0410-8976-d6e080ef94d8> | 2010-11-15 21:33:32 +0000 |
| commit | 50c237c503ff6de52cba0b0b2306ff12386b98e4 (patch) | |
| tree | b9e1c6a28237e43eac10419e7f11e80a1685d338 | |
| parent | ee12f8efc515c288f78dbc737267d562b26dbe94 (diff) | |
| download | sos-50c237c503ff6de52cba0b0b2306ff12386b98e4.tar.gz | |
Mistaken again.
dm-* should actually be skipped (unless someone plays with kpartx too much.)
Unlike r1000, actually closes RHBZ#653498.
git-svn-id: svn+ssh://svn.fedorahosted.org/svn/sos/trunk@1002 ef72aa8b-4018-0410-8976-d6e080ef94d8
| -rw-r--r-- | sos/plugins/filesys.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sos/plugins/filesys.py b/sos/plugins/filesys.py index 93568361..3a9a9f13 100644 --- a/sos/plugins/filesys.py +++ b/sos/plugins/filesys.py @@ -56,13 +56,13 @@ class filesys(sos.plugintools.PluginBase): start_geo = hdparm.strip().split("\n")[-1].strip().split()[-1] if(start_geo == "0"): devlist.append(dev) - # RHEL* does not ship hdparm for S390(x) - # Cheaper heuristic handling at least dm-.* correctly + # Cheaper heuristic as RHEL* does not ship hdparm for S390(x) + # Skips least dm-.* correctly else: - part_in_disk = re.compile("/dev/[a-z]+[0-9]+$") + part_in_disk = re.compile("^/dev/[a-z]+$") for dev in partlist: print part_in_disk.match(dev) - if not bool(part_in_disk.match(dev)): + if bool(part_in_disk.match(dev)): devlist.append(dev) for i in devlist: |