Re: [PATCH] Do not implement the setup and tear down functions for dmraid.

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

 





On 04/06/2009 11:28 AM, Joel Granados Moreno wrote:
These were implemented to be consistent with the other classes in the
storage/devices.py file.  Adding this tears down the dmraid device after
scanning.  This presents trouble further on when parts of the installer
expect device nodes to be there.

Looks good, ack.

Regards,

Hans


---
  storage/devices.py |   24 ------------------------
  1 files changed, 0 insertions(+), 24 deletions(-)

diff --git a/storage/devices.py b/storage/devices.py
index 04b562f..99306d9 100644
--- a/storage/devices.py
+++ b/storage/devices.py
@@ -2429,30 +2429,6 @@ class DMRaidArrayDevice(DiskDevice):
          else:
              self.sysfsPath = ''

-    def teardown(self, recursive=None):
-        """ Close, or tear down, a device. """
-        log_method_call(self, self.name, status=self.status)
-        if not self.exists:
-            raise DeviceError("device has not been created")
-
-        if self.format.exists:
-            self.format.teardown()
-
-        # This call already checks if the set is not active.
-        self._raidSet.deactivate()
-
-    def setup(self, intf=None):
-        """ Open, or set up, a device. """
-        log_method_call(self, self.name, status=self.status)
-        if not self.exists:
-            raise DeviceError("device has not been created")
-
-        # This call already checks if the set is active.
-        self._raidSet.activate(mknod=True)
-
-        udev_settle()
-
-
  class MultipathDevice(DMDevice):
      """ A multipath device """
      _type = "dm-multipath"

_______________________________________________
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