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@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list