[PATCH f17-branch 2/2] Activate devices as needed when parsing /etc/fstab. (#813120)

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

 



Any non-root device that appears in /etc/fstab and requires activation
(eg: lvm, md) must be active prior to instantiating an FS instance
for an existing filesystem on that device due to current/minimum size
calculations triggered by the FS constructor.
---
 pyanaconda/storage/__init__.py |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/pyanaconda/storage/__init__.py b/pyanaconda/storage/__init__.py
index 8483eca..5b278ff 100644
--- a/pyanaconda/storage/__init__.py
+++ b/pyanaconda/storage/__init__.py
@@ -2004,10 +2004,12 @@ class FSSet(object):
                                                                 fstype))
             raise UnrecognizedFSTabEntryError()
 
+        device.setup()
         fmt = getFormat(fstype, device=device.path, exists=True)
         if fstype != "auto" and None in (device.format.type, fmt.type):
             log.info("Unrecognized filesystem type for %s (%s)"
                      % (device.name, fstype))
+            device.teardown()
             raise UnrecognizedFSTabEntryError()
 
         # make sure, if we're using a device from the tree, that
@@ -2020,6 +2022,7 @@ class FSSet(object):
                 # XXX we should probably disallow migration for this fs
                 device.format = fmt
             else:
+                device.teardown()
                 raise FSTabTypeMismatchError("%s: detected as %s, fstab says %s"
                                              % (mountpoint, dtype, ftype))
         del ftype
-- 
1.7.6

_______________________________________________
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