[PATCH 5/5] Autopart PVs require enough space for a default-sized partition.

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

 



When identifying disks that will contain autopart PVs we are only
checking that each disk contain at least 100MB of contiguous free
space, but then we are allocating partitions with a base size of
500MB for the PVs. Hook both pieces to the same source to prevent
confusion.

Note that we can still get caught in many cases, eg: 900MB of free
space on the device /boot will be allocated from. We detect 900MB of
free space, so we mark it for a new PV. We allocate /boot, and then
don't have enough space left to allocate the PV.
---
 pyanaconda/storage/partitioning.py |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/pyanaconda/storage/partitioning.py b/pyanaconda/storage/partitioning.py
index 61f03ae..0530163 100644
--- a/pyanaconda/storage/partitioning.py
+++ b/pyanaconda/storage/partitioning.py
@@ -42,7 +42,7 @@ log = logging.getLogger("storage")
 
 def _createFreeSpacePartitions(anaconda):
     # get a list of disks that have at least one free space region of at
-    # least 100MB
+    # least the default size for new partitions
     disks = []
     for disk in anaconda.storage.partitioned:
         if anaconda.storage.clearPartDisks and \
@@ -55,7 +55,7 @@ def _createFreeSpacePartitions(anaconda):
                 part = part.nextPartition()
                 continue
 
-            if part.getSize(unit="MB") > 100:
+            if part.getSize(unit="MB") > PartitionDevice.defaultSize:
                 disks.append(disk)
                 break
 
-- 
1.6.6.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