Re: [PATCH] Schedule destruction of any existing formatting along with the device.

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

 



Looks good.

On 03/10/2009 03:06 PM, David Lehman wrote:
Schedule destruction of any existing formats the caller neglected to
destroy before calling Storage.destroyDevice to avoid things like
mdadm hanging the install trying to ask if we're sure we want to make
an array on what looks like an ext3 filesystem.
---
  storage/__init__.py |    4 ++++
  1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/storage/__init__.py b/storage/__init__.py
index 63fa622..1e1aa7f 100644
--- a/storage/__init__.py
+++ b/storage/__init__.py
@@ -568,6 +568,10 @@ class Storage(object):

      def destroyDevice(self, device):
          """ Schedule destruction of a device. """
+        if device.format.exists:
+            # schedule destruction of any formatting while we're at it
+            self.devicetree.registerAction(ActionDestroyFormat(device))
+
          action = ActionDestroyDevice(device)
          self.devicetree.registerAction(action)



--
David Cantrell <dcantrell@xxxxxxxxxx>
Red Hat / Honolulu, HI

_______________________________________________
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