[PATCH 1/4] Add multipath member with addUdevDiskDevice instead of DiskDevice (#582254)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Multipath members also need to be distinguished between the different
subclasses of DiskDevice for dracutSetupString to work.
addUdevDiskDevice already does this distinction for us and knows
iSCSI, FCoE, MD container, DASD, zFCP, or DiskDevice for everything
else.

(Patch originally from Steffen Maier <maier@xxxxxxxxxxxxxxxxxx>)
---
 storage/devicetree.py |   11 ++---------
 1 files changed, 2 insertions(+), 9 deletions(-)

diff --git a/storage/devicetree.py b/storage/devicetree.py
index 6a167e9..895bbae 100644
--- a/storage/devicetree.py
+++ b/storage/devicetree.py
@@ -1244,15 +1244,8 @@ class DeviceTree(object):
         #
         # The first step is to either look up or create the device
         #
-        if udev_device_is_multipath_member(info):
-            device = DiskDevice(name,
-                            major=udev_device_get_major(info),
-                            minor=udev_device_get_minor(info),
-                            sysfsPath=sysfs_path, exists=True,
-                            serial=udev_device_get_serial(info),
-                            vendor=udev_device_get_vendor(info),
-                            model=udev_device_get_model(info))
-            self._addDevice(device)
+        if udev_device_is_multipath_member(info) and device is None:
+            device = self.addUdevDiskDevice(info)
         elif udev_device_is_dm(info) and \
                devicelibs.dm.dm_is_multipath(info):
             log.debug("%s is a multipath device" % name)
-- 
1.7.1

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list


[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux